On Thu, Sep 13, 2012 at 10:44:00AM +0900, Simon Horman wrote: > On Wed, Sep 12, 2012 at 12:06:50PM -0700, Ben Pfaff wrote: > > On Wed, Sep 12, 2012 at 05:44:31PM +0900, Simon Horman wrote: > > > From: Isaku Yamahata <yamah...@valinux.co.jp> > > > > > > Signed-off-by: Isaku Yamahata <yamah...@valinux.co.jp> > > > Signed-off-by: Simon Horman <ho...@verge.net.au> > > > > I think that, if I'm reading this right, then attempting to convert > > many actions to OF1.2 will cause an assertion failure. We need to do > > something more graceful than that. > > Is the problem the use of NOT_REACHED() in ofpact_to_openflow12() > and ofpact_to_openflow12_common()? > > If so, I wonder if the code could be re-factored to allow an error to be > returned.
I don't think it's useful to report an error in response to a request to dump the flow table. > I also notice that ofpacts_put_openflow12_actions() isn't actually used, > rather ofpacts_put_openflow11_actions is used for Open Flow 1.2, but that > should be easy enough to fix. OF1.2 actions are a superset of OF1.1 actions, so I'm not sure there's any value in distinguishing for the purpose of outputting actions. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
