On Fri, 2012-01-27 at 01:37 -0600, Mike Christie wrote: > On 01/27/2012 01:04 AM, Bart Van Assche wrote: > > On Fri, Jan 27, 2012 at 4:13 AM, <michaelc@xxxxxxxxxxx> wrote: > >> NETLINK_CREDS's pid now returns 0, so I guess we are supposed to > >> be using NETLINK_CB. This changed while the patch to export the > >> pid was getting merged upstream, so it was not noticed until both > >> the network and iscsi changes were in the same tree. > >> > >> Signed-off-by: Mike Christie <michaelc@xxxxxxxxxxx> > > > > If my analysis is correct this patch has not only to be applied on the > > master branch but also on kernel 3.2. Shouldn't "Cc: > > stable@xxxxxxxxxxxxxxx #3.2" be added here ? See also > > http://www.spinics.net/lists/netdev/msg185565.html. > > > > The iscsi code that uses the pid just got merged in the 3.3 feature > window, so no need. > > Also for stable, I think we normally just tell James it should go in > stable and then he does some magic. In the past, when he knows it should > go there, when the patch gets merged and sent upstream the stable people > were notified automagically. That's the cc: stable@xxxxxxxxxx tag in the commit ... but don't necessarily rely on me to add this. I do add it in a few cases, but I do need people to indicate should go to stable by adding the cc: in their commit, otherwise I have to guess. James -- 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
