return;
j->on_network =
- (unsigned __SWORD_TYPE) sfs.f_type == CIFS_MAGIC_NUMBER ||
- (unsigned __SWORD_TYPE) sfs.f_type == CODA_SUPER_MAGIC ||
- (unsigned __SWORD_TYPE) sfs.f_type == NCP_SUPER_MAGIC ||
- (unsigned __SWORD_TYPE) sfs.f_type == NFS_SUPER_MAGIC ||
- (unsigned __SWORD_TYPE) sfs.f_type == SMB_SUPER_MAGIC;
+ CMP_F_TYPE(sfs.f_type, CIFS_MAGIC_NUMBER) ||
+ CMP_F_TYPE(sfs.f_type, CODA_SUPER_MAGIC) ||
+ CMP_F_TYPE(sfs.f_type, NCP_SUPER_MAGIC) ||
+ CMP_F_TYPE(sfs.f_type, NFS_SUPER_MAGIC) ||
+ CMP_F_TYPE(sfs.f_type, SMB_SUPER_MAGIC);
}
static int add_file(sd_journal *j, const char *prefix, const char *filename) {
on_ssd = fs_on_ssd(root) > 0;
log_debug("On SSD: %s", yes_no(on_ssd));
- on_btrfs = statfs(root, &sfs) >= 0 && (unsigned __SWORD_TYPE) sfs.f_type == BTRFS_SUPER_MAGIC;
+ on_btrfs = statfs(root, &sfs) >= 0 && CMP_F_TYPE(sfs.f_type, BTRFS_SUPER_MAGIC);
log_debug("On btrfs: %s", yes_no(on_btrfs));
if (asprintf(&pack_fn_new, "%s/.readahead.new", root) < 0) {
} \
} while(false)
+/* Remove this macro, when the kernel has f_type as unsigned int or long
+ * for every architecure. Currently some 64bit architecures (like s390x)
+ * have int in the kernel, but long in userspace for f_type, so glibc
+ * extends the int to long and carries over the sign. Negative numbers are
+ * caused by the 32bit magic constants in linux/magic.h stuffed into the
+ * signed int in the kernel and these negative numbers are extended to
+ * long, which cannot be simply compared to the magic constants anymore.
+ */
+#define CMP_F_TYPE(f, c) \
+ __extension__ ({ \
+ __SWORD_TYPE _f = (f); \
+ const __SWORD_TYPE _c = (c); \
+ const int _c32 = (c); \
+ (_f == _c || _f == _c32 ); \
+ })
+
+
/* Returns the number of chars needed to format variables of the
* specified type as a decimal string. Adds in extra space for a
* negative '-' prefix. */
static int is_temporary_fs(struct statfs *s) {
assert(s);
- return (unsigned __SWORD_TYPE) s->f_type == TMPFS_MAGIC ||
- (unsigned __SWORD_TYPE) s->f_type == RAMFS_MAGIC;
+ return
+ CMP_F_TYPE(s->f_type, TMPFS_MAGIC) ||
+ CMP_F_TYPE(s->f_type, RAMFS_MAGIC);
}
int rm_rf_children(int fd, bool only_dirs, bool honour_sticky, struct stat *root_dev) {