lkml.org 
[lkml]   [2003]   [Sep]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: HT not working by default since 2.4.22
Len,

I think you're missing Jeffs point.
He's really saying that the user want to select features. The user
doesn't really care how it is implemented. And there is no requirement
(I think) that some source files match one on one with a configuration
option.

As a user I like Jeffs proposal very much. It allows me to indicate what
I want without bothering about the implementation.
I think it would be wise to indicate in the help that HT does include
parts of ACPI.

As a programmer, I can understand your point too. But perhaps you should
do something like this (in the Makefile):
if CONFIG_HT
include part of ACPI needed for HT
endif
if CONFIG_ACPI
include all of acpi
endif
And let make fix things up.

-- Jan Evert

Brown, Len wrote:
>>Now that I've thought of it (aren't I humble), I rather like
>>CONFIG_HT.
>>It's simple and it's effects should be obvious to both developer and
>>user:
>>
>> CONFIG_HT, CONFIG_ACPI == ACPI
>> !CONFIG_HT, CONFIG_ACPI == ACPI
>> CONFIG_HT, !CONFIG_ACPI == HT-only ACPI
>> !CONFIG_HT, !CONFIG_ACPI == no ACPI
>>
>>Following the "autoconf model", what we really want to be testing with
>>CONFIG_xxx is _features_, where possible. "hyperthreading: yes/no" is
>>IMO more clear than "do I want ht-only ACPI or full ACPI",
>>while at the
>>same time being more fine-grained and future-proof.
>
>
> I like positive logic too.
> I went so far as to try to implement this back when I deleted "noht".
>
> The problem is that "!CONFIG_HT" is meaningless. It implies that
> you can have CONFIG_ACPI but still "config-out" HT, which you can't.
>
> Ie. The 2nd row above says to give me ACPI w/o HT.
> If you delete that row and reverse the polarity you get:
>
> !CONFIG_ACPI_HT_ONLY, CONFIG_ACPI == ACPI
> CONFIG_ACPI_HT_ONLY, !CONFIG_ACPI == HT-only ACPI
> !CONFIG_ACPI_HT_ONLY, !CONFIG_ACPI == no ACPI
>
> Here we can use config to emphasize that it is not possible to select
> CONFIG_ACPI and CONFIG_ACPI_HT_ONLY at the same time.
>
> Cheers,
> -Len
>
> Ps. Note that in 2.6 CONFIG_X86_HT exists and covers the sibling code.
> It depends on CONFIG_SMP, and CONFIG_ACPI_HT_ONLY depends on it. (in the
> ACPI patch)
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:48    [W:0.062 / U:0.700 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site