On Fri, Jun 16, 2017 at 7:42 PM, Brian Haley <haleyb....@gmail.com> wrote:
> On 06/15/2017 04:39 AM, nusid...@redhat.com wrote: > >> From: Numan Siddique <nusid...@redhat.com> >> >> Renamed 'struct dhcp_opts_map' to 'struct gen_opts_map' and >> renamed ovn-dhcp.h to ovn-l7.h. An upcoming commit to support IPv6 >> Router Advertisement, will make use of the refactored code to store >> the IPv6 ND RA options in 'struct gen_opts_map'. >> > > <snip> > > @@ -233,16 +233,16 @@ struct ovnact_put_mac_bind { >> struct expr_field mac; /* 48-bit Ethernet address. */ >> }; >> -struct ovnact_dhcp_option { >> - const struct dhcp_opts_map *option; >> +struct ovnact_gen_option { >> + const struct gen_opts_map *option; >> struct expr_constant_set value; >> }; >> /* OVNACT_PUT_DHCPV4_OPTS, OVNACT_PUT_DHCPV6_OPTS. */ > > > Super-nit: guess this comment should change here or in patch 5/6, > otherwise looks good. Thanks Brian for the review. I think it should change in patch 5 to also include OVNACT_PUT_ND_RA_OPTS. I will incorporate it in my next patch set. Numan > > > -struct ovnact_put_dhcp_opts { >> +struct ovnact_put_opts { >> struct ovnact ovnact; >> struct expr_field dst; /* 1-bit destination field. */ >> - struct ovnact_dhcp_option *options; >> + struct ovnact_gen_option *options; >> size_t n_options; >> }; >> > > -Brian > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev