From 12f622b769bb34f8dba78c4712220828a3cce029 Mon Sep 17 00:00:00 2001 From: Darek Stojaczyk Date: Fri, 19 Apr 2019 03:40:39 +0200 Subject: [PATCH] virtio: zero-out virtqueue structures at init They used to be allocated with spdk_dma_zmalloc() which did provide zeroed memory, but we unintentionally changed that when switching to posix_memalign. The structure might have some unitialized memory, so with this patch we just memset it right after allocating it. Change-Id: Id5a5685e09419901513925abaeed605c36f5199a Signed-off-by: Darek Stojaczyk Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/451546 Tested-by: SPDK CI Jenkins Reviewed-by: Jim Harris Reviewed-by: Ben Walker --- lib/virtio/virtio.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/virtio/virtio.c b/lib/virtio/virtio.c index 1d01e719f..997de52f0 100644 --- a/lib/virtio/virtio.c +++ b/lib/virtio/virtio.c @@ -145,6 +145,7 @@ virtio_init_queue(struct virtio_dev *dev, uint16_t vtpci_queue_idx) SPDK_ERRLOG("can not allocate vq\n"); return -ENOMEM; } + memset(vq, 0, size); dev->vqs[vtpci_queue_idx] = vq; vq->vdev = dev;