Re: [ovs-dev] [merge native tunneling and patch port 1/7] ofproto-dpif: Unfreeze within clone

2017-09-27 Thread Andy Zhou
On Thu, Sep 21, 2017 at 9:44 AM, Greg Rose wrote: > On 09/12/2017 12:49 PM, Andy Zhou wrote: >> >> When translating actions within open flow clone, actions generated >> by finish_freezeing() should also be enclosed within the datapath >> clone netlink encoding. >> >>

Re: [ovs-dev] [merge native tunneling and patch port 1/7] ofproto-dpif: Unfreeze within clone

2017-09-19 Thread Greg Rose
On 09/19/2017 04:11 PM, Andy Zhou wrote: On Tue, Sep 19, 2017 at 1:56 PM, Greg Rose wrote: On 09/19/2017 01:26 PM, Andy Zhou wrote: Yes, I just tried on a VM running Centos 7.3 with the 4.9 kernel and it passed there. /shrug? OK, I'll continue with review then.

Re: [ovs-dev] [merge native tunneling and patch port 1/7] ofproto-dpif: Unfreeze within clone

2017-09-19 Thread Andy Zhou
On Tue, Sep 19, 2017 at 1:56 PM, Greg Rose wrote: > On 09/19/2017 01:26 PM, Andy Zhou wrote: >> >> On Tue, Sep 19, 2017 at 9:55 AM, Greg Rose wrote: >>> >>> On 09/12/2017 12:49 PM, Andy Zhou wrote: When translating actions within open

Re: [ovs-dev] [merge native tunneling and patch port 1/7] ofproto-dpif: Unfreeze within clone

2017-09-19 Thread Greg Rose
On 09/19/2017 01:26 PM, Andy Zhou wrote: On Tue, Sep 19, 2017 at 9:55 AM, Greg Rose wrote: On 09/12/2017 12:49 PM, Andy Zhou wrote: When translating actions within open flow clone, actions generated by finish_freezeing() should also be enclosed within the datapath clone

Re: [ovs-dev] [merge native tunneling and patch port 1/7] ofproto-dpif: Unfreeze within clone

2017-09-19 Thread Andy Zhou
On Tue, Sep 19, 2017 at 9:55 AM, Greg Rose wrote: > On 09/12/2017 12:49 PM, Andy Zhou wrote: >> >> When translating actions within open flow clone, actions generated >> by finish_freezeing() should also be enclosed within the datapath >> clone netlink encoding. >> >>

Re: [ovs-dev] [merge native tunneling and patch port 1/7] ofproto-dpif: Unfreeze within clone

2017-09-19 Thread Greg Rose
On 09/12/2017 12:49 PM, Andy Zhou wrote: When translating actions within open flow clone, actions generated by finish_freezeing() should also be enclosed within the datapath clone netlink encoding. Signed-off-by: Andy Zhou Andy, I am reviewing and testing your patches. I