fix-up schedule_timeout() usage
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest18 of 32 patches 10 repliesIn review
18 of 32 patches are in this archive. The cover letter announces 32.
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/32 | include: update jiffies/{m,u}secs conversion functionsinclude/ include/ | +24 | 2 replies | |
| 2/32 | fs: fix-up schedule_timeout() usagefs/ fs/ fs/ +8 more | +23 | 5 replies | |
| 3/32 | kernel: fix-up schedule_timeout() usagekernel/ kernel/ kernel/ | +10 | 0 replies | |
| 4/32 | mm: fix-up schedule_timeout() usagemm/ mm/ | +2 | 0 replies | |
| 6/32 | sound: fix-up schedule_timeout() usagesound/ sound/ sound/ +25 more | +51 | 0 replies | |
| 9/32 | ia64: fix-up schedule_timeout() usagearch/ | +2 | 0 replies | |
| 11/32 | mips: fix-up schedule_timeout() usagearch/ arch/ | +3 | 0 replies | |
| 12/32 | ppc: fix-up schedule_timeout() usagearch/ arch/ | +8 | 0 replies | |
| 14/32 | um: fix-up schedule_timeout() usagearch/ | +2 | 1 reply | |
| 17/32 | drivers/cdrom: fix-up schedule_timeout() usagedrivers/ drivers/ | +2 | 0 replies | |
| 19/32 | drivers/dlm: fix-up schedule_timeout() usagedrivers/ | +1 | 0 replies | |
| 23/32 | drivers/macintosh: fix-up schedule_timeout() usagedrivers/ | +1 | 0 replies | |
| 24/32 | drivers/md: fix-up schedule_timeout() usagedrivers/ drivers/ | +2 | 0 replies | |
| 26/32 | message: fix-up schedule_timeout() usagedrivers/ drivers/ drivers/ | +11 | 0 replies | |
| 28/32 | drivers/sbus: fix-up schedule_timeout() usagedrivers/ drivers/ | +2 | 0 replies | |
| 29/32 | drivers/scsi: fix-up schedule_timeout() usagedrivers/ drivers/ drivers/ +5 more | +29 | 2 replies | |
| 30/32 | serial: fix-up schedule_timeout() usagedrivers/ | +3 | 0 replies | |
| 31/32 | telephony: fix-up schedule_timeout() usagedrivers/ | +18 | 0 replies |
Reviews trailers per patch
1 of 18 patches has a Reviewed-by or Acked-by.
| Patch | Jeff Dike |
|---|---|
| 1/32 | |
| 2/32 | |
| 3/32 | |
| 4/32 | |
| 6/32 | |
| 9/32 | |
| 11/32 | |
| 12/32 | |
| 14/32 | A-b |
| 17/32 | |
| 19/32 | |
| 23/32 | |
| 24/32 | |
| 26/32 | |
| 28/32 | |
| 29/32 | |
| 30/32 | |
| 31/32 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [-mm PATCH 0/32] fix-up schedule_timeout() usage
Hello Andrew,
Sorry for the lack of a combined To:/Cc: list, but it would have been excessive if I included everyone (I won't send all of the patches to LKML, unless there is no other list entry in MAINTAINERS - I think that is only 9 out of the 32 patches :)
Andrew recently added schedule_timeout_interruptible() and schedule_timeout_uninterruptible() to the -mm tree, as schedule_timeout() called without the state set results in an immediate return; thus, it makes little sense to ever call it that way. Almost all the kernel invocations of schedule_timeout(), additionally, follow:
set_current_state(TASK_{,UN}INTERRUPTIBLE);
schedule_timeout(some_value_in_jiffies);Andrew pointed out that creating non-inline wrappers for this common usage should reduce the kernel size. The following 32 patches (split up by directory/subsystem) convert the schedule_timeout() callers in 2.6.13-rc5-mm1, where appropriate.
Here's the list of patches as I split them up:
01/32: include/
02/32: fs/
03/32: kernel/
04/32: mm/
05/32: net/
06/32: sound/
07/32: arch/alpha
08/32: arch/i386
09/32: arch/ia64
10/32: arch/m68k
11/32: arch/mips
12/32: arch/ppc
13/32: arch/ppc64
14/32: arch/um
15/32: drivers/acpi
16/32: drivers/block
17/32: drivers/cdrom
18/32: drivers/char
19/32: drivers/dlm
20/32: drivers/net
21/32: drivers/ieee1394
22/32: drivers/isdn
23/32: drivers/macintosh
24/32: drivers/md
25/32: drivers/media
26/32: drivers/message
27/32: drivers/parport
28/32: drivers/sbus
29/32: drivers/scsi
30/32: drivers/serial
31/32: drivers/telephony
32/32: drivers/usb
Thanks,
Nish