On Tue, Nov 24, 2015 at 09:40:20AM -0800, Jarno Rajahalme wrote: > > > On Nov 24, 2015, at 9:25 AM, Ben Pfaff <b...@ovn.org> wrote: > > > > On Thu, Nov 19, 2015 at 01:33:18PM -0800, Jarno Rajahalme wrote: > >> Currently ovs-ofctl replace-flows and diff-flows commands only support > >> flows in table 0. Extend this to cover all possible tables. > >> > >> Signed-off-by: Jarno Rajahalme <ja...@ovn.org> > > > > There's one oddity that may deserve consideration. It depends on how > > careful we want to be. > > > > OpenFlow 1.0 does not define a way to add a flow to a particular table. > > The switch is responsible for deciding which table is most appropriate > > for a given flow. For example, a switch might have one table that > > supports wildcards and another one that is exact-match (this is in fact > > specifically envisioned by OF1.0 through its insistence that exact-match > > flows have the highest priority). > > > > This means that when talking to an OF1.0 switch, "ovs-ofctl > > replace-flows" (and friends) should ignore the table number. If > > a flow on the switch is in table 1, but the input file says it is in > > table 0 (probably because it doesn't specify a table at all), ovs-ofctl > > should do nothing, because that's the desired state. > > > > So for an OF1.0 switch without the Table ID extension we should ignore > table numbers both ways, when reading from the file and when reading > from the switch, essentially pretend that there is only one table?
Yes. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
