From: "Yinghai Lu" <yhlu.kernel@gmail.com>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: "Ingo Molnar" <mingo@elte.hu>,
"Thomas Gleixner" <tglx@linutronix.de>,
jbarnes@virtuousgeek.org,
"Jack Howarth" <howarth@bromo.msbb.uc.edu>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] x86,pci: dmi check for mackpro 2.2 mmconf
Date: Fri, 18 Jul 2008 10:54:28 -0700 [thread overview]
Message-ID: <86802c440807181054p422a77bdv901ebdc44a8d1b43@mail.gmail.com> (raw)
In-Reply-To: <4880D7B9.8000007@zytor.com>
On Fri, Jul 18, 2008 at 10:49 AM, H. Peter Anvin <hpa@zytor.com> wrote:
> Yinghai Lu wrote:
>>
>> On Fri, Jul 18, 2008 at 10:44 AM, H. Peter Anvin <hpa@zytor.com> wrote:
>>>
>>> Yinghai Lu wrote:
>>>>
>>>> change the mconf bus range from [0,0xff] to to [0, 0x3f]
>>>> to match range [0xf0000000, 0xf4000000) in e820 tables.
>>>>
>>>> Signed-off-by: Yinghai Lu <yhlu.kernel@gmail.com>
>>>> Tested-by: Jack Howarth <howarth@bromo.msbb.uc.edu>
>>>
>>> Why is this hard-coded instead of discovered dynamically?
>>
>> you mean according to the range reserved in e820 to find out bus range
>> that should be used for mmconf?
>
> That would make sense, since you're truncating the mmconf range based on
> some arbitrary limit. Even better would be to query the hardware register
> which is responsible for the truncation.
OK
will rework pci_mmcfg_reject_broken to
reduce size and check with is_acpi_reserved/e820_mapped to update end_bus_number
YH
next prev parent reply other threads:[~2008-07-18 17:54 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-18 16:41 Yinghai Lu
2008-07-18 16:50 ` Jack Howarth
2008-07-18 17:44 ` H. Peter Anvin
2008-07-18 17:47 ` Yinghai Lu
2008-07-18 17:49 ` H. Peter Anvin
2008-07-18 17:54 ` Yinghai Lu [this message]
2008-07-18 20:19 ` Ingo Molnar
2008-07-18 20:21 ` Yinghai Lu
2008-07-18 20:21 ` Ingo Molnar
2008-07-18 20:26 ` Yinghai Lu
2008-07-19 0:58 ` Jack Howarth
2008-07-19 1:22 ` Yinghai Lu
2008-07-19 3:28 ` Jack Howarth
2008-07-19 3:43 ` Justin Mattock
2008-07-19 4:45 ` Yinghai Lu
2008-07-19 5:12 ` Jack Howarth
2008-07-19 5:28 ` Yinghai Lu
2008-07-19 17:26 ` Jesse Barnes
2008-07-19 18:19 ` Jack Howarth
2008-07-19 18:40 ` Jack Howarth
2008-07-19 19:04 ` Jesse Barnes
2008-07-19 19:14 ` Jack Howarth
2008-07-21 6:49 ` Shaohua Li
2008-07-22 21:15 ` Jesse Barnes
2008-07-18 20:22 ` [PATCH] x86,pci: detect end_bus_number according to acpi/e820 reserved Yinghai Lu
2008-07-18 20:48 ` Ingo Molnar
2008-07-18 21:48 ` Jack Howarth
2008-07-18 22:27 ` Yinghai Lu
2008-07-18 22:42 ` Jesse Barnes
2008-07-20 6:48 ` Ingo Molnar
2008-07-19 9:16 ` [PATCH] x86,pci: detect end_bus_number according to acpi/e820 reserved - fix Yinghai Lu
2008-07-20 6:44 ` Ingo Molnar
2008-07-22 23:37 [PATCH] x86,pci: dmi check for mackpro 2.2 mmconf Jack Howarth
2008-07-23 0:23 ` Jesse Barnes
2008-07-23 1:00 ` Jack Howarth
2008-07-23 1:32 ` Jesse Barnes
2008-07-23 1:41 ` Jack Howarth
2008-07-23 1:54 ` Shaohua Li
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=86802c440807181054p422a77bdv901ebdc44a8d1b43@mail.gmail.com \
--to=yhlu.kernel@gmail.com \
--cc=howarth@bromo.msbb.uc.edu \
--cc=hpa@zytor.com \
--cc=jbarnes@virtuousgeek.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®