[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1A61D8EA6755AF458F06EA669A4EC81815FDA736@JKLMAIL02.ixonos.local>
Date: Tue, 12 Oct 2010 14:23:15 +0000
From: Penttilä Mika <mika.penttila@...nos.com>
To: Andi Kleen <andi@...stfloor.org>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] HWPOISON: Implement hwpoison-on-free for soft offlining
>
> >> --- a/mm/page_alloc.c
> >> +++ b/mm/page_alloc.c
> >> @@ -564,6 +564,8 @@ static inline int free_pages_check(struct page
> *page)
> >> (page->mapping != NULL) |
> >> (atomic_read(&page->_count) != 0) |
> >> (page->flags & PAGE_FLAGS_CHECK_AT_FREE))) {
> >> + if (PageHWPoisonOnFree(page))
> >> + hwpoison_page_on_free(page);
> >
> > hwpoison_page_on_free() seems to be undefined when
> > CONFIG_HWPOISON_ON_FREE is not defined.
>
> The optimizer takes care of that.
>
> >
> >> bad_page(page);
> >> return 1;
> >> }
> >
> > And of course PAGE_FLAGS_CHECK_AT_FREE should include
> > pahehwpoisononfree this to be effective
>
> It is of course (__PG_HWPOISON_ON_FREE)
>
Yes but it's not included in PAGE_FLAGS_CHECK_AT_FREE, if I'm not missing something
--Mika
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists