2011-05-21 00:59:23

by Suresh Siddha

[permalink] [raw]
Subject: [patch 0/4] x86, apic: apicdriver probe cleanups - V2

Updated patchset for cleaning up apic probe sequence by using apicdriver
section.

Changes from V1:
* split up the patch into four small patches
* misc cleanups, specifying section alignment, removing duplicate messages
(from Cyrill and Yinghai's review comments)
* fixed a crash on smaller x64 systems


2011-05-21 07:46:06

by Cyrill Gorcunov

[permalink] [raw]
Subject: Re: [patch 0/4] x86, apic: apicdriver probe cleanups - V2

On 05/21/2011 04:51 AM, Suresh Siddha wrote:
> Updated patchset for cleaning up apic probe sequence by using apicdriver
> section.
>
> Changes from V1:
> * split up the patch into four small patches
> * misc cleanups, specifying section alignment, removing duplicate messages
> (from Cyrill and Yinghai's review comments)
> * fixed a crash on smaller x64 systems
>

I'll give them a spin tonight (if noone manage earlier). Thanks a lot Suresh!

--
Cyrill

2011-05-21 21:37:49

by Cyrill Gorcunov

[permalink] [raw]
Subject: Re: [patch 0/4] x86, apic: apicdriver probe cleanups - V2

On 05/21/2011 04:51 AM, Suresh Siddha wrote:
> Updated patchset for cleaning up apic probe sequence by using apicdriver
> section.
>
> Changes from V1:
> * split up the patch into four small patches
> * misc cleanups, specifying section alignment, removing duplicate messages
> (from Cyrill and Yinghai's review comments)
> * fixed a crash on smaller x64 systems
>

FWIW it runs fine on my Core 2 machine, setting apic driver to flat as expected.
For flat apic mode

Tested-by: Cyrill Gorcunov <[email protected]>

Thanks a lot Suresh!
--
Cyrill

2011-05-22 10:40:01

by Ingo Molnar

[permalink] [raw]
Subject: Re: [patch 0/4] x86, apic: apicdriver probe cleanups - V2


* Suresh Siddha <[email protected]> wrote:

> Updated patchset for cleaning up apic probe sequence by using apicdriver
> section.
>
> Changes from V1:
> * split up the patch into four small patches
> * misc cleanups, specifying section alignment, removing duplicate messages
> (from Cyrill and Yinghai's review comments)
> * fixed a crash on smaller x64 systems

Ok, this split-up looks a lot nicer. Applied - thanks guys!

Ingo