Re: [PATCH] Documentation/livepatch: Update terminology in livepatch

From: Bagas Sanjaya
Date: Mon Dec 25 2023 - 23:52:11 EST


On Sat, Dec 23, 2023 at 03:08:54PM -0700, Jonathan Corbet wrote:
> attreyee-muk <tintinm2017@xxxxxxxxx> writes:
>
> > Update the sentence in livepatch.rst to: "Functions are there for a reason. Take some input parameters, acquire or release locks, read, process, and write some data in a defined way."
> >
> > Signed-off-by: Attreyee Mukherjee <tintinm2017@xxxxxxxxx>
> > ---
> > Documentation/livepatch/livepatch.rst | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> So this is a classic example of saying what you have done, but not why.
> What makes this a change that we want?

I think what he intended was "The word 'get' is not the correct antonym to
'release' in the context of locking. Replace it with 'acquire'".

Thanks.

--
An old man doll... just what I always wanted! - Clara

Attachment: signature.asc
Description: PGP signature