From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-yk0-f176.google.com (mail-yk0-f176.google.com [209.85.160.176]) by dpdk.org (Postfix) with ESMTP id BCB1D8E78 for ; Sat, 26 Sep 2015 13:35:10 +0200 (CEST) Received: by ykdz138 with SMTP id z138so136867835ykd.2 for ; Sat, 26 Sep 2015 04:35:10 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:subject:to:references:cc:from:message-id:date :user-agent:mime-version:in-reply-to:content-type :content-transfer-encoding; bh=c5pJ1B9J3s7MeAodA7a7iUQm14KdW1ohK9YPkTNqLbg=; b=HzHjliddRGTrOmtz5I/qCsJ69O6bSpVD2LXiFty7JjWeevhRRqlzXpNkLlG1nenyMG mFTGtVFmCC7ufInA8THilh+A15F5PKc+jNCggfSBNdOXxVlQRkLIT/XIQEEYmCjl2+uS 46tadtUciH4AtiSaZgCQrX9foWvyMgw2Svdi479IaqLDnzyaq/LvRtDsoaLYjZ+a+XDe LhKn38Yadcnw+3ooyL5sA5i6mhGIbuMapolF8el44i1jnFaZjxCVipxb1LSY5HItBoWP GupRyDI3YKOpzuCJq03ToQWV98Il+iOjpAwbPjEqUSuG4mdAXcwo2kKiYSUuYIrTAqwC a39Q== X-Gm-Message-State: ALoCoQk/57cHxghlRdp7JTBvPa3YYO8jyiXfZjwFoFe8mBrEQ+O+mYfz6nN915nuOqWRkZRr41ux X-Received: by 10.170.116.207 with SMTP id i198mr8812319ykb.128.1443267310133; Sat, 26 Sep 2015 04:35:10 -0700 (PDT) Received: from [10.11.3.147] ([216.53.198.14]) by smtp.googlemail.com with ESMTPSA id y187sm5941237ywe.23.2015.09.26.04.35.08 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 26 Sep 2015 04:35:09 -0700 (PDT) To: Ravi Kerur , David Marchand References: <1440013341-29659-1-git-send-email-rkerur@gmail.com> <1440013376-29715-1-git-send-email-rkerur@gmail.com> <55D53655.1040808@igel.co.jp> <55D69C00.2020609@igel.co.jp> From: Tetsuya Mukawa Message-ID: <560682EF.3030904@igel.co.jp> Date: Sat, 26 Sep 2015 20:35:11 +0900 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH v2] Change rte_eal_vdev_init to update port_id 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: Sat, 26 Sep 2015 11:35:11 -0000 Hi Ravi, I am out of office now, and will be back 29th. After backing, I will check it and send reply. Tetsuya On 2015/09/24 6:22, Ravi Kerur wrote: > Hi David, Tetsuya, > > I have sent V3 (changes isolated to rte_ether component) for formal review. > Please look into it and let me know your inputs. > > @David: I looked at "rte_eth_dev_get_port_by_name()", this function is > similar to "rte_eth_dev_get_name_by_port" and I have used same logic. Let > me know if this not correct I can fix both. > > Thanks, > Ravi > > > On Tue, Sep 15, 2015 at 4:28 AM, Ravi Kerur wrote: > >> Hi David, >> >> >> On Thu, Sep 3, 2015 at 7:04 AM, David Marchand >> wrote: >> >>> Hello Ravi, Tetsuya, >>> >>> On Tue, Aug 25, 2015 at 7:59 PM, Ravi Kerur wrote: >>> >>>> Let us know how you want us to fix this? To fix rte_eal_vdev_init and >>>> rte_eal_pci_probe_one to return allocated port_id we had 2 approaches >>>> mentioned in earlier discussion. In addition to those we have another >>>> approach with changes isolated only to rte_ether component. I am attaching >>>> diffs (preliminary) with this email. Please let us know your inputs since >>>> it involves EAL component. >>>> >>> - This patch looks like a good ethdev cleanup (even if it really lacks >>> some context / commit log). >>> >>> I wonder just why you only take the first part of the name in >>> rte_eth_dev_get_port_by_name(). >>> Would not this match, let's say, both toto and toto0 vdevs ? >>> Is this intended ? >>> >>> It was not intended, i will look into it. >>> - In the end, with this patch, do we still need to update eal ? >>> Looking at the code, I am not sure anymore. >>> >> Approach 3 (preliminary diffs sent as an attachment) doesn't involve EAL >> but the other two solutions do. So please let us know which one you prefer. >> I will send updated patch. >> >> Thanks, >> Ravi >> >> >>> >>> >>> -- >>> David Marchand >>> >>