Re: [PATCH] eal/bsd: fix spinlock not unlock in alarm callback

2023-02-19 Thread Thomas Monjalon
16/02/2023 10:22, Bruce Richardson: > On Thu, Feb 16, 2023 at 09:28:17AM +0800, fengchengwen wrote: > > add cc maintainer > > > > On 2023/2/13 20:44, Chengwen Feng wrote: > > > The spinlock should unlock when clock_gettime() failed. This patch > > > fixes it by invoking clock_gettime() before lock

Re: [PATCH] eal/bsd: fix spinlock not unlock in alarm callback

2023-02-16 Thread Bruce Richardson
On Thu, Feb 16, 2023 at 09:28:17AM +0800, fengchengwen wrote: > add cc maintainer > > On 2023/2/13 20:44, Chengwen Feng wrote: > > The spinlock should unlock when clock_gettime() failed. This patch > > fixes it by invoking clock_gettime() before lock. > > > > Fixes: 26021a715067 ("eal/bsd: suppor

Re: [PATCH] eal/bsd: fix spinlock not unlock in alarm callback

2023-02-15 Thread fengchengwen
add cc maintainer On 2023/2/13 20:44, Chengwen Feng wrote: > The spinlock should unlock when clock_gettime() failed. This patch > fixes it by invoking clock_gettime() before lock. > > Fixes: 26021a715067 ("eal/bsd: support alarm API") > Cc: sta...@dpdk.org > > Signed-off-by: Chengwen Feng > ---

[PATCH] eal/bsd: fix spinlock not unlock in alarm callback

2023-02-13 Thread Chengwen Feng
The spinlock should unlock when clock_gettime() failed. This patch fixes it by invoking clock_gettime() before lock. Fixes: 26021a715067 ("eal/bsd: support alarm API") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- lib/eal/freebsd/eal_alarm.c | 6 +++--- 1 file changed, 3 insertions(+), 3