Re: [PATCH] mm: Warn once when a page is freed with PG_mlocked set V2

From: KOSAKI Motohiro
Date: Thu Jul 16 2009 - 20:17:55 EST


> On Thu, Jul 16, 2009 at 09:54:49AM -0400, Christoph Lameter wrote:
> > On Thu, 16 Jul 2009, KOSAKI Motohiro wrote:
> >
> > > I like this patch. but can you please separate two following patches?
> > > - introduce __TESTCLEARFLAG()
> > > - non-atomic test-clear of PG_mlocked on free
> >
> > That would mean introducing the macro without any use case? It is fine the
> > way it is I think.
>
> Yeah, it's borderline. In any case, I have the split version here as
> well. Andrew, you choose :)
>
> > Reviewed-by: Christoph Lameter <cl@xxxxxxxxxxxxxxxxxxxx>
>
> Thanks,

OK, I can agree with Christoph. you don't need change the patch.

Reviewed-by: KOSAKI Motohiro <kosaki.motohiro@xxxxxxxxxxxxxx>


--
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/