DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] RTE_PIPELINE_ACTION_PORT_META fix for non-default entries
@ 2015-02-21  8:31 Ildar Mustafin
  2015-02-24  1:01 ` Thomas Monjalon
  0 siblings, 1 reply; 4+ messages in thread
From: Ildar Mustafin @ 2015-02-21  8:31 UTC (permalink / raw)
  To: dev

Signed-off-by: Ildar Mustafin <imustafin@bk.ru>
---
 lib/librte_pipeline/rte_pipeline.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/lib/librte_pipeline/rte_pipeline.c b/lib/librte_pipeline/rte_pipeline.c
index ac7e887..36d92c9 100644
--- a/lib/librte_pipeline/rte_pipeline.c
+++ b/lib/librte_pipeline/rte_pipeline.c
@@ -999,6 +999,7 @@ rte_pipeline_compute_masks(struct rte_pipeline *p, uint64_t pkts_mask)
 {
 	p->action_mask1[RTE_PIPELINE_ACTION_DROP] = 0;
 	p->action_mask1[RTE_PIPELINE_ACTION_PORT] = 0;
+	p->action_mask1[RTE_PIPELINE_ACTION_PORT_META] = 0;
 	p->action_mask1[RTE_PIPELINE_ACTION_TABLE] = 0;
 
 	if ((pkts_mask & (pkts_mask + 1)) == 0) {
@@ -1224,6 +1225,7 @@ rte_pipeline_run(struct rte_pipeline *p)
 		pkts_mask = RTE_LEN2MASK(n_pkts, uint64_t);
 		p->action_mask0[RTE_PIPELINE_ACTION_DROP] = 0;
 		p->action_mask0[RTE_PIPELINE_ACTION_PORT] = 0;
+		p->action_mask0[RTE_PIPELINE_ACTION_PORT_META] = 0;
 		p->action_mask0[RTE_PIPELINE_ACTION_TABLE] = 0;
 
 		/* Input port user actions */
@@ -1300,6 +1302,9 @@ rte_pipeline_run(struct rte_pipeline *p)
 				p->action_mask0[RTE_PIPELINE_ACTION_PORT] |=
 					p->action_mask1[
 						RTE_PIPELINE_ACTION_PORT];
+				p->action_mask0[RTE_PIPELINE_ACTION_PORT_META] |=
+					p->action_mask1[
+						RTE_PIPELINE_ACTION_PORT_META];
 				p->action_mask0[RTE_PIPELINE_ACTION_TABLE] |=
 					p->action_mask1[
 						RTE_PIPELINE_ACTION_TABLE];
-- 
1.9.1

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

* Re: [dpdk-dev] [PATCH] RTE_PIPELINE_ACTION_PORT_META fix for non-default entries
  2015-02-21  8:31 [dpdk-dev] [PATCH] RTE_PIPELINE_ACTION_PORT_META fix for non-default entries Ildar Mustafin
@ 2015-02-24  1:01 ` Thomas Monjalon
  0 siblings, 0 replies; 4+ messages in thread
From: Thomas Monjalon @ 2015-02-24  1:01 UTC (permalink / raw)
  To: Ildar Mustafin; +Cc: dev

2015-02-21 11:31, Ildar Mustafin:
> Signed-off-by: Ildar Mustafin <imustafin@bk.ru>

An explanation is generally preferred.

Applied, thanks

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

* Re: [dpdk-dev] [PATCH] RTE_PIPELINE_ACTION_PORT_META fix for non-default entries
  2015-02-20 22:35 Ildar Mustafin
