Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Robert Richter <rrichter@marvell.com>
To:     John Garry <john.garry@huawei.com>
CC:     Borislav Petkov <bp@alien8.de>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        James Morse <james.morse@arm.com>,
        "tony.luck@intel.com" <tony.luck@intel.com>,
        "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        wanghuiqiang <wanghuiqiang@huawei.com>,
        Xiaofei Tan <tanxiaofei@huawei.com>,
        Linuxarm <linuxarm@huawei.com>,
        "Huangming (Mark)" <huangming23@huawei.com>
Subject: Re: linuxnext-2019119 edac warns (was Re: edac KASAN warning in
 experimental arm64 allmodconfig boot)
Thread-Topic: linuxnext-2019119 edac warns (was Re: edac KASAN warning in
 experimental arm64 allmodconfig boot)
Thread-Index: AQHVoSgDZLrxCc1zPkWOGGwtnIkgYA==
Date:   Fri, 22 Nov 2019 11:28:50 +0000
Message-ID: <20191122112842.tmf4lkj52hpv6tqd@rric.localdomain>
References: <304df85b-8b56-b77e-1a11-aa23769f2e7c@huawei.com>
 <93bdc04e-9e8f-b766-6e97-9fd9e1460a8c@huawei.com>
 <20191121142302.rhvgkgqpiubidhtu@rric.localdomain>
 <4ff7631f-fbb7-e45f-87dd-9223beca4da7@huawei.com>
