mbox series

[v2,00/21] use C11 alignof

Message ID 1707849292-19519-1-git-send-email-roretzla@linux.microsoft.com (mailing list archive)
Headers
Series use C11 alignof |

Message

Tyler Retzlaff Feb. 13, 2024, 6:34 p.m. UTC 
  Replace use of __alignof__(T) and __alignof__(e) with C11 alignof(T)
and alignof(typeof(e)) respectively to improve portability of the code
between toolchains.

v2:
    * expand series to replace use in entire source tree, now
      includes app, drivers, examples in addition to lib.

Tyler Retzlaff (21):
  ring: use C11 alignof
  mbuf: use C11 alignof
  ethdev: use C11 alignof
  eventdev: use C11 alignof
  stack: use C11 alignof
  node: use C11 alignof
  pdcp: use C11 alignof
  reorder: use C11 alignof
  security: use C11 alignof
  test: use C11 alignof
  examples: use C11 alignof
  app/eventdev: use C11 alignof
  drivers/bus: use C11 alignof
  event/sw: use C11 alignof
  net/bnxt: use C11 alignof
  net/iavf: use C11 alignof
  net/nfp: use C11 alignof
  net/sfc: use C11 alignof
  net/virtio: use C11 alignof
  net/vmxnet3: use C11 alignof
  devtools: forbid direct use of compiler alignof and typeof

 app/test-eventdev/test_order_common.c |  6 ++++--
 app/test/test_distributor.c           |  3 ++-
 app/test/test_graph.c                 |  3 ++-
 app/test/test_mbuf.c                  |  9 +++++----
 app/test/test_ring_stress_impl.h      |  6 ++++--
 devtools/checkpatches.sh              | 16 ++++++++++++++++
 drivers/bus/dpaa/dpaa_bus.c           |  3 ++-
 drivers/bus/fslmc/fslmc_bus.c         |  3 ++-
 drivers/event/sw/sw_evdev_selftest.c  |  3 ++-
 drivers/net/bnxt/bnxt_ethdev.c        |  3 ++-
 drivers/net/iavf/iavf_ethdev.c        |  3 ++-
 drivers/net/iavf/iavf_ipsec_crypto.c  |  4 +++-
 drivers/net/iavf/iavf_testpmd.c       |  3 ++-
 drivers/net/nfp/nfp_ipsec.c           |  4 +++-
 drivers/net/sfc/sfc_dp.c              |  5 +++--
 drivers/net/virtio/virtio_rxtx.c      |  7 ++++---
 drivers/net/vmxnet3/vmxnet3_ethdev.c  |  3 ++-
 examples/bbdev_app/main.c             |  3 ++-
 examples/rxtx_callbacks/main.c        |  3 ++-
 lib/ethdev/ethdev_driver.c            |  3 ++-
 lib/ethdev/rte_flow.c                 |  3 ++-
 lib/eventdev/rte_eventdev.c           |  3 ++-
 lib/mbuf/rte_mbuf_dyn.c               |  3 ++-
 lib/node/node_private.h               |  4 +++-
 lib/pdcp/rte_pdcp.c                   |  4 +++-
 lib/reorder/rte_reorder.c             |  3 ++-
 lib/ring/rte_ring.c                   |  3 ++-
 lib/security/rte_security.c           |  5 +++--
 lib/stack/rte_stack.c                 |  3 ++-
 29 files changed, 87 insertions(+), 37 deletions(-)

Comments

David Marchand Feb. 14, 2024, 12:19 p.m. UTC | #1 
On Tue, Feb 13, 2024 at 7:35 PM Tyler Retzlaff
<roretzla@linux.microsoft.com> wrote:
>
> Replace use of __alignof__(T) and __alignof__(e) with C11 alignof(T)
> and alignof(typeof(e)) respectively to improve portability of the code
> between toolchains.
>
> v2:
>     * expand series to replace use in entire source tree, now
>       includes app, drivers, examples in addition to lib.
>
> Tyler Retzlaff (21):
>   ring: use C11 alignof
>   mbuf: use C11 alignof
>   ethdev: use C11 alignof
>   eventdev: use C11 alignof
>   stack: use C11 alignof
>   node: use C11 alignof
>   pdcp: use C11 alignof
>   reorder: use C11 alignof
>   security: use C11 alignof
>   test: use C11 alignof
>   examples: use C11 alignof
>   app/eventdev: use C11 alignof
>   drivers/bus: use C11 alignof
>   event/sw: use C11 alignof
>   net/bnxt: use C11 alignof
>   net/iavf: use C11 alignof
>   net/nfp: use C11 alignof
>   net/sfc: use C11 alignof
>   net/virtio: use C11 alignof
>   net/vmxnet3: use C11 alignof
>   devtools: forbid direct use of compiler alignof and typeof
>
>  app/test-eventdev/test_order_common.c |  6 ++++--
>  app/test/test_distributor.c           |  3 ++-
>  app/test/test_graph.c                 |  3 ++-
>  app/test/test_mbuf.c                  |  9 +++++----
>  app/test/test_ring_stress_impl.h      |  6 ++++--
>  devtools/checkpatches.sh              | 16 ++++++++++++++++
>  drivers/bus/dpaa/dpaa_bus.c           |  3 ++-
>  drivers/bus/fslmc/fslmc_bus.c         |  3 ++-
>  drivers/event/sw/sw_evdev_selftest.c  |  3 ++-
>  drivers/net/bnxt/bnxt_ethdev.c        |  3 ++-
>  drivers/net/iavf/iavf_ethdev.c        |  3 ++-
>  drivers/net/iavf/iavf_ipsec_crypto.c  |  4 +++-
>  drivers/net/iavf/iavf_testpmd.c       |  3 ++-
>  drivers/net/nfp/nfp_ipsec.c           |  4 +++-
>  drivers/net/sfc/sfc_dp.c              |  5 +++--
>  drivers/net/virtio/virtio_rxtx.c      |  7 ++++---
>  drivers/net/vmxnet3/vmxnet3_ethdev.c  |  3 ++-
>  examples/bbdev_app/main.c             |  3 ++-
>  examples/rxtx_callbacks/main.c        |  3 ++-
>  lib/ethdev/ethdev_driver.c            |  3 ++-
>  lib/ethdev/rte_flow.c                 |  3 ++-
>  lib/eventdev/rte_eventdev.c           |  3 ++-
>  lib/mbuf/rte_mbuf_dyn.c               |  3 ++-
>  lib/node/node_private.h               |  4 +++-
>  lib/pdcp/rte_pdcp.c                   |  4 +++-
>  lib/reorder/rte_reorder.c             |  3 ++-
>  lib/ring/rte_ring.c                   |  3 ++-
>  lib/security/rte_security.c           |  5 +++--
>  lib/stack/rte_stack.c                 |  3 ++-
>  29 files changed, 87 insertions(+), 37 deletions(-)

Thanks Tyler.
Series applied.