Thread (89 messages) 89 messages, 13 authors, 2017-09-18

Re: [PATCH v06 35/36] uapi linux/tls.h: don't include <net/tcp.h> in user space

From: Mikko Rapeli <hidden>
Date: 2017-09-18 12:57:45
Also in: lkml, netdev

On Mon, Sep 04, 2017 at 07:15:11PM +0300, Dmitry V. Levin wrote:
On Wed, Aug 09, 2017 at 02:25:54AM +0300, Dmitry V. Levin wrote:
quoted
On Sun, Aug 06, 2017 at 06:44:26PM +0200, Mikko Rapeli wrote:
quoted
It is not needed and not part of uapi headers, but causes
user space compilation error:

fatal error: net/tcp.h: No such file or directory
 #include <net/tcp.h>
                     ^

Signed-off-by: Mikko Rapeli <redacted>
Cc: Dave Watson <redacted>
Cc: Ilya Lesokhin <ilyal-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Cc: Aviad Yehezkel <aviadye-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
---
 include/uapi/linux/tls.h | 2 ++
 1 file changed, 2 insertions(+)
diff --git a/include/uapi/linux/tls.h b/include/uapi/linux/tls.h
index cc1d21db35d8..d87c698623f2 100644
--- a/include/uapi/linux/tls.h
+++ b/include/uapi/linux/tls.h
@@ -37,7 +37,9 @@
 #include <asm/byteorder.h>
 #include <linux/socket.h>
 #include <linux/tcp.h>
+#ifdef __KERNEL__
 #include <net/tcp.h>
+#endif
Let's move it to include/net/tls.h instead.
So everybody ignored this and *new* uapi header was released
in a totally unusable form along with v4.13.
Should issues like these be filed into bugzilla instead?

Maybe emails are easier to ignore than bugzilla tickets.

-Mikko
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help