diff options
| author | Jonathan Corbet <corbet@lwn.net> | 2008-05-16 21:40:30 +0200 |
|---|---|---|
| committer | Jonathan Corbet <corbet@lwn.net> | 2008-06-20 22:05:50 +0200 |
| commit | d21c95c569c462da20d491b75d0a45bd70ddc1bf (patch) | |
| tree | c5efcd4a1ae50101a56abc1f5a27ef5173da209d /drivers/char/dtlk.c | |
| parent | tty: cdev lock_kernel() pushdown (diff) | |
| download | linux-d21c95c569c462da20d491b75d0a45bd70ddc1bf.tar.xz linux-d21c95c569c462da20d491b75d0a45bd70ddc1bf.zip | |
Add "no BKL needed" comments to several drivers
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>
Diffstat (limited to 'drivers/char/dtlk.c')
| -rw-r--r-- | drivers/char/dtlk.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/char/dtlk.c b/drivers/char/dtlk.c index abde6ddefe69..433388c60235 100644 --- a/drivers/char/dtlk.c +++ b/drivers/char/dtlk.c @@ -288,6 +288,8 @@ static int dtlk_ioctl(struct inode *inode, } } +/* No BKL needed here; "dtlk_busy" is the only global resource, + and it is not ever set by anybody (test is broken) */ static int dtlk_open(struct inode *inode, struct file *file) { TRACE_TEXT("(dtlk_open"); |