* [PATCH] include linux/stddef.h in swab.h uapi header
@ 2018-09-13 0:56 Khem Raj
0 siblings, 0 replies; 2+ messages in thread
From: Khem Raj @ 2018-09-13 0:56 UTC (permalink / raw)
To: linux-kernel
Cc: Khem Raj, Philippe Ombredanne, Kate Stewart, Greg Kroah-Hartman,
Thomas Gleixner
swab.h uses __always_inline without including the header where it is
defined, this is exposed by musl based distributions where this macro is
not defined by system C library headers unlike glibc where it is defined
in sys/cdefs.h and that header gets pulled in indirectly via
features.h -> sys/cdefs.h
and features.h gets pulled in a lot of headers. Therefore it may work in
cases where features.h is includes but not otherwise.
Adding linux/stddef.h here ensures that __always_inline is always
defined independent of which C library is used in userspace
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Cc: Philippe Ombredanne <pombredanne@nexb.com>
Cc: Kate Stewart <kstewart@linuxfoundation.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
---
include/uapi/linux/swab.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/uapi/linux/swab.h b/include/uapi/linux/swab.h
index 23cd84868cc3..acddbe50a20d 100644
--- a/include/uapi/linux/swab.h
+++ b/include/uapi/linux/swab.h
@@ -3,6 +3,7 @@
#define _UAPI_LINUX_SWAB_H
#include <linux/types.h>
+#include <linux/stddef.h>
#include <linux/compiler.h>
#include <asm/swab.h>
--
2.19.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH] include linux/stddef.h in swab.h uapi header
@ 2018-12-13 18:01 Khem Raj
0 siblings, 0 replies; 2+ messages in thread
From: Khem Raj @ 2018-12-13 18:01 UTC (permalink / raw)
To: linux-kernel
Cc: Khem Raj, Philippe Ombredanne, Kate Stewart, Greg Kroah-Hartman,
Thomas Gleixner
swab.h uses __always_inline without including the header where it is
defined, this is exposed by musl based distributions where this macro is
not defined by system C library headers unlike glibc where it is defined
in sys/cdefs.h and that header gets pulled in indirectly via
features.h -> sys/cdefs.h
and features.h gets pulled in a lot of headers. Therefore it may work in
cases where features.h is includes but not otherwise.
Adding linux/stddef.h here ensures that __always_inline is always
defined independent of which C library is used in userspace
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Cc: Philippe Ombredanne <pombredanne@nexb.com>
Cc: Kate Stewart <kstewart@linuxfoundation.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
---
include/uapi/linux/swab.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/uapi/linux/swab.h b/include/uapi/linux/swab.h
index 23cd84868cc3..acddbe50a20d 100644
--- a/include/uapi/linux/swab.h
+++ b/include/uapi/linux/swab.h
@@ -3,6 +3,7 @@
#define _UAPI_LINUX_SWAB_H
#include <linux/types.h>
+#include <linux/stddef.h>
#include <linux/compiler.h>
#include <asm/swab.h>
--
2.20.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-12-13 18:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-13 0:56 [PATCH] include linux/stddef.h in swab.h uapi header Khem Raj
2018-12-13 18:01 Khem Raj
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).