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 <j...@perches.com> > To: Rasmus Villemoes <li...@rasmusvillemoes.dk>, Evgeny > Cherkashin/Russia/IBM@IBMRU, Frank Blaschka > <frank.blasc...@de.ibm.com>, Ursula Braun <ursula.br...@de.ibm.com> > Cc: Kashyap Desai <kashyap.de...@avagotech.com>, Sumit Saxena > <sumit.sax...@avagotech.com>, Uday Lingala > <uday.ling...@avagotech.com>, "James E.J. Bottomley" > <jbottom...@parallels.com>, "PDL,MEGARAIDLINUX" > <megaraidlinux....@avagotech.com>, linux-s...@vger.kernel.org, > linux-kernel@vger.kernel.org > 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 <kashyap.de...@avagotech.com> > wrote: > > > > > Acked-by: Kashyap Desai <kashyap.de...@avagotech.com> > > > > > >> -----Original Message----- > > >> From: Rasmus Villemoes [mailto:li...@rasmusvillemoes.dk] > > >> Sent: Friday, February 06, 2015 8:04 PM > > >> To: Kashyap Desai; Sumit Saxena; Uday Lingala; James E.J. > Bottomley > > >> Cc: Rasmus Villemoes; megaraidlinux....@avagotech.com; linux- > > >> s...@vger.kernel.org; linux-kernel@vger.kernel.org > > >> 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 <li...@rasmusvillemoes.dk> > > > > 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
