lkml.org 
[lkml]   [2006]   [Feb]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] amd76x_pm: C3 powersaving for AMD K7
    On Fri, Feb 03, 2006 at 03:45:59AM -0500, Brown, Len wrote:
    > >- Enabling C2/C3 in the BIOS would be a very bad thing IMHO.
    > > From all he testing with amd76x_pm I found that is very tricky
    > > to go into C2/C3 "the right way".
    >
    > Who defines the "right way"? Is it guaranteed to work on all
    > models and all configurations? Exactly what is the reward
    > for the cost we'd be paying and the risk we'd be taking?
    >
    I'd be glad to know "the right way". All I have is something that seems
    to work on a number of boxes :-(
    There are some benefits from using it and there are some known issues.
    Whoever wants to use this must decide for himself if he's willing to
    take the risk. I'll emphasise this in the documentation.

    > > Simply reading the PM register without a
    > > suitable logic around leads to all kinds of instabilities. You need
    > > to implement this logic and then enable the hardware. The BIOS cannot
    > > do this.
    >
    > How about if we put it this way...
    > If the ACPI maintainer were an AMD employee,
    > and he accepted a patch like this specific to Intel hardware --
    > a patch that rejects whatever validation Intel, the BIOS
    > vendor and the board vendor have put into the product --
    > I'd call for his expulsion for ineptitude.

    Don't get me wrong: I didn't ask for inclusion of this patch into
    something official. All I want is to tell people: here is something
    that is useful for me and might be useful for you.

    My point was to say, that *for me* this stuff doesn't look that
    dangerous in exactly the environment it was written for:
    AMD K7 + 762 + 766/768.

    You have a well-defined position not to include this into the ACPI
    subsystem, you pointed out your reasons for (not) doing so and I accept
    that.

    For me there are just a few questions left:
    What is an appropiate way of announcing this patch on linux-kernel in the
    future? Does anybody feel uncomfortable with the way I did it in the
    past? Shall I cc: linux-acpi in the future, as suggested by Andrew?

    -jo

    --
    -rw-r--r-- 1 jo users 63 2006-02-03 14:28 /home/jo/.signature
    -
    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: 2006-02-03 15:19    [W:2.610 / U:0.880 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site