lkml.org 
[lkml]   [2009]   [Aug]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] md: avoid use of broken kzalloc mempool
On Tue, 11 Aug 2009, Andrew Morton wrote:

> On Tue, 4 Aug 2009 08:03:52 +1000 (EST)
> "NeilBrown" <neilb@suse.de> wrote:
>
> > On Tue, August 4, 2009 6:54 am, Sage Weil wrote:
> > > The kzalloc mempool does not re-zero items that have been used and then
> > > returned to the pool. Manually zero the allocated multipath_bh instead.
> > >
> > > CC: linux-raid@vger.kernel.org
> > > CC: Neil Brown <neilb@suse.de>
> > > CC: <stable@kernel.org>
>
> Sage, why did you cc stable@kernel.org? The patch fixes some bug?
> What is it? Why is the fix sufficiently important to warrant
> backporting?

I thought it fixed a bug in multipath (though as Neil points out it turns
out that code wasn't relying on the structure being zeroed, so there's no
need to backport). It would have been pretty rare, only affecting
multipath after low-memory conditions. I'm not sure what the criteria are
for stable, so I thought I'd be on the safe side.

sage


> > Acked-by: NeilBrown <neilb@suse.de>
> >
>
> Thanks, I'll merge it once the above is understood.
>
> >
> > In fact, you don't even need the memset. After allocation
> > no assumptions are made about the content of the structure.
> > Every field is immediately initialised except retry_list, which
> > is never accessed until after the object is placed on a list.h list.
> >
> > So this code never needed kzalloc_pool.
>
> Well, that's a bit of a functional change so let us (ie: you ;)) do
> that separately?
>
>


\
 
 \ /
  Last update: 2009-08-12 00:05    [W:1.414 / U:0.124 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site