Re: [ovs-dev] [PATCH 1/2] sparse: Add rte_memcpy.h replacement header.

2017-05-08 Thread Ben Pfaff
Darrell, do you plan to review patch 2? On Sun, May 07, 2017 at 08:58:35AM -0400, Ben Pfaff wrote: > OK, I added that it affects i386 and applied this to master. Thank you! > > On Sat, May 06, 2017 at 08:27:47PM +, Darrell Ball wrote: > > I am not able to reproduce this using 64bit arch., al

Re: [ovs-dev] [PATCH 1/2] sparse: Add rte_memcpy.h replacement header.

2017-05-07 Thread Ben Pfaff
OK, I added that it affects i386 and applied this to master. Thank you! On Sat, May 06, 2017 at 08:27:47PM +, Darrell Ball wrote: > I am not able to reproduce this using 64bit arch., although Sparse does work > for me. >(I guess I should install a 32 bit VM at some point, but I am low on

Re: [ovs-dev] [PATCH 1/2] sparse: Add rte_memcpy.h replacement header.

2017-05-06 Thread Darrell Ball
I am not able to reproduce this using 64bit arch., although Sparse does work for me. (I guess I should install a 32 bit VM at some point, but I am low on disk space - too many 64bit VMs) I tried tip of master and about 2 months old master Based on the above data point and the error output bel

[ovs-dev] [PATCH 1/2] sparse: Add rte_memcpy.h replacement header.

2017-05-05 Thread Ben Pfaff
Without this replacement header, building netdev-dpdk.c provokes several "sparse" warnings: /usr/include/dpdk/rte_memcpy.h:515:33: warning: incorrect type in argument 1 (different type sizes) /usr/include/dpdk/rte_memcpy.h:515:33:expected long long const [usertype] *__P /usr/include/dpdk/rte