[lkml]   [2009]   [Apr]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [RESEND][PATCH -tip 2/3] x86, mce: Revert "add mce=nopoll option to disable timer polling"\
    Andi Kleen wrote:
    >> 1) There is a agent checking/clearing corrected errors
    >> (such as BIOS) other than OS.
    >> In this case, clearing MSRs by OS is not applicable.
    >> So ignore_ce is better option here.
    > Yes, but how do you make sure the option is set when the agent is active?
    > I think relying on a kernel boot option here is quite fragile.
    > Also it's something the administrator shouldn't be involved in.

    Chicken and egg?

    If there is a agent, and if the administrator cannot deactivate the agent,
    then use this option. For example, there are no updated BIOS.

    >>> Also for your use case really the better way would be to use
    >>> some way to let the firmware communicate that it doesn't want the OS
    >>> to log.
    >> Yes. However AFAIK there is no way to do it yet.
    > We can just define one. We had Linux specific extensions before.
    > Just needs a new bit somewhere. I think that would give a better
    > experience for your customers.

    You are right in part.
    But even if I defined one, it doesn't work until it actually used by
    someone. In other words, for example, it will require update of BIOS.

    > There might be something forthcomming that could be usable in fact.

    Some standard interface will be appreciated.
    Having linux specific thing is not happy for hardware vendors, I guess.

    >> The original of this repost were posted about three weeks ago (Apr.2)...
    >> I think your patches will go smoothly if my revert patches added before
    >> them.
    > The bug fixes should go in 2.6.30. After all that is what the rc stage
    > is for -- applying bug fixed.

    Maybe you overlooked that patches here are against -tip tree, don't you?
    I'd like to fix -tip tree, that will be merged in next window, 2.6.31-rc1.

    >> BTW, could you give me your Acked-by on this 2/3 too?
    > Not yet sorry.

    You have acked on 3/3... So you mean "keep mce=nopoll option in -tip" ?


     \ /
      Last update: 2009-04-20 12:49    [W:0.023 / U:49.576 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site