[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"
    Hidetoshi Seto <> writes:

    > Disabling only polling but not cmci is pointless setting.
    > Instead of "mce=nopoll" which tend to be paired with cmci disablement,
    > it rather make sense to have a "mce=ignore_ce" option that disable
    > both of polling and cmci at once. A patch for this new implementation
    > will follow this reverting patch.
    > OTOH, once booted, we can disable polling by setting check_interval
    > to 0, but there are no mention about the fact. Later Andi will post
    > updated documents that can respond this issue.

    I still think that patch has bad semantics because you leave around
    the events in the machine check registers and never clear
    them. Especially with MCA recovery that has very unfortunate side
    effects -- it means the OVER bit will be set and a in principle
    recoverable MCA will require a panic. Even without MCA recovery it has
    similar problems and will lead to confusing log output for non CE

    I think a patch to not log corrected errors would be reasonable,
    but you still need to clear the events from the machine check
    banks at least.

    So I would recommend you add a mce=dont_log_ce or somesuch
    that just guards the mce_log() call in machine_check_poll()

    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.

    Also BTW before adding new features like this it would be a good
    idea to first add the bug fixes I posted two weeks ago.


    -- -- Speaking for myself only.

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