From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by dpdk.org (Postfix) with ESMTP id 38C5623C for ; Fri, 4 May 2018 15:09:08 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.rdu2.redhat.com [10.11.54.4]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id C8F03FDC7F; Fri, 4 May 2018 13:09:07 +0000 (UTC) Received: from [10.36.112.52] (ovpn-112-52.ams2.redhat.com [10.36.112.52]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 1D349202342D; Fri, 4 May 2018 13:09:06 +0000 (UTC) To: "Loftus, Ciara" , "dev@dpdk.org" Cc: "Chen, Junjie J" References: <1524838798-5445-1-git-send-email-ciara.loftus@intel.com> <63cf2950-e7e6-deee-e84a-645cce2c4222@redhat.com> <74F120C019F4A64C9B78E802F6AD4CC278F13AE0@IRSMSX106.ger.corp.intel.com> <74F120C019F4A64C9B78E802F6AD4CC278F180BC@IRSMSX106.ger.corp.intel.com> From: Maxime Coquelin Message-ID: <374cfc3c-b01c-2f39-6560-3c9e15819eb9@redhat.com> Date: Fri, 4 May 2018 15:09:05 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: <74F120C019F4A64C9B78E802F6AD4CC278F180BC@IRSMSX106.ger.corp.intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.78 on 10.11.54.4 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.1]); Fri, 04 May 2018 13:09:07 +0000 (UTC) X-Greylist: inspected by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.1]); Fri, 04 May 2018 13:09:07 +0000 (UTC) for IP:'10.11.54.4' DOMAIN:'int-mx04.intmail.prod.int.rdu2.redhat.com' HELO:'smtp.corp.redhat.com' FROM:'maxime.coquelin@redhat.com' RCPT:'' Subject: Re: [dpdk-dev] [PATCH] net/vhost: Initialise vid to -1 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 04 May 2018 13:09:08 -0000 On 05/03/2018 03:18 PM, Loftus, Ciara wrote: >> >>> >>> On 04/27/2018 04:19 PM, Ciara Loftus wrote: >>>> rte_eth_vhost_get_vid_from_port_id returns a value of 0 if called before >>>> the first call to the new_device callback. A vid value >=0 suggests the >>>> device is active which is not the case in this instance. Initialise vid >>>> to a negative value to prevent this. >>>> >>>> Signed-off-by: Ciara Loftus >>>> --- >>>> drivers/net/vhost/rte_eth_vhost.c | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/drivers/net/vhost/rte_eth_vhost.c >>> b/drivers/net/vhost/rte_eth_vhost.c >>>> index 99a7727..f47950c 100644 >>>> --- a/drivers/net/vhost/rte_eth_vhost.c >>>> +++ b/drivers/net/vhost/rte_eth_vhost.c >>>> @@ -1051,6 +1051,7 @@ eth_rx_queue_setup(struct rte_eth_dev *dev, >>> uint16_t rx_queue_id, >>>> return -ENOMEM; >>>> } >>>> >>>> + vq->vid = -1; >>>> vq->mb_pool = mb_pool; >>>> vq->virtqueue_id = rx_queue_id * VIRTIO_QNUM + VIRTIO_TXQ; >>>> dev->data->rx_queues[rx_queue_id] = vq; >>>> >>> >>> Reviewed-by: Maxime Coquelin >>> >>> Thanks, >>> Maxime >> >> On second thoughts, self-NACK. >> >> We need to provision for the case where we want to call >> eth_rx_queue_setup AFTER new_device. For instance when we want to >> change the mb_pool. In this case we need to maintain the same vid and not >> reset it to -1. >> >> Without this patch the original problem still exists and need to find an >> alternative workaround. > > Junjie's patches fix the issue I was observing. Thanks Junjie! > https://dpdk.org/browse/dpdk/commit/?id=30a701a53737a0b6f7953412cc3b3d36c1d49122 > https://dpdk.org/browse/dpdk/commit/?id=e6722dee533cda3756fbc5c9ea4ddfbf30276f1b > > Along with the v2 of this patch could they be considered for the 17.11 stable branch? Yes, I think it is a good idea. It wasn't planned for -stable initially as it fixed a new use-case in v18.05. Junjie, can you please generate a backport against v17.11 and post it to stable@dpdk.org, adding "PATCH v17.11 LTS" as subject prefix, and using -x option of when cherry-picking so that it references the patches in master? Thanks in advance, Maxime > Thanks, > Ciara > >> >> Thanks, >> Ciara