[OE-core] [PATCH] linux-libc-headers: if_tunnel: remove include of if/ip/in6.h

[Khem Raj]

On Oct 12, 2016 10:21 PM, "Randy MacLeod" <randy.macleod at windriver.com>
wrote:
>
> On 2016-10-08 09:17 PM, Huang, Jie (Jackie) wrote:
>>
>>
>>
>>> -----Original Message-----
>>> From: MacLeod, Randy
>>> Sent: Friday, October 07, 2016 4:54 AM
>>> To: Bruce Ashfield; Khem Raj; Huang, Jie (Jackie)
>>> Cc: Patches and discussions about the oe-core layer
>>> Subject: Re: [OE-core] [PATCH] linux-libc-headers: if_tunnel: remove
include of if/ip/in6.h
>>>
>>> On 2016-10-03 08:27 PM, Bruce Ashfield wrote:
>>>>
>>>>
>>>> It's something we could drop in the future, but these are mods that
have
>>>> been done since -rc4, and could very well be undone if they start
>>>> breaking things like this.
>>>>
>>>> The alternative is to try and fix the userspace applications, but I
>>>> currently don't see a fast way to do that (and I'm now officially on
>>>> vacation).
>>>>
>>>
>>> I've sent a fix to the net-tools list:
>>>
>>>
https://sourceforge.net/p/net-tools/mailman/net-tools-devel/?viewmonth=201610
>>>
>>> Jackie will be back at work on Saturday and he'll revert
>>> Bruce's change, test all qemu builds and boot an image
>>> to test iptunnel before sending a patch to the list.
>>
>>
>> I just sent patches for this to the list.
>>
>> Thanks,
>> Jackie
>
>
>
> Jackie,
>
> Thanks for fixing the net-tools bug. A fedora dev acked the patch
> on the SF list so hopefully it will be integrated upstream.
>
> Bruce, Khem
>
> What, if anything, should we do about the other yocto specific changes to
linux-libc-headers:
>
> $ ls -1 meta/recipes-kernel/linux-libc-headers/linux-libc-headers
> 0001-libc-compat.h-fix-some-issues-arising-from-in6.h.patch
> 0001-ppp-fix-in-if.h-includes.patch
> 0002-libc-compat.h-prevent-redefinition-of-struct-ethhdr.patch
> 0003-remove-inclusion-of-sysinfo.h-in-kernel.h.patch
>
> It seems like some of the patches (ppp) could be addressed and
> dropped and that we should do that before releasing oe-core-2.2
> or wait for 2.3.
>

Imo if we can fix apps that should be first choice. Second choice would be
run remaining patches by lkml or atleast report the problem. Real lesson is
to not wait for kernel version updates so late in release cycle.

> ../Randy
>
>
> $ git log
meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-libc-compat.h-fix-some-issues-arising-from-in6.h.patch
>
>
> commit 12451a412fb7b5706c1553618ee7b704234876cc
> Author: Bruce Ashfield <bruce.ashfield at windriver.com>
> Date:   Mon Oct 3 01:54:36 2016 -0400
>
>     linux-libc-headers: fix in/if.h includes
>
>     The following kernel commits broke the compilation of ppp, due to
redefined
>     structures.
>
>     Nothing else breaks in userspace with or without these uapi changes,
so we
>     revert them to keep everything building.
>
>        commit 05ee5de7451796cf9a8aeb2f05a57790d4fd2336
>        Author: Mikko Rapeli <mikko.rapeli at iki.fi>
>        Date:   Mon Aug 22 20:32:42 2016 +0200
>
>            include/uapi/linux/if_pppol2tp.h: include linux/in.h and
linux/in6.h
>
>            Fixes userspace compilation errors like:
>
>            error: field <E2><80><98>addr<E2><80><99> has incomplete type
>             struct sockaddr_in addr; /* IP address and port to send to */
>                                ^
>            error: field <E2><80><98>addr<E2><80><99> has incomplete type
>             struct sockaddr_in6 addr; /* IP address and port to send to */
>
>            Signed-off-by: Mikko Rapeli <mikko.rapeli at iki.fi>
>            Signed-off-by: David S. Miller <davem at davemloft.net>
>
>
>
>
>
>>
>>>
>>> In oe-core-2.3, we should probably drop net-tools completely or
>>> at least from packagegroup-core-full-cmdline.bb since iproute2
>>> replaces it:
>>>    http://xmodulo.com/linux-tcpip-networking-net-tools-iproute2.html
>>>
>>> ( and I'm now officially on vacation for a few days. :) )
>>>
>>>
>>> --
>>> # Randy MacLeod. SMTS, Linux, Wind River
>>> Direct: 613.963.1350 | 350 Terry Fox Drive, Suite 200, Ottawa, ON,
>>> Canada, K2K 2W5
>>
>>
>
>
> --
> # Randy MacLeod. SMTS, Linux, Wind River
> Direct: 613.963.1350 | 350 Terry Fox Drive, Suite 200, Ottawa, ON,
Canada, K2K 2W5
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20161012/4edf4c74/attachment-0002.html>