netdev - Re: [PATCH net-next v2 1/5] net: don't reforward packets already forwarded by offload device

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:	Fri, 17 Jul 2015 12:59:11 -0700
From:	Scott Feldman <sfeldma@...il.com>
To:	Nicolas Dichtel <nicolas.dichtel@...nd.com>
Cc:	Netdev <netdev@...r.kernel.org>,
	Jiří Pírko <jiri@...nulli.us>,
	Roopa Prabhu <roopa@...ulusnetworks.com>,
	"simon.horman@...ronome.com" <simon.horman@...ronome.com>
Subject: Re: [PATCH net-next v2 1/5] net: don't reforward packets already
 forwarded by offload device

On Fri, Jul 17, 2015 at 7:37 AM, Nicolas Dichtel
<nicolas.dichtel@...nd.com> wrote:
> Le 16/07/2015 10:04, sfeldma@...il.com a écrit :
>>
>> From: Scott Feldman <sfeldma@...il.com>
>>
>> Just before queuing skb for xmit on port, check if skb has been marked by
>> switchdev port driver as already fordwarded by device.  If so, drop skb.
>> A
>> non-zero skb->offload_fwd_mark field is set by the switchdev port
>> driver/device on ingress to indicate the skb has already been forwarded by
>> the device to egress ports with matching dev->skb_mark.  The switchdev
>> port
>> driver would assign a non-zero dev->skb_mark for each device port netdev
>> during registration, for example.
>>
>> Signed-off-by: Scott Feldman <sfeldma@...il.com>
>>
>> ---
>>   include/linux/netdevice.h |    6 ++++++
>>   include/linux/skbuff.h    |   11 ++++++++++-
>>   net/core/dev.c            |   10 ++++++++++
>>   3 files changed, 26 insertions(+), 1 deletion(-)
>>
>> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
>> index 45cfd79..8364f29 100644
>> --- a/include/linux/netdevice.h
>> +++ b/include/linux/netdevice.h
>> @@ -1456,6 +1456,8 @@ enum netdev_priv_flags {
>>    *
>>    *    @xps_maps:      XXX: need comments on this one
>>    *
>> + *     @offload_fwd_mark:      Offload device fwding mark
>> + *
>>    *    @trans_start:           Time (in jiffies) of last Tx
>>    *    @watchdog_timeo:        Represents the timeout that is used by
>>    *                            the watchdog ( see dev_watchdog() )
>> @@ -1697,6 +1699,10 @@ struct net_device {
>>         struct xps_dev_maps __rcu *xps_maps;
>>   #endif
>>
>> +#ifdef CONFIG_NET_SWITCHDEV
>> +       u32                     offload_fwd_mark;
>> +#endif
>> +
>>         /* These may be needed for future network-power-down code. */
>>
>>         /*
>> diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h
>> index d6cdd6e..2edcf50 100644
>> --- a/include/linux/skbuff.h
>> +++ b/include/linux/skbuff.h
>> @@ -506,6 +506,7 @@ static inline u32 skb_mstamp_us_delta(const struct
>> skb_mstamp *t1,
>>    *    @no_fcs:  Request NIC to treat last 4 bytes as Ethernet FCS
>>     *   @napi_id: id of the NAPI struct this skb came from
>>    *    @secmark: security marking
>> + *     @offload_fwd_mark: fwding offload mark
>>    *    @mark: Generic packet mark
>>    *    @vlan_proto: vlan encapsulation protocol
>>    *    @vlan_tci: vlan tag control information
>> @@ -650,9 +651,17 @@ struct sk_buff {
>>                 unsigned int    sender_cpu;
>>         };
>>   #endif
>> +       union {
>>   #ifdef CONFIG_NETWORK_SECMARK
>> -       __u32                   secmark;
>> +               __u32           secmark;
>> +#endif
>> +#ifdef CONFIG_NET_SWITCHDEV
>> +               __u32           offload_fwd_mark;
>>   #endif
>> +       };
>> +
>> +       union {};
>> +
>
> Everybody seems to ack. For my knowledge, why did you put this empty union?

My bad...sending v3...
--
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