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: Dmitry V. Levin <hidden>
Date: 2017-09-04 16:15:14
Also in: lkml, netdev

On Wed, Aug 09, 2017 at 02:25:54AM +0300, Dmitry V. Levin wrote:
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 <redacted>
Cc: Aviad Yehezkel <redacted>
---
 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.


-- 
ldv

Attachments

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