Re: [PATCH net-next v2 10/12] gtp: add IPv6 support

2020-12-12 Thread Jonas Bonn
Hi Harald, On 12/12/2020 12:22, Harald Welte wrote: Hi Jonas, thanks again for your patches, they are very much appreciated. However, I don't think that it is "that easy". PDP contexts (at least) in GPRS/EDGE and UMTS come in three flavors: * IPv4 only * IPv6 only * IPv4v6 (i.e. both an IPv4

Re: [PATCH net-next v2 10/12] gtp: add IPv6 support

2020-12-12 Thread Harald Welte
Hi Jonas, thanks again for your patches, they are very much appreciated. However, I don't think that it is "that easy". PDP contexts (at least) in GPRS/EDGE and UMTS come in three flavors: * IPv4 only * IPv6 only * IPv4v6 (i.e. both an IPv4 and an IPv6 address within the same tunnel) See for ex

Re: [PATCH net-next v2 10/12] gtp: add IPv6 support

2020-12-12 Thread Harald Welte
Hi Jonas, On Sat, Dec 12, 2020 at 08:05:40AM +0100, Jonas Bonn wrote: > Yes, you're probably right. Given that IPv6 isn't really optional in > contexts where this driver is relevant, [...] I strongly contest this statement. GTP is used in a lot of legacy contexts without any IPv6 requirements wh

Re: [PATCH net-next v2 10/12] gtp: add IPv6 support

2020-12-12 Thread Jonas Bonn
Hi Pravin, On 12/12/2020 06:51, Pravin Shelar wrote: On Fri, Dec 11, 2020 at 4:29 AM Jonas Bonn wrote: This patch adds support for handling IPv6. Both the GTP tunnel and the tunneled packets may be IPv6; as they constitute independent streams, both v4-over-v6 and v6-over-v4 are supported, as

Re: [PATCH net-next v2 10/12] gtp: add IPv6 support

2020-12-11 Thread Pravin Shelar
On Fri, Dec 11, 2020 at 4:29 AM Jonas Bonn wrote: > > This patch adds support for handling IPv6. Both the GTP tunnel and the > tunneled packets may be IPv6; as they constitute independent streams, > both v4-over-v6 and v6-over-v4 are supported, as well. > > This patch includes only the driver fun

[PATCH net-next v2 10/12] gtp: add IPv6 support

2020-12-11 Thread Jonas Bonn
This patch adds support for handling IPv6. Both the GTP tunnel and the tunneled packets may be IPv6; as they constitute independent streams, both v4-over-v6 and v6-over-v4 are supported, as well. This patch includes only the driver functionality for IPv6 support. A follow-on patch will add suppo