In OVS, commit titles start with a capital letter and ends with a .
I suggest:
netdev-dpdk: Fix build with gcc 13.


On Tue, May 30, 2023 at 4:23 PM Robin Jarry <rja...@redhat.com> wrote:
>
> GCC now reports uninitialized warnings from function return values.
>
> ../lib/netdev-dpdk.c: In function 'netdev_dpdk_mempool_configure':
> ../lib/netdev-dpdk.c:964:22: warning: 'dmp' may be used uninitialized 
> [-Wmaybe-uninitialized]
>   964 |         dev->dpdk_mp = dmp;
>       |         ~~~~~~~~~~~~~^~~~~
> ../lib/netdev-dpdk.c:854:21: note: 'dmp' was declared here
>   854 |     struct dpdk_mp *dmp, *next;
>       |                     ^~~
>
> Signed-off-by: Robin Jarry <rja...@redhat.com>

This looks like a false positive to me..
gcc 13 probably fails to see the link between reuse and dmp.

I don't have a better idea to make gcc happy and your proposal is
simple so I am ok with it.
If you send a v2, feel free to add:
Reviewed-by: David Marchand <david.march...@redhat.com>


-- 
David Marchand

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to