On Fri 20-07-18 14:34:21, David Hildenbrand wrote: > The reserved bit once was used to hinder pages from getting swapped. While > this still works,
Does it? There is no single PageReserved check in the reclaim path. I have no idea when we stopped checking but it must be loooong ago. > the semantics are a little bit stronger nowadays: The > page should never be touched by anybody in the system except by the owner. > The original comment already gave a hint about that. > > So especially, these pages should also not be dumped by dumping tools. > Let's make that more clear by updating the comment. > > This will be useful especially in the future in virtual environments where > pages marked with the reserved bit might no longer be accessible. > > Cc: Andrew Morton <[email protected]> > Cc: Michal Hocko <[email protected]> > Cc: Vlastimil Babka <[email protected]> > Cc: "Kirill A. Shutemov" <[email protected]> > Cc: Matthew Wilcox <[email protected]> > Cc: Greg Kroah-Hartman <[email protected]> > Cc: David Hildenbrand <[email protected]> > Cc: Huang Ying <[email protected]> > Cc: Pavel Tatashin <[email protected]> > Cc: Miles Chen <[email protected]> > Cc: Dave Young <[email protected]> > Cc: Baoquan He <[email protected]> > Cc: "Marc-André Lureau" <[email protected]> > Cc: Petr Tesarik <[email protected]> > Signed-off-by: David Hildenbrand <[email protected]> Anyway Acked-by: Michal Hocko <[email protected]> for this change > --- > include/linux/page-flags.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/linux/page-flags.h b/include/linux/page-flags.h > index 901943e4754b..ba81e11a868c 100644 > --- a/include/linux/page-flags.h > +++ b/include/linux/page-flags.h > @@ -17,8 +17,8 @@ > /* > * Various page->flags bits: > * > - * PG_reserved is set for special pages, which can never be swapped out. Some > - * of them might not even exist... > + * PG_reserved is set for special pages, which should never be touched (read/ > + * write) by anybody except their owner. Some of them might not even exist. > * > * The PG_private bitflag is set on pagecache pages if they contain > filesystem > * specific data (which is normally at page->private). It can be used by > -- > 2.17.1 > -- Michal Hocko SUSE Labs
