| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <alpine.LFD.2.00.1004271112420.3318@localhost> Date: Tue, 27 Apr 2010 11:14:39 +0200 (CEST) From: John Kacur <jkacur@...hat.com> To: Arnd Bergmann <arnd@...db.de> cc: LKML <linux-kernel@...r.kernel.org>, Frederic Weisbecker <fweisbec@...il.com>, Thomas Gleixner <tglx@...utronix.de>, Al Viro <viro@...iv.linux.org.uk>, Jan Blunck <jblunck@...e.de>, Ingo Molnar <mingo@...e.hu>, John Kacur <jkacur@...hat.com>, Linus Torvalds <torvalds@...ux-foundation.org> Subject: Re: [PATCH 0/6] Push down BKL into device drivers 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@...r.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists