[ovs-dev] [PATCH v4 01/10] ofp-actions: allow ofpacts_put_openflow10() return error

[Ben Pfaff]

On Wed, Aug 15, 2012 at 11:36:03AM +0900, Isaku Yamahata wrote:
> On Tue, Aug 14, 2012 at 09:43:16AM -0700, Ben Pfaff wrote:
> > On Thu, Aug 02, 2012 at 12:24:06AM +0900, Isaku Yamahata wrote:
> > > When OF11+ only actions is introduced, ofpacts_put_openflow10() will be
> > > unable to handle all ofp-actions. So allow it return error in that case.
> > > 
> > > Signed-off-by: Isaku Yamahata <yamahata at valinux.co.jp>
> > 
> > I'd prefer to make it possible for OpenFlow 1.0 to handle all OF1.1
> > actions, by introducing extensions for them as necessary.
> 
> I see. So the problem is how to encode instructions/actions.
> How about the following?

I've been thinking about this stuff for a while (not ignoring you;
sorry it has taken for a while).

I think the answer is basically "I'm not sure yet."  So, how about
this for now: when an action or an instruction cannot be encoded in
OF1.0, simply do not encode it at all, that is, omit it from the
actions.  When I have a proper answer later, we can fix it up.

Thanks,

Ben.