diff options
author | Mathieu Segaud <mathieu.segaud@regala.cx> | 2008-04-18 16:29:38 -0400 |
---|---|---|
committer | Dave Airlie <airlied@linux.ie> | 2008-04-26 04:43:20 -0400 |
commit | 09aa356b5584090aab6810ec8002936d710cd4ac (patch) | |
tree | 3a36423ce815c31b72a062e674071ec2cd51c011 /drivers/char | |
parent | 4ab92bcf773e7b9e1367897047d5fa4d151d9e90 (diff) |
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) | |||
967 | return 0; | 967 | return 0; |
968 | } | 968 | } |
969 | 969 | ||
970 | static int agp_ioctl(struct inode *inode, struct file *file, | 970 | static long agp_ioctl(struct file *file, |
971 | unsigned int cmd, unsigned long arg) | 971 | unsigned int cmd, unsigned long arg) |
972 | { | 972 | { |
973 | struct agp_file_private *curr_priv = file->private_data; | 973 | struct agp_file_private *curr_priv = file->private_data; |
@@ -1058,7 +1058,7 @@ static const struct file_operations agp_fops = | |||
1058 | .llseek = no_llseek, | 1058 | .llseek = no_llseek, |
1059 | .read = agp_read, | 1059 | .read = agp_read, |
1060 | .write = agp_write, | 1060 | .write = agp_write, |
1061 | .ioctl = agp_ioctl, | 1061 | .unlocked_ioctl = agp_ioctl, |
1062 | #ifdef CONFIG_COMPAT | 1062 | #ifdef CONFIG_COMPAT |
1063 | .compat_ioctl = compat_agp_ioctl, | 1063 | .compat_ioctl = compat_agp_ioctl, |
1064 | #endif | 1064 | #endif |