2.4.0 - lseek on /proc broken? [with patch]

From: Salvador Ortiz Garcia (sog@msg.com.mx)
Date: Tue Jan 16 2001 - 01:53:47 EST


Hi:

After diging around for some problems (shutdown/unmount related) I found
that some processes where hidden from ps, pidoff, ls /proc, etc.

A strace reveled that:

open("/proc", O_RDONLY|O_NONBLOCK|O_DIRECTORY) = 7
fstat(7, {st_mode=S_IFDIR|0555, st_size=0, ...}) = 0
fcntl(7, F_SETFD, FD_CLOEXEC) = 0
getdents(7, /* 58 entries */, 984) = 980
lseek(7, 265, SEEK_SET) = -1 EINVAL (Invalid argument)

So I change proc_root_operations to use proc_file_lseek and the problems
vanished.

Comments?

Salvador Ortiz.
please CCs to me.

 
=========== cut ===========
diff -u linux/fs/proc/generic.c linux-2.4.0-ac7/fs/proc/generic.c
--- linux/fs/proc/generic.c Mon Dec 11 15:45:42 2000
+++ linux-2.4.0-msg/fs/proc/generic.c Tue Jan 16 00:05:24 2001
@@ -22,7 +22,7 @@
                               size_t nbytes, loff_t *ppos);
 static ssize_t proc_file_write(struct file * file, const char * buffer,
                                size_t count, loff_t *ppos);
-static loff_t proc_file_lseek(struct file *, loff_t, int);
+loff_t proc_file_lseek(struct file *, loff_t, int);
 
 int proc_match(int len, const char *name,struct proc_dir_entry * de)
 {
@@ -137,7 +137,7 @@
 }
 
 
-static loff_t
+loff_t
 proc_file_lseek(struct file * file, loff_t offset, int orig)
 {
     switch (orig) {
diff -u linux/fs/proc/root.c linux-2.4.0-ac7/fs/proc/root.c
--- linux/fs/proc/root.c Thu Nov 23 11:07:36 2000
+++ linux-2.4.0-msg/fs/proc/root.c Tue Jan 16 00:05:27 2001
@@ -81,7 +81,9 @@
  * <pid> directories. Thus we don't use the generic
  * directory handling functions for that..
  */
+extern loff_t proc_file_lseek(struct file *, loff_t, int);
 static struct file_operations proc_root_operations = {
+ llseek: proc_file_lseek,
         read: generic_read_dir,
         readdir: proc_root_readdir,
 };

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



This archive was generated by hypermail 2b29 : Tue Jan 23 2001 - 21:00:12 EST