All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steve Longerbeam <stevel@mvista.com>
To: Andi Kleen <ak@muc.de>
Cc: Hugh Dickins <hugh@veritas.com>, Ray Bryant <raybry@sgi.com>,
	Hirokazu Takahashi <taka@valinux.co.jp>,
	Dave Hansen <haveblue@us.ibm.com>,
	Marcello Tosatti <marcelo.tosatti@cyclades.com>,
	Kernel Mailing List <linux-kernel@vger.kernel.org>,
	linux-mm <linux-mm@kvack.org>, andrew morton <akpm@osdl.org>
Subject: Re: page migration patchset
Date: Thu, 06 Jan 2005 15:43:39 -0800	[thread overview]
Message-ID: <41DDCD2B.4060709@mvista.com> (raw)
In-Reply-To: <20050106144307.GB59451@muc.de>

Andi Kleen wrote:

>On Wed, Jan 05, 2005 at 03:56:29PM -0800, Steve Longerbeam wrote:
>  
>
>>Hugetlbfs is also defining its own shared policy RB tree in its
>>inode info struct, but it doesn't seem to be used, just initialized
>>and freed at alloc/destroy inode time. Does anyone know why that
>>is there? A place-holder for future hugetlbfs mempolicy support?
>>If so, it can be removed and use the generic_file policies instead.
>>    
>>
>
>You need lazy hugetlbfs to use it (= allocate at page fault time,
>not mmap time). Otherwise the policy can never be applied. I implemented 
>my own version of lazy allocation for SLES9, but when I wanted to 
>merge it into mainline some other people told they had a much better 
>singing&dancing lazy hugetlb patch. So I waited for them, but they 
>never went forward with their stuff and their code seems to be dead
>now. So this is still a dangling end :/
>
>If nothing happens soon regarding the "other" hugetlb code I will
>forward port my SLES9 code. It already has NUMA policy support.
>
>For now you can remove the hugetlb policy code from mainline if you
>want, it would be easy to readd it when lazy hugetlbfs is merged.
>  
>

if you don't mind I'd like to. Sounds as if lazy hugetlbfs would be able to
make use of the generic file mapping->policy instead of a hugetlb-specific
policy anyway. Same goes for shmem.

Steve



WARNING: multiple messages have this Message-ID (diff)
From: Steve Longerbeam <stevel@mvista.com>
To: Andi Kleen <ak@muc.de>
Cc: Hugh Dickins <hugh@veritas.com>, Ray Bryant <raybry@sgi.com>,
	Hirokazu Takahashi <taka@valinux.co.jp>,
	Dave Hansen <haveblue@us.ibm.com>,
	Marcello Tosatti <marcelo.tosatti@cyclades.com>,
	Kernel Mailing List <linux-kernel@vger.kernel.org>,
	linux-mm <linux-mm@kvack.org>, andrew morton <akpm@osdl.org>
Subject: Re: page migration patchset
Date: Thu, 06 Jan 2005 15:43:39 -0800	[thread overview]
Message-ID: <41DDCD2B.4060709@mvista.com> (raw)
In-Reply-To: <20050106144307.GB59451@muc.de>

Andi Kleen wrote:

>On Wed, Jan 05, 2005 at 03:56:29PM -0800, Steve Longerbeam wrote:
>  
>
>>Hugetlbfs is also defining its own shared policy RB tree in its
>>inode info struct, but it doesn't seem to be used, just initialized
>>and freed at alloc/destroy inode time. Does anyone know why that
>>is there? A place-holder for future hugetlbfs mempolicy support?
>>If so, it can be removed and use the generic_file policies instead.
>>    
>>
>
>You need lazy hugetlbfs to use it (= allocate at page fault time,
>not mmap time). Otherwise the policy can never be applied. I implemented 
>my own version of lazy allocation for SLES9, but when I wanted to 
>merge it into mainline some other people told they had a much better 
>singing&dancing lazy hugetlb patch. So I waited for them, but they 
>never went forward with their stuff and their code seems to be dead
>now. So this is still a dangling end :/
>
>If nothing happens soon regarding the "other" hugetlb code I will
>forward port my SLES9 code. It already has NUMA policy support.
>
>For now you can remove the hugetlb policy code from mainline if you
>want, it would be easy to readd it when lazy hugetlbfs is merged.
>  
>

if you don't mind I'd like to. Sounds as if lazy hugetlbfs would be able to
make use of the generic file mapping->policy instead of a hugetlb-specific
policy anyway. Same goes for shmem.

Steve