@ 2015-02-20 22:58 ` De Lara Guarch, Pablo
  0 siblings, 0 replies; 4+ messages in thread
From: De Lara Guarch, Pablo @ 2015-02-20 22:58 UTC (permalink / raw)
  To: Ildar Mustafin, dev



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ildar Mustafin
> Sent: Friday, February 20, 2015 10:35 PM
> To: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH] RTE_PIPELINE_ACTION_PORT_META fix for
> non-default entries
> 
> ---
>  lib/librte_pipeline/rte_pipeline.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/lib/librte_pipeline/rte_pipeline.c
> b/lib/librte_pipeline/rte_pipeline.c
> index ac7e887..36d92c9 100644
> --- a/lib/librte_pipeline/rte_pipeline.c
> +++ b/lib/librte_pipeline/rte_pipeline.c
> @@ -999,6 +999,7 @@ rte_pipeline_compute_masks(struct rte_pipeline *p,
> uint64_t pkts_mask)
>  {
>  	p->action_mask1[RTE_PIPELINE_ACTION_DROP] = 0;
>  	p->action_mask1[RTE_PIPELINE_ACTION_PORT] = 0;
> +	p->action_mask1[RTE_PIPELINE_ACTION_PORT_META] = 0;
>  	p->action_mask1[RTE_PIPELINE_ACTION_TABLE] = 0;
> 
>  	if ((pkts_mask & (pkts_mask + 1)) == 0) {
> @@ -1224,6 +1225,7 @@ rte_pipeline_run(struct rte_pipeline *p)
>  		pkts_mask = RTE_LEN2MASK(n_pkts, uint64_t);
>  		p->action_mask0[RTE_PIPELINE_ACTION_DROP] = 0;
>  		p->action_mask0[RTE_PIPELINE_ACTION_PORT] = 0;
> +		p->action_mask0[RTE_PIPELINE_ACTION_PORT_META] = 0;
>  		p->action_mask0[RTE_PIPELINE_ACTION_TABLE] = 0;
> 
>  		/* Input port user actions */
> @@ -1300,6 +1302,9 @@ rte_pipeline_run(struct rte_pipeline *p)
>  				p-
> >action_mask0[RTE_PIPELINE_ACTION_PORT] |=
>  					p->action_mask1[
> 
> 	RTE_PIPELINE_ACTION_PORT];
> +				p-
> >action_mask0[RTE_PIPELINE_ACTION_PORT_META] |=
> +					p->action_mask1[
> +
> 	RTE_PIPELINE_ACTION_PORT_META];
>  				p-
> >action_mask0[RTE_PIPELINE_ACTION_TABLE] |=
>  					p->action_mask1[
> 
> 	RTE_PIPELINE_ACTION_TABLE];
> --
> 1.9.1

Hi Ildar,

You forgot the signature. Use git commit --signoff, considering that your user name is set.

Thanks,
Pablo

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

* [dpdk-dev] [PATCH] RTE_PIPELINE_ACTION_PORT_META fix for non-default entries
@ 2015-02-20 22:35 Ildar Mustafin
  2015-02-20 22:58 ` De Lara Guarch, Pablo
  0 siblings, 1 reply; 4+ messages in thread
From: Ildar Mustafin @ 2015-02-20 22:35 UTC (permalink / raw)
  To: dev

---
 lib/librte_pipeline/rte_pipeline.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/lib/librte_pipeline/rte_pipeline.c b/lib/librte_pipeline/rte_pipeline.c
index ac7e887..36d92c9 100644
--- a/lib/librte_pipeline/rte_pipeline.c
+++ b/lib/librte_pipeline/rte_pipeline.c
@@ -999,6 +999,7 @@ rte_pipeline_compute_masks(struct rte_pipeline *p, uint64_t pkts_mask)
 {
 	p->action_mask1[RTE_PIPELINE_ACTION_DROP] = 0;
 	p->action_mask1[RTE_PIPELINE_ACTION_PORT] = 0;
+	p->action_mask1[RTE_PIPELINE_ACTION_PORT_META] = 0;
 	p->action_mask1[RTE_PIPELINE_ACTION_TABLE] = 0;
 
 	if ((pkts_mask & (pkts_mask + 1)) == 0) {
@@ -1224,6 +1225,7 @@ rte_pipeline_run(struct rte_pipeline *p)
 		pkts_mask = RTE_LEN2MASK(n_pkts, uint64_t);
 		p->action_mask0[RTE_PIPELINE_ACTION_DROP] = 0;
 		p->action_mask0[RTE_PIPELINE_ACTION_PORT] = 0;
+		p->action_mask0[RTE_PIPELINE_ACTION_PORT_META] = 0;
 		p->action_mask0[RTE_PIPELINE_ACTION_TABLE] = 0;
 
 		/* Input port user actions */
@@ -1300,6 +1302,9 @@ rte_pipeline_run(struct rte_pipeline *p)
 				p->action_mask0[RTE_PIPELINE_ACTION_PORT] |=
 					p->action_mask1[
 						RTE_PIPELINE_ACTION_PORT];
+				p->action_mask0[RTE_PIPELINE_ACTION_PORT_META] |=
+					p->action_mask1[
+						RTE_PIPELINE_ACTION_PORT_META];
 				p->action_mask0[RTE_PIPELINE_ACTION_TABLE] |=
 					p->action_mask1[
 						RTE_PIPELINE_ACTION_TABLE];
-- 
1.9.1

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

end of thread, other threads:[~2015-02-24  1:01 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-21  8:31 [dpdk-dev] [PATCH] RTE_PIPELINE_ACTION_PORT_META fix for non-default entries Ildar Mustafin
2015-02-24  1:01 ` Thomas Monjalon
  -- strict thread matches above, loose matches on Subject: below --
2015-02-20 22:35 Ildar Mustafin
2015-02-20 22:58 ` De Lara Guarch, Pablo

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).