linux-kernel - Re: [PATCH] agp: remove read/write stubs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAPM=9tyAmEQq9CZn9FFOxLbE7J9TMkTMu=XHEnzaRFBBByDyew@mail.gmail.com>
Date:	Wed, 9 Jul 2014 08:48:02 +1000
From:	Dave Airlie <airlied@...il.com>
To:	Mathias Krause <minipli@...glemail.com>
Cc:	David Airlie <airlied@...ux.ie>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Alexander Viro <viro@...iv.linux.org.uk>
Subject: Re: [PATCH] agp: remove read/write stubs

On 2 July 2014 16:15, Mathias Krause <minipli@...glemail.com> wrote:
> On 15 June 2014 23:02, Mathias Krause <minipli@...glemail.com> wrote:
>> The VFS layer handles those in the very same way, if unset. No need for
>> additional stubs.
>>
>> Signed-off-by: Mathias Krause <minipli@...glemail.com>
>> Cc: Alexander Viro <viro@...iv.linux.org.uk>
>> ---
>>  drivers/char/agp/frontend.c |   15 ---------------
>>  1 file changed, 15 deletions(-)
>>
>> diff --git a/drivers/char/agp/frontend.c b/drivers/char/agp/frontend.c
>> index b29703324e..09f17eb734 100644
>> --- a/drivers/char/agp/frontend.c
>> +++ b/drivers/char/agp/frontend.c
>> @@ -710,19 +710,6 @@ static int agp_open(struct inode *inode, struct file *file)
>>         return 0;
>>  }
>>
>> -
>> -static ssize_t agp_read(struct file *file, char __user *buf,
>> -                       size_t count, loff_t * ppos)
>> -{
>> -       return -EINVAL;
>> -}
>> -
>> -static ssize_t agp_write(struct file *file, const char __user *buf,
>> -                        size_t count, loff_t * ppos)
>> -{
>> -       return -EINVAL;
>> -}
>> -
>>  static int agpioc_info_wrap(struct agp_file_private *priv, void __user *arg)
>>  {
>>         struct agp_info userinfo;
>> @@ -1047,8 +1034,6 @@ static const struct file_operations agp_fops =
>>  {
>>         .owner          = THIS_MODULE,
>>         .llseek         = no_llseek,
>> -       .read           = agp_read,
>> -       .write          = agp_write,
>>         .unlocked_ioctl = agp_ioctl,
>>  #ifdef CONFIG_COMPAT
>>         .compat_ioctl   = compat_agp_ioctl,
>> --
>> 1.7.10.4
>>
>
> Ping. Any objections, comments? David?

Are you sure VFS handles this the same way? can you point me where it
checks ->read and returns -EINVAL, I'm probably just missing it in my
quick look over fs/read_write.c

Dave.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/