From: "H. Peter Anvin" <hpa@zytor.com>
To: "Pan, Jacob jun" <jacob.jun.pan@intel.com>
Cc: Yinghai Lu <yinghai@kernel.org>,
"mingo@redhat.com" <mingo@redhat.com>,
"tglx@linutronix.de" <tglx@linutronix.de>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"Du, Alek" <alek.du@intel.com>,
"Siddha, Suresh B" <suresh.b.siddha@intel.com>,
Jesse Barnes <jbarnes@virtuousgeek.org>
Subject: Re: [PATCH] x86/pci: exclude mrst pci code for other platform
Date: Wed, 24 Feb 2010 13:44:49 -0800 [thread overview]
Message-ID: <4B859DD1.601@zytor.com> (raw)
In-Reply-To: <43F901BD926A4E43B106BF17856F0755A321EB54@orsmsx508.amr.corp.intel.com>
On 02/24/2010 01:03 PM, Pan, Jacob jun wrote:
> Our goal is to have a binary compatible kernel runs on Moorestown and PC, can
> we keep the Moorestown PCI code for this purpose?
CONFIG_X86_MRST is supposed to reflect the *ability* to run on
Moorestown, not the *requirement* of running on Moorestown. As such,
Yinghai's patch seems correct to me.
If you cannot boot a kernel with CONFIG_X86_MRST=y on PC hardware, that
is a bug which should be fixed.
-hpa
next prev parent reply other threads:[~2010-02-24 21:45 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-13 1:40 [PATCH 5/9] x86/ioapic: improve handling of i8259a irq init Pan, Jacob jun
2010-02-20 1:28 ` [tip:x86/mrst] x86, ioapic: Improve handling of i8259A " tip-bot for Alek Du
2010-02-24 2:49 ` [PATCH] x86: fix compiling warning with legacy_pci_xxx Yinghai Lu
2010-02-24 4:17 ` [PATCH] x86: remove left over nr_legacy_irqs Yinghai Lu
2010-02-24 4:27 ` [PATCH -v2] " Yinghai Lu
2010-02-24 8:09 ` [tip:x86/mrst] x86, legacy_irq: Remove " tip-bot for Yinghai Lu
2010-02-24 19:04 ` tip-bot for Yinghai Lu
2010-02-24 20:39 ` [PATCH] x86/pci: exclude mrst pci code for other platform Yinghai Lu
2010-02-24 21:03 ` Pan, Jacob jun
2010-02-24 21:44 ` H. Peter Anvin [this message]
2010-02-24 22:10 ` Yinghai Lu
2010-02-24 22:31 ` H. Peter Anvin
2010-02-24 22:42 ` Yinghai Lu
2010-02-25 0:41 ` [PATCH] x86/mrst: add dependencies for Moorestown Pan, Jacob jun
2010-02-25 6:01 ` Yinghai Lu
2010-02-25 22:27 ` [tip:x86/mrst] x86, pci: Exclude Moorestown PCI code if CONFIG_X86_MRST=n tip-bot for Yinghai Lu
2010-02-24 8:09 ` [tip:x86/mrst] x86, legacy_irq: Remove duplicate vector assigment tip-bot for Yinghai Lu
2010-02-24 19:04 ` tip-bot for Yinghai Lu
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=4B859DD1.601@zytor.com \
--to=hpa@zytor.com \
--cc=alek.du@intel.com \
--cc=jacob.jun.pan@intel.com \
--cc=jbarnes@virtuousgeek.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=suresh.b.siddha@intel.com \
--cc=tglx@linutronix.de \
--cc=yinghai@kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.