diff options
| author | Jonathan Corbet <corbet@lwn.net> | 2008-05-16 13:40:30 -0600 |
|---|---|---|
| committer | Jonathan Corbet <corbet@lwn.net> | 2008-06-20 14:05:50 -0600 |
| commit | d21c95c569c462da20d491b75d0a45bd70ddc1bf (patch) | |
| tree | c5efcd4a1ae50101a56abc1f5a27ef5173da209d /drivers/char/dtlk.c | |
| parent | 39d95b9d857ad9ed335dd1a2d6c6de1f1ee69ce1 (diff) | |
| download | op-kernel-dev-d21c95c569c462da20d491b75d0a45bd70ddc1bf.zip op-kernel-dev-d21c95c569c462da20d491b75d0a45bd70ddc1bf.tar.gz | |
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 abde6dd..433388c 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"); |
