[ovs-dev] [PATCH v3 2/3] ovn: Add ct_commit(ct_mark=INT, ct_label=INT); action.
Russell Bryant
russell at ovn.org
Mon Mar 21 18:23:10 UTC 2016
On Mon, Mar 21, 2016 at 11:16 AM, Guru Shetty <guru at ovn.org> wrote:
>
>
> On 21 March 2016 at 07:54, Russell Bryant <russell at ovn.org> wrote:
>
>> Update the "ct_commit;" logical flow action to optionally take
>> one or two parameters, setting the value of "ct_mark" or "ct_label".
>> Supported ct_commit syntax now includes:
>>
>> ct_commit;
>> ct_commit();
>> ct_commit(ct_mark=1);
>> ct_commit(ct_label=1);
>> ct_commit(ct_mark=1, ct_label=1);
>>
>> Setting ct_mark via this type of logical flow results in an OpenFlow
>> flow that looks like:
>>
>>
>> actions=ct(commit,zone=NXM_NX_REG5[0..15],exec(set_field:0x1->ct_mark))
>>
>> Similarly, setting ct_label results in:
>>
>>
>> actions=ct(commit,zone=NXM_NX_REG5[0..15],exec(set_field:0x1->ct_label))
>>
>
> I think this feature makes it tricky to share zones with other stateful
> additions. If you want to commit only once for all stateful services, then
> set-field for ct_mark and ct_label will need to be loaded to registers in
> advance, which I guess would mean that you loose 2 registers for this
> purpose.
>
Do you have any thoughts on how I could change this to cause you less pain
on the LB series? The next patch shows how this is used. In practice, it
only uses ct_label.
--
Russell Bryant
More information about the dev
mailing list