Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-08 Thread William Lee Irwin III
On Fri, 06 Apr 2007 17:02:58 -0700 Jeremy Fitzhardinge <[EMAIL PROTECTED]> wrote: >> You're too kind. wli's comment on the first version of this patch was >> something along the lines of "this patch causes a surprising amount of >> damage for what little it achieves". On Fri, Apr 06, 2007 at

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-08 Thread William Lee Irwin III
On Fri, 06 Apr 2007 17:02:58 -0700 Jeremy Fitzhardinge [EMAIL PROTECTED] wrote: You're too kind. wli's comment on the first version of this patch was something along the lines of this patch causes a surprising amount of damage for what little it achieves. On Fri, Apr 06, 2007 at 05:28:44PM

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Jeremy Fitzhardinge
Andrew Morton wrote: > Thanks. A replacement would suit. > Subject: Allow paravirt backend to choose kernel PMD sharing Normally when running in PAE mode, the 4th PMD maps the kernel address space, which can be shared among all processes (since they all need the same kernel mappings). Xen,

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Andrew Morton
On Fri, 06 Apr 2007 17:40:13 -0700 Jeremy Fitzhardinge <[EMAIL PROTECTED]> wrote: > Andrew Morton wrote: > > Something like that. I don't want to redo my patch if we're going to change > > your patch ;) > > > > OK. I won't specifically redo it on top of your patches, but I'll > rework it

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Jeremy Fitzhardinge
Andrew Morton wrote: > Something like that. I don't want to redo my patch if we're going to change > your patch ;) > OK. I won't specifically redo it on top of your patches, but I'll rework it to remove the inline function and add the if() statement. Do you want an incremental update or a

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Andrew Morton
On Fri, 06 Apr 2007 17:02:58 -0700 Jeremy Fitzhardinge <[EMAIL PROTECTED]> wrote: > Andrew Morton wrote: > > All this paravirt stuff isn't making the kernel any prettier, is it? > > > > You're too kind. wli's comment on the first version of this patch was > something along the lines of

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Jeremy Fitzhardinge
Andrew Morton wrote: > All this paravirt stuff isn't making the kernel any prettier, is it? > You're too kind. wli's comment on the first version of this patch was something along the lines of "this patch causes a surprising amount of damage for what little it achieves". >> ... >> >>

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Andrew Morton
On Wed, 04 Apr 2007 12:11:58 -0700 Jeremy Fitzhardinge <[EMAIL PROTECTED]> wrote: > Normally when running in PAE mode, the 4th PMD maps the kernel address > space, which can be shared among all processes (since they all need > the same kernel mappings). > > Xen, however, does not allow guests

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Andrew Morton
On Wed, 04 Apr 2007 12:11:58 -0700 Jeremy Fitzhardinge [EMAIL PROTECTED] wrote: Normally when running in PAE mode, the 4th PMD maps the kernel address space, which can be shared among all processes (since they all need the same kernel mappings). Xen, however, does not allow guests to have

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Jeremy Fitzhardinge
Andrew Morton wrote: All this paravirt stuff isn't making the kernel any prettier, is it? You're too kind. wli's comment on the first version of this patch was something along the lines of this patch causes a surprising amount of damage for what little it achieves. ... -#ifndef

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Andrew Morton
On Fri, 06 Apr 2007 17:02:58 -0700 Jeremy Fitzhardinge [EMAIL PROTECTED] wrote: Andrew Morton wrote: All this paravirt stuff isn't making the kernel any prettier, is it? You're too kind. wli's comment on the first version of this patch was something along the lines of this patch

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Jeremy Fitzhardinge
Andrew Morton wrote: Something like that. I don't want to redo my patch if we're going to change your patch ;) OK. I won't specifically redo it on top of your patches, but I'll rework it to remove the inline function and add the if() statement. Do you want an incremental update or a

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Andrew Morton
On Fri, 06 Apr 2007 17:40:13 -0700 Jeremy Fitzhardinge [EMAIL PROTECTED] wrote: Andrew Morton wrote: Something like that. I don't want to redo my patch if we're going to change your patch ;) OK. I won't specifically redo it on top of your patches, but I'll rework it to remove the

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-06 Thread Jeremy Fitzhardinge
Andrew Morton wrote: Thanks. A replacement would suit. Subject: Allow paravirt backend to choose kernel PMD sharing Normally when running in PAE mode, the 4th PMD maps the kernel address space, which can be shared among all processes (since they all need the same kernel mappings). Xen,

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-04 Thread Chris Wright
* Christoph Lameter ([EMAIL PROTECTED]) wrote: > Acked-by: Christoph Lameter <[EMAIL PROTECTED]> > > for all thats worth since I am not a i386 specialist. > > How much of the issues with page struct sharing between slab and arch code > does this address? I think the answer is 'none yet.' It

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-04 Thread Jeremy Fitzhardinge
Christoph Lameter wrote: > Acked-by: Christoph Lameter <[EMAIL PROTECTED]> > > for all thats worth since I am not a i386 specialist. > > How much of the issues with page struct sharing between slab and arch code > does this address? > I haven't been following that thread as closely as I

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-04 Thread Christoph Lameter
Acked-by: Christoph Lameter <[EMAIL PROTECTED]> for all thats worth since I am not a i386 specialist. How much of the issues with page struct sharing between slab and arch code does this address? - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message

[patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-04 Thread Jeremy Fitzhardinge
Normally when running in PAE mode, the 4th PMD maps the kernel address space, which can be shared among all processes (since they all need the same kernel mappings). Xen, however, does not allow guests to have the kernel pmd shared between page tables, so parameterize pgtable.c to allow both

[patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-04 Thread Jeremy Fitzhardinge
Normally when running in PAE mode, the 4th PMD maps the kernel address space, which can be shared among all processes (since they all need the same kernel mappings). Xen, however, does not allow guests to have the kernel pmd shared between page tables, so parameterize pgtable.c to allow both

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-04 Thread Christoph Lameter
Acked-by: Christoph Lameter [EMAIL PROTECTED] for all thats worth since I am not a i386 specialist. How much of the issues with page struct sharing between slab and arch code does this address? - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-04 Thread Jeremy Fitzhardinge
Christoph Lameter wrote: Acked-by: Christoph Lameter [EMAIL PROTECTED] for all thats worth since I am not a i386 specialist. How much of the issues with page struct sharing between slab and arch code does this address? I haven't been following that thread as closely as I should be, so

Re: [patch 07/20] Allow paravirt backend to choose kernel PMD sharing

2007-04-04 Thread Chris Wright
* Christoph Lameter ([EMAIL PROTECTED]) wrote: Acked-by: Christoph Lameter [EMAIL PROTECTED] for all thats worth since I am not a i386 specialist. How much of the issues with page struct sharing between slab and arch code does this address? I think the answer is 'none yet.' It uses page