RE: [PATCH] examples/l2fwd-event: fix function args of worker cleanup

RE: [PATCH] examples/l2fwd-event: fix function args of worker cleanup

[Pavan Nikhilesh Bhagavatula]

> -----Original Message-----
> From: Shijith Thotton <sthotton@marvell.com>
> Sent: Monday, December 12, 2022 9:19 PM
> To: Jerin Jacob Kollanukkaran <jerinj@marvell.com>
> Cc: Shijith Thotton <sthotton@marvell.com>; dev@dpdk.org; Pavan
> Nikhilesh Bhagavatula <pbhagavatula@marvell.com>
> Subject: [PATCH] examples/l2fwd-event: fix function args of worker cleanup
> 
> Fixed function call arguments of l2fwd worker cleanup. Number of events
> enqueued and dequeued are set the opposite.
> 
> Fixes: 622ebb6b4a7d ("examples/l2fwd-event: clean up worker state before
> exit")
Cc: stable@dpdk.org
> 
> Signed-off-by: Shijith Thotton <sthotton@marvell.com>
Acked-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> ---
>  examples/l2fwd-event/l2fwd_event.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/examples/l2fwd-event/l2fwd_event.c b/examples/l2fwd-
> event/l2fwd_event.c
> index 63450537fe..4b5a032e35 100644
> --- a/examples/l2fwd-event/l2fwd_event.c
> +++ b/examples/l2fwd-event/l2fwd_event.c
> @@ -284,7 +284,7 @@ l2fwd_event_loop_burst(struct l2fwd_resources
> *rsrc,
>  		}
>  	}
> 
> -	l2fwd_event_worker_cleanup(event_d_id, port_id, ev, nb_rx,
> nb_tx, 0);
> +	l2fwd_event_worker_cleanup(event_d_id, port_id, ev, nb_tx,
> nb_rx, 0);
>  }
> 
>  static __rte_always_inline void
> @@ -468,7 +468,7 @@ l2fwd_event_loop_vector(struct l2fwd_resources
> *rsrc, const uint32_t flags)
>  		}
>  	}
> 
> -	l2fwd_event_worker_cleanup(event_d_id, port_id, ev, nb_rx,
> nb_tx, 1);
> +	l2fwd_event_worker_cleanup(event_d_id, port_id, ev, nb_tx,
> nb_rx, 1);
>  }
> 
>  static void __rte_noinline
> --
> 2.25.1

Re: [PATCH] examples/l2fwd-event: fix function args of worker cleanup

[Jerin Jacob]

On Tue, Dec 13, 2022 at 8:43 PM Pavan Nikhilesh Bhagavatula
<pbhagavatula@marvell.com> wrote:
>
>
>
> > -----Original Message-----
> > From: Shijith Thotton <sthotton@marvell.com>
> > Sent: Monday, December 12, 2022 9:19 PM
> > To: Jerin Jacob Kollanukkaran <jerinj@marvell.com>
> > Cc: Shijith Thotton <sthotton@marvell.com>; dev@dpdk.org; Pavan
> > Nikhilesh Bhagavatula <pbhagavatula@marvell.com>
> > Subject: [PATCH] examples/l2fwd-event: fix function args of worker cleanup
> >
> > Fixed function call arguments of l2fwd worker cleanup. Number of events
> > enqueued and dequeued are set the opposite.
> >
> > Fixes: 622ebb6b4a7d ("examples/l2fwd-event: clean up worker state before
> > exit")
> Cc: stable@dpdk.org
> >
> > Signed-off-by: Shijith Thotton <sthotton@marvell.com>
> Acked-by: Pavan Nikhilesh <pbhagavatula@marvell.com>

Applied to dpdk-next-net-eventdev/for-main. Thanks




> > ---
> >  examples/l2fwd-event/l2fwd_event.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/examples/l2fwd-event/l2fwd_event.c b/examples/l2fwd-
> > event/l2fwd_event.c
> > index 63450537fe..4b5a032e35 100644
> > --- a/examples/l2fwd-event/l2fwd_event.c
> > +++ b/examples/l2fwd-event/l2fwd_event.c
> > @@ -284,7 +284,7 @@ l2fwd_event_loop_burst(struct l2fwd_resources
> > *rsrc,
> >               }
> >       }
> >
> > -     l2fwd_event_worker_cleanup(event_d_id, port_id, ev, nb_rx,
> > nb_tx, 0);
> > +     l2fwd_event_worker_cleanup(event_d_id, port_id, ev, nb_tx,
> > nb_rx, 0);
> >  }
> >
> >  static __rte_always_inline void
> > @@ -468,7 +468,7 @@ l2fwd_event_loop_vector(struct l2fwd_resources
> > *rsrc, const uint32_t flags)
> >               }
> >       }
> >
> > -     l2fwd_event_worker_cleanup(event_d_id, port_id, ev, nb_rx,
> > nb_tx, 1);
> > +     l2fwd_event_worker_cleanup(event_d_id, port_id, ev, nb_tx,
> > nb_rx, 1);
> >  }
> >
> >  static void __rte_noinline
> > --
> > 2.25.1
>