Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Remove assertion for truncated

2017-10-10 Thread Iwase Yusuke
Hi Andy, Thanks so much! I will test it with my controller application. Thanks, Iwase On 2017年10月11日 04:29, Andy Zhou wrote: On Tue, Oct 10, 2017 at 12:28 AM, Iwase Yusuke wrote: Hi Andy, Thank you for your quick response. Your patch looks good to me. I couldn't judge whether w

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Remove assertion for truncated

2017-10-10 Thread Iwase Yusuke
Hi Andy, Thank you for your quick response. Your patch looks good to me. I couldn't judge whether we can "truncate" completely or not, that sounds great! Thanks, Iwase On 2017年10月10日 16:21, Andy Zhou wrote: On Mon, Oct 9, 2017 at 8:18 PM, Iwase Yusuke wrote: Hi Ben and A

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Remove assertion for truncated

2017-10-09 Thread Iwase Yusuke
added the assertions. I sent him a message asking about it. On Tue, Oct 10, 2017 at 11:39:42AM +0900, Iwase Yusuke wrote: Hi, I'm very sorry for disturbing you. Could someone review this patch? Or are there some more procedures for submitting patch? Thanks, Iwase On 2017年10月04日 22:54, IWA

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Remove assertion for truncated

2017-10-09 Thread Iwase Yusuke
Hi, I'm very sorry for disturbing you. Could someone review this patch? Or are there some more procedures for submitting patch? Thanks, Iwase On 2017年10月04日 22:54, IWASE Yusuke wrote: Because OpenFlow Spec does not clearly stipulate that "max_len" in OUTPUT action must be

[ovs-dev] [PATCH] ofproto-dpif-xlate: Remove assertion for truncated

2017-10-04 Thread IWASE Yusuke
so this assertion causes the lack of the interoperability with some controller implementations. This patch removes these redundant assertions of if truncated or not. Signed-off-by: IWASE Yusuke --- ofproto/ofproto-dpif-xlate.c | 5 - 1 file changed, 5 deletions(-) diff --git a/ofproto/ofproto-dpi

Re: [ovs-dev] [PATCH] python/setup: "-fPIC" extra compile flag for C extension

2017-01-10 Thread Iwase Yusuke
atch... Thanks, Iwase On 2017年01月11日 11:16, Iwase Yusuke wrote: > Hi, > > On 2017年01月11日 11:01, Ben Pfaff wrote: >> On Wed, Jan 11, 2017 at 09:38:13AM +0900, IWASE Yusuke wrote: >>> To build shared library object, it is required to built as "Position >>>

Re: [ovs-dev] [PATCH] python/setup: "-fPIC" extra compile flag for C extension

2017-01-10 Thread Iwase Yusuke
Hi, On 2017年01月11日 11:01, Ben Pfaff wrote: > On Wed, Jan 11, 2017 at 09:38:13AM +0900, IWASE Yusuke wrote: >> To build shared library object, it is required to built as "Position >> Independent Code" (on x86-64, for example). >> This patch adds "-fPIC&q

[ovs-dev] [PATCH] python/setup: "-fPIC" extra compile flag for C extension

2017-01-10 Thread IWASE Yusuke
To build shared library object, it is required to built as "Position Independent Code" (on x86-64, for example). This patch adds "-fPIC" extra compile flag to make sure that extension being PIC while building a shared library object. Signed-off-by: IWASE Yusuke ---