Received-SPF: pass (google.com: domain of linux-kernel+bounces-207443-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Sun, 9 Jun 2024 19:33:21 +0200
From: Alejandro Colomar <alx@kernel.org>
To: Mark Brown <broonie@kernel.org>
Cc: Dev Jain <dev.jain@arm.com>, linux-man@vger.kernel.org, 
	mingo@kernel.org, tglx@linutronix.de, mark.rutland@arm.com, ryan.roberts@arm.com, 
	suzuki.poulose@arm.com, Anshuman.Khandual@arm.com, DeepakKumar.Mishra@arm.com, 
	AneeshKumar.KizhakeVeetil@arm.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] signal.7: Clearly describe ucontext kernel dump to
 userspace
Message-ID: <slkndx4223o6we7rygfatca6sa46hsw2fwbiovn2wx3mvyc55c@tyklfmkbe37r>
References: <20240607123119.769044-1-dev.jain@arm.com>
 <20240607123119.769044-2-dev.jain@arm.com>
 <ZmMptdPt_Zw9Xrlt@finisterre.sirena.org.uk>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="t7t3p7hirvo5uufk"
Content-Disposition: inline
In-Reply-To: <ZmMptdPt_Zw9Xrlt@finisterre.sirena.org.uk>


--t7t3p7hirvo5uufk
Content-Type: text/plain; protected-headers=v1; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
From: Alejandro Colomar <alx@kernel.org>
To: Mark Brown <broonie@kernel.org>
Cc: Dev Jain <dev.jain@arm.com>, linux-man@vger.kernel.org, 
	mingo@kernel.org, tglx@linutronix.de, mark.rutland@arm.com, ryan.roberts@arm.com, 
	suzuki.poulose@arm.com, Anshuman.Khandual@arm.com, DeepakKumar.Mishra@arm.com, 
	AneeshKumar.KizhakeVeetil@arm.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] signal.7: Clearly describe ucontext kernel dump to
 userspace
References: <20240607123119.769044-1-dev.jain@arm.com>
 <20240607123119.769044-2-dev.jain@arm.com>
 <ZmMptdPt_Zw9Xrlt@finisterre.sirena.org.uk>
MIME-Version: 1.0
In-Reply-To: <ZmMptdPt_Zw9Xrlt@finisterre.sirena.org.uk>

Hi Mark, Dev,

On Fri, Jun 07, 2024 at 04:39:33PM GMT, Mark Brown wrote:
> On Fri, Jun 07, 2024 at 06:01:18PM +0530, Dev Jain wrote:
>=20
> >  .I ucontext_t
> > -object that is pointed to by the third argument of the signal handler.)
> > +object that is pointed to by the third argument of the signal handler.
> > +We emphasize on the fact that this object contains context information
>=20
> s/We emphasize on the fact that/Note that/
>=20
> feels more idiomatic.

Even more idiomatic is to jump straight to the point, without noting it.
If it weren't notable, we wouldn't say it.  :)

>=20
> > +of the thread, present before jumping into the handler; the set of
> > +blocked signals for the current thread would be updated from struct
>=20
> s/would/will/
>=20
> > +sigaction only after
> > +.I ucontext_t
> > +has been dumped to userspace. This semantically makes sense since the

Please use semantic newlines.  See man-pages:

$ MANWIDTH=3D72 man man-pages | sed -n '/Use semantic newlines/,/^$/p'
   Use semantic newlines
     In the source of a manual page, new sentences should be started on
     new lines, long sentences should be split  into  lines  at  clause
     breaks  (commas,  semicolons, colons, and so on), and long clauses
     should be split at phrase boundaries.  This convention,  sometimes
     known as "semantic newlines", makes it easier to see the effect of
     patches, which often operate at the level of individual sentences,
     clauses, or phrases.

> > +context for which the signals have been blocked, remains only during
> > +execution of the handler.)
>=20
> I'd drop the "this semantically makes sense" and reword the last bit to
> be something like "The ucontext reflects the state at the time the
> signal is delivered rather than in the handler" for idiom reasons.

Thank you both.

Have a lovely day!
Alex


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

--t7t3p7hirvo5uufk
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEE6jqH8KTroDDkXfJAnowa+77/2zIFAmZl52EACgkQnowa+77/
2zISPw//YFBwIkNkOCIegtXdYoUMN4j9lZMglkuqPrE/WMJLHVb6XJ5URr2fDRBD
yLDzJ+ow05U73WYmRsXwM3q1NwD3t3CCvl50gxCwQ+IGZ1TnBu1NNzo79hoM/lgH
dgdbBMHGieK6XwYBtiWZLle/shy1wdBVvRae5lcfuCBh8NfYGWwwB5x4hSXf9Xao
pfeSTRD6OAZoouu3oO6ZKlp/6CcvYbYxdMxR+UEQImPprXp0WyBP9fLIydY5JSpq
+TloFpJXWkTYrlxbRLe8fcsIwJ5iDtz1hD5GpIPXosp/PHLjexot/d89Ixs/7pa6
rzdyrXMvrSYF+8vd4wCJjuRB+abewra5mn03b8pBs2Y9Lbqg8rb2QLo/OT3zIom5
Km5lVLCTpitZCJKnp32rk1eomz7FHiSzYsPdPqXg71DG9J0y0JU20ZBy8ZCu+ZsM
mzJ+XKkbC9bdAyuGnX1tbPxj43PNhVWfxRq8ZTmDr6/OaJDBoFS8mG+suP0THW13
rVOANNeljHABKHleiUEpXZKExZTq5n0RaNHCw3C8aoKN2Drsb1iCqLqyuwklin/b
/ILzChdYE2988Olnpm86pAaWAsR6T6jkHTFZaSDEH4lL0Z55GtpRWV7wF0kVOD6i
LcCtG0mhXbNwMrXlRost/n+DM3vc0/RfLDPGqDlpHAeh1Ly5s98=
=W4Or
-----END PGP SIGNATURE-----

--t7t3p7hirvo5uufk--