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

On Thu, 15 Feb 2007 22:14:45 -0800 Dan Hecht <[email protected]> wrote:

> >  config PREEMPT
> >  	bool "Preemptible Kernel (Low-Latency Desktop)"
> > +	depends on !XEN
> >  	help
> >  	  This option reduces the latency of the kernel by making
> >  	  all kernel code (that is not executing in a critical section)
> > 

Oh, so that's why it doesn't break when CONFIG_PREEMPT=y.  In which case
that preempt_disable() I spotted is wrong-and-unneeded.

Why doesn't Xen work with preemption??

> I hate to sound like a broken record, but this really isn't the right 
> way to do this.  If you are going to inhibit config settings when Xen 
> support is compiled, then it should be:
> 
> config XEN
>      depends on !KEXEC && !DOUBLEFAULT && HZ_100 && !PREEMPT
> 

Agree.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: [patch 14/21] Xen-paravirt: Add XEN config options and disable unsupported config options.
    • From: Jeremy Fitzhardinge <[email protected]>

• References:
  • [patch 00/21] Xen-paravirt: Xen guest implementation for paravirt_ops interface
    • From: Jeremy Fitzhardinge <[email protected]>
  • [patch 14/21] Xen-paravirt: Add XEN config options and disable unsupported config options.
    • From: Jeremy Fitzhardinge <[email protected]>
  • Re: [patch 14/21] Xen-paravirt: Add XEN config options and disable unsupported config options.
    • From: Dan Hecht <[email protected]>

• Prev by Date: Re: [patch 11/21] Xen-paravirt: Add apply_to_page_range() which applies a function to a pte range.
• Next by Date: Re: [patch 12/21] Xen-paravirt: Allocate and free vmalloc areas
• Previous by thread: Re: [patch 14/21] Xen-paravirt: Add XEN config options and disable unsupported config options.
• Next by thread: Re: [patch 14/21] Xen-paravirt: Add XEN config options and disable unsupported config options.
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]