[dpdk-dev] [PATCH v3 01/15] app/testpmd: replace uint with unsigned int

[Iremonger, Bernard]

Hi Nataneal,

> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Natanael Copa
> Sent: Wednesday, March 13, 2019 5:07 PM
> To: dev at dpdk.org
> Cc: Natanael Copa <ncopa at alpinelinux.org>
> Subject: [dpdk-dev] [PATCH v3 01/15] app/testpmd: replace uint with
> unsigned int

This should have the fix keyword in the commit line and a fixes line should be added.
Suggest something like the following:

" app/testpmd: fix build error with musl libc"
 
> Improve portability by avoid use non-standard 'uint'.
> 
> This fixes following build error when building with musl libc:
> 
> In file included from ../app/test-pmd/cmdline.c:75:
> ../app/test-pmd/testpmd.h:809:29: error: unknown type name 'uint'; did you
> mean 'int'?
>           uint8_t *hash_key, uint hash_key_len);
>                              ^~~~
>                              int
> 

./dpdk /devtools/check-git-log.sh -1
Line too long:
        ../app/test-pmd/testpmd.h:809:29: error: unknown type name 'uint'; did you mean 'int'?
Wrong tag:
        This fixes following build error when building with musl libc:

Need to wrap the compile error message to pass the check-git-log.sh checks.

> Signed-off-by: Natanael Copa <ncopa at alpinelinux.org>
> ---
>  app/test-pmd/testpmd.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h index
> fa4887853..84ef3fc30 100644
> --- a/app/test-pmd/testpmd.h
> +++ b/app/test-pmd/testpmd.h
> @@ -806,7 +806,7 @@ int set_vf_rate_limit(portid_t port_id, uint16_t vf,
> uint16_t rate,
> 
>  void port_rss_hash_conf_show(portid_t port_id, int show_rss_key);  void
> port_rss_hash_key_update(portid_t port_id, char rss_type[],
> -			      uint8_t *hash_key, uint hash_key_len);

uint should be replaced by uint8_t instead of unsigned int.
The  rss_key_len is a uint8_t type.

> +			      uint8_t *hash_key, unsigned int hash_key_len);
>  int rx_queue_id_is_invalid(queueid_t rxq_id);  int
> tx_queue_id_is_invalid(queueid_t txq_id);  void setup_gro(const char
> *onoff, portid_t port_id);
> --
> 2.21.0

Regards,

Bernard.