diff options
author | Andi Kleen <ak@suse.de> | 2008-02-08 07:21:23 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-02-08 12:22:38 -0500 |
commit | d59d0b1b88c5b0f9cec219d236758d8882a59d6b (patch) | |
tree | e8af344ea9a75e25f583e2c90eb98469b8dcfaca | |
parent | de5c9edee7a3cfdc6dd1a31c4794dc41ef3c70f9 (diff) |
BKL-Removal: convert pipe to use unlocked_ioctl too
No BKL needed in pipe_ioctl
Signed-off-by: Andi Kleen <ak@suse.de>
Cc: Jens Axboe <jens.axboe@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | fs/pipe.c | 16 |
1 files changed, 7 insertions, 9 deletions
@@ -576,9 +576,7 @@ bad_pipe_w(struct file *filp, const char __user *buf, size_t count, | |||
576 | return -EBADF; | 576 | return -EBADF; |
577 | } | 577 | } |
578 | 578 | ||
579 | static int | 579 | static long pipe_ioctl(struct file *filp, unsigned int cmd, unsigned long arg) |
580 | pipe_ioctl(struct inode *pino, struct file *filp, | ||
581 | unsigned int cmd, unsigned long arg) | ||
582 | { | 580 | { |
583 | struct inode *inode = filp->f_path.dentry->d_inode; | 581 | struct inode *inode = filp->f_path.dentry->d_inode; |
584 | struct pipe_inode_info *pipe; | 582 | struct pipe_inode_info *pipe; |
@@ -785,7 +783,7 @@ const struct file_operations read_fifo_fops = { | |||
785 | .aio_read = pipe_read, | 783 | .aio_read = pipe_read, |
786 | .write = bad_pipe_w, | 784 | .write = bad_pipe_w, |
787 | .poll = pipe_poll, | 785 | .poll = pipe_poll, |
788 | .ioctl = pipe_ioctl, | 786 | .unlocked_ioctl = pipe_ioctl, |
789 | .open = pipe_read_open, | 787 | .open = pipe_read_open, |
790 | .release = pipe_read_release, | 788 | .release = pipe_read_release, |
791 | .fasync = pipe_read_fasync, | 789 | .fasync = pipe_read_fasync, |
@@ -797,7 +795,7 @@ const struct file_operations write_fifo_fops = { | |||
797 | .write = do_sync_write, | 795 | .write = do_sync_write, |
798 | .aio_write = pipe_write, | 796 | .aio_write = pipe_write, |
799 | .poll = pipe_poll, | 797 | .poll = pipe_poll, |
800 | .ioctl = pipe_ioctl, | 798 | .unlocked_ioctl = pipe_ioctl, |
801 | .open = pipe_write_open, | 799 | .open = pipe_write_open, |
802 | .release = pipe_write_release, | 800 | .release = pipe_write_release, |
803 | .fasync = pipe_write_fasync, | 801 | .fasync = pipe_write_fasync, |
@@ -810,7 +808,7 @@ const struct file_operations rdwr_fifo_fops = { | |||
810 | .write = do_sync_write, | 808 | .write = do_sync_write, |
811 | .aio_write = pipe_write, | 809 | .aio_write = pipe_write, |
812 | .poll = pipe_poll, | 810 | .poll = pipe_poll, |
813 | .ioctl = pipe_ioctl, | 811 | .unlocked_ioctl = pipe_ioctl, |
814 | .open = pipe_rdwr_open, | 812 | .open = pipe_rdwr_open, |
815 | .release = pipe_rdwr_release, | 813 | .release = pipe_rdwr_release, |
816 | .fasync = pipe_rdwr_fasync, | 814 | .fasync = pipe_rdwr_fasync, |
@@ -822,7 +820,7 @@ static const struct file_operations read_pipe_fops = { | |||
822 | .aio_read = pipe_read, | 820 | .aio_read = pipe_read, |
823 | .write = bad_pipe_w, | 821 | .write = bad_pipe_w, |
824 | .poll = pipe_poll, | 822 | .poll = pipe_poll, |
825 | .ioctl = pipe_ioctl, | 823 | .unlocked_ioctl = pipe_ioctl, |
826 | .open = pipe_read_open, | 824 | .open = pipe_read_open, |
827 | .release = pipe_read_release, | 825 | .release = pipe_read_release, |
828 | .fasync = pipe_read_fasync, | 826 | .fasync = pipe_read_fasync, |
@@ -834,7 +832,7 @@ static const struct file_operations write_pipe_fops = { | |||
834 | .write = do_sync_write, | 832 | .write = do_sync_write, |
835 | .aio_write = pipe_write, | 833 | .aio_write = pipe_write, |
836 | .poll = pipe_poll, | 834 | .poll = pipe_poll, |
837 | .ioctl = pipe_ioctl, | 835 | .unlocked_ioctl = pipe_ioctl, |
838 | .open = pipe_write_open, | 836 | .open = pipe_write_open, |
839 | .release = pipe_write_release, | 837 | .release = pipe_write_release, |
840 | .fasync = pipe_write_fasync, | 838 | .fasync = pipe_write_fasync, |
@@ -847,7 +845,7 @@ static const struct file_operations rdwr_pipe_fops = { | |||
847 | .write = do_sync_write, | 845 | .write = do_sync_write, |
848 | .aio_write = pipe_write, | 846 | .aio_write = pipe_write, |
849 | .poll = pipe_poll, | 847 | .poll = pipe_poll, |
850 | .ioctl = pipe_ioctl, | 848 | .unlocked_ioctl = pipe_ioctl, |
851 | .open = pipe_rdwr_open, | 849 | .open = pipe_rdwr_open, |
852 | .release = pipe_rdwr_release, | 850 | .release = pipe_rdwr_release, |
853 | .fasync = pipe_rdwr_fasync, | 851 | .fasync = pipe_rdwr_fasync, |