Re: [dpdk-dev] [PATCH v3 00/15] Build fixes for musl libc

[David Marchand <david.marchand@redhat.com>]

On Sun, Mar 31, 2019 at 12:57 AM Thomas Monjalon <thomas@monjalon.net>
wrote:

> 13/03/2019 18:06, Natanael Copa:
> > A set of patches to fix build with musl libc. I also did a few cleanups
> wrt
> > macros and fixed a few scary compiler warnings while at it.
> >
> > Please note that those are only compile tested on x86_64 with musl libc.
> >
> > v2 has some code style fixes reported by checkpatch
> > v3 is a full reroll of the entire patch set to enable automatted tests
> >    and logic for patchwork patchset.
> >
> > Natanael Copa (15):
> >   app/testpmd: replace uint with unsigned int
> >   net/cxgbe: replace uint with unsigned int
> >   bus/pci: add fallback for out[lwb]_p for non GNU libc
> >   bus/pci: factor out various ifdefs in pci_uio_ioport_{read,write}
> >   bus/fslmc: fix compile error with musl libc
> >   bus/fslmc: remove unused include of error.h
> >   net/nfp: build fix for musl libc
> >   app/test: include fcntl.h due to use of O_RDONLY
> >   app/test: fix setting of -D_GNU_SOURCE with meson
> >   bus/dpaa: use warn(3) instead of error(3) to improve portability
> >   bus/dpaa: fix warning: "__WORDSIZE" is not defined, evaluates to 0
> >   crypto/dpaa2_sec: build fix for musl libc
> >   crypto/dpaa2_sec: simplify pr_{debug,err,warn} macros
> >   net/netvsc: fix compile warning for fcntl.h include
> >   eal/linux: simplify debug message in sigbus_handler
>
> Thanks for the cleanup.
>
> Some patches may deserve more reviews.
> If you send a new version, please use --to-cmd devtools/get-maintainer.sh
> so the maintainers will be Cc'ed.
>
> All these patches are fixes and may be interesting to backport them
> in the latest LTS or other stable releases.
> In order to help with backporting, please fill "Fixes:" tag
> and add Cc: stable when relevant.
> The exact recommendation is described in this doc chapter:
> http://doc.dpdk.org/guides/contributing/patches.html#commit-messages-body
>
> In order to help, I already fixed 7 commit messages and merged them.
> The remaining patches, requiring a v4, are:
>   app/testpmd: replace uint with unsigned int
>   net/cxgbe: replace uint with unsigned int
>   bus/pci: add fallback for out[lwb]_p for non GNU libc
>   bus/pci: factor out various ifdefs in pci_uio_ioport_{read,write}
>   bus/dpaa: use warn(3) instead of error(3) to improve portability
>   bus/dpaa: fix warning: "__WORDSIZE" is not defined, evaluates to 0
>   crypto/dpaa2_sec: build fix for musl libc
>   crypto/dpaa2_sec: simplify pr_{debug,err,warn} macros
>


If you intend to send a v4, I noticed a few more "uint" in the tree, I
suppose this is worth fixing them:
$ git grep -w uint
app/test-pmd/config.c:                   uint hash_key_len)
examples/tep_termination/vxlan.c:       uint vport_id;



-- 
David Marchand