Hi Chris, It's not the kernel, but some other code I'm writing. However, I expect it could be used for kmalloc() and friends. Nobody is using malloc_ident yet, but I noticed other xxx_ident that are not used nad are present in ident-list.h so I assumed it would be ok. I'll try to add a test case. Cheers, Emil. This e-mail, and any associated attachments have been classified as: -------------------------------------------------------------------- [ ] Public [x] Freescale Semiconductor Internal Use Only [ ] Freescale Semiconductor Confidential Proprietary > -----Original Message----- > From: christ.li@xxxxxxxxx [mailto:christ.li@xxxxxxxxx] On > Behalf Of Christopher Li > Sent: Thursday, October 25, 2007 10:16 PM > To: Medve Emilian-EMMEDVE1 > Cc: josh@xxxxxxxxxxxxxxx; linux-sparse@xxxxxxxxxxxxxxx > Subject: Re: [PATCH] Handle ignored attribute malloc > > Which kernel source code triggers this? > > If nobody is using malloc_ident directly. You don't need to add it > to ident-list.h. > > Having a test case in validations directory will be nice too. > > Chris > > On 10/25/07, Emil Medve <Emilian.Medve@xxxxxxxxxxxxx> wrote: > > This avoids error messages like this: > > > > error: attribute 'malloc': unknown attribute > > > > Signed-off-by: Emil Medve <Emilian.Medve@xxxxxxxxxxxxx> > > --- > > > > Please copy me in replies as I'm not subscribed to the list > > > > ident-list.h | 1 + > > parse.c | 1 + > > 2 files changed, 2 insertions(+), 0 deletions(-) > > > > diff --git a/ident-list.h b/ident-list.h > > index b183eac..8fcd7de 100644 > > --- a/ident-list.h > > +++ b/ident-list.h > > @@ -69,6 +69,7 @@ IDENT(model); IDENT(__model__); > > IDENT(format_arg); IDENT(__format_arg__); > > IDENT(nothrow); IDENT(__nothrow); IDENT(__nothrow__); > > IDENT(__transparent_union__); > > +IDENT(malloc); > > IDENT(__malloc__); > > IDENT(nonnull); IDENT(__nonnull); IDENT(__nonnull__); > > IDENT(constructor); IDENT(__constructor__); > > diff --git a/parse.c b/parse.c > > index 47b5204..a41939d 100644 > > --- a/parse.c > > +++ b/parse.c > > @@ -283,6 +283,7 @@ static struct init_keyword { > > { "nothrow", NS_KEYWORD, .op = &ignore_attr_op }, > > { "__nothrow", NS_KEYWORD, .op = &ignore_attr_op }, > > { "__nothrow__", NS_KEYWORD, .op = > &ignore_attr_op }, > > + { "malloc", NS_KEYWORD, .op = &ignore_attr_op }, > > { "__malloc__", NS_KEYWORD, .op = &ignore_attr_op }, > > { "nonnull", NS_KEYWORD, .op = &ignore_attr_op }, > > { "__nonnull", NS_KEYWORD, .op = &ignore_attr_op }, > > -- > > 1.5.3.GIT > > > > - > > To unsubscribe from this list: send the line "unsubscribe > linux-sparse" in > > the body of a message to majordomo@xxxxxxxxxxxxxxx > > More majordomo info at http://vger.kernel.org/majordomo-info.html - To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
