2017-10-31 18:13:52

by Bart Van Assche

[permalink] [raw]
Subject: Re: [PATCH] scsi: qla2xxx: Convert timers to use timer_setup()

On Tue, 2017-10-31 at 18:03 +0000, Madhani, Himanshu wrote:
> On Oct 31, 2017, at 8:49 AM, Martin K. Petersen <[email protected]> wrote:
> > > In preparation for unconditionally passing the struct timer_list
> > > pointer to all timer callbacks, switch to using the new timer_setup()
> > > and from_timer() to pass the timer pointer explicitly.
> >
> > Cavium folks: Please verify!
>
> I’ve checked on my setup and i am seeing regression with this patch applied.
>
> Driver load locks up the system with this patch applied.
>
> NACK

That feedback is not very helpful ...

Anyway, what kernel source tree did you use in your testing? I may be able to
free up some time to look into this myself.

Bart.?&ן6????n?׭=????S?㜷}u??=?~>?M4?M{_???1? ???Ͷ?5??u??=?U?f????m?l??ۣ?jנ??????