lkml.org 
[lkml]   [2009]   [Jun]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH RFC] x86/acpi: don't ignore I/O APICs just because there's no local APIC
On 06/15/09 14:58, Eric W. Biederman wrote:
>>> The only clean way I can see to handle this is to make xen dom0 it's own
>>> weird separate subarch that does all of the table parsing of the
>>> firmware tables in completely separate code. Then once we have something
>>> that works factoring out the commonalities into a helper library for
>>> better long term maintenance.
>>>
>>>
>> That seems like overkill. We can get things working under Xen with 3 changes:
>>
>
> All of the subtle assumptions sound like the come out differently. Which means
> you can very easily start down the road of just reusing small bits and then
> you find so many assumptions are different you have to scrap/replace or gunk
> up with if (xen) tests.
>

I think we're getting off into the weeds a bit here. I'm looking at
other options of how to fit Xen interrupt handling into the kernel in a
clean way; we may end up with a different model from the previous patch
postings (not this particular one under discussion; the ones from last
month). We can reopen this discussion when I post those patches.

However, the kernel will still need information about the I/O APICs from
ACPI so that it can perform basic interrupt routing for PCI devices (ie,
regardless of how the interrupt gets delivered, and who programs the
APIC hardware, we still need the basic information of "what io apic+pin
is this PCI device connected to?"). This particular patch is my attempt
to achieve this.

> To be very clear. We have mechanism and policy mixed in the mptable
> and related code today. While we continue to have that mixed I think
> even attempting to reuse it for Xen dom0 is a horrifically bad move.
>
> Nacked-by: "Eric W. Biederman"<ebiederm@xmission.com>
>

The only effect of this patch is to parse the I/O APIC parts of the MADT
even if it skips the local APIC parts; it causes no change in behaviour
in normal circumstances (unless you actually have a physical machine
with ACPI and I/O APICs but CPUs with no local APICs, which is guess is
possible in principle).

Can you give an example of how mechanism and policy are mixed? In what
ways could it break? Would you agree to a patch which attempts to
decouple policy and mechanism to solve these problems?

J


\
 
 \ /
  Last update: 2009-06-16 21:41    [W:0.282 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site