mbox series

[0/1] Spelling code fixes*

Message ID 20220112072808.59713-1-jsoref@gmail.com (mailing list archive)
Headers show
Series Spelling code fixes* | expand

Message

Josh Soref Jan. 12, 2022, 7:28 a.m. UTC
This is a followup to https://github.com/DPDK/dpdk/commit/7be78d027918dbc846e502780faf94d5acdf5f75

The engine that identifies items runs here:
  https://github.com/jsoref/dpdk/actions/workflows/spelling.yml

I generally use Google Sheets to select automatic corrections for words.

I'll sometimes make additional fixes beyond what Google Sheets suggests.
Unfortunately, this means that while I may have sent one series of fixes,
there can easily be more, especially if I spend a bit more time reviewing
the word list.

The current list of words I am not correcting is here:
https://raw.githubusercontent.com/jsoref/dpdk/spell-check-with-spelling/.github/actions/spelling/expect.txt

Note that this isn't a complete list of misspelled words. As this
repository is written in C++, I'm experimenting with using a c++
orientated dictionary. Unfortunately, I know it includes quite a few
misspelled words of its own, which means those words if present in this
repository will be handled as false-negatives and not reported.

fixed spellings:

* accelerator
* acceptable
* account
* accounting
* acknowledge
* acknowledgement
* across
* action
* actually
* adapter
* adaptive
* added
* addition
* address
* addresses
* adds
* adjustment
* affinity
* aggregate
* aggregator
* alignment
* allocate
* allocated
* allocates
* allocation
* already
* annotation
* approx
* arithmetic
* array
* arrays
* arrive
* asserted
* assigned
* at most
* attributes
* authentic
* autogreen
* between
* bitstream
* boundaries
* build
* burst
* byte
* bytes
* cached
* calculate
* cannot
* capabilities
* capability
* celsius
* certificates
* chained
* chunk
* cipher
* cleanness
* coalescing
* command
* commit
* communicated
* compiler
* configuration
* connecting
* consequence
* contains
* contexts
* continuously
* convenience
* conversion
* counters
* create
* crypto
* detach
* discrimination
* empty
* endian
* erroneous
* expansion
* explicit
* feature
* fiber
* fields
* fragment
* free
* generator
* groupid
* grpmask
* interim
* interrupt
* intrinsics
* lapsed
* nonexistent
* octeontx
* offload
* offset
* overhead
* partner
* pending
* per port
* platform
* policies
* polynomials
* populate
* portctl
* prefetch
* properties
* qgroup
* registers
* replace
* resettable
* resolution
* retransmission
* retrieving
* revision
* session
* shaping
* ticks
* traditional
* traditionally
* traffic
* translation
* unexpectedly
* unsupported
* vectors
* verify
* virtchnl
* warnings
* written
* xstats

