lkml.org 
[lkml]   [2009]   [Jan]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] lib/idr.c: Zero memory properly in idr_remove_all
On Wed, 14 Jan 2009 09:59:07 +0100 Stefan Richter <stefanr@s5r6.in-berlin.de> wrote:

> > --- a/lib/idr.c~lib-idrc-use-kmem_cache_zalloc-for-the-idr_layer-cache
> > +++ a/lib/idr.c
> > @@ -121,7 +121,7 @@ int idr_pre_get(struct idr *idp, gfp_t g
> > {
> > while (idp->id_free_cnt < IDR_FREE_MAX) {
> > struct idr_layer *new;
> > - new = kmem_cache_alloc(idr_layer_cache, gfp_mask);
> > + new = kmem_cache_zalloc(idr_layer_cache, gfp_mask);
> > if (new == NULL)
> > return (0);
> > move_to_free_list(idp, new);
> ...
>
> I wonder if it would be more robust --- or even necessary --- to instead
> add proper initialization code to get_from_free_list().
>
> As far as David and I tested the new idr using code in firewire, we
> called idr_remove_all() *and* idr_destroy() before any subsequent
> idr_get_new(). But in practice, idr_get_new() may of course also happen
> between idr_remove_all() and idr_destroy().
>
> And then this fix won't be sufficient, would it?

Maybe I'm having a thick day, but I'm not following you at all here.

What do you think the remaining problem is? get_from_free_list()
starts out with a not-fully-zeroed object? Something else?



\
 
 \ /
  Last update: 2009-01-14 10:27    [W:0.917 / U:0.268 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site