linux-kernel - Re: [PATCH v1] dtivers: ntb: fix parameter check in perf_setup

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] [day] [month] [year] [list]

Message-ID: <k7tdkycc2r4f4zjyzv3zdhufkt5lpeyyouptgf6pij4mpxi3yt@aulhd5he5ydm>
Date:   Sun, 13 Aug 2023 17:48:25 +0300
From:   Serge Semin <fancer.lancer@...il.com>
To:     Minjie Du <duminjie@...o.com>
Cc:     Jon Mason <jdmason@...zu.us>, Dave Jiang <dave.jiang@...el.com>,
        Allen Hubbe <allenbh@...il.com>,
        Stephen Rothwell <sfr@...b.auug.org.au>,
        "open list:NTB DRIVER CORE" <ntb@...ts.linux.dev>,
        open list <linux-kernel@...r.kernel.org>,
        opensource.kernel@...o.com
Subject: Re: [PATCH v1] dtivers: ntb: fix parameter check in
 perf_setup_dbgfs()

On Thu, Jul 13, 2023 at 11:47:27AM +0800, Minjie Du wrote:
> Make IS_ERR() judge the debugfs_create_dir() function return
> in perf_setup_dbgfs().

The patch itself is correct for sure:
Reviewed-by: Serge Semin <fancer.lancer@...il.com>

But as I already noted in a similar patch submitted for ntb_tool.c:
https://lore.kernel.org/ntb/5d0cd0e0-d92e-42d3-a6d9-ec9fc3229b7b@kadam.mountain/T/#m78d5823691d2f205d43aaa9e09028674d57296ab

Neither NTB perf, nor NTB tool, nor NTB pong-ping (nor likely NTB MSI
test) drivers make any sense without DebugFS because their
functionality has been developed based on the DebugFS nodes. The
better change would be to just fail the NTB devices probing in these
drivers if DebugFS is unsupported.

-Serge(y)

> 
> Signed-off-by: Minjie Du <duminjie@...o.com>
> ---
>  drivers/ntb/test/ntb_perf.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/ntb/test/ntb_perf.c b/drivers/ntb/test/ntb_perf.c
> index 65e1e5cf1..553f1f46b 100644
> --- a/drivers/ntb/test/ntb_perf.c
> +++ b/drivers/ntb/test/ntb_perf.c
> @@ -1355,7 +1355,7 @@ static void perf_setup_dbgfs(struct perf_ctx *perf)
>  	struct pci_dev *pdev = perf->ntb->pdev;
>  
>  	perf->dbgfs_dir = debugfs_create_dir(pci_name(pdev), perf_dbgfs_topdir);
> -	if (!perf->dbgfs_dir) {
> +	if (IS_ERR(perf->dbgfs_dir)) {
>  		dev_warn(&perf->ntb->dev, "DebugFS unsupported\n");
>  		return;
>  	}
> -- 
> 2.39.0
> 
>