>-----Original Message-----
>From: Shahaf Shuler <shah...@mellanox.com>
>Sent: 18. juni 2018 16:29
>To: Andrew Rybchenko <arybche...@solarflare.com>; Finn Christensen
><f...@napatech.com>; ian.sto...@intel.com
>Cc: d...@openvswitch.org; simon.hor...@netronome.com; f...@redhat.com
>Subject: RE: [ovs-dev] [PATCH v10 4/7] netdev-dpdk: implement flow offload
>with rte flow
>
>Monday, June 18, 2018 1:17 PM, Andrew Rybchenko:
>> Subject: Re: [ovs-dev] [PATCH v10 4/7] netdev-dpdk: implement flow
>> offload with rte flow
>>
>> On 05/18/2018 12:14 PM, Shahaf Shuler wrote:
>> > From: Finn Christensen <f...@napatech.com>
>> >
>> > The basic yet the major part of this patch is to translate the "match"
>> > to rte flow patterns. And then, we create a rte flow with MARK + RSS
>> > actions. Afterwards, all packets match the flow will have the mark
>> > id in the mbuf.
>> >
>> > The reason RSS is needed is, for most NICs, a MARK only action is
>> > not allowed. It has to be used together with some other actions,
>> > such as QUEUE, RSS, etc. However, QUEUE action can specify one queue
>> > only, which may break the rss. Likely, RSS action is currently the
>> > best we could now. Thus, RSS action is choosen.
>> >
>> > For any unsupported flows, such as MPLS, -1 is returned, meaning the
>> > flow offload is failed and then skipped.
>> >
>> > Co-authored-by: Yuanhan Liu <y...@fridaylinux.org>
>> > Signed-off-by: Finn Christensen <f...@napatech.com>
>> > Signed-off-by: Yuanhan Liu <y...@fridaylinux.org>
>> > Signed-off-by: Shahaf Shuler <shah...@mellanox.com>
>>
>> <...>
>>
>> > + /* VLAN */
>> > + struct rte_flow_item_vlan vlan_spec;
>> > + struct rte_flow_item_vlan vlan_mask;
>> > + memset(&vlan_spec, 0, sizeof(vlan_spec));
>> > + memset(&vlan_mask, 0, sizeof(vlan_mask));
>> > + if (match->wc.masks.vlans[0].tci && match->flow.vlans[0].tci) {
>> > + vlan_spec.tci = match->flow.vlans[0].tci;
>> > + vlan_mask.tci = match->wc.masks.vlans[0].tci;
>>
>> As I understand VLAN_CFI bit is used inside OVS to distinguish no VLAN
>> and zero VLAN cases (aka VLAN_TAG_PRESENT). So above two lines should
>> drop the VLAN_CFI bit, something like:
>>
>> vlan_spec.tci = match->flow.vlans[0].tci & ~htons(VLAN_CFI);
>> vlan_mask.tci = match->wc.masks.vlans[0].tci & ~htons(VLAN_CFI);
>
>Yes, thanks for spotting that. I guess matching on the CFI bit is not so
>important from offload perspective, I don't think we will have different
>actions of those.
>Will be addressed on v11.
Yes, I agree with this. Thanks Shahaf for correcting this in v11.
Finn
>
>>
>> Andrew.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
