Re: UPDATE: liburcu

On 6/4/24 02:52, Brad Smith wrote:
> Here is a diff to add futex support for liburcu.
>

Copying for others on the list - I've tested it out net/knot in two
production servers and it's working well.

OK aisha


> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/devel/liburcu/Makefile,v
> retrieving revision 1.28
> diff -u -p -u -p -r1.28 Makefile
> --- Makefile 17 Oct 2023 06:17:19 -0000 1.28
> +++ Makefile 4 Jun 2024 04:00:59 -0000
> @@ -3,7 +3,7 @@ COMMENT = user-level RCU implementation
> V = 0.14.0
> DISTNAME = userspace-rcu-$V
> PKGNAME = liburcu-$V
> -REVISION = 0
> +REVISION = 1
>
> SHARED_LIBS += urcu-bp 3.0 # 9.0
> SHARED_LIBS += urcu-cds 2.1 # 9.0
> @@ -34,5 +34,13 @@ COMPILER = base-clang ports-gcc
> USE_GMAKE = Yes
>
> CONFIGURE_STYLE = gnu
> +
> +TEST_DEPENDS += shells/bash
> +
> +# Adjust shell scripts to allow Bash in other locations
> +# da56d5cad05a280a8171ef51f185e6d0d29610f0
> +post-extract:
> + find ${WRKSRC}/tests -exec grep -q '/bin/bash' {} \; -print |\
> + xargs sed -i "s#/bin/bash#/usr/bin/env bash#g"
>
> .include <bsd.port.mk>
> Index: patches/patch-include_urcu_futex_h
> ===================================================================
> RCS file: patches/patch-include_urcu_futex_h
> diff -N patches/patch-include_urcu_futex_h
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ patches/patch-include_urcu_futex_h 4 Jun 2024 04:00:59 -0000
> @@ -0,0 +1,121 @@
> +- Use futex on OpenBSD
> + e9af364c26b0e474b87a7fe5fb2222a399f8e180
> +- futex.h: Use urcu_posix_assert to validate unused values
> + e463c38f0ec65d06e544681d1916991808a6a2b9
> +
> +Index: include/urcu/futex.h
> +--- include/urcu/futex.h.orig
> ++++ include/urcu/futex.h
> +@@ -39,20 +39,28 @@
> + #include <errno.h>
> + #include <urcu/compiler.h>
> + #include <urcu/arch.h>
> ++#include <urcu/assert.h>
> +
> + #elif defined(__FreeBSD__)
> +
> + #include <sys/types.h>
> + #include <sys/umtx.h>
> +
> ++#elif defined(__OpenBSD__)
> ++
> ++#include <sys/time.h>
> ++#include <sys/futex.h>
> ++
> +