[v2] module: Use RCU instead of RCU-sched.

[PATCH v2 00/28] module: Use RCU instead of RCU-sched.

Posted by Sebastian Andrzej Siewior 12 months ago

Hi,

This is an updated version of the inital post after PeterZ made me aware
that there are users outside of the module directory.
The goal is replace the mix auf rcu_read_lock(), rcu_read_lock_sched()
and preempt_disable() with just rcu_read_lock().

I've splitted it into smaller chunks which can be applied/ reviewed
independently. I'm just not sure about the cfi patch (28/28) so I added
just a comment instead.

v1…v2: https://lore.kernel.org/all/20241205215102.hRywUW2A@linutronix.de/
  - Splitted into smaller patches.
  - Converted all users.

Sebastian

Re: [PATCH v2 00/28] module: Use RCU instead of RCU-sched.

Posted by Christophe Leroy 11 months, 2 weeks ago

Hi Sebastian,

Le 20/12/2024 à 18:41, Sebastian Andrzej Siewior a écrit :
> Hi,
> 
> This is an updated version of the inital post after PeterZ made me aware
> that there are users outside of the module directory.
> The goal is replace the mix auf rcu_read_lock(), rcu_read_lock_sched()
> and preempt_disable() with just rcu_read_lock().
> 
> I've splitted it into smaller chunks which can be applied/ reviewed
> independently. I'm just not sure about the cfi patch (28/28) so I added
> just a comment instead.
> 
> v1…v2: https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore.kernel.org%2Fall%2F20241205215102.hRywUW2A%40linutronix.de%2F&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7Cbad0e7f9344a47371a0808dd211fdefd%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C638703142988931970%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=a%2FV5g5zRqceJWedAAZJ4zt6XJpZ0Yqm%2BqKPrsItuEXs%3D&reserved=0
>    - Splitted into smaller patches.
>    - Converted all users.
> 

How did you generate that cover letter ?

It should contain the full list of files modified by the series, so that 
I can see if any of them is of interest to me without going into each patch.

This is done automatically when you use 'git format-patch --cover-letter'.

Christophe

Re: [PATCH v2 00/28] module: Use RCU instead of RCU-sched.

Posted by Sebastian Andrzej Siewior 11 months, 2 weeks ago

On 2025-01-02 13:47:06 [+0100], Christophe Leroy wrote:
> Hi Sebastian,
Hi Christophe,

> How did you generate that cover letter ?
> 
> It should contain the full list of files modified by the series, so that I
> can see if any of them is of interest to me without going into each patch.
> 
> This is done automatically when you use 'git format-patch --cover-letter'.

I could do so for v3. This was format-patch without --cover-letter.

> Christophe

Sebastian

Re: [PATCH v2 00/28] module: Use RCU instead of RCU-sched.

Posted by Peter Zijlstra 11 months, 2 weeks ago

On Fri, Dec 20, 2024 at 06:41:14PM +0100, Sebastian Andrzej Siewior wrote:
> Hi,
> 
> This is an updated version of the inital post after PeterZ made me aware
> that there are users outside of the module directory.
> The goal is replace the mix auf rcu_read_lock(), rcu_read_lock_sched()
> and preempt_disable() with just rcu_read_lock().
> 
> I've splitted it into smaller chunks which can be applied/ reviewed
> independently. I'm just not sure about the cfi patch (28/28) so I added
> just a comment instead.
> 
> v1…v2: https://lore.kernel.org/all/20241205215102.hRywUW2A@linutronix.de/
>   - Splitted into smaller patches.
>   - Converted all users.

Thanks for doing the cleanup!

Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>