[OpenWrt-Devel] [PATCH fstools 2/5] block: support hierarchical mount/umount
Yousong Zhou
yszhou4tech at gmail.com
Tue Oct 29 08:39:47 EDT 2019
This is needed when e.g. sda1 needs to be mounted to /mnt and sda2
/mnt/srv. We need to make sure sda1 was mounted first.
"devices" is now an ordered list with elements pointing to both "struct
probe_info" and "struct mount". The order is firstly on presence of
mount info, then strlen(mount.target), then strcmp(probe_info.dev)
Resolves FS#2214
Signed-off-by: Yousong Zhou <yszhou4tech at gmail.com>
---
block.c | 152 +++++++++++++++++++++++++++++++++++++-------------------
1 file changed, 101 insertions(+), 51 deletions(-)
diff --git a/block.c b/block.c
index 99672fb..66dcf9c 100644
--- a/block.c
+++ b/block.c
@@ -82,9 +82,20 @@ struct mount {
unsigned int prio;
};
-static struct vlist_tree mounts;
+struct device {
+ struct vlist_node node;
+
+ struct probe_info *pr;
+ struct mount *m;
+};
+
+static void vlist_nop_update(struct vlist_tree *tree, struct vlist_node *node_new,
+ struct vlist_node *node_old);
+static int devices_cmp(const void *k1, const void *k2, void *ptr);
+
static struct blob_buf b;
-static LIST_HEAD(devices);
+static VLIST_TREE(mounts, avl_strcmp, vlist_nop_update, false, false);
+static VLIST_TREE(devices, devices_cmp, vlist_nop_update, false, false);
static int anon_mount, anon_swap, auto_mount, auto_swap, check_fs;
static unsigned int delay_root;
@@ -409,9 +420,10 @@ static struct mount* find_block(const char *uuid, const char *label, const char
return NULL;
}
-static void mounts_update(struct vlist_tree *tree, struct vlist_node *node_new,
+static void vlist_nop_update(struct vlist_tree *tree, struct vlist_node *node_new,
struct vlist_node *node_old)
{
+ // NOTE: free on delete skipped
}
static struct uci_package * config_try_load(struct uci_context *ctx, char *path)
@@ -442,8 +454,6 @@ static int config_load(char *cfg)
struct uci_element *e;
char path[64];
- vlist_init(&mounts, avl_strcmp, mounts_update);
-
if (cfg) {
snprintf(path, sizeof(path), "%s/upper/etc/config/fstab", cfg);
pkg = config_try_load(ctx, path);
@@ -482,15 +492,15 @@ static int config_load(char *cfg)
static struct probe_info* _probe_path(char *path)
{
- struct probe_info *pr;
+ struct device *dev;
char tmppath[64];
/* skip ubi device if ubiblock device is present */
if (path[5] == 'u' && path[6] == 'b' && path[7] == 'i' &&
path[8] >= '0' && path[8] <= '9' ) {
snprintf(tmppath, sizeof(tmppath), "/dev/ubiblock%s", path + 8);
- list_for_each_entry(pr, &devices, list)
- if (!strcasecmp(pr->dev, tmppath))
+ vlist_for_each_element(&devices, dev, node)
+ if (!strcasecmp(dev->pr->dev, tmppath))
return NULL;
}
@@ -502,14 +512,20 @@ static int _cache_load(const char *path)
int gl_flags = GLOB_NOESCAPE | GLOB_MARK;
int j;
glob_t gl;
+ struct device *dev;
if (glob(path, gl_flags, NULL, &gl) < 0)
return -1;
for (j = 0; j < gl.gl_pathc; j++) {
struct probe_info *pr = _probe_path(gl.gl_pathv[j]);
- if (pr)
- list_add_tail(&pr->list, &devices);
+ if (pr) {
+ dev = malloc(sizeof(struct device));
+ dev->pr = pr;
+ dev->m = find_block(pr->uuid, pr->label,
+ basename(pr->dev), NULL);
+ vlist_add(&devices, &dev->node, dev);
+ }
}
globfree(&gl);
@@ -517,8 +533,30 @@ static int _cache_load(const char *path)
return 0;
}
+static int devices_cmp(const void *k1, const void *k2, void *ptr)
+{
+ struct device *dev1 = (struct device *)k1;
+ struct device *dev2 = (struct device *)k2;
+
+ if (dev1->m) {
+ if (!dev2->m)
+ return -1;
+ if (dev1->m->type == TYPE_MOUNT && dev2->m->type == TYPE_MOUNT &&
+ dev1->m->target && dev2->m->target) {
+ int len1 = strlen(dev1->m->target);
+ int len2 = strlen(dev2->m->target);
+ if (len1 != len2)
+ return len1 - len2;
+ }
+ } else if (dev2->m) {
+ return 1;
+ }
+ return strcmp(dev1->pr->dev, dev2->pr->dev);
+}
+
static void cache_load(int mtd)
{
+ vlist_update(&devices);
if (mtd) {
_cache_load("/dev/mtdblock*");
_cache_load("/dev/ubiblock*");
@@ -533,6 +571,7 @@ static void cache_load(int mtd)
_cache_load("/dev/vd*");
_cache_load("/dev/xvd*");
_cache_load("/dev/mapper/*");
+ vlist_flush(&devices);
}
@@ -553,24 +592,24 @@ static int print_block_uci(struct probe_info *pr)
return 0;
}
-static struct probe_info* find_block_info(char *uuid, char *label, char *path)
+static struct device* find_block_device(char *uuid, char *label, char *path)
{
- struct probe_info *pr = NULL;
+ struct device *dev;
if (uuid)
- list_for_each_entry(pr, &devices, list)
- if (pr->uuid && !strcasecmp(pr->uuid, uuid))
- return pr;
+ vlist_for_each_element(&devices, dev, node)
+ if (dev->pr->uuid && !strcasecmp(dev->pr->uuid, uuid))
+ return dev;
if (label)
- list_for_each_entry(pr, &devices, list)
- if (pr->label && !strcmp(pr->label, label))
- return pr;
+ vlist_for_each_element(&devices, dev, node)
+ if (dev->pr->label && !strcmp(dev->pr->label, label))
+ return dev;
if (path)
- list_for_each_entry(pr, &devices, list)
- if (pr->dev && !strcmp(basename(pr->dev), basename(path)))
- return pr;
+ vlist_for_each_element(&devices, dev, node)
+ if (dev->pr->dev && !strcmp(basename(dev->pr->dev), basename(path)))
+ return dev;
return NULL;
}
@@ -1022,18 +1061,20 @@ static int blockd_notify(char *device, struct mount *m, struct probe_info *pr)
return err;
}
-static int mount_device(struct probe_info *pr, int type)
+static int mount_device(struct device *dev, int type)
{
struct mount *m;
+ struct probe_info *pr;
char _target[32];
char *target;
char *device;
char *mp;
int err;
- if (!pr)
+ if (!dev)
return -1;
+ pr = dev->pr;
device = basename(pr->dev);
if (!strcmp(pr->type, "swap")) {
@@ -1053,7 +1094,7 @@ static int mount_device(struct probe_info *pr, int type)
return -1;
}
- m = find_block(pr->uuid, pr->label, device, NULL);
+ m = dev->m;
if (m && m->extroot)
return -1;
@@ -1170,7 +1211,7 @@ static int mount_action(char *action, char *device, int type)
return -1;
cache_load(0);
- return mount_device(find_block_info(NULL, NULL, path), type);
+ return mount_device(find_block_device(NULL, NULL, path), type);
}
static int main_hotplug(int argc, char **argv)
@@ -1186,19 +1227,21 @@ static int main_autofs(int argc, char **argv)
return -1;
if (!strcmp(argv[2], "start")) {
+ struct device *dev;
struct probe_info *pr;
if (config_load(NULL))
return -1;
cache_load(0);
- list_for_each_entry(pr, &devices, list) {
+ vlist_for_each_element(&devices, dev, node) {
struct mount *m;
+ pr = dev->pr;
if (!strcmp(pr->type, "swap"))
continue;
- m = find_block(pr->uuid, pr->label, NULL, NULL);
+ m = dev->m;
if (m && m->extroot)
continue;
@@ -1362,7 +1405,8 @@ static int test_fs_support(const char *name)
static int check_extroot(char *path)
{
- struct probe_info *pr = NULL;
+ struct device *dev;
+ struct probe_info *pr;
char devpath[32];
#ifdef UBIFS_EXTROOT
@@ -1385,7 +1429,8 @@ static int check_extroot(char *path)
}
#endif
- list_for_each_entry(pr, &devices, list) {
+ vlist_for_each_element(&devices, dev, node) {
+ pr = dev->pr;
if (!strcmp(pr->dev, devpath)) {
struct stat s;
FILE *fp = NULL;
@@ -1443,6 +1488,7 @@ static int mount_extroot(char *cfg)
char overlay[] = "/tmp/extroot/overlay";
char mnt[] = "/tmp/extroot/mnt";
char *path = mnt;
+ struct device *dev;
struct probe_info *pr;
struct mount *m;
int err = -1;
@@ -1463,16 +1509,17 @@ static int mount_extroot(char *cfg)
}
/* Find block device pointed by the mount config */
- pr = find_block_info(m->uuid, m->label, m->device);
+ dev = find_block_device(m->uuid, m->label, m->device);
- if (!pr && delay_root){
+ if (!dev && delay_root){
ULOG_INFO("extroot: device not present, retrying in %u seconds\n", delay_root);
sleep(delay_root);
make_devs();
cache_load(0);
- pr = find_block_info(m->uuid, m->label, m->device);
+ dev = find_block_device(m->uuid, m->label, m->device);
}
- if (pr) {
+ if (dev) {
+ pr = dev->pr;
if (strncmp(pr->type, "ext", 3) &&
strncmp(pr->type, "f2fs", 4) &&
strncmp(pr->type, "btrfs", 5) &&
@@ -1516,7 +1563,6 @@ static int mount_extroot(char *cfg)
static int main_extroot(int argc, char **argv)
{
- struct probe_info *pr;
char blkdev_path[32] = { 0 };
int err = -1;
#ifdef UBIFS_EXTROOT
@@ -1545,8 +1591,8 @@ static int main_extroot(int argc, char **argv)
/* Start with looking for MTD partition */
find_block_mtd("\"rootfs_data\"", blkdev_path, sizeof(blkdev_path));
if (blkdev_path[0]) {
- pr = find_block_info(NULL, NULL, blkdev_path);
- if (pr && !strcmp(pr->type, "jffs2")) {
+ struct device *dev = find_block_device(NULL, NULL, blkdev_path);
+ if (dev && !strcmp(dev->pr->type, "jffs2")) {
char cfg[] = "/tmp/jffs_cfg";
/*
@@ -1592,14 +1638,14 @@ static int main_extroot(int argc, char **argv)
static int main_mount(int argc, char **argv)
{
- struct probe_info *pr;
+ struct device *dev;
if (config_load(NULL))
return -1;
cache_load(1);
- list_for_each_entry(pr, &devices, list)
- mount_device(pr, TYPE_DEV);
+ vlist_for_each_element(&devices, dev, node)
+ mount_device(dev, TYPE_DEV);
handle_swapfiles(true);
@@ -1608,6 +1654,7 @@ static int main_mount(int argc, char **argv)
static int main_umount(int argc, char **argv)
{
+ struct device *dev;
struct probe_info *pr;
bool all = false;
@@ -1621,13 +1668,14 @@ static int main_umount(int argc, char **argv)
if (argc == 3)
all = !strcmp(argv[2], "-a");
- list_for_each_entry(pr, &devices, list) {
+ vlist_for_each_element_reverse(&devices, dev, node) {
struct mount *m;
+ pr = dev->pr;
if (!strcmp(pr->type, "swap"))
continue;
- m = find_block(pr->uuid, pr->label, basename(pr->dev), NULL);
+ m = dev->m;
if (m && m->extroot)
continue;
@@ -1639,7 +1687,7 @@ static int main_umount(int argc, char **argv)
static int main_detect(int argc, char **argv)
{
- struct probe_info *pr;
+ struct device *dev;
cache_load(0);
printf("config 'global'\n");
@@ -1649,8 +1697,8 @@ static int main_detect(int argc, char **argv)
printf("\toption\tauto_mount\t'1'\n");
printf("\toption\tdelay_root\t'5'\n");
printf("\toption\tcheck_fs\t'0'\n\n");
- list_for_each_entry(pr, &devices, list)
- print_block_uci(pr);
+ vlist_for_each_element(&devices, dev, node)
+ print_block_uci(dev->pr);
return 0;
}
@@ -1658,12 +1706,12 @@ static int main_detect(int argc, char **argv)
static int main_info(int argc, char **argv)
{
int i;
- struct probe_info *pr;
+ struct device *dev;
cache_load(1);
if (argc == 2) {
- list_for_each_entry(pr, &devices, list)
- print_block_info(pr);
+ vlist_for_each_element(&devices, dev, node)
+ print_block_info(dev->pr);
return 0;
};
@@ -1679,9 +1727,9 @@ static int main_info(int argc, char **argv)
ULOG_ERR("%s is not a block device\n", argv[i]);
continue;
}
- pr = find_block_info(NULL, NULL, argv[i]);
- if (pr)
- print_block_info(pr);
+ dev = find_block_device(NULL, NULL, argv[i]);
+ if (dev)
+ print_block_info(dev->pr);
}
return 0;
@@ -1703,6 +1751,7 @@ static int main_swapon(int argc, char **argv)
FILE *fp;
char *lineptr;
size_t s;
+ struct device *dev;
struct probe_info *pr;
int flags = 0;
int pri;
@@ -1727,7 +1776,8 @@ static int main_swapon(int argc, char **argv)
return 0;
case 'a':
cache_load(0);
- list_for_each_entry(pr, &devices, list) {
+ vlist_for_each_element(&devices, dev, node) {
+ pr = dev->pr;
if (strcmp(pr->type, "swap"))
continue;
if (swapon(pr->dev, 0))
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel
More information about the openwrt-devel
mailing list