diff options
author | Mathieu Segaud <mathieu.segaud@regala.cx> | 2008-04-18 13:29:38 -0700 |
---|---|---|
committer | Dave Airlie <airlied@linux.ie> | 2008-04-26 18:43:20 +1000 |
commit | 09aa356b5584090aab6810ec8002936d710cd4ac (patch) | |
tree | 3a36423ce815c31b72a062e674071ec2cd51c011 /drivers/char | |
parent | 4ab92bcf773e7b9e1367897047d5fa4d151d9e90 (diff) | |
download | blackbird-op-linux-09aa356b5584090aab6810ec8002936d710cd4ac.tar.gz blackbird-op-linux-09aa356b5584090aab6810ec8002936d710cd4ac.zip |
agp: convert drivers/char/agp/frontend.c to use unlocked_ioctl
As of now, agp_compat_ioctl already runs without the BKL. Mutual exclusion
is enforced by agp_fe.agp_mutex in agp_ioctl() and agp_compat_ioctl().
Apply the same locking rationale to the two functions allowing BKL cleanup.
Signed-off-by: Mathieu Segaud <mathieu.segaud@regala.cx>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Dave Airlie <airlied@linux.ie>
Diffstat (limited to 'drivers/char')
-rw-r--r-- | drivers/char/agp/frontend.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/char/agp/frontend.c b/drivers/char/agp/frontend.c index 55d7a82bd071..857b26227d87 100644 --- a/drivers/char/agp/frontend.c +++ b/drivers/char/agp/frontend.c @@ -967,7 +967,7 @@ int agpioc_chipset_flush_wrap(struct agp_file_private *priv) return 0; } -static int agp_ioctl(struct inode *inode, struct file *file, +static long agp_ioctl(struct file *file, unsigned int cmd, unsigned long arg) { struct agp_file_private *curr_priv = file->private_data; @@ -1058,7 +1058,7 @@ static const struct file_operations agp_fops = .llseek = no_llseek, .read = agp_read, .write = agp_write, - .ioctl = agp_ioctl, + .unlocked_ioctl = agp_ioctl, #ifdef CONFIG_COMPAT .compat_ioctl = compat_agp_ioctl, #endif |