[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <m1hcuaml17.fsf@ebiederm.dsl.xmission.com>
Date: Sun, 28 Jan 2007 22:58:28 -0700
From: ebiederm@...ssion.com (Eric W. Biederman)
To: David Miller <davem@...emloft.net>
Cc: benh@...nel.crashing.org, jeff@...zik.org, greg@...ah.com,
tony.luck@...el.com, grundler@...isc-linux.org, mingo@...e.hu,
linux-kernel@...r.kernel.org, kyle@...isc-linux.org,
linuxppc-dev@...abs.org, brice@...i.com, shaohua.li@...el.com,
linux-pci@...ey.karlin.mff.cuni.cz
Subject: Re: [PATCH 0/6] MSI portability cleanups
David Miller <davem@...emloft.net> writes:
> From: ebiederm@...ssion.com (Eric W. Biederman)
> Date: Sun, 28 Jan 2007 22:18:59 -0700
>
>> Regardless of my opinion on the sanity of the hypervisor architects.
>> I have not seen anything that indicates it will be hard to support
>> the hypervisor doing everything or most of everything for us, so
>> I see no valid technical objection to it. Nor have I ever.
>>
>> So I have no problem with additional patches in that direction.
>
> Ok, that's great to hear.
>
> I know your bi-directional approach isn't exactly what Ben
> wants but he can support his machines with it. Maybe after
> some time we can agree to move from that more towards the
> totally abstracted scheme.
Moving farther has been my intention the entire time, even
while writing those patches. I'm just not prepared to do it in
one giant patch where bug hunting becomes impossible.
I think I have moved msi.c to the point it won't be a horror to
work with, so we can start seriously looking at what it will
take to support hypervisors that do this.
I don't believe there is anything generic we can do in the general
hypervisor case, so we need a way for the architecture code in
the case where it is inappropriate to write directly to the msi
and msi-x registers to have a completely different implementation of:
pci_enable_msi, pci_disable_msi, pci_enable_msix, psi_disable_msix,
and whatever other driver interface bits we have in there.
One small step at a time and we should get there soon.
Eric
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists