DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 1/6] config: enable lpm, port, table, pipeline, acl, sched libraries for ppc64le
@ 2016-07-07  2:56 Gowrishankar
  0 siblings, 0 replies; 5+ messages in thread
From: Gowrishankar @ 2016-07-07  2:56 UTC (permalink / raw)
  To: dev
  Cc: Chao Zhu, Bruce Richardson, Konstantin Ananyev,
	Cristian Dumitrescu, pradeep, gowrishankar

From: gowrishankar <gowrishankar.m@linux.vnet.ibm.com>

In this patch, DPDK libraries such as LPM, port, table, pipeline, ACL,
and sched are enabled for ppc64le.

Signed-off-by: Gowrishankar <gowrishankar.m@linux.vnet.ibm.com>
---
 config/defconfig_ppc_64-power8-linuxapp-gcc | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/config/defconfig_ppc_64-power8-linuxapp-gcc b/config/defconfig_ppc_64-power8-linuxapp-gcc
index 9eb0cc4..02d15bc 100644
--- a/config/defconfig_ppc_64-power8-linuxapp-gcc
+++ b/config/defconfig_ppc_64-power8-linuxapp-gcc
@@ -57,9 +57,9 @@ CONFIG_RTE_LIBRTE_ENIC_PMD=n
 CONFIG_RTE_LIBRTE_FM10K_PMD=n
 
 # This following libraries are not available on Power. So they're turned off.
-CONFIG_RTE_LIBRTE_LPM=n
-CONFIG_RTE_LIBRTE_ACL=n
-CONFIG_RTE_LIBRTE_SCHED=n
-CONFIG_RTE_LIBRTE_PORT=n
-CONFIG_RTE_LIBRTE_TABLE=n
-CONFIG_RTE_LIBRTE_PIPELINE=n
+CONFIG_RTE_LIBRTE_LPM=y
+CONFIG_RTE_LIBRTE_ACL=y
+CONFIG_RTE_LIBRTE_SCHED=y
+CONFIG_RTE_LIBRTE_PORT=y
+CONFIG_RTE_LIBRTE_TABLE=y
+CONFIG_RTE_LIBRTE_PIPELINE=y
-- 
1.9.1

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH 1/6] config: enable lpm, port, table, pipeline, acl, sched libraries for ppc64le
  2016-07-09 14:38     ` gowrishankar
@ 2016-07-09 20:53       ` Thomas Monjalon
  0 siblings, 0 replies; 5+ messages in thread
From: Thomas Monjalon @ 2016-07-09 20:53 UTC (permalink / raw)
  To: gowrishankar
  Cc: dev, Chao Zhu, Bruce Richardson, Konstantin Ananyev,
	Cristian Dumitrescu, pradeep

2016-07-09 20:08, gowrishankar:
> Hi Thomas,
> 
> On Friday 08 July 2016 06:27 PM, Thomas Monjalon wrote:
> > The comment and these lines can be removed and rely on default 
> > enabling in config/common_base. I suggest to enable (remove disabling) 
> > a feature at a time in the patch fixing the feature for PPC. 
> 
> Sure. So, do you suggest to send individual patchset (or patch) for each 
> feature being enabled, one by one ?.

No, my comment was about dropping the patch #1 and update following patches.
Example: in the patch fixing LPM, you can remove the line disabling LPM
in the config file.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH 1/6] config: enable lpm, port, table, pipeline, acl, sched libraries for ppc64le
  2016-07-08 12:57   ` Thomas Monjalon
@ 2016-07-09 14:38     ` gowrishankar
  2016-07-09 20:53       ` Thomas Monjalon
  0 siblings, 1 reply; 5+ messages in thread
From: gowrishankar @ 2016-07-09 14:38 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: dev, Chao Zhu, Bruce Richardson, Konstantin Ananyev,
	Cristian Dumitrescu, pradeep

Hi Thomas,

On Friday 08 July 2016 06:27 PM, Thomas Monjalon wrote:
> The comment and these lines can be removed and rely on default 
> enabling in config/common_base. I suggest to enable (remove disabling) 
> a feature at a time in the patch fixing the feature for PPC. 

Sure. So, do you suggest to send individual patchset (or patch) for each 
feature being enabled, one by one ?.

Thanks,
Gowrishankar

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH 1/6] config: enable lpm, port, table, pipeline, acl, sched libraries for ppc64le
  2016-07-07  2:57 ` [dpdk-dev] [PATCH 1/6] config: enable lpm, port, table, pipeline, acl, sched libraries for ppc64le Gowrishankar
@ 2016-07-08 12:57   ` Thomas Monjalon
  2016-07-09 14:38     ` gowrishankar
  0 siblings, 1 reply; 5+ messages in thread
From: Thomas Monjalon @ 2016-07-08 12:57 UTC (permalink / raw)
  To: Gowrishankar
  Cc: dev, Chao Zhu, Bruce Richardson, Konstantin Ananyev,
	Cristian Dumitrescu, pradeep

2016-07-07 08:27, Gowrishankar:
> --- a/config/defconfig_ppc_64-power8-linuxapp-gcc
> +++ b/config/defconfig_ppc_64-power8-linuxapp-gcc
> @@ -57,9 +57,9 @@ CONFIG_RTE_LIBRTE_ENIC_PMD=n
>  CONFIG_RTE_LIBRTE_FM10K_PMD=n
>  
>  # This following libraries are not available on Power. So they're turned off.
> -CONFIG_RTE_LIBRTE_LPM=n
> -CONFIG_RTE_LIBRTE_ACL=n
> -CONFIG_RTE_LIBRTE_SCHED=n
> -CONFIG_RTE_LIBRTE_PORT=n
> -CONFIG_RTE_LIBRTE_TABLE=n
> -CONFIG_RTE_LIBRTE_PIPELINE=n
> +CONFIG_RTE_LIBRTE_LPM=y
> +CONFIG_RTE_LIBRTE_ACL=y
> +CONFIG_RTE_LIBRTE_SCHED=y
> +CONFIG_RTE_LIBRTE_PORT=y
> +CONFIG_RTE_LIBRTE_TABLE=y
> +CONFIG_RTE_LIBRTE_PIPELINE=y

The comment and these lines can be removed and rely on default enabling
in config/common_base.

I suggest to enable (remove disabling) a feature at a time in the patch
fixing the feature for PPC.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [dpdk-dev] [PATCH 1/6] config: enable lpm, port, table, pipeline, acl, sched libraries for ppc64le
  2016-07-07  2:57 [dpdk-dev] [PATCH 0/6] enable lpm, acl and other missing libraries in ppc64le Gowrishankar
@ 2016-07-07  2:57 ` Gowrishankar
  2016-07-08 12:57   ` Thomas Monjalon
  0 siblings, 1 reply; 5+ messages in thread
From: Gowrishankar @ 2016-07-07  2:57 UTC (permalink / raw)
  To: dev
  Cc: Chao Zhu, Bruce Richardson, Konstantin Ananyev,
	Cristian Dumitrescu, pradeep, gowrishankar

From: gowrishankar <gowrishankar.m@linux.vnet.ibm.com>

In this patch, DPDK libraries such as LPM, port, table, pipeline, ACL,
and sched are enabled for ppc64le.

Signed-off-by: Gowrishankar <gowrishankar.m@linux.vnet.ibm.com>
---
 config/defconfig_ppc_64-power8-linuxapp-gcc | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/config/defconfig_ppc_64-power8-linuxapp-gcc b/config/defconfig_ppc_64-power8-linuxapp-gcc
index 9eb0cc4..02d15bc 100644
--- a/config/defconfig_ppc_64-power8-linuxapp-gcc
+++ b/config/defconfig_ppc_64-power8-linuxapp-gcc
@@ -57,9 +57,9 @@ CONFIG_RTE_LIBRTE_ENIC_PMD=n
 CONFIG_RTE_LIBRTE_FM10K_PMD=n
 
 # This following libraries are not available on Power. So they're turned off.
-CONFIG_RTE_LIBRTE_LPM=n
-CONFIG_RTE_LIBRTE_ACL=n
-CONFIG_RTE_LIBRTE_SCHED=n
-CONFIG_RTE_LIBRTE_PORT=n
-CONFIG_RTE_LIBRTE_TABLE=n
-CONFIG_RTE_LIBRTE_PIPELINE=n
+CONFIG_RTE_LIBRTE_LPM=y
+CONFIG_RTE_LIBRTE_ACL=y
+CONFIG_RTE_LIBRTE_SCHED=y
+CONFIG_RTE_LIBRTE_PORT=y
+CONFIG_RTE_LIBRTE_TABLE=y
+CONFIG_RTE_LIBRTE_PIPELINE=y
-- 
1.9.1

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-07-09 20:53 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-07  2:56 [dpdk-dev] [PATCH 1/6] config: enable lpm, port, table, pipeline, acl, sched libraries for ppc64le Gowrishankar
2016-07-07  2:57 [dpdk-dev] [PATCH 0/6] enable lpm, acl and other missing libraries in ppc64le Gowrishankar
2016-07-07  2:57 ` [dpdk-dev] [PATCH 1/6] config: enable lpm, port, table, pipeline, acl, sched libraries for ppc64le Gowrishankar
2016-07-08 12:57   ` Thomas Monjalon
2016-07-09 14:38     ` gowrishankar
2016-07-09 20:53       ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).