From patchwork Fri Aug 18 17:45:24 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 75 Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id BAB774309A; Fri, 18 Aug 2023 19:46:46 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 918C440ED9; Fri, 18 Aug 2023 19:46:46 +0200 (CEST) Received: from mail-pf1-f177.google.com (mail-pf1-f177.google.com [209.85.210.177]) by mails.dpdk.org (Postfix) with ESMTP id C799640395 for ; Fri, 18 Aug 2023 19:46:44 +0200 (CEST) Received: by mail-pf1-f177.google.com with SMTP id d2e1a72fcca58-6887c3aac15so1095162b3a.2 for ; Fri, 18 Aug 2023 10:46:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20221208.gappssmtp.com; s=20221208; t=1692380803; x=1692985603; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=tIdByWZgQU0CS5DN88ZeQhsWtoYR8zwy2Xwaz6kuNm4=; b=DMB2xElflPWEpTXGcmAA6B80poptxzFE7Vu7GizVq/qQm08ulLIPQ+/xjytBiU6o1A YiLazgkn8k2wh7eFTKMl0OP/0Ze6oKIxbTGea9IZ/lXPTRsW/DDUEOyaXOZvK+mJDLH6 WRSLbPgFYcfViR5F8ne94a4vjxHdsK8/rPEjYwQNVSOlXYd1BAe9KnMd0VgqBTmXfHpo KieZPPgRGVb7Xns2XypNdYk+aXT1dJvE9pEHCz4mbredBTVKPk7nE3DKqxmdlQREDOgy Vz4ABny6un7GrA5jtnSrhrGSorbteNlBj3tbUwWw6SUQMuPQKFGw1TWM/LIeMUAU9tl1 DupQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692380803; x=1692985603; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=tIdByWZgQU0CS5DN88ZeQhsWtoYR8zwy2Xwaz6kuNm4=; b=lsJHNhHVSbABEFvk/K4gB4EnQrqGkJUlQykryoIWuADX2GRRFNspOJAtM6lnLuL18Q jo4cWpK0LMi7rpSNqYCKYsZAr+3v0xxmbwvHoZ1eLIeC1vekr7NpTY4c2HNk756D3lBO LCYCPJjlEqljixTP9QcoSmqoWS7fSMB7vN/SLYsk2TS29PhTmJG3puP98zgcG2rZ8Ja7 hv3MVlb8epHDWGmsqTC0+ZlgthQiN8RoKCg5iloQaJkBILpqyo/IXHYFIxrNW6qwOz7j ShDsqiLygE9I62DLniyAw2UApXLSUT0Az8Rf2pIljoNwE1O9Xv0Lon33A7WhKqDWrGWu dM/g== X-Gm-Message-State: AOJu0YyUhAZPG49zAbhhXfXRhYb9RDTHbEtCiC4wExLUN00Ajm0A7hE8 vTkVCs44AUqwRHRXz95sd1FtJ3rtmOIzJ3ubq0CKyQ== X-Google-Smtp-Source: AGHT+IEwzpJjLkw2PeTHGS8r1+nKJZdo+mD9It6rNBTMZsmFtUsC3JKTlcgtZe6LknQBOn2ld5vgaA== X-Received: by 2002:a05:6a00:23d5:b0:688:71b9:89f3 with SMTP id g21-20020a056a0023d500b0068871b989f3mr3343888pfc.24.1692380803380; Fri, 18 Aug 2023 10:46:43 -0700 (PDT) Received: from hermes.local (204-195-127-207.wavecable.com. [204.195.127.207]) by smtp.gmail.com with ESMTPSA id u20-20020aa78394000000b0066a2e8431a0sm1804610pfm.183.2023.08.18.10.46.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 18 Aug 2023 10:46:43 -0700 (PDT) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger Subject: [PATCH v2 00/13] Replace us of term abort Date: Fri, 18 Aug 2023 10:45:24 -0700 Message-Id: <20230818174537.290222-1-stephen@networkplumber.org> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20230818025249.33994-1-stephen@networkplumber.org> References: <20230818025249.33994-1-stephen@networkplumber.org> MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org The term 'abort' is on the inclusive namin Tier 1 word list as replace when possible. It is possible to do this across DPDK except the few places that directly call the lib C abort() function. v2 - drop changes to sfc since requires changes to base driver - cleanup some checkpatch and missing spots Rationale (from https://inclusivenaming.org/word-lists/tier-1/abort/) The term “abort” frequently appeared in Inclusive Language Initiative surveys and standards reviews. Multiple organizations felt that usage of the word posed an issue worth addressing in their individual companies and projects. Given this widespread interest, the INI has decided to offer its own guidance. INI recommends replacing “abort” wherever possible. In accordance with the INI’s language framework, the term does not necessarily constitute a first-order concern. However, because it is such a charged term outside of computing, “abort” fails to provide a clear description of the action being taken, and serves primarily to distract. There are numerous other words in the English language that can serve the same purpose in computing without invoking the emotionally charged cultural context of “abort.” Although the INI debated whether the word “abort” itself or the procedure commonly associated with it (“abortion”) caused the aforementioned distractions, the etymology of the word has a direct and unambiguous link to the termination of a pregnancy. Alternative uses of the word “abort” are in use today, such as in rocketry. However, the INI concluded that the term itself was insufficiently distanced from its original meaning for those alternative definitions to be its primary association. All this being said, the INI does not advocate for a blanket replacement of the term. “Abort” appears in many standards organization documents, and is deeply embedded in some operating systems. As such, the INI acknowledges that the term may need to be retained in certain contexts to remain in compliance with those standards, or to preserve accurate documentation for bedrock functions and processes that are too fundamental to be changed. Some organizations have noted that using “abort” in contexts other than the medical or political serves to de-stigmatize the term, thereby promoting reproductive rights and bodily autonomy. Conversely, discouraging the term could be interpreted as accepting a framing that denies a pregnant person’s right to control their body. The judgment of the INI is that the term causes discomfort or offense without providing a necessary degree of technical clarity, and therefore it should be avoided. At some future point, because language changes over time, “abort” may become a less contentious term; at that juncture, the appropriateness of the term may be revisited. Stephen Hemminger (13): jobstats: change jobstats_abort to jobstats_cancel all: remove use of word abort pipeline: remove use of term abort net/vmxnet3: replace abort() with rte_panic() event/dlb2: remove word abort in comments net/vdev_netvsc: replace use of term abort net/netvsc: replace abort with cancel net/ionic: remove word aborting net/bnx2x: replace abort with cancel net/mlx5: replace abort with cancel net/softnic: replace abort with cancel net/mlx4: remove word abort net/axgbe: replace word abort .../comp_perf_test_cyclecount.c | 2 +- .../comp_perf_test_throughput.c | 2 +- .../comp_perf_test_verify.c | 2 +- app/test/test_dmadev.c | 2 +- doc/guides/prog_guide/bpf_lib.rst | 2 +- doc/guides/prog_guide/traffic_management.rst | 2 +- drivers/common/mlx5/mlx5_common_mr.c | 6 ++-- drivers/common/mlx5/mlx5_prm.h | 4 +-- drivers/dma/hisilicon/hisi_dmadev.c | 4 +-- drivers/dma/hisilicon/hisi_dmadev.h | 2 +- drivers/event/dlb2/dlb2.c | 2 +- drivers/event/dlb2/pf/base/dlb2_resource.c | 2 +- drivers/net/af_xdp/rte_eth_af_xdp.c | 2 +- drivers/net/axgbe/axgbe_common.h | 4 +-- drivers/net/axgbe/axgbe_ethdev.h | 2 +- drivers/net/axgbe/axgbe_i2c.c | 26 ++++++++--------- drivers/net/bnx2x/bnx2x.c | 18 ++++++------ drivers/net/bnx2x/bnx2x_vfpf.c | 4 +-- drivers/net/bnx2x/ecore_hsi.h | 6 ++-- drivers/net/bnx2x/ecore_reg.h | 20 ++++++------- drivers/net/ionic/ionic_dev_pci.c | 6 ++-- drivers/net/ionic/ionic_ethdev.c | 18 ++++++------ drivers/net/ionic/ionic_lif.c | 2 +- drivers/net/mlx4/mlx4_mr.c | 6 ++-- drivers/net/mlx5/hws/mlx5dr_rule.c | 8 ++--- drivers/net/mlx5/hws/mlx5dr_send.c | 2 +- drivers/net/mlx5/hws/mlx5dr_send.h | 2 +- drivers/net/mlx5/linux/mlx5_os.c | 4 +-- drivers/net/mlx5/mlx5_rx.c | 2 +- drivers/net/netvsc/rndis.h | 2 +- drivers/net/softnic/rte_eth_softnic_cli.c | 12 ++++---- drivers/net/vdev_netvsc/vdev_netvsc.c | 4 +-- drivers/net/vmxnet3/vmxnet3_rxtx.c | 6 ++-- examples/ipsec-secgw/rt.c | 4 +-- examples/l2fwd-jobstats/main.c | 2 +- examples/pipeline/cli.c | 20 ++++++------- lib/dmadev/rte_dmadev.h | 6 ++-- .../include/generic/rte_power_intrinsics.h | 10 +++---- lib/eal/include/generic/rte_rwlock.h | 4 +-- lib/eal/include/generic/rte_spinlock.h | 8 ++--- lib/eal/include/rte_seqlock.h | 2 +- lib/eal/x86/include/rte_rtm.h | 16 +++++----- lib/eal/x86/include/rte_spinlock.h | 12 ++++---- lib/eal/x86/rte_power_intrinsics.c | 8 ++--- lib/ethdev/rte_tm.h | 2 +- lib/hash/rte_cuckoo_hash.h | 2 +- lib/jobstats/rte_jobstats.c | 2 +- lib/jobstats/rte_jobstats.h | 9 +++++- lib/jobstats/version.map | 2 +- lib/pipeline/rte_swx_ctl.c | 29 ++++++++++--------- lib/pipeline/rte_swx_ctl.h | 8 ++--- lib/pipeline/version.map | 2 +- lib/ring/rte_ring_peek_elem_pvt.h | 4 +-- 53 files changed, 174 insertions(+), 166 deletions(-)