Le lundi 12 mai 2014 à 12:36 +0200, Bart Van Assche a écrit : > On 05/12/14 12:04, Yann Droneaud wrote: > > Le lundi 12 mai 2014 à 10:30 +0200, Bart Van Assche a écrit : > >> container_of() never returns NULL. Hence remove the code that tests > >> whether container_of() returns NULL. > >> > >> Signed-off-by: Bart Van Assche <[email protected]> > >> Cc: Alex Chiang <[email protected]> > >> Cc: <[email protected]> > > The patch seems fine. > > > > But I don't think this one qualify for linux-stable. > > One chunk in patch 3/3 touches code that is also touched by this patch. > This means that if only patches 2/3 and 3/3 would be sent to stable that > these wouldn't apply properly. Hence the "Cc: stable" tag in this patch. >
You could re-order the patches to make this one apply on top of the two others. Anyway, it's a minor issue that doesn't justifiy by itself to respin the patchset. But if you're going to take account of my others remarks, it might be worth to reorder the patchset. Regards. -- Yann Droneaud OPTEYA -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