--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"aart@kvack.org"> aart@kvack.org </a>

  parent reply	other threads:[~2005-01-06 23:45 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-01-05  0:32 page migration patchset Ray Bryant
2005-01-05  0:32 ` Ray Bryant
2005-01-05  2:07 ` Andi Kleen
2005-01-05  2:07   ` Andi Kleen
2005-01-05  3:20   ` Ray Bryant
2005-01-05  3:20     ` Ray Bryant
2005-01-05 18:41     ` Steve Longerbeam
2005-01-05 18:41       ` Steve Longerbeam
2005-01-05 19:23       ` Ray Bryant
2005-01-05 19:23         ` Ray Bryant
2005-01-05 23:00         ` Steve Longerbeam
2005-01-05 23:16           ` Ray Bryant
2005-01-05 23:16             ` Ray Bryant
2005-01-05 20:55       ` Hugh Dickins
2005-01-05 20:55         ` Hugh Dickins
     [not found]         ` <41DC7EAD.8010407@mvista.com>
2005-01-06 14:43           ` Andi Kleen
2005-01-06 14:43             ` Andi Kleen
2005-01-06 16:00             ` Ray Bryant
2005-01-06 16:00               ` Ray Bryant
2005-01-06 17:50               ` Christoph Lameter
2005-01-06 17:50                 ` Christoph Lameter
2005-01-06 19:29                 ` Andi Kleen
2005-01-06 19:29                   ` Andi Kleen
2005-01-06 22:30             ` William Lee Irwin III
2005-01-06 22:30               ` William Lee Irwin III
2005-01-06 23:08               ` Andrew Morton
2005-01-06 23:08                 ` Andrew Morton
2005-01-06 23:15                 ` William Lee Irwin III
2005-01-06 23:15                   ` William Lee Irwin III
2005-01-06 23:21               ` Ray Bryant
2005-01-06 23:21                 ` Ray Bryant
2005-01-06 23:35                 ` William Lee Irwin III
2005-01-06 23:35                   ` William Lee Irwin III
2005-01-06 23:53               ` Anton Blanchard
2005-01-06 23:53                 ` Anton Blanchard
2005-01-07  0:06                 ` William Lee Irwin III
2005-01-07  0:06                   ` William Lee Irwin III
2005-01-07  0:31                 ` Andi Kleen
2005-01-07  0:31                   ` Andi Kleen
2005-01-06 23:43             ` Steve Longerbeam [this message]
2005-01-06 23:43               ` Steve Longerbeam
2005-01-06 23:58               ` William Lee Irwin III
2005-01-06 23:58                 ` William Lee Irwin III
2005-01-11 15:38       ` Ray Bryant
2005-01-11 15:38         ` Ray Bryant
2005-01-11 19:00         ` Steve Longerbeam
2005-01-11 19:00           ` Steve Longerbeam
2005-01-11 19:30           ` Ray Bryant
2005-01-11 19:30             ` Ray Bryant
2005-01-11 20:59             ` Steve Longerbeam
2005-01-11 20:59               ` Steve Longerbeam
2005-01-12 12:35         ` Robin Holt
2005-01-12 12:35           ` Robin Holt
2005-01-12 18:12           ` Hugh Dickins
2005-01-12 18:12             ` Hugh Dickins
2005-01-12 18:45             ` Ray Bryant
2005-01-12 18:45               ` Ray Bryant
2005-01-12 18:53             ` Andrew Morton
2005-01-12 18:53               ` Andrew Morton
2005-01-14 13:55               ` swapspace layout improvements advocacy Tim Schmielau
2005-01-14 18:15                 ` Andrew Morton
2005-01-14 22:52                 ` Barry K. Nathan
2005-01-15  0:33                   ` Alan Cox
2005-01-15  2:26                   ` Tim Schmielau
2005-01-15  8:55                   ` Pasi Savolainen
  -- strict thread matches above, loose matches on Subject: below --
2005-01-06 20:59 page migration patchset Ray Bryant
2005-01-06 20:59 ` Ray Bryant
2005-01-06 23:04 ` Andi Kleen
2005-01-06 23:04   ` Andi Kleen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=41DDCD2B.4060709@mvista.com \
    --to=stevel@mvista.com \
    --cc=ak@muc.de \
    --cc=akpm@osdl.org \
    --cc=haveblue@us.ibm.com \
    --cc=hugh@veritas.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=marcelo.tosatti@cyclades.com \
    --cc=raybry@sgi.com \
    --cc=taka@valinux.co.jp \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.