On 14/10/2016 12:42, Fam Zheng wrote: >> > diff --git a/block/qed.c b/block/qed.c >> > index 1a7ef0a..dcb5fb9 100644 >> > --- a/block/qed.c >> > +++ b/block/qed.c >> > @@ -354,7 +354,9 @@ static void qed_start_need_check_timer(BDRVQEDState *s) >> > static void qed_cancel_need_check_timer(BDRVQEDState *s) >> > { >> > trace_qed_cancel_need_check_timer(s); >> > - timer_del(s->need_check_timer); >> > + if (s->need_check_timer) { >> > + timer_del(s->need_check_timer); >> > + } >> > } > > This belongs to a separate patch, or deserves an explanation in the commit > message.
It probably belongs in no patch, actually. Paolo