From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f42.google.com (mail-pa0-f42.google.com [209.85.220.42]) by dpdk.org (Postfix) with ESMTP id EBC50CF9 for ; Thu, 9 Jul 2015 04:04:44 +0200 (CEST) Received: by pactm7 with SMTP id tm7so141389653pac.2 for ; Wed, 08 Jul 2015 19:04:44 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:message-id:date:from:user-agent:mime-version:to :subject:references:in-reply-to:content-type :content-transfer-encoding; bh=wMJP3jv5XDWjYjy1RpYomPiFjvJ6nhyXos9nklKAjJQ=; b=QWGWSU6Ub0N4VgZ5nn75heJGItOeFpkzeBvQdqrnmZNTW5LShptjM0j8C9SnIo0DAs Dtz2YHqpS7P5IxoydpWyqRQFbDzvp/UX/WjLuTazZNw4toYn2ZmNmZnTHWo+9HvX+Glr CFjbF/p0WgCG7PhHKvCrrKN88Li61Kgm7TaU0P8NQfVn2amMaD/FVlqdfpJSiWIUAPU4 o/DhDJavIHCi9R9g7bsXsEEecRgST2GEXCAKWBi6WD0JIC3Hfh+a3Ct7QA6vxbFiAksz IELlrk5piKMa6LefPreCSdSSYPCrEdBN6NQkMSQO9P+Q8b/92gKAl5pqVi+GNS9FEA1+ h2GA== X-Gm-Message-State: ALoCoQkBZtL/r2tDTgrr8i9J5QJX5YRfibuRlWoKC7z5ziiK+rs7RFxJPyQpafq9h6GN0VelggQ9 X-Received: by 10.68.237.105 with SMTP id vb9mr25698091pbc.149.1436407484245; Wed, 08 Jul 2015 19:04:44 -0700 (PDT) Received: from [10.16.129.101] (napt.igel.co.jp. [219.106.231.132]) by smtp.googlemail.com with ESMTPSA id xs13sm3933071pac.3.2015.07.08.19.04.42 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 08 Jul 2015 19:04:43 -0700 (PDT) Message-ID: <559DD6B9.9050909@igel.co.jp> Date: Thu, 09 Jul 2015 11:04:41 +0900 From: Tetsuya Mukawa User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: "Ouyang, Changchun" , "Iremonger, Bernard" , "dev@dpdk.org" References: <1436260687-28549-1-git-send-email-bernard.iremonger@intel.com> <1436260687-28549-5-git-send-email-bernard.iremonger@intel.com> In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v4 4/4] test-pmd: remove call to rte_eth_promiscuous_disable() from detach_port() 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: , X-List-Received-Date: Thu, 09 Jul 2015 02:04:45 -0000 On 2015/07/08 9:42, Ouyang, Changchun wrote: >> -----Original Message----- >> From: Iremonger, Bernard >> Sent: Tuesday, July 7, 2015 5:18 PM >> To: dev@dpdk.org >> Cc: Ouyang, Changchun; Iremonger, Bernard >> Subject: [PATCH v4 4/4] test-pmd: remove call to >> rte_eth_promiscuous_disable() from detach_port() >> >> At this point the stop() and close() functions have already been called. >> The rte_eth_promiscuous_disable() function does not return on the VM. > I think we need root the cause why it doesn't return on the VM. It seems this patch is correct. The root cause is that when close() is called, virtio-net device will be reset by vtpci_reset(), and at this point all virtqueues will be invalid in virtio-net device. After resetting device, we should not call rte_eth_promiscuous_disable(), because it will use an one of virtqueues. Anyway, promiscuous configuration will be initialized when the device is reset. Regards, Tetsuya > >> Signed-off-by: Bernard Iremonger >> --- >> app/test-pmd/testpmd.c | 4 +--- >> 1 files changed, 1 insertions(+), 3 deletions(-) >> >> diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index >> 82b465d..4769533 100644 >> --- a/app/test-pmd/testpmd.c >> +++ b/app/test-pmd/testpmd.c >> @@ -1,7 +1,7 @@ >> /*- >> * BSD LICENSE >> * >> - * Copyright(c) 2010-2014 Intel Corporation. All rights reserved. >> + * Copyright(c) 2010-2015 Intel Corporation. All rights reserved. >> * All rights reserved. >> * >> * Redistribution and use in source and binary forms, with or without >> @@ -1542,8 +1542,6 @@ detach_port(uint8_t port_id) >> return; >> } >> >> - rte_eth_promiscuous_disable(port_id); >> - > It seems a workaround. > >> if (rte_eth_dev_detach(port_id, name)) >> return; >> >> -- >> 1.7.4.1