[ovs-dev] [PATCH v3 05/16] ofp-actions: Add clone action.

William Tu u9012063 at gmail.com
Wed Dec 21 00:39:39 UTC 2016


Hi Joe,
Thanks I will take a look.
William

On Tue, Dec 20, 2016 at 10:25 AM, Joe Stringer <joe at ovn.org> wrote:
> On 18 December 2016 at 00:18, Ben Pfaff <blp at ovn.org> wrote:
>> From: William Tu <u9012063 at gmail.com>
>>
>> This patch adds OpenFlow clone action with syntax as below:
>> "clone([action][,action...])".  The clone() action makes a copy of the
>> current packet and executes the list of actions against the packet,
>> without affecting the packet after the "clone(...)" action.  In other
>> word, the packet before the clone() and after the clone() is the same,
>> no matter what actions executed inside the clone().
>>
>> Use case 1:
>> Set different fields and output to different ports without unset
>> actions=
>>   clone(mod_dl_src:<mac1>, output:1), clone(mod_dl_dst:<mac2>, output:2), output:3
>> Since each clone() has independent packet, output:1 has only dl_src modified,
>> output:2 has only dl_dst modified, output:3 has original packet.
>>
>> Similar to case1
>> actions=
>>   push_vlan(...), output:2, pop_vlan, push_vlan(...), output:3
>> can be changed to
>> actions=
>>   clone(push_vlan(...), output:2),clone(push_vlan(...), output:3)
>> without having to add pop_vlan.
>>
>> case 2: resubmit to another table without worrying packet being modified
>>   actions=clone(resubmit(1,2)), ...
>>
>> Signed-off-by: William Tu <u9012063 at gmail.com>
>> [blp at ovn.org revised this to omit the "sample" action]
>> Signed-off-by: Ben Pfaff <blp at ovn.org>
>
> It looks like the newly added system-traffic tests are failing on all
> platforms with this patch applied.. William, will you take a look?


More information about the dev mailing list