lkml.org 
[lkml]   [2015]   [Sep]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 0/3] x86/paravirt: Fix baremetal paravirt MSR ops
    From
    Date
    On 9/17/2015 8:29 AM, Paolo Bonzini wrote:
    >
    >
    > On 17/09/2015 17:27, Arjan van de Ven wrote:
    >>>
    >>>> ( We should double check that rdmsr()/wrmsr() results are never left
    >>>> uninitialized, but are set to zero or so, for cases where the
    >>>> return code is not
    >>>> checked. )
    >>>
    >>> It sure looks like native_read_msr_safe doesn't clear the output if
    >>> the rdmsr fails.
    >>
    >> I'd suggest to return some poison not just 0...
    >
    > What about 0 + WARN?

    why 0?

    0xdeadbeef or any other pattern (even 0x3636363636) makes more sense (of course also WARN... but most folks don't read dmesg for WARNs)

    (it's the same thing we do for list or slab poison stuff)



    \
     
     \ /
      Last update: 2015-09-17 18:01    [W:4.120 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site