[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aEsmDyc44P8amm5p@smile.fi.intel.com>
Date: Thu, 12 Jun 2025 22:10:07 +0300
From: Andy Shevchenko <andriy.shevchenko@...el.com>
To: Pavel Machek <pavel@....cz>
Cc: Nicolas Frattaroli <nicolas.frattaroli@...labora.com>,
David Lechner <dlechner@...libre.com>,
linux-rockchip@...ts.infradead.org,
Michael Hennerich <michael.hennerich@...log.com>,
Lars-Peter Clausen <lars@...afoo.de>,
Jonathan Cameron <jic23@...nel.org>,
Nuno Sá <nuno.sa@...log.com>,
Andy Shevchenko <andy@...nel.org>,
Matthias Brugger <matthias.bgg@...il.com>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
Heiko Stuebner <heiko@...ech.de>,
Maxime Coquelin <mcoquelin.stm32@...il.com>,
Alexandre Torgue <alexandre.torgue@...s.st.com>,
Francesco Dolcini <francesco@...cini.it>,
João Paulo Gonçalves <jpaulo.silvagoncalves@...il.com>,
Leonard Göhrs <l.goehrs@...gutronix.de>,
kernel@...gutronix.de, Oleksij Rempel <o.rempel@...gutronix.de>,
Roan van Dijk <roan@...tonic.nl>,
Tomasz Duszynski <tomasz.duszynski@...akon.com>,
Jacopo Mondi <jacopo@...ndi.org>,
Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@....com>,
Mudit Sharma <muditsharma.info@...il.com>,
Javier Carrasco <javier.carrasco.cruz@...il.com>,
Ondřej Jirman <megi@....cz>,
Andreas Klinger <ak@...klinger.de>,
Petre Rodan <petre.rodan@...dimension.ro>,
linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
linux-mediatek@...ts.infradead.org,
linux-stm32@...md-mailman.stormreply.com
Subject: Re: [PATCH 00/28] iio: zero init stack with { } instead of memset()
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
Powered by blists - more mailing lists