netdev - Re: [PATCH net-next] net: flow_dissector.c missing include linux/export.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 24 Jan 2012 22:42:44 +0100
From:	Jesper Dangaard Brouer <hawk@...x.dk>
To:	David Miller <davem@...emloft.net>
Cc:	paul.gortmaker@...driver.com, eric.dumazet@...il.com,
	netdev@...r.kernel.org, sfr@...b.auug.org.au
Subject: Re: [PATCH net-next] net: flow_dissector.c missing include
 linux/export.h

On Tue, 2012-01-24 at 15:51 -0500, David Miller wrote:
> Jesper did you actually hit this problem and get a real build
> failure? 

Yes, but I didn't explain the circumstances, sorry.

I was backporting Erics SFQ bug fix (sch_sfq: dont put new flow at the
end of flows / d47a0ac7b66) to stable kernel 3.2.1.

But to avoid conflicts I also cherry-picked the other SFQ changes, plus
the skb_flow_dissect changes, and cherry-picked:

 # sch_sfq: use skb_flow_dissect()
 # (Need flow_dissect APIs)
 git cherry-pick 11fca931d35a34d01ce209eb8d51ff667c9f5e7c

 # net: introduce skb_flow_dissect()
 git cherry-pick 0744dd00c1b1be99a25b62b1b48df440e82e57e0

 # flow_dissector: use a 64bit load/store
 git cherry-pick 4d77d2b567ec66a443792d99e96ac760991d80d0

 # sch_sfq: rehash queues in perturb timer
 git cherry-pick 225d9b89c937633dfeec502741a174fe0bab5b9f

 # sch_sfq: dont put new flow at the end of flows
 git cherry-pick  d47a0ac7b66883987275598d6039f902f4410ca9

 # net_sched: sfq: fix mem alloc error recovery
 git cherry-pick bd16a6cce2a7f169b559abc5672fd2c66e91fb36

So, I guess the "problem" is because I didn't have the patch, which adds
the include net/netprio_cgroup.h in linux/netdevice.h, and have the side
effect of including module.h, the patch:
 net: add network priority cgroup infrastructure (v4) / 5bc1421e34ecfe

git describe --contains 5bc1421e34ecfe
v3.3-rc1~182^2~519

Still think it more correct to include linux/export.h in
flow_dissector.c.

-- 
Best regards,
  Jesper Dangaard Brouer
  ComX Networks A/S
  Linux Network Kernel Developer
  Cand. Scient Datalog / MSc.CS
  Author of http://www.iptv-analyzer.org
  LinkedIn: http://www.linkedin.com/in/brouer

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html