Thread (10 messages) flat view 10 messages, 4 authors, 2016-10-06

Re: [GIT] Networking

From: Stephen Rothwell <hidden>
Date: 2016-10-06 00:52:40
Also in: lkml, netfilter-devel

Hi Linus,

On Wed, 5 Oct 2016 15:37:17 -0700 Linus Torvalds [off-list ref] wrote:
On Wed, Oct 5, 2016 at 3:29 PM, Stephen Rothwell [off-list ref] wrote:
quoted
I have been carrying the following merge fix patch (for the merge of
the net-next tree with Linus' tree) for a while now which seems to have
got missed:  
Ugh. It doesn't seem to be a merge error, because that double iph
assignment came from the original patch that introduced this function:
commit ddc8b6027ad0 ("netfilter: introduce nft_set_pktinfo_{ipv4,
ipv6}_validate()").
Except that commit effectively moved that function from
net/netfilter/nf_tables_netdev.c to
include/net/netfilter/nf_tables_ipv4.h while commit c73c24849011
("netfilter: nf_tables_netdev: remove redundant ip_hdr assignment")
removed the assignment in the original file (and has been in your tree
since v4.8-rc7) and that is where I originally actually got a conflict.
So I wouldn't call it a merge error - it just looks like a bug in the
network layer. So I'm not going to apply your patch even though it
looks plausible to me, simply because it's outside my area of
expertise.
no worries.
-- 
Cheers,
Stephen Rothwell
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help