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
