linux-kernel - Re: [PATCH] fs: kill default

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <1850345881.414857.1304964238493.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com>
Date:	Mon, 9 May 2011 14:03:58 -0400 (EDT)
From:	Dave Anderson <anderson@...hat.com>
To:	linux-kernel@...r.kernel.org
Cc:	Josef Bacik <jbacik@...hat.com>
Subject: Re: [PATCH] fs: kill default_llseek


On Thursday 05 May 2011 16:27:57 Josef Bacik wrote:
> Looking at this llseek stuff I noticed that default_llseek is the exact same as
> generic_file_llseek, so kill default_llseek.  I patched this using spatch with
> just a simple
> 
> @@
> @@
> 
> - default_llseek
> + generic_file_llseek

...

> diff --git a/fs/proc/kcore.c b/fs/proc/kcore.c
> index d245cb2..6f37c39 100644
> --- a/fs/proc/kcore.c
> +++ b/fs/proc/kcore.c
> @@ -558,7 +558,7 @@ static int open_kcore(struct inode *inode, struct file *filp)
>  static const struct file_operations proc_kcore_operations = {
>  	.read		= read_kcore,
>  	.open		= open_kcore,
> -	.llseek		= default_llseek,
> +	.llseek		= generic_file_llseek,
>  };
 
> diff --git a/fs/proc/vmcore.c b/fs/proc/vmcore.c
> index 74802bc5..0cafd9e 100644
> --- a/fs/proc/vmcore.c
> +++ b/fs/proc/vmcore.c
> @@ -163,7 +163,7 @@ static ssize_t read_vmcore(struct file *file, char __user *buffer,
>  
>  static const struct file_operations proc_vmcore_operations = {
>  	.read		= read_vmcore,
> -	.llseek		= default_llseek,
> +	.llseek		= generic_file_llseek,
>  };
 
Both /proc/kcore and /proc/vmcore currently require default_llseek().  
They were both changed to use generic_file_llseek(), but then subsequently
reverted back to default_llseek():

commit c227e69028473c7c7994a9b0a2cc0034f3f7e0fe
Author: Arnd Bergmann <arnd@...db.de>
Date:   Wed Sep 22 13:04:54 2010 -0700

    /proc/vmcore: fix seeking
    
    Commit 73296bc611 ("procfs: Use generic_file_llseek in /proc/vmcore")
    broke seeking on /proc/vmcore.  This changes it back to use default_llseek
    in order to restore the original behaviour.
    ...


commit ceff1a770933e2ca2bf995b453dade4ec47a9878
Author: Dave Anderson <anderson@...hat.com>
Date:   Wed Jan 12 17:00:36 2011 -0800

    /proc/kcore: fix seeking
    
    Commit 34aacb2920 ("procfs: Use generic_file_llseek in /proc/kcore") broke
    seeking on /proc/kcore.  This changes it back to use default_llseek in
    order to restore the original behavior.
    ...

Dave Anderson
--
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/