* linux-next: build warning after merge of the netfilter-next tree
@ 2018-05-07 0:55 Stephen Rothwell
2018-05-08 0:21 ` Stephen Rothwell
0 siblings, 1 reply; 7+ messages in thread
From: Stephen Rothwell @ 2018-05-07 0:55 UTC (permalink / raw)
To: Pablo Neira Ayuso, NetFilter
Cc: Linux-Next Mailing List, Linux Kernel Mailing List,
Fernando Fernandez Mancera
[-- Attachment #1: Type: text/plain, Size: 380 bytes --]
Hi all,
After merging the netfilter-next tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
./usr/include/linux/netfilter/nf_osf.h:25: found __[us]{8,16,32,64} type without #include <linux/types.h>
Introduced by commit
bfb15f2a95cb ("netfilter: extract Passive OS fingerprint infrastructure from xt_osf")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build warning after merge of the netfilter-next tree
2018-05-07 0:55 linux-next: build warning after merge of the netfilter-next tree Stephen Rothwell
@ 2018-05-08 0:21 ` Stephen Rothwell
2018-05-08 7:17 ` Florian Westphal
2018-05-08 8:05 ` [PATCH nf-next] netfilter: fix fallout from xt/nf osf separation Florian Westphal
0 siblings, 2 replies; 7+ messages in thread
From: Stephen Rothwell @ 2018-05-08 0:21 UTC (permalink / raw)
To: David Miller
Cc: Pablo Neira Ayuso, NetFilter, Linux-Next Mailing List,
Linux Kernel Mailing List, Fernando Fernandez Mancera
[-- Attachment #1: Type: text/plain, Size: 544 bytes --]
Hi all,
On Mon, 7 May 2018 10:55:19 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> After merging the netfilter-next tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> ./usr/include/linux/netfilter/nf_osf.h:25: found __[us]{8,16,32,64} type without #include <linux/types.h>
>
> Introduced by commit
>
> bfb15f2a95cb ("netfilter: extract Passive OS fingerprint infrastructure from xt_osf")
This warning now occurs after the merge of the net-next tree.
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build warning after merge of the netfilter-next tree
2018-05-08 0:21 ` Stephen Rothwell
@ 2018-05-08 7:17 ` Florian Westphal
2018-05-15 10:54 ` Geert Uytterhoeven
2018-05-08 8:05 ` [PATCH nf-next] netfilter: fix fallout from xt/nf osf separation Florian Westphal
1 sibling, 1 reply; 7+ messages in thread
From: Florian Westphal @ 2018-05-08 7:17 UTC (permalink / raw)
To: Stephen Rothwell
Cc: David Miller, Pablo Neira Ayuso, NetFilter,
Linux-Next Mailing List, Linux Kernel Mailing List,
Fernando Fernandez Mancera
Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> On Mon, 7 May 2018 10:55:19 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >
> > After merging the netfilter-next tree, today's linux-next build (x86_64
> > allmodconfig) produced this warning:
> >
> > ./usr/include/linux/netfilter/nf_osf.h:25: found __[us]{8,16,32,64} type without #include <linux/types.h>
>
> > Introduced by commit
> >
> > bfb15f2a95cb ("netfilter: extract Passive OS fingerprint infrastructure from xt_osf")
I'll send a fix for this, thanks for reporting.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH nf-next] netfilter: fix fallout from xt/nf osf separation
2018-05-08 0:21 ` Stephen Rothwell
2018-05-08 7:17 ` Florian Westphal
@ 2018-05-08 8:05 ` Florian Westphal
2018-05-17 11:52 ` Pablo Neira Ayuso
1 sibling, 1 reply; 7+ messages in thread
From: Florian Westphal @ 2018-05-08 8:05 UTC (permalink / raw)
To: netfilter-devel
Cc: sfr, linux-next, linux-kernel, ffmancera, Florian Westphal
Stephen Rothwell says:
today's linux-next build (x86_64 allmodconfig) produced this warning:
./usr/include/linux/netfilter/nf_osf.h:25: found __[us]{8,16,32,64} type without #include <linux/types.h>
Fix that up and also move kernel-private struct out of uapi (it was not
exposed in any released kernel version).
tested via allmodconfig build + make headers_check.
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Fixes: bfb15f2a95cb ("netfilter: extract Passive OS fingerprint infrastructure from xt_osf")
Signed-off-by: Florian Westphal <fw@strlen.de>
---
include/linux/netfilter/nf_osf.h | 6 ++++++
include/uapi/linux/netfilter/nf_osf.h | 8 ++------
2 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/include/linux/netfilter/nf_osf.h b/include/linux/netfilter/nf_osf.h
index a2b39602e87d..0e114c492fb8 100644
--- a/include/linux/netfilter/nf_osf.h
+++ b/include/linux/netfilter/nf_osf.h
@@ -21,6 +21,12 @@ enum osf_fmatch_states {
FMATCH_OPT_WRONG,
};
+struct nf_osf_finger {
+ struct rcu_head rcu_head;
+ struct list_head finger_entry;
+ struct nf_osf_user_finger finger;
+};
+
bool nf_osf_match(const struct sk_buff *skb, u_int8_t family,
int hooknum, struct net_device *in, struct net_device *out,
const struct nf_osf_info *info, struct net *net,
diff --git a/include/uapi/linux/netfilter/nf_osf.h b/include/uapi/linux/netfilter/nf_osf.h
index 45376eae31ef..8f2f2f403183 100644
--- a/include/uapi/linux/netfilter/nf_osf.h
+++ b/include/uapi/linux/netfilter/nf_osf.h
@@ -1,6 +1,8 @@
#ifndef _NF_OSF_H
#define _NF_OSF_H
+#include <linux/types.h>
+
#define MAXGENRELEN 32
#define NF_OSF_GENRE (1 << 0)
@@ -57,12 +59,6 @@ struct nf_osf_user_finger {
struct nf_osf_opt opt[MAX_IPOPTLEN];
};
-struct nf_osf_finger {
- struct rcu_head rcu_head;
- struct list_head finger_entry;
- struct nf_osf_user_finger finger;
-};
-
struct nf_osf_nlmsg {
struct nf_osf_user_finger f;
struct iphdr ip;
--
2.16.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: linux-next: build warning after merge of the netfilter-next tree
2018-05-08 7:17 ` Florian Westphal
@ 2018-05-15 10:54 ` Geert Uytterhoeven
2018-05-15 17:51 ` Florian Westphal
0 siblings, 1 reply; 7+ messages in thread
From: Geert Uytterhoeven @ 2018-05-15 10:54 UTC (permalink / raw)
To: Florian Westphal
Cc: Stephen Rothwell, David Miller, Pablo Neira Ayuso, NetFilter,
Linux-Next Mailing List, Linux Kernel Mailing List,
Fernando Fernandez Mancera
Hi Florian,
On Tue, May 8, 2018 at 9:17 AM, Florian Westphal <fw@strlen.de> wrote:
> Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>> On Mon, 7 May 2018 10:55:19 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>> >
>> > After merging the netfilter-next tree, today's linux-next build (x86_64
>> > allmodconfig) produced this warning:
>> >
>> > ./usr/include/linux/netfilter/nf_osf.h:25: found __[us]{8,16,32,64} type without #include <linux/types.h>
>>
>> > Introduced by commit
>> >
>> > bfb15f2a95cb ("netfilter: extract Passive OS fingerprint infrastructure from xt_osf")
>
> I'll send a fix for this, thanks for reporting.
+config NF_OSF
+ tristate 'Passive OS fingerprint infrastructure'
"There is no help available for this option."
Is this meant to be a user-visible symbol?
I noticed it is selected by NETFILTER_XT_MATCH_OSF.
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build warning after merge of the netfilter-next tree
2018-05-15 10:54 ` Geert Uytterhoeven
@ 2018-05-15 17:51 ` Florian Westphal
0 siblings, 0 replies; 7+ messages in thread
From: Florian Westphal @ 2018-05-15 17:51 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Florian Westphal, Stephen Rothwell, David Miller,
Pablo Neira Ayuso, NetFilter, Linux-Next Mailing List,
Linux Kernel Mailing List, Fernando Fernandez Mancera
Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> On Tue, May 8, 2018 at 9:17 AM, Florian Westphal <fw@strlen.de> wrote:
> > Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >> On Mon, 7 May 2018 10:55:19 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >> >
> >> > After merging the netfilter-next tree, today's linux-next build (x86_64
> >> > allmodconfig) produced this warning:
> >> >
> >> > ./usr/include/linux/netfilter/nf_osf.h:25: found __[us]{8,16,32,64} type without #include <linux/types.h>
> >>
> >> > Introduced by commit
> >> >
> >> > bfb15f2a95cb ("netfilter: extract Passive OS fingerprint infrastructure from xt_osf")
> >
> > I'll send a fix for this, thanks for reporting.
>
> +config NF_OSF
> + tristate 'Passive OS fingerprint infrastructure'
>
> "There is no help available for this option."
>
> Is this meant to be a user-visible symbol?
No, its not. I can send a patch in case you're too busy, let me know.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH nf-next] netfilter: fix fallout from xt/nf osf separation
2018-05-08 8:05 ` [PATCH nf-next] netfilter: fix fallout from xt/nf osf separation Florian Westphal
@ 2018-05-17 11:52 ` Pablo Neira Ayuso
0 siblings, 0 replies; 7+ messages in thread
From: Pablo Neira Ayuso @ 2018-05-17 11:52 UTC (permalink / raw)
To: Florian Westphal
Cc: netfilter-devel, sfr, linux-next, linux-kernel, ffmancera
On Tue, May 08, 2018 at 10:05:38AM +0200, Florian Westphal wrote:
> Stephen Rothwell says:
> today's linux-next build (x86_64 allmodconfig) produced this warning:
> ./usr/include/linux/netfilter/nf_osf.h:25: found __[us]{8,16,32,64} type without #include <linux/types.h>
>
> Fix that up and also move kernel-private struct out of uapi (it was not
> exposed in any released kernel version).
>
> tested via allmodconfig build + make headers_check.
Applied, thanks Florian.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-05-17 11:52 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-07 0:55 linux-next: build warning after merge of the netfilter-next tree Stephen Rothwell
2018-05-08 0:21 ` Stephen Rothwell
2018-05-08 7:17 ` Florian Westphal
2018-05-15 10:54 ` Geert Uytterhoeven
2018-05-15 17:51 ` Florian Westphal
2018-05-08 8:05 ` [PATCH nf-next] netfilter: fix fallout from xt/nf osf separation Florian Westphal
2018-05-17 11:52 ` Pablo Neira Ayuso
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).