[lttng-dev] Compile fix for urcu-bp.c

[lttng-dev] Compile fix for urcu-bp.c

[Duncan Sands via lttng-dev]

--- src/urcu-bp.c
+++ src/urcu-bp.c
@@ -409,7 +409,7 @@ void expand_arena(struct registry_arena *arena)
  		new_chunk_size_bytes, 0);
  	if (new_chunk != MAP_FAILED) {
  		/* Should not have moved. */
-		assert(new_chunk == last_chunk);
+		urcu_posix_assert(new_chunk == last_chunk);
  		memset((char *) last_chunk + old_chunk_size_bytes, 0,
  			new_chunk_size_bytes - old_chunk_size_bytes);
  		last_chunk->capacity = new_capacity;
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Re: [lttng-dev] Compile fix for urcu-bp.c

[Mathieu Desnoyers via lttng-dev]

Hi,

There are a few things missing before I can take this patch:

- Missing commit message describing the issue,
- Missing "Signed-off-by" tag.

Thanks!

Mathieu

On 2024-03-29 10:06, Duncan Sands via lttng-dev wrote:
> --- src/urcu-bp.c
> +++ src/urcu-bp.c
> @@ -409,7 +409,7 @@ void expand_arena(struct registry_arena *arena)
>           new_chunk_size_bytes, 0);
>       if (new_chunk != MAP_FAILED) {
>           /* Should not have moved. */
> -        assert(new_chunk == last_chunk);
> +        urcu_posix_assert(new_chunk == last_chunk);
>           memset((char *) last_chunk + old_chunk_size_bytes, 0,
>               new_chunk_size_bytes - old_chunk_size_bytes);
>           last_chunk->capacity = new_capacity;
> _______________________________________________
> lttng-dev mailing list
> lttng-dev@lists.lttng.org
> https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

-- 
Mathieu Desnoyers
EfficiOS Inc.
https://www.efficios.com

_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev