linux-kernel - Re: [PATCH] Add __GFP_MOVABLE for callers to flag allocations that may be migrated

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20061201110103.08d0cf3d.akpm@osdl.org>
Date:	Fri, 1 Dec 2006 11:01:03 -0800
From:	Andrew Morton <akpm@...l.org>
To:	Mel Gorman <mel@....ul.ie>
Cc:	clameter@....com,
	Linux Memory Management List <linux-mm@...ck.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] Add __GFP_MOVABLE for callers to flag allocations that
 may be migrated

On Fri, 1 Dec 2006 09:54:11 +0000 (GMT)
Mel Gorman <mel@....ul.ie> wrote:

> >> @@ -65,7 +65,7 @@ static inline void clear_user_highpage(s
> >>  static inline struct page *
> >>  alloc_zeroed_user_highpage(struct vm_area_struct *vma, unsigned long vaddr)
> >>  {
> >> -	struct page *page = alloc_page_vma(GFP_HIGHUSER, vma, vaddr);
> >> +	struct page *page = alloc_page_vma(GFP_HIGH_MOVABLE, vma, vaddr);
> >>
> >>  	if (page)
> >>  		clear_user_highpage(page, vaddr);
> >
> > But this change is presumptuous.  alloc_zeroed_user_highpage() doesn't know
> > that its caller is going to use the page for moveable purposes.  (Ditto lots
> > of other places in this patch).
> >
> 
> according to grep -r, alloc_zeroed_user_highpage() is only used in two 
> places, do_wp_page() (when write faulting the zero page)[1] and 
> do_anonymous_page() (when mapping the zero page for the first time and 
> writing). In these cases, they are known to be movable. What am I missing?

We shouldn't implement a function which "knows" how its callers are using
it in this manner.

You've gone and changed alloc_zeroed_user_highpage() into alloc_user_zeroed_highpage_which_you_must_use_in_an_application_where_it_is_movable().
Now, if we want to put a big fat comment over these functions saying that the caller
must honour the promise we've made on the caller's behalf then OK(ish).  But it'd
be better (albeit perhaps bloaty) to require the caller to pass in the gfp-flags.
-
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/