In-Reply-To: <4ff7631f-fbb7-e45f-87dd-9223beca4da7@huawei.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-messagesentrepresentingtype: 1
x-originating-ip: [31.208.96.227]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: dc1e841e-e277-4d65-3281-08d76f3f25e9
x-ms-traffictypediagnostic: MN2PR18MB2575:
x-microsoft-antispam-prvs: <MN2PR18MB2575AD6C209BBCD3435F354DD9490@MN2PR18MB2575.namprd18.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-forefront-prvs: 02296943FF
x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(4636009)(136003)(39860400002)(366004)(376002)(396003)(346002)(189003)(199004)(102836004)(7416002)(52116002)(8936002)(64756008)(66446008)(66556008)(66476007)(66946007)(316002)(3846002)(6116002)(8676002)(4326008)(229853002)(305945005)(7736002)(71190400001)(6512007)(9686003)(2906002)(81166006)(81156014)(54906003)(386003)(6506007)(53546011)(71200400001)(256004)(76176011)(6436002)(99286004)(6916009)(66066001)(508600001)(6486002)(86362001)(446003)(11346002)(25786009)(1076003)(5660300002)(26005)(6246003)(186003)(14454004);DIR:OUT;SFP:1101;SCL:1;SRVR:MN2PR18MB2575;H:MN2PR18MB3408.namprd18.prod.outlook.com;FPR:;SPF:None;LANG:en;PTR:InfoNoRecords;MX:1;A:1;
received-spf: None (protection.outlook.com: marvell.com does not designate
 permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: =?us-ascii?Q?QDNnY9Vqsr8sNw9WiTtTxN1ImXvM//odnjkrKL3S1t7Yfgd4KulDZnIIKRSb?=
 =?us-ascii?Q?vnoQmJnirqC9O3Q6L5xUYOyIAPsoNAYcfMuhVQLVZ30RUJiY08mcZnS2SWZL?=
 =?us-ascii?Q?XPzoUhkl+cmvFrPHUBGmEakuNsAAK6N+gv2HCS7GkIL+igvIltFIXfcff1Q0?=
 =?us-ascii?Q?H5HorNDEEcxEPwXqBy0+pXvrcSq/X8sjVqkL+GjtFiOCLwo3fGNwiUHbW1V+?=
 =?us-ascii?Q?+XMEG7cavfA3P+zSQffjDja8WeJ7gdRn9t8CQoDZ0YSfFvqtXlRMV3TdAuzk?=
 =?us-ascii?Q?USc0VtZzi7ZHsNjVzFTlIKU7WiY7YTiUertmf68gi22n4bLUkX6bGcBbsHS1?=
 =?us-ascii?Q?tyLXghrbpdW2krFCmOPT8Nl93dnsYD7fm5glFStkjR/GU+Rgw88/vkrsJ00g?=
 =?us-ascii?Q?3npOrH3RHmms7jkeMpplq/qUWQC9RiTpdUsuo3kHCBVoVFv5mdO7mBUv+kGg?=
 =?us-ascii?Q?KfRO0tRCsdQ4b3k/wB2IyxeJhGkUci5gsBH326twUXi+3IkETddo/z7Z0tr0?=
 =?us-ascii?Q?W+G1n9pfVya4ARas5F/Itv1S0KQh7xql0Yeq/IRJ9DIGlDzsoNWaK2n9pjYS?=
 =?us-ascii?Q?jendncNy08nMsrR3U6wm4lTIcN5sERUHozHGLRxYZrpG3VJajJM45D/qbgnq?=
 =?us-ascii?Q?KazpvD6EHiOojdzlLYuYoldyUzLa8NwTsPvfY19/G2RtUdhOYx2oBwMbpyhJ?=
 =?us-ascii?Q?x5Zl8zSCJKuexDTC0OkTzK11nQpCYHHnDb0AiqOT3vCCwZtd/4qCPfy6Yr5r?=
 =?us-ascii?Q?Ds6BElSf6WEmqN44zkf2GzE71d14qpWRDq/Bd0D/M90jSFA9Bsp0OOoBcb0A?=
 =?us-ascii?Q?ZiI5cvwnYcuQ3HD2DdbpLjnSJEk5jAoL0N5kJ/0PcbStg2VCAIPP6O3qvKKG?=
 =?us-ascii?Q?8iX8N5wf09E6/QGLFQaYOD5clz1rQHGSUSu5B9FyPB9I4NQHRFYH2MZonZ21?=
 =?us-ascii?Q?JbtFLd4FVsgiN3cwmp8bwn9XV7NVBxrEzCoEEoYc8CUOA/Y1D6Bur8h4hHOZ?=
 =?us-ascii?Q?YiXrAhnE9Hrbhhf3Q5x6nppz92CLDLZFr4bdEPTfWJCN1Phrw/SVFgKszntC?=
 =?us-ascii?Q?f6HgmaUhes6v1JZpreXe8f/4YFbvznJh7+nblWLi2YHFHqXTqljHyPJ6SDZ+?=
 =?us-ascii?Q?YPHlEtChLUp8/jKpSYYcUN/hfHIi7xx9gyYQ3l6g/x+C2Ka9g8n1N8j0jUCG?=
 =?us-ascii?Q?HcR4EMCU27s51clE31GaPs5Dlmjzld+Anjw1xKID86tsuNFAzTdv2U21JSrL?=
 =?us-ascii?Q?fF6PbBwQucAEIw3cvQy2nUvwbRfFPpw9CjhODRTsDUMGsWwzlk3RJsrT7vVe?=
 =?us-ascii?Q?lF0C79ie8wyQBtU+PZCdbUHpmI3ActkjmROoXojjnUfR5IeWSh/RGFAowfmc?=
 =?us-ascii?Q?RHsItoMnehlz6krFHww5tR3V+lzr9VCVrl7uUCT8bx0ULKcy1Ly+0/g1MRaY?=
 =?us-ascii?Q?TImK5bMZmxpfmfateFzy72Tn13qKNsLUUnp6SHNh30qyhbRUUcjqpWPmrHMM?=
 =?us-ascii?Q?R2zTgi71fIRpL4CZ0DTw0UdLfJXyLoDCUgoPMVNMzBGiiMD0im543gOkIbmF?=
 =?us-ascii?Q?a6KNnMhpB3qF8OqSOZ/Is7GMp+FevYkSDUvk5j9VMEBNcQ3GPBjsyZHnnJaY?=
 =?us-ascii?Q?v7xNAgyoMemhzCuXSi0BOcksvcYi9YkwGt4wkKn24VwOBWl/Opi00oLmWKvs?=
 =?us-ascii?Q?EQ5ssqHB6XzwfciDdiJMvBXOPPWkr0Zo+FdpWLK3Yi6Lis0V?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="us-ascii"
Content-ID: <0F5FF74244E9F442967D6CA075EBEB59@namprd18.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: dc1e841e-e277-4d65-3281-08d76f3f25e9
X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Nov 2019 11:28:50.8695
 (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: Jyze51Ybnyb9LS2rZhKHYxtHb8o17/8UbBXOX9/DNKqApJyYYwN28+xFOcMxbKAcYQ+VdHGRh+MbCBkkAvpUAg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR18MB2575
X-OriginatorOrg: marvell.com
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.95,18.0.572
 definitions=2019-11-22_02:2019-11-21,2019-11-22 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 21.11.19 15:23:42, John Garry wrote:
> On 21/11/2019 14:23, Robert Richter wrote:
> > On 21.11.19 12:34:22, John Garry wrote:

> > > [   22.046666] EDAC MC: bug in low-level driver: attempt to assign
> > > [   22.046666]     duplicate mc_idx 0 in add_mc_to_global_list()
> > > [   22.058311] ghes_edac: Can't register at EDAC core
> > > [   22.065402] EDAC MC: bug in low-level driver: attempt to assign
> > > [   22.065402]     duplicate mc_idx 0 in add_mc_to_global_list()
> > > [   22.077080] ghes_edac: Can't register at EDAC core
> > > [   22.084140] EDAC MC: bug in low-level driver: attempt to assign
> > > [   22.084140]     duplicate mc_idx 0 in add_mc_to_global_list()
> > > [   22.095789] ghes_edac: Can't register at EDAC core
> > > [   22.102873] EDAC MC: bug in low-level driver: attempt to assign
> > > [   22.102873]     duplicate mc_idx 0 in add_mc_to_global_list()
> > > [   22.115442] ghes_edac: Can't register at EDAC core
> > > [   22.122536] EDAC MC: bug in low-level driver: attempt to assign
> > > [   22.122536]     duplicate mc_idx 0 in add_mc_to_global_list()
> > > [   22.134344] ghes_edac: Can't register at EDAC core
> > > [   22.141441] EDAC MC: bug in low-level driver: attempt to assign
> > > [   22.141441]     duplicate mc_idx 0 in add_mc_to_global_list()
> > > [   22.153089] ghes_edac: Can't register at EDAC core
> > > [   22.160161] EDAC MC: bug in low-level driver: attempt to assign
> > > [   22.160161]     duplicate mc_idx 0 in add_mc_to_global_list()
> > > [   22.171810] ghes_edac: Can't register at EDAC core
> >=20
> > What I am more concerned is this here. In total this implies 8 ghes
> > users that all try to register a (single-instance) ghes mc device. For
> > non-x86 only one instance is allowed (see ghes_edac_register(), idx =3D
> > 0).

I also looked into this: With refcount_inc_checked() enabled, the
refcount is *not* increased from 0 to 1. Under the hood only
refcount_inc_not_zero() is called instead of refcount_inc(). So the
refcount is still zero after an edac mc device was registered. Instead
of sharing the edac mc device, the driver tries to allocate another mc
device for each GHESv2 entry in the HEST table. This causes the
'duplicate mc_idx' message. Also, it is ok to have multiple GHESv2
entries (your system seems to have 8 entries), e.g. to serve different
kind of errors in the system.

Thanks,

-Robert