diff options
| author | Christoph Hellwig <hch@lst.de> | 2008-02-08 07:20:27 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-02-08 12:22:34 -0500 |
| commit | 9261303ab7589cda6a3b95f9f80c9063538dc335 (patch) | |
| tree | 03a46d0bb81dd0d0e56f3f3d73a15a42e698076a | |
| parent | 8b88b0998e35d239e74446cc30f354bdab86df89 (diff) | |
libfs: make simple attributes interruptible
Use mutex_lock_interruptible in simple_attr_read/write.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Cc: <stefano.brivio@polimi.it>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Greg KH <greg@kroah.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | fs/libfs.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/fs/libfs.c b/fs/libfs.c index 2319415ddb5e..d6de56a6e183 100644 --- a/fs/libfs.c +++ b/fs/libfs.c | |||
| @@ -634,7 +634,10 @@ ssize_t simple_attr_read(struct file *file, char __user *buf, | |||
| 634 | if (!attr->get) | 634 | if (!attr->get) |
| 635 | return -EACCES; | 635 | return -EACCES; |
| 636 | 636 | ||
| 637 | mutex_lock(&attr->mutex); | 637 | ret = mutex_lock_interruptible(&attr->mutex); |
| 638 | if (ret) | ||
| 639 | return ret; | ||
| 640 | |||
| 638 | if (*ppos) { /* continued read */ | 641 | if (*ppos) { /* continued read */ |
| 639 | size = strlen(attr->get_buf); | 642 | size = strlen(attr->get_buf); |
| 640 | } else { /* first read */ | 643 | } else { /* first read */ |
| @@ -666,7 +669,10 @@ ssize_t simple_attr_write(struct file *file, const char __user *buf, | |||
| 666 | if (!attr->set) | 669 | if (!attr->set) |
| 667 | return -EACCES; | 670 | return -EACCES; |
| 668 | 671 | ||
| 669 | mutex_lock(&attr->mutex); | 672 | ret = mutex_lock_interruptible(&attr->mutex); |
| 673 | if (ret) | ||
| 674 | return ret; | ||
| 675 | |||
| 670 | ret = -EFAULT; | 676 | ret = -EFAULT; |
| 671 | size = min(sizeof(attr->set_buf) - 1, len); | 677 | size = min(sizeof(attr->set_buf) - 1, len); |
| 672 | if (copy_from_user(attr->set_buf, buf, size)) | 678 | if (copy_from_user(attr->set_buf, buf, size)) |
