Joe, thanks, we will add your qeth patch to our local library and submit it with one of our next shipments.
Regards, Ursula Braun > > From: Joe Perches <[email protected]> > To: Rasmus Villemoes <[email protected]>, Evgeny > Cherkashin/Russia/IBM@IBMRU, Frank Blaschka > <[email protected]>, Ursula Braun <[email protected]> > Cc: Kashyap Desai <[email protected]>, Sumit Saxena > <[email protected]>, Uday Lingala > <[email protected]>, "James E.J. Bottomley" > <[email protected]>, "PDL,MEGARAIDLINUX" > <[email protected]>, [email protected], > [email protected] > Date: 05/12/2015 05:52 PM > Subject: Re: [PATCH] scsi: megaraid_sas: Prevent future %p > disaster > > ______________________________________________________________________ > > > > On Tue, 2015-05-12 at 16:04 +0200, Rasmus Villemoes wrote: > > On Tue, Feb 10 2015, Kashyap Desai <[email protected]> > wrote: > > > > > Acked-by: Kashyap Desai <[email protected]> > > > > > >> -----Original Message----- > > >> From: Rasmus Villemoes [mailto:[email protected]] > > >> Sent: Friday, February 06, 2015 8:04 PM > > >> To: Kashyap Desai; Sumit Saxena; Uday Lingala; James E.J. > Bottomley > > >> Cc: Rasmus Villemoes; [email protected]; linux- > > >> [email protected]; [email protected] > > >> Subject: [PATCH] scsi: megaraid_sas: Prevent future %p disaster > > >> > > >> There is currently no %po format extension, so currently the > letters > > > "on" are > > >> simply skipped and the pointer is printed as expected (while > missing the > > > word > > >> on). However, it is probably only a matter of time before someone > comes > > > up > > >> with a %po extension, at which point this is likely to fail > > > spectacularly. > > >> > > >> Signed-off-by: Rasmus Villemoes <[email protected]> > > > > Any chance either this minimal fix or Joe's more thorough cleanup > could > > get applied? There's still no %po extension, but there's certainly a > lot > > of proposals for more %p extensions flying around LKML, so I still > think > > the "only a matter of time" holds. > > There's also this one with %pM6: > --- > drivers/s390/net/qeth_l2_main.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/s390/net/qeth_l2_main.c > b/drivers/s390/net/qeth_l2_main.c > index 0ea0869..dc783047 100644 > --- a/drivers/s390/net/qeth_l2_main.c > +++ b/drivers/s390/net/qeth_l2_main.c > @@ -1420,8 +1420,8 @@ static void qeth_bridge_emit_host_event(struct > qeth_card *card, > env[i] = str[i]; i > ++; > } > if (code & > IPA_ADDR_CHANGE_CODE_MACADDR) { > - snprintf(str[i], > sizeof(str[i]), "MAC=%pM6", > - > &addr_lnid->mac); > + snprintf(str[i], > sizeof(str[i]), "MAC=%pM", > + > addr_lnid->mac); > env[i] = str[i]; i > ++; > } > snprintf(str[i], sizeof(str[i]), > "NTOK_BUSID=%x.%x.%04x", > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
