[nf-next:master 14/17] include/uapi/linux/netfilter_ipv6/ip6t_LOG.h:5:2: warning: #warning "Please update iptables, this file will be removed soon!"

[nf-next:master 14/17] include/uapi/linux/netfilter_ipv6/ip6t_LOG.h:5:2: warning: #warning "Please update iptables, this file will be removed soon!"

[kbuild test robot]

[-- Attachment #1: Type: text/plain, Size: 2148 bytes --]

tree:   https://kernel.googlesource.com/pub/scm/linux/kernel/git/pablo/nf-next.git master
head:   105333435b4f3b21ffc325f32fae17719310db64
commit: 2a475c409fe81a76fb26a6b023509d648237bbe6 [14/17] kbuild: remove all netfilter headers from header-test blacklist.
config: sparc64-allmodconfig (attached as .config)
compiler: sparc64-linux-gcc (GCC) 7.4.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 2a475c409fe81a76fb26a6b023509d648237bbe6
        # save the attached .config to linux build tree
        GCC_VERSION=7.4.0 make.cross ARCH=sparc64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   In file included from <command-line>:0:0:
>> include/uapi/linux/netfilter_ipv6/ip6t_LOG.h:5:2: warning: #warning "Please update iptables, this file will be removed soon!" [-Wcpp]
    #warning "Please update iptables, this file will be removed soon!"
     ^~~~~~~
--
   In file included from <command-line>:0:0:
>> include/uapi/linux/netfilter_ipv4/ipt_LOG.h:5:2: warning: #warning "Please update iptables, this file will be removed soon!" [-Wcpp]
    #warning "Please update iptables, this file will be removed soon!"
     ^~~~~~~

vim +5 include/uapi/linux/netfilter_ipv6/ip6t_LOG.h

^1da177e4c3f41 include/linux/netfilter_ipv6/ip6t_LOG.h Linus Torvalds     2005-04-16  4  
6939c33a757bd0 include/linux/netfilter_ipv6/ip6t_LOG.h Richard Weinberger 2012-02-10 @5  #warning "Please update iptables, this file will be removed soon!"
6939c33a757bd0 include/linux/netfilter_ipv6/ip6t_LOG.h Richard Weinberger 2012-02-10  6  

:::::: The code at line 5 was first introduced by commit
:::::: 6939c33a757bd006c5e0b8b5fd429fc587a4d0f4 netfilter: merge ipt_LOG and ip6_LOG into xt_LOG

:::::: TO: Richard Weinberger <richard@nod.at>
:::::: CC: Pablo Neira Ayuso <pablo@netfilter.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 58706 bytes --]

Re: [nf-next:master 14/17] include/uapi/linux/netfilter_ipv6/ip6t_LOG.h:5:2: warning: #warning "Please update iptables, this file will be removed soon!"

[Pablo Neira Ayuso]

Hi Jeremy,

On Wed, Aug 14, 2019 at 06:05:49AM +0800, kbuild test robot wrote:
> tree:   https://kernel.googlesource.com/pub/scm/linux/kernel/git/pablo/nf-next.git master
> head:   105333435b4f3b21ffc325f32fae17719310db64
> commit: 2a475c409fe81a76fb26a6b023509d648237bbe6 [14/17] kbuild: remove all netfilter headers from header-test blacklist.
> config: sparc64-allmodconfig (attached as .config)
> compiler: sparc64-linux-gcc (GCC) 7.4.0
> reproduce:
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout 2a475c409fe81a76fb26a6b023509d648237bbe6
>         # save the attached .config to linux build tree
>         GCC_VERSION=7.4.0 make.cross ARCH=sparc64 
> 
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> All warnings (new ones prefixed by >>):
> 
>    In file included from <command-line>:0:0:
> >> include/uapi/linux/netfilter_ipv6/ip6t_LOG.h:5:2: warning: #warning "Please update iptables, this file will be removed soon!" [-Wcpp]
>     #warning "Please update iptables, this file will be removed soon!"
>      ^~~~~~~

I'd suggest you send me a patch to remove this #warning.

userspace iptables still refer to this header. The intention was to
use xt_LOG.h instead and remove these, but userspace was never
updated.

Re: [nf-next:master 14/17] include/uapi/linux/netfilter_ipv6/ip6t_LOG.h:5:2: warning: #warning "Please update iptables, this file will be removed soon!"

[Jeremy Sowden]

