lkml.org 
[lkml]   [2008]   [Sep]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC/PATCH] i386: handle all exceptions with interrupts off initially

* H. Peter Anvin <hpa@zytor.com> wrote:

> heukelum@fastmail.fm wrote:
>> From: Alexander van Heukelum <heukelum@fastmail.fm>
>>
>> x86_64 handles all exceptions with interrupts off initially, this
>> bisectable patch set does the same for i386, in (very) small steps.
>> If this is acceptable, it would make further unification of traps_32.c
>> and traps_64.c a lot easier. If it is not... why?
>>
>
> The only reason not to is that one generally doesn't want to disable
> interrupts unless necessary (bad for latency.) On 64 bits there are
> stack switches which make disabling interrupts mandatory. The only
> pitfall is if there is any code which is likely to take time, but I
> highly doubt it.

the entry paths here are really short (we enable irqs almost
immediately) so it's a non-issue in terms of worst-case latencies.

> In other words, it's not something we want to do "just because", but
> to the extent that it provides real benefit, it makes sense.

this is historically pretty fragile code so bringing the 32-bit and
64-bit variants more in line sounds like a good reason to me. For
example we had various long-living irq state annotation bugs (the
combination of kprobes and lockdep, etc.) that remained unfixed partly
due to this assymetry.

There will be details i'm sure, but the series looks quite bisectable.

Ingo


\
 
 \ /
  Last update: 2008-09-10 07:57    [W:0.137 / U:0.492 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site