On Mon, Sep 10, 2012 at 04:58:02PM +0400, Glauber Costa wrote: > On 09/10/2012 04:57 PM, Fengguang Wu wrote: > > Glauber, > > > > The patch entitled > > > > sl[au]b: Allocate objects from memcg cache > > > > changes > > > > include/linux/slub_def.h | 15 ++++++++++----- > > > > which triggers this warning: > > > > drivers/video/riva/fbdev.c:281:9: sparse: preprocessor token MAX_LEVEL redefined > > > > It's the MAX_LEVEL that is defined in include/linux/idr.h. > > > > MAX_LEVEL is obviously too generic. Better adding some prefix to it? > > > > I don't see any MAX_LEVEL definition in this patch. You say it is > defined in include/linux/idr.h, and as the diffstat shows, I am not > touching this file. It's a rather *unexpected* side effect. You changed slub_def.h to include memcontrol.h/cgroup.h which in turn includes idr.h. > I think this needs patching independently. Yes, sure. And perhaps send it for quick inclusion before your patches? Thanks, Fengguang -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html