On Wed, Jun 17, 2020 at 8:50 AM Van Haaren, Harry <[email protected]> wrote: > > > -----Original Message----- > > From: William Tu <[email protected]> > > Sent: Tuesday, June 16, 2020 4:42 PM > > To: Van Haaren, Harry <[email protected]> > > Cc: ovs-dev <[email protected]>; Stokes, Ian <[email protected]>; > > Ilya Maximets <[email protected]>; Federico Iezzi <[email protected]> > > Subject: Re: [PATCH v3 3/7] dpif-netdev: add subtable-lookup-get command for > > usability > > > > On Wed, Jun 10, 2020 at 3:47 AM Harry van Haaren > > <[email protected]> wrote: > > > > > > This commit introduces a new command, "dpif-netdev/subtable-lookup-get" > > add prio > > Hey William, > > Just reviewing your FB in detail here, today the commands work as follows: > - the "set" command is "subtable-lookup-prio-set" > - the "get" command is "subtable-lookup-get" // Note : no "prio" here > > This is done on purpose - the get returns the list of subtable lookup > implementations, > and is not really technically directly to do with "prio" . It does however, > return the priority too...
I see, thanks > > Open to suggestions, would the more similar commands below be better? > subtable-lookup-prio-get > subtable-lookup-prio-set I don't have strong opinion, but I prefer subtable-lookup-prio-get subtable-lookup-prio-set btw, when we issue ovs-appctl subtable-lookup-get, does it show "for each subtable", its corresponding lookup function? Because each subtable might not always uses the highest prio lookup function, correct? William _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
