"dev" <dev-boun...@openvswitch.org> wrote on 06/09/2016 12:37:58 AM:
> From: Ben Pfaff <b...@ovn.org> > To: dev@openvswitch.org > Cc: Ben Pfaff <b...@ovn.org> > Date: 06/09/2016 12:58 AM > Subject: [ovs-dev, PATCH 3/4] ovn-controller: Add 'put_dhcp_opts' action in ovn-controller > Sent by: "dev" <dev-boun...@openvswitch.org> > > From: Numan Siddique <nusid...@redhat.com> > > This patch adds a new OVN action 'put_dhcp_opts' to support native > DHCP in OVN. > > ovn-controller parses this action and adds a NXT_PACKET_IN2 > OF flow with 'pause' flag set and the DHCP options stored in > 'userdata' field. > > When the valid DHCP packet is received by ovn-controller, it frames a > new DHCP reply packet with the DHCP options present in the > 'userdata' field and resumes the packet and stores 1 in the OVS > register provded in the 'put_dhcp_opts' parameter. If the packet is invalid, > it resumes the packet without any modifying and stores 0 in the OVS register. > > Eg. put_dhcp_opts(reg0, efferip = 10.0.0.4, router = 10.0.0.1, > netmask = 255.255.255.0, lease_time = 3600,....) > > A new 'DHCP_Options' table is added in SB DB which stores > the supported DHCP options with DHCP code and type. ovn-northd is > expected to popule this table. > > The next patch will add logical flows with this action. > > Signed-off-by: Numan Siddique <nusid...@redhat.com> > Co-authored-by: Ben Pfaff <b...@ovn.org> > Signed-off-by: Ben Pfaff <b...@ovn.org> > --- As a counter example to the last email, this patch set has a goto which I'm comfortable with (as the alternative hits my personal cyclomatic complexity limit) Acked-by: Ryan Moats <rmo...@us.ibm.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