Josh Soref (1):
  fix spelling in code

 app/test-crypto-perf/cperf_test_vectors.h     |  4 +-
 app/test/test_acl.c                           |  6 +-
 app/test/test_link_bonding.c                  |  4 +-
 app/test/test_link_bonding_mode4.c            | 30 ++++----
 app/test/test_table.h                         |  2 +-
 doc/guides/nics/octeontx2.rst                 |  2 +-
 drivers/bus/dpaa/base/fman/netcfg_layer.c     |  4 +-
 drivers/bus/fslmc/fslmc_vfio.h                |  2 +-
 drivers/common/cnxk/roc_cpt.c                 | 10 +--
 drivers/common/cnxk/roc_cpt_priv.h            |  2 +-
 drivers/common/cnxk/roc_mbox.h                |  4 +-
 drivers/common/cnxk/roc_tim.c                 |  2 +-
 drivers/common/dpaax/caamflib/desc/ipsec.h    |  4 +-
 .../common/dpaax/caamflib/rta/operation_cmd.h |  6 +-
 drivers/common/mlx5/mlx5_devx_cmds.c          | 24 +++---
 drivers/common/mlx5/mlx5_devx_cmds.h          | 10 +--
 drivers/common/mlx5/mlx5_prm.h                |  4 +-
 drivers/common/octeontx2/otx2_mbox.h          |  4 +-
 .../sfc_efx/base/ef10_signed_image_layout.h   |  2 +-
 drivers/common/sfc_efx/base/efx_port.c        |  2 +-
 drivers/common/sfc_efx/base/efx_regs.h        |  2 +-
 drivers/common/sfc_efx/base/efx_types.h       |  2 +-
 drivers/compress/octeontx/otx_zip.h           |  2 +-
 drivers/compress/qat/dev/qat_comp_pmd_gen1.c  |  4 +-
 drivers/compress/qat/qat_comp.c               | 12 +--
 drivers/compress/qat/qat_comp_pmd.c           |  6 +-
 drivers/compress/qat/qat_comp_pmd.h           |  2 +-
 drivers/crypto/caam_jr/caam_jr.c              |  4 +-
 drivers/crypto/ccp/ccp_crypto.h               |  2 +-
 drivers/crypto/cnxk/cnxk_cryptodev_ops.h      |  4 +-
 drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c   | 30 ++++----
 drivers/crypto/dpaa_sec/dpaa_sec.c            |  2 +-
 drivers/crypto/qat/qat_sym_session.h          |  4 -
 drivers/crypto/virtio/virtio_cryptodev.c      |  6 +-
 drivers/crypto/virtio/virtqueue.c             |  2 +-
 drivers/crypto/virtio/virtqueue.h             |  2 +-
 drivers/dma/ioat/ioat_dmadev.c                |  2 +-
 drivers/dma/ioat/ioat_hw_defs.h               |  2 +-
 drivers/event/octeontx2/otx2_tim_evdev.c      |  4 +-
 drivers/net/ark/ark_ethdev.c                  |  4 +-
 drivers/net/ark/ark_rqp.c                     |  4 +-
 drivers/net/ark/ark_rqp.h                     |  4 +-
 drivers/net/bnx2x/bnx2x.c                     | 18 ++---
 drivers/net/bnx2x/bnx2x.h                     |  6 +-
 drivers/net/bnx2x/bnx2x_stats.c               |  8 +-
 drivers/net/bnx2x/bnx2x_stats.h               |  4 +-
 drivers/net/bnx2x/ecore_hsi.h                 | 38 +++++-----
 drivers/net/bnx2x/ecore_init.h                |  2 +-
 drivers/net/bnx2x/ecore_reg.h                 | 12 +--
 drivers/net/bnx2x/ecore_sp.c                  | 36 ++++-----
 drivers/net/bnx2x/ecore_sp.h                  |  2 +-
 drivers/net/bnx2x/elink.c                     | 32 ++++----
 drivers/net/bnx2x/elink.h                     |  2 +-
 drivers/net/bonding/eth_bond_8023ad_private.h |  2 +-
 drivers/net/cxgbe/base/adapter.h              |  2 +-
 drivers/net/cxgbe/base/t4_chip_type.h         |  2 +-
 drivers/net/cxgbe/base/t4_hw.c                |  8 +-
 drivers/net/dpaa/fmlib/fm_port_ext.h          |  4 +-
 drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h  |  2 +-
 drivers/net/dpaa2/dpaa2_flow.c                |  4 +-
 drivers/net/dpaa2/dpaa2_mux.c                 |  4 +-
 drivers/net/dpaa2/mc/dpdmux.c                 |  8 +-
 drivers/net/dpaa2/mc/dpni.c                   |  2 +-
 drivers/net/dpaa2/mc/fsl_dpdmux.h             |  4 +-
 drivers/net/e1000/base/e1000_82575.c          |  2 +-
 drivers/net/e1000/base/e1000_phy.c            |  2 +-
 drivers/net/enic/base/vnic_devcmd.h           |  2 +-
 drivers/net/enic/enic_flow.c                  | 48 ++++++------
 drivers/net/fm10k/base/fm10k_mbx.c            |  2 +-
 drivers/net/fm10k/base/fm10k_pf.c             |  2 +-
 drivers/net/fm10k/base/fm10k_vf.c             |  4 +-
 drivers/net/hinic/hinic_pmd_ethdev.c          |  4 +-
 drivers/net/hinic/hinic_pmd_flow.c            |  4 +-
 drivers/net/hinic/hinic_pmd_rx.c              | 30 ++++----
 drivers/net/hns3/hns3_dcb.c                   | 76 +++++++++----------
 drivers/net/hns3/hns3_dcb.h                   | 24 +++---
 drivers/net/hns3/hns3_ethdev.c                | 18 ++---
 drivers/net/hns3/hns3_fdir.c                  |  4 +-
 drivers/net/hns3/hns3_tm.c                    |  4 +-
 drivers/net/i40e/base/i40e_adminq_cmd.h       |  2 +-
 drivers/net/i40e/base/i40e_common.c           |  4 +-
 drivers/net/iavf/iavf_ethdev.c                |  8 +-
 drivers/net/iavf/iavf_hash.c                  |  6 +-
 drivers/net/iavf/iavf_ipsec_crypto.c          |  4 +-
 drivers/net/iavf/iavf_ipsec_crypto.h          |  4 +-
 drivers/net/ice/base/ice_adminq_cmd.h         |  2 +-
 drivers/net/ice/ice_switch_filter.c           | 26 +++----
 drivers/net/igc/base/igc_defines.h            |  2 +-
 drivers/net/ipn3ke/ipn3ke_tm.c                |  4 +-
 drivers/net/ixgbe/base/ixgbe_82598.c          |  2 +-
 drivers/net/ixgbe/ixgbe_bypass.c              | 18 ++---
 drivers/net/ixgbe/ixgbe_bypass_defines.h      |  2 +-
 drivers/net/mlx5/mlx5.c                       |  4 +-
 drivers/net/mlx5/mlx5.h                       |  4 +-
 drivers/net/mlx5/mlx5_flow.c                  | 14 ++--
 drivers/net/mlx5/mlx5_flow.h                  |  2 +-
 drivers/net/mlx5/mlx5_flow_aso.c              |  8 +-
 drivers/net/mlx5/mlx5_flow_dv.c               | 44 +++++------
 drivers/net/mlx5/mlx5_flow_meter.c            |  6 +-
 drivers/net/mlx5/mlx5_rx.c                    |  2 +-
 drivers/net/mlx5/mlx5_tx.c                    |  2 +-
 drivers/net/ngbe/ngbe_ethdev.c                |  8 +-
 drivers/net/pfe/pfe_ethdev.c                  |  4 +-
 drivers/net/qede/base/ecore_chain.h           |  4 +-
 drivers/net/qede/base/ecore_cxt.c             |  8 +-
 drivers/net/qede/base/ecore_dev.c             |  2 +-
 drivers/net/qede/base/ecore_dev_api.h         |  2 +-
 drivers/net/qede/base/ecore_hsi_eth.h         |  4 +-
 drivers/net/qede/base/ecore_hw_defs.h         |  4 +-
 drivers/net/qede/base/ecore_init_fw_funcs.c   |  6 +-
 drivers/net/qede/base/ecore_init_fw_funcs.h   |  4 +-
 drivers/net/qede/base/ecore_int.c             |  2 +-
 drivers/net/qede/base/ecore_iov_api.h         |  2 +-
 drivers/net/qede/base/ecore_l2.c              |  2 +-
 drivers/net/qede/base/ecore_mcp.h             |  2 +-
 drivers/net/qede/base/ecore_mcp_api.h         |  2 +-
 drivers/net/qede/base/ecore_spq.c             |  2 +-
 drivers/net/qede/base/ecore_spq.h             |  2 +-
 drivers/net/qede/base/ecore_sriov.c           |  4 +-
 drivers/net/qede/base/ecore_sriov.h           |  4 +-
 drivers/net/qede/base/ecore_vf.c              |  2 +-
 drivers/net/qede/base/ecore_vfpf_if.h         |  2 +-
 drivers/net/qede/base/mcp_public.h            |  4 +-
 drivers/net/qede/qede_debug.c                 | 10 +--
 drivers/net/sfc/sfc_mae.c                     | 50 ++++++------
 drivers/net/sfc/sfc_tso.h                     | 10 +--
 drivers/net/txgbe/txgbe_ethdev.c              |  8 +-
 drivers/net/virtio/virtio_ethdev.c            |  2 +-
 drivers/net/virtio/virtqueue.c                |  2 +-
 drivers/net/virtio/virtqueue.h                |  2 +-
 drivers/net/vmxnet3/base/upt1_defs.h          |  2 +-
 drivers/raw/ifpga/base/ifpga_defines.h        |  6 +-
 drivers/raw/ifpga/base/ifpga_feature_dev.c    |  2 +-
 drivers/raw/ifpga/base/ifpga_fme_pr.c         |  2 +-
 drivers/raw/ifpga/base/opae_hw_api.h          |  4 +-
 drivers/raw/ioat/ioat_rawdev.c                |  2 +-
 drivers/raw/ioat/ioat_spec.h                  |  2 +-
 drivers/regex/mlx5/mlx5_regex_fastpath.c      |  2 +-
 drivers/vdpa/mlx5/mlx5_vdpa.h                 |  2 +-
 drivers/vdpa/mlx5/mlx5_vdpa_virtq.c           |  4 +-
 examples/ipsec-secgw/ipsec_process.c          |  2 +-
 examples/vhost/virtio_net.c                   | 50 ++++++------
 lib/bpf/bpf_validate.c                        | 10 +--
 lib/cryptodev/rte_cryptodev.h                 |  2 +-
 lib/eal/common/eal_common_trace_ctf.c         |  8 +-
 lib/fib/trie_avx512.c                         |  2 +-
 lib/graph/graph_populate.c                    |  4 +-
 lib/graph/graph_stats.c                       |  4 +-
 lib/hash/rte_crc_arm64.h                      |  2 +-
 lib/hash/rte_thash.c                          |  2 +-
 lib/ip_frag/ip_frag_internal.c                |  2 +-
 lib/ipsec/ipsec_sad.c                         | 10 +--
 lib/vhost/vhost_user.h                        |  2 +-
 lib/vhost/virtio_net.c                        | 10 +--
 154 files changed, 563 insertions(+), 567 deletions(-)

