Re: [ovs-dev] [PATCH v2 2/3] ofp-util: New data structure for mapping between table names and numbers.

2018-02-01 Thread Yifeng Sun
Sure, I just reviewed the 3rd patch. On Wed, Jan 31, 2018 at 11:36 AM, Ben Pfaff wrote: > Thanks for the reviews of patches 1 and 2. Do you plan to review patch > 3 as well? I rebased and sent out v3 just now. > > On Tue, Jan 30, 2018 at 01:29:56PM -0800, Yifeng Sun wrote: > > Looks good to me

Re: [ovs-dev] [PATCH v2 2/3] ofp-util: New data structure for mapping between table names and numbers.

2018-01-31 Thread Ben Pfaff
Thanks for the reviews of patches 1 and 2. Do you plan to review patch 3 as well? I rebased and sent out v3 just now. On Tue, Jan 30, 2018 at 01:29:56PM -0800, Yifeng Sun wrote: > Looks good to me, thanks. > > Reviewed-by: Yifeng Sun > > > On Fri, Jan 12, 2018 at 12:57 PM, Ben Pfaff wrote:

Re: [ovs-dev] [PATCH v2 2/3] ofp-util: New data structure for mapping between table names and numbers.

2018-01-30 Thread Yifeng Sun
Looks good to me, thanks. Reviewed-by: Yifeng Sun On Fri, Jan 12, 2018 at 12:57 PM, Ben Pfaff wrote: > This shares the infrastructure for mapping port names and numbers. It will > be used in an upcoming commit. > > Signed-off-by: Ben Pfaff > --- > include/openvswitch/ofp-util.h | 33 +

[ovs-dev] [PATCH v2 2/3] ofp-util: New data structure for mapping between table names and numbers.

2018-01-12 Thread Ben Pfaff
This shares the infrastructure for mapping port names and numbers. It will be used in an upcoming commit. Signed-off-by: Ben Pfaff --- include/openvswitch/ofp-util.h | 33 +++-- lib/ofp-util.c | 150 ++--- 2 files changed, 138 insertions(