diff mbox series

net: dev_is_mac_header_xmit() true for ARPHRD_RAWIP

Message ID 20190124110702.140834-1-zenczykowski@gmail.com
State Accepted
Delegated to: David Miller
Headers show
Series net: dev_is_mac_header_xmit() true for ARPHRD_RAWIP | expand

Commit Message

Maciej Żenczykowski Jan. 24, 2019, 11:07 a.m. UTC 
From: Maciej Żenczykowski <maze@google.com>

__bpf_redirect() and act_mirred checks this boolean
to determine whether to prefix an ethernet header.

Signed-off-by: Maciej Żenczykowski <maze@google.com>
---
 include/linux/if_arp.h | 1 +
 1 file changed, 1 insertion(+)

Comments

Daniel Borkmann Jan. 24, 2019, 11:27 a.m. UTC | #1 
On 01/24/2019 12:07 PM, Maciej Żenczykowski wrote:
> From: Maciej Żenczykowski <maze@google.com>
> 
> __bpf_redirect() and act_mirred checks this boolean
> to determine whether to prefix an ethernet header.
> 
> Signed-off-by: Maciej Żenczykowski <maze@google.com>

I presume David would pick this up for net tree, so:

Acked-by: Daniel Borkmann <daniel@iogearbox.net>
David Miller Jan. 25, 2019, 6:45 a.m. UTC | #2 
From: Maciej Żenczykowski <zenczykowski@gmail.com>
Date: Thu, 24 Jan 2019 03:07:02 -0800

> From: Maciej Żenczykowski <maze@google.com>
> 
> __bpf_redirect() and act_mirred checks this boolean
> to determine whether to prefix an ethernet header.
> 
> Signed-off-by: Maciej Żenczykowski <maze@google.com>

Applied.
diff mbox series

Patch

diff --git a/include/linux/if_arp.h b/include/linux/if_arp.h
index 6756fea18b69..e44746de95cd 100644
--- a/include/linux/if_arp.h
+++ b/include/linux/if_arp.h
@@ -54,6 +54,7 @@  static inline bool dev_is_mac_header_xmit(const struct net_device *dev)
 	case ARPHRD_IPGRE:
 	case ARPHRD_VOID:
 	case ARPHRD_NONE:
+	case ARPHRD_RAWIP:
 		return false;
 	default:
 		return true;