From patchwork Thu Nov 12 04:10:41 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yuanhan Liu X-Patchwork-Id: 8870 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id C29848E9F; Thu, 12 Nov 2015 05:06:43 +0100 (CET) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 5D6228D39 for ; Thu, 12 Nov 2015 05:06:41 +0100 (CET) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga102.jf.intel.com with ESMTP; 11 Nov 2015 20:06:40 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,280,1444719600"; d="scan'208";a="849088226" Received: from yliu-dev.sh.intel.com ([10.239.66.49]) by fmsmga002.fm.intel.com with ESMTP; 11 Nov 2015 20:06:39 -0800 From: Yuanhan Liu To: dev@dpdk.org Date: Thu, 12 Nov 2015 12:10:41 +0800 Message-Id: <1447301441-10590-2-git-send-email-yuanhan.liu@linux.intel.com> X-Mailer: git-send-email 1.9.0 In-Reply-To: <1447301441-10590-1-git-send-email-yuanhan.liu@linux.intel.com> References: <1447301441-10590-1-git-send-email-yuanhan.liu@linux.intel.com> Subject: [dpdk-dev] [PATCH] vhost: reset device properly X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Currently, we reset all fields of a device to zero when reset happens, which is wrong, since for some fields like device_fh, ifname, and virt_qp_nb, they should be same and be kept after reset until the device is removed. And this is what's the new helper function reset_device() for. And use rte_zmalloc() instead of rte_malloc, so that we could avoid init_device(), which basically dose zero reset only so far. Hence, init_device() is dropped in this patch. This patch also removes a hack of using the offset a specific field (which is virtqueue now) inside of `virtio_net' structure to do reset, which could be broken easily if someone changed the field order without caution. Cc: Tetsuya Mukawa Cc: Xie Huawei Signed-off-by: Yuanhan Liu Acked-by: Rich Lane --- This patch is based on: http://dpdk.org/dev/patchwork/patch/8818/ --- lib/librte_vhost/virtio-net.c | 27 ++++++++++----------------- 1 file changed, 10 insertions(+), 17 deletions(-) diff --git a/lib/librte_vhost/virtio-net.c b/lib/librte_vhost/virtio-net.c index 39a6a5e..cc917da 100644 --- a/lib/librte_vhost/virtio-net.c +++ b/lib/librte_vhost/virtio-net.c @@ -204,6 +204,7 @@ cleanup_device(struct virtio_net *dev) munmap((void *)(uintptr_t)dev->mem->mapped_address, (size_t)dev->mem->mapped_size); free(dev->mem); + dev->mem = NULL; } for (i = 0; i < dev->virt_qp_nb; i++) { @@ -306,20 +307,18 @@ alloc_vring_queue_pair(struct virtio_net *dev, uint32_t qp_idx) } /* - * Initialise all variables in device structure. + * Reset some variables in device structure, while keeping few + * others untouched, such as device_fh, ifname, virt_qp_nb: they + * should be same unless the device is removed. */ static void -init_device(struct virtio_net *dev) +reset_device(struct virtio_net *dev) { - int vq_offset; uint32_t i; - /* - * Virtqueues have already been malloced so - * we don't want to set them to NULL. - */ - vq_offset = offsetof(struct virtio_net, virtqueue); - memset(dev, 0, vq_offset); + dev->features = 0; + dev->protocol_features = 0; + dev->flags = 0; for (i = 0; i < dev->virt_qp_nb; i++) init_vring_queue_pair(dev, i); @@ -336,7 +335,7 @@ new_device(struct vhost_device_ctx ctx) struct virtio_net_config_ll *new_ll_dev; /* Setup device and virtqueues. */ - new_ll_dev = rte_malloc(NULL, sizeof(struct virtio_net_config_ll), 0); + new_ll_dev = rte_zmalloc(NULL, sizeof(struct virtio_net_config_ll), 0); if (new_ll_dev == NULL) { RTE_LOG(ERR, VHOST_CONFIG, "(%"PRIu64") Failed to allocate memory for dev.\n", @@ -344,9 +343,6 @@ new_device(struct vhost_device_ctx ctx) return -1; } - /* Initialise device and virtqueues. */ - init_device(&new_ll_dev->dev); - new_ll_dev->next = NULL; /* Add entry to device configuration linked list. */ @@ -430,7 +426,6 @@ static int reset_owner(struct vhost_device_ctx ctx) { struct virtio_net *dev; - uint64_t device_fh; dev = get_device(ctx); if (dev == NULL) @@ -439,10 +434,8 @@ reset_owner(struct vhost_device_ctx ctx) if (dev->flags & VIRTIO_DEV_RUNNING) notify_ops->destroy_device(dev); - device_fh = dev->device_fh; cleanup_device(dev); - init_device(dev); - dev->device_fh = device_fh; + reset_device(dev); return 0; }