From: John Kacur <jkacur@redhat.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: LKML <linux-kernel@vger.kernel.org>,
Frederic Weisbecker <fweisbec@gmail.com>,
Thomas Gleixner <tglx@linutronix.de>,
Al Viro <viro@zeniv.linux.org.uk>, Jan Blunck <jblunck@suse.de>,
Ingo Molnar <mingo@elte.hu>, John Kacur <jkacur@redhat.com>,
Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: [PATCH 0/6] Push down BKL into device drivers
Date: Tue, 27 Apr 2010 11:14:39 +0200 (CEST) [thread overview]
Message-ID: <alpine.LFD.2.00.1004271112420.3318@localhost> (raw)
In-Reply-To: <1272320645-8671-1-git-send-email-arnd@arndb.de>
On Tue, 27 Apr 2010, Arnd Bergmann wrote:
> This is half the work of getting rid of the BKL in
> the ioctl file operation, the rest in arch/ and fs/
> still needs to be done, maybe two more hours of
> work (for someone else than me ;-)).
>
> Pushdown is straightforward. In many cases, it's
> rather obvious that the BKL is not needed at all,
> but let's not mix the removal with the pushdown.
Hi Arnd. These pushdowns conflicted with Linus's renaming.
In addition I found a few errors due to the change to
unlocked_ioctl.
I will resubmit fixes to these problems in a new mail.
Thanks
>
> Arnd Bergmann (6):
> dvb: push down BKL into ioctl functions
> scsi: push down BKL into ioctl functions
> isdn: push down BKL into ioctl functions
> staging: push down BKL into ioctl functions
> v4l: always use unlocked_ioctl
> drivers: push down BKL into various drivers
>
> drivers/block/pktcdvd.c | 13 ++++++--
> drivers/char/apm-emulation.c | 8 +++--
> drivers/char/applicom.c | 13 +++++---
> drivers/char/ds1620.c | 16 ++++++++-
> drivers/char/dtlk.c | 15 +++++----
> drivers/char/generic_nvram.c | 17 ++++++++--
> drivers/char/genrtc.c | 16 ++++++++-
> drivers/char/hpet.c | 14 +++++---
> drivers/char/i8k.c | 21 ++++++++++--
> drivers/char/ipmi/ipmi_devintf.c | 26 +++++++++++++---
> drivers/char/ipmi/ipmi_watchdog.c | 17 +++++++++-
> drivers/char/nvram.c | 10 ++++--
> drivers/char/nwflash.c | 7 +++-
> drivers/char/raw.c | 42 ++++++++++++++-----------
> drivers/hwmon/fschmd.c | 9 +++--
> drivers/hwmon/w83793.c | 10 ++++--
> drivers/input/misc/hp_sdc_rtc.c | 34 ++++++++++++++------
> drivers/isdn/capi/capi.c | 17 ++++++++--
> drivers/isdn/divert/divert_procfs.c | 19 ++++++++++--
> drivers/isdn/i4l/isdn_common.c | 18 +++++++++--
> drivers/isdn/mISDN/timerdev.c | 10 ++++--
> drivers/macintosh/nvram.c | 2 +-
> drivers/macintosh/via-pmu.c | 17 ++++++++--
> drivers/media/dvb/dvb-core/dmxdev.c | 31 +++++++++++++-----
> drivers/media/dvb/dvb-core/dvb_ca_en50221.c | 17 +++++++---
> drivers/media/dvb/dvb-core/dvb_frontend.c | 30 +++++++++---------
> drivers/media/dvb/dvb-core/dvb_net.c | 15 +++++++--
> drivers/media/dvb/dvb-core/dvbdev.c | 17 +++++++----
> drivers/media/dvb/dvb-core/dvbdev.h | 11 ++----
> drivers/media/dvb/firewire/firedtv-ci.c | 5 +--
> drivers/media/dvb/ttpci/av7110.c | 4 +-
> drivers/media/dvb/ttpci/av7110_av.c | 8 ++--
> drivers/media/dvb/ttpci/av7110_ca.c | 5 +--
> drivers/media/video/v4l2-dev.c | 17 ++++++++--
> drivers/mtd/mtdchar.c | 19 ++++++++---
> drivers/pcmcia/pcmcia_ioctl.c | 17 ++++++++--
> drivers/rtc/rtc-m41t80.c | 16 ++++++++-
> drivers/sbus/char/openprom.c | 44 +++++++++++++++-----------
> drivers/scsi/3w-9xxx.c | 10 ++++--
> drivers/scsi/3w-sas.c | 7 +++-
> drivers/scsi/3w-xxxx.c | 10 ++++--
> drivers/scsi/aacraid/linit.c | 11 +++++--
> drivers/scsi/dpt_i2o.c | 20 ++++++++++--
> drivers/scsi/gdth.c | 20 +++++++++---
> drivers/scsi/megaraid.c | 20 ++++++++++--
> drivers/scsi/megaraid/megaraid_mm.c | 22 ++++++++++---
> drivers/scsi/osst.c | 14 ++++++--
> drivers/scsi/sg.c | 17 ++++++++--
> drivers/staging/crystalhd/crystalhd_lnx.c | 13 +++++--
> drivers/staging/dt3155/dt3155_drv.c | 24 +++++++++++----
> drivers/staging/poch/poch.c | 17 +++++++++-
> drivers/staging/vme/devices/vme_user.c | 18 +++++++++--
> drivers/usb/mon/mon_bin.c | 23 ++++++++++----
> drivers/usb/mon/mon_stat.c | 3 +-
> 54 files changed, 631 insertions(+), 245 deletions(-)
>
> --
> 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/
>
next prev parent reply other threads:[~2010-04-27 9:14 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-16 3:56 [GIT PULL] Preparation for BKL'ed ioctl removal Frederic Weisbecker
2010-04-22 0:48 ` [GIT PULL v2] " Frederic Weisbecker
2010-04-24 15:25 ` Frederic Weisbecker
2010-04-24 18:36 ` Linus Torvalds
2010-04-24 18:47 ` Linus Torvalds
2010-04-24 19:54 ` Arnd Bergmann
2010-04-24 20:01 ` Linus Torvalds
2010-04-24 20:40 ` Arnd Bergmann
2010-04-24 22:15 ` Linus Torvalds
2010-04-25 17:39 ` Frederic Weisbecker
2010-04-25 17:49 ` Linus Torvalds
2010-04-25 18:05 ` Frederic Weisbecker
2010-04-26 8:30 ` Arnd Bergmann
2010-04-26 18:08 ` Linus Torvalds
2010-04-26 19:12 ` Arnd Bergmann
2010-04-26 20:36 ` Linus Torvalds
2010-04-26 22:23 ` [PATCH 0/6] Push down BKL into device drivers Arnd Bergmann
2010-04-27 9:14 ` John Kacur [this message]
2010-04-26 22:24 ` [PATCH 1/6] dvb: push down BKL into ioctl functions Arnd Bergmann
2010-04-26 22:24 ` [PATCH 2/6] scsi: " Arnd Bergmann
2010-04-26 22:24 ` [PATCH 3/6] isdn: " Arnd Bergmann
2010-04-26 22:24 ` [PATCH 4/6] staging: " Arnd Bergmann
2010-04-27 18:15 ` Frederic Weisbecker
2010-04-27 18:33 ` Greg KH
2010-04-26 22:24 ` [PATCH 5/6] v4l: always use unlocked_ioctl Arnd Bergmann
2010-04-26 22:24 ` [PATCH 6/6] drivers: push down BKL into various drivers Arnd Bergmann
2010-04-26 20:42 ` [GIT PULL v2] Preparation for BKL'ed ioctl removal David Miller
2010-04-26 22:09 ` Frederic Weisbecker
2010-04-26 22:32 ` Linus Torvalds
2010-04-26 23:04 ` Frederic Weisbecker
2010-04-26 7:25 ` Ingo Molnar
2010-04-26 11:29 ` Arnd Bergmann
2010-04-27 9:25 ` Ingo Molnar
2010-04-28 13:21 ` Frederic Weisbecker
2010-04-28 13:37 ` Ingo Molnar
2010-04-28 14:05 ` Arnd Bergmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.LFD.2.00.1004271112420.3318@localhost \
--to=jkacur@redhat.com \
--cc=arnd@arndb.de \
--cc=fweisbec@gmail.com \
--cc=jblunck@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
--cc=viro@zeniv.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox