[dpdk-dev] [PATCH v3 3/7] pci: build on Windows
Dmitry Kozlyuk
dmitry.kozliuk at gmail.com
Fri May 8 01:27:41 CEST 2020
On 2020-05-07 15:16 GMT+0300 talshn at mellanox.com wrote:
> From: Tal Shnaiderman <talshn at mellanox.com>
>
> Added off_t in Windows header file as a supported
> type since it is needed by PCI.
>
> Signed-off-by: Tal Shnaiderman <talshn at mellanox.com>
> ---
> lib/librte_eal/windows/include/rte_os.h | 1 +
> lib/meson.build | 5 ++++-
> 2 files changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/lib/librte_eal/windows/include/rte_os.h b/lib/librte_eal/windows/include/rte_os.h
> index 62805a307..1c433b976 100644
> --- a/lib/librte_eal/windows/include/rte_os.h
> +++ b/lib/librte_eal/windows/include/rte_os.h
> @@ -48,6 +48,7 @@ extern "C" {
>
> /* as in <windows.h> */
> typedef long long ssize_t;
> +typedef long off_t;
Windows actually has off_t in <sys/types.h>, where it is by POSIX. Shouldn't
rte_pci.h include <sys/types.h> instead? IMO, shims should be kept at minimum.
--
Dmitry Kozlyuk
More information about the dev
mailing list