Re: [dpdk-dev] [PATCH v2 0/4] Hyper-V Netvsc PMD

[Raslan Darawsheh <rasland@mellanox.com>]

I'm facing this on gcc version 5.4.0 on 64bit.

Kindest regards,
Raslan Darawsheh

-----Original Message-----
From: Stephen Hemminger [mailto:stephen@networkplumber.org] 
Sent: Tuesday, April 17, 2018 7:22 PM
To: Raslan Darawsheh <rasland@mellanox.com>
Cc: dev@dpdk.org; Stephen Hemminger <sthemmin@microsoft.com>; Thomas Monjalon <thomasm@mellanox.com>
Subject: Re: [dpdk-dev] [PATCH v2 0/4] Hyper-V Netvsc PMD

On Tue, 17 Apr 2018 09:02:45 +0000
Raslan Darawsheh <rasland@mellanox.com> wrote:

> Hi Stephan,
> 
> I've just tried to apply your patches on top of current master branch and I have the following two issues:
> 1- compilation is failing with the following error when compiling with RTE_TARGET=x86_64-native-linuxapp-gcc:
> /root/dpdk/x86_64-native-linuxapp-gcc/include/rte_common.h:122:30: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
>  #define RTE_PTR_ADD(ptr, x) ((void*)((uintptr_t)(ptr) + (x)))
>                               ^
> /root/dpdk/buildtools/pmdinfogen/pmdinfogen.c:99:9: note: in expansion of macro 'RTE_PTR_ADD'
>   return RTE_PTR_ADD(info->hdr,
>          ^
> 2- compilation is also failing with the following errors when compiling with RTE_TARGET=i686-native-linuxapp-gcc
> CC linux/vmbus_uio.o
> /root/dpdk/drivers/bus/vmbus/linux/vmbus_bus.c: In function 'rte_vmbus_map_device':
> /root/dpdk/drivers/bus/vmbus/linux/vmbus_bus.c:194:39: error: passing argument 2 of 'eal_parse_sysfs_value' from incompatible pointer type [-Werror=incompatible-pointer-types]
>    if (eal_parse_sysfs_value(filename, &res->len) < 0) {
>                                        ^
> In file included from /root/dpdk/drivers/bus/vmbus/linux/vmbus_bus.c:22:0:
> /root/dpdk/lib/librte_eal/common/eal_filesystem.h:123:5: note: expected 'long unsigned int *' but argument is of type 'uint64_t * {aka long long unsigned int *}'
>  int eal_parse_sysfs_value(const char *filename, unsigned long *val);
> 
> 
> I noticed one additional new dependency which is uuid-dev  and I think you might be missing the documentation of that.
> 
> Kindest regards,
> Raslan Darawsheh

I rebased against dpdk.org master yesterday.

What version of Gcc? I don't see this on Gcc 7.3.0 and build on 64 bit.

Will update the documentation and fix a couple of typos.