[EXT] [PATCH] examples/l2fwd-event: fix missing unlock issue

Jerin Jacob Kollanukkaran jerinj at marvell.com
Sat Nov 4 08:51:33 CET 2023



> -----Original Message-----
> From: Weiguo Li <liwg06 at foxmail.com>
> Sent: Saturday, November 4, 2023 12:52 PM
> To: Sunil Kumar Kori <skori at marvell.com>; Pavan Nikhilesh Bhagavatula
> <pbhagavatula at marvell.com>
> Cc: dev at dpdk.org; stable at dpdk.org; Weiguo Li <liweiguo at xencore.cn>
> Subject: [EXT] [PATCH] examples/l2fwd-event: fix missing unlock issue
> 
> External Email
> 
> ----------------------------------------------------------------------
> The function l2fwd_get_free_event_port acquires a lock on 'evt_rsrc->evp.lock'
> at the beginning. This lock is expected to be released at the first return
> statement, when no free event port is available.
> 
> Fixes: 080f57bceca4 ("examples/l2fwd-event: add eventdev main loop")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Weiguo Li <liweiguo at xencore.cn>

Please Fix issue found by CI at http://mails.dpdk.org/archives/test-report/2023-November/501341.html

> ---
>  examples/l2fwd-event/l2fwd_event.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/examples/l2fwd-event/l2fwd_event.c b/examples/l2fwd-
> event/l2fwd_event.c
> index 4b5a032e35..842fd816e5 100644
> --- a/examples/l2fwd-event/l2fwd_event.c
> +++ b/examples/l2fwd-event/l2fwd_event.c
> @@ -140,6 +140,7 @@ l2fwd_get_free_event_port(struct
> l2fwd_event_resources *evt_rsrc)
> 
>  	rte_spinlock_lock(&evt_rsrc->evp.lock);
>  	if (index >= evt_rsrc->evp.nb_ports) {
> +		rte_spinlock_unlock(&evt_rsrc->evp.lock);
>  		printf("No free event port is available\n");
>  		return -1;
>  	}
> --
> 2.34.1



More information about the stable mailing list