Thread (49 messages) 49 messages, 5 authors, 2026-02-18

Re: [PATCH v13] futex_waitv.2: new page

From: наб <hidden>
Date: 2026-02-17 16:17:05

On Tue, Feb 17, 2026 at 04:46:28PM +0100, Alejandro Colomar wrote:
silence; let me know your opinion:

	.tmp/man/man2/futex_waitv.2.d/futex_waitv.c:36:13: error: implicit use of sequentially-consistent atomic may incur stronger memory barriers than necessary [clang-diagnostic-atomic-implicit-seq-cst,-warnings-as-errors]
	   36 |      usleep(*futex * 10000);
	      |             ^


	.tmp/man/man2/futex_waitv.2.d/futex_waitv.c:37:13: error: implicit use of sequentially-consistent atomic may incur stronger memory barriers than necessary [clang-diagnostic-atomic-implicit-seq-cst,-warnings-as-errors]
	   37 |      *futex *= 2;
	      |             ^

I have little knowledge about atomics, so can't judge, but the code
seems good to me.  Please confirm.
Yeah, that's okay. Worst-case is it'll be marginally slower than
something more verbose that specifies... acqrel? which would be minimal here.
But that doesn't matter for example code.

Best,

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help