linux-kernel - Re: of c4iw_fill_res_qp

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: <ZwKehJ34PzNN6FQx@gallifrey>
Date: Sun, 6 Oct 2024 14:28:20 +0000
From: "Dr. David Alan Gilbert" <linux@...blig.org>
To: Leon Romanovsky <leon@...nel.org>
Cc: maorg@...dia.com, bharat@...lsio.com, jgg@...pe.ca,
	linux-rdma@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: of c4iw_fill_res_qp_entry

* Leon Romanovsky (leon@...nel.org) wrote:
> On Fri, Oct 04, 2024 at 02:16:08PM +0000, Dr. David Alan Gilbert wrote:
> > Hi,
> >   One of my scripts noticed that c4iw_fill_res_qp_entry is not called
> > anywhere; It came from:
> > 
> > commit 5cc34116ccec60032dbaa92768f41e95ce2d8ec7
> > Author: Maor Gottlieb <maorg@...lanox.com>
> > Date:   Tue Jun 23 14:30:38 2020 +0300
> > 
> >     RDMA: Add dedicated QP resource tracker function
> >     
> > I was going to send a patch to deadcode it, but is it really a bug and
> > it should be assigned in c4iw_dev_ops in cxgb4/provider.c ?
> > 
> > (Note I know nothing about the innards of your driver, I'm just spotting
> > the unused function).

Thanks for the reply,

> It is a bug, something like that should be done.

Ah good I spotted; out of curiosity, what would be the symptom?

I don't have the hardware to test this, can I suggest that you send that patch?

Dave

> diff --git a/drivers/infiniband/hw/cxgb4/provider.c b/drivers/infiniband/hw/cxgb4/provider.c
> index 10a4c738b59f..e059f92d90fd 100644
> --- a/drivers/infiniband/hw/cxgb4/provider.c
> +++ b/drivers/infiniband/hw/cxgb4/provider.c
> @@ -473,6 +473,7 @@ static const struct ib_device_ops c4iw_dev_ops = {
>         .fill_res_cq_entry = c4iw_fill_res_cq_entry,
>         .fill_res_cm_id_entry = c4iw_fill_res_cm_id_entry,
>         .fill_res_mr_entry = c4iw_fill_res_mr_entry,
> +       .fill_res_qp_entry = c4iw_fill_res_qp_entry,
>         .get_dev_fw_str = get_dev_fw_str,
>         .get_dma_mr = c4iw_get_dma_mr,
>         .get_hw_stats = c4iw_get_mib,
> (END)
> 
> 
> > 
> > Thoughts?
> > 
> > Dave
> > -- 
> >  -----Open up your eyes, open up your mind, open up your code -------   
> > / Dr. David Alan Gilbert    |       Running GNU/Linux       | Happy  \ 
> > \        dave @ treblig.org |                               | In Hex /
> >  \ _________________________|_____ http://www.treblig.org   |_______/
> > 
> 
-- 
 -----Open up your eyes, open up your mind, open up your code -------   
/ Dr. David Alan Gilbert    |       Running GNU/Linux       | Happy  \ 
\        dave @ treblig.org |                               | In Hex /
 \ _________________________|_____ http://www.treblig.org   |_______/