Received-SPF: pass (google.com: domain of linux-kernel+bounces-1820-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Sat, 16 Dec 2023 07:27:20 +0800
From: Kent Gibson <warthog618@gmail.com>
To: Andy Shevchenko <andy@kernel.org>
Cc: linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, brgl@bgdev.pl,
	linus.walleij@linaro.org
Subject: Re: [PATCH v3 3/5] gpiolib: cdev: reduce locking in
 gpio_desc_to_lineinfo()
Message-ID: <ZXzg2DkAPPQV8vft@rigel>
References: <20231215023805.63289-1-warthog618@gmail.com>
 <20231215023805.63289-4-warthog618@gmail.com>
 <ZXyAh2hfhb3l5ogO@smile.fi.intel.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <ZXyAh2hfhb3l5ogO@smile.fi.intel.com>

On Fri, Dec 15, 2023 at 06:36:23PM +0200, Andy Shevchenko wrote:
> On Fri, Dec 15, 2023 at 10:38:03AM +0800, Kent Gibson wrote:
> > Reduce the time holding the gpio_lock by snapshotting the desc flags,
> > rather than testing them individually while holding the lock.
> >
> > Accept that the calculation of the used field is inherently racy, and
> > only check the availability of the line from pinctrl if other checks
> > pass, so avoiding the check for lines that are otherwise in use.
>
> Just wondering if you are using --histogram (diff algo) when preparing
> the patches...
>

Was just using the default myers.
Only changes patch 4, which it does make slightly more readable, so
will use it for v4.

Cheers,
Kent.