[dpdk-stable] [PATCH 2/4] compress/mlx5: fix constant size in QP creation
Michael Baum
michaelba at nvidia.com
Mon Jun 28 16:21:04 CEST 2021
External email: Use caution opening links or attachments
> -----Original Message-----
> From: Thomas Monjalon <thomas at monjalon.net>
> Sent: Wednesday, June 23, 2021 9:48 AM
> To: Michael Baum <michaelba at nvidia.com>
> Cc: dev at dpdk.org; stable at dpdk.org; Matan Azrad <matan at nvidia.com>;
> Raslan Darawsheh <rasland at nvidia.com>; Slava Ovsiienko
> <viacheslavo at nvidia.com>
> Subject: Re: [dpdk-stable] [PATCH 2/4] compress/mlx5: fix constant size in
> QP creation
>
> External email: Use caution opening links or attachments
>
>
> 01/06/2021 09:11, Michael Baum:
> > The mlx5_compress_qp_setup function makes shifting to the numeric
> > constant 1, then sends it as a parameter to rte_calloc function.
> >
> > The rte_calloc function expects to get size_t (64 bits, unsigned) and
>
> No on 32-bit systems, size_t is 32 bits.
Thanks for the comment, I'll send v2.
> > instead gets a 32-bit variable, because the numeric constant size is a
> > 32-bit.
>
> Most of the patches of this series say "constant" where it is a variable.
>
> > In case the shift is greater than 32 the variable will lose its value
> > even though the function can get 64-bit argument.
> >
> > Change the size of the numeric constant 1 to 64-bit.
> [...]
> > - opaq_buf = rte_calloc(__func__, 1u << log_ops_n,
> > + opaq_buf = rte_calloc(__func__, RTE_BIT64(log_ops_n),
>
>
More information about the stable
mailing list