Re: [PATCH 1/5] generic __remove_pages() support

[Badari Pulavarty]

Yasunori Goto wrote:
> Hi Badari-san.
>
>   
> > On Thu, 2008-03-06 at 12:54 -0800, Dave Hansen wrote:
> >     
> > > On Thu, 2008-03-06 at 10:55 -0800, Badari Pulavarty wrote:
> > >       
> > > > +               if (memmap)
> > > > +                       __kfree_section_memmap(memmap, PAGES_PER_SECTION);
> > > > +               return;
> > > > +       }
> > > > +
> > > > +       /*
> > > > +        * Allocations came from bootmem - how do I free up ?
> > > > +        */
> > > > +
> > > >         
> > > Shouldn't we figure this one out before merging?
> > >
> > > I think we at least need a printk() there.
> > >       
> > I can add a printk(). I am hoping Yasunori Goto has something to
> > handle this, before we really merge into mainline.
> >     
>
> Ah, yes. 
> I'm making patches for around here. I'm sorry for your waiting.
>
> BTW, do you hurry for merging your patch? 
> To be honest, I would like to solve not only here
> but also some other issues.
> But, if you hurry, I'll concentrate to solve only this.
>
> Bye.
>
>   
I am hoping to get all of this merged into 2.6.26.  Till then I would 
like this to
be tested in -mm. I am not in a hurry, but I would like to make sure 
some one is
working on the issue. Please let me know, if you have something to test 
- I will
be happy to help out.

Thanks,
Badari

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/