On Wed, Sep 28, 2016 at 11:47 PM, Ben Pfaff wrote:
> On Wed, Sep 28, 2016 at 03:17:04AM +, Zongkai LI wrote:
>> From: Zongkai LI
>>
>> This patch aims to extend table Address_Set to Set, make it more common to
>> accept variable set, not only address. And by that, we can skinny down ACLs,
>>
On Thu, Sep 15, 2016 at 7:48 AM, Ben Pfaff wrote:
> On Thu, Sep 01, 2016 at 09:56:30AM +, Zongkai LI wrote:
>> This patch adds default_dhcpv4_options and default_dhcpv6_options columns for
>> Logical_Switch, which should help CMS not to calculate and set dhcpv4_options
>> and dhcpv6_options co
On Thu, Sep 8, 2016 at 9:39 AM, Justin Pettit wrote:
>
>> On Sep 2, 2016, at 10:09 PM, Zongkai LI wrote:
>>
>> This patch updates method is_nd to let type ND_ROUTER_SOLICIT,
>> ND_ROUTER_ADVERT, ND_REDIRECT can be recoginzed.
>> And introduces method is_nd_neighbor for inherit current is_nd behav
On Thu, Sep 8, 2016 at 9:13 AM, Justin Pettit wrote:
>
>> On Sep 2, 2016, at 10:09 PM, Zongkai LI wrote:
>>
>> This patch updates symbols in logical-fields for ND, such as "nd",
>> "nd.target", "nd.sll", "nd.tll", to describe more clear about "icmp6.type"
>> predicate.
>> It adds new symbols:
>>
hi, guys, please help review serial patches supporting Router
Solicitation responder.
http://patchwork.ozlabs.org/patch/665380/ [ovs-dev,v2,1/8] packets:
add compose_nd_ra to compose Router Advertisement packet
http://patchwork.ozlabs.org/patch/665381/ [ovs-dev,v2,2/8] lib: rename
ovs_nd_opt to ov
On Thu, Sep 1, 2016 at 6:05 PM, Numan Siddique wrote:
>
>
> On Wed, Aug 31, 2016 at 8:44 AM, Zong Kai Li wrote:
>>
>> On Tue, Aug 30, 2016 at 8:28 PM, Numan Siddique
>> wrote:
>> >
>> >
>> > On Tue, Aug 30, 2016 at 9:40 AM, Ben Pfaff wrot
On Tue, Aug 30, 2016 at 8:28 PM, Numan Siddique wrote:
>
>
> On Tue, Aug 30, 2016 at 9:40 AM, Ben Pfaff wrote:
>>
>> The patch will get reviewed, there's no need to ask again.
>>
>> On Tue, Aug 30, 2016 at 11:09:23AM +0800, Zong Kai Li wrote:
>> >
Hi, Numan and Justin.
I wish you can help review patch "allow DHCPv6 respond multiple IA
Address Options".
http://patchwork.ozlabs.org/patch/663493/
For current DHCPv6 implementation, it will respond one IPv6 address
for a lswitch port in a DHCPv6 Reply message. The "one IPv6 address",
I mean, one
From: Zongkai LI
This patch adds default_dhcpv4_options and default_dhcpv6_options columns for
Logical_Switch, which should help CMS not to calculate and set dhcpv4_options
and dhcpv6_options columns for lswitch ports on lswitchs one by one, when
most of lswitch ports on the same lswitch are usin
From: Zongkai LI
This patch tries to allow OVN native DHCPv6 responder to respond multiple
IA Address Options in a DHCPv6 reply message, this will help a lswitch port
to configure all IPv6 addresses belongs to it in DHCPv6 processing.
This fixes issue lswitch port can only get one IPv6 address,
From: Zongkai LI
This patch adds specific inner actions 'put_nd_ra', 'put_nd_opt_prefix',
'put_nd_opt_mtu', 'put_nd_opt_sll' for action 'nd_ra'.
Action put_nd_ra will set cur_hop_limit(8-bit unsigned integer),
"Managed address configuration" and "Other configuration" flags(8-bit
unsigned integer
From: Zongkai LI
This patch adds a new logical flow table to support Router Advertisement
responder. And it also adds flow support for SLAAC.
---
ovn/northd/ovn-northd.c | 97 +---
ovn/ovn-nb.ovsschema| 5 ++-
ovn/ovn-nb.xml | 11 +
tests/
From: Zongkai LI
This patchs add a new action 'nd_ra' against current 'nd_na' action, to work as
a Router Solicitation packet responder. The inner actions to set fields, flags,
options in RA packet will be introduced in the following patch.
---
include/ovn/actions.h | 7 ++
ovn/controll
From: Zongkai LI
This serial patches try to support Stateless Address Autoconfiguration(SLAAC)
for IPv6, via Router Solicitation (RS) message responder, which will reply
Router Advertisement (RA) message.
User case:
After this serial patches, a user can follow the following steps to let all
From: Zongkai LI
This patch updates method "is_nd" to let type ND_ROUTER_SOLICIT,
ND_ROUTER_ADVERT, ND_REDIRECT can be recoginzed.
---
lib/flow.h | 5 -
lib/nx-match.c | 4 +++-
lib/odp-util.c | 6 +-
ovn/controller/pinctrl.c | 10 ++
4 files
From: Zongkai LI
This patch introduces methods to compose a Router Advertisement (RA) packet,
introduces flags for RA, and renames ovs_nd_opt to ovs_nd_lla_opt to specify
it's Source/Target Link-layer Address option.
---
lib/flow.c| 26 -
lib/odp-execute.c | 20 +++
lib/pac
From: Zongkai LI
This patch updates symbols in logical-fields for ND, such as "nd",
"nd.target", "nd.sll", "nd.tll", to describe more clear about "icmp6.type"
predicate.
And it adds new symbols:
- "nd_rs" stands for Router Solicitation message,
- "nd_ra" stands for Router Advertisement message,
On Fri, Aug 12, 2016 at 1:33 AM, Russell Bryant wrote:
>
> On Tue, Aug 9, 2016 at 11:55 PM, Zong Kai LI wrote:
>>
>> This patch renames table Address_Set to Set, Address_Set.addresses to
>> Set.members to reflect a more broad purpose, that we can define other
>> typ
On Thu, Aug 11, 2016 at 1:51 PM, Han Zhou wrote:
>
>
> On Tue, Aug 9, 2016 at 8:55 PM, Zong Kai LI wrote:
>>
>> This patch renames table Address_Set to Set, Address_Set.addresses to
>> Set.members to reflect a more broad purpose, that we can define other
>>
On Thu, Aug 4, 2016 at 7:46 PM, Russell Bryant wrote:
>
>
> On Wed, Aug 3, 2016 at 6:31 PM, Ben Pfaff wrote:
>>
>> On Wed, Aug 03, 2016 at 11:14:20PM +0800, Zong Kai LI wrote:
>> > This patch aims to extend Address_Set to Macros, make it more common to
>> >
Zong Kai LI
---
ovn/controller/lflow.c| 22 +++---
ovn/northd/ovn-northd.c | 26 +-
ovn/ovn-nb.ovsschema | 12 ++--
ovn/ovn-nb.xml| 13 +++--
ovn/ovn-sb.ovsschema | 12 ++--
ovn/ovn-sb.xml
On Thu, Aug 4, 2016 at 6:31 AM, Ben Pfaff wrote:
> On Wed, Aug 03, 2016 at 11:14:20PM +0800, Zong Kai LI wrote:
>> This patch aims to extend Address_Set to Macros, make it more common to
>> accept variable set, not only address. And by that, we can skinny down ACLs,
>> if we
n
installation plenty of similar ACLs/lflows.
For port comes and leaves scenario, when there are no special ACL entries
assigned to ports, it will be the ones first come and last leave to update ACL
uuid refers in Logical_Switch.acl.
TODO:
- go check how will ovn-northd use port sets defi
On Wed, Aug 3, 2016 at 7:56 PM, Russell Bryant wrote:
>
>
> On Wed, Aug 3, 2016 at 7:01 AM, Zong Kai LI wrote:
>>
>> This patch aims to extend Address_Set to Macros_Set, make it more common
>> to
>> accept variable set, not only address. And by that, we can
make parsing work quicker.
Port set defined under Macros_Set will improve ovn-controller performance on
installation plenty of similar ACLs/lflows, but not for port comes and leaves
scenario.
Potential drawback: not sure yet.
TODO: go check how will ovn-northd use port sets defined under Macros
This patch tries to implement Router Advertisement (RA) responder for SLAAC
on ovn-controller side.
It parses lflows which have:
- match: inport == LRP_NAME && ip6.dst == ff02::2 && nd_rs
(nd_rs: icmp6.type == 133 && icmp6.code == 0 && ttl == 255)
- action: nd_ra{slaac(prefix,...,MTU, mac_add
This patch tries to implement Router Advertisement (RA) responder for SLAAC
on ovn-northd side.
It tries to build lflows per each Logical Router Port, who have IPv6 networks
and set their 'slaac' column to true.
The lflows will look like:
match=(inport == "lrp-32a71e0b-8b19-4c52-8cde-058325e4df5
This patch introduces methods to compose a Router Advertisement (RA) packet,
introduces flags for RA, renames ovs_nd_opt to ovs_nd_lla_opt to specify it's
Source/Target Link-layer Address option.
Signed-off-by: Zong Kai LI
---
lib/flow.c| 26 -
lib/odp-execute.c
ess, such as fa:16:3e:32:3c:e0.
After a RA packet is composed, the left nested actions will make RA packet
transmitted back to the inport, where Router Solicitation (RS) packet comes.
For more details, will be described in each following patches.
Zong Kai LI (1):
ovn: add SLAAC support for IP
Action "flags.loopback" is introduced recently, but symtab in test-ovn
doesn't recognize it yet. This may cause action parsing test broken.
This patch fixes that.
Signed-off-by: Zong Kai LI
---
tests/test-ovn.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/tests/tes
On Thu, Jul 28, 2016 at 2:04 PM, Ben Pfaff wrote:
> On Wed, Jul 27, 2016 at 04:02:11PM +0800, Zong Kai LI wrote:
>> This patch introduces(from north to south):
>> - OVN NB DB:
>> * mtu(integer) column for Logical_Switch, used by Router Advertisement
>>(RA) mess
rts,
which means ovn-controller will compose several RA packets for one RS packet,
I'm not sure whether it's good to keep use 'continuation'.
Signed-off-by: Zong Kai LI
---
lib/packets.c| 105 +++
lib/packets.h
It missed comma as constant seperator in match string for ND responder.
Signed-off-by: Zong Kai LI
---
ovn/northd/ovn-northd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c
index a3d1672..8e5c5c5 100644
--- a/ovn/northd/ovn
> http://openvswitch.org/mailman/listinfo/dev
I tested this patch on my 2 chassis environment, with manually
rebased, with ping test between VMs from different subnets attached on
the same router. And it works.
Thanks.
Zong Kai, LI
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
: Zong Kai LI
---
python/ovs/db/data.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/python/ovs/db/data.py b/python/ovs/db/data.py
index 747acd5..6d87f89 100644
--- a/python/ovs/db/data.py
+++ b/python/ovs/db/data.py
@@ -521,7 +521,7 @@ class Datum(object):
ka
On Wed, Jul 20, 2016 at 4:50 AM, Justin Pettit wrote:
>
>> On Jul 13, 2016, at 3:19 AM, Zong Kai Li wrote:
>>
>> Hi, Justin. I tried your patches one by one. P1~P5 are OK.
>> But something wrong seems happened to P6.
>> In my OpenStack integrated environment, af
on iterable
sequence type from python.
Signed-off-by: Zong Kai LI
---
python/ovs/db/data.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/python/ovs/db/data.py b/python/ovs/db/data.py
index 747acd5..6d87f89 100644
--- a/python/ovs/db/data.py
+++ b/python/ovs/db/data.py
@
>On Wed, Jul 13, 2016 at 6:19 PM, Zong Kai Li wrote:
> Hi, Justin. I tried your patches one by one. P1~P5 are OK.
> But something wrong seems happened to P6.
> In my OpenStack integrated environment, after using P6, I cannot ping
> VM dhcp netns. And later when I dump ovs flows, I
MFF_LOG_DNAT_ZONE...MFF_LOG_OUTPORT, and
this time, table OFTABLE_PHY_TO_LOG comes back.
I hope you can check this.
Thanks,
Zong Kai, LI
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
>On Wed, Jul 13, 2016 at 11:24 AM, Justin Pettit wrote:
>
>> On Jul 12, 2016, at 7:53 PM, Zong Kai Li wrote:
>>
>>> +
>>> +The regX symbols are 32-bit integers.
>>> +The xxregX symbols are 128-bit integers,
>>> +
MFF_REG7 to MFF_REG11...MFF_REG15. REGBIT_CONNTRACK_**
which are using MFF_REG0 should be renumbered too.
Thanks.
Zong Kai, LI
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
d connect to either a switch or another router, and a
lsp.peer should either be empty or be set with another lsp.name.
So when a lrp has its nbr->peer been set, we should parse current
network and mac column for it. For peer router port still these to
build static route.
And when a lrp has empty nbr->peer, we can let lsp processing branch
in join_logical_ports to extract addresses for it.
And even, for op is lrp and op->peer->nbs is NULL case, we can
directly use ip, network, mask via op->peer, without extracting
addresses for lrp at first.
I can't figure why a patched lsp and lrp could have different
addresses, networks, masks, macs. So I think this should make sense.
:)
Thanks.
Zong Kai, LI
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
ND_OPT_LEN);
>
Hi, Justin. It's quite a long time to wait support for IPv6, but I
think it worth. Thanks. :)
About using ALIGNED_CAST, would you mind checking
http://patchwork.ozlabs.org/patch/632026/ ?
Ben has a comment said "Using ALIGNED_CAST does not solve a pro
remote VM inport, it will have
50 + C flows been installed in table ls_in_port_sec_l2 on each node.
Signed-off-by: Zong Kai LI
---
ovn/controller/lflow.c | 42 +++--
ovn/controller/lflow.h | 3 ++-
ovn/controller/ovn-controller.c | 2 +-
ovn/northd/
ingress pipeline, when it tries to
match
a VM(or VIF) port that doesn't belong to current chassis.
This can help ovn-controller to avoid to install many unnecessary flows.
Signed-off-by: Zong Kai LI
---
ovn/controller/lflow.c | 42
+++--
ovn/contro
On Wed, Jul 6, 2016 at 5:14 PM, Numan Siddique wrote:
>
>
> On Tue, Jul 5, 2016 at 12:04 PM, Zong Kai LI wrote:
>
>> +/* Logical switch ingress table 10 and 11: DHCP options and response
>>> + * priority 100 flows. */
>>> +HMAP_FOR_EACH (op
So I
suppose this is another place will cause load balance failure.
Thanks.
Zong Kai, LI
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
dd(lflows, od, S_SWITCH_IN_DHCP_OPTIONS, 0, "1",
> "next;");
> +ovn_lflow_add(lflows, od, S_SWITCH_IN_DHCP_RESPONSE, 0, "1",
> "next;");
> +}
>
Great work, but only one thing makes me feel uncomfortable.
DHCP Options flows are genera
&& tcp && tcp.dst == %d",
> port);
> +}
> +ovn_lflow_add(lflows, od, S_SWITCH_IN_LB,
> + 120, ds_cstr(&match), action);
> +} else {
> +ovn_lflow_add(lflows, od, S_SWITCH_IN_LB,
> + 110, ds_cstr(&match), action);
> +}
>
S_SWITCH_IN_LB, I think you missed to put them into method build_lb.
> +
> +ds_destroy(&match);
> +free(action);
> + }
> +}
> }
>
>
Thanks.
Zong Kai, LI
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
t;30.0.0.1:8000"'='"10.0.0.1:80,
10.0.0.2:80,10.0.0.3:80"' vips:'"30.0.0.1:5000"'='"10.0.0.1:80,10.0.0.2:80,
10.0.0.3:80"', this time it get correctly parsed in ovn-northd.
Woud you mind to update the description here to notice
efault. */
> +ovn_lflow_add(lflows, od, S_SWITCH_IN_STATEFUL, 0, "1", "next;");
> +ovn_lflow_add(lflows, od, S_SWITCH_OUT_STATEFUL, 0, "1", "next;");
>
if we allow non-stateful stuff to skip PRE_STATEFUL and STATEFUL table, we
will don't need the default next action in the two tables.
Thanks,
Zong Kai, LI
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
t; + REGBIT_CONNTRACK_DEFRAG" == 1", "ct_next;");
> +ovn_lflow_add(lflows, od, S_SWITCH_OUT_PRE_STATEFUL, 100,
> + REGBIT_CONNTRACK_DEFRAG" == 1", "ct_next;");
> +}
>
I hope you can also modify the default
from-lport" to make table name simpler and clearer?
> Ingress table 4: from-lport ACLs
>
> - This is similar to ingress table 3 except for to-lport
> - traffic.
> + This is similar to ingress table Pre-ACLs except for
> + to-lport traffic.
>
I've tested this serial patches, it works as expected.
Only one thing I'm not sure:
diff --git a/ovn/utilities/ovn-sbctl.c b/ovn/utilities/ovn-sbctl.c
> index a12b247..d85efd5 100644
> --- a/ovn/utilities/ovn-sbctl.c
> +++ b/ovn/utilities/ovn-sbctl.c
> @@ -779,6 +779,10 @@ static const struct ctl
error);
> +struct in6_addr ipv6;
> +error = ipv6_parse_cidr(ctx->argv[1], &ipv6, &plen);
> +if (error) {
> +free(error);
> +VLOG_WARN("Invalid cidr format '%s'", ctx->argv[1]);
> +return;
>
ditto.
> +ovn-nbctl -- --id=@d1 create DHCP_Options cidr=10.0.0.0/24 \
> +options="\"server_id\"=\"10.0.0.1\" \"server_mac\"=\"ff:10:00:00:00:01\" \
> +\"lease_time\"=\"3600\" \"router\"=\"10.0.0.1\"" \
>
how about using nbctl_dhcp_options_create here?
+-- add Logical_Switch_Port ls1-lp1 dhcpv4_options @d1 \
> +-- add Logical_Switch_Port ls1-lp2 dhcpv4_options @d1
> +
> +ovn-nbctl -- --id=@d2 create DHCP_Options cidr=30.0.0.0/24 \
> +options="\"server_id\"=\"30.0.0.1\" \"server_mac\"=\"ff:10:00:00:00:02\" \
> +\"lease_time\"=\"3600\"" -- add Logical_Switch_Port ls2-lp2
> dhcpv4_options @d2
>
ditto
Thanks,
Zong Kai, LI
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
if (build_dhcpv4_action(op, laddrs.ipv4_addrs[j].addr,
> +&options_action,
> &response_action)) {
>
There are some dhcp_options relevant but not lsp address relevant checking
in build_dhcpv4_action.
So think about it, it's unnecessary to do duplicated
>
> +static void
> +update_address_sets(struct controller_ctx *ctx)
> +{
> +/* Remember the names of all address sets currently in
> expr_address_sets
> + * so we can detect address sets that have been deleted. */
> +struct sset cur_addr_set_names =
> SSET_INITIALIZER(&cur_addr_set_name
e;
> +}
> +
> +cs->type = EXPR_C_INTEGER;
>
Hi, Babu.
It's odd you don't directly fix here with assign_constant_set_type, but in
the following patch.
Hope it doesn't matter to others.
Thanks.
Zong Kai, LI
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
ent ACL lflows for ND, not to do conntrack
on ND and NA packets in following tables:
- S_SWITCH_IN_PRE_ACL
- S_SWITCH_OUT_PRE_ACL
- S_SWITCH_IN_ACL
- S_SWITCH_OUT_ACL
(Rebase on upstream)
Signed-off-by: Zong Kai LI
---
lib/packets.c| 29 ++
}
> +
> +sset_destroy(&cur_address_sets);
I think you don't need manually call sset_delete, sset_destroy will do
it for you.
> +SHASH_FOR_EACH_SAFE (node, next, &sb_address_sets) {
> + sbrec_address_set_delete(node->data);
> +shash_delete(&sb_address_sets,
On Fri, Jun 24, 2016 at 4:43 AM, Justin Pettit wrote:
>
> > diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c
> > index 17713ec..5851fca 100644
> > --- a/ovn/northd/ovn-northd.c
> > +++ b/ovn/northd/ovn-northd.c
> > @@ -612,11 +612,12 @@ join_logical_ports(struct northd_context *ctx,
ple addresses
for IPv6 address case, but it's peer lrp port has only 1 address in
'network' column. It's inconsistent, this patch tries to fix that.
Signed-off-by: Zong Kai LI
---
ovn/northd/ovn-northd.c | 5 +++--
ovn/ovn-nb.ovsschema | 6 --
ent ACL lflows for ND, not to do conntrack
on ND and NA packets in following tables:
- S_SWITCH_IN_PRE_ACL
- S_SWITCH_OUT_PRE_ACL
- S_SWITCH_IN_ACL
- S_SWITCH_OUT_ACL
Signed-off-by: Zong Kai LI
---
lib/packets.c| 29 ++
lib/packets.h| 4 ++
ovn/controller/pinctr
On Wed, Jun 15, 2016 at 10:17 PM, Ryan Moats wrote:
> "dev" wrote on 06/14/2016 10:09:37 PM:
>
> > From: Zong Kai LI
> > To: dev@openvswitch.org
> > Cc: Zong Kai LI
> > Date: 06/14/2016 10:11 PM
> > Subject: [ovs-dev] [PATCH] [PATCH, v4, 1
ent ACL lflows for ND, not to do conntrack
on ND and NA packets in following tables:
- S_SWITCH_IN_PRE_ACL
- S_SWITCH_OUT_PRE_ACL
- S_SWITCH_IN_ACL
- S_SWITCH_OUT_ACL
Signed-off-by: Zong Kai LI
---
lib/packets.c| 29 ++
lib/packets.h| 4 ++
ovn/controller/pinctr
tput; };)
And new lflows will be set in tabel ls_in_arp_nd_rsp, which is renamed
from previous ls_in_arp_rsp.
This patch also modified current ACL lflows for ND, not to do conntrack
on ND packets in following tables:
- S_SWITCH_IN_PRE_ACL
- S_SWITCH_OUT_PRE_ACL
- S_SWITCH_IN_ACL
- S_SWITCH_OUT_ACL
S
ill do logical flows works for this action.
Signed-off-by: Zong Kai LI
---
lib/packets.c| 29 ++
lib/packets.h| 4 ++
ovn/controller/pinctrl.c | 134 ++-
ovn/lib/actions.c| 43 +++
ovn/lib/actio
On Thu, Jun 9, 2016 at 12:43 AM, Justin Pettit wrote:
>
> > On Jun 8, 2016, at 1:12 AM, Zong Kai LI wrote:
> >
> > I know Justin is working on implement router patch port for IPv6 switch,
> but I'm not sure whether will Justin also work on implement ND or not.
On Thu, Jun 9, 2016 at 5:25 AM, Ben Pfaff wrote:
> On Wed, Jun 08, 2016 at 03:26:30PM +0800, Zong Kai LI wrote:
> > This patch adds a new OVN action 'na' to support ND versus ARP.
> >
> > When ovn-controller received a ND packet, it frames a NA packet for
> >
On Thu, Jun 9, 2016 at 5:32 AM, Ben Pfaff wrote:
> On Wed, Jun 08, 2016 at 03:28:58PM +0800, Zong Kai LI wrote:
> > This patch adds some lflows for 'na' action to support ND versus ARP.
> >
> > For ovn-northd, it will generate lflows per each IPv6 address on
>
ese are default values that the nested actions will
>> probably want
>> +to change:
>> +
>> +
>> +
>> +reg0 = 0x1(Mark as replied by
>> ovn-controller)
>> +outport copied from inport
>> +
On Tue, Jun 7, 2016 at 5:08 AM, Ben Pfaff wrote:
> On Fri, May 27, 2016 at 10:32:56AM +0800, Zong Kai LI wrote:
> > From: lzklibj
> >
> > This patch tries to implement ND for Logical_Port in OVN.
> >
> > Signed-off-by: lzklibj
>
> Thanks for workin
ention that such kind of NA packets are replied
by ovn-controller, and for these packets, dont do conntrack on them.
Also modfiy current table 32 and table 48, to make these packets
output directly.
Signed-off-by: Zong Kai LI
---
ovn/controller/physical.c |
atch will do logical flows works for this action.
Signed-off-by: Zong Kai LI
---
lib/packets.c| 33 +
lib/packets.h| 4 ++
ovn/controller/pinctrl.c | 119 +--
ovn/lib/actions.c| 50 +++
atch will do logical flows works for this action.
Signed-off-by: Zong Kai LI
---
lib/packets.c| 33 +
lib/packets.h| 4 ++
ovn/controller/pinctrl.c | 119 +--
ovn/lib/actions.c| 50 +++
ention that such kind of NA packets are replied
by ovn-controller, and for these packets, they dont need conntrack.
So I also modfiy current table 32 and table 48, to make these packets
output directly.
(Will try to add test once I figure out what to test.)
Signed-off-by: Zong Kai LI
---
ovn/control
lows for this action.
Signed-off-by: Zong Kai LI
---
ovn/controller/pinctrl.c | 119 +--
ovn/lib/actions.c| 50
ovn/lib/actions.h| 6 +++
ovn/lib/expr.c | 47 ++-
ovn/lib/expr.h | 43
Add a method compose_na to compose a NA packet.
The ND feature of OVN will base on this.
Signed-off-by: Zong Kai LI
---
lib/packets.c | 33 +
lib/packets.h | 4
2 files changed, 37 insertions(+)
diff --git a/lib/packets.c b/lib/packets.c
index 6a55d6f
From: lzklibj
This patch tries to implement ND for Logical_Port in OVN.
Signed-off-by: lzklibj
---
lib/packets.c| 2 +-
lib/packets.h| 3 +
ovn/controller/pinctrl.c | 156 +++
ovn/lib/actions.c| 50 +++
79 matches
Mail list logo
