diff options
author | Len Brown <len.brown@intel.com> | 2010-02-19 00:09:22 -0500 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2010-03-14 20:58:38 -0400 |
commit | 68ca406930d6380b3be7ada5f15fcf85bfcbd552 (patch) | |
tree | 866864defb2733d71bce04e342638c60f92fb928 /arch/x86/kernel/acpi/boot.c | |
parent | 4c81ba4900ab4eb24c7d2ba1aca594c644b6ce4c (diff) |
ACPI: delete the "acpi=ht" boot option
acpi=ht was important in 2003 -- before ACPI was
universally deployed and enabled by default in
the major Linux distributions.
At that time, there were a fair number of people who
or chose to, or needed to, run with acpi=off,
yet also wanted access to Hyper-threading.
Today we find that many invocations of "acpi=ht"
are accidental, and thus is it possible that it
is doing more harm than good.
In 2.6.34, we warn on invocation of acpi=ht.
In 2.6.35, we delete the boot option.
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'arch/x86/kernel/acpi/boot.c')
-rw-r--r-- | arch/x86/kernel/acpi/boot.c | 19 |
1 files changed, 3 insertions, 16 deletions
diff --git a/arch/x86/kernel/acpi/boot.c b/arch/x86/kernel/acpi/boot.c index 7914ab0ad76..63bcf39f8f2 100644 --- a/arch/x86/kernel/acpi/boot.c +++ b/arch/x86/kernel/acpi/boot.c | |||
@@ -62,7 +62,6 @@ EXPORT_SYMBOL(acpi_disabled); | |||
62 | int acpi_noirq; /* skip ACPI IRQ initialization */ | 62 | int acpi_noirq; /* skip ACPI IRQ initialization */ |
63 | int acpi_pci_disabled; /* skip ACPI PCI scan and IRQ initialization */ | 63 | int acpi_pci_disabled; /* skip ACPI PCI scan and IRQ initialization */ |
64 | EXPORT_SYMBOL(acpi_pci_disabled); | 64 | EXPORT_SYMBOL(acpi_pci_disabled); |
65 | int acpi_ht __initdata = 1; /* enable HT */ | ||
66 | 65 | ||
67 | int acpi_lapic; | 66 | int acpi_lapic; |
68 | int acpi_ioapic; | 67 | int acpi_ioapic; |
@@ -1460,9 +1459,8 @@ void __init acpi_boot_table_init(void) | |||
1460 | 1459 | ||
1461 | /* | 1460 | /* |
1462 | * If acpi_disabled, bail out | 1461 | * If acpi_disabled, bail out |
1463 | * One exception: acpi=ht continues far enough to enumerate LAPICs | ||
1464 | */ | 1462 | */ |
1465 | if (acpi_disabled && !acpi_ht) | 1463 | if (acpi_disabled) |
1466 | return; | 1464 | return; |
1467 | 1465 | ||
1468 | /* | 1466 | /* |
@@ -1493,9 +1491,8 @@ int __init early_acpi_boot_init(void) | |||
1493 | { | 1491 | { |
1494 | /* | 1492 | /* |
1495 | * If acpi_disabled, bail out | 1493 | * If acpi_disabled, bail out |
1496 | * One exception: acpi=ht continues far enough to enumerate LAPICs | ||
1497 | */ | 1494 | */ |
1498 | if (acpi_disabled && !acpi_ht) | 1495 | if (acpi_disabled) |
1499 | return 1; | 1496 | return 1; |
1500 | 1497 | ||
1501 | /* | 1498 | /* |
@@ -1513,9 +1510,8 @@ int __init acpi_boot_init(void) | |||
1513 | 1510 | ||
1514 | /* | 1511 | /* |
1515 | * If acpi_disabled, bail out | 1512 | * If acpi_disabled, bail out |
1516 | * One exception: acpi=ht continues far enough to enumerate LAPICs | ||
1517 | */ | 1513 | */ |
1518 | if (acpi_disabled && !acpi_ht) | 1514 | if (acpi_disabled) |
1519 | return 1; | 1515 | return 1; |
1520 | 1516 | ||
1521 | acpi_table_parse(ACPI_SIG_BOOT, acpi_parse_sbf); | 1517 | acpi_table_parse(ACPI_SIG_BOOT, acpi_parse_sbf); |
@@ -1550,21 +1546,12 @@ static int __init parse_acpi(char *arg) | |||
1550 | /* acpi=force to over-ride black-list */ | 1546 | /* acpi=force to over-ride black-list */ |
1551 | else if (strcmp(arg, "force") == 0) { | 1547 | else if (strcmp(arg, "force") == 0) { |
1552 | acpi_force = 1; | 1548 | acpi_force = 1; |
1553 | acpi_ht = 1; | ||
1554 | acpi_disabled = 0; | 1549 | acpi_disabled = 0; |
1555 | } | 1550 | } |
1556 | /* acpi=strict disables out-of-spec workarounds */ | 1551 | /* acpi=strict disables out-of-spec workarounds */ |
1557 | else if (strcmp(arg, "strict") == 0) { | 1552 | else if (strcmp(arg, "strict") == 0) { |
1558 | acpi_strict = 1; | 1553 | acpi_strict = 1; |
1559 | } | 1554 | } |
1560 | /* Limit ACPI just to boot-time to enable HT */ | ||
1561 | else if (strcmp(arg, "ht") == 0) { | ||
1562 | if (!acpi_force) { | ||
1563 | printk(KERN_WARNING "acpi=ht will be removed in Linux-2.6.35\n"); | ||
1564 | disable_acpi(); | ||
1565 | } | ||
1566 | acpi_ht = 1; | ||
1567 | } | ||
1568 | /* acpi=rsdt use RSDT instead of XSDT */ | 1555 | /* acpi=rsdt use RSDT instead of XSDT */ |
1569 | else if (strcmp(arg, "rsdt") == 0) { | 1556 | else if (strcmp(arg, "rsdt") == 0) { |
1570 | acpi_rsdt_forced = 1; | 1557 | acpi_rsdt_forced = 1; |