Re: [PATCH v13] futex_waitv.2: new page
From: наб <hidden>
Date: 2026-02-17 16:17:05
Attachments
- signature.asc [application/pgp-signature] 833 bytes
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,