All of lore.kernel.org
 help / color / mirror / Atom feed
* recreate monitor page table?
@ 2007-04-18  2:13 Li, Xin B
  2007-04-18 12:52 ` Tim Deegan
  2007-04-18 13:03 ` Keir Fraser
  0 siblings, 2 replies; 7+ messages in thread
From: Li, Xin B @ 2007-04-18  2:13 UTC (permalink / raw)
  To: Tim Deegan; +Cc: xen-devel

What's the intent to recreate monitor page table when shadow levels
changes in sh_update_paging_modes?
-Xin

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: recreate monitor page table?
  2007-04-18  2:13 recreate monitor page table? Li, Xin B
@ 2007-04-18 12:52 ` Tim Deegan
  2007-04-18 15:46   ` Li, Xin B
  2007-04-18 13:03 ` Keir Fraser
  1 sibling, 1 reply; 7+ messages in thread
From: Tim Deegan @ 2007-04-18 12:52 UTC (permalink / raw)
  To: Li, Xin B; +Cc: Tim Deegan, xen-devel

At 10:13 +0800 on 18 Apr (1176891228), Li, Xin B wrote:
> What's the intent to recreate monitor page table when shadow levels
> changes in sh_update_paging_modes?

The monitor tables have to have a different layout in different shadow
paging modes so that linear maps of the shadows can work properly.

Cheers,

Tim.

-- 
Tim Deegan <Tim.Deegan@xensource.com>, XenSource UK Limited
Registered office c/o EC2Y 5EB, UK; company number 05334508

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: recreate monitor page table?
  2007-04-18  2:13 recreate monitor page table? Li, Xin B
  2007-04-18 12:52 ` Tim Deegan
@ 2007-04-18 13:03 ` Keir Fraser
  1 sibling, 0 replies; 7+ messages in thread
From: Keir Fraser @ 2007-04-18 13:03 UTC (permalink / raw)
  To: Li, Xin B, Tim Deegan; +Cc: xen-devel




On 18/4/07 03:13, "Li, Xin B" <xin.b.li@intel.com> wrote:

> What's the intent to recreate monitor page table when shadow levels
> changes in sh_update_paging_modes?
> -Xin

The format of the monitor table depends on the guest and/or shadow paging
levels, doesn't it? So presumably the easiest way to tackle that is to
regenerate the monitor table from scratch.

 -- Keir

^ permalink raw reply	[flat|nested] 7+ messages in thread

* RE: recreate monitor page table?
  2007-04-18 12:52 ` Tim Deegan
@ 2007-04-18 15:46   ` Li, Xin B
  2007-04-18 15:54     ` Keir Fraser
  2007-04-18 15:59     ` Tim Deegan
  0 siblings, 2 replies; 7+ messages in thread
From: Li, Xin B @ 2007-04-18 15:46 UTC (permalink / raw)
  To: Tim Deegan; +Cc: xen-devel

>At 10:13 +0800 on 18 Apr (1176891228), Li, Xin B wrote:
>> What's the intent to recreate monitor page table when shadow levels
>> changes in sh_update_paging_modes?
>
>The monitor tables have to have a different layout in different shadow
>paging modes so that linear maps of the shadows can work properly.
>

On x86_64, monitor page table should always be 4 level, and we can
uninstall the old shadow linear entry and install new shadow paging mode
entry, why even recreate the monitor top level page?
-Xin

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: RE: recreate monitor page table?
  2007-04-18 15:46   ` Li, Xin B
@ 2007-04-18 15:54     ` Keir Fraser
  2007-04-18 15:59     ` Tim Deegan
  1 sibling, 0 replies; 7+ messages in thread
From: Keir Fraser @ 2007-04-18 15:54 UTC (permalink / raw)
  To: Li, Xin B, Tim Deegan; +Cc: xen-devel




On 18/4/07 16:46, "Li, Xin B" <xin.b.li@intel.com> wrote:

>> At 10:13 +0800 on 18 Apr (1176891228), Li, Xin B wrote:
>>> What's the intent to recreate monitor page table when shadow levels
>>> changes in sh_update_paging_modes?
>> 
>> The monitor tables have to have a different layout in different shadow
>> paging modes so that linear maps of the shadows can work properly.
>> 
> 
> On x86_64, monitor page table should always be 4 level, and we can
> uninstall the old shadow linear entry and install new shadow paging mode
> entry, why even recreate the monitor top level page?

It could probably be optimised, but is it worth it?

 -- Keir

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: recreate monitor page table?
  2007-04-18 15:46   ` Li, Xin B
  2007-04-18 15:54     ` Keir Fraser
@ 2007-04-18 15:59     ` Tim Deegan
  2007-04-18 16:02       ` Li, Xin B
  1 sibling, 1 reply; 7+ messages in thread
From: Tim Deegan @ 2007-04-18 15:59 UTC (permalink / raw)
  To: Li, Xin B; +Cc: Tim Deegan, xen-devel

Hi :)

At 23:46 +0800 on 18 Apr (1176940011), Li, Xin B wrote:
> On x86_64, monitor page table should always be 4 level, and we can
> uninstall the old shadow linear entry and install new shadow paging mode
> entry, why even recreate the monitor top level page?

We could have made a routine to adjust the layout of the monitor table
given the old and new modes, but it would have to understand two modes
at once, which nothing else in multi.c does.  It would also reproduce 
code that's already in sh_make_monitor_table and sh_destroy_monitor_table.
Since we don't expect it to be worth optimizing this path, we just used
the existing routines.

Cheers,

Tim.  

-- 
Tim Deegan <Tim.Deegan@xensource.com>, XenSource UK Limited
Registered office c/o EC2Y 5EB, UK; company number 05334508

^ permalink raw reply	[flat|nested] 7+ messages in thread

* RE: recreate monitor page table?
  2007-04-18 15:59     ` Tim Deegan
@ 2007-04-18 16:02       ` Li, Xin B
  0 siblings, 0 replies; 7+ messages in thread
From: Li, Xin B @ 2007-04-18 16:02 UTC (permalink / raw)
  To: Tim Deegan; +Cc: xen-devel

>At 23:46 +0800 on 18 Apr (1176940011), Li, Xin B wrote:
>> On x86_64, monitor page table should always be 4 level, and we can
>> uninstall the old shadow linear entry and install new shadow 
>paging mode
>> entry, why even recreate the monitor top level page?
>
>We could have made a routine to adjust the layout of the monitor table
>given the old and new modes, but it would have to understand two modes
>at once, which nothing else in multi.c does.  It would also reproduce 
>code that's already in sh_make_monitor_table and
sh_destroy_monitor_table.
>Since we don't expect it to be worth optimizing this path, we just used
>the existing routines.


Thanks
Quite clear to me now :-)

-Xin

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2007-04-18 16:02 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-04-18  2:13 recreate monitor page table? Li, Xin B
2007-04-18 12:52 ` Tim Deegan
2007-04-18 15:46   ` Li, Xin B
2007-04-18 15:54     ` Keir Fraser
2007-04-18 15:59     ` Tim Deegan
2007-04-18 16:02       ` Li, Xin B
2007-04-18 13:03 ` Keir Fraser

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.