[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.LFD.0.9999.0712222040520.21557@woody.linux-foundation.org>
Date: Sat, 22 Dec 2007 20:44:37 -0800 (PST)
From: Linus Torvalds <torvalds@...ux-foundation.org>
To: Jeff Garzik <jeff@...zik.org>
cc: Arjan van de Ven <arjan@...radead.org>,
linux-kernel@...r.kernel.org, gregkh@...e.de,
linux-pci@...ey.karlin.mff.cuni.cz,
Benjamin Herrenschmidt <benh@...nel.crashing.org>
Subject: Re: [patch] Make MMCONFIG space (extended PCI config space) a driver
opt-in issue
On Sat, 22 Dec 2007, Jeff Garzik wrote:
>
> Jeff Garzik wrote:
> > Maybe that day will never come, but it is nonetheless quite possible without
> > today's PCI Express spec for this to happen.
>
> er, s/without/within/
You're talking specs. I'm talking machines.
I agree with you 100% that as per specs, you need to support MMCONFIG,
because anything can happen.
As per *reality*, though, machines sold today simply don't need it. So
there is no upside, and there is definitely downside.
I want to limit that downside. Right now, the easiest way to limit it
seems to be to say that those (very very few) drivers that actually care
could enable it. That way, we automatically limit it to only those
machines that have hardware that cares.
And yes, if you want the capability following to notice automatically when
capabilities really do go into the 0x100+ range, that's fine. I suspect
that there aren't really very many cards that do that (because they
wouldn't work with WinXP etc), so I doubt it will actually enable MMCONFIG
for any noticeable amount of hardware sold today. Which is exactly what I
want. I do *not* want to enable MMCONFIG unless it is shown to actually be
needed.
And no, "specs" do not show it is needed. MMCONFIG is needed depending on
the actual *hardware* the kernel runs on, not based on some external
specs.
Linus
--
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