Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: ipv6 redefinition build issue with 4.15-rc8
To:     Hauke Mehrtens <hauke@hauke-m.de>,
        Neil MacLeod <neil@nmacleod.com>,
        Jonas Bonn <jonas@southpole.se>
Cc:     connman@lists.01.org,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "David S. Miller" <davem@davemloft.net>
References: <CAFbqK8m9grXyAV_mWSzMGS12Mikekg6NsOM-B7pkXiXpnWVgZg@mail.gmail.com>
 <794202f5-e993-444f-f3fd-891276795926@monom.org>
 <b4f9df2d-09cb-b9ba-4742-1957c08a5a51@southpole.se>
 <CAFbqK8mtYtgLiviaBQvk_hT+yMr2VcNZux_5+c8gN36iCeCCiA@mail.gmail.com>
 <CAFbqK8=k0F6Xszz-nu8ECs17ckLiO3h1E_X2SV+cabBUOZxxew@mail.gmail.com>
 <0d51f6a6-0edc-1c18-3412-5a37cdc83873@hauke-m.de>
From:   Daniel Wagner <wagi@monom.org>
Message-ID: <ba894de3-f968-52c5-6504-0e0d21b385a0@monom.org>
Date:   Wed, 17 Jan 2018 23:34:37 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.5.2
MIME-Version: 1.0
In-Reply-To: <0d51f6a6-0edc-1c18-3412-5a37cdc83873@hauke-m.de>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


On 01/17/2018 11:20 PM, Hauke Mehrtens wrote:
> On 01/17/2018 08:31 PM, Neil MacLeod wrote:
>> All
>>
>> Further to my previous reply (reproduced below having been bounced by
>> linux-kernel) I have successfully built LibreELEC when using the
>> ConnMan patch from Jonas - there were no other failures.
>>
>> I have also built a number of network related packages (iftop, iperf,
>> ngrp, nmap, sshfs, tcpdump, udpxy, wireless-tools), again without
>> issue, so this particular 4.15-rc8 kernel change is only affecting
>> ConnMan as far as I can tell.
> 
> Thanks for testing.
> 
>> Regards
>> Neil
>>
>>> All
>>>
>>> Many thanks for the replies.
>>>
>>> To ensure my build environment is sane I tested again without reverting the kernel commit, and reproduced the connman build failure.
>>>
>>> Next I tested the change suggested by Hauke (kernel patch: http://ix.io/Eh5) and connman fails to build, however it fails with a different error this time: http://ix.io/Eh2
>>>
>>> I then tested the change suggested by Jonas (connman patch: http://ix.io/Eh6) and connman builds successfully, no failure, so this might be a potential fix.
> 
> You should import the libc header files first and then the Linux header
> files in user space applications, this is the supported order.
> 
> Can you try this patch please:
> 
> --- a/src/tethering.c
> +++ b/src/tethering.c
> @@ -31,11 +31,11 @@
>   #include <stdio.h>
>   #include <sys/ioctl.h>
>   #include <net/if.h>
> -#include <linux/sockios.h>
>   #include <string.h>
>   #include <fcntl.h>
> -#include <linux/if_tun.h>
>   #include <netinet/in.h>
> +#include <linux/sockios.h>
> +#include <linux/if_tun.h>
>   #include <linux/if_bridge.h>
> 
>   #include "connman.h"
> 
> 
> Do we want to do any changes to the kernel header files? I do not know
> of any clean workaround to make this work, we can probably hack
> something for connman, but I think it is not worth the trouble.

Nah, don't worry, we just fix this in ConnMan. Thanks a lot for the 
excellent input!

I'll cook a patch for this.

Thanks,
Daniel