netdev - Re: netif_set_xps_queue() before register_netdev(): correct from an API perspective?

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: <20230802152259.uwmstxftdk6wnjfg@skbuf>
Date: Wed, 2 Aug 2023 18:22:59 +0300
From: Vladimir Oltean <vladimir.oltean@....com>
To: Alexander Duyck <alexander.duyck@...il.com>
Cc: Ioana Ciornei <ioana.ciornei@....com>,
	Antoine Tenart <atenart@...nel.org>, netdev@...r.kernel.org
Subject: Re: netif_set_xps_queue() before register_netdev(): correct from an
 API perspective?

On Wed, Aug 02, 2023 at 08:04:01AM -0700, Alexander Duyck wrote:
> We really shouldn't be calling it before the netdev is registered.
> 
> That said though the easiest way to clean it up would probably be to
> call netdev_reset_tc in case of a failure.

Thanks. What else will happen that's bad with XPS being configured on
TXQs of unregistered net devices? The call path has existed since 2017 -
commit 93ddf0b211a0 ("staging: fsl-dpaa2/eth: Flow affinity for
non-forwarded traffic").