netdev - Re: [PATCH RFC net-next v20] vmxnet3: Add XDP support.

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]

Date:   Tue, 18 Apr 2023 06:59:58 -0700
From:   William Tu <u9012063@...il.com>
To:     Simon Horman <simon.horman@...igine.com>
Cc:     netdev@...r.kernel.org, jsankararama@...are.com, gyang@...are.com,
        doshir@...are.com, alexander.duyck@...il.com,
        alexandr.lobakin@...el.com, bang@...are.com,
        maciej.fijalkowski@...el.com, witu@...dia.com,
        horatiu.vultur@...rochip.com, error27@...il.com,
        Alexander Duyck <alexanderduyck@...com>
Subject: Re: [PATCH RFC net-next v20] vmxnet3: Add XDP support.

On Tue, Apr 18, 2023 at 5:11 AM Simon Horman <simon.horman@...igine.com> wrote:
>
> On Wed, Apr 12, 2023 at 04:44:34PM -0700, William Tu wrote:
> > From: William Tu <u9012063@...il.com>
> >
> > The patch adds native-mode XDP support: XDP DROP, PASS, TX, and REDIRECT.
>
> ...
>
> > +/* ndo_xdp_xmit */
> > +int
> > +vmxnet3_xdp_xmit(struct net_device *dev,
> > +              int n, struct xdp_frame **frames, u32 flags)
> > +{
> > +     struct vmxnet3_adapter *adapter = netdev_priv(dev);
> > +     struct vmxnet3_tx_queue *tq;
> > +     struct netdev_queue *nq;
> > +     int i;
> > +
> > +     if (unlikely(test_bit(VMXNET3_STATE_BIT_QUIESCED, &adapter->state)))
> > +             return -ENETDOWN;
> > +     if (unlikely(test_bit(VMXNET3_STATE_BIT_RESETTING, &adapter->state)))
> > +             return -EINVAL;
> > +
> > +     tq = vmxnet3_xdp_get_tq(adapter);
> > +     if (tq->stopped)
> > +             return -ENETDOWN;
> > +
> > +     nq = netdev_get_tx_queue(adapter->netdev, tq->qid);
>
> Hi William,
>
> gcc-12 with W=1 tells me that:
>
>  drivers/net/vmxnet3/vmxnet3_xdp.c:228:23: warning: variable 'nq' set but not used [-Wunused-but-set-variable]
>          struct netdev_queue *nq;
>
Hi Simon,
Thanks a lot for reviewing and found this error.
I will fix it in next version.
William