lkml.org 
[lkml]   [2018]   [Feb]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRE: [PATCH 2/3] x86/MCE/AMD, EDAC/mce_amd: Enumerate Reserved SMCA bank type
    Date
    > -----Original Message-----
    > From: linux-edac-owner@vger.kernel.org [mailto:linux-edac-
    > owner@vger.kernel.org] On Behalf Of Borislav Petkov
    > Sent: Thursday, February 8, 2018 10:15 AM
    > To: Ghannam, Yazen <Yazen.Ghannam@amd.com>
    > Cc: linux-edac@vger.kernel.org; linux-kernel@vger.kernel.org; bp@suse.de;
    > tony.luck@intel.com; x86@kernel.org
    > Subject: Re: [PATCH 2/3] x86/MCE/AMD, EDAC/mce_amd: Enumerate Reserved
    > SMCA bank type
    >
    > On Thu, Feb 01, 2018 at 12:48:12PM -0600, Yazen Ghannam wrote:
    > > From: Yazen Ghannam <yazen.ghannam@amd.com>
    > >
    > > Currently, bank 4 is reserved on Fam17h, so we chose not to initialize
    > > bank 4 in the smca_banks array. This means that when we check if a bank
    > > is initialized, like during boot or resume, we will see that bank 4 is
    > > not initialized and try to initialize it. This may cause a call trace,
    > > when resuming from suspend, due to *on_cpu() calls in the init path.
    >
    > Please be more specific: the rdmsr_*_on_cpu() calls issue an IPI but we're
    > running with interrupts disabled, which triggers:
    >
    > WARNING: CPU: 0 PID: 11523 at kernel/smp.c:291
    > smp_call_function_single+0xdc/0xe0
    >

    Okay.

    ...
    > > @@ -433,6 +437,9 @@ static u32 get_block_address(unsigned int cpu, u32
    > current_addr, u32 low, u32 hi
    > > u32 addr = 0, offset = 0;
    > >
    > > if (mce_flags.smca) {
    >
    > As a last patch in the series: please carve the code in this
    > if-statement into a smca_get_block_address() helper. And it doesn't need
    > the stable tag as it is only a cleanup.
    >

    Will do.

    Thanks,
    Yazen
    \
     
     \ /
      Last update: 2018-02-14 17:29    [W:3.384 / U:0.060 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site