[-- Attachment #1: Type: text/plain, Size: 1599 bytes --]

On 2019-08-14, at 09:45:39 +0200, Pablo Neira Ayuso wrote:
> On Wed, Aug 14, 2019 at 06:05:49AM +0800, kbuild test robot wrote:
> > tree:   https://kernel.googlesource.com/pub/scm/linux/kernel/git/pablo/nf-next.git master
> > head:   105333435b4f3b21ffc325f32fae17719310db64
> > commit: 2a475c409fe81a76fb26a6b023509d648237bbe6 [14/17] kbuild: remove all netfilter headers from header-test blacklist.
> > config: sparc64-allmodconfig (attached as .config)
> > compiler: sparc64-linux-gcc (GCC) 7.4.0
> > reproduce:
> >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         git checkout 2a475c409fe81a76fb26a6b023509d648237bbe6
> >         # save the attached .config to linux build tree
> >         GCC_VERSION=7.4.0 make.cross ARCH=sparc64
> >
> > If you fix the issue, kindly add following tag
> > Reported-by: kbuild test robot <lkp@intel.com>
> >
> > All warnings (new ones prefixed by >>):
> >
> >    In file included from <command-line>:0:0:
> > >> include/uapi/linux/netfilter_ipv6/ip6t_LOG.h:5:2: warning: #warning "Please update iptables, this file will be removed soon!" [-Wcpp]
> >     #warning "Please update iptables, this file will be removed soon!"
> >      ^~~~~~~
>
> I'd suggest you send me a patch to remove this #warning.
>
> userspace iptables still refer to this header. The intention was to
> use xt_LOG.h instead and remove these, but userspace was never
> updated.

Was just preparing one to add "#ifndef __KERNEL__" guards to the
warnings.  I'll remove them instead.

J.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[PATCH nf-next] netfilter: remove deprecation warnings from uapi headers.

[Jeremy Sowden]

There are two netfilter userspace headers which contain deprecation
warnings.  While these headers are not used within the kernel, they are
compiled stand-alone for header-testing.

Pablo informs me that userspace iptables still refer to these headers,
and the intention was to use xt_LOG.h instead and remove these, but
userspace was never updated.

Remove the warnings.

Fixes: 2a475c409fe8 ("kbuild: remove all netfilter headers from header-test blacklist.")
Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Jeremy Sowden <jeremy@azazel.net>
---
 include/uapi/linux/netfilter_ipv4/ipt_LOG.h  | 2 --
 include/uapi/linux/netfilter_ipv6/ip6t_LOG.h | 2 --
 2 files changed, 4 deletions(-)

diff --git a/include/uapi/linux/netfilter_ipv4/ipt_LOG.h b/include/uapi/linux/netfilter_ipv4/ipt_LOG.h
index 6dec14ba851b..b7cf2c669f40 100644
--- a/include/uapi/linux/netfilter_ipv4/ipt_LOG.h
+++ b/include/uapi/linux/netfilter_ipv4/ipt_LOG.h
@@ -2,8 +2,6 @@
 #ifndef _IPT_LOG_H
 #define _IPT_LOG_H
 
-#warning "Please update iptables, this file will be removed soon!"
-
 /* make sure not to change this without changing netfilter.h:NF_LOG_* (!) */
 #define IPT_LOG_TCPSEQ		0x01	/* Log TCP sequence numbers */
 #define IPT_LOG_TCPOPT		0x02	/* Log TCP options */
diff --git a/include/uapi/linux/netfilter_ipv6/ip6t_LOG.h b/include/uapi/linux/netfilter_ipv6/ip6t_LOG.h
index 7553a434e4da..23e91a9c2583 100644
--- a/include/uapi/linux/netfilter_ipv6/ip6t_LOG.h
+++ b/include/uapi/linux/netfilter_ipv6/ip6t_LOG.h
@@ -2,8 +2,6 @@
 #ifndef _IP6T_LOG_H
 #define _IP6T_LOG_H
 
-#warning "Please update iptables, this file will be removed soon!"
-
 /* make sure not to change this without changing netfilter.h:NF_LOG_* (!) */
 #define IP6T_LOG_TCPSEQ		0x01	/* Log TCP sequence numbers */
 #define IP6T_LOG_TCPOPT		0x02	/* Log TCP options */
-- 
2.20.1

Re: [PATCH nf-next] netfilter: remove deprecation warnings from uapi headers.

[Pablo Neira Ayuso]

On Wed, Aug 14, 2019 at 09:01:28AM +0100, Jeremy Sowden wrote:
> There are two netfilter userspace headers which contain deprecation
> warnings.  While these headers are not used within the kernel, they are
> compiled stand-alone for header-testing.
> 
> Pablo informs me that userspace iptables still refer to these headers,
> and the intention was to use xt_LOG.h instead and remove these, but
> userspace was never updated.
> 
> Remove the warnings.

Applied, thanks.