mbox series

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

Message ID 20241220174731.514432-1-bigeasy@linutronix.de (mailing list archive)
Headers show
Series module: Use RCU instead of RCU-sched. | expand

Message

Sebastian Andrzej Siewior Dec. 20, 2024, 5:41 p.m. UTC
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

Comments

Christophe Leroy Jan. 2, 2025, 12:47 p.m. UTC | #1
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
Peter Zijlstra Jan. 7, 2025, 8:48 a.m. UTC | #2
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>
Sebastian Andrzej Siewior Jan. 7, 2025, 3:45 p.m. UTC | #3
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