Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 3 Jun 2020 08:56:58 +0200
From:   Robert Richter <rrichter@marvell.com>
To:     Borislav Petkov <bp@alien8.de>
CC:     Mauro Carvalho Chehab <mchehab@kernel.org>,
        Tony Luck <tony.luck@intel.com>,
        James Morse <james.morse@arm.com>,
        Aristeu Rozanski <aris@redhat.com>,
        Matthias Brugger <mbrugger@suse.com>,
        <linux-edac@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v4] EDAC/ghes: Setup DIMM label from DMI and use it in
 error reports
Message-ID: <20200603065657.zrxer5jtgryqlznt@rric.localdomain>
References: <20200528101307.23245-1-rrichter@marvell.com>
 <20200602154843.GD11634@zn.tnic>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200602154843.GD11634@zn.tnic>
User-Agent: NeoMutt/20170113 (1.7.2)
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from rric.localdomain (31.208.96.227) by HE1PR05CA0343.eurprd05.prod.outlook.com (2603:10a6:7:92::38) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3066.18 via Frontend Transport; Wed, 3 Jun 2020 06:57:06 +0000
X-Originating-IP: [31.208.96.227]
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 55d263da-3389-4cd0-42c6-08d8078b54ea
X-MS-TrafficTypeDiagnostic: BYAPR18MB2807:
X-Microsoft-Antispam-PRVS: <BYAPR18MB28071DC95ACBED6828BC89A6D9880@BYAPR18MB2807.namprd18.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:6790;
X-Forefront-PRVS: 04238CD941
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: YukT8xA7Vu23dTq1HM1cs/pHNvu/oQbf/SceYZyq4vOqeikTHavPT24MLLpEZemdnp9M8k2X7NWaPsB9j6Q+9emHkPHP+hscHRrmMk/JDOXvZwjqj548LYGysJA5wVw636H+M7HuL9BSwjPotYatXtCYa3Z6bCec1uh5/3ZHc05aksqHsrlpFBWa/Yi6mccjrgHoo0xmB9wRpP3zhpt81Yh5jOQx67N2y9rPmbWQIKoM+fEDVISGD/n2xyVs/pGzD2SzWsHy+SSgWoVqI214Siwd/1ajc4HEzFaKkAhhj2LBYVik7zfrK2E6mT2czA7OdnAYHtaVusKfXSt+nCJhIA==
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BYAPR18MB2661.namprd18.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(346002)(39850400004)(136003)(376002)(366004)(396003)(5660300002)(1076003)(6666004)(6916009)(4744005)(4326008)(478600001)(956004)(2906002)(83380400001)(8936002)(8676002)(54906003)(16526019)(53546011)(6506007)(55016002)(52116002)(7696005)(86362001)(9686003)(66946007)(66476007)(66556008)(316002)(186003)(26005);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData: W7CtzK3ygJTeTVkYKjwqxBIN0DbyOp20rtEHqwUC2AjwHO+uGoh0jIUtRPGkPXMsiqnQ1zSkbtIkVbzvjB9Ola2iGGoCpn/kwynulUQuXOVjvVjqSqv5Gz212UL9KvCJbc6HkjXZmO3N08xx2AcbZvy6MHjm/w1YPa6xDwFy/G9mHrGx81DE6TEVQVR8vLxGYuwwoVmDRCbSH52v/6ia0ChsbmPUxBQ65OpZARD1SGSkpXvvdlcZQcpwV79Q9GmJab6uuf2o8/zlJAldWJAGDA7Nrmo2eB72xSV0Y5zHXPMgXS5lTqs0lmIQy4hCNguZLvWOlSW2c6P5DXV51Zh1Lnc3BhoMVZDwdL9KOq0kxpAsxUeZXp2jCs63U5hGlv8N0gikr3tr5baUOFjah43JIJpMGD9VtgO9iEnSmHB7o1URz51WtwUsmiw/KmoKrNmVFKc7b6uUdItp7P0k8/NYe1cP0X4HK5JI94o89N1HuNY=
X-MS-Exchange-CrossTenant-Network-Message-Id: 55d263da-3389-4cd0-42c6-08d8078b54ea
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Jun 2020 06:57:08.1791
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 70e1fb47-1155-421d-87fc-2e58f638b6e0
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 0WcG/7yDAvBqD9wf156BsnOA4C5cUJToRfoaG8fZCe3D6VyyU2c6Jw8jfcmURTacsjQQ5b5eXGbi6yTAS8LLfw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR18MB2807
X-OriginatorOrg: marvell.com
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.216,18.0.687
 definitions=2020-06-03_06:2020-06-02,2020-06-03 signatures=0
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 02.06.20 17:48:43, Borislav Petkov wrote:
> On Thu, May 28, 2020 at 12:13:06PM +0200, Robert Richter wrote:

> > v4:
> > 
> >  * dimm->label: Only update dimm->label in if bank/device is found in
> >    the SMBIOS table, this keeps current behavior for machines that do
> >    not provide this information.
> > 
> >  * e->location: Keep current behavior how e->location is written.
> > 
> >  * e->label: Use dimm->label if a DIMM was found by its handle and
> >    "unknown memory" otherwise. This aligns with the edac_mc
> >    implementation.
> > 
> > Signed-off-by: Robert Richter <rrichter@marvell.com>
> > ---
> >  drivers/edac/ghes_edac.c | 37 ++++++++++++++++++++++++++-----------
> >  1 file changed, 26 insertions(+), 11 deletions(-)
> 
> Yap, looks good. I'll queue it after the merge window.

Great, thanks.

-Robert