chiark
/
gitweb
/
~ianmdlvl
/
elogind.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
stop complaining about unknown kernel cmdline options
[elogind.git]
/
src
/
gpt-auto-generator
/
gpt-auto-generator.c
diff --git
a/src/gpt-auto-generator/gpt-auto-generator.c
b/src/gpt-auto-generator/gpt-auto-generator.c
index be91977de9aa298b79653d288a9d7cb8287ca535..7dcbbfe497a52b66f0bf23de38222813b63de115 100644
(file)
--- a/
src/gpt-auto-generator/gpt-auto-generator.c
+++ b/
src/gpt-auto-generator/gpt-auto-generator.c
@@
-101,7
+101,7
@@
static int add_swap(const char *path) {
return 0;
}
return 0;
}
-static int add_cryptsetup(const char *id, const char *what, char **device) {
+static int add_cryptsetup(const char *id, const char *what,
bool rw,
char **device) {
_cleanup_free_ char *e = NULL, *n = NULL, *p = NULL, *d = NULL, *to = NULL;
_cleanup_fclose_ FILE *f = NULL;
char *from, *ret;
_cleanup_free_ char *e = NULL, *n = NULL, *p = NULL, *d = NULL, *to = NULL;
_cleanup_fclose_ FILE *f = NULL;
char *from, *ret;
@@
-149,10
+149,10
@@
static int add_cryptsetup(const char *id, const char *what, char **device) {
"Type=oneshot\n"
"RemainAfterExit=yes\n"
"TimeoutSec=0\n" /* the binary handles timeouts anyway */
"Type=oneshot\n"
"RemainAfterExit=yes\n"
"TimeoutSec=0\n" /* the binary handles timeouts anyway */
- "ExecStart=" SYSTEMD_CRYPTSETUP_PATH " attach '%s' '%s'\n"
+ "ExecStart=" SYSTEMD_CRYPTSETUP_PATH " attach '%s' '%s'
'' '%s'
\n"
"ExecStop=" SYSTEMD_CRYPTSETUP_PATH " detach '%s'\n",
d, d,
"ExecStop=" SYSTEMD_CRYPTSETUP_PATH " detach '%s'\n",
d, d,
- id, what,
+ id, what,
rw ? "" : "read-only",
id);
fflush(f);
id);
fflush(f);
@@
-223,7
+223,7
@@
static int add_mount(
const char *what,
const char *where,
const char *fstype,
const char *what,
const char *where,
const char *fstype,
-
const char *options
,
+
bool rw
,
const char *description,
const char *post) {
const char *description,
const char *post) {
@@
-240,7
+240,7
@@
static int add_mount(
if (streq_ptr(fstype, "crypto_LUKS")) {
if (streq_ptr(fstype, "crypto_LUKS")) {
- r = add_cryptsetup(id, what, &crypto_what);
+ r = add_cryptsetup(id, what,
rw,
&crypto_what);
if (r < 0)
return r;
if (r < 0)
return r;
@@
-286,8
+286,7
@@
static int add_mount(
if (fstype)
fprintf(f, "Type=%s\n", fstype);
if (fstype)
fprintf(f, "Type=%s\n", fstype);
- if (options)
- fprintf(f, "Options=%s\n", options);
+ fprintf(f, "Options=%s\n", rw ? "rw" : "ro");
fflush(f);
if (ferror(f)) {
fflush(f);
if (ferror(f)) {
@@
-314,6
+313,7
@@
static int probe_and_add_mount(
const char *id,
const char *what,
const char *where,
const char *id,
const char *what,
const char *where,
+ bool rw,
const char *description,
const char *post) {
const char *description,
const char *post) {
@@
-365,7
+365,7
@@
static int probe_and_add_mount(
what,
where,
fstype,
what,
where,
fstype,
-
NULL
,
+
rw
,
description,
post);
}
description,
post);
}
@@
-381,6
+381,7
@@
static int enumerate_partitions(dev_t devnum) {
struct udev_device *parent = NULL;
const char *node, *pttype, *devtype;
int home_nr = -1, srv_nr = -1;
struct udev_device *parent = NULL;
const char *node, *pttype, *devtype;
int home_nr = -1, srv_nr = -1;
+ bool home_rw = true, srv_rw = true;
blkid_partlist pl;
int r, k;
dev_t pn;
blkid_partlist pl;
int r, k;
dev_t pn;
@@
-394,22
+395,30
@@
static int enumerate_partitions(dev_t devnum) {
return log_oom();
parent = udev_device_get_parent(d);
return log_oom();
parent = udev_device_get_parent(d);
- if (!parent)
- return log_oom();
+ if (!parent) {
+ log_debug("Not a partitioned device, ignoring.");
+ return 0;
+ }
/* Does it have a devtype? */
devtype = udev_device_get_devtype(parent);
/* Does it have a devtype? */
devtype = udev_device_get_devtype(parent);
- if (!devtype)
+ if (!devtype) {
+ log_debug("Parent doesn't have a device type, ignoring.");
return 0;
return 0;
+ }
/* Is this a disk or a partition? We only care for disks... */
/* Is this a disk or a partition? We only care for disks... */
- if (!streq(devtype, "disk"))
+ if (!streq(devtype, "disk")) {
+ log_debug("Parent isn't a raw disk, ignoring.");
return 0;
return 0;
+ }
/* Does it have a device node? */
node = udev_device_get_devnode(parent);
/* Does it have a device node? */
node = udev_device_get_devnode(parent);
- if (!node)
+ if (!node) {
+ log_debug("Parent device does not have device node, ignoring.");
return 0;
return 0;
+ }
log_debug("Root device %s.", node);
log_debug("Root device %s.", node);
@@
-427,8
+436,6
@@
static int enumerate_partitions(dev_t devnum) {
return -errno;
}
return -errno;
}
- blkid_probe_enable_superblocks(b, 1);
- blkid_probe_set_superblocks_flags(b, BLKID_SUBLKS_TYPE);
blkid_probe_enable_partitions(b, 1);
blkid_probe_set_partitions_flags(b, BLKID_PARTS_ENTRY_DETAILS);
blkid_probe_enable_partitions(b, 1);
blkid_probe_set_partitions_flags(b, BLKID_PARTS_ENTRY_DETAILS);
@@
-453,8
+460,10
@@
static int enumerate_partitions(dev_t devnum) {
}
/* We only do this all for GPT... */
}
/* We only do this all for GPT... */
- if (!streq_ptr(pttype, "gpt"))
+ if (!streq_ptr(pttype, "gpt")) {
+ log_debug("Not a GPT partition table, ignoring.");
return 0;
return 0;
+ }
errno = 0;
pl = blkid_probe_get_partitions(b);
errno = 0;
pl = blkid_probe_get_partitions(b);
@@
-492,6
+501,7
@@
static int enumerate_partitions(dev_t devnum) {
blkid_partition pp;
dev_t qn;
int nr;
blkid_partition pp;
dev_t qn;
int nr;
+ unsigned long long flags;
q = udev_device_new_from_syspath(udev, udev_list_entry_get_name(item));
if (!q)
q = udev_device_new_from_syspath(udev, udev_list_entry_get_name(item));
if (!q)
@@
-515,6
+525,13
@@
static int enumerate_partitions(dev_t devnum) {
if (!pp)
continue;
if (!pp)
continue;
+ flags = blkid_partition_get_flags(pp);
+
+ /* Ignore partitions that are not marked for automatic
+ * mounting on discovery */
+ if (flags & GPT_FLAG_NO_AUTO)
+ continue;
+
nr = blkid_partition_get_partno(pp);
if (nr < 0)
continue;
nr = blkid_partition_get_partno(pp);
if (nr < 0)
continue;
@@
-528,6
+545,11
@@
static int enumerate_partitions(dev_t devnum) {
if (sd_id128_equal(type_id, GPT_SWAP)) {
if (sd_id128_equal(type_id, GPT_SWAP)) {
+ if (flags & GPT_FLAG_READ_ONLY) {
+ log_debug("%s marked as read-only swap partition, which is bogus, ignoring.", subnode);
+ continue;
+ }
+
k = add_swap(subnode);
if (k < 0)
r = k;
k = add_swap(subnode);
if (k < 0)
r = k;
@@
-539,6
+561,7
@@
static int enumerate_partitions(dev_t devnum) {
continue;
home_nr = nr;
continue;
home_nr = nr;
+ home_rw = !(flags & GPT_FLAG_READ_ONLY),
free(home);
home = strdup(subnode);
free(home);
home = strdup(subnode);
@@
-552,6
+575,7
@@
static int enumerate_partitions(dev_t devnum) {
continue;
srv_nr = nr;
continue;
srv_nr = nr;
+ srv_rw = !(flags & GPT_FLAG_READ_ONLY),
free(srv);
srv = strdup(node);
free(srv);
srv = strdup(node);
@@
-561,13
+585,13
@@
static int enumerate_partitions(dev_t devnum) {
}
if (home) {
}
if (home) {
- k = probe_and_add_mount("home", home, "/home", "Home Partition", SPECIAL_LOCAL_FS_TARGET);
+ k = probe_and_add_mount("home", home, "/home",
home_rw,
"Home Partition", SPECIAL_LOCAL_FS_TARGET);
if (k < 0)
r = k;
}
if (srv) {
if (k < 0)
r = k;
}
if (srv) {
- k = probe_and_add_mount("srv", srv, "/srv", "Server Data Partition", SPECIAL_LOCAL_FS_TARGET);
+ k = probe_and_add_mount("srv", srv, "/srv",
srv_rw,
"Server Data Partition", SPECIAL_LOCAL_FS_TARGET);
if (k < 0)
r = k;
}
if (k < 0)
r = k;
}
@@
-671,8
+695,6
@@
static int parse_proc_cmdline_item(const char *key, const char *value) {
arg_root_rw = true;
else if (streq(key, "ro") && !value)
arg_root_rw = false;
arg_root_rw = true;
else if (streq(key, "ro") && !value)
arg_root_rw = false;
- else if (startswith(key, "systemd.gpt-auto.") || startswith(key, "rd.systemd.gpt-auto."))
- log_warning("Unknown kernel switch %s. Ignoring.", key);
return 0;
}
return 0;
}
@@
-702,10
+724,10
@@
static int add_root_mount(void) {
return add_mount(
"root",
return add_mount(
"root",
- "/dev/
disk/by-id/
gpt-auto-root",
+ "/dev/gpt-auto-root",
in_initrd() ? "/sysroot" : "/",
NULL,
in_initrd() ? "/sysroot" : "/",
NULL,
- arg_root_rw
? "rw" : "ro"
,
+ arg_root_rw,
"Root Partition",
in_initrd() ? SPECIAL_INITRD_ROOT_FS_TARGET : SPECIAL_LOCAL_FS_TARGET);
#else
"Root Partition",
in_initrd() ? SPECIAL_INITRD_ROOT_FS_TARGET : SPECIAL_LOCAL_FS_TARGET);
#else