[dpdk-dev] [PATCH] ip_pipeline: fix parsing error in TM port section
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jasvinder Singh
> Sent: Thursday, June 16, 2016 10:05 AM
> To: dev at dpdk.org
> Cc: Dumitrescu, Cristian
> Subject: [dpdk-dev] [PATCH] ip_pipeline: fix parsing error in TM port section
>
> Replace APP_PARAM_ADD_LINK_FOR_TXQ with
> APP_PARAM_ADD_LINK_FOR_TM macro in TM (Traffic Manager) port
> section parsing function. This macro adds nic ports entry specified in TM port
> section of the application configuration file to the application parameters
> structure.
>
> Fixes: e5a1cd8a4847 ("examples/ip_pipeline: clean up configuration parser")
>
> Signed-off-by: Jasvinder Singh
> Acked-by: Cristian Dumitrescu
> ---
> examples/ip_pipeline/config_parse.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/examples/ip_pipeline/config_parse.c
> b/examples/ip_pipeline/config_parse.c
> index 504018e..5e70d78 100644
Self-NACK ; fix applied by another patch.
[dpdk-dev] [PATCH] ip_pipeline: fix parsing error in TM port section
> -Original Message-
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Tuesday, June 21, 2016 9:49 PM
> To: Dumitrescu, Cristian
> Cc: dev at dpdk.org; Singh, Jasvinder
> Subject: Re: [dpdk-dev] [PATCH] ip_pipeline: fix parsing error in TM port
> section
>
> 2016-06-16 10:04, Jasvinder Singh:
> > Replace APP_PARAM_ADD_LINK_FOR_TXQ with
> APP_PARAM_ADD_LINK_FOR_TM macro
> > in TM (Traffic Manager) port section parsing function. This macro adds
> > nic ports entry specified in TM port section of the application
> > configuration file to the application parameters structure.
> >
> > Fixes: e5a1cd8a4847 ("examples/ip_pipeline: clean up configuration
> parser")
> >
> > Signed-off-by: Jasvinder Singh
> > Acked-by: Cristian Dumitrescu
>
> Already fixed in 81d084dd2aa8 ("examples/ip_pipeline: support KNI")
>
> Cristian, you should not ack a patch which do more than announced in
> the title. This pipeline KNI patch do a lot of things without any
> explanation for the fixes.
> 1 explanation = 1 patch
Yeah, there were several small issues to fix and several people spotted them at
the same time. I agree we should do a better job at separating different fixes
into different patches. Sorry about this.
[dpdk-dev] [PATCH] ip_pipeline: fix parsing error in TM port section
2016-06-16 10:04, Jasvinder Singh:
> Replace APP_PARAM_ADD_LINK_FOR_TXQ with APP_PARAM_ADD_LINK_FOR_TM macro
> in TM (Traffic Manager) port section parsing function. This macro adds
> nic ports entry specified in TM port section of the application
> configuration file to the application parameters structure.
>
> Fixes: e5a1cd8a4847 ("examples/ip_pipeline: clean up configuration parser")
>
> Signed-off-by: Jasvinder Singh
> Acked-by: Cristian Dumitrescu
Already fixed in 81d084dd2aa8 ("examples/ip_pipeline: support KNI")
Cristian, you should not ack a patch which do more than announced in
the title. This pipeline KNI patch do a lot of things without any
explanation for the fixes.
1 explanation = 1 patch
[dpdk-dev] [PATCH] ip_pipeline: fix parsing error in TM port section
Replace APP_PARAM_ADD_LINK_FOR_TXQ with APP_PARAM_ADD_LINK_FOR_TM macro
in TM (Traffic Manager) port section parsing function. This macro adds
nic ports entry specified in TM port section of the application
configuration file to the application parameters structure.
Fixes: e5a1cd8a4847 ("examples/ip_pipeline: clean up configuration parser")
Signed-off-by: Jasvinder Singh
Acked-by: Cristian Dumitrescu
---
examples/ip_pipeline/config_parse.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/examples/ip_pipeline/config_parse.c
b/examples/ip_pipeline/config_parse.c
index 504018e..5e70d78 100644
--- a/examples/ip_pipeline/config_parse.c
+++ b/examples/ip_pipeline/config_parse.c
@@ -1816,7 +1816,7 @@ parse_tm(struct app_params *app,
param = >tm_params[param_idx];
PARSE_CHECK_DUPLICATE_SECTION(param);
- APP_PARAM_ADD_LINK_FOR_TXQ(app, section_name);
+ APP_PARAM_ADD_LINK_FOR_TM(app, section_name);
for (i = 0; i < n_entries; i++) {
struct rte_cfgfile_entry *ent = [i];
--
2.5.5
