From patchwork Wed Apr 27 08:53:58 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Huawei Xie X-Patchwork-Id: 12278 X-Patchwork-Delegate: yuanhan.liu@linux.intel.com 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 BB4FB37B4; Wed, 27 Apr 2016 16:58:48 +0200 (CEST) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 5C68937B4 for ; Wed, 27 Apr 2016 16:58:47 +0200 (CEST) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga104.fm.intel.com with ESMTP; 27 Apr 2016 07:58:34 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,542,1455004800"; d="scan'208";a="92960347" Received: from ovsfrank.sh.intel.com ([10.239.129.132]) by fmsmga004.fm.intel.com with ESMTP; 27 Apr 2016 07:58:33 -0700 From: Huawei Xie To: dev@dpdk.org Cc: stephen@networkplumber.org, mukawa@igel.co.jp, kevin.traynor@intel.com, jianfeng.tan@intel.com, yuanhan.liu@linux.intel.com, Huawei Xie Date: Wed, 27 Apr 2016 04:53:58 -0400 Message-Id: <1461747238-124446-1-git-send-email-huawei.xie@intel.com> X-Mailer: git-send-email 2.4.11 In-Reply-To: <1461259092-9309-1-git-send-email-huawei.xie@intel.com> References: <1461259092-9309-1-git-send-email-huawei.xie@intel.com> Subject: [dpdk-dev] [PATCH] virtio: avoid avail ring entry index update if equal 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" Avail ring is updated by the frontend and consumed by the backend. There are frequent core to core cache transfers for the avail ring. This optmization avoids avail ring entry index update if the entry already holds the same value. As DPDK virtio PMD implements FIFO free descriptor list (also for performance reason of CACHE), in which descriptors are allocated from the head and freed to the tail, with this patch in most cases avail ring will remain the same, then it would be valid in both caches of frontend and backend. Signed-off-by: Huawei Xie Suggested-by: ms >> Michael S. Tsirkin Acked-by: Yuanhan Liu --- drivers/net/virtio/virtqueue.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/virtio/virtqueue.h b/drivers/net/virtio/virtqueue.h index 4e9239e..8c46a83 100644 --- a/drivers/net/virtio/virtqueue.h +++ b/drivers/net/virtio/virtqueue.h @@ -302,7 +302,8 @@ vq_update_avail_ring(struct virtqueue *vq, uint16_t desc_idx) * descriptor. */ avail_idx = (uint16_t)(vq->vq_avail_idx & (vq->vq_nentries - 1)); - vq->vq_ring.avail->ring[avail_idx] = desc_idx; + if (unlikely(vq->vq_ring.avail->ring[avail_idx] != desc_idx)) + vq->vq_ring.avail->ring[avail_idx] = desc_idx; vq->vq_avail_idx++; }