proc: unsigned file descriptors
authorAlexey Dobriyan <adobriyan@gmail.com>
Thu, 1 Sep 2016 21:42:02 +0000 (00:42 +0300)
committerAl Viro <viro@zeniv.linux.org.uk>
Tue, 27 Sep 2016 22:47:38 +0000 (18:47 -0400)
Make struct proc_inode::fd unsigned.

This allows better code generation on x86_64 (less sign extensions).

Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/proc/fd.c
fs/proc/fd.h
fs/proc/internal.h

index 01df23cc81f62d4a6d1c3aadcb1b48495b0d7917..d21dafef31029deb345e98b36fe133dc73599e00 100644 (file)
@@ -31,7 +31,7 @@ static int seq_show(struct seq_file *m, void *v)
        put_task_struct(task);
 
        if (files) {
-               int fd = proc_fd(m->private);
+               unsigned int fd = proc_fd(m->private);
 
                spin_lock(&files->file_lock);
                file = fcheck_files(files, fd);
@@ -86,7 +86,7 @@ static int tid_fd_revalidate(struct dentry *dentry, unsigned int flags)
        struct task_struct *task;
        const struct cred *cred;
        struct inode *inode;
-       int fd;
+       unsigned int fd;
 
        if (flags & LOOKUP_RCU)
                return -ECHILD;
@@ -158,7 +158,7 @@ static int proc_fd_link(struct dentry *dentry, struct path *path)
        }
 
        if (files) {
-               int fd = proc_fd(d_inode(dentry));
+               unsigned int fd = proc_fd(d_inode(dentry));
                struct file *fd_file;
 
                spin_lock(&files->file_lock);
@@ -253,7 +253,7 @@ static int proc_readfd_common(struct file *file, struct dir_context *ctx,
                        continue;
                rcu_read_unlock();
 
-               len = snprintf(name, sizeof(name), "%d", fd);
+               len = snprintf(name, sizeof(name), "%u", fd);
                if (!proc_fill_cache(file, ctx,
                                     name, len, instantiate, p,
                                     (void *)(unsigned long)fd))
index 7c047f256ae2ccb945aab10d8688d6305b5a0eb7..46dafadd0083a5b59fe7c7a5bc6c8d33282a74be 100644 (file)
@@ -11,7 +11,7 @@ extern const struct inode_operations proc_fdinfo_inode_operations;
 
 extern int proc_fd_permission(struct inode *inode, int mask);
 
-static inline int proc_fd(struct inode *inode)
+static inline unsigned int proc_fd(struct inode *inode)
 {
        return PROC_I(inode)->fd;
 }
index 7931c558c19250ab87fd911b6c09ff359d4d05e1..5378441ec1b7bccb0c98a6a5ce077be9b4da921f 100644 (file)
@@ -60,7 +60,7 @@ union proc_op {
 
 struct proc_inode {
        struct pid *pid;
-       int fd;
+       unsigned int fd;
        union proc_op op;
        struct proc_dir_entry *pde;
        struct ctl_table_header *sysctl;