[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAM_iQpX-V60iTHBz3nFnJkNbEy1JvGkxu_8T1mwYYwd67WxBYA@mail.gmail.com>
Date: Tue, 4 Nov 2014 16:39:59 -0800
From: Cong Wang <xiyou.wangcong@...il.com>
To: Gregory Fong <gregory.0xf0@...il.com>
Cc: Linux Kernel Network Developers <netdev@...r.kernel.org>,
linux-api@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>,
carlos@...hat.com, eblake@...hat.com,
Kumar Gala <galak@...eaurora.org>,
Florian Fainelli <f.fainelli@...il.com>,
David Miller <davem@...emloft.net>
Subject: Re: [PATCH] bridge: include in6.h in if_bridge.h for struct in6_addr
On Tue, Nov 4, 2014 at 11:21 AM, Gregory Fong <gregory.0xf0@...il.com> wrote:
> if_bridge.h uses struct in6_addr ip6, but wasn't including the in6.h
> header. Thomas Backlund originally sent a patch to do this, but this
> revealed a redefinition issue: https://lkml.org/lkml/2013/1/13/116
>
> The redefinition issue should have been fixed by the following Linux
> commits:
> ee262ad827f89e2dc7851ec2986953b5b125c6bc inet: defines IPPROTO_* needed for module alias generation
> cfd280c91253cc28e4919e349fa7a813b63e71e8 net: sync some IP headers with glibc
>
> and the following glibc commit:
> 6c82a2f8d7c8e21e39237225c819f182ae438db3 Coordinate IPv6 definitions for Linux and glibc
>
> so actually include the header now.
>
> Reported-by: Colin Guthrie <colin@...eia.org>
> Reported-by: Christiaan Welvaart <cjw@...eel.dyndns.org>
> Reported-by: Thomas Backlund <tmb@...eia.org>
> Cc: Florian Fainelli <f.fainelli@...il.com>
> Cc: Cong Wang <xiyou.wangcong@...il.com>
> Cc: David Miller <davem@...emloft.net>
> Signed-off-by: Gregory Fong <gregory.0xf0@...il.com>
Acked-by: Cong Wang <xiyou.wangcong@...il.com>
Thanks for working on it!
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists