Re: [ovs-dev] [no-slow 1/6] ofproto-dpif: Add ability to look up an ofproto by UUID.

2018-01-09 Thread Justin Pettit
> On Jan 3, 2018, at 3:31 PM, Ben Pfaff wrote: > > On Tue, Jan 02, 2018 at 10:24:21AM -0800, Justin Pettit wrote: >> >> >>> On Jan 2, 2018, at 8:40 AM, Ben Pfaff wrote: >>> >>> On Thu, Dec 21, 2017 at 02:25:10PM -0800, Justin Pettit wrote: This will have

Re: [ovs-dev] [no-slow 1/6] ofproto-dpif: Add ability to look up an ofproto by UUID.

2018-01-03 Thread Ben Pfaff
On Tue, Jan 02, 2018 at 10:24:21AM -0800, Justin Pettit wrote: > > > > On Jan 2, 2018, at 8:40 AM, Ben Pfaff wrote: > > > > On Thu, Dec 21, 2017 at 02:25:10PM -0800, Justin Pettit wrote: > >> This will have callers in the future. > >> > >> Signed-off-by: Justin Pettit

Re: [ovs-dev] [no-slow 1/6] ofproto-dpif: Add ability to look up an ofproto by UUID.

2018-01-02 Thread Justin Pettit
> On Jan 2, 2018, at 8:40 AM, Ben Pfaff wrote: > > On Thu, Dec 21, 2017 at 02:25:10PM -0800, Justin Pettit wrote: >> This will have callers in the future. >> >> Signed-off-by: Justin Pettit > > The new comment in struct ofproto_dpif only refers to one of the >

Re: [ovs-dev] [no-slow 1/6] ofproto-dpif: Add ability to look up an ofproto by UUID.

2018-01-02 Thread Ben Pfaff
On Thu, Dec 21, 2017 at 02:25:10PM -0800, Justin Pettit wrote: > This will have callers in the future. > > Signed-off-by: Justin Pettit The new comment in struct ofproto_dpif only refers to one of the hmap_nodes. Acked-by: Ben Pfaff

Re: [ovs-dev] [no-slow 1/6] ofproto-dpif: Add ability to look up an ofproto by UUID.

2017-12-27 Thread Justin Pettit
> On Dec 26, 2017, at 2:14 PM, Gregory Rose wrote: > > I'll continue to look into it. I looked at this patch more carefully, and I don't see where it would cause issues. It mostly renames a couple of things and adds a new function which isn't used in this patch. Of

Re: [ovs-dev] [no-slow 1/6] ofproto-dpif: Add ability to look up an ofproto by UUID.

2017-12-26 Thread Gregory Rose
On 12/26/2017 10:15 AM, Gregory Rose wrote: On 12/26/2017 10:08 AM, Justin Pettit wrote: Thanks for trying it out, Greg.  I don't know why that would affect the kernel checks, and I don't see them on my system with this patch: -=-=-=-=-=-=-=- ## -- ## ##

Re: [ovs-dev] [no-slow 1/6] ofproto-dpif: Add ability to look up an ofproto by UUID.

2017-12-26 Thread Gregory Rose
On 12/26/2017 10:08 AM, Justin Pettit wrote: Thanks for trying it out, Greg. I don't know why that would affect the kernel checks, and I don't see them on my system with this patch: -=-=-=-=-=-=-=- ## -- ## ## openvswitch 2.8.90 test suite. ## ##

Re: [ovs-dev] [no-slow 1/6] ofproto-dpif: Add ability to look up an ofproto by UUID.

2017-12-26 Thread Justin Pettit
Thanks for trying it out, Greg. I don't know why that would affect the kernel checks, and I don't see them on my system with this patch: -=-=-=-=-=-=-=- ## -- ## ## openvswitch 2.8.90 test suite. ## ## -- ## datapath-sanity 1: datapath

[ovs-dev] [no-slow 1/6] ofproto-dpif: Add ability to look up an ofproto by UUID.

2017-12-21 Thread Justin Pettit
This will have callers in the future. Signed-off-by: Justin Pettit --- ofproto/ofproto-dpif-trace.c | 2 +- ofproto/ofproto-dpif.c | 92 +++- ofproto/ofproto-dpif.h | 13 +-- 3 files changed, 77 insertions(+), 30