Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-17 Thread Rik van Riel
Dan Hecht wrote: Yes, and regardless of whether you run your periodic timer slower than HZ, calibrating time in a VM is always difficult due to the fact the kernel is time sharing the physical cpu. Why not just ask the underlying hypervisor? Upstream Xen does just that. I'm guessing we'll

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-17 Thread Rik van Riel
Dan Hecht wrote: Yes, and regardless of whether you run your periodic timer slower than HZ, calibrating time in a VM is always difficult due to the fact the kernel is time sharing the physical cpu. Why not just ask the underlying hypervisor? Upstream Xen does just that. I'm guessing we'll

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-16 Thread Dan Hecht
On 02/16/2007 01:51 PM, Zachary Amsden wrote: Keir Fraser wrote: On 16/2/07 17:46, "Jeremy Fitzhardinge" <[EMAIL PROTECTED]> wrote: Keir Fraser wrote: This initial patchset does not include save/restore support anyway, so in fact it would be consistent to have CONFIG_PREEMPT

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-16 Thread Zachary Amsden
Keir Fraser wrote: On 16/2/07 17:46, "Jeremy Fitzhardinge" <[EMAIL PROTECTED]> wrote: Keir Fraser wrote: This initial patchset does not include save/restore support anyway, so in fact it would be consistent to have CONFIG_PREEMPT configurable. I'm sure that we are going to have some

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-16 Thread Jeremy Fitzhardinge
Keir Fraser wrote: > We can extend the Xen timer interface quite easily and get rid of this one > too. In fact it doesn't *much* matter if the CONFIG_HZ differs from the Xen > ticker rate -- we modified the Linux timer ISR to handle timer interrupts at > arbitrary times already. The only drawback

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-16 Thread Keir Fraser
On 16/2/07 17:46, "Jeremy Fitzhardinge" <[EMAIL PROTECTED]> wrote: > Keir Fraser wrote: >> This initial patchset does not include save/restore support anyway, so in >> fact it would be consistent to have CONFIG_PREEMPT configurable. I'm sure >> that we are going to have some nasty bugs to fix up

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-16 Thread Jeremy Fitzhardinge
Keir Fraser wrote: > This initial patchset does not include save/restore support anyway, so in > fact it would be consistent to have CONFIG_PREEMPT configurable. I'm sure > that we are going to have some nasty bugs to fix up as a result, but we > can't fix them until we find them! Then we can

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-16 Thread Jeremy Fitzhardinge
Keir Fraser wrote: This initial patchset does not include save/restore support anyway, so in fact it would be consistent to have CONFIG_PREEMPT configurable. I'm sure that we are going to have some nasty bugs to fix up as a result, but we can't fix them until we find them! Then we can convert

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-16 Thread Keir Fraser
On 16/2/07 17:46, Jeremy Fitzhardinge [EMAIL PROTECTED] wrote: Keir Fraser wrote: This initial patchset does not include save/restore support anyway, so in fact it would be consistent to have CONFIG_PREEMPT configurable. I'm sure that we are going to have some nasty bugs to fix up as a

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-16 Thread Jeremy Fitzhardinge
Keir Fraser wrote: We can extend the Xen timer interface quite easily and get rid of this one too. In fact it doesn't *much* matter if the CONFIG_HZ differs from the Xen ticker rate -- we modified the Linux timer ISR to handle timer interrupts at arbitrary times already. The only drawback is

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-16 Thread Zachary Amsden
Keir Fraser wrote: On 16/2/07 17:46, Jeremy Fitzhardinge [EMAIL PROTECTED] wrote: Keir Fraser wrote: This initial patchset does not include save/restore support anyway, so in fact it would be consistent to have CONFIG_PREEMPT configurable. I'm sure that we are going to have some nasty

Re: [Xen-devel] Re: [patch 14/21] Xen-paravirt: Add XEN config options and disableunsupported config options.

2007-02-16 Thread Dan Hecht
On 02/16/2007 01:51 PM, Zachary Amsden wrote: Keir Fraser wrote: On 16/2/07 17:46, Jeremy Fitzhardinge [EMAIL PROTECTED] wrote: Keir Fraser wrote: This initial patchset does not include save/restore support anyway, so in fact it would be consistent to have CONFIG_PREEMPT