On Wed, 9 Sep 2015, Huang Rui wrote: > On Wed, Sep 09, 2015 at 04:40:07PM +0800, Ken Xue wrote: > > On Wed, 2015-09-09 at 16:30 +0800, Huang Rui wrote: > > > On Wed, Sep 09, 2015 at 03:02:11PM +0800, Ken Xue wrote: > > > > From 844ebfcecad7ddaf7206e0474c600b0146b4ef21 Mon Sep 17 00:00:00 2001 > > > > From: Ken Xue <Ken.Xue@xxxxxxx> > > > > Date: Wed, 9 Sep 2015 14:55:21 +0800 > > > > Subject: [PATCH] SCSI:Refine the way to fix NULL pointer dereference in > > > > runtime PM > > > > > > > > There was a patch about Bugzilla #101371. > > > > > > Which Bugzilla? I guess you should put a link or oops message of NULL > > > pointer dereference here. Then other guys can know detail infomation > > > of this fix. > > > > > You can find out previous patch with commit ID 49718f. > > You also can find log with below link > > https://bugzilla.kernel.org/show_bug.cgi?id=101371 > > > > I see. So maybe we would better mark the commit ID. :) > > Alan, could you please take a look? Yes. Ken, your fix was good but the description was bad. The description did not explain what the problem was or how you were going to fix it. In fact, this change should be split up into two patches. The first patch should simply revert commit 49718f0fb8c9 ("SCSI: Fix NULL pointer dereference in runtime PM") and the description should explain why that commit didn't work properly for the sr driver. The second patch should make the new changes to the block core. The description should explain why the changes are needed, and it can reference Bugzilla #101371. Both patches should be tagged for the -stable kernels. When you have done all that, you can add: Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> to both patches. Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
