Re: [PATCH 00/10] bkl: pushdowns from Arnd, and compile fixes

[John Kacur <jkacur@redhat.com>]

On Tue, 27 Apr 2010, Arnd Bergmann wrote:

> On Tuesday 27 April 2010, John Kacur wrote:
> > These are bkl pushdowns from Arnd that conflicted with
> > Linus's "Preparation for BKL'ed ioctl removal patch". I fixed-up the merge
> > conflicts. In addition - during allyesconfig compile testing on x86_64 I found
> > a number of issues that I fixed-up
> 
> Thanks for catching the scsi bugs! It was getting late yesterday night, so
> I didn't go through a full allyesconfig then.
> 
> > I pushed these to my own new linus-bkl tree.
> > git://www.kernel.org/pub/scm/linux/kernel/git/jkacur/jk-2.6.git linux-bkl
> > 
> > Frederic, if you want to grab those, pls do so. also, we might want to combine
> > my compile fixes with Arnd's push-down patches, for better git bisectability.
> 
> Yes, your fixes (except the v4l2 patch, see comment there) should best be
> merged into the main patches.
> 
> The idea was that if we manage to kill all fops->ioctl users right away,
> we wouldn't even need Linus' patch and move to dungeon level 2 directly.
> Either way works for me though.
> 
> Should we just split up the rest? If you, Frederic and me each take one
> of these chunks, we're done. According to the diffstat from Linus' patch,
> this is what's left:

Okay - good plan. I AM trying to balance doing my own day-to-day work,
and writing slides for a conference but I'm sure you and Frederic are
 just as pressed for time as me, so lets go-ahead and divide it up!
That will prevent us from doing redundant work.
I assume the target is 2.6.35 so, we have a little bit of time anyway.

I suggest we divide up a) and b) first and postone c) until the first
parts are done.

Arranging our names by alphabetical order and shuffling the files
off one at a time (for variety), I get this. Is everybody happy with that?
36 files / 3 = 12 files each.
Jan - speak up if you want in.

Arnd
----
arch/cris/arch-v10/drivers/ds1302.c
arch/cris/arch-v10/drivers/pcf8563.c
arch/cris/arch-v32/drivers/i2c.c
arch/cris/arch-v32/drivers/pcf8563.c
arch/ia64/sn/kernel/sn2/sn_hwperf.c
arch/m68k/mvme16x/rtc.c
arch/um/drivers/mmapper_kern.c
fs/coda/psdev.c
fs/hfsplus/dir.c
fs/logfs/file.c
fs/ntfs/dir.c
fs/smbfs/file.c

Frederic
--------
arch/cris/arch-v10/drivers/gpio.c
arch/cris/arch-v10/drivers/sync_serial.c
arch/cris/arch-v32/drivers/mach-a3/gpio.c
arch/cris/arch-v32/drivers/sync_serial.c
arch/m68k/bvme6000/rtc.c
arch/um/drivers/harddog_kern.c
fs/autofs/root.c
fs/fat/dir.c
fs/hfsplus/inode.c
fs/ncpfs/dir.c
fs/ntfs/file.c
fs/udf/dir.c

John
----
arch/cris/arch-v10/drivers/i2c.c
arch/cris/arch-v32/drivers/cryptocop.c
arch/cris/arch-v32/drivers/mach-fs/gpio.c
arch/ia64/kernel/perfmon.c
arch/m68k/bvme6000/rtc.c
arch/um/drivers/hostaudio_kern.c
fs/coda/pioctl.c
fs/fat/file.c
fs/logfs/dir.c
fs/ncpfs/file.c
fs/smbfs/dir.c
fs/udf/file.c

> 
> a) arch specific drivers
>  arch/cris/arch-v10/drivers/ds1302.c         |    2 +-
>  arch/cris/arch-v10/drivers/gpio.c           |    2 +-
>  arch/cris/arch-v10/drivers/i2c.c            |    2 +-
>  arch/cris/arch-v10/drivers/pcf8563.c        |    2 +-
>  arch/cris/arch-v10/drivers/sync_serial.c    |    2 +-
>  arch/cris/arch-v32/drivers/cryptocop.c      |    2 +-
>  arch/cris/arch-v32/drivers/i2c.c            |    2 +-
>  arch/cris/arch-v32/drivers/mach-a3/gpio.c   |    2 +-
>  arch/cris/arch-v32/drivers/mach-fs/gpio.c   |    2 +-
>  arch/cris/arch-v32/drivers/pcf8563.c        |    2 +-
>  arch/cris/arch-v32/drivers/sync_serial.c    |    2 +-
>  arch/ia64/kernel/perfmon.c                  |    2 +-
>  arch/ia64/sn/kernel/sn2/sn_hwperf.c         |    2 +-
>  arch/m68k/bvme6000/rtc.c                    |    2 +-
>  arch/m68k/mvme16x/rtc.c                     |    2 +-
>  arch/um/drivers/harddog_kern.c              |    2 +-
>  arch/um/drivers/hostaudio_kern.c            |    4 +-
>  arch/um/drivers/mmapper_kern.c              |    2 +-
> 
> b) file systems (my other series has patches for some of these already)
>  fs/autofs/root.c                            |    2 +-
>  fs/autofs4/root.c                           |    2 +-
>  fs/coda/pioctl.c                            |    2 +-
>  fs/coda/psdev.c                             |    2 +-
>  fs/fat/dir.c                                |    2 +-
>  fs/fat/file.c                               |    2 +-
>  fs/hfsplus/dir.c                            |    2 +-
>  fs/hfsplus/inode.c                          |    2 +-
>  fs/logfs/dir.c                              |    2 +-
>  fs/logfs/file.c                             |    2 +-
>  fs/ncpfs/dir.c                              |    2 +-
>  fs/ncpfs/file.c                             |    2 +-
>  fs/ntfs/dir.c                               |    2 +-
>  fs/ntfs/file.c                              |    2 +-
>  fs/smbfs/dir.c                              |    2 +-
>  fs/smbfs/file.c                             |    2 +-
>  fs/udf/dir.c                                |    2 +-
>  fs/udf/file.c                               |    2 +-
> 
> c) special stuff (this should go last)
>  Documentation/filesystems/vfs.txt           |    2 +-
>  fs/bad_inode.c                              |    2 +-
>  fs/ioctl.c                                  |    4 +-
>  fs/compat_ioctl.c                           |    2 +-
>  fs/ecryptfs/file.c                          |    9 ++++---
>  fs/proc/inode.c                             |    8 +++---
>  include/linux/fs.h                          |    2 -
> 
> 	Arnd
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>