Ping.
On Mon, Aug 22, 2022 at 15:42, Min Zhou wrote:
This patch fixes the following build failure seen on CentOS 8
with gcc 12.1 because of uninitialized struct variable:
[..]
../lib/vhost/virtio_net.c:1159:18: warning: 'buf_vec[0].buf_addr' may be used
uninitialized [-Wmaybe-uninitialized]
1159 | buf_addr = buf_vec[vec_idx].buf_addr;
| ~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
[..]
Fixes: 873e8dad6f49 ("vhost: support packed ring in async datapath")
Signed-off-by: Min Zhou <zhou...@loongson.cn>
---
lib/vhost/virtio_net.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/vhost/virtio_net.c b/lib/vhost/virtio_net.c
index 35fa4670fd..4878bb2d8a 100644
--- a/lib/vhost/virtio_net.c
+++ b/lib/vhost/virtio_net.c
@@ -1837,6 +1837,7 @@ virtio_dev_rx_async_packed(struct virtio_net *dev, struct
vhost_virtqueue *vq,
{
struct buf_vector buf_vec[BUF_VECTOR_MAX];
+ memset(buf_vec, 0, sizeof(buf_vec));
if (unlikely(vhost_enqueue_async_packed(dev, vq, pkt, buf_vec,
nr_descs, nr_buffers) < 0)) {
VHOST_LOG_DATA(dev->ifname, DEBUG, "failed to get enough desc from
vring\n");
--
Thanks,
Min Zhou