linux-kernel - Re: [PATCH] i2c: i2c-elektor: Allow building on SMP kernels

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <b6670aba21de847615cec7534bacce3826e657c9.camel@physik.fu-berlin.de>
Date: Mon, 24 Nov 2025 23:42:02 +0100
From: John Paul Adrian Glaubitz <glaubitz@...sik.fu-berlin.de>
To: "Maciej W. Rozycki" <macro@...am.me.uk>
Cc: Magnus Lindholm <linmag7@...il.com>, linux-i2c@...r.kernel.org, 
	linux-kernel@...r.kernel.org, linux-alpha@...r.kernel.org,
 andi.shyti@...nel.org, 	wsa+renesas@...g-engineering.com
Subject: Re: [PATCH] i2c: i2c-elektor: Allow building on SMP kernels

On Mon, 2025-11-24 at 20:49 +0000, Maciej W. Rozycki wrote:
> On Sat, 22 Nov 2025, John Paul Adrian Glaubitz wrote:
> 
> > > there has since been some fixes/cleanup work to get rid of cli/sti and
> > 
> > s/has/have/
> 
>  An ambiguity here comes from the mixture of plural and uncountable for 
> "fixes" vs "... work".
> 
> > > rely on spinlocks instead (as pointed out by Wolfram Sang). Tested this
> > 
> > The sentence just ends after "instead". I think you meant to say "instead,
> > let's allow building the driver on SMP kernels again." or something like
> > that.
> 
>  FWIW I can see nothing wrong with the sentence, "... get rid of cli/sti 
> and rely on spinlocks instead." seems like standard English usage to me.

Yes, I was probably misreading it. Either way, I think the v2 looks fine.

Adrian

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913