From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jozsef Kadlecsik Subject: Re: your mail Date: Wed, 31 Aug 2011 15:24:38 +0200 (CEST) Message-ID: References: Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Cc: Netfilter Developer Mailing List To: Jan Engelhardt Return-path: Received: from smtp1.kfki.hu ([148.6.0.26]:43722 "EHLO smtp1.kfki.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755757Ab1HaNYj (ORCPT ); Wed, 31 Aug 2011 09:24:39 -0400 In-Reply-To: Sender: netfilter-devel-owner@vger.kernel.org List-ID: On Wed, 31 Aug 2011, Jan Engelhardt wrote: > On Wednesday 2011-08-31 14:49, Jozsef Kadlecsik wrote: > > >On Wed, 31 Aug 2011, Jan Engelhardt wrote: > > > >> On Wednesday 2011-08-31 14:16, Jozsef Kadlecsik wrote: > >> > >> >On Wed, 31 Aug 2011, Jan Engelhardt wrote: > >> > > >> >> On Wednesday 2011-08-31 13:56, Jozsef Kadlecsik wrote: > >> >> > >> >> >On Sun, 28 Aug 2011, Jan Engelhardt wrote: > >> >> > > >> >> >> libxt_SET.c requires IPSET_INVALID_ID, but this constant has moved into > >> >> >> a #ifdef __KERNEL__ section lately and is thus absent from userspace. > >> >> >> (In linux-kernel/include/linux/netfilter/ipset/ip_set.h.) > >> >> > > >> >> >After rewieving your patches, I don't get it: what userspace tool would > >> >> >use linux-kernel/include/linux/netfilter/ipset/ip_set.h? iptables does > >> >> >definitely not use it and ipset either. So why your patches are > >> >> >required? > >> >> > >> >> iptables/extensions/xt_SET.c #includes which > >> >> #includes ip_set.h to get at the definition of ip_set.h. > >> >> Patch will be resent. > >> > > >> >iptables/extensions/libxt_SET.c includes and > >> >xt_set.h includes nothing. > >> > >> 14:44 seven:../linux/netfilter > git show v3.0:include/linux/netfilter/xt_set.h | grep includ > >> #include > >> #include > > > >This is the kernel source tree. You wrote about libxt_SET.c and userspace: > > > >kadlec@mentat:/usr/src/git/iptables$ rgrep ip_set.h * > >kadlec@mentat:/usr/src/git/iptables$ > > Of course the header files in the iptables source tree come from the > (sanitized) kernel headers, and time and again, new copies from the > kernel are brought into the iptables tree. > > (Cf. commits b4af04be14560b3fcc6cf23200148d408014a2f5, > 350661a6eb089f3e54e67e022db9e16ea280499f, > 978e27e8f8c2e49d0528c6c4ae3a56627fbe8492, > e0bba47e550420e371c97425cc6d39909a6e059b, and possibly more in the > past.) Then I'm going to apply your patches. After that some more work is required from me, because ipset userspace assumes those parts to be excluded by #ifdef __KERNEL__. Best regards, Jozsef - E-mail : kadlec@blackhole.kfki.hu, kadlec@mail.kfki.hu PGP key : http://www.kfki.hu/~kadlec/pgp_public_key.txt Address : KFKI Research Institute for Particle and Nuclear Physics H-1525 Budapest 114, POB. 49, Hungary