| Message ID | 1444823465-21264-3-git-send-email-bill.fischofer@linaro.org |
|---|---|
| State | Accepted |
| Commit | fe3764711bf126facac764af40d37953f34f36ca |
| Headers | show |
I think it's better to have single patch from patches 3,4 and 5, since they do the same thing. Maxim. On 10/14/2015 14:51, Bill Fischofer wrote: > Signed-off-by: Bill Fischofer <bill.fischofer@linaro.org> > --- > example/classifier/odp_classifier.c | 2 +- > example/generator/odp_generator.c | 2 +- > example/ipsec/odp_ipsec.c | 2 +- > example/packet/odp_pktio.c | 2 +- > 4 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/example/classifier/odp_classifier.c b/example/classifier/odp_classifier.c > index 05a1eaa..18cd3bb 100644 > --- a/example/classifier/odp_classifier.c > +++ b/example/classifier/odp_classifier.c > @@ -227,7 +227,7 @@ static odp_pktio_t create_pktio(const char *dev, odp_pool_t pool) > int ret; > odp_pktio_param_t pktio_param; > > - memset(&pktio_param, 0, sizeof(pktio_param)); > + odp_pktio_param_init(&pktio_param); > pktio_param.in_mode = ODP_PKTIN_MODE_SCHED; > > /* Open a packet IO instance */ > diff --git a/example/generator/odp_generator.c b/example/generator/odp_generator.c > index 5a0d5a4..f80104c 100644 > --- a/example/generator/odp_generator.c > +++ b/example/generator/odp_generator.c > @@ -330,7 +330,7 @@ static odp_pktio_t create_pktio(const char *dev, odp_pool_t pool) > odp_queue_t inq_def; > odp_pktio_param_t pktio_param; > > - memset(&pktio_param, 0, sizeof(pktio_param)); > + odp_pktio_param_init(&pktio_param); > pktio_param.in_mode = ODP_PKTIN_MODE_SCHED; > > /* Open a packet IO instance */ > diff --git a/example/ipsec/odp_ipsec.c b/example/ipsec/odp_ipsec.c > index 9c1e03c..474f4a5 100644 > --- a/example/ipsec/odp_ipsec.c > +++ b/example/ipsec/odp_ipsec.c > @@ -496,7 +496,7 @@ void initialize_intf(char *intf) > char src_mac_str[MAX_STRING]; > odp_pktio_param_t pktio_param; > > - memset(&pktio_param, 0, sizeof(pktio_param)); > + odp_pktio_param_init(&pktio_param); > > if (getenv("ODP_IPSEC_USE_POLL_QUEUES")) > pktio_param.in_mode = ODP_PKTIN_MODE_POLL; > diff --git a/example/packet/odp_pktio.c b/example/packet/odp_pktio.c > index 4a8686e..93c38f5 100644 > --- a/example/packet/odp_pktio.c > +++ b/example/packet/odp_pktio.c > @@ -118,7 +118,7 @@ static odp_pktio_t create_pktio(const char *dev, odp_pool_t pool, int mode) > int ret; > odp_pktio_param_t pktio_param; > > - memset(&pktio_param, 0, sizeof(pktio_param)); > + odp_pktio_param_init(&pktio_param); > > switch (mode) { > case APPL_MODE_PKT_BURST:
I can certainly combine them, but I thought our policy was to separate examples from validation, etc. On Thu, Oct 15, 2015 at 6:05 AM, Maxim Uvarov <maxim.uvarov@linaro.org> wrote: > I think it's better to have single patch from patches 3,4 and 5, since > they do the same thing. > > Maxim. > > > On 10/14/2015 14:51, Bill Fischofer wrote: > >> Signed-off-by: Bill Fischofer <bill.fischofer@linaro.org> >> --- >> example/classifier/odp_classifier.c | 2 +- >> example/generator/odp_generator.c | 2 +- >> example/ipsec/odp_ipsec.c | 2 +- >> example/packet/odp_pktio.c | 2 +- >> 4 files changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/example/classifier/odp_classifier.c >> b/example/classifier/odp_classifier.c >> index 05a1eaa..18cd3bb 100644 >> --- a/example/classifier/odp_classifier.c >> +++ b/example/classifier/odp_classifier.c >> @@ -227,7 +227,7 @@ static odp_pktio_t create_pktio(const char *dev, >> odp_pool_t pool) >> int ret; >> odp_pktio_param_t pktio_param; >> - memset(&pktio_param, 0, sizeof(pktio_param)); >> + odp_pktio_param_init(&pktio_param); >> pktio_param.in_mode = ODP_PKTIN_MODE_SCHED; >> /* Open a packet IO instance */ >> diff --git a/example/generator/odp_generator.c >> b/example/generator/odp_generator.c >> index 5a0d5a4..f80104c 100644 >> --- a/example/generator/odp_generator.c >> +++ b/example/generator/odp_generator.c >> @@ -330,7 +330,7 @@ static odp_pktio_t create_pktio(const char *dev, >> odp_pool_t pool) >> odp_queue_t inq_def; >> odp_pktio_param_t pktio_param; >> - memset(&pktio_param, 0, sizeof(pktio_param)); >> + odp_pktio_param_init(&pktio_param); >> pktio_param.in_mode = ODP_PKTIN_MODE_SCHED; >> /* Open a packet IO instance */ >> diff --git a/example/ipsec/odp_ipsec.c b/example/ipsec/odp_ipsec.c >> index 9c1e03c..474f4a5 100644 >> --- a/example/ipsec/odp_ipsec.c >> +++ b/example/ipsec/odp_ipsec.c >> @@ -496,7 +496,7 @@ void initialize_intf(char *intf) >> char src_mac_str[MAX_STRING]; >> odp_pktio_param_t pktio_param; >> - memset(&pktio_param, 0, sizeof(pktio_param)); >> + odp_pktio_param_init(&pktio_param); >> if (getenv("ODP_IPSEC_USE_POLL_QUEUES")) >> pktio_param.in_mode = ODP_PKTIN_MODE_POLL; >> diff --git a/example/packet/odp_pktio.c b/example/packet/odp_pktio.c >> index 4a8686e..93c38f5 100644 >> --- a/example/packet/odp_pktio.c >> +++ b/example/packet/odp_pktio.c >> @@ -118,7 +118,7 @@ static odp_pktio_t create_pktio(const char *dev, >> odp_pool_t pool, int mode) >> int ret; >> odp_pktio_param_t pktio_param; >> - memset(&pktio_param, 0, sizeof(pktio_param)); >> + odp_pktio_param_init(&pktio_param); >> switch (mode) { >> case APPL_MODE_PKT_BURST: >> > > _______________________________________________ > lng-odp mailing list > lng-odp@lists.linaro.org > https://lists.linaro.org/mailman/listinfo/lng-odp >
diff --git a/example/classifier/odp_classifier.c b/example/classifier/odp_classifier.c index 05a1eaa..18cd3bb 100644 --- a/example/classifier/odp_classifier.c +++ b/example/classifier/odp_classifier.c @@ -227,7 +227,7 @@ static odp_pktio_t create_pktio(const char *dev, odp_pool_t pool) int ret; odp_pktio_param_t pktio_param; - memset(&pktio_param, 0, sizeof(pktio_param)); + odp_pktio_param_init(&pktio_param); pktio_param.in_mode = ODP_PKTIN_MODE_SCHED; /* Open a packet IO instance */ diff --git a/example/generator/odp_generator.c b/example/generator/odp_generator.c index 5a0d5a4..f80104c 100644 --- a/example/generator/odp_generator.c +++ b/example/generator/odp_generator.c @@ -330,7 +330,7 @@ static odp_pktio_t create_pktio(const char *dev, odp_pool_t pool) odp_queue_t inq_def; odp_pktio_param_t pktio_param; - memset(&pktio_param, 0, sizeof(pktio_param)); + odp_pktio_param_init(&pktio_param); pktio_param.in_mode = ODP_PKTIN_MODE_SCHED; /* Open a packet IO instance */ diff --git a/example/ipsec/odp_ipsec.c b/example/ipsec/odp_ipsec.c index 9c1e03c..474f4a5 100644 --- a/example/ipsec/odp_ipsec.c +++ b/example/ipsec/odp_ipsec.c @@ -496,7 +496,7 @@ void initialize_intf(char *intf) char src_mac_str[MAX_STRING]; odp_pktio_param_t pktio_param; - memset(&pktio_param, 0, sizeof(pktio_param)); + odp_pktio_param_init(&pktio_param); if (getenv("ODP_IPSEC_USE_POLL_QUEUES")) pktio_param.in_mode = ODP_PKTIN_MODE_POLL; diff --git a/example/packet/odp_pktio.c b/example/packet/odp_pktio.c index 4a8686e..93c38f5 100644 --- a/example/packet/odp_pktio.c +++ b/example/packet/odp_pktio.c @@ -118,7 +118,7 @@ static odp_pktio_t create_pktio(const char *dev, odp_pool_t pool, int mode) int ret; odp_pktio_param_t pktio_param; - memset(&pktio_param, 0, sizeof(pktio_param)); + odp_pktio_param_init(&pktio_param); switch (mode) { case APPL_MODE_PKT_BURST:
Signed-off-by: Bill Fischofer <bill.fischofer@linaro.org> --- example/classifier/odp_classifier.c | 2 +- example/generator/odp_generator.c | 2 +- example/ipsec/odp_ipsec.c | 2 +- example/packet/odp_pktio.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-)