Re: [PATCH 00/28] iio: zero init stack with { } instead of memset()

From: Andy Shevchenko <andriy.shevchenko@intel.com>
To: Pavel Machek <pavel@ucw.cz>
Cc: "Nicolas Frattaroli" <nicolas.frattaroli@collabora.com>,
	"David Lechner" <dlechner@baylibre.com>,
	linux-rockchip@lists.infradead.org,
	"Michael Hennerich" <michael.hennerich@analog.com>,
	"Lars-Peter Clausen" <lars@metafoo.de>,
	"Jonathan Cameron" <jic23@kernel.org>,
	"Nuno Sá" <nuno.sa@analog.com>,
	"Andy Shevchenko" <andy@kernel.org>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	"AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Heiko Stuebner" <heiko@sntech.de>,
	"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
	"Alexandre Torgue" <alexandre.torgue@foss.st.com>,
	"Francesco Dolcini" <francesco@dolcini.it>,
	"João Paulo Gonçalves" <jpaulo.silvagoncalves@gmail.com>,
	"Leonard Göhrs" <l.goehrs@pengutronix.de>,
	kernel@pengutronix.de, "Oleksij Rempel" <o.rempel@pengutronix.de>,
	"Roan van Dijk" <roan@protonic.nl>,
	"Tomasz Duszynski" <tomasz.duszynski@octakon.com>,
	"Jacopo Mondi" <jacopo@jmondi.org>,
	"Jean-Baptiste Maneyrol" <jean-baptiste.maneyrol@tdk.com>,
	"Mudit Sharma" <muditsharma.info@gmail.com>,
	"Javier Carrasco" <javier.carrasco.cruz@gmail.com>,
	"Ondřej Jirman" <megi@xff.cz>,
	"Andreas Klinger" <ak@it-klinger.de>,
	"Petre Rodan" <petre.rodan@subdimension.ro>,
	linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org,
	linux-stm32@st-md-mailman.stormreply.com
Subject: Re: [PATCH 00/28] iio: zero init stack with { } instead of memset()
Date: Thu, 12 Jun 2025 22:10:07 +0300	[thread overview]
Message-ID: <aEsmDyc44P8amm5p@smile.fi.intel.com> (raw)
In-Reply-To: <aEsiTy++yKGe1p9W@duo.ucw.cz>

On Thu, Jun 12, 2025 at 08:54:07PM +0200, Pavel Machek wrote:
> > On Thursday, 12 June 2025 11:17:52 Central European Summer Time Pavel Machek wrote:
> > > 
> > > > Jonathan mentioned recently that he would like to get away from using
> > > > memset() to zero-initialize stack memory in the IIO subsystem. And we
> > > > have it on good authority that initializing a struct or array with = { }
> > > > is the preferred way to do this in the kernel [1]. So here is a series
> > > > to take care of that.
> > > 
> > > 1) Is it worth the churn?
> > > 
> > > 2) Will this fail to initialize padding with some obscure compiler?
> > 
> > as of right now, the only two C compilers that are supported are
> > GCC >= 8.1, and Clang >= 13.0.1. If anyone even manages to get the
> > kernel
> 
> Well... I'm pretty sure parts of this would make it into -stable as a
> dependency, or because AUTOSEL decides it is a bugfix. So..
> 
> GNU C                  4.9              gcc --version
> Clang/LLVM (optional)  10.0.1           clang --version

Even though, what the kernel versions are you referring to? I am sure there
plenty of cases with {} there.

-- 
With Best Regards,
Andy Shevchenko