Re: [ovs-dev] [PATCH 2/3] odp-execute: Consolidate callbacks.

2013-12-30 Thread Jarno Rajahalme
Pushed to master, Jarno On Dec 30, 2013, at 3:57 PM, Jarno Rajahalme wrote: > Not dropped, just worked on something else recently. I’ll push these ASAP. > > Jarno > > On Dec 30, 2013, at 2:26 PM, Ben Pfaff wrote: > >> On Mon, Dec 16, 2013 at 10:25:48PM -0800, Ben Pfaff wrote: >>> On Fri,

Re: [ovs-dev] [PATCH 2/3] odp-execute: Consolidate callbacks.

2013-12-30 Thread Jarno Rajahalme
Not dropped, just worked on something else recently. I’ll push these ASAP. Jarno On Dec 30, 2013, at 2:26 PM, Ben Pfaff wrote: > On Mon, Dec 16, 2013 at 10:25:48PM -0800, Ben Pfaff wrote: >> On Fri, Nov 15, 2013 at 03:12:19PM -0800, Jarno Rajahalme wrote: >>> Use one callback instead of many,

Re: [ovs-dev] [PATCH 2/3] odp-execute: Consolidate callbacks.

2013-12-30 Thread Ben Pfaff
On Mon, Dec 16, 2013 at 10:25:48PM -0800, Ben Pfaff wrote: > On Fri, Nov 15, 2013 at 03:12:19PM -0800, Jarno Rajahalme wrote: > > Use one callback instead of many, helps in adding new functionality > > later on. > > > > Signed-off-by: Jarno Rajahalme > > I see in the last round of reviews I only

Re: [ovs-dev] [PATCH 2/3] odp-execute: Consolidate callbacks.

2013-12-16 Thread Ben Pfaff
On Fri, Nov 15, 2013 at 03:12:19PM -0800, Jarno Rajahalme wrote: > Use one callback instead of many, helps in adding new functionality > later on. > > Signed-off-by: Jarno Rajahalme I see in the last round of reviews I only asked for an extra comment. I see that you added one, so: Acked-by: Be

[ovs-dev] [PATCH 2/3] odp-execute: Consolidate callbacks.

2013-11-15 Thread Jarno Rajahalme
Use one callback instead of many, helps in adding new functionality later on. Signed-off-by: Jarno Rajahalme --- lib/dpif-netdev.c| 65 ++ lib/dpif.c | 46 +++--- lib/odp-execute.c| 58