From 6be15dce13becd40682f799f7871fa4c2f862178 Mon Sep 17 00:00:00 2001 From: Daan De Meyer Date: Tue, 29 Apr 2025 15:46:48 +0200 Subject: [PATCH] basic: Remove assertions from missing_fs.h These assertions impose an include order between and "missing_fs.h", specifically can't be included before "missing_fs.h". This makes automated include refactoring very painful, so let's get rid of these assertions and instead assume that linux/fs.h does the right thing. --- src/basic/missing_fs.h | 42 ----------------------------------- src/shared/btrfs-util.c | 2 ++ src/shared/copy.c | 1 + src/shared/resize-fs.c | 1 + src/udev/udev-builtin-btrfs.c | 1 + 5 files changed, 5 insertions(+), 42 deletions(-) diff --git a/src/basic/missing_fs.h b/src/basic/missing_fs.h index c3b2746b07..1e2f07e55f 100644 --- a/src/basic/missing_fs.h +++ b/src/basic/missing_fs.h @@ -1,50 +1,8 @@ /* SPDX-License-Identifier: LGPL-2.1-or-later */ #pragma once -/* temporary undef definitions in bits/uio-ext.h, which is included by sys/uio.h */ -#include - -#define __RWF_HIPRI_SAVED__ RWF_HIPRI -#undef RWF_HIPRI -#define __RWF_DSYNC_SAVED__ RWF_DSYNC -#undef RWF_DSYNC -#define __RWF_SYNC_SAVED__ RWF_SYNC -#undef RWF_SYNC -#define __RWF_NOWAIT_SAVED__ RWF_NOWAIT -#undef RWF_NOWAIT -#define __RWF_APPEND_SAVED__ RWF_APPEND -#undef RWF_APPEND -#define __RWF_NOAPPEND_SAVED__ RWF_NOAPPEND -#undef RWF_NOAPPEND -#if defined(RWF_ATOMIC) -#define __RWF_ATOMIC_SAVED__ RWF_ATOMIC -#undef RWF_ATOMIC -#else -#define __RWF_ATOMIC_SAVED__ 0x00000040 -#endif -#if defined(RWF_DONTCACHE) -#define __RWF_DONTCACHE_SAVED__ RWF_DONTCACHE -#undef RWF_DONTCACHE -#else -#define __RWF_DONTCACHE_SAVED__ 0x00000080 -#endif - -#include -#include #include -#include "macro.h" - -/* check RWF_xyz are redefined by linux/fs.h */ -assert_cc(RWF_HIPRI == __RWF_HIPRI_SAVED__); -assert_cc(RWF_DSYNC == __RWF_DSYNC_SAVED__); -assert_cc(RWF_SYNC == __RWF_SYNC_SAVED__); -assert_cc(RWF_NOWAIT == __RWF_NOWAIT_SAVED__); -assert_cc(RWF_APPEND == __RWF_APPEND_SAVED__); -assert_cc(RWF_NOAPPEND == __RWF_NOAPPEND_SAVED__); -assert_cc(RWF_ATOMIC == __RWF_ATOMIC_SAVED__); -assert_cc(RWF_DONTCACHE == __RWF_DONTCACHE_SAVED__); - /* Not exposed yet. Defined at fs/ext4/ext4.h */ #ifndef EXT4_IOC_RESIZE_FS #define EXT4_IOC_RESIZE_FS _IOW('f', 16, __u64) diff --git a/src/shared/btrfs-util.c b/src/shared/btrfs-util.c index 713ecb4eb5..d296ac682c 100644 --- a/src/shared/btrfs-util.c +++ b/src/shared/btrfs-util.c @@ -3,6 +3,8 @@ #include #include #include +#include +#include #include #include #include diff --git a/src/shared/copy.c b/src/shared/copy.c index c662853ac8..4288bae1ce 100644 --- a/src/shared/copy.c +++ b/src/shared/copy.c @@ -2,6 +2,7 @@ #include #include +#include #include #include #include diff --git a/src/shared/resize-fs.c b/src/shared/resize-fs.c index b913aedef3..0f28869ea0 100644 --- a/src/shared/resize-fs.c +++ b/src/shared/resize-fs.c @@ -1,5 +1,6 @@ /* SPDX-License-Identifier: LGPL-2.1-or-later */ +#include #include #include #include diff --git a/src/udev/udev-builtin-btrfs.c b/src/udev/udev-builtin-btrfs.c index a1238f1e1b..118b934b76 100644 --- a/src/udev/udev-builtin-btrfs.c +++ b/src/udev/udev-builtin-btrfs.c @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: LGPL-2.1-or-later */ #include +#include #include #include