From e636c8d336489bf3eed5878299e6cc045bbad077 Mon Sep 17 00:00:00 2001
From: hc <hc@nodka.com>
Date: Tue, 20 Feb 2024 01:17:29 +0000
Subject: [PATCH] debug lk
---
kernel/fs/incfs/vfs.c | 1691 ++++++++++++++++++++++-----------------------------------
1 files changed, 662 insertions(+), 1,029 deletions(-)
diff --git a/kernel/fs/incfs/vfs.c b/kernel/fs/incfs/vfs.c
index 8386722..f393f17 100644
--- a/kernel/fs/incfs/vfs.c
+++ b/kernel/fs/incfs/vfs.c
@@ -4,32 +4,28 @@
*/
#include <linux/blkdev.h>
-#include <linux/cred.h>
-#include <linux/eventpoll.h>
+#include <linux/compat.h>
+#include <linux/delay.h>
#include <linux/file.h>
#include <linux/fs.h>
#include <linux/fs_stack.h>
+#include <linux/fsnotify.h>
+#include <linux/fsverity.h>
+#include <linux/mmap_lock.h>
#include <linux/namei.h>
#include <linux/parser.h>
-#include <linux/poll.h>
#include <linux/seq_file.h>
-#include <linux/syscalls.h>
-#include <linux/xattr.h>
#include <uapi/linux/incrementalfs.h>
#include "vfs.h"
+
#include "data_mgmt.h"
#include "format.h"
-#include "integrity.h"
#include "internal.h"
-
-#define INCFS_PENDING_READS_INODE 2
-#define INCFS_LOG_INODE 3
-#define INCFS_START_INO_RANGE 10
-#define READ_FILE_MODE 0444
-#define READ_EXEC_FILE_MODE 0555
-#define READ_WRITE_FILE_MODE 0666
+#include "pseudo_files.h"
+#include "sysfs.h"
+#include "verity.h"
static int incfs_remount_fs(struct super_block *sb, int *flags, char *data);
@@ -52,23 +48,19 @@
static int read_single_page(struct file *f, struct page *page);
static long dispatch_ioctl(struct file *f, unsigned int req, unsigned long arg);
-static ssize_t pending_reads_read(struct file *f, char __user *buf, size_t len,
- loff_t *ppos);
-static __poll_t pending_reads_poll(struct file *file, poll_table *wait);
-static int pending_reads_open(struct inode *inode, struct file *file);
-static int pending_reads_release(struct inode *, struct file *);
-
-static ssize_t log_read(struct file *f, char __user *buf, size_t len,
- loff_t *ppos);
-static __poll_t log_poll(struct file *file, poll_table *wait);
-static int log_open(struct inode *inode, struct file *file);
-static int log_release(struct inode *, struct file *);
+#ifdef CONFIG_COMPAT
+static long incfs_compat_ioctl(struct file *file, unsigned int cmd,
+ unsigned long arg);
+#endif
static struct inode *alloc_inode(struct super_block *sb);
static void free_inode(struct inode *inode);
static void evict_inode(struct inode *inode);
static int incfs_setattr(struct dentry *dentry, struct iattr *ia);
+static int incfs_getattr(const struct path *path,
+ struct kstat *stat, u32 request_mask,
+ unsigned int query_flags);
static ssize_t incfs_getxattr(struct dentry *d, const char *name,
void *value, size_t size);
static ssize_t incfs_setxattr(struct dentry *d, const char *name,
@@ -109,8 +101,6 @@
.iterate = iterate_incfs_dir,
.open = file_open,
.release = file_release,
- .unlocked_ioctl = dispatch_ioctl,
- .compat_ioctl = dispatch_ioctl
};
static const struct dentry_operations incfs_dentry_ops = {
@@ -123,51 +113,54 @@
/* .readpages = readpages */
};
-static const struct file_operations incfs_file_ops = {
+static vm_fault_t incfs_fault(struct vm_fault *vmf)
+{
+ vmf->flags &= ~FAULT_FLAG_ALLOW_RETRY;
+ return filemap_fault(vmf);
+}
+
+static const struct vm_operations_struct incfs_file_vm_ops = {
+ .fault = incfs_fault,
+ .map_pages = filemap_map_pages,
+ .page_mkwrite = filemap_page_mkwrite,
+};
+
+/* This is used for a general mmap of a disk file */
+
+static int incfs_file_mmap(struct file *file, struct vm_area_struct *vma)
+{
+ struct address_space *mapping = file->f_mapping;
+
+ if (!mapping->a_ops->readpage)
+ return -ENOEXEC;
+ file_accessed(file);
+ vma->vm_ops = &incfs_file_vm_ops;
+ return 0;
+}
+
+const struct file_operations incfs_file_ops = {
.open = file_open,
.release = file_release,
.read_iter = generic_file_read_iter,
- .mmap = generic_file_mmap,
+ .mmap = incfs_file_mmap,
.splice_read = generic_file_splice_read,
.llseek = generic_file_llseek,
.unlocked_ioctl = dispatch_ioctl,
- .compat_ioctl = dispatch_ioctl
+#ifdef CONFIG_COMPAT
+ .compat_ioctl = incfs_compat_ioctl,
+#endif
};
-enum FILL_PERMISSION {
- CANT_FILL = 0,
- CAN_FILL = 1,
-};
-
-static const struct file_operations incfs_pending_read_file_ops = {
- .read = pending_reads_read,
- .poll = pending_reads_poll,
- .open = pending_reads_open,
- .release = pending_reads_release,
- .llseek = noop_llseek,
- .unlocked_ioctl = dispatch_ioctl,
- .compat_ioctl = dispatch_ioctl
-};
-
-static const struct file_operations incfs_log_file_ops = {
- .read = log_read,
- .poll = log_poll,
- .open = log_open,
- .release = log_release,
- .llseek = noop_llseek,
- .unlocked_ioctl = dispatch_ioctl,
- .compat_ioctl = dispatch_ioctl
-};
-
-static const struct inode_operations incfs_file_inode_ops = {
+const struct inode_operations incfs_file_inode_ops = {
.setattr = incfs_setattr,
- .getattr = simple_getattr,
+ .getattr = incfs_getattr,
.listxattr = incfs_listxattr
};
static int incfs_handler_getxattr(const struct xattr_handler *xh,
struct dentry *d, struct inode *inode,
- const char *name, void *buffer, size_t size)
+ const char *name, void *buffer, size_t size,
+ int flags)
{
return incfs_getxattr(d, name, buffer, size);
}
@@ -191,56 +184,41 @@
NULL,
};
-/* State of an open .pending_reads file, unique for each file descriptor. */
-struct pending_reads_state {
- /* A serial number of the last pending read obtained from this file. */
- int last_pending_read_sn;
-};
-
-/* State of an open .log file, unique for each file descriptor. */
-struct log_file_state {
- struct read_log_state state;
-};
-
struct inode_search {
unsigned long ino;
struct dentry *backing_dentry;
size_t size;
+
+ bool verity;
};
enum parse_parameter {
Opt_read_timeout,
Opt_readahead_pages,
- Opt_no_backing_file_cache,
- Opt_no_backing_file_readahead,
Opt_rlog_pages,
Opt_rlog_wakeup_cnt,
+ Opt_report_uid,
+ Opt_sysfs_name,
Opt_err
-};
-
-static const char pending_reads_file_name[] = INCFS_PENDING_READS_FILENAME;
-static struct mem_range pending_reads_file_name_range = {
- .data = (u8 *)pending_reads_file_name,
- .len = ARRAY_SIZE(pending_reads_file_name) - 1
-};
-
-static const char log_file_name[] = INCFS_LOG_FILENAME;
-static struct mem_range log_file_name_range = {
- .data = (u8 *)log_file_name,
- .len = ARRAY_SIZE(log_file_name) - 1
};
static const match_table_t option_tokens = {
{ Opt_read_timeout, "read_timeout_ms=%u" },
{ Opt_readahead_pages, "readahead=%u" },
- { Opt_no_backing_file_cache, "no_bf_cache=%u" },
- { Opt_no_backing_file_readahead, "no_bf_readahead=%u" },
{ Opt_rlog_pages, "rlog_pages=%u" },
{ Opt_rlog_wakeup_cnt, "rlog_wakeup_cnt=%u" },
+ { Opt_report_uid, "report_uid" },
+ { Opt_sysfs_name, "sysfs_name=%s" },
{ Opt_err, NULL }
};
+
+static void free_options(struct mount_options *opts)
+{
+ kfree(opts->sysfs_name);
+ opts->sysfs_name = NULL;
+}
static int parse_options(struct mount_options *opts, char *str)
{
@@ -251,12 +229,13 @@
if (opts == NULL)
return -EFAULT;
- opts->read_timeout_ms = 1000; /* Default: 1s */
- opts->readahead_pages = 10;
- opts->read_log_pages = 2;
- opts->read_log_wakeup_count = 10;
- opts->no_backing_file_cache = false;
- opts->no_backing_file_readahead = false;
+ *opts = (struct mount_options) {
+ .read_timeout_ms = 1000, /* Default: 1s */
+ .readahead_pages = 10,
+ .read_log_pages = 2,
+ .read_log_wakeup_count = 10,
+ };
+
if (str == NULL || *str == 0)
return 0;
@@ -272,22 +251,14 @@
case Opt_read_timeout:
if (match_int(&args[0], &value))
return -EINVAL;
+ if (value > 3600000)
+ return -EINVAL;
opts->read_timeout_ms = value;
break;
case Opt_readahead_pages:
if (match_int(&args[0], &value))
return -EINVAL;
opts->readahead_pages = value;
- break;
- case Opt_no_backing_file_cache:
- if (match_int(&args[0], &value))
- return -EINVAL;
- opts->no_backing_file_cache = (value != 0);
- break;
- case Opt_no_backing_file_readahead:
- if (match_int(&args[0], &value))
- return -EINVAL;
- opts->no_backing_file_readahead = (value != 0);
break;
case Opt_rlog_pages:
if (match_int(&args[0], &value))
@@ -299,26 +270,19 @@
return -EINVAL;
opts->read_log_wakeup_count = value;
break;
+ case Opt_report_uid:
+ opts->report_uid = true;
+ break;
+ case Opt_sysfs_name:
+ opts->sysfs_name = match_strdup(&args[0]);
+ break;
default:
+ free_options(opts);
return -EINVAL;
}
}
return 0;
-}
-
-static struct super_block *file_superblock(struct file *f)
-{
- struct inode *inode = file_inode(f);
-
- return inode->i_sb;
-}
-
-static struct mount_info *get_mount_info(struct super_block *sb)
-{
- struct mount_info *result = sb->s_fs_info;
-
- return result;
}
/* Read file size from the attribute. Quicker than reading the header */
@@ -336,100 +300,66 @@
return le64_to_cpu(attr_value);
}
+/* Read verity flag from the attribute. Quicker than reading the header */
+static bool read_verity_attr(struct dentry *backing_dentry)
+{
+ return vfs_getxattr(backing_dentry, INCFS_XATTR_VERITY_NAME, NULL, 0)
+ >= 0;
+}
+
static int inode_test(struct inode *inode, void *opaque)
{
struct inode_search *search = opaque;
struct inode_info *node = get_incfs_node(inode);
+ struct inode *backing_inode = d_inode(search->backing_dentry);
if (!node)
return 0;
- if (search->backing_dentry) {
- struct inode *backing_inode = d_inode(search->backing_dentry);
-
- return (node->n_backing_inode == backing_inode) &&
- inode->i_ino == search->ino;
- } else
- return inode->i_ino == search->ino;
+ return node->n_backing_inode == backing_inode &&
+ inode->i_ino == search->ino;
}
static int inode_set(struct inode *inode, void *opaque)
{
struct inode_search *search = opaque;
struct inode_info *node = get_incfs_node(inode);
+ struct dentry *backing_dentry = search->backing_dentry;
+ struct inode *backing_inode = d_inode(backing_dentry);
- if (search->backing_dentry) {
- /* It's a regular inode that has corresponding backing inode */
- struct dentry *backing_dentry = search->backing_dentry;
- struct inode *backing_inode = d_inode(backing_dentry);
+ fsstack_copy_attr_all(inode, backing_inode);
+ if (S_ISREG(inode->i_mode)) {
+ u64 size = search->size;
- fsstack_copy_attr_all(inode, backing_inode);
- if (S_ISREG(inode->i_mode)) {
- u64 size = search->size;
-
- inode->i_size = size;
- inode->i_blocks = get_blocks_count_for_size(size);
- inode->i_mapping->a_ops = &incfs_address_space_ops;
- inode->i_op = &incfs_file_inode_ops;
- inode->i_fop = &incfs_file_ops;
- inode->i_mode &= ~0222;
- } else if (S_ISDIR(inode->i_mode)) {
- inode->i_size = 0;
- inode->i_blocks = 1;
- inode->i_mapping->a_ops = &incfs_address_space_ops;
- inode->i_op = &incfs_dir_inode_ops;
- inode->i_fop = &incfs_dir_fops;
- } else {
- pr_warn_once("incfs: Unexpected inode type\n");
- return -EBADF;
- }
-
- ihold(backing_inode);
- node->n_backing_inode = backing_inode;
- node->n_mount_info = get_mount_info(inode->i_sb);
- inode->i_ctime = backing_inode->i_ctime;
- inode->i_mtime = backing_inode->i_mtime;
- inode->i_atime = backing_inode->i_atime;
- inode->i_ino = backing_inode->i_ino;
- if (backing_inode->i_ino < INCFS_START_INO_RANGE) {
- pr_warn("incfs: ino conflict with backing FS %ld\n",
- backing_inode->i_ino);
- }
-
- return 0;
- } else if (search->ino == INCFS_PENDING_READS_INODE) {
- /* It's an inode for .pending_reads pseudo file. */
-
- inode->i_ctime = (struct timespec64){};
- inode->i_mtime = inode->i_ctime;
- inode->i_atime = inode->i_ctime;
- inode->i_size = 0;
- inode->i_ino = INCFS_PENDING_READS_INODE;
- inode->i_private = NULL;
-
- inode_init_owner(inode, NULL, S_IFREG | READ_WRITE_FILE_MODE);
-
+ inode->i_size = size;
+ inode->i_blocks = get_blocks_count_for_size(size);
+ inode->i_mapping->a_ops = &incfs_address_space_ops;
inode->i_op = &incfs_file_inode_ops;
- inode->i_fop = &incfs_pending_read_file_ops;
-
- } else if (search->ino == INCFS_LOG_INODE) {
- /* It's an inode for .log pseudo file. */
-
- inode->i_ctime = (struct timespec64){};
- inode->i_mtime = inode->i_ctime;
- inode->i_atime = inode->i_ctime;
+ inode->i_fop = &incfs_file_ops;
+ inode->i_mode &= ~0222;
+ if (search->verity)
+ inode_set_flags(inode, S_VERITY, S_VERITY);
+ } else if (S_ISDIR(inode->i_mode)) {
inode->i_size = 0;
- inode->i_ino = INCFS_LOG_INODE;
- inode->i_private = NULL;
-
- inode_init_owner(inode, NULL, S_IFREG | READ_WRITE_FILE_MODE);
-
- inode->i_op = &incfs_file_inode_ops;
- inode->i_fop = &incfs_log_file_ops;
-
+ inode->i_blocks = 1;
+ inode->i_mapping->a_ops = &incfs_address_space_ops;
+ inode->i_op = &incfs_dir_inode_ops;
+ inode->i_fop = &incfs_dir_fops;
} else {
- /* Unknown inode requested. */
- return -EINVAL;
+ pr_warn_once("incfs: Unexpected inode type\n");
+ return -EBADF;
+ }
+
+ ihold(backing_inode);
+ node->n_backing_inode = backing_inode;
+ node->n_mount_info = get_mount_info(inode->i_sb);
+ inode->i_ctime = backing_inode->i_ctime;
+ inode->i_mtime = backing_inode->i_mtime;
+ inode->i_atime = backing_inode->i_atime;
+ inode->i_ino = backing_inode->i_ino;
+ if (backing_inode->i_ino < INCFS_START_INO_RANGE) {
+ pr_warn("incfs: ino conflict with backing FS %ld\n",
+ backing_inode->i_ino);
}
return 0;
@@ -443,222 +373,7 @@
.ino = backing_inode->i_ino,
.backing_dentry = backing_dentry,
.size = read_size_attr(backing_dentry),
- };
- struct inode *inode = iget5_locked(sb, search.ino, inode_test,
- inode_set, &search);
-
- if (!inode)
- return ERR_PTR(-ENOMEM);
-
- if (inode->i_state & I_NEW)
- unlock_new_inode(inode);
-
- return inode;
-}
-
-static ssize_t pending_reads_read(struct file *f, char __user *buf, size_t len,
- loff_t *ppos)
-{
- struct pending_reads_state *pr_state = f->private_data;
- struct mount_info *mi = get_mount_info(file_superblock(f));
- struct incfs_pending_read_info *reads_buf = NULL;
- size_t reads_to_collect = len / sizeof(*reads_buf);
- int last_known_read_sn = READ_ONCE(pr_state->last_pending_read_sn);
- int new_max_sn = last_known_read_sn;
- int reads_collected = 0;
- ssize_t result = 0;
- int i = 0;
-
- if (!incfs_fresh_pending_reads_exist(mi, last_known_read_sn))
- return 0;
-
- reads_buf = (struct incfs_pending_read_info *)get_zeroed_page(GFP_NOFS);
- if (!reads_buf)
- return -ENOMEM;
-
- reads_to_collect =
- min_t(size_t, PAGE_SIZE / sizeof(*reads_buf), reads_to_collect);
-
- reads_collected = incfs_collect_pending_reads(
- mi, last_known_read_sn, reads_buf, reads_to_collect);
- if (reads_collected < 0) {
- result = reads_collected;
- goto out;
- }
-
- for (i = 0; i < reads_collected; i++)
- if (reads_buf[i].serial_number > new_max_sn)
- new_max_sn = reads_buf[i].serial_number;
-
- /*
- * Just to make sure that we don't accidentally copy more data
- * to reads buffer than userspace can handle.
- */
- reads_collected = min_t(size_t, reads_collected, reads_to_collect);
- result = reads_collected * sizeof(*reads_buf);
-
- /* Copy reads info to the userspace buffer */
- if (copy_to_user(buf, reads_buf, result)) {
- result = -EFAULT;
- goto out;
- }
-
- WRITE_ONCE(pr_state->last_pending_read_sn, new_max_sn);
- *ppos = 0;
-out:
- if (reads_buf)
- free_page((unsigned long)reads_buf);
- return result;
-}
-
-
-static __poll_t pending_reads_poll(struct file *file, poll_table *wait)
-{
- struct pending_reads_state *state = file->private_data;
- struct mount_info *mi = get_mount_info(file_superblock(file));
- __poll_t ret = 0;
-
- poll_wait(file, &mi->mi_pending_reads_notif_wq, wait);
- if (incfs_fresh_pending_reads_exist(mi,
- state->last_pending_read_sn))
- ret = EPOLLIN | EPOLLRDNORM;
-
- return ret;
-}
-
-static int pending_reads_open(struct inode *inode, struct file *file)
-{
- struct pending_reads_state *state = NULL;
-
- state = kzalloc(sizeof(*state), GFP_NOFS);
- if (!state)
- return -ENOMEM;
-
- file->private_data = state;
- return 0;
-}
-
-static int pending_reads_release(struct inode *inode, struct file *file)
-{
- kfree(file->private_data);
- return 0;
-}
-
-static struct inode *fetch_pending_reads_inode(struct super_block *sb)
-{
- struct inode_search search = {
- .ino = INCFS_PENDING_READS_INODE
- };
- struct inode *inode = iget5_locked(sb, search.ino, inode_test,
- inode_set, &search);
-
- if (!inode)
- return ERR_PTR(-ENOMEM);
-
- if (inode->i_state & I_NEW)
- unlock_new_inode(inode);
-
- return inode;
-}
-
-static int log_open(struct inode *inode, struct file *file)
-{
- struct log_file_state *log_state = NULL;
- struct mount_info *mi = get_mount_info(file_superblock(file));
-
- log_state = kzalloc(sizeof(*log_state), GFP_NOFS);
- if (!log_state)
- return -ENOMEM;
-
- log_state->state = incfs_get_log_state(mi);
- file->private_data = log_state;
- return 0;
-}
-
-static int log_release(struct inode *inode, struct file *file)
-{
- kfree(file->private_data);
- return 0;
-}
-
-static ssize_t log_read(struct file *f, char __user *buf, size_t len,
- loff_t *ppos)
-{
- struct log_file_state *log_state = f->private_data;
- struct mount_info *mi = get_mount_info(file_superblock(f));
- int total_reads_collected = 0;
- int rl_size;
- ssize_t result = 0;
- struct incfs_pending_read_info *reads_buf;
- ssize_t reads_to_collect = len / sizeof(*reads_buf);
- ssize_t reads_per_page = PAGE_SIZE / sizeof(*reads_buf);
-
- rl_size = READ_ONCE(mi->mi_log.rl_size);
- if (rl_size == 0)
- return 0;
-
- reads_buf = (struct incfs_pending_read_info *)__get_free_page(GFP_NOFS);
- if (!reads_buf)
- return -ENOMEM;
-
- reads_to_collect = min_t(ssize_t, rl_size, reads_to_collect);
- while (reads_to_collect > 0) {
- struct read_log_state next_state;
- int reads_collected;
-
- memcpy(&next_state, &log_state->state, sizeof(next_state));
- reads_collected = incfs_collect_logged_reads(
- mi, &next_state, reads_buf,
- min_t(ssize_t, reads_to_collect, reads_per_page));
- if (reads_collected <= 0) {
- result = total_reads_collected ?
- total_reads_collected *
- sizeof(*reads_buf) :
- reads_collected;
- goto out;
- }
- if (copy_to_user(buf, reads_buf,
- reads_collected * sizeof(*reads_buf))) {
- result = total_reads_collected ?
- total_reads_collected *
- sizeof(*reads_buf) :
- -EFAULT;
- goto out;
- }
-
- memcpy(&log_state->state, &next_state, sizeof(next_state));
- total_reads_collected += reads_collected;
- buf += reads_collected * sizeof(*reads_buf);
- reads_to_collect -= reads_collected;
- }
-
- result = total_reads_collected * sizeof(*reads_buf);
- *ppos = 0;
-out:
- if (reads_buf)
- free_page((unsigned long)reads_buf);
- return result;
-}
-
-static __poll_t log_poll(struct file *file, poll_table *wait)
-{
- struct log_file_state *log_state = file->private_data;
- struct mount_info *mi = get_mount_info(file_superblock(file));
- int count;
- __poll_t ret = 0;
-
- poll_wait(file, &mi->mi_log.ml_notif_wq, wait);
- count = incfs_get_uncollected_logs_count(mi, &log_state->state);
- if (count >= mi->mi_options.read_log_wakeup_count)
- ret = EPOLLIN | EPOLLRDNORM;
-
- return ret;
-}
-
-static struct inode *fetch_log_inode(struct super_block *sb)
-{
- struct inode_search search = {
- .ino = INCFS_LOG_INODE
+ .verity = read_verity_attr(backing_dentry),
};
struct inode *inode = iget5_locked(sb, search.ino, inode_test,
inode_set, &search);
@@ -679,7 +394,7 @@
struct mount_info *mi = get_mount_info(file_superblock(file));
bool root;
- if (!dir || !mi) {
+ if (!dir) {
error = -EBADF;
goto out;
}
@@ -687,29 +402,15 @@
root = dir->backing_dir->f_inode
== d_inode(mi->mi_backing_dir_path.dentry);
- if (root && ctx->pos == 0) {
- if (!dir_emit(ctx, pending_reads_file_name,
- ARRAY_SIZE(pending_reads_file_name) - 1,
- INCFS_PENDING_READS_INODE, DT_REG)) {
- error = -EINVAL;
+ if (root) {
+ error = emit_pseudo_files(ctx);
+ if (error)
goto out;
- }
- ctx->pos++;
}
- if (root && ctx->pos == 1) {
- if (!dir_emit(ctx, log_file_name,
- ARRAY_SIZE(log_file_name) - 1,
- INCFS_LOG_INODE, DT_REG)) {
- error = -EINVAL;
- goto out;
- }
- ctx->pos++;
- }
-
- ctx->pos -= 2;
+ ctx->pos -= PSEUDO_FILE_COUNT;
error = iterate_dir(dir->backing_dir, ctx);
- ctx->pos += 2;
+ ctx->pos += PSEUDO_FILE_COUNT;
file->f_pos = dir->backing_dir->f_pos;
out:
if (error)
@@ -736,29 +437,10 @@
return 0;
}
-static struct dentry *incfs_lookup_dentry(struct dentry *parent,
- const char *name)
+static struct dentry *open_or_create_special_dir(struct dentry *backing_dir,
+ const char *name,
+ bool *created)
{
- struct inode *inode;
- struct dentry *result = NULL;
-
- if (!parent)
- return ERR_PTR(-EFAULT);
-
- inode = d_inode(parent);
- inode_lock_nested(inode, I_MUTEX_PARENT);
- result = lookup_one_len(name, parent, strlen(name));
- inode_unlock(inode);
-
- if (IS_ERR(result))
- pr_warn("%s err:%ld\n", __func__, PTR_ERR(result));
-
- return result;
-}
-
-static struct dentry *open_or_create_index_dir(struct dentry *backing_dir)
-{
- static const char name[] = ".index";
struct dentry *index_dentry;
struct inode *backing_inode = d_inode(backing_dir);
int err = 0;
@@ -770,6 +452,7 @@
return index_dentry;
} else if (d_really_is_positive(index_dentry)) {
/* Index already exists. */
+ *created = false;
return index_dentry;
}
@@ -783,12 +466,68 @@
return ERR_PTR(err);
}
- if (!d_really_is_positive(index_dentry)) {
+ if (!d_really_is_positive(index_dentry) ||
+ unlikely(d_unhashed(index_dentry))) {
dput(index_dentry);
return ERR_PTR(-EINVAL);
}
+ *created = true;
return index_dentry;
+}
+
+static int read_single_page_timeouts(struct data_file *df, struct file *f,
+ int block_index, struct mem_range range,
+ struct mem_range tmp,
+ unsigned int *delayed_min_us)
+{
+ struct mount_info *mi = df->df_mount_info;
+ struct incfs_read_data_file_timeouts timeouts = {
+ .max_pending_time_us = U32_MAX,
+ };
+ int uid = current_uid().val;
+ int i;
+
+ spin_lock(&mi->mi_per_uid_read_timeouts_lock);
+ for (i = 0; i < mi->mi_per_uid_read_timeouts_size /
+ sizeof(*mi->mi_per_uid_read_timeouts); ++i) {
+ struct incfs_per_uid_read_timeouts *t =
+ &mi->mi_per_uid_read_timeouts[i];
+
+ if(t->uid == uid) {
+ timeouts.min_time_us = t->min_time_us;
+ timeouts.min_pending_time_us = t->min_pending_time_us;
+ timeouts.max_pending_time_us = t->max_pending_time_us;
+ break;
+ }
+ }
+ spin_unlock(&mi->mi_per_uid_read_timeouts_lock);
+ if (timeouts.max_pending_time_us == U32_MAX) {
+ u64 read_timeout_us = (u64)mi->mi_options.read_timeout_ms *
+ 1000;
+
+ timeouts.max_pending_time_us = read_timeout_us <= U32_MAX ?
+ read_timeout_us : U32_MAX;
+ }
+
+ return incfs_read_data_file_block(range, f, block_index, tmp,
+ &timeouts, delayed_min_us);
+}
+
+static int usleep_interruptible(u32 us)
+{
+ /* See:
+ * https://www.kernel.org/doc/Documentation/timers/timers-howto.txt
+ * for explanation
+ */
+ if (us < 10) {
+ udelay(us);
+ return 0;
+ } else if (us < 20000) {
+ usleep_range(us, us + us / 10);
+ return 0;
+ } else
+ return msleep_interruptible(us / 1000);
}
static int read_single_page(struct file *f, struct page *page)
@@ -799,28 +538,36 @@
ssize_t read_result = 0;
struct data_file *df = get_incfs_data_file(f);
int result = 0;
- void *page_start = kmap(page);
+ void *page_start;
int block_index;
- int timeout_ms;
+ unsigned int delayed_min_us = 0;
- if (!df)
+ if (!df) {
+ SetPageError(page);
+ unlock_page(page);
return -EBADF;
+ }
+ page_start = kmap(page);
offset = page_offset(page);
- block_index = offset / INCFS_DATA_FILE_BLOCK_SIZE;
+ block_index = (offset + df->df_mapped_offset) /
+ INCFS_DATA_FILE_BLOCK_SIZE;
size = df->df_size;
- timeout_ms = df->df_mount_info->mi_options.read_timeout_ms;
if (offset < size) {
struct mem_range tmp = {
.len = 2 * INCFS_DATA_FILE_BLOCK_SIZE
};
-
tmp.data = (u8 *)__get_free_pages(GFP_NOFS, get_order(tmp.len));
+ if (!tmp.data) {
+ read_result = -ENOMEM;
+ goto err;
+ }
bytes_to_read = min_t(loff_t, size - offset, PAGE_SIZE);
- read_result = incfs_read_data_file_block(
- range(page_start, bytes_to_read), f, block_index,
- timeout_ms, tmp);
+
+ read_result = read_single_page_timeouts(df, f, block_index,
+ range(page_start, bytes_to_read), tmp,
+ &delayed_min_us);
free_pages((unsigned long)tmp.data, get_order(tmp.len));
} else {
@@ -828,6 +575,7 @@
read_result = 0;
}
+err:
if (read_result < 0)
result = read_result;
else if (read_result < PAGE_SIZE)
@@ -841,140 +589,12 @@
flush_dcache_page(page);
kunmap(page);
unlock_page(page);
+ if (delayed_min_us)
+ usleep_interruptible(delayed_min_us);
return result;
}
-static char *file_id_to_str(incfs_uuid_t id)
-{
- char *result = kmalloc(1 + sizeof(id.bytes) * 2, GFP_NOFS);
- char *end;
-
- if (!result)
- return NULL;
-
- end = bin2hex(result, id.bytes, sizeof(id.bytes));
- *end = 0;
- return result;
-}
-
-static struct mem_range incfs_copy_signature_info_from_user(u8 __user *original,
- u64 size)
-{
- u8 *result;
-
- if (!original)
- return range(NULL, 0);
-
- if (size > INCFS_MAX_SIGNATURE_SIZE)
- return range(ERR_PTR(-EFAULT), 0);
-
- result = kzalloc(size, GFP_NOFS | __GFP_COMP);
- if (!result)
- return range(ERR_PTR(-ENOMEM), 0);
-
- if (copy_from_user(result, original, size)) {
- kfree(result);
- return range(ERR_PTR(-EFAULT), 0);
- }
-
- return range(result, size);
-}
-
-static int init_new_file(struct mount_info *mi, struct dentry *dentry,
- incfs_uuid_t *uuid, u64 size, struct mem_range attr,
- u8 __user *user_signature_info, u64 signature_size)
-{
- struct path path = {};
- struct file *new_file;
- int error = 0;
- struct backing_file_context *bfc = NULL;
- u32 block_count;
- struct mem_range raw_signature = { NULL };
- struct mtree *hash_tree = NULL;
-
- if (!mi || !dentry || !uuid)
- return -EFAULT;
-
- /* Resize newly created file to its true size. */
- path = (struct path) {
- .mnt = mi->mi_backing_dir_path.mnt,
- .dentry = dentry
- };
- new_file = dentry_open(&path, O_RDWR | O_NOATIME | O_LARGEFILE,
- current_cred());
-
- if (IS_ERR(new_file)) {
- error = PTR_ERR(new_file);
- goto out;
- }
-
- bfc = incfs_alloc_bfc(mi, new_file);
- fput(new_file);
- if (IS_ERR(bfc)) {
- error = PTR_ERR(bfc);
- bfc = NULL;
- goto out;
- }
-
- mutex_lock(&bfc->bc_mutex);
- error = incfs_write_fh_to_backing_file(bfc, uuid, size);
- if (error)
- goto out;
-
- if (attr.data && attr.len) {
- error = incfs_write_file_attr_to_backing_file(bfc,
- attr, NULL);
- if (error)
- goto out;
- }
-
- block_count = (u32)get_blocks_count_for_size(size);
-
- if (user_signature_info) {
- raw_signature = incfs_copy_signature_info_from_user(
- user_signature_info, signature_size);
-
- if (IS_ERR(raw_signature.data)) {
- error = PTR_ERR(raw_signature.data);
- raw_signature.data = NULL;
- goto out;
- }
-
- hash_tree = incfs_alloc_mtree(raw_signature, block_count);
- if (IS_ERR(hash_tree)) {
- error = PTR_ERR(hash_tree);
- hash_tree = NULL;
- goto out;
- }
-
- error = incfs_write_signature_to_backing_file(
- bfc, raw_signature, hash_tree->hash_tree_area_size);
- if (error)
- goto out;
-
- block_count += get_blocks_count_for_size(
- hash_tree->hash_tree_area_size);
- }
-
- if (block_count)
- error = incfs_write_blockmap_to_backing_file(bfc, block_count);
-
- if (error)
- goto out;
-out:
- if (bfc) {
- mutex_unlock(&bfc->bc_mutex);
- incfs_free_bfc(bfc);
- }
- incfs_free_mtree(hash_tree);
- kfree(raw_signature.data);
-
- if (error)
- pr_debug("incfs: %s error: %d\n", __func__, error);
- return error;
-}
-
-static int incfs_link(struct dentry *what, struct dentry *where)
+int incfs_link(struct dentry *what, struct dentry *where)
{
struct dentry *parent_dentry = dget_parent(where);
struct inode *pinode = d_inode(parent_dentry);
@@ -988,7 +608,7 @@
return error;
}
-static int incfs_unlink(struct dentry *dentry)
+int incfs_unlink(struct dentry *dentry)
{
struct dentry *parent_dentry = dget_parent(dentry);
struct inode *pinode = d_inode(parent_dentry);
@@ -1016,274 +636,108 @@
return error;
}
-static int dir_relative_path_resolve(
- struct mount_info *mi,
- const char __user *relative_path,
- struct path *result_path)
+static void notify_unlink(struct dentry *dentry, const char *file_id_str,
+ const char *special_directory)
{
- struct path *base_path = &mi->mi_backing_dir_path;
- int dir_fd = get_unused_fd_flags(0);
- struct file *dir_f = NULL;
+ struct dentry *root = dentry;
+ struct dentry *file = NULL;
+ struct dentry *dir = NULL;
int error = 0;
+ bool take_lock = root->d_parent != root->d_parent->d_parent;
- if (dir_fd < 0)
- return dir_fd;
+ while (root != root->d_parent)
+ root = root->d_parent;
- dir_f = dentry_open(base_path, O_RDONLY | O_NOATIME, current_cred());
+ if (take_lock)
+ dir = incfs_lookup_dentry(root, special_directory);
+ else
+ dir = lookup_one_len(special_directory, root,
+ strlen(special_directory));
- if (IS_ERR(dir_f)) {
- error = PTR_ERR(dir_f);
+ if (IS_ERR(dir)) {
+ error = PTR_ERR(dir);
goto out;
}
- fd_install(dir_fd, dir_f);
-
- if (!relative_path) {
- /* No relative path given, just return the base dir. */
- *result_path = *base_path;
- path_get(result_path);
+ if (d_is_negative(dir)) {
+ error = -ENOENT;
goto out;
}
- error = user_path_at_empty(dir_fd, relative_path,
- LOOKUP_FOLLOW | LOOKUP_DIRECTORY, result_path, NULL);
+ file = incfs_lookup_dentry(dir, file_id_str);
+ if (IS_ERR(file)) {
+ error = PTR_ERR(file);
+ goto out;
+ }
+ if (d_is_negative(file)) {
+ error = -ENOENT;
+ goto out;
+ }
+
+ fsnotify_unlink(d_inode(dir), file);
+ d_delete(file);
out:
- ksys_close(dir_fd);
if (error)
- pr_debug("incfs: %s %d\n", __func__, error);
- return error;
+ pr_warn("%s failed with error %d\n", __func__, error);
+
+ dput(dir);
+ dput(file);
}
-static int validate_name(char *file_name)
+static void handle_file_completed(struct file *f, struct data_file *df)
{
- struct mem_range name = range(file_name, strlen(file_name));
- int i = 0;
-
- if (name.len > INCFS_MAX_NAME_LEN)
- return -ENAMETOOLONG;
-
- if (incfs_equal_ranges(pending_reads_file_name_range, name))
- return -EINVAL;
-
- for (i = 0; i < name.len; i++)
- if (name.data[i] == '/')
- return -EINVAL;
-
- return 0;
-}
-
-static int chmod(struct dentry *dentry, umode_t mode)
-{
- struct inode *inode = dentry->d_inode;
- struct inode *delegated_inode = NULL;
- struct iattr newattrs;
+ struct backing_file_context *bfc;
+ struct mount_info *mi = df->df_mount_info;
+ char *file_id_str = NULL;
+ struct dentry *incomplete_file_dentry = NULL;
+ const struct cred *old_cred = override_creds(mi->mi_owner);
int error;
-retry_deleg:
- inode_lock(inode);
- newattrs.ia_mode = (mode & S_IALLUGO) | (inode->i_mode & ~S_IALLUGO);
- newattrs.ia_valid = ATTR_MODE | ATTR_CTIME;
- error = notify_change(dentry, &newattrs, &delegated_inode);
- inode_unlock(inode);
- if (delegated_inode) {
- error = break_deleg_wait(&delegated_inode);
- if (!error)
- goto retry_deleg;
- }
- return error;
-}
+ /* Truncate file to remove any preallocated space */
+ bfc = df->df_backing_file_context;
+ if (bfc) {
+ struct file *f = bfc->bc_file;
-static long ioctl_create_file(struct mount_info *mi,
- struct incfs_new_file_args __user *usr_args)
-{
- struct incfs_new_file_args args;
- char *file_id_str = NULL;
- struct dentry *index_file_dentry = NULL;
- struct dentry *named_file_dentry = NULL;
- struct path parent_dir_path = {};
- struct inode *index_dir_inode = NULL;
- __le64 size_attr_value = 0;
- char *file_name = NULL;
- char *attr_value = NULL;
- int error = 0;
- bool locked = false;
+ if (f) {
+ loff_t size = i_size_read(file_inode(f));
- if (!mi || !mi->mi_index_dir) {
- error = -EFAULT;
- goto out;
- }
-
- if (copy_from_user(&args, usr_args, sizeof(args)) > 0) {
- error = -EFAULT;
- goto out;
- }
-
- file_name = strndup_user(u64_to_user_ptr(args.file_name), PATH_MAX);
- if (IS_ERR(file_name)) {
- error = PTR_ERR(file_name);
- file_name = NULL;
- goto out;
- }
-
- error = validate_name(file_name);
- if (error)
- goto out;
-
- file_id_str = file_id_to_str(args.file_id);
- if (!file_id_str) {
- error = -ENOMEM;
- goto out;
- }
-
- error = mutex_lock_interruptible(&mi->mi_dir_struct_mutex);
- if (error)
- goto out;
- locked = true;
-
- /* Find a directory to put the file into. */
- error = dir_relative_path_resolve(mi,
- u64_to_user_ptr(args.directory_path),
- &parent_dir_path);
- if (error)
- goto out;
-
- if (parent_dir_path.dentry == mi->mi_index_dir) {
- /* Can't create a file directly inside .index */
- error = -EBUSY;
- goto out;
- }
-
- /* Look up a dentry in the parent dir. It should be negative. */
- named_file_dentry = incfs_lookup_dentry(parent_dir_path.dentry,
- file_name);
- if (!named_file_dentry) {
- error = -EFAULT;
- goto out;
- }
- if (IS_ERR(named_file_dentry)) {
- error = PTR_ERR(named_file_dentry);
- named_file_dentry = NULL;
- goto out;
- }
- if (d_really_is_positive(named_file_dentry)) {
- /* File with this path already exists. */
- error = -EEXIST;
- goto out;
- }
- /* Look up a dentry in the .index dir. It should be negative. */
- index_file_dentry = incfs_lookup_dentry(mi->mi_index_dir, file_id_str);
- if (!index_file_dentry) {
- error = -EFAULT;
- goto out;
- }
- if (IS_ERR(index_file_dentry)) {
- error = PTR_ERR(index_file_dentry);
- index_file_dentry = NULL;
- goto out;
- }
- if (d_really_is_positive(index_file_dentry)) {
- /* File with this ID already exists in index. */
- error = -EEXIST;
- goto out;
- }
-
- /* Creating a file in the .index dir. */
- index_dir_inode = d_inode(mi->mi_index_dir);
- inode_lock_nested(index_dir_inode, I_MUTEX_PARENT);
- error = vfs_create(index_dir_inode, index_file_dentry, args.mode | 0222,
- true);
- inode_unlock(index_dir_inode);
-
- if (error)
- goto out;
- if (!d_really_is_positive(index_file_dentry)) {
- error = -EINVAL;
- goto out;
- }
-
- error = chmod(index_file_dentry, args.mode | 0222);
- if (error) {
- pr_debug("incfs: chmod err: %d\n", error);
- goto delete_index_file;
- }
-
- /* Save the file's ID as an xattr for easy fetching in future. */
- error = vfs_setxattr(index_file_dentry, INCFS_XATTR_ID_NAME,
- file_id_str, strlen(file_id_str), XATTR_CREATE);
- if (error) {
- pr_debug("incfs: vfs_setxattr err:%d\n", error);
- goto delete_index_file;
- }
-
- /* Save the file's size as an xattr for easy fetching in future. */
- size_attr_value = cpu_to_le64(args.size);
- error = vfs_setxattr(index_file_dentry, INCFS_XATTR_SIZE_NAME,
- (char *)&size_attr_value, sizeof(size_attr_value),
- XATTR_CREATE);
- if (error) {
- pr_debug("incfs: vfs_setxattr err:%d\n", error);
- goto delete_index_file;
- }
-
- /* Save the file's attribute as an xattr */
- if (args.file_attr_len && args.file_attr) {
- if (args.file_attr_len > INCFS_MAX_FILE_ATTR_SIZE) {
- error = -E2BIG;
- goto delete_index_file;
+ error = vfs_truncate(&f->f_path, size);
+ if (error)
+ /* No useful action on failure */
+ pr_warn("incfs: Failed to truncate complete file: %d\n",
+ error);
}
-
- attr_value = kmalloc(args.file_attr_len, GFP_NOFS);
- if (!attr_value) {
- error = -ENOMEM;
- goto delete_index_file;
- }
-
- if (copy_from_user(attr_value,
- u64_to_user_ptr(args.file_attr),
- args.file_attr_len) > 0) {
- error = -EFAULT;
- goto delete_index_file;
- }
-
- error = vfs_setxattr(index_file_dentry,
- INCFS_XATTR_METADATA_NAME,
- attr_value, args.file_attr_len,
- XATTR_CREATE);
-
- if (error)
- goto delete_index_file;
}
- /* Initializing a newly created file. */
- error = init_new_file(mi, index_file_dentry, &args.file_id, args.size,
- range(attr_value, args.file_attr_len),
- (u8 __user *)args.signature_info,
- args.signature_size);
- if (error)
- goto delete_index_file;
-
- /* Linking a file with it's real name from the requested dir. */
- error = incfs_link(index_file_dentry, named_file_dentry);
-
- if (!error)
+ /* This is best effort - there is no useful action to take on failure */
+ file_id_str = file_id_to_str(df->df_id);
+ if (!file_id_str)
goto out;
-delete_index_file:
- incfs_unlink(index_file_dentry);
+ incomplete_file_dentry = incfs_lookup_dentry(
+ df->df_mount_info->mi_incomplete_dir,
+ file_id_str);
+ if (!incomplete_file_dentry || IS_ERR(incomplete_file_dentry)) {
+ incomplete_file_dentry = NULL;
+ goto out;
+ }
+
+ if (!d_really_is_positive(incomplete_file_dentry))
+ goto out;
+
+ vfs_fsync(df->df_backing_file_context->bc_file, 0);
+ error = incfs_unlink(incomplete_file_dentry);
+ if (error) {
+ pr_warn("incfs: Deleting incomplete file failed: %d\n", error);
+ goto out;
+ }
+
+ notify_unlink(f->f_path.dentry, file_id_str, INCFS_INCOMPLETE_NAME);
out:
- if (error)
- pr_debug("incfs: %s err:%d\n", __func__, error);
-
+ dput(incomplete_file_dentry);
kfree(file_id_str);
- kfree(file_name);
- kfree(attr_value);
- dput(named_file_dentry);
- dput(index_file_dentry);
- path_put(&parent_dir_path);
- if (locked)
- mutex_unlock(&mi->mi_dir_struct_mutex);
- return error;
+ revert_creds(old_cred);
}
static long ioctl_fill_blocks(struct file *f, void __user *arg)
@@ -1292,15 +746,17 @@
struct incfs_fill_blocks fill_blocks;
struct incfs_fill_block __user *usr_fill_block_array;
struct data_file *df = get_incfs_data_file(f);
+ struct incfs_file_data *fd = f->private_data;
const ssize_t data_buf_size = 2 * INCFS_DATA_FILE_BLOCK_SIZE;
u8 *data_buf = NULL;
ssize_t error = 0;
int i = 0;
+ bool complete = false;
if (!df)
return -EBADF;
- if ((uintptr_t)f->private_data != CAN_FILL)
+ if (!fd || fd->fd_fill_permission != CAN_FILL)
return -EPERM;
if (copy_from_user(&fill_blocks, usr_fill_blocks, sizeof(fill_blocks)))
@@ -1337,7 +793,7 @@
data_buf);
} else {
error = incfs_process_new_data_block(df, &fill_block,
- data_buf);
+ data_buf, &complete);
}
if (error)
break;
@@ -1345,6 +801,9 @@
if (data_buf)
free_pages((unsigned long)data_buf, get_order(data_buf_size));
+
+ if (complete)
+ handle_file_completed(f, df);
/*
* Only report the error if no records were processed, otherwise
@@ -1354,53 +813,6 @@
return error;
return i;
-}
-
-static long ioctl_permit_fill(struct file *f, void __user *arg)
-{
- struct incfs_permit_fill __user *usr_permit_fill = arg;
- struct incfs_permit_fill permit_fill;
- long error = 0;
- struct file *file = NULL;
-
- if (f->f_op != &incfs_pending_read_file_ops)
- return -EPERM;
-
- if (copy_from_user(&permit_fill, usr_permit_fill, sizeof(permit_fill)))
- return -EFAULT;
-
- file = fget(permit_fill.file_descriptor);
- if (IS_ERR(file))
- return PTR_ERR(file);
-
- if (file->f_op != &incfs_file_ops) {
- error = -EPERM;
- goto out;
- }
-
- if (file->f_inode->i_sb != f->f_inode->i_sb) {
- error = -EPERM;
- goto out;
- }
-
- switch ((uintptr_t)file->private_data) {
- case CANT_FILL:
- file->private_data = (void *)CAN_FILL;
- break;
-
- case CAN_FILL:
- pr_debug("CAN_FILL already set");
- break;
-
- default:
- pr_warn("Invalid file private data");
- error = -EFAULT;
- goto out;
- }
-
-out:
- fput(file);
- return error;
}
static long ioctl_read_file_signature(struct file *f, void __user *arg)
@@ -1456,18 +868,19 @@
struct incfs_get_filled_blocks_args __user *args_usr_ptr = arg;
struct incfs_get_filled_blocks_args args = {};
struct data_file *df = get_incfs_data_file(f);
+ struct incfs_file_data *fd = f->private_data;
int error;
- if (!df)
+ if (!df || !fd)
return -EINVAL;
- if ((uintptr_t)f->private_data != CAN_FILL)
+ if (fd->fd_fill_permission != CAN_FILL)
return -EPERM;
if (copy_from_user(&args, args_usr_ptr, sizeof(args)) > 0)
return -EINVAL;
- error = incfs_get_filled_blocks(df, &args);
+ error = incfs_get_filled_blocks(df, fd, &args);
if (copy_to_user(args_usr_ptr, &args, sizeof(args)))
return -EFAULT;
@@ -1475,25 +888,80 @@
return error;
}
+static long ioctl_get_block_count(struct file *f, void __user *arg)
+{
+ struct incfs_get_block_count_args __user *args_usr_ptr = arg;
+ struct incfs_get_block_count_args args = {};
+ struct data_file *df = get_incfs_data_file(f);
+
+ if (!df)
+ return -EINVAL;
+
+ args.total_data_blocks_out = df->df_data_block_count;
+ args.filled_data_blocks_out = atomic_read(&df->df_data_blocks_written);
+ args.total_hash_blocks_out = df->df_total_block_count -
+ df->df_data_block_count;
+ args.filled_hash_blocks_out = atomic_read(&df->df_hash_blocks_written);
+
+ if (copy_to_user(args_usr_ptr, &args, sizeof(args)))
+ return -EFAULT;
+
+ return 0;
+}
+
+static int incfs_ioctl_get_flags(struct file *f, void __user *arg)
+{
+ u32 flags = IS_VERITY(file_inode(f)) ? FS_VERITY_FL : 0;
+
+ return put_user(flags, (int __user *) arg);
+}
+
static long dispatch_ioctl(struct file *f, unsigned int req, unsigned long arg)
{
- struct mount_info *mi = get_mount_info(file_superblock(f));
-
switch (req) {
- case INCFS_IOC_CREATE_FILE:
- return ioctl_create_file(mi, (void __user *)arg);
case INCFS_IOC_FILL_BLOCKS:
return ioctl_fill_blocks(f, (void __user *)arg);
- case INCFS_IOC_PERMIT_FILL:
- return ioctl_permit_fill(f, (void __user *)arg);
case INCFS_IOC_READ_FILE_SIGNATURE:
return ioctl_read_file_signature(f, (void __user *)arg);
case INCFS_IOC_GET_FILLED_BLOCKS:
return ioctl_get_filled_blocks(f, (void __user *)arg);
+ case INCFS_IOC_GET_BLOCK_COUNT:
+ return ioctl_get_block_count(f, (void __user *)arg);
+ case FS_IOC_ENABLE_VERITY:
+ return incfs_ioctl_enable_verity(f, (const void __user *)arg);
+ case FS_IOC_GETFLAGS:
+ return incfs_ioctl_get_flags(f, (void __user *) arg);
+ case FS_IOC_MEASURE_VERITY:
+ return incfs_ioctl_measure_verity(f, (void __user *)arg);
+ case FS_IOC_READ_VERITY_METADATA:
+ return incfs_ioctl_read_verity_metadata(f, (void __user *)arg);
default:
return -EINVAL;
}
}
+
+#ifdef CONFIG_COMPAT
+static long incfs_compat_ioctl(struct file *file, unsigned int cmd,
+ unsigned long arg)
+{
+ switch (cmd) {
+ case FS_IOC32_GETFLAGS:
+ cmd = FS_IOC_GETFLAGS;
+ break;
+ case INCFS_IOC_FILL_BLOCKS:
+ case INCFS_IOC_READ_FILE_SIGNATURE:
+ case INCFS_IOC_GET_FILLED_BLOCKS:
+ case INCFS_IOC_GET_BLOCK_COUNT:
+ case FS_IOC_ENABLE_VERITY:
+ case FS_IOC_MEASURE_VERITY:
+ case FS_IOC_READ_VERITY_METADATA:
+ break;
+ default:
+ return -ENOIOCTLCMD;
+ }
+ return dispatch_ioctl(file, cmd, (unsigned long) compat_ptr(arg));
+}
+#endif
static struct dentry *dir_lookup(struct inode *dir_inode, struct dentry *dentry,
unsigned int flags)
@@ -1503,40 +971,18 @@
struct dentry *backing_dentry = NULL;
struct path dir_backing_path = {};
struct inode_info *dir_info = get_incfs_node(dir_inode);
- struct mem_range name_range =
- range((u8 *)dentry->d_name.name, dentry->d_name.len);
int err = 0;
+
+ if (!mi || !dir_info || !dir_info->n_backing_inode)
+ return ERR_PTR(-EBADF);
if (d_inode(mi->mi_backing_dir_path.dentry) ==
dir_info->n_backing_inode) {
/* We do lookup in the FS root. Show pseudo files. */
-
- if (incfs_equal_ranges(pending_reads_file_name_range,
- name_range)) {
- struct inode *inode = fetch_pending_reads_inode(
- dir_inode->i_sb);
-
- if (IS_ERR(inode)) {
- err = PTR_ERR(inode);
- goto out;
- }
-
- d_add(dentry, inode);
+ err = dir_lookup_pseudo_files(dir_inode->i_sb, dentry);
+ if (err != -ENOENT)
goto out;
- }
-
- if (incfs_equal_ranges(log_file_name_range, name_range)) {
- struct inode *inode = fetch_log_inode(
- dir_inode->i_sb);
-
- if (IS_ERR(inode)) {
- err = PTR_ERR(inode);
- goto out;
- }
-
- d_add(dentry, inode);
- goto out;
- }
+ err = 0;
}
dir_dentry = dget_parent(dentry);
@@ -1622,7 +1068,7 @@
if (!backing_dentry) {
err = -EBADF;
- goto out;
+ goto path_err;
}
if (backing_dentry->d_parent == mi->mi_index_dir) {
@@ -1631,13 +1077,19 @@
goto out;
}
+ if (backing_dentry->d_parent == mi->mi_incomplete_dir) {
+ /* Can't create a subdir inside .incomplete */
+ err = -EBUSY;
+ goto out;
+ }
inode_lock_nested(dir_node->n_backing_inode, I_MUTEX_PARENT);
err = vfs_mkdir(dir_node->n_backing_inode, backing_dentry, mode | 0222);
inode_unlock(dir_node->n_backing_inode);
if (!err) {
struct inode *inode = NULL;
- if (d_really_is_negative(backing_dentry)) {
+ if (d_really_is_negative(backing_dentry) ||
+ unlikely(d_unhashed(backing_dentry))) {
err = -EINVAL;
goto out;
}
@@ -1654,23 +1106,33 @@
if (d_really_is_negative(dentry))
d_drop(dentry);
path_put(&backing_path);
+
+path_err:
mutex_unlock(&mi->mi_dir_struct_mutex);
if (err)
pr_debug("incfs: %s err:%d\n", __func__, err);
return err;
}
-/* Delete file referenced by backing_dentry and also its hardlink from .index */
-static int final_file_delete(struct mount_info *mi,
- struct dentry *backing_dentry)
+/*
+ * Delete file referenced by backing_dentry and if appropriate its hardlink
+ * from .index and .incomplete
+ */
+static int file_delete(struct mount_info *mi, struct dentry *dentry,
+ struct dentry *backing_dentry, int nlink)
{
struct dentry *index_file_dentry = NULL;
+ struct dentry *incomplete_file_dentry = NULL;
/* 2 chars per byte of file ID + 1 char for \0 */
char file_id_str[2 * sizeof(incfs_uuid_t) + 1] = {0};
ssize_t uuid_size = 0;
int error = 0;
WARN_ON(!mutex_is_locked(&mi->mi_dir_struct_mutex));
+
+ if (nlink > 3)
+ goto just_unlink;
+
uuid_size = vfs_getxattr(backing_dentry, INCFS_XATTR_ID_NAME,
file_id_str, 2 * sizeof(incfs_uuid_t));
if (uuid_size < 0) {
@@ -1686,17 +1148,50 @@
index_file_dentry = incfs_lookup_dentry(mi->mi_index_dir, file_id_str);
if (IS_ERR(index_file_dentry)) {
error = PTR_ERR(index_file_dentry);
+ index_file_dentry = NULL;
goto out;
}
- error = incfs_unlink(backing_dentry);
- if (error)
- goto out;
+ if (d_really_is_positive(index_file_dentry) && nlink > 0)
+ nlink--;
- if (d_really_is_positive(index_file_dentry))
+ if (nlink > 2)
+ goto just_unlink;
+
+ incomplete_file_dentry = incfs_lookup_dentry(mi->mi_incomplete_dir,
+ file_id_str);
+ if (IS_ERR(incomplete_file_dentry)) {
+ error = PTR_ERR(incomplete_file_dentry);
+ incomplete_file_dentry = NULL;
+ goto out;
+ }
+
+ if (d_really_is_positive(incomplete_file_dentry) && nlink > 0)
+ nlink--;
+
+ if (nlink > 1)
+ goto just_unlink;
+
+ if (d_really_is_positive(index_file_dentry)) {
error = incfs_unlink(index_file_dentry);
+ if (error)
+ goto out;
+ notify_unlink(dentry, file_id_str, INCFS_INDEX_NAME);
+ }
+
+ if (d_really_is_positive(incomplete_file_dentry)) {
+ error = incfs_unlink(incomplete_file_dentry);
+ if (error)
+ goto out;
+ notify_unlink(dentry, file_id_str, INCFS_INCOMPLETE_NAME);
+ }
+
+just_unlink:
+ error = incfs_unlink(backing_dentry);
+
out:
dput(index_file_dentry);
+ dput(incomplete_file_dentry);
if (error)
pr_debug("incfs: delete_file_from_index err:%d\n", error);
return error;
@@ -1709,6 +1204,9 @@
struct kstat stat;
int err = 0;
+ if (!mi)
+ return -EBADF;
+
err = mutex_lock_interruptible(&mi->mi_dir_struct_mutex);
if (err)
return err;
@@ -1716,11 +1214,17 @@
get_incfs_backing_path(dentry, &backing_path);
if (!backing_path.dentry) {
err = -EBADF;
- goto out;
+ goto path_err;
}
if (backing_path.dentry->d_parent == mi->mi_index_dir) {
/* Direct unlink from .index are not allowed. */
+ err = -EBUSY;
+ goto out;
+ }
+
+ if (backing_path.dentry->d_parent == mi->mi_incomplete_dir) {
+ /* Direct unlink from .incomplete are not allowed. */
err = -EBUSY;
goto out;
}
@@ -1730,20 +1234,12 @@
if (err)
goto out;
- if (stat.nlink == 2) {
- /*
- * This is the last named link to this file. The only one left
- * is in .index. Remove them both now.
- */
- err = final_file_delete(mi, backing_path.dentry);
- } else {
- /* There are other links to this file. Remove just this one. */
- err = incfs_unlink(backing_path.dentry);
- }
+ err = file_delete(mi, dentry, backing_path.dentry, stat.nlink);
d_drop(dentry);
out:
path_put(&backing_path);
+path_err:
if (err)
pr_debug("incfs: %s err:%d\n", __func__, err);
mutex_unlock(&mi->mi_dir_struct_mutex);
@@ -1758,6 +1254,9 @@
struct path backing_new_path = {};
int error = 0;
+ if (!mi)
+ return -EBADF;
+
error = mutex_lock_interruptible(&mi->mi_dir_struct_mutex);
if (error)
return error;
@@ -1767,6 +1266,12 @@
if (backing_new_path.dentry->d_parent == mi->mi_index_dir) {
/* Can't link to .index */
+ error = -EBUSY;
+ goto out;
+ }
+
+ if (backing_new_path.dentry->d_parent == mi->mi_incomplete_dir) {
+ /* Can't link to .incomplete */
error = -EBUSY;
goto out;
}
@@ -1804,6 +1309,9 @@
struct path backing_path = {};
int err = 0;
+ if (!mi)
+ return -EBADF;
+
err = mutex_lock_interruptible(&mi->mi_dir_struct_mutex);
if (err)
return err;
@@ -1811,11 +1319,17 @@
get_incfs_backing_path(dentry, &backing_path);
if (!backing_path.dentry) {
err = -EBADF;
- goto out;
+ goto path_err;
}
if (backing_path.dentry == mi->mi_index_dir) {
/* Can't delete .index */
+ err = -EBUSY;
+ goto out;
+ }
+
+ if (backing_path.dentry == mi->mi_incomplete_dir) {
+ /* Can't delete .incomplete */
err = -EBUSY;
goto out;
}
@@ -1825,6 +1339,8 @@
d_drop(dentry);
out:
path_put(&backing_path);
+
+path_err:
if (err)
pr_debug("incfs: %s err:%d\n", __func__, err);
mutex_unlock(&mi->mi_dir_struct_mutex);
@@ -1843,14 +1359,19 @@
struct dentry *trap;
int error = 0;
- if (!mi)
- return -EBADF;
-
error = mutex_lock_interruptible(&mi->mi_dir_struct_mutex);
if (error)
return error;
backing_old_dentry = get_incfs_dentry(old_dentry)->backing_path.dentry;
+
+ if (!backing_old_dentry || backing_old_dentry == mi->mi_index_dir ||
+ backing_old_dentry == mi->mi_incomplete_dir) {
+ /* Renaming .index or .incomplete not allowed */
+ error = -EBUSY;
+ goto exit;
+ }
+
backing_new_dentry = get_incfs_dentry(new_dentry)->backing_path.dentry;
dget(backing_old_dentry);
dget(backing_new_dentry);
@@ -1859,8 +1380,9 @@
backing_new_dir_dentry = dget_parent(backing_new_dentry);
target_inode = d_inode(new_dentry);
- if (backing_old_dir_dentry == mi->mi_index_dir) {
- /* Direct moves from .index are not allowed. */
+ if (backing_old_dir_dentry == mi->mi_index_dir ||
+ backing_old_dir_dentry == mi->mi_incomplete_dir) {
+ /* Direct moves from .index or .incomplete are not allowed. */
error = -EBUSY;
goto out;
}
@@ -1897,6 +1419,7 @@
dput(backing_new_dentry);
dput(backing_old_dentry);
+exit:
mutex_unlock(&mi->mi_dir_struct_mutex);
if (error)
pr_debug("incfs: %s err:%d\n", __func__, error);
@@ -1910,12 +1433,21 @@
struct file *backing_file = NULL;
struct path backing_path = {};
int err = 0;
+ int flags = O_NOATIME | O_LARGEFILE |
+ (S_ISDIR(inode->i_mode) ? O_RDONLY : O_RDWR);
const struct cred *old_cred;
+ WARN_ON(file->private_data);
+
+ if (!mi)
+ return -EBADF;
+
get_incfs_backing_path(file->f_path.dentry, &backing_path);
+ if (!backing_path.dentry)
+ return -EBADF;
+
old_cred = override_creds(mi->mi_owner);
- backing_file = dentry_open(&backing_path,
- O_RDWR | O_NOATIME | O_LARGEFILE, current_cred());
+ backing_file = dentry_open(&backing_path, flags, current_cred());
revert_creds(old_cred);
path_put(&backing_path);
@@ -1926,8 +1458,25 @@
}
if (S_ISREG(inode->i_mode)) {
+ struct incfs_file_data *fd = kzalloc(sizeof(*fd), GFP_NOFS);
+
+ if (!fd) {
+ err = -ENOMEM;
+ goto out;
+ }
+
+ *fd = (struct incfs_file_data) {
+ .fd_fill_permission = CANT_FILL,
+ };
+ file->private_data = fd;
+
err = make_inode_ready_for_data_ops(mi, inode, backing_file);
- file->private_data = (void *)CANT_FILL;
+ if (err)
+ goto out;
+
+ err = incfs_fsverity_file_open(inode, file);
+ if (err)
+ goto out;
} else if (S_ISDIR(inode->i_mode)) {
struct dir_file *dir = NULL;
@@ -1940,9 +1489,17 @@
err = -EBADF;
out:
- if (err)
- pr_debug("incfs: %s name:%s err: %d\n", __func__,
- file->f_path.dentry->d_name.name, err);
+ if (err) {
+ pr_debug("name:%s err: %d\n",
+ file->f_path.dentry->d_name.name, err);
+ if (S_ISREG(inode->i_mode))
+ kfree(file->private_data);
+ else if (S_ISDIR(inode->i_mode))
+ incfs_free_dir_file(file->private_data);
+
+ file->private_data = NULL;
+ }
+
if (backing_file)
fput(backing_file);
return err;
@@ -1951,9 +1508,8 @@
static int file_release(struct inode *inode, struct file *file)
{
if (S_ISREG(inode->i_mode)) {
- /* Do nothing.
- * data_file is released only by inode eviction.
- */
+ kfree(file->private_data);
+ file->private_data = NULL;
} else if (S_ISDIR(inode->i_mode)) {
struct dir_file *dir = get_incfs_dir_file(file);
@@ -2056,6 +1612,10 @@
if (ia->ia_valid & ATTR_SIZE)
return -EINVAL;
+ if ((ia->ia_valid & (ATTR_KILL_SUID|ATTR_KILL_SGID)) &&
+ (ia->ia_valid & ATTR_MODE))
+ return -EINVAL;
+
if (!di)
return -EINVAL;
backing_dentry = di->backing_path.dentry;
@@ -2085,6 +1645,43 @@
return simple_setattr(dentry, ia);
}
+
+static int incfs_getattr(const struct path *path,
+ struct kstat *stat, u32 request_mask,
+ unsigned int query_flags)
+{
+ struct inode *inode = d_inode(path->dentry);
+
+ generic_fillattr(inode, stat);
+
+ if (inode->i_ino < INCFS_START_INO_RANGE)
+ return 0;
+
+ stat->attributes &= ~STATX_ATTR_VERITY;
+ if (IS_VERITY(inode))
+ stat->attributes |= STATX_ATTR_VERITY;
+ stat->attributes_mask |= STATX_ATTR_VERITY;
+
+ if (request_mask & STATX_BLOCKS) {
+ struct kstat backing_kstat;
+ struct dentry_info *di = get_incfs_dentry(path->dentry);
+ int error = 0;
+ struct path *backing_path;
+
+ if (!di)
+ return -EFSCORRUPTED;
+ backing_path = &di->backing_path;
+ error = vfs_getattr(backing_path, &backing_kstat, STATX_BLOCKS,
+ AT_STATX_SYNC_AS_STAT);
+ if (error)
+ return error;
+
+ stat->blocks = backing_kstat.blocks;
+ }
+
+ return 0;
+}
+
static ssize_t incfs_getxattr(struct dentry *d, const char *name,
void *value, size_t size)
{
@@ -2092,9 +1689,7 @@
struct mount_info *mi = get_mount_info(d->d_sb);
char *stored_value;
size_t stored_size;
-
- if (!mi)
- return -EBADF;
+ int i;
if (di && di->backing_path.dentry)
return vfs_getxattr(di->backing_path.dentry, name, value, size);
@@ -2102,16 +1697,14 @@
if (strcmp(name, "security.selinux"))
return -ENODATA;
- if (!strcmp(d->d_iname, INCFS_PENDING_READS_FILENAME)) {
- stored_value = mi->pending_read_xattr;
- stored_size = mi->pending_read_xattr_size;
- } else if (!strcmp(d->d_iname, INCFS_LOG_FILENAME)) {
- stored_value = mi->log_xattr;
- stored_size = mi->log_xattr_size;
- } else {
+ for (i = 0; i < PSEUDO_FILE_COUNT; ++i)
+ if (!strcmp(d->d_iname, incfs_pseudo_file_names[i].data))
+ break;
+ if (i == PSEUDO_FILE_COUNT)
return -ENODATA;
- }
+ stored_value = mi->pseudo_file_xattr[i].data;
+ stored_size = mi->pseudo_file_xattr[i].len;
if (!stored_value)
return -ENODATA;
@@ -2120,7 +1713,6 @@
memcpy(value, stored_value, stored_size);
return stored_size;
-
}
@@ -2129,11 +1721,9 @@
{
struct dentry_info *di = get_incfs_dentry(d);
struct mount_info *mi = get_mount_info(d->d_sb);
- void **stored_value;
+ u8 **stored_value;
size_t *stored_size;
-
- if (!mi)
- return -EBADF;
+ int i;
if (di && di->backing_path.dentry)
return vfs_setxattr(di->backing_path.dentry, name, value, size,
@@ -2145,16 +1735,14 @@
if (size > INCFS_MAX_FILE_ATTR_SIZE)
return -E2BIG;
- if (!strcmp(d->d_iname, INCFS_PENDING_READS_FILENAME)) {
- stored_value = &mi->pending_read_xattr;
- stored_size = &mi->pending_read_xattr_size;
- } else if (!strcmp(d->d_iname, INCFS_LOG_FILENAME)) {
- stored_value = &mi->log_xattr;
- stored_size = &mi->log_xattr_size;
- } else {
+ for (i = 0; i < PSEUDO_FILE_COUNT; ++i)
+ if (!strcmp(d->d_iname, incfs_pseudo_file_names[i].data))
+ break;
+ if (i == PSEUDO_FILE_COUNT)
return -ENODATA;
- }
+ stored_value = &mi->pseudo_file_xattr[i].data;
+ stored_size = &mi->pseudo_file_xattr[i].len;
kfree (*stored_value);
*stored_value = kzalloc(size, GFP_NOFS);
if (!*stored_value)
@@ -2175,22 +1763,18 @@
return vfs_listxattr(di->backing_path.dentry, list, size);
}
-static int incfs_test_super(struct super_block *s, void *p)
-{
- return s->s_fs_info != NULL;
-}
-
struct dentry *incfs_mount_fs(struct file_system_type *type, int flags,
const char *dev_name, void *data)
{
struct mount_options options = {};
struct mount_info *mi = NULL;
struct path backing_dir_path = {};
- struct dentry *index_dir;
+ struct dentry *index_dir = NULL;
+ struct dentry *incomplete_dir = NULL;
struct super_block *src_fs_sb = NULL;
struct inode *root_inode = NULL;
- struct super_block *sb = sget(type, incfs_test_super, set_anon_super,
- flags, NULL);
+ struct super_block *sb = sget(type, NULL, set_anon_super, flags, NULL);
+ bool dir_created = false;
int error = 0;
if (IS_ERR(sb))
@@ -2199,7 +1783,7 @@
sb->s_op = &incfs_super_ops;
sb->s_d_op = &incfs_dentry_ops;
sb->s_flags |= S_NOATIME;
- sb->s_magic = (long)INCFS_MAGIC_NUMBER;
+ sb->s_magic = INCFS_MAGIC_NUMBER;
sb->s_time_gran = 1;
sb->s_blocksize = INCFS_DATA_FILE_BLOCK_SIZE;
sb->s_blocksize_bits = blksize_bits(sb->s_blocksize);
@@ -2207,17 +1791,23 @@
BUILD_BUG_ON(PAGE_SIZE != INCFS_DATA_FILE_BLOCK_SIZE);
+ if (!dev_name) {
+ pr_err("incfs: Backing dir is not set, filesystem can't be mounted.\n");
+ error = -ENOENT;
+ goto err_deactivate;
+ }
+
error = parse_options(&options, (char *)data);
if (error != 0) {
pr_err("incfs: Options parsing error. %d\n", error);
- goto err;
+ goto err_deactivate;
}
sb->s_bdi->ra_pages = options.readahead_pages;
if (!dev_name) {
pr_err("incfs: Backing dir is not set, filesystem can't be mounted.\n");
error = -ENOENT;
- goto err;
+ goto err_free_opts;
}
error = kern_path(dev_name, LOOKUP_FOLLOW | LOOKUP_DIRECTORY,
@@ -2226,62 +1816,80 @@
!d_really_is_positive(backing_dir_path.dentry)) {
pr_err("incfs: Error accessing: %s.\n",
dev_name);
- goto err;
+ goto err_free_opts;
}
src_fs_sb = backing_dir_path.dentry->d_sb;
sb->s_maxbytes = src_fs_sb->s_maxbytes;
+ sb->s_stack_depth = src_fs_sb->s_stack_depth + 1;
- if (!sb->s_fs_info) {
- mi = incfs_alloc_mount_info(sb, &options, &backing_dir_path);
-
- if (IS_ERR_OR_NULL(mi)) {
- error = PTR_ERR(mi);
- pr_err("incfs: Error allocating mount info. %d\n",
- error);
- mi = NULL;
- goto err;
- }
- sb->s_fs_info = mi;
- } else {
- mi = sb->s_fs_info;
+ if (sb->s_stack_depth > FILESYSTEM_MAX_STACK_DEPTH) {
+ error = -EINVAL;
+ goto err_put_path;
}
- index_dir = open_or_create_index_dir(backing_dir_path.dentry);
+ mi = incfs_alloc_mount_info(sb, &options, &backing_dir_path);
+ if (IS_ERR_OR_NULL(mi)) {
+ error = PTR_ERR(mi);
+ pr_err("incfs: Error allocating mount info. %d\n", error);
+ goto err_put_path;
+ }
+
+ sb->s_fs_info = mi;
+ mi->mi_backing_dir_path = backing_dir_path;
+ index_dir = open_or_create_special_dir(backing_dir_path.dentry,
+ INCFS_INDEX_NAME, &dir_created);
if (IS_ERR_OR_NULL(index_dir)) {
error = PTR_ERR(index_dir);
pr_err("incfs: Can't find or create .index dir in %s\n",
dev_name);
- goto err;
+ /* No need to null index_dir since we don't put it */
+ goto err_put_path;
}
+
mi->mi_index_dir = index_dir;
+ mi->mi_index_free = dir_created;
+
+ incomplete_dir = open_or_create_special_dir(backing_dir_path.dentry,
+ INCFS_INCOMPLETE_NAME,
+ &dir_created);
+ if (IS_ERR_OR_NULL(incomplete_dir)) {
+ error = PTR_ERR(incomplete_dir);
+ pr_err("incfs: Can't find or create .incomplete dir in %s\n",
+ dev_name);
+ /* No need to null incomplete_dir since we don't put it */
+ goto err_put_path;
+ }
+ mi->mi_incomplete_dir = incomplete_dir;
+ mi->mi_incomplete_free = dir_created;
root_inode = fetch_regular_inode(sb, backing_dir_path.dentry);
if (IS_ERR(root_inode)) {
error = PTR_ERR(root_inode);
- goto err;
+ goto err_put_path;
}
+ sb->s_root = d_make_root(root_inode);
if (!sb->s_root) {
- sb->s_root = d_make_root(root_inode);
- if (!sb->s_root) {
- error = -ENOMEM;
- goto err;
- }
- error = incfs_init_dentry(sb->s_root, &backing_dir_path);
- if (error)
- goto err;
+ error = -ENOMEM;
+ goto err_put_path;
}
+ error = incfs_init_dentry(sb->s_root, &backing_dir_path);
+ if (error)
+ goto err_put_path;
- mi->mi_backing_dir_path = backing_dir_path;
+ path_put(&backing_dir_path);
sb->s_flags |= SB_ACTIVE;
pr_debug("incfs: mount\n");
return dget(sb->s_root);
-err:
- sb->s_fs_info = NULL;
+
+err_put_path:
path_put(&backing_dir_path);
- incfs_free_mount_info(mi);
+err_free_opts:
+ free_options(&options);
+err_deactivate:
deactivate_locked_super(sb);
+ pr_err("incfs: mount failed %d\n", error);
return ERR_PTR(error);
}
@@ -2291,39 +1899,62 @@
struct mount_info *mi = get_mount_info(sb);
int err = 0;
- if (!mi)
- return err;
-
sync_filesystem(sb);
err = parse_options(&options, (char *)data);
if (err)
return err;
+ if (options.report_uid != mi->mi_options.report_uid) {
+ pr_err("incfs: Can't change report_uid mount option on remount\n");
+ err = -EOPNOTSUPP;
+ goto out;
+ }
+
err = incfs_realloc_mount_info(mi, &options);
if (err)
- return err;
+ goto out;
pr_debug("incfs: remount\n");
- return 0;
+
+out:
+ free_options(&options);
+ return err;
}
void incfs_kill_sb(struct super_block *sb)
{
struct mount_info *mi = sb->s_fs_info;
+ struct inode *dinode = NULL;
pr_debug("incfs: unmount\n");
- vfs_rmdir(d_inode(mi->mi_backing_dir_path.dentry), mi->mi_index_dir);
+
+ /*
+ * We must kill the super before freeing mi, since killing the super
+ * triggers inode eviction, which triggers the final update of the
+ * backing file, which uses certain information for mi
+ */
kill_anon_super(sb);
- incfs_free_mount_info(mi);
- sb->s_fs_info = NULL;
+
+ if (mi) {
+ if (mi->mi_backing_dir_path.dentry)
+ dinode = d_inode(mi->mi_backing_dir_path.dentry);
+
+ if (dinode) {
+ if (mi->mi_index_dir && mi->mi_index_free)
+ vfs_rmdir(dinode, mi->mi_index_dir);
+
+ if (mi->mi_incomplete_dir && mi->mi_incomplete_free)
+ vfs_rmdir(dinode, mi->mi_incomplete_dir);
+ }
+
+ incfs_free_mount_info(mi);
+ sb->s_fs_info = NULL;
+ }
}
static int show_options(struct seq_file *m, struct dentry *root)
{
struct mount_info *mi = get_mount_info(root->d_sb);
-
- if (!mi)
- return -EBADF;
seq_printf(m, ",read_timeout_ms=%u", mi->mi_options.read_timeout_ms);
seq_printf(m, ",readahead=%u", mi->mi_options.readahead_pages);
@@ -2332,9 +1963,11 @@
seq_printf(m, ",rlog_wakeup_cnt=%u",
mi->mi_options.read_log_wakeup_count);
}
- if (mi->mi_options.no_backing_file_cache)
- seq_puts(m, ",no_bf_cache");
- if (mi->mi_options.no_backing_file_readahead)
- seq_puts(m, ",no_bf_readahead");
+ if (mi->mi_options.report_uid)
+ seq_puts(m, ",report_uid");
+
+ if (mi->mi_sysfs_node)
+ seq_printf(m, ",sysfs_name=%s",
+ kobject_name(&mi->mi_sysfs_node->isn_sysfs_node));
return 0;
}
--
Gitblit v1.6.2