Commit c9c554f2 authored by Al Viro's avatar Al Viro

alloc_file(): switch to passing O_... flags instead of FMODE_... mode

... so that it could set both ->f_flags and ->f_mode, without callers
having to set ->f_flags manually.
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 6b4e8085
...@@ -102,12 +102,11 @@ static struct file *cxl_getfile(const char *name, ...@@ -102,12 +102,11 @@ static struct file *cxl_getfile(const char *name,
path.mnt = mntget(cxl_vfs_mount); path.mnt = mntget(cxl_vfs_mount);
d_instantiate(path.dentry, inode); d_instantiate(path.dentry, inode);
file = alloc_file(&path, OPEN_FMODE(flags), fops); file = alloc_file(&path, flags & (O_ACCMODE | O_NONBLOCK), fops);
if (IS_ERR(file)) { if (IS_ERR(file)) {
path_put(&path); path_put(&path);
goto err_fs; goto err_fs;
} }
file->f_flags = flags & (O_ACCMODE | O_NONBLOCK);
file->private_data = priv; file->private_data = priv;
return file; return file;
......
...@@ -129,7 +129,7 @@ static struct file *ocxlflash_getfile(struct device *dev, const char *name, ...@@ -129,7 +129,7 @@ static struct file *ocxlflash_getfile(struct device *dev, const char *name,
path.mnt = mntget(ocxlflash_vfs_mount); path.mnt = mntget(ocxlflash_vfs_mount);
d_instantiate(path.dentry, inode); d_instantiate(path.dentry, inode);
file = alloc_file(&path, OPEN_FMODE(flags), fops); file = alloc_file(&path, flags & (O_ACCMODE | O_NONBLOCK), fops);
if (IS_ERR(file)) { if (IS_ERR(file)) {
rc = PTR_ERR(file); rc = PTR_ERR(file);
dev_err(dev, "%s: alloc_file failed rc=%d\n", dev_err(dev, "%s: alloc_file failed rc=%d\n",
...@@ -138,7 +138,6 @@ static struct file *ocxlflash_getfile(struct device *dev, const char *name, ...@@ -138,7 +138,6 @@ static struct file *ocxlflash_getfile(struct device *dev, const char *name,
goto err3; goto err3;
} }
file->f_flags = flags & (O_ACCMODE | O_NONBLOCK);
file->private_data = priv; file->private_data = priv;
out: out:
return file; return file;
......
...@@ -234,14 +234,10 @@ static struct file *aio_private_file(struct kioctx *ctx, loff_t nr_pages) ...@@ -234,14 +234,10 @@ static struct file *aio_private_file(struct kioctx *ctx, loff_t nr_pages)
path.mnt = mntget(aio_mnt); path.mnt = mntget(aio_mnt);
d_instantiate(path.dentry, inode); d_instantiate(path.dentry, inode);
file = alloc_file(&path, FMODE_READ | FMODE_WRITE, &aio_ring_fops); file = alloc_file(&path, O_RDWR, &aio_ring_fops);
if (IS_ERR(file)) { if (IS_ERR(file))
path_put(&path); path_put(&path);
return file; return file;
}
file->f_flags = O_RDWR;
return file;
} }
static struct dentry *aio_mount(struct file_system_type *fs_type, static struct dentry *aio_mount(struct file_system_type *fs_type,
......
...@@ -102,12 +102,11 @@ struct file *anon_inode_getfile(const char *name, ...@@ -102,12 +102,11 @@ struct file *anon_inode_getfile(const char *name,
d_instantiate(path.dentry, anon_inode_inode); d_instantiate(path.dentry, anon_inode_inode);
file = alloc_file(&path, OPEN_FMODE(flags), fops); file = alloc_file(&path, flags & (O_ACCMODE | O_NONBLOCK), fops);
if (IS_ERR(file)) if (IS_ERR(file))
goto err_dput; goto err_dput;
file->f_mapping = anon_inode_inode->i_mapping; file->f_mapping = anon_inode_inode->i_mapping;
file->f_flags = flags & (O_ACCMODE | O_NONBLOCK);
file->private_data = priv; file->private_data = priv;
return file; return file;
......
...@@ -153,10 +153,10 @@ struct file *get_empty_filp(void) ...@@ -153,10 +153,10 @@ struct file *get_empty_filp(void)
* alloc_file - allocate and initialize a 'struct file' * alloc_file - allocate and initialize a 'struct file'
* *
* @path: the (dentry, vfsmount) pair for the new file * @path: the (dentry, vfsmount) pair for the new file
* @mode: the mode with which the new file will be opened * @flags: O_... flags with which the new file will be opened
* @fop: the 'struct file_operations' for the new file * @fop: the 'struct file_operations' for the new file
*/ */
struct file *alloc_file(const struct path *path, fmode_t mode, struct file *alloc_file(const struct path *path, int flags,
const struct file_operations *fop) const struct file_operations *fop)
{ {
struct file *file; struct file *file;
...@@ -165,19 +165,20 @@ struct file *alloc_file(const struct path *path, fmode_t mode, ...@@ -165,19 +165,20 @@ struct file *alloc_file(const struct path *path, fmode_t mode,
if (IS_ERR(file)) if (IS_ERR(file))
return file; return file;
file->f_mode = OPEN_FMODE(flags);
file->f_flags = flags;
file->f_path = *path; file->f_path = *path;
file->f_inode = path->dentry->d_inode; file->f_inode = path->dentry->d_inode;
file->f_mapping = path->dentry->d_inode->i_mapping; file->f_mapping = path->dentry->d_inode->i_mapping;
file->f_wb_err = filemap_sample_wb_err(file->f_mapping); file->f_wb_err = filemap_sample_wb_err(file->f_mapping);
if ((mode & FMODE_READ) && if ((file->f_mode & FMODE_READ) &&
likely(fop->read || fop->read_iter)) likely(fop->read || fop->read_iter))
mode |= FMODE_CAN_READ; file->f_mode |= FMODE_CAN_READ;
if ((mode & FMODE_WRITE) && if ((file->f_mode & FMODE_WRITE) &&
likely(fop->write || fop->write_iter)) likely(fop->write || fop->write_iter))
mode |= FMODE_CAN_WRITE; file->f_mode |= FMODE_CAN_WRITE;
file->f_mode = mode;
file->f_op = fop; file->f_op = fop;
if ((mode & (FMODE_READ | FMODE_WRITE)) == FMODE_READ) if ((file->f_mode & (FMODE_READ | FMODE_WRITE)) == FMODE_READ)
i_readcount_inc(path->dentry->d_inode); i_readcount_inc(path->dentry->d_inode);
return file; return file;
} }
......
...@@ -1375,8 +1375,7 @@ struct file *hugetlb_file_setup(const char *name, size_t size, ...@@ -1375,8 +1375,7 @@ struct file *hugetlb_file_setup(const char *name, size_t size,
inode->i_size = size; inode->i_size = size;
clear_nlink(inode); clear_nlink(inode);
file = alloc_file(&path, FMODE_WRITE | FMODE_READ, file = alloc_file(&path, O_RDWR, &hugetlbfs_file_operations);
&hugetlbfs_file_operations);
if (IS_ERR(file)) if (IS_ERR(file))
goto out_dentry; /* inode is already attached */ goto out_dentry; /* inode is already attached */
......
...@@ -760,16 +760,17 @@ int create_pipe_files(struct file **res, int flags) ...@@ -760,16 +760,17 @@ int create_pipe_files(struct file **res, int flags)
d_instantiate(path.dentry, inode); d_instantiate(path.dentry, inode);
f = alloc_file(&path, FMODE_WRITE, &pipefifo_fops); f = alloc_file(&path, O_WRONLY | (flags & (O_NONBLOCK | O_DIRECT)),
&pipefifo_fops);
if (IS_ERR(f)) { if (IS_ERR(f)) {
err = PTR_ERR(f); err = PTR_ERR(f);
goto err_dentry; goto err_dentry;
} }
f->f_flags = O_WRONLY | (flags & (O_NONBLOCK | O_DIRECT));
f->private_data = inode->i_pipe; f->private_data = inode->i_pipe;
res[0] = alloc_file(&path, FMODE_READ, &pipefifo_fops); res[0] = alloc_file(&path, O_RDONLY | (flags & O_NONBLOCK),
&pipefifo_fops);
if (IS_ERR(res[0])) { if (IS_ERR(res[0])) {
put_pipe_info(inode, inode->i_pipe); put_pipe_info(inode, inode->i_pipe);
fput(f); fput(f);
...@@ -778,7 +779,6 @@ int create_pipe_files(struct file **res, int flags) ...@@ -778,7 +779,6 @@ int create_pipe_files(struct file **res, int flags)
path_get(&path); path_get(&path);
res[0]->private_data = inode->i_pipe; res[0]->private_data = inode->i_pipe;
res[0]->f_flags = O_RDONLY | (flags & O_NONBLOCK);
res[1] = f; res[1] = f;
return 0; return 0;
......
...@@ -18,7 +18,7 @@ struct file_operations; ...@@ -18,7 +18,7 @@ struct file_operations;
struct vfsmount; struct vfsmount;
struct dentry; struct dentry;
struct path; struct path;
extern struct file *alloc_file(const struct path *, fmode_t mode, extern struct file *alloc_file(const struct path *, int flags,
const struct file_operations *fop); const struct file_operations *fop);
static inline void fput_light(struct file *file, int fput_needed) static inline void fput_light(struct file *file, int fput_needed)
......
...@@ -1362,7 +1362,7 @@ long do_shmat(int shmid, char __user *shmaddr, int shmflg, ...@@ -1362,7 +1362,7 @@ long do_shmat(int shmid, char __user *shmaddr, int shmflg,
struct ipc_namespace *ns; struct ipc_namespace *ns;
struct shm_file_data *sfd; struct shm_file_data *sfd;
struct path path; struct path path;
fmode_t f_mode; int f_flags;
unsigned long populate = 0; unsigned long populate = 0;
err = -EINVAL; err = -EINVAL;
...@@ -1395,11 +1395,11 @@ long do_shmat(int shmid, char __user *shmaddr, int shmflg, ...@@ -1395,11 +1395,11 @@ long do_shmat(int shmid, char __user *shmaddr, int shmflg,
if (shmflg & SHM_RDONLY) { if (shmflg & SHM_RDONLY) {
prot = PROT_READ; prot = PROT_READ;
acc_mode = S_IRUGO; acc_mode = S_IRUGO;
f_mode = FMODE_READ; f_flags = O_RDONLY;
} else { } else {
prot = PROT_READ | PROT_WRITE; prot = PROT_READ | PROT_WRITE;
acc_mode = S_IRUGO | S_IWUGO; acc_mode = S_IRUGO | S_IWUGO;
f_mode = FMODE_READ | FMODE_WRITE; f_flags = O_RDWR;
} }
if (shmflg & SHM_EXEC) { if (shmflg & SHM_EXEC) {
prot |= PROT_EXEC; prot |= PROT_EXEC;
...@@ -1449,7 +1449,7 @@ long do_shmat(int shmid, char __user *shmaddr, int shmflg, ...@@ -1449,7 +1449,7 @@ long do_shmat(int shmid, char __user *shmaddr, int shmflg,
goto out_nattch; goto out_nattch;
} }
file = alloc_file(&path, f_mode, file = alloc_file(&path, f_flags,
is_file_hugepages(shp->shm_file) ? is_file_hugepages(shp->shm_file) ?
&shm_file_operations_huge : &shm_file_operations_huge :
&shm_file_operations); &shm_file_operations);
......
...@@ -326,7 +326,7 @@ SYSCALL_DEFINE2(memfd_create, ...@@ -326,7 +326,7 @@ SYSCALL_DEFINE2(memfd_create,
goto err_fd; goto err_fd;
} }
file->f_mode |= FMODE_LSEEK | FMODE_PREAD | FMODE_PWRITE; file->f_mode |= FMODE_LSEEK | FMODE_PREAD | FMODE_PWRITE;
file->f_flags |= O_RDWR | O_LARGEFILE; file->f_flags |= O_LARGEFILE;
if (flags & MFD_ALLOW_SEALING) { if (flags & MFD_ALLOW_SEALING) {
file_seals = memfd_file_seals_ptr(file); file_seals = memfd_file_seals_ptr(file);
......
...@@ -3942,8 +3942,7 @@ static struct file *__shmem_file_setup(struct vfsmount *mnt, const char *name, l ...@@ -3942,8 +3942,7 @@ static struct file *__shmem_file_setup(struct vfsmount *mnt, const char *name, l
if (IS_ERR(res)) if (IS_ERR(res))
goto put_path; goto put_path;
res = alloc_file(&path, FMODE_WRITE | FMODE_READ, res = alloc_file(&path, O_RDWR, &shmem_file_operations);
&shmem_file_operations);
if (IS_ERR(res)) if (IS_ERR(res))
goto put_path; goto put_path;
......
...@@ -411,7 +411,7 @@ struct file *sock_alloc_file(struct socket *sock, int flags, const char *dname) ...@@ -411,7 +411,7 @@ struct file *sock_alloc_file(struct socket *sock, int flags, const char *dname)
d_instantiate(path.dentry, SOCK_INODE(sock)); d_instantiate(path.dentry, SOCK_INODE(sock));
file = alloc_file(&path, FMODE_READ | FMODE_WRITE, file = alloc_file(&path, O_RDWR | (flags & O_NONBLOCK),
&socket_file_ops); &socket_file_ops);
if (IS_ERR(file)) { if (IS_ERR(file)) {
/* drop dentry, keep inode for a bit */ /* drop dentry, keep inode for a bit */
...@@ -423,7 +423,6 @@ struct file *sock_alloc_file(struct socket *sock, int flags, const char *dname) ...@@ -423,7 +423,6 @@ struct file *sock_alloc_file(struct socket *sock, int flags, const char *dname)
} }
sock->file = file; sock->file = file;
file->f_flags = O_RDWR | (flags & O_NONBLOCK);
file->private_data = sock; file->private_data = sock;
return file; return file;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment