Re: [ovs-dev] [PATCH v3] dpdk: add support for v2.1.0

2015-09-08 Thread Daniele Di Proietto
The patch looks good, I added you and Michal in AUTHORS and pushed
it to master.

Thanks

On 08/09/2015 07:56, "Puha, TimoX"  wrote:

>Hi,
>
>> From: Puha, TimoX
>> Subject: [PATCH v3] dpdk: add support for v2.1.0
>> 
>> Update relevant artifacts to add support for DPDK v2.1.0
>>  - INSTALL.DPDK.md
>>  - acinclude.m4: Change DPDK library name
>>  - netdev-dpdk: Limit minimum mbuf size to to adapt to DPDK bug fix that
>>changes the treatment of the requested mbuf size
>>  - build.sh: Change DPDK version number
>> 
>> Note that this breaks compatibility with DPDK v2.0.0 although only
>> for the library name change.
>> 
>> Note that throughput for vhost ports with mergeable buffers is reduced
>> about 10% due to a necessary bug fix in DPDK vhost code.
>> 
>> Signed-off-by: Mark Kavanagh 
>> Signed-off-by: Michal Weglicki 
>> Signed-off-by: Timo Puha 
>
>Any comments on this patch version?  This v3 only corrects a stray
>whitespace compared to v2, but v2 accommodates all the comments for v1 as
>far as I understood them right.
>
>-Timo
>___
>dev mailing list
>dev@openvswitch.org
>https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailma
>n_listinfo_dev&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=Sm
>B5nZacmXNq0gKCC1s_Cw5yUNjxgD4v5kJqZ2uWLlE&m=S87yaK-mAgMhyH9VLwPLZhSk_pXfWg
>vnjPcktpC60os&s=5SA9gsO7MxToyuNdf6Dv3kibISZrkoOWMWEsib_-pDs&e= 

___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev


Re: [ovs-dev] [PATCH v3] dpdk: add support for v2.1.0

2015-09-07 Thread Puha, TimoX
Hi,

> From: Puha, TimoX
> Subject: [PATCH v3] dpdk: add support for v2.1.0
> 
> Update relevant artifacts to add support for DPDK v2.1.0
>  - INSTALL.DPDK.md
>  - acinclude.m4: Change DPDK library name
>  - netdev-dpdk: Limit minimum mbuf size to to adapt to DPDK bug fix that
>changes the treatment of the requested mbuf size
>  - build.sh: Change DPDK version number
> 
> Note that this breaks compatibility with DPDK v2.0.0 although only
> for the library name change.
> 
> Note that throughput for vhost ports with mergeable buffers is reduced
> about 10% due to a necessary bug fix in DPDK vhost code.
> 
> Signed-off-by: Mark Kavanagh 
> Signed-off-by: Michal Weglicki 
> Signed-off-by: Timo Puha 

Any comments on this patch version?  This v3 only corrects a stray whitespace 
compared to v2, but v2 accommodates all the comments for v1 as far as I 
understood them right. 

-Timo
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev