lkml.org 
[lkml]   [2019]   [Aug]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v3 0/8] AMD64 EDAC fixes
    On Mon, Aug 26, 2019 at 02:19:18PM +0000, Ghannam, Yazen wrote:
    > I was tracking down the failure with ECC disabled, and that seems to be it.
    >
    > So I think we should return 0 "if (!edac_has_mcs())", because we'd only get
    > there if ECC is disabled on all nodes and there wasn't some other initialization
    > error.
    >
    > I'll send a patch for this soon.
    >
    > Adam, would you mind testing this patch?

    You can't return 0 when ECC is disabled on all nodes because then the
    driver remains loaded without driving anything. That silly userspace
    needs to understand that ENODEV means "stop trying to load this driver".

    --
    Regards/Gruss,
    Boris.

    Good mailing practices for 400: avoid top-posting and trim the reply.

    \
     
     \ /
      Last update: 2019-08-26 17:00    [W:4.312 / U:0.996 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site