Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B7F75C678D5 for ; Wed, 8 Mar 2023 20:23:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229706AbjCHUXt (ORCPT ); Wed, 8 Mar 2023 15:23:49 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44860 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229484AbjCHUXq (ORCPT ); Wed, 8 Mar 2023 15:23:46 -0500 Received: from mail-vs1-xe2a.google.com (mail-vs1-xe2a.google.com [IPv6:2607:f8b0:4864:20::e2a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 72874126DE for ; Wed, 8 Mar 2023 12:23:24 -0800 (PST) Received: by mail-vs1-xe2a.google.com with SMTP id o32so16507149vsv.12 for ; Wed, 08 Mar 2023 12:23:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1678307003; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=+RPaT5MrHSvj4O9LtIvda00nyyIYBJhBMMQAbW0pyRs=; b=F9JkXcVoblDoQmKlLDOVuticNu5VhGherOTRVqdN12OzIO5B0v1TVWd406EsRrlLIs db+cOdWorzKZyRoNnq8sLXcGkQm56hKnDE1J4Q3NMwU/kh/rek0T2Y31POsARkcCwc9g rbkZPzwZTdMLA7RyOtumdEoEL+ma5XpwLbpU4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678307003; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=+RPaT5MrHSvj4O9LtIvda00nyyIYBJhBMMQAbW0pyRs=; b=jRxk5n13xUeZDxb6JyprsgeBZiu2C2vtDVVxdkoPDpTx3aMnG/ABhXsc/dCOIHtypg 7a3hyBfxD6BwfPugohRlPeCl3AVRi6IAMyFqFkuehGphhJDGIb/nR+NSWE5hvb+d5etP UFGOYSL3KO8tyijq1sWX0n2KB7i1DAZiyKTX4KNesLHs+riQwM83ivtVJqXgGrCdlXOl VeH5BOaH0UU0R/LPyQLMETyX+6IN6HNFDgv0de557Abz70yXvvfe+zKWvH1ll/0S06w3 1eUlqMI+rpluNzEo9pXsFqMthIbO8wZSsaSoqgKbwRDj/ga4dmvsbMa7PVD3M0/Fe6/d qKBQ== X-Gm-Message-State: AO0yUKWnvnKDlCbXGTo9Jk3427B7jBcl2UPHu7NNfwRWkwQEaNoPfq0u hBwODgvHbkGt+9ZEtbw/fK301abN6dhapUn59CuslQ== X-Google-Smtp-Source: AK7set819vxr9EQa/2YNU/n4kHNJ5k7TG52gGkvDn+JASTWTeZxSkmZibh/gpAVYxJG81DMZiVDj2IX8Hf+16OcA3NU= X-Received: by 2002:a05:6102:1261:b0:412:11d2:bd97 with SMTP id q1-20020a056102126100b0041211d2bd97mr12491049vsg.2.1678307003370; Wed, 08 Mar 2023 12:23:23 -0800 (PST) MIME-Version: 1.0 References: <20230308201827.GA1039517@bhelgaas> In-Reply-To: <20230308201827.GA1039517@bhelgaas> From: Grant Grundler Date: Wed, 8 Mar 2023 12:23:12 -0800 Message-ID: Subject: Re: [PATCH] PCI/AER: correctable error message as KERN_INFO To: Bjorn Helgaas Cc: Grant Grundler , Mahesh J Salgaonkar , "Oliver O 'Halloran" , Bjorn Helgaas , Rajat Khandelwal , linuxppc-dev@lists.ozlabs.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Rajat Jain Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Mar 8, 2023 at 12:18=E2=80=AFPM Bjorn Helgaas = wrote: > > On Wed, Mar 08, 2023 at 12:00:48PM -0800, Grant Grundler wrote: > > Ping? Did I miss an email or other work that this patch collides with? > > Nope, we typically make topic branches based on -rc1, so not much > happens during the merge window. -rc1 was tagged Sunday, so things > will start appearing in -next soon. Ah ok! Thanks for clarifying Bjorn! cheers, grant > > Bjorn > > > On Tue, Feb 28, 2023 at 10:05=E2=80=AFPM Grant Grundler wrote: > > > > > > Since correctable errors have been corrected (and counted), the dmesg= output > > > should not be reported as a warning, but rather as "informational". > > > > > > Otherwise, using a certain well known vendor's PCIe parts in a USB4 d= ocking > > > station, the dmesg buffer can be spammed with correctable errors, 717= bytes > > > per instance, potentially many MB per day. > > > > > > Given the "WARN" priority, these messages have already confused the t= ypical > > > user that stumbles across them, support staff (triaging feedback repo= rts), > > > and more than a few linux kernel devs. Changing to INFO will hide the= se > > > messages from most audiences. > > > > > > Signed-off-by: Grant Grundler > > > --- > > > This patch will likely conflict with: > > > https://lore.kernel.org/all/20230103165548.570377-1-rajat.khandelwa= l@linux.intel.com/ > > > > > > which I'd also like to see upstream. Please let me know to resubmit m= ine if Rajat's patch lands first. Or feel free to fix up this one. > > > > > > drivers/pci/pcie/aer.c | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git a/drivers/pci/pcie/aer.c b/drivers/pci/pcie/aer.c > > > index f6c24ded134c..e4cf3ec40d66 100644 > > > --- a/drivers/pci/pcie/aer.c > > > +++ b/drivers/pci/pcie/aer.c > > > @@ -692,7 +692,7 @@ static void __aer_print_error(struct pci_dev *dev= , > > > > > > if (info->severity =3D=3D AER_CORRECTABLE) { > > > strings =3D aer_correctable_error_string; > > > - level =3D KERN_WARNING; > > > + level =3D KERN_INFO; > > > } else { > > > strings =3D aer_uncorrectable_error_string; > > > level =3D KERN_ERR; > > > @@ -724,7 +724,7 @@ void aer_print_error(struct pci_dev *dev, struct = aer_err_info *info) > > > layer =3D AER_GET_LAYER_ERROR(info->severity, info->status); > > > agent =3D AER_GET_AGENT(info->severity, info->status); > > > > > > - level =3D (info->severity =3D=3D AER_CORRECTABLE) ? KERN_WARN= ING : KERN_ERR; > > > + level =3D (info->severity =3D=3D AER_CORRECTABLE) ? KERN_INFO= : KERN_ERR; > > > > > > pci_printk(level, dev, "PCIe Bus Error: severity=3D%s, type= =3D%s, (%s)\n", > > > aer_error_severity_string[info->severity], > > > -- > > > 2.39.2.722.g9855ee24e9-goog > > >