Received-SPF: pass (google.com: domain of linux-wireless+bounces-8185-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <117ec723-fc35-42de-8e09-9ed25654a15f@bootlin.com>
Date: Tue, 28 May 2024 11:57:57 +0200
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [bug report] wifi: wilc1000: convert list management to RCU
To: Kalle Valo <kvalo@kernel.org>
Cc: Dan Carpenter <dan.carpenter@linaro.org>, linux-wireless@vger.kernel.org,
 "Ajay.Kathat" <Ajay.Kathat@microchip.com>,
 Thomas Petazzoni <thomas.petazzoni@bootlin.com>
References: <3b46ec7c-baee-49fd-b760-3bc12fb12eaf@moroto.mountain>
 <0a9c592d-7db3-454e-a83f-901b1865b414@bootlin.com>
 <878qzu8e4k.fsf@kernel.org>
From: =?UTF-8?Q?Alexis_Lothor=C3=A9?= <alexis.lothore@bootlin.com>
Content-Language: en-US
In-Reply-To: <878qzu8e4k.fsf@kernel.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Hello Kalle,

On 5/28/24 11:44, Kalle Valo wrote:
> Alexis Lothoré <alexis.lothore@bootlin.com> writes:
>> Because of the second point which would bring a not-so-clean fix, and the third
>> one for which I still don't have a proper fix, I am considering to submit a
>> revert for my RCU conversion series, to come back to SRCU. I don´t know if those
>> issues do or do not make SRCU usage more legitimate, but at least I feel like
>> really fixing it need slightly larger rework. I will still search for better
>> options, but if I do not find any, I will submit the revert.
> 
> Thanks for the good summary.Maybe it's easier just to revert the commit
> immediately so that you don't have to waste more time on this?

Yes, I have used already quite some time for this, let's do as you suggest and
opt for the revert directly.

> Especially if Ajay is missing.
> 
> But if would be nice if you could also include a separate patch which
> documents in the code why SRCU is needed. Just to avoid duplicate work
> later :)

Sure, I'll make sure to include this.

Thanks,

Alexis

-- 
Alexis Lothoré, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com