chiark
/
gitweb
/
~ianmdlvl
/
elogind.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
core: convert log_unit_*() to log_unit_*_errno()
[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 19c5eea9de2fc24a5a97cb3589d77d08f20d39c7..9803fca420f271acc787d3befd86be21a4c63660 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;
@@
-124,7
+124,7
@@
static int add_cryptsetup(const char *id, const char *what, char **device) {
return log_oom();
p = strjoin(arg_dest, "/", n, NULL);
return log_oom();
p = strjoin(arg_dest, "/", n, NULL);
- if (!
n
)
+ if (!
p
)
return log_oom();
f = fopen(p, "wxe");
return log_oom();
f = fopen(p, "wxe");
@@
-144,15
+144,14
@@
static int add_cryptsetup(const char *id, const char *what, char **device) {
"Before=umount.target cryptsetup.target\n"
"After=%s\n"
"IgnoreOnIsolate=true\n"
"Before=umount.target cryptsetup.target\n"
"After=%s\n"
"IgnoreOnIsolate=true\n"
- "After=systemd-readahead-collect.service systemd-readahead-replay.service\n\n"
"[Service]\n"
"Type=oneshot\n"
"RemainAfterExit=yes\n"
"TimeoutSec=0\n" /* the binary handles timeouts anyway */
"[Service]\n"
"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);
@@
-206,7
+205,7
@@
static int add_cryptsetup(const char *id, const char *what, char **device) {
"[Unit]\n"
"JobTimeoutSec=0\n"); /* the binary handles timeouts anyway */
if (r < 0) {
"[Unit]\n"
"JobTimeoutSec=0\n"); /* the binary handles timeouts anyway */
if (r < 0) {
- log_error
("Failed to write device drop-in: %s", strerror(-r)
);
+ log_error
_errno(r, "Failed to write device drop-in: %m"
);
return r;
}
return r;
}
@@
-223,7
+222,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
+239,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
+285,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
+312,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
+364,7
@@
static int probe_and_add_mount(
what,
where,
fstype,
what,
where,
fstype,
-
NULL
,
+
rw
,
description,
post);
}
description,
post);
}
@@
-381,6
+380,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
+394,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)
+ if (!parent) {
+ log_debug("Not a partitioned device, ignoring.");
return 0;
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
+435,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
+459,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);
@@
-480,7
+488,7
@@
static int enumerate_partitions(dev_t devnum) {
r = udev_enumerate_scan_devices(e);
if (r < 0) {
r = udev_enumerate_scan_devices(e);
if (r < 0) {
- log_error
("Failed to enumerate partitions on %s: %s", node, strerror(-r)
);
+ log_error
_errno(r, "Failed to enumerate partitions on %s: %m", node
);
return r;
}
return r;
}
@@
-492,6
+500,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
+524,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
+544,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
+560,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
+574,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
+584,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;
}
@@
-657,8
+680,8
@@
static int parse_proc_cmdline_item(const char *key, const char *value) {
r = parse_boolean(value);
if (r < 0)
log_warning("Failed to parse gpt-auto switch %s. Ignoring.", value);
r = parse_boolean(value);
if (r < 0)
log_warning("Failed to parse gpt-auto switch %s. Ignoring.", value);
-
- arg_enabled = r;
+ else
+
arg_enabled = r;
} else if (streq(key, "root") && value) {
} else if (streq(key, "root") && value) {
@@
-671,8
+694,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;
}
@@
-692,7
+713,7
@@
static int add_root_mount(void) {
log_debug("EFI loader partition unknown, exiting.");
return 0;
} else if (r < 0) {
log_debug("EFI loader partition unknown, exiting.");
return 0;
} else if (r < 0) {
- log_error
("Failed to read ESP partition UUID: %s", strerror(-r)
);
+ log_error
_errno(r, "Failed to read ESP partition UUID: %m"
);
return r;
}
return r;
}
@@
-705,7
+726,7
@@
static int add_root_mount(void) {
"/dev/gpt-auto-root",
in_initrd() ? "/sysroot" : "/",
NULL,
"/dev/gpt-auto-root",
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
@@
-719,7
+740,7
@@
static int add_mounts(void) {
r = get_block_device("/", &devno);
if (r < 0) {
r = get_block_device("/", &devno);
if (r < 0) {
- log_error
("Failed to determine block device of root file system: %s", strerror(-r)
);
+ log_error
_errno(r, "Failed to determine block device of root file system: %m"
);
return r;
} else if (r == 0) {
log_debug("Root file system not on a (single) block device.");
return r;
} else if (r == 0) {
log_debug("Root file system not on a (single) block device.");
@@
-751,8
+772,9
@@
int main(int argc, char *argv[]) {
return EXIT_SUCCESS;
}
return EXIT_SUCCESS;
}
- if (parse_proc_cmdline(parse_proc_cmdline_item) < 0)
- return EXIT_FAILURE;
+ r = parse_proc_cmdline(parse_proc_cmdline_item);
+ if (r < 0)
+ log_warning_errno(r, "Failed to parse kernel command line, ignoring: %m");
if (!arg_enabled) {
log_debug("Disabled, exiting.");
if (!arg_enabled) {
log_debug("Disabled, exiting.");