diff mbox

[net] l2tp: define SOL_PPPOL2TP in uapi

Message ID 28884134c7944f005e4872256742b10765cc2770.1491402366.git.g.nault@alphalink.fr
State Rejected, archived
Delegated to: David Miller
Headers show

Commit Message

Guillaume Nault April 5, 2017, 2:52 p.m. UTC
Userspace needs SOL_PPPOL2TP to be defined for using PPPOL2TP_SO_*
socket options.

Signed-off-by: Guillaume Nault <g.nault@alphalink.fr>
---
 include/uapi/linux/if_pppol2tp.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

David Miller April 6, 2017, 8:25 p.m. UTC | #1
From: Guillaume Nault <g.nault@alphalink.fr>
Date: Wed, 5 Apr 2017 16:52:35 +0200

> Userspace needs SOL_PPPOL2TP to be defined for using PPPOL2TP_SO_*
> socket options.
> 
> Signed-off-by: Guillaume Nault <g.nault@alphalink.fr>

We really can't do this, there is no precedence for defining the SOL_*
values in the kernel headers, it needs to come from the libc headers just
like the other SOL_* values do for applications.

Thank you.
Guillaume Nault April 7, 2017, 7:48 a.m. UTC | #2
On Thu, Apr 06, 2017 at 01:25:34PM -0700, David Miller wrote:
> From: Guillaume Nault <g.nault@alphalink.fr>
> Date: Wed, 5 Apr 2017 16:52:35 +0200
> 
> > Userspace needs SOL_PPPOL2TP to be defined for using PPPOL2TP_SO_*
> > socket options.
> > 
> > Signed-off-by: Guillaume Nault <g.nault@alphalink.fr>
> 
> We really can't do this, there is no precedence for defining the SOL_*
> values in the kernel headers, it needs to come from the libc headers just
> like the other SOL_* values do for applications.
> 
Got it, thanks.
diff mbox

Patch

diff --git a/include/uapi/linux/if_pppol2tp.h b/include/uapi/linux/if_pppol2tp.h
index 6418c4d10241..90ce1abd2c4b 100644
--- a/include/uapi/linux/if_pppol2tp.h
+++ b/include/uapi/linux/if_pppol2tp.h
@@ -73,6 +73,8 @@  struct pppol2tpv3in6_addr {
 	struct sockaddr_in6 addr;	/* IP address and port to send to */
 };
 
+#define SOL_PPPOL2TP 273
+
 /* Socket options:
  * DEBUG	- bitmask of debug message categories
  * SENDSEQ	- 0 => don't send packets with sequence numbers