From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: schwidefsky@de.ibm.com
Cc: linux-mm@kvack.org, linux-arch@vger.kernel.org,
linux-s390@vger.kernel.org, borntraeger@de.ibm.com
Subject: Re: [patch 3/6] arch_update_pgd call
Date: Fri, 26 Oct 2007 17:59:03 +1000 [thread overview]
Message-ID: <1193385543.13638.0.camel@pasglop> (raw)
In-Reply-To: <1193384437.31831.3.camel@localhost>
On Fri, 2007-10-26 at 09:40 +0200, Martin Schwidefsky wrote:
> On Fri, 2007-10-26 at 06:48 +1000, Benjamin Herrenschmidt wrote:
> > On Thu, 2007-10-25 at 20:15 +0200, Martin Schwidefsky wrote:
> > > plain text document attachment (003-mm-update-pgd.diff)
> > > From: Martin Schwidefsky <schwidefsky@de.ibm.com>
> > >
> > > In order to change the layout of the page tables after an mmap has
> > > crossed the adress space limit of the current page table layout a
> > > architecture hook in get_unmapped_area is needed. The arguments
> > > are the address of the new mapping and the length of it.
> > >
> > > Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
> >
> > I'm not at all fan of the hook there and it's name...
> >
> > Any reason why you can't do that in your arch gua ?
> >
> > If not, then why can't you call it something nicer, like
> > arch_rebalance_pgtables() ?
>
> The name can be changed in no time. I've tried to use one of the
> existing arch calls like arch_mmap_check or arch_get_unmapped_area but
> it didn't work out. I really need the final address to make the call to
> extend the page tables.
You arch get_unmapped_area() has it...
Ben.
WARNING: multiple messages have this Message-ID (diff)
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: schwidefsky@de.ibm.com
Cc: linux-mm@kvack.org, linux-arch@vger.kernel.org,
linux-s390@vger.kernel.org, borntraeger@de.ibm.com
Subject: Re: [patch 3/6] arch_update_pgd call
Date: Fri, 26 Oct 2007 17:59:03 +1000 [thread overview]
Message-ID: <1193385543.13638.0.camel@pasglop> (raw)
In-Reply-To: <1193384437.31831.3.camel@localhost>
On Fri, 2007-10-26 at 09:40 +0200, Martin Schwidefsky wrote:
> On Fri, 2007-10-26 at 06:48 +1000, Benjamin Herrenschmidt wrote:
> > On Thu, 2007-10-25 at 20:15 +0200, Martin Schwidefsky wrote:
> > > plain text document attachment (003-mm-update-pgd.diff)
> > > From: Martin Schwidefsky <schwidefsky@de.ibm.com>
> > >
> > > In order to change the layout of the page tables after an mmap has
> > > crossed the adress space limit of the current page table layout a
> > > architecture hook in get_unmapped_area is needed. The arguments
> > > are the address of the new mapping and the length of it.
> > >
> > > Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
> >
> > I'm not at all fan of the hook there and it's name...
> >
> > Any reason why you can't do that in your arch gua ?
> >
> > If not, then why can't you call it something nicer, like
> > arch_rebalance_pgtables() ?
>
> The name can be changed in no time. I've tried to use one of the
> existing arch calls like arch_mmap_check or arch_get_unmapped_area but
> it didn't work out. I really need the final address to make the call to
> extend the page tables.
You arch get_unmapped_area() has it...
Ben.
--
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:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2007-10-26 7:59 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-25 18:15 [patch 0/6] s390 page tables on steroids Martin Schwidefsky
2007-10-25 18:15 ` Martin Schwidefsky
2007-10-25 18:15 ` [patch 1/6] add mm argument to pte/pmd/pud/pgd_free Martin Schwidefsky
2007-10-25 18:15 ` Martin Schwidefsky, Benjamin Herrenschmidt, Martin Schwidefsky
2007-10-25 18:15 ` [patch 2/6] CONFIG_HIGHPTE vs. sub-page page tables Martin Schwidefsky
2007-10-25 18:15 ` Martin Schwidefsky, Martin Schwidefsky
2007-10-25 20:47 ` Benjamin Herrenschmidt
2007-10-25 20:47 ` Benjamin Herrenschmidt
2007-10-26 7:42 ` Martin Schwidefsky
2007-10-26 7:42 ` Martin Schwidefsky
2007-10-26 8:00 ` Benjamin Herrenschmidt
2007-10-26 8:00 ` Benjamin Herrenschmidt
2007-10-26 8:08 ` Martin Schwidefsky
2007-10-26 8:08 ` Martin Schwidefsky
2007-10-26 8:06 ` Martin Schwidefsky
2007-10-26 8:06 ` Martin Schwidefsky
2007-10-25 18:15 ` [patch 3/6] arch_update_pgd call Martin Schwidefsky
2007-10-25 18:15 ` Martin Schwidefsky, Martin Schwidefsky
2007-10-25 20:48 ` Benjamin Herrenschmidt
2007-10-25 20:48 ` Benjamin Herrenschmidt
2007-10-25 22:49 ` Benjamin Herrenschmidt
2007-10-25 22:49 ` Benjamin Herrenschmidt
2007-10-26 7:40 ` Martin Schwidefsky
2007-10-26 7:40 ` Martin Schwidefsky
2007-10-26 7:59 ` Benjamin Herrenschmidt [this message]
2007-10-26 7:59 ` Benjamin Herrenschmidt
2007-10-26 8:22 ` Martin Schwidefsky
2007-10-26 8:22 ` Martin Schwidefsky
2007-10-26 11:28 ` Benjamin Herrenschmidt
2007-10-26 11:28 ` Benjamin Herrenschmidt
2007-10-25 18:15 ` [patch 4/6] s390: 1K/2K page table pages Martin Schwidefsky
2007-10-25 18:15 ` Martin Schwidefsky, Martin Schwidefsky
2007-10-25 18:15 ` [patch 5/6] s390: Add four level page tables for CONFIG_64BIT=y Martin Schwidefsky
2007-10-25 18:15 ` Martin Schwidefsky, Martin Schwidefsky
2007-10-25 18:15 ` [patch 6/6] s390: dynamic page tables Martin Schwidefsky
2007-10-25 18:15 ` Martin Schwidefsky, Martin Schwidefsky
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=1193385543.13638.0.camel@pasglop \
--to=benh@kernel.crashing.org \
--cc=borntraeger@de.ibm.com \
--cc=linux-arch@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linux-s390@vger.kernel.org \
--cc=schwidefsky@de.ibm.com \
/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.