On Fri, 30 Apr 2010, Bill Pemberton wrote: > Fixes sparse warning: > > fs/jbd2/journal.c:1892:9: warning: Using plain integer as NULL pointer > > Signed-off-by: Bill Pemberton <wfp5p@xxxxxxxxxxxx> > CC: linux-ext4@xxxxxxxxxxxxxxx > --- > fs/jbd2/journal.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/fs/jbd2/journal.c b/fs/jbd2/journal.c > index c03d4dc..bc2ff59 100644 > --- a/fs/jbd2/journal.c > +++ b/fs/jbd2/journal.c > @@ -1889,7 +1889,7 @@ static struct kmem_cache *get_slab(size_t size) > BUG_ON(i >= JBD2_MAX_SLABS); > if (unlikely(i < 0)) > i = 0; > - BUG_ON(jbd2_slab[i] == 0); > + BUG_ON(jbd2_slab[i] == NULL); > return jbd2_slab[i]; > } This doesn't seem to be picked up as per today's linux-next. I have applied the patch to my queue. -- Jiri Kosina SUSE Labs, Novell Inc. -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html