"dev" <[email protected]> wrote on 06/09/2016 12:37:58 AM:
> From: Ben Pfaff <[email protected]> > To: [email protected] > Cc: Ben Pfaff <[email protected]> > 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" <[email protected]> > > From: Numan Siddique <[email protected]> > > 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 <[email protected]> > Co-authored-by: Ben Pfaff <[email protected]> > Signed-off-by: Ben Pfaff <[email protected]> > --- 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 <[email protected]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
