Re: [PATCH, RFC] char dev BKL pushdown

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linus Torvalds
Date: Saturday, May 17, 2008 - 2:58 pm

Btw, Jonathan, would you be willing to maintain some kind of tree of these 
BKL removal patches? 

This is different from the work Ingo is doing in the sense that these 
things should be (a) safe and reasonably obvious and thus (b) presumably 
ready to be merged in the next merge window. Ingo's BKL debugging tree is 
likely a good thing to use to find places that need work, but actually 
removing the BKL from some subsystem is a different issue.

(And when I say "safe and reasonably obvious" I obviously don't mean that 
there can't be bugs. Mistakes happen, and some BKL use might be overly 
subtle like the issue that Alan pointed out with an empty ->open routine 
almost accidentally serializing with initialization, but that's why I'd 
not merge these things after -rc2 anyway, but in the next merge window).

Because if you're willing to maintain a BKL-cleanup tree that gets merged 
into linux-next etc, I'd submit my VFAT/MSDOS BKL removal patch to you. 
The reason I did that one was that Thomas actually reported that to be a 
major source of latency problems on one of his embedded systems (80ms 
latency!), so it would be nice to have that patch in some place where it 
might get tested.

		Linus
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [announce] "kill the Big Kernel Lock (BKL)" tree, Frederik Deweerdt, (Wed May 14, 1:05 pm)
[PATCH, RFC] char dev BKL pushdown , Jonathan Corbet, (Fri May 16, 8:44 am)
Re: [PATCH, RFC] char dev BKL pushdown, Christoph Hellwig, (Fri May 16, 8:49 am)
[PATCH] kill empty chardev open/release methods, Christoph Hellwig, (Fri May 16, 9:03 am)
Re: [PATCH, RFC] char dev BKL pushdown, Alan Cox, (Fri May 16, 9:22 am)
Re: [PATCH, RFC] char dev BKL pushdown , Linus Torvalds, (Fri May 16, 9:30 am)
Re: [PATCH, RFC] char dev BKL pushdown , Jonathan Corbet, (Fri May 16, 9:43 am)
Re: [PATCH, RFC] char dev BKL pushdown, Arnd Bergmann, (Sat May 17, 2:15 pm)
Re: [PATCH, RFC] char dev BKL pushdown , Linus Torvalds, (Sat May 17, 2:58 pm)
Re: [PATCH] kill empty chardev open/release methods , Jonathan Corbet, (Sun May 18, 12:46 pm)
Re: [PATCH] kill empty chardev open/release methods, Alan Cox, (Sun May 18, 12:58 pm)
Re: [PATCH, RFC] char dev BKL pushdown , Jonathan Corbet, (Sun May 18, 1:07 pm)
Re: [PATCH, RFC] char dev BKL pushdown , Jonathan Corbet, (Sun May 18, 1:26 pm)
Re: [PATCH, RFC] char dev BKL pushdown, Arnd Bergmann, (Mon May 19, 4:07 pm)
[PATCH 1/3, RFC] misc char dev BKL pushdown, Arnd Bergmann, (Mon May 19, 4:26 pm)
[PATCH 3/3, RFC] remove BKL from misc_open(), Arnd Bergmann, (Mon May 19, 4:34 pm)
Re: [PATCH 1/3, RFC] misc char dev BKL pushdown, Mike Frysinger, (Mon May 19, 5:07 pm)
Re: [PATCH 1/3, RFC] misc char dev BKL pushdown , Jonathan Corbet, (Mon May 19, 5:21 pm)
Re: [PATCH 1/3, RFC] misc char dev BKL pushdown, Mike Frysinger, (Mon May 19, 5:46 pm)
Re: [PATCH 1/3, RFC] misc char dev BKL pushdown, Alan Cox, (Tue May 20, 1:46 am)
Re: [PATCH, RFC] char dev BKL pushdown , Jonathan Corbet, (Tue May 20, 8:13 am)
Re: [PATCH, RFC] char dev BKL pushdown, Arnd Bergmann, (Tue May 20, 10:21 am)
Re: [PATCH, RFC] char dev BKL pushdown, Alan Cox, (Tue May 20, 11:51 am)
Re: [PATCH 1/3, RFC] misc char dev BKL pushdown, Mike Frysinger, (Tue May 20, 4:01 pm)
Re: [PATCH 1/3, RFC] misc char dev BKL pushdown , Jonathan Corbet, (Tue May 20, 4:25 pm)
Re: [PATCH 1/3, RFC] misc char dev BKL pushdown, Mike Frysinger, (Wed May 21, 9:22 am)