linux-kernel - RE: [EXTERNAL] [PATCH 3/9] qed: avoid truncating work queue length

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: 
 <CO1PR18MB4666DF3B7684D340C3A3646DA1342@CO1PR18MB4666.namprd18.prod.outlook.com>
Date: Wed, 27 Mar 2024 14:04:08 +0000
From: Subbaraya Sundeep Bhatta <sbhatta@...vell.com>
To: Arnd Bergmann <arnd@...nel.org>,
        "llvm@...ts.linux.dev"
	<llvm@...ts.linux.dev>,
        Ariel Elior <aelior@...vell.com>, Manish Chopra
	<manishc@...vell.com>
CC: Arnd Bergmann <arnd@...db.de>, "David S. Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>,
        Paolo
 Abeni <pabeni@...hat.com>,
        Nathan Chancellor <nathan@...nel.org>,
        Nick
 Desaulniers <ndesaulniers@...gle.com>,
        Bill Wendling <morbo@...gle.com>,
        Justin Stitt <justinstitt@...gle.com>, Simon Horman <horms@...nel.org>,
        Konstantin Khorenko <khorenko@...tuozzo.com>,
        Sudarsana Reddy Kalluru
	<sudarsana.kalluru@...ium.com>,
        "netdev@...r.kernel.org"
	<netdev@...r.kernel.org>,
        "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
Subject: RE: [EXTERNAL] [PATCH 3/9] qed: avoid truncating work queue length

Hi,

>-----Original Message-----
>From: Arnd Bergmann <arnd@...nel.org>
>Sent: Wednesday, March 27, 2024 4:08 AM
>To: llvm@...ts.linux.dev; Ariel Elior <aelior@...vell.com>; Manish Chopra
><manishc@...vell.com>
>Cc: Arnd Bergmann <arnd@...db.de>; David S. Miller <davem@...emloft.net>;
>Eric Dumazet <edumazet@...gle.com>; Jakub Kicinski <kuba@...nel.org>; Paolo
>Abeni <pabeni@...hat.com>; Nathan Chancellor <nathan@...nel.org>; Nick
>Desaulniers <ndesaulniers@...gle.com>; Bill Wendling <morbo@...gle.com>;
>Justin Stitt <justinstitt@...gle.com>; Simon Horman <horms@...nel.org>;
>Konstantin Khorenko <khorenko@...tuozzo.com>; Sudarsana Reddy Kalluru
><sudarsana.kalluru@...ium.com>; netdev@...r.kernel.org; linux-
>kernel@...r.kernel.org
>Subject: [PATCH 3/9] qed: avoid truncating work queue length
>
>From: Arnd Bergmann <arnd@...db.de>
>
>clang complains that the temporary string for the name passed into
>alloc_workqueue() is too short for its contents:
>
>drivers/net/ethernet/qlogic/qed/qed_main.c:1218:3: error: 'snprintf' will always
>be truncated; specified size is 16, but format string expands to at least 18 [-
>Werror,-Wformat-truncation]
>
>There is no need for a temporary buffer, and the actual name of a workqueue	
>is 32 bytes (WQ_NAME_LEN), so just use the interface as intended to avoid
>the truncation.
>
>Fixes: 59ccf86fe69a ("qed: Add driver infrastucture for handling mfw requests.")
>Signed-off-by: Arnd Bergmann <arnd@...db.de>
>---
> drivers/net/ethernet/qlogic/qed/qed_main.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
>diff --git a/drivers/net/ethernet/qlogic/qed/qed_main.c
>b/drivers/net/ethernet/qlogic/qed/qed_main.c
>index c278f8893042..8159b4c315b5 100644
>--- a/drivers/net/ethernet/qlogic/qed/qed_main.c
>+++ b/drivers/net/ethernet/qlogic/qed/qed_main.c
>@@ -1206,7 +1206,6 @@ static void qed_slowpath_task(struct work_struct
>*work)
> static int qed_slowpath_wq_start(struct qed_dev *cdev)
> {
> 	struct qed_hwfn *hwfn;
>-	char name[NAME_SIZE];
> 	int i;
>
> 	if (IS_VF(cdev))
>@@ -1215,11 +1214,11 @@ static int qed_slowpath_wq_start(struct qed_dev
>*cdev)
> 	for_each_hwfn(cdev, i) {
> 		hwfn = &cdev->hwfns[i];
>
>-		snprintf(name, NAME_SIZE, "slowpath-%02x:%02x.%02x",
>-			 cdev->pdev->bus->number,
>-			 PCI_SLOT(cdev->pdev->devfn), hwfn->abs_pf_id);
>+		hwfn->slowpath_wq = alloc_workqueue("slowpath-
>%02x:%02x.%02x",
>+					 0, 0, cdev->pdev->bus->number,
>+					 PCI_SLOT(cdev->pdev->devfn),
>+					 hwfn->abs_pf_id);

Confused. This should be alloc_workqueue("slowpath-%02x:%02x.%02x",  cdev->pdev->bus->number, PCI_SLOT(cdev->pdev->devfn), hwfn->abs_pf_id, 0, 0);
Right?

Thanks,
Sundeep
>
>-		hwfn->slowpath_wq = alloc_workqueue(name, 0, 0);
> 		if (!hwfn->slowpath_wq) {
> 			DP_NOTICE(hwfn, "Cannot create slowpath
>workqueue\n");
> 			return -ENOMEM;
>--
>2.39.2
>