From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from alln-iport-6.cisco.com (alln-iport-6.cisco.com [173.37.142.93]) by dpdk.org (Postfix) with ESMTP id 8751E2BD3 for ; Tue, 26 Mar 2019 07:39:21 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=2000; q=dns/txt; s=iport; t=1553582361; x=1554791961; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=dvjkj5V4XOo14hT8uBskg3oOOfH6DC2XsFzPDP0gJk4=; b=OKmAc7aEU0s8/1ZQbodwunPum1djJaYwIebncyuodn+7cR2jyliqMME+ SWT6qkju5xtMRdkyQQdj8UyIKOk5EVtoC8DNsE3qWJgezlyDb4U5tnQAp kj+gL/Yt8jubUSLq3OJFVCIPWWcaS8nVfLjWjP3636xZ4KFPSKx8q42hl 4=; X-IronPort-AV: E=Sophos;i="5.60,271,1549929600"; d="scan'208";a="251774138" Received: from alln-core-6.cisco.com ([173.36.13.139]) by alln-iport-6.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 26 Mar 2019 06:39:20 +0000 Received: from HYONKIM-7R0DR.cisco.com ([10.24.34.232]) by alln-core-6.cisco.com (8.15.2/8.15.2) with ESMTPS id x2Q6dEbn005010 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Tue, 26 Mar 2019 06:39:17 GMT Date: Tue, 26 Mar 2019 15:39:14 +0900 From: Hyong Youb Kim To: =?iso-8859-1?Q?Ga=EBtan?= Rivet Cc: Ferruh Yigit , Thomas Monjalon , "John Daley (johndale)" , Andrew Rybchenko , Qi Zhang , "dev@dpdk.org" , Shahaf Shuler , Jerin Jacob , David Marchand , Maxime Coquelin , Konstantin Ananyev , Hemant Agrawal , Stephen Hemminger Message-ID: <20190326063912.GA24848@HYONKIM-7R0DR.cisco.com> References: <20190305055659.3095-1-hyonkim@cisco.com> <3429576.gmczXWLcQl@xps> <09e2672e-c94f-5f7d-84e3-343a7be7452e@intel.com> <3306757.Lg3ZuM36Da@xps> <698fd89f-10c1-41a3-2b60-6a77783716e8@intel.com> <20190325132627.hn64wxypxzzb4hz7@bidouze.vm.6wind.com> <20190325133341.wqzl4jhoaopiskgk@bidouze.vm.6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190325133341.wqzl4jhoaopiskgk@bidouze.vm.6wind.com> User-Agent: Mutt/1.10.0 (2018-05-17) X-Outbound-SMTP-Client: 10.24.34.232, [10.24.34.232] X-Outbound-Node: alln-core-6.cisco.com Subject: Re: [dpdk-dev] [PATCH v2] net/enic: add private API to set ingress VLAN rewrite mode 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: Tue, 26 Mar 2019 06:39:22 -0000 [...] > > > I see it can work if an application always wants this config option to have > > > *same* value. So it can set this in eal_init() always. > > > > > > This requires "driver=xxx,key=value" kind of support in devargs. > > > > > > > > > John, Hyong, > > > > > > I guess some level of devargs support is already there, Thomas/Gaetan can > > > provide more information on latest status of it, can it be possible to get some > > > support from you to finalize this effort? > > > > > > And when it is ready enic can benefit from it for this usecase. > > > > > > Thanks, > > > ferruh > > > > Hi Thomas, Ferruh, John, Hyong, > > > > driver=xxx,key=value could work, as it is not dependent on the > > devargs framework, only on the driver implementation. Nothing specific > > should be needed from EAL PoV (regarding this feature only). What will > > be missing is the new devargs support in general. > > Sorry I spoke too quickly, specific development and some passing of > arguments would be needed. > > > > > Regarding the new devargs: this dev was stalled 2 versions ago at the > > --dev inclusion step. This parameter was necessary for PMD maintainers > > to be able to use the new init path with their drivers and transition to > > rte_eth_devargs_parse() for devargs parsing. > > > > --dev was proposed, but its patch was not kept by Thomas during the > > final crunch. I probably did not shout loud enough about it and let it > > go, but I think this was a mistake: this feature was low-risk and central > > in the transition process (it was in parallel to -w/b and --vdev). Hi All, Thanks for tracking this issue. Yes, we will work on "driver=x,k=v" and try to submit patches early in the 19.08 cycle. Cannot keep free-riding :-) As for vpp, it can have its own local DPDK patches. So, we added an enic patch to change the default rewrite mode to 'untag', which has the same effect as 'driver=enic,rewrite=untag'. -Hyong From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id 5D4A7A05D3 for ; Tue, 26 Mar 2019 07:39:24 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 3A8192C19; Tue, 26 Mar 2019 07:39:23 +0100 (CET) Received: from alln-iport-6.cisco.com (alln-iport-6.cisco.com [173.37.142.93]) by dpdk.org (Postfix) with ESMTP id 8751E2BD3 for ; Tue, 26 Mar 2019 07:39:21 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=2000; q=dns/txt; s=iport; t=1553582361; x=1554791961; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=dvjkj5V4XOo14hT8uBskg3oOOfH6DC2XsFzPDP0gJk4=; b=OKmAc7aEU0s8/1ZQbodwunPum1djJaYwIebncyuodn+7cR2jyliqMME+ SWT6qkju5xtMRdkyQQdj8UyIKOk5EVtoC8DNsE3qWJgezlyDb4U5tnQAp kj+gL/Yt8jubUSLq3OJFVCIPWWcaS8nVfLjWjP3636xZ4KFPSKx8q42hl 4=; X-IronPort-AV: E=Sophos;i="5.60,271,1549929600"; d="scan'208";a="251774138" Received: from alln-core-6.cisco.com ([173.36.13.139]) by alln-iport-6.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 26 Mar 2019 06:39:20 +0000 Received: from HYONKIM-7R0DR.cisco.com ([10.24.34.232]) by alln-core-6.cisco.com (8.15.2/8.15.2) with ESMTPS id x2Q6dEbn005010 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Tue, 26 Mar 2019 06:39:17 GMT Date: Tue, 26 Mar 2019 15:39:14 +0900 From: Hyong Youb Kim To: =?iso-8859-1?Q?Ga=EBtan?= Rivet Cc: Ferruh Yigit , Thomas Monjalon , "John Daley (johndale)" , Andrew Rybchenko , Qi Zhang , "dev@dpdk.org" , Shahaf Shuler , Jerin Jacob , David Marchand , Maxime Coquelin , Konstantin Ananyev , Hemant Agrawal , Stephen Hemminger Message-ID: <20190326063912.GA24848@HYONKIM-7R0DR.cisco.com> References: <20190305055659.3095-1-hyonkim@cisco.com> <3429576.gmczXWLcQl@xps> <09e2672e-c94f-5f7d-84e3-343a7be7452e@intel.com> <3306757.Lg3ZuM36Da@xps> <698fd89f-10c1-41a3-2b60-6a77783716e8@intel.com> <20190325132627.hn64wxypxzzb4hz7@bidouze.vm.6wind.com> <20190325133341.wqzl4jhoaopiskgk@bidouze.vm.6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Disposition: inline In-Reply-To: <20190325133341.wqzl4jhoaopiskgk@bidouze.vm.6wind.com> User-Agent: Mutt/1.10.0 (2018-05-17) X-Outbound-SMTP-Client: 10.24.34.232, [10.24.34.232] X-Outbound-Node: alln-core-6.cisco.com Subject: Re: [dpdk-dev] [PATCH v2] net/enic: add private API to set ingress VLAN rewrite mode 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Message-ID: <20190326063914.LBnBQ-qeO75GWnVtK0JT00l-YCJU6OoOZwJZfvI8xMs@z> [...] > > > I see it can work if an application always wants this config option to have > > > *same* value. So it can set this in eal_init() always. > > > > > > This requires "driver=xxx,key=value" kind of support in devargs. > > > > > > > > > John, Hyong, > > > > > > I guess some level of devargs support is already there, Thomas/Gaetan can > > > provide more information on latest status of it, can it be possible to get some > > > support from you to finalize this effort? > > > > > > And when it is ready enic can benefit from it for this usecase. > > > > > > Thanks, > > > ferruh > > > > Hi Thomas, Ferruh, John, Hyong, > > > > driver=xxx,key=value could work, as it is not dependent on the > > devargs framework, only on the driver implementation. Nothing specific > > should be needed from EAL PoV (regarding this feature only). What will > > be missing is the new devargs support in general. > > Sorry I spoke too quickly, specific development and some passing of > arguments would be needed. > > > > > Regarding the new devargs: this dev was stalled 2 versions ago at the > > --dev inclusion step. This parameter was necessary for PMD maintainers > > to be able to use the new init path with their drivers and transition to > > rte_eth_devargs_parse() for devargs parsing. > > > > --dev was proposed, but its patch was not kept by Thomas during the > > final crunch. I probably did not shout loud enough about it and let it > > go, but I think this was a mistake: this feature was low-risk and central > > in the transition process (it was in parallel to -w/b and --vdev). Hi All, Thanks for tracking this issue. Yes, we will work on "driver=x,k=v" and try to submit patches early in the 19.08 cycle. Cannot keep free-riding :-) As for vpp, it can have its own local DPDK patches. So, we added an enic patch to change the default rewrite mode to 'untag', which has the same effect as 'driver=enic,rewrite=untag'. -Hyong