lkml.org 
[lkml]   [2008]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectBUG: lock held when returning to user space
Hi Ingo,

we provided a device driver vmur dealing with z/VM virtual unit record
devices (reader, punch, printer). A corresponding user space tool
provides functions similar to the CMS commands RECEIVE, PUNCH, PRINT.
Unit record devices are not meant for concurrent read or write by
multiple users, that's why we need to serialize access. The driver's
open method uses mutex_trylock or mutex_lock_interruptible to ensure
exclusive access to the device, while its release method uses
mutex_unlock.

As a consequence, lockdep complains about locks being held when
returning to user space. We used a very simple char device driver
(appended below) to produce this message:
================================================
[ BUG: lock held when returning to user space! ]
------------------------------------------------
testapp/2683 is leaving the kernel with locks still held!
1 lock held by testapp/2683:
#0: (&test_mutex){--..}, at: [<000003e00003316c>] test_open+0x30/0x64
[test]

For the vmur device driver it is crucial to have only one process access
a given unit record device node at a given time. So having open hold the
mutex and return to user space is exactly what we want. Is there any
annotation to tell lockdep to suppress or bypass this kind of warning?

Thanks in advance,

Frank

#include <linux/module.h>
#include <linux/init.h>
#include <linux/fs.h>
#include <linux/types.h>
#include <linux/cdev.h>

#define DEVICE_NAME "test"
#define MODULE_NAME "test"

static dev_t Devno;
static struct cdev test_cdev;
static struct mutex test_mutex;

static ssize_t test_read(struct file *filp, char *buff, size_t len,
loff_t *whence)
{
return 1;
}

static ssize_t test_write(struct file *filp, const char *buff,
size_t len, loff_t *whence)
{
return 1;
}

static int test_open(struct inode *ino, struct file *filp)
{
if (mutex_lock_interruptible(&test_mutex))
return -ERESTARTSYS;
printk("%s: test device is open.\n", MODULE_NAME);
return 0;
}

static int test_close(struct inode *ino, struct file *filp)
{
printk("%s: test device is closed.\n", MODULE_NAME);
mutex_unlock(&test_mutex);
return 0;
}

static struct file_operations Fops = { .owner = THIS_MODULE,
.read = test_read,
.write = test_write,
.open = test_open,
.release = test_close };

static int __init test_init(void)
{
int rc;
rc = alloc_chrdev_region(&Devno, 0, 1, DEVICE_NAME);
if (rc < 0)
{
printk("%s: Registration failed...\n",MODULE_NAME);
return rc;
}
printk("%s: Registration %s at major number %d\n", MODULE_NAME,
DEVICE_NAME, MAJOR(Devno));
cdev_init(&test_cdev, &Fops);
test_cdev.owner = THIS_MODULE;
rc = cdev_add(&test_cdev, Devno, 1);
if (rc < 0) {
printk("%s: Device object not added!\n", MODULE_NAME);
unregister_chrdev_region(Devno, 1);
return rc;
}
printk("%s: Device added.\n", MODULE_NAME);
mutex_init(&test_mutex);
return 0;
}

static void __exit test_exit(void)
{
cdev_del(&test_cdev);
unregister_chrdev_region(Devno, 1);
printk("%s: Module removed.\n", MODULE_NAME);
}

module_init(test_init);
module_exit(test_exit);

MODULE_LICENSE("GPL");
MODULE_AUTHOR("Frank");
MODULE_DESCRIPTION("Simple char driver.");
\
 
 \ /
  Last update: 2008-03-12 16:45    [W:0.573 / U:0.352 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site