* R: [Bug #14886] Asus P2B-DS not detected as SMP moterboard
@ 2010-01-11 9:15 flinco-VGgt2q2+T+FeoWH0uzbU5w
2010-01-11 19:40 ` Rafael J. Wysocki
0 siblings, 1 reply; 6+ messages in thread
From: flinco-VGgt2q2+T+FeoWH0uzbU5w @ 2010-01-11 9:15 UTC (permalink / raw)
To: rjw-KKrjLPT3xs0, Linux Kernel Mailing List; +Cc: Kernel Testers List
Tried 2.6.32.3. The issue is still present.
Regards,
Lorenzo Buzzi
>----
Messaggio originale----
>Da: rjw-KKrjLPT3xs0@public.gmane.org
>Data: 10/01/2010 23.56
>A: "Linux
Kernel Mailing List"<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
>Cc: "Kernel Testers List"
<kernel-testers-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, "Lorenzo Buzzi"<flinco-VGgt2q2+T+FeoWH0uzbU5w@public.gmane.org>
>Ogg: [Bug
#14886] Asus P2B-DS not detected as SMP moterboard
>
>This message has been
generated automatically as a part of a report
>of regressions introduced
between 2.6.31 and 2.6.32.
>
>The following bug entry is on the current list of
known regressions
>introduced between 2.6.31 and 2.6.32. Please verify if it
still should
>be listed and let me know (either way).
>
>
>Bug-Entry : http:
//bugzilla.kernel.org/show_bug.cgi?id=14886
>Subject : Asus P2B-DS not
detected as SMP moterboard
>Submitter : Lorenzo Buzzi <flinco-VGgt2q2+T+FeoWH0uzbU5w@public.gmane.org>
>Date
: 2009-12-27 17:20 (15 days old)
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: R: [Bug #14886] Asus P2B-DS not detected as SMP moterboard
2010-01-11 9:15 R: [Bug #14886] Asus P2B-DS not detected as SMP moterboard flinco-VGgt2q2+T+FeoWH0uzbU5w
@ 2010-01-11 19:40 ` Rafael J. Wysocki
[not found] ` <201001112040.45514.rjw-KKrjLPT3xs0@public.gmane.org>
0 siblings, 1 reply; 6+ messages in thread
From: Rafael J. Wysocki @ 2010-01-11 19:40 UTC (permalink / raw)
To: flinco@libero.it; +Cc: Linux Kernel Mailing List, Kernel Testers List
On Monday 11 January 2010, flinco@libero.it wrote:
> Tried 2.6.32.3. The issue is still present.
Thanks for the update.
Rafael
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: R: [Bug #14886] Asus P2B-DS not detected as SMP moterboard
[not found] ` <201001112040.45514.rjw-KKrjLPT3xs0@public.gmane.org>
@ 2010-01-26 15:53 ` Dmitry Artamonow
2010-01-27 3:35 ` Feng Tang
0 siblings, 1 reply; 6+ messages in thread
From: Dmitry Artamonow @ 2010-01-26 15:53 UTC (permalink / raw)
To: Linux Kernel Mailing List
Cc: flinco-VGgt2q2+T+FeoWH0uzbU5w@public.gmane.org, Rafael J. Wysocki,
Kernel Testers List, Arkadiusz Miskiewicz, H. Peter Anvin,
Andrew Morton, Feng Tang, Len Brown
On 20:40 Mon 11 Jan , Rafael J. Wysocki wrote:
> On Monday 11 January 2010, flinco-VGgt2q2+T+FeoWH0uzbU5w@public.gmane.org wrote:
> > Tried 2.6.32.3. The issue is still present.
>
> Thanks for the update.
>
> Rafael
(I have some troubles with registering on kernel bugzilla, so posting here,
adding people from bug to CC: list)
Hi!
I'm also using P2B-DS and can confirm that starting with kernel 2.6.32
SMP stopped working (and don't work still - tested with current git
v2.6.33-rc5-238-g158c168) The issue seems to have something to do with the
fact that ACPI is blacklisted on P2B-DS. I used to workaround this bug
on newer kernels (>=2.6.32) by passing "acpi=force" in kernel arguments.
Finally, yesterday I found some time to write simple automated bisection
script and leaved it to run on machine overnight. Here's result:
---------------------------------------------------------------------------
e5b8fc6ac158f65598f58dba2c0d52ba3b412f52 is the first bad commit
commit e5b8fc6ac158f65598f58dba2c0d52ba3b412f52
Author: Len Brown <len.brown-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Date: Tue Jul 7 23:22:58 2009 -0400
ACPI: check acpi_disabled in acpi_table_parse() and acpi_table_parse_entries()
Allow consumers of the acpi_table_parse()/acpi_table_parse_entries() API
to gracefully handle the acpi_disabled=1 case via return value
rather than checking the global flag themselves.
Signed-off-by: Feng Tang <feng.tang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Signed-off-by: Len Brown <len.brown-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
---------------------------------------------------------------------------
I re-checked this result and yes - reverting this commit on both 2.6.32 and
current git (v2.6.33-rc5-238-g158c168) makes problem go away.
--
Best regards,
Dmitry "MAD" Artamonow
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: R: [Bug #14886] Asus P2B-DS not detected as SMP moterboard
2010-01-26 15:53 ` Dmitry Artamonow
@ 2010-01-27 3:35 ` Feng Tang
2010-01-27 6:21 ` Arkadiusz Miskiewicz
2010-02-01 14:54 ` [PATCH] ACPI: Remove old blacklist entries Matthew Garrett
0 siblings, 2 replies; 6+ messages in thread
From: Feng Tang @ 2010-01-27 3:35 UTC (permalink / raw)
To: Dmitry Artamonow
Cc: Linux Kernel Mailing List, flinco@libero.it, Rafael J. Wysocki,
Kernel Testers List, Arkadiusz Miskiewicz, H. Peter Anvin,
Andrew Morton, Brown, Len
On Tue, 26 Jan 2010 23:53:30 +0800
Dmitry Artamonow <mad_soft@inbox.ru> wrote:
> Hi!
> I'm also using P2B-DS and can confirm that starting with kernel 2.6.32
> SMP stopped working (and don't work still - tested with current git
> v2.6.33-rc5-238-g158c168) The issue seems to have something to do
> with the fact that ACPI is blacklisted on P2B-DS. I used to
> workaround this bug on newer kernels (>=2.6.32) by passing
> "acpi=force" in kernel arguments. Finally, yesterday I found some
> time to write simple automated bisection script and leaved it to run
> on machine overnight. Here's result:
> ---------------------------------------------------------------------------
> e5b8fc6ac158f65598f58dba2c0d52ba3b412f52 is the first bad commit
> commit e5b8fc6ac158f65598f58dba2c0d52ba3b412f52 Author: Len Brown
> <len.brown@intel.com> Date: Tue Jul 7 23:22:58 2009 -0400
>
> ACPI: check acpi_disabled in acpi_table_parse() and
> acpi_table_parse_entries()
> Allow consumers of the
> acpi_table_parse()/acpi_table_parse_entries() API to gracefully
> handle the acpi_disabled=1 case via return value rather than checking
> the global flag themselves.
> Signed-off-by: Feng Tang <feng.tang@intel.com>
> Signed-off-by: Len Brown <len.brown@intel.com>
> ---------------------------------------------------------------------------
Hi Dmitry,
That commit is
diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c
index 646d39c..f336bca 100644
--- a/drivers/acpi/tables.c
+++ b/drivers/acpi/tables.c
@@ -213,6 +213,9 @@ acpi_table_parse_entries(char *id,
unsigned long table_end;
acpi_size tbl_size;
+ if (acpi_disabled)
+ return -ENODEV;
+
if (!handler)
return -EINVAL;
@@ -277,6 +280,9 @@ int __init acpi_table_parse(char *id, acpi_table_handler handler)
struct acpi_table_header *table = NULL;
acpi_size tbl_size;
+ if (acpi_disabled)
+ return -ENODEV;
+
if (!handler)
return -EINVAL;
Which only enforces the "acpi_disabled" check and should have no
logical problem.
And I guess your platform is blacklisted and acpi_disabled is set to 1,
while it still need parse ACPI tables to get SMP info. So I would suggest
to add a "acpi=force" for your case.
Thanks,
Feng
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: R: [Bug #14886] Asus P2B-DS not detected as SMP moterboard
2010-01-27 3:35 ` Feng Tang
@ 2010-01-27 6:21 ` Arkadiusz Miskiewicz
2010-02-01 14:54 ` [PATCH] ACPI: Remove old blacklist entries Matthew Garrett
1 sibling, 0 replies; 6+ messages in thread
From: Arkadiusz Miskiewicz @ 2010-01-27 6:21 UTC (permalink / raw)
To: Feng Tang
Cc: Dmitry Artamonow, Linux Kernel Mailing List,
flinco-VGgt2q2+T+FeoWH0uzbU5w@public.gmane.org, Rafael J. Wysocki,
Kernel Testers List, H. Peter Anvin, Andrew Morton, Brown, Len
On Wednesday 27 of January 2010, Feng Tang wrote:
> On Tue, 26 Jan 2010 23:53:30 +0800
>
> Dmitry Artamonow <mad_soft-aPYA7nAdAYY@public.gmane.org> wrote:
> > Hi!
> > I'm also using P2B-DS and can confirm that starting with kernel 2.6.32
> > SMP stopped working (and don't work still - tested with current git
> > v2.6.33-rc5-238-g158c168) The issue seems to have something to do
> > with the fact that ACPI is blacklisted on P2B-DS. I used to
> > workaround this bug on newer kernels (>=2.6.32) by passing
> > "acpi=force" in kernel arguments. Finally, yesterday I found some
> > time to write simple automated bisection script and leaved it to run
> > on machine overnight. Here's result:
> > -------------------------------------------------------------------------
> > -- e5b8fc6ac158f65598f58dba2c0d52ba3b412f52 is the first bad commit
> > commit e5b8fc6ac158f65598f58dba2c0d52ba3b412f52 Author: Len Brown
> > <len.brown-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> Date: Tue Jul 7 23:22:58 2009 -0400
> >
> > ACPI: check acpi_disabled in acpi_table_parse() and
> >
> > acpi_table_parse_entries()
> >
> > Allow consumers of the
> >
> > acpi_table_parse()/acpi_table_parse_entries() API to gracefully
> > handle the acpi_disabled=1 case via return value rather than checking
> > the global flag themselves.
> >
> > Signed-off-by: Feng Tang <feng.tang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> > Signed-off-by: Len Brown <len.brown-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> >
> > -------------------------------------------------------------------------
> > --
>
> Hi Dmitry,
>
> That commit is
> diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c
> index 646d39c..f336bca 100644
> --- a/drivers/acpi/tables.c
> +++ b/drivers/acpi/tables.c
> @@ -213,6 +213,9 @@ acpi_table_parse_entries(char *id,
> unsigned long table_end;
> acpi_size tbl_size;
>
> + if (acpi_disabled)
> + return -ENODEV;
> +
> if (!handler)
> return -EINVAL;
>
> @@ -277,6 +280,9 @@ int __init acpi_table_parse(char *id,
> acpi_table_handler handler) struct acpi_table_header *table = NULL;
> acpi_size tbl_size;
>
> + if (acpi_disabled)
> + return -ENODEV;
> +
> if (!handler)
> return -EINVAL;
>
> Which only enforces the "acpi_disabled" check and should have no
> logical problem.
>
> And I guess your platform is blacklisted and acpi_disabled is set to 1,
> while it still need parse ACPI tables to get SMP info. So I would suggest
> to add a "acpi=force" for your case.
Is there a force ht-only option? acpi=ht doesn't work.
> Thanks,
> Feng
--
Arkadiusz Mi≈õkiewicz PLD/Linux Team
arekm / maven.pl http://ftp.pld-linux.org/
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ACPI: Remove old blacklist entries
2010-01-27 3:35 ` Feng Tang
2010-01-27 6:21 ` Arkadiusz Miskiewicz
@ 2010-02-01 14:54 ` Matthew Garrett
1 sibling, 0 replies; 6+ messages in thread
From: Matthew Garrett @ 2010-02-01 14:54 UTC (permalink / raw)
To: Feng Tang
Cc: Dmitry Artamonow, Linux Kernel Mailing List,
flinco-VGgt2q2+T+FeoWH0uzbU5w@public.gmane.org, Rafael J. Wysocki,
Kernel Testers List, Arkadiusz Miskiewicz, H. Peter Anvin,
Andrew Morton, Brown, Len
On Wed, Jan 27, 2010 at 11:35:20AM +0800, Feng Tang wrote:
> Which only enforces the "acpi_disabled" check and should have no
> logical problem.
>
> And I guess your platform is blacklisted and acpi_disabled is set to 1,
> while it still need parse ACPI tables to get SMP info. So I would suggest
> to add a "acpi=force" for your case.
It's actually set to force_ht, so it sounds like some blacklisting is
now stricter than it used to be. However, given that acpi=force seems to
work, how about just doing the following?
commit a8d9241dad684f7dda46889f00c9e627773e868e
Author: Matthew Garrett <mjg-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Date: Mon Feb 1 09:51:44 2010 -0500
ACPI: Remove old blacklist entries
The kernel has a set of blacklist entries that disable ACPI functionality
on various machines. These all seem to date from pre-git days and most
have no indication of what they were meant to fix. Let's work on the
assumption that we've fixed whatever it was that was broken before and so
remove most of the entries.
Signed-off-by: Matthew Garrett <mjg-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
diff --git a/arch/x86/kernel/acpi/boot.c b/arch/x86/kernel/acpi/boot.c
index fb1035c..086f0b6 100644
--- a/arch/x86/kernel/acpi/boot.c
+++ b/arch/x86/kernel/acpi/boot.c
@@ -1256,35 +1256,6 @@ static int __init disable_acpi_pci(const struct dmi_system_id *d)
return 0;
}
-static int __init dmi_disable_acpi(const struct dmi_system_id *d)
-{
- if (!acpi_force) {
- printk(KERN_NOTICE "%s detected: acpi off\n", d->ident);
- disable_acpi();
- } else {
- printk(KERN_NOTICE
- "Warning: DMI blacklist says broken, but acpi forced\n");
- }
- return 0;
-}
-
-/*
- * Limit ACPI to CPU enumeration for HT
- */
-static int __init force_acpi_ht(const struct dmi_system_id *d)
-{
- if (!acpi_force) {
- printk(KERN_NOTICE "%s detected: force use of acpi=ht\n",
- d->ident);
- disable_acpi();
- acpi_ht = 1;
- } else {
- printk(KERN_NOTICE
- "Warning: acpi=force overrules DMI blacklist: acpi=ht\n");
- }
- return 0;
-}
-
/*
* Force ignoring BIOS IRQ0 pin2 override
*/
@@ -1308,116 +1279,6 @@ static int __init dmi_ignore_irq0_timer_override(const struct dmi_system_id *d)
* works for you, please contact linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
*/
static struct dmi_system_id __initdata acpi_dmi_table[] = {
- /*
- * Boxes that need ACPI disabled
- */
- {
- .callback = dmi_disable_acpi,
- .ident = "IBM Thinkpad",
- .matches = {
- DMI_MATCH(DMI_BOARD_VENDOR, "IBM"),
- DMI_MATCH(DMI_BOARD_NAME, "2629H1G"),
- },
- },
-
- /*
- * Boxes that need acpi=ht
- */
- {
- .callback = force_acpi_ht,
- .ident = "FSC Primergy T850",
- .matches = {
- DMI_MATCH(DMI_SYS_VENDOR, "FUJITSU SIEMENS"),
- DMI_MATCH(DMI_PRODUCT_NAME, "PRIMERGY T850"),
- },
- },
- {
- .callback = force_acpi_ht,
- .ident = "HP VISUALIZE NT Workstation",
- .matches = {
- DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"),
- DMI_MATCH(DMI_PRODUCT_NAME, "HP VISUALIZE NT Workstation"),
- },
- },
- {
- .callback = force_acpi_ht,
- .ident = "Compaq Workstation W8000",
- .matches = {
- DMI_MATCH(DMI_SYS_VENDOR, "Compaq"),
- DMI_MATCH(DMI_PRODUCT_NAME, "Workstation W8000"),
- },
- },
- {
- .callback = force_acpi_ht,
- .ident = "ASUS P2B-DS",
- .matches = {
- DMI_MATCH(DMI_BOARD_VENDOR, "ASUSTeK Computer INC."),
- DMI_MATCH(DMI_BOARD_NAME, "P2B-DS"),
- },
- },
- {
- .callback = force_acpi_ht,
- .ident = "ASUS CUR-DLS",
- .matches = {
- DMI_MATCH(DMI_BOARD_VENDOR, "ASUSTeK Computer INC."),
- DMI_MATCH(DMI_BOARD_NAME, "CUR-DLS"),
- },
- },
- {
- .callback = force_acpi_ht,
- .ident = "ABIT i440BX-W83977",
- .matches = {
- DMI_MATCH(DMI_BOARD_VENDOR, "ABIT <http://www.abit.com>"),
- DMI_MATCH(DMI_BOARD_NAME, "i440BX-W83977 (BP6)"),
- },
- },
- {
- .callback = force_acpi_ht,
- .ident = "IBM Bladecenter",
- .matches = {
- DMI_MATCH(DMI_BOARD_VENDOR, "IBM"),
- DMI_MATCH(DMI_BOARD_NAME, "IBM eServer BladeCenter HS20"),
- },
- },
- {
- .callback = force_acpi_ht,
- .ident = "IBM eServer xSeries 360",
- .matches = {
- DMI_MATCH(DMI_BOARD_VENDOR, "IBM"),
- DMI_MATCH(DMI_BOARD_NAME, "eServer xSeries 360"),
- },
- },
- {
- .callback = force_acpi_ht,
- .ident = "IBM eserver xSeries 330",
- .matches = {
- DMI_MATCH(DMI_BOARD_VENDOR, "IBM"),
- DMI_MATCH(DMI_BOARD_NAME, "eserver xSeries 330"),
- },
- },
- {
- .callback = force_acpi_ht,
- .ident = "IBM eserver xSeries 440",
- .matches = {
- DMI_MATCH(DMI_BOARD_VENDOR, "IBM"),
- DMI_MATCH(DMI_PRODUCT_NAME, "eserver xSeries 440"),
- },
- },
-
- /*
- * Boxes that need ACPI PCI IRQ routing disabled
- */
- {
- .callback = disable_acpi_irq,
- .ident = "ASUS A7V",
- .matches = {
- DMI_MATCH(DMI_BOARD_VENDOR, "ASUSTeK Computer INC"),
- DMI_MATCH(DMI_BOARD_NAME, "<A7V>"),
- /* newer BIOS, Revision 1011, does work */
- DMI_MATCH(DMI_BIOS_VERSION,
- "ASUS A7V ACPI BIOS Revision 1007"),
- },
- },
{
/*
* Latest BIOS for IBM 600E (1.16) has bad pcinum
@@ -1454,14 +1315,6 @@ static struct dmi_system_id __initdata acpi_dmi_table[] = {
DMI_MATCH(DMI_BIOS_DATE, "03/21/2003")
},
},
- {
- .callback = disable_acpi_pci,
- .ident = "Acer TravelMate 36x Laptop",
- .matches = {
- DMI_MATCH(DMI_SYS_VENDOR, "Acer"),
- DMI_MATCH(DMI_PRODUCT_NAME, "TravelMate 360"),
- },
- },
{}
};
--
Matthew Garrett | mjg59-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2010-02-01 14:54 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-11 9:15 R: [Bug #14886] Asus P2B-DS not detected as SMP moterboard flinco-VGgt2q2+T+FeoWH0uzbU5w
2010-01-11 19:40 ` Rafael J. Wysocki
[not found] ` <201001112040.45514.rjw-KKrjLPT3xs0@public.gmane.org>
2010-01-26 15:53 ` Dmitry Artamonow
2010-01-27 3:35 ` Feng Tang
2010-01-27 6:21 ` Arkadiusz Miskiewicz
2010-02-01 14:54 ` [PATCH] ACPI: Remove old blacklist entries Matthew Garrett
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).