Re: [PATCH v1 0/1] futex_waitv.2: Move text to a new PARAMETERS section

[Alejandro Colomar <alx@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 2565 bytes --]

Hi Branden, наб,

On 2026-02-18T14:40:12-0600, G. Branden Robinson wrote:
> At 2026-02-18T21:30:53+0100, наб wrote:
> > On Wed, Feb 18, 2026 at 09:26:38PM +0100, Alejandro Colomar wrote:
> > > On 2026-02-18T21:16:06+0100, наб wrote:
> > > > On Wed, Feb 18, 2026 at 01:41:38AM +0100, Alejandro Colomar wrote:
> > > > > Please let me know what you think of this patch?
> > > > > Here's how the patch changes the page:
> > > [...]
> > > > > What do you think?  I think the formatted page is more readable.
> > > > I really hate it.
> > > Okay.  Do you like any part, or is it all hateful?  :-)
> > The format annoys me, the layout infuriates me, I find the tone
> > insulting.
> 
> When dealing with formally complex (meaning: decomposable) proposals,
> rapprochement is difficult with a binary oracle.
> 
> Alex,
> 
> The onus might be on you to correspondingly decompose наб's black box.
> 
> Some obvious candidates include the two hunks of the end of the
> diff of the formatted document that change only a few words, and the
> existence of a "PARAMETERS" section of the page.

Hmmm, I've now pushed a much smaller change:

	$ MANWIDTH=64 diffman-git HEAD
	--- HEAD^:man/man2/futex_waitv.2
	+++ HEAD:man/man2/futex_waitv.2
	@@ -54,11 +54,16 @@ DESCRIPTION
	      Futex words to monitor are given by struct futex_waitv,
	      whose fields are analogous to FUTEX_WAIT(2const) parame‐
	      ters, except .__reserved must be 0 and .flags must contain
	-     one of FUTEX2_SIZE_* ORed with some of the flags below.
	+     exactly one size flag, ORed with some other flags.
	 
	      FUTEX2_SIZE_U32
		     .val and .uaddr[] are 32‐bit unsigned integers.
	 
	+     FUTEX2_SIZE_U8
	+     FUTEX2_SIZE_U16
	+     FUTEX2_SIZE_U64
	+            These are defined, but not supported (EINVAL).
	+
	      FUTEX2_NUMA
		     The futex word is followed by another word of the
		     same size (.uaddr points to uintN_t[2] rather than
	@@ -132,9 +137,8 @@ ERRORS
		     Any waiters[].flags field contains an unknown flag.
	 
	      EINVAL
	-            Any waiters[].flags field is missing a FU‐
	-            TEX2_SIZE_* flag or has a size flag different than
	-            FUTEX2_SIZE_U32 set.
	+            Any waiters[].flags field does not contain exactly
	+            one size flag, or it contains an unsupported one.
	 
	      EINVAL
		     Any waiters[].__reserved field is not 0.

I think this one should be uncontroversial.


Cheers,
Alex

-- 
<https://www.alejandro-colomar.es>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]