Comments

Thomas Monjalon Jan. 12, 2022, 11:49 a.m. UTC | #1
12/01/2022 08:28, Josh Soref:
> fixed spellings:
> 
> * accelerator
> * acceptable
> * account
> * accounting
> * acknowledge
> * acknowledgement
> * across
> * action
> * actually
> * adapter
> * adaptive
> * added
> * addition
> * address
> * addresses
> * adds
> * adjustment
> * affinity
...

It would be more convenient to review the changes, example:
	* acelerator -> accelerator
And it would be nice to make some categories,
like shorter and longer replacement,
because shorter is almost always a good fix,
but a longer replacement may not be required and require more attention.
Josh Soref Jan. 12, 2022, 12:48 p.m. UTC | #2
On Wed, Jan 12, 2022, 6:50 AM Thomas Monjalon <thomas@monjalon.net> wrote:

> 12/01/2022 08:28, Josh Soref:
> > fixed spellings:
> >
> > * accelerator
> > * acceptable
> > * account
> > * accounting
> > * acknowledge
> > * acknowledgement
> > * across
> > * action
> > * actually
> > * adapter
> > * adaptive
> > * added
> > * addition
> > * address
> > * addresses
> > * adds
> > * adjustment
> > * affinity
> ...
>
> It would be more convenient to review the changes, example:
>         * acelerator -> accelerator
> And it would be nice to make some categories,
> like shorter and longer replacement,
> because shorter is almost always a good fix,
> but a longer replacement may not be required and require more attention.
>

The original task actually has this. But I generally fold all of the
replacements together, so a given commit for `addresses` at this point may
be a mix of changes from `addresse`, `adddresses`, and `Addressses`