On 5/31/23 11:37, Robin Jarry 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;
> | ^~~
>
> NB: this looks like a false positive, gcc 13 probably fails to see the link
> between reuse and dmp in dpdk_mp_get().
>
> Signed-off-by: Robin Jarry
> Reviewed-by: David Marchand
> ---
>
> Notes:
> v2: Fixed commit title and added note about possible false positive.
>
> lib/netdev-dpdk.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
> index fb0dd43f75c5..e11508cd5c31 100644
> --- a/lib/netdev-dpdk.c
> +++ b/lib/netdev-dpdk.c
> @@ -840,7 +840,7 @@ dpdk_mp_create(struct netdev_dpdk *dev, int mtu)
> static struct dpdk_mp *
> dpdk_mp_get(struct netdev_dpdk *dev, int mtu)
> {
> -struct dpdk_mp *dmp, *next;
> +struct dpdk_mp *dmp = NULL, *next;
> bool reuse = false;
>
> ovs_mutex_lock(&dpdk_mp_mutex);
Yeah, this definitely looks like a false-positive. But I can reproduce
the warning and this patch is fixing it. So, applied to branches down
to 2.17. Thanks!
Best regards, Ilya Maximets.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev