linux-kernel - Re: [PATCH v2 0/2] Convert regulator drivers to use devm_kmemdup

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <Z8mBQEKAJfZd6a7G@smile.fi.intel.com>
Date: Thu, 6 Mar 2025 13:04:32 +0200
From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To: Raag Jadav <raag.jadav@...el.com>
Cc: Mark Brown <broonie@...nel.org>, lgirdwood@...il.com,
	bleung@...omium.org, groeck@...omium.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 0/2] Convert regulator drivers to use
 devm_kmemdup_array()

On Thu, Mar 06, 2025 at 12:14:59PM +0200, Raag Jadav wrote:
> On Thu, Mar 06, 2025 at 11:19:31AM +0200, Andy Shevchenko wrote:
> > On Thu, Mar 06, 2025 at 04:15:55AM +0200, Raag Jadav wrote:
> > > On Fri, Feb 28, 2025 at 09:22:47PM +0000, Mark Brown wrote:
> > > > On Fri, 28 Feb 2025 12:50:55 +0530, Raag Jadav wrote:
> > > > > This series converts regulator drivers to use the newly introduced[1]
> > > > > devm_kmemdup_array() helper. This depends on changes available on
> > > > > immutable tag[2].
> > > > > 
> > > > > [1] https://lore.kernel.org/r/20250212062513.2254767-1-raag.jadav@intel.com
> > > > > [2] https://lore.kernel.org/r/Z7xGpz3Q4Zj6YHx7@black.fi.intel.com
> > > > > 
> > > > > [...]
> > > > 
> > > > Applied to
> > > > 
> > > >    https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next
> > > 
> > > Thank you.
> > > 
> > > Unless there's a nuance intended in the merge series title that I'm unable
> > > to understand, it probably seems incomplete.
> > 
> > I believe it's an issue somewhere in the scripts. The long Subject line is
> > split in the mailbox and that's probably is not supported by the machinery.
> > 
> > You are not the only one who reports this issue.
> 
> While I'm not well educated on the machinery, I'm also seeing devm_kmemdup_array()
> introduction commit reordered in -next and thinking perhaps it can cause issues
> with bisect, especially after final merge into Linus' tree?

I;m not sure what you exactly pointing out here. Mark seems applied only
necessary part of the immutable tag, which have the same effect. The Git merges
only once the stuff as long as it has the same hash.

-- 
With Best Regards,
Andy Shevchenko