On Sun, Apr 6, 2008 at 12:26 AM, Yinghai Lu <yhlu.kernel@gmail.com> wrote:
quoted text
>
> On Sun, Apr 6, 2008 at 7:31 AM, Zhao Yakui <yakui.zhao@intel.com> wrote:
>  > On Thu, 2008-04-03 at 17:00 -0700, Andrew Morton wrote:
>  >  > On Thu, 3 Apr 2008 16:37:01 -0700 Greg KH <greg@kroah.com> wrote:
>  >
>  > >
>  >  > > On Thu, Apr 03, 2008 at 02:27:17PM -0700, akpm@linux-foundation.org wrote:
>  >  > > >
>  >  > > > The patch titled
>  >  > > >      acpi: fix bug 10124
>  >  > > > has been added to the -mm tree.  Its filename is
>  >  > > >      acpi-fix-bug-10124.patch
>  >  > > >
>  >  > > > Before you just go and hit "reply", please:
>  >  > > >    a) Consider who else should be cc'ed
>  >  > > >    b) Prefer to cc a suitable mailing list as well
>  >  > > >    c) Ideally: find the original patch on the mailing list and do a
>  >  > > >       reply-to-all to that, adding suitable additional cc's
>  >  > > >
>  >  > > > *** Remember to use Documentation/SubmitChecklist when testing your code ***
>  >  > > >
>  >  > > > See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find
>  >  > > > out what to do about this
>  >  > > >
>  >  > > > The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/
>  >  > > >
>  >  > > > ------------------------------------------------------
>  >  > > > Subject: acpi: fix bug 10124
>  >  > > >
>  >  > > >
>  >  > > > Unchangelogged patch, taken from
>  >  > > > http://bugzilla.kernel.org/show_bug.cgi?id=10124
>  >  > > >
>  >  > > > Cc: <yakui.zhao@intel.com>
>  >  > > > Cc: Stian Jordet <stian@jordet.net>
>  >  > > > Cc: Len Brown <lenb@kernel.org>
>  >  > > > Cc: Greg KH <greg@kroah.com>
>  >  > > > Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
>  >  > >
>  >  > > Acked-by: Greg Kroah-Hartman <gregkh@suse.de>
>  >  > >
>  >  > > This should go through Len to Linus for 2.6.25.
>  >  > >
>  >  >
>  >  > That patch was written over a month ago!  Something is wrong here.
>  >  >
>  >  > Could someone please write a changelog for it asap?  Then if there's no
>  >  > fur
>  >  The patch is out of date. Yinghai Lu sent the updated patch on March
>  >  4th. But the patch is not merged into the git tree.
>  >  I will  refresh the patch from the Yinghai Lu and send it again.
>  >  Thanks.
>  >  > ther movement I'll merge it this weekend.
>  >  >
>
>  Ingo,
>
>  A: - in x86.git/latest
>  commit 3dab95bd91b6061a887b8a199261c1d92074da85
>  Author: Yinghai Lu <yhlu.kernel@gmail.com>
>  Date:   Tue Mar 4 11:34:23 2008 -0800
>
>     ACPI: Unneccessary to scan the PCI bus already scanned.
>
>
>     http://bugzilla.kernel.org/show_bug.cgi?id=10124
>
>  is the one Yakui mentioned, but it is after
>
>  B: - in x86.git/latest
>  commit e5491e8eb56d2e4bb0e4a6795bf0662c6703d5ac
>  Author: Yinghai Lu <Yinghai.Lu@Sun.COM>
>  Date:   Tue Feb 19 03:20:09 2008 -0800
>
>     x86: get mp_bus_to_node early
>
>  C: - in x86.git/latest
>  commit c0f03f8c4c8dc0aa5fb5d0b791e76f6dff84a288
>  Author: Yinghai Lu <Yinghai.Lu@Sun.COM>
>  Date:   Wed Feb 20 12:41:52 2008 -0800
>
>     x86/acpi: make dev_to_node return online node
>
>  if the A kind patch is merged to Linus tree via Andrew, we may need
>  update B and C. to make sure pci/acpi.c is same between
>  B+C+A with new A + new B + new C

Andrew,

please use following for changelog

-------------------
    ACPI: Unneccessary to scan the PCI bus already scanned.

    http://bugzilla.kernel.org/show_bug.cgi?id=10124

    this change:

      commit 08f1c192c3c32797068bfe97738babb3295bbf42
      Author: Muli Ben-Yehuda <muli@il.ibm.com>
      Date:   Sun Jul 22 00:23:39 2007 +0300

         x86-64: introduce struct pci_sysdata to facilitate sharing of ->sysdata

         This patch introduces struct pci_sysdata to x86 and x86-64, and
         converts the existing two users (NUMA, Calgary) to use it.

         This lays the groundwork for having other users of sysdata, such as
         the PCI domains work.

         The Calgary bits are tested, the NUMA bits just look ok.

    replaces pcibios_scan_root by pci_scan_bus_parented...

    but in pcibios_scan_root we have a check about scanned busses.
---------------------


Ingo,

when new A is merged, please use attached as new B. and C seem ok...


YH