Re: [PATCH v2 15/21] net/bnxt: use C11 alignof

[Ajit Khaparde]

On Tue, Feb 13, 2024 at 10:34 AM Tyler Retzlaff
<roret...@linux.microsoft.com> wrote:
>
> Replace use of __alignof__(T) with C11 alignof(T) to improve portability
> between toolchains.
>
> Signed-off-by: Tyler Retzlaff <roret...@linux.microsoft.com>
Acked-by: Ajit Khaparde <ajit.khapa...@broadcom.com>

> ---
>  drivers/net/bnxt/bnxt_ethdev.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
> index acf7e6e..45d840d 100644
> --- a/drivers/net/bnxt/bnxt_ethdev.c
> +++ b/drivers/net/bnxt/bnxt_ethdev.c
> @@ -4,6 +4,7 @@
>   */
>
>  #include <inttypes.h>
> +#include <stdalign.h>
>  #include <stdbool.h>
>
>  #include <dev_driver.h>
> @@ -5920,7 +5921,7 @@ static int bnxt_drv_init(struct rte_eth_dev *eth_dev)
>                 static const struct rte_mbuf_dynfield 
> bnxt_cfa_code_dynfield_desc = {
>                         .name = RTE_PMD_BNXT_CFA_CODE_DYNFIELD_NAME,
>                         .size = sizeof(bnxt_cfa_code_dynfield_t),
> -                       .align = __alignof__(bnxt_cfa_code_dynfield_t),
> +                       .align = alignof(bnxt_cfa_code_dynfield_t),
>                 };
>                 bnxt_cfa_code_dynfield_offset =
>                         
> rte_mbuf_dynfield_register(&bnxt_cfa_code_dynfield_desc);
> --
> 1.8.3.1
>

smime.p7s
Description: S/MIME Cryptographic Signature