summaryrefslogtreecommitdiffstats
path: root/arch/cris/arch-v32/mm (unfollow)
Commit message (Collapse)AuthorFilesLines
2008-06-20Add cycle_kernel_lock()Jonathan Corbet1-0/+13
A number of driver functions are so obviously trivial that they do not need the big kernel lock - at least not overtly. It turns out that the acquisition of the BKL in driver open() functions can perform a sort of poor-hacker's serialization function, delaying the open operation until the driver is certain to have completed its initialization. Add a simple cycle_kernel_lock() function for these cases to make it clear that there is no need to *hold* the BKL, just to be sure that we can acquire it. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20videodev: BKL pushdownJonathan Corbet1-0/+4
Put explicit lock_kernel() calls into videodev_open(). That function itself seems OK, but one never knows about all the open() functions provided by underlying video drivers. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20usbdev: BKL pushdownJonathan Corbet1-0/+2
Add explicit lock_kernel() calls to usbdev_open() Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20USB Monitor: BKL pushdownJonathan Corbet1-0/+6
Add explicit lock_kernel() calls to mon_bin_open() Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20printer gadget: BKL pushdownJonathan Corbet1-1/+2
Add explicit lock_kernel() calls to printer_open() Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20adb: BKL pushdownJonathan Corbet1-5/+13
Put explicit lock_kernel() calls in adb_open(). The fact that adb_release() already has them suggests this is necessary. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20divamnt: BKL pushdownJonathan Corbet1-5/+11
Put explicit lock_kernel() calls into maint_open(). Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20CAPI: BKL pushdownJonathan Corbet1-6/+11
Put explicit lock_kernel() calls into capi_open() Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20changer: BKL pushdownJonathan Corbet1-0/+4
Add lock_kernel() calls to ch_open(), though the existing locking looks adequate. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20dpt_i20: BKL pushdownJonathan Corbet1-0/+5
Add lock_kernel() calls to adpt_open() Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20videopix: BKL pushdownJonathan Corbet1-0/+5
Add explicit lock_kernel() calls to vfc_open(). Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20bpp: bkl pushdownJonathan Corbet1-0/+3
Put explicit lock_kernel() calls into bpp_open(). It has locking, but I'm not convinced it won't race with ioctl(). Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20phantom: BKL pushdownJonathan Corbet1-2/+7
Add explicit lock_kernel calls to phantom_open(). Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20infiniband: more BKL pushdownJonathan Corbet2-3/+13
Be extra-cautious and protect the remaining open() functions. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20dsp56k: BKL pushdownJonathan Corbet1-5/+11
Put explicit lock_kernel calls into dsp56k_open(). [Stupid missing label error fixed] Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20raw: BKL pushdownJonathan Corbet1-0/+3
Put explicit lock_kernel() calls into raw_open(), even though the existing locking looks adequate. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20tlckl: BKL pushdownJonathan Corbet1-7/+12
Put explicit lock_kernel calls into tlclk_open() Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20xilinx icap: BKL pushdownJonathan Corbet1-1/+5
Add explicit lock_kernel() calls to hwicap_open() even though the existing locking looks adequate. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20vcs: BKL pushdownJonathan Corbet1-2/+7
Add explicit BKL to vcs_open(). Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20spidev: BKL pushdownJonathan Corbet1-0/+3
Add the BKL to spidev_open(), even though the existing locking looks adequate. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20Add "no BKL needed" comments to several driversJonathan Corbet16-4/+23
This documents the fact that somebody looked at the relevant open() functions and concluded that, due to their trivial nature, no locking was needed. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20tty: cdev lock_kernel() pushdownJonathan Corbet1-2/+25
Parts of the serial code actually BUG() if we don't do this.
2008-06-20snsc: cdev lock_kernel() pushdownJonathan Corbet1-1/+4
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20sound: cdev lock_kernel() pushdownJonathan Corbet2-1/+19
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20fbmem: cdev lock_kernel() pushdownJonathan Corbet1-4/+11
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20dvb: cdev lock_kernel() pushdownJonathan Corbet1-0/+4
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20usbcore: cdev lock_kernel() pushdownJonathan Corbet1-0/+3
usb_open() is protected by a down_read(&minor_rwsem), but I'm not sure I trust it to protect everything including subsidiary open() functions. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20isdn: cdev lock_kernel() pushdownJonathan Corbet1-1/+2
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20gdth: cdev lock_kernel() pushdownJonathan Corbet1-0/+3
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20st: cdev lock_kernel() pushdownJonathan Corbet1-2/+9
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20aacraid: cdev lock_kernel() pushdownJonathan Corbet1-0/+3
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20osst: cdev lock_kernel() pushdown.Jonathan Corbet1-1/+14
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20sg: cdev lock_kernel() pushdownJonathan Corbet1-3/+13
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20ide-tape: cdev lock_kernel() pushdownJonathan Corbet1-1/+6
->release() already has explicit lock_kernel() calls... Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20phonedev: cdev lock_kernel() pushdownJonathan Corbet1-0/+3
phone_open() looks OK, but I don't trust the subsidiary drivers (and ixj in particular). Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20drm: cdev lock_kernel() pushdownJonathan Corbet1-2/+7
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20lp: cdev lock_kernel() pushdownJonathan Corbet1-12/+26
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20mbcs: cdev lock_kernel() pushdownJonathan Corbet1-0/+5
This driver would appear to have no internal locking at all. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20viotape: cdev lock_kernel pushdown ()Jonathan Corbet1-0/+3
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20misc: cdev lock_kernel() pushdownJonathan Corbet1-0/+3
misc_open() looks fine, but who knows what all of the misc drivers are doing in their open() functions? Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20mem: cdev lock_kernel() pushdownJonathan Corbet1-2/+8
It's really hard to tell if this is necessary - lots of weird magic happens by way of map_devmem() Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20ipmi: cdev lock_kernel() pushdownJonathan Corbet1-2/+6
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20UIO: cdev lock_kernel() pushdownJonathan Corbet1-5/+12
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20Input: cdev lock_kernel() pushdownJonathan Corbet1-4/+12
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20HID: cdev lock_kernel() pushdownJonathan Corbet1-0/+3
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20UBI: cdev lock_kernel() pushdownJonathan Corbet1-1/+6
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20mtdchar: cdev lock_kernel() pushdownJonathan Corbet1-6/+16
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20paride: cdev lock_kernel() pushdownJonathan Corbet2-7/+23
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20AoE: cdev lock_kernel() pushdownJonathan Corbet1-1/+6
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2008-06-20drivers/s390: cdev lock_kernel() pushdownJonathan Corbet4-14/+41
Signed-off-by: Jonathan Corbet <corbet@lwn.net>