Project

General

Profile

Bug #14476

rbd: TaskFinisher::cancel should remove event from SafeTimer

Added by Douglas Fuller over 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Target version:
-
Start date:
01/22/2016
Due date:
% Done:

0%

Source:
Development
Tags:
Backport:
hammer, infernalis
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:

Description

TaskFinisher needs to call SafeTimer::cancel_event, otherwise canceled events fire anyway.


Related issues

Copied to rbd - Backport #14552: infernalis: rbd: TaskFinisher::cancel should remove event from SafeTimer Resolved
Copied to rbd - Backport #14553: hammer: rbd: TaskFinisher::cancel should remove event from SafeTimer Resolved

Associated revisions

Revision 2aa0f318 (diff)
Added by Douglas Fuller over 3 years ago

rbd: remove canceled tasks from timer thread

When canceling scheduled tasks using the timer thread, TaskFinisher::cancel
does not call SafeTimer::cancel_event, so events fire anyway. Add this call.

Fixes: #14476
Signed-off-by: Douglas Fuller <>

Revision 46d626d9 (diff)
Added by Douglas Fuller over 3 years ago

rbd: remove canceled tasks from timer thread

When canceling scheduled tasks using the timer thread, TaskFinisher::cancel
does not call SafeTimer::cancel_event, so events fire anyway. Add this call.

Fixes: #14476
Signed-off-by: Douglas Fuller <>
(cherry picked from commit 2aa0f318c862dbe3027d74d345671506605778eb)

Revision f870ba9a (diff)
Added by Douglas Fuller over 3 years ago

rbd: remove canceled tasks from timer thread

When canceling scheduled tasks using the timer thread, TaskFinisher::cancel
does not call SafeTimer::cancel_event, so events fire anyway. Add this call.

Fixes: #14476
Signed-off-by: Douglas Fuller <>
(cherry picked from commit 2aa0f318c862dbe3027d74d345671506605778eb)

History

#2 Updated by Douglas Fuller over 3 years ago

  • Status changed from New to Need Review

#3 Updated by Jason Dillaman over 3 years ago

  • Backport set to hammer, infernalis

#4 Updated by Douglas Fuller over 3 years ago

  • Status changed from Need Review to Pending Backport

#5 Updated by Loic Dachary over 3 years ago

  • Copied to Backport #14552: infernalis: rbd: TaskFinisher::cancel should remove event from SafeTimer added

#6 Updated by Loic Dachary over 3 years ago

  • Copied to Backport #14553: hammer: rbd: TaskFinisher::cancel should remove event from SafeTimer added

#7 Updated by Loic Dachary over 3 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF