lkml.org 
[lkml]   [2012]   [Feb]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/3] mce: Add a msg string to the MCE tracepoint
    (2012/02/29 1:11), Borislav Petkov wrote:
    > From: Borislav Petkov <borislav.petkov@amd.com>
    >
    > The idea here is to pass an additional decoded MCE message through
    > the tracepoint and into the ring buffer for userspace to consume. The
    > designated consumers are RAS daemons and other tools collecting RAS
    > information.

    I could not catch the point... Why you need this msg field?

    I think that all of information about the error is already packed in
    the record and that we can make a string from the bits in the record
    soon afterward. From my point of view it seems that what you are
    doing here is just consuming the ring buffer by repeating same
    contents in another format with dynamic length which might be short
    but otherwise could be too long.

    And one more unacceptable point is that filling this msg field is
    expected to be done in machine check context where have many
    limitations in kernel's subsystems such as use of memory allocators.

    Suggestion; How about having a kind of translator function for
    userland, e.g. an exported function named mce_record_to_msg()?
    Tool obtains raw data from the record in the tracepoint's ring buffer,
    and if it likes, optionally it can pass the record to the translator
    function to get some accomplished string.

    >
    > Drop unneeded fields while at it, thus saving some room in the ring
    > buffer.

    Really unneeded and should be killed?


    Thanks,
    H.Seto



    \
     
     \ /
      Last update: 2012-02-29 02:17    [W:0.028 / U:0.044 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site