Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Ingo Molnar
Date: Tuesday, March 20, 2007 - 10:00 am

* Linus Torvalds <torvalds@linux-foundation.org> wrote:


-rt doesnt wrap cli/sti anymore: spin_lock_irq*() doesnt disable irq 
flags on -rt, thus the amount of real irqs-off sections is very small 
and reviewable.

But nevertheless an incarnation of the code survived and is upstream 
already, in the form of TRACE_IRQFLAGS lockdep code ;) This implements a 
soft hardirq flag _today_: all that would be needed is for Xen to define 
raw_local_irq_disable() as a NOP, and to use the 
current->hardirqs_enabled as 'soft IRQ-off flag'.

Note that ->hardirqs_enabled is self-maintained, i.e. it's not just a 
stupid shadow of the hardirq flag, it's an independently maintained flag 
that does not rely on the existence of the hard flag.

[ this code even has its own debugging code, so out-of-sync-flags,
  double-off and double-on is detected and complained about. So all the 
  hard stuff has already been done as part of lockdep, and it's even 
  long-term maintainable because under a native lockdep kernel we check 
  the soft flag against the hard flag. ]

so i think a soft cli/sti flag support should be merged/unified with the 
trace_hardirqs_on()/trace_hardirqs_off() code.

	Ingo
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ..., Jeremy Fitzhardinge, (Thu Mar 1, 4:24 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Fri Mar 16, 10:36 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Fri Mar 16, 12:16 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Fri Mar 16, 1:38 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Fri Mar 16, 5:40 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Sun Mar 18, 12:59 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Sun Mar 18, 8:58 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Sun Mar 18, 10:29 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Sun Mar 18, 4:46 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Mon Mar 19, 10:58 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Eric W. Biederman, (Mon Mar 19, 11:25 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Mon Mar 19, 12:10 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Mon Mar 19, 12:33 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Mon Mar 19, 1:06 pm)
[PATCH] x86_64 : Suppress __jiffies, Eric Dumazet, (Mon Mar 19, 2:55 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Mon Mar 19, 6:45 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Eric W. Biederman, (Mon Mar 19, 9:19 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Mon Mar 19, 10:54 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Eric W. Biederman, (Tue Mar 20, 8:58 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Eric W. Biederman, (Tue Mar 20, 9:25 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Tue Mar 20, 9:26 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Tue Mar 20, 9:31 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Ingo Molnar, (Tue Mar 20, 10:00 am)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Tue Mar 20, 4:33 pm)
Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap para ..., Jeremy Fitzhardinge, (Tue Mar 20, 5:20 pm)