From patchwork Mon Feb 5 17:43:29 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136388 X-Patchwork-Delegate: thomas@monjalon.net 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 0B0CF43A3D; Mon, 5 Feb 2024 19:03:46 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id E384F40DDA; Mon, 5 Feb 2024 19:03:41 +0100 (CET) Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) by mails.dpdk.org (Postfix) with ESMTP id E906640269 for ; Mon, 5 Feb 2024 19:03:39 +0100 (CET) Received: by mail-pl1-f173.google.com with SMTP id d9443c01a7336-1d98fc5ebceso8421225ad.1 for ; Mon, 05 Feb 2024 10:03:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156219; x=1707761019; darn=dpdk.org; 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=0UU5x8eOSKj/KaWrqX6oh+A5NqhkELkaQHAqy20ER9Q=; b=JRzLTjWSe9OXwqDsBRotZOFaztHXbbHIiThIgDAYB0G+EKX0R1cUFNYWeAspMP3K1u 7f3RnTSmjzgXgs7v4k4ChPukWICpiIhb+vzTqSRfN4+nP/HK04NoJZoqm/qoZx/Iy3vL swarv1vX4A1OU7+UZ/flAKfpuHeXvznR2o04fj+gWCtHo/8E5YbAYqaxSHrao8WIn6NN n6OBbmxnRifQwXn483UVAsJaQo1zKL0O5ZP1pZVoyWlEGNOJ9/NFNFoVUpVrlQxaHIM9 RkGrUFPCkcIM9H5uftzLO48cdhGz8fsyCJuohmiCde/1iaxtmX9k66d6skLliq/uRT1R fj3A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156219; x=1707761019; 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=0UU5x8eOSKj/KaWrqX6oh+A5NqhkELkaQHAqy20ER9Q=; b=Agzoh1E0P7zWLO9jaupGRAV++4YD2EJXGBdfb1ljglOJ8AjodZSs2BLXIZ9QuN4ry8 toDTDW3B+Z2tVUwhSrbDey+gxyCQ9xnm5XtUp3jQnGiJmeq+O6aRjrYpEAYLAViYxV06 at3QSRvdoCvwlQSMEuj1VWdwrz/zOo/jO4kWSCfcK1MRXITE8QN8CQomYx4Dp7q1fKQI xV7J04CQBXIJBypJsA2IqyjnmMDSP8NaSIgk1DXuMrMs5XGEPNn5K2sb2LRJQP/cMdq7 SLmdif+YbvsTQlGi5ErnMgqE4OshONqWzT3lKJwzYgEujJkYvt4lrisPyhBYGpCR6lSn WbFA== X-Gm-Message-State: AOJu0YxviGC5tTUGbMyn77nV8MKBtVSWQv9Vfq352y2x3fldsPYQiCZF 20nZMneG3hIVJdgKigtVxluIZYEslKLOy6FmfLqxsBnM0EEUZ6PCGHzTjez5jfpy04UoOMorO3T wP2U= X-Google-Smtp-Source: AGHT+IHMH29o1LnT9fdZzvqN0SdPi2nT8LDMahhk3atFdr1wwpnHBl0iR4CIusQzR5jpT0QD3f/Bcg== X-Received: by 2002:a17:902:cf4a:b0:1d9:5038:f115 with SMTP id e10-20020a170902cf4a00b001d95038f115mr342542plg.4.1707156219155; Mon, 05 Feb 2024 10:03:39 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCXT1FtyMkfKr1V+gB7G9IAgVIuZ2JDs8ZVmuZAdlTsS8IBJfWB4kEjZBsHpSfRKm2R2IP0MfXaVb07yByREreVNi1sq Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:38 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Thomas Monjalon Subject: [PATCH v9 01/23] devtools: add script to check for non inclusive naming Date: Mon, 5 Feb 2024 09:43:29 -0800 Message-ID: <20240205180328.131019-2-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Add a new script to find words that should not be used. It is a wrapper around git grep command. By default it prints matches but can also display counts. Uses the word lists from Inclusive Naming Initiative see https://inclusivenaming.org/word-lists/ Note: the JSON list has extra comma at end of list of elements which is not valid in basic JSON but is allowed in user-friendly JSON5 (https://json5.org/) To handle this the tool uses the PyPi package for parsing json5 format. Examples: $ ./devtools/check-inclusive-naming.py -c | head -5 app/test/test_common.c:1 app/test/test_eal_flags.c:8 app/test/test_hash.c:1 app/test/test_hash_readwrite_lf_perf.c:1 app/test/test_link_bonding_mode4.c:1 $ ./devtools/check-inclusive-naming.py lib/pcapng lib/pcapng/rte_pcapng.c: /* sanity check that is really a pcapng mbuf */ $ ./devtools/check-inclusive-naming.py -l lib/eal lib/eal/common/eal_common_memory.c lib/eal/common/eal_common_proc.c lib/eal/common/eal_common_trace.c lib/eal/common/eal_memcfg.h lib/eal/common/rte_malloc.c lib/eal/freebsd/eal.c lib/eal/linux/eal.c lib/eal/windows/eal.c Signed-off-by: Stephen Hemminger --- MAINTAINERS | 1 + devtools/check-inclusive-naming.py | 135 +++++++++++++++++++++++++++++ 2 files changed, 136 insertions(+) create mode 100755 devtools/check-inclusive-naming.py diff --git a/MAINTAINERS b/MAINTAINERS index 5fb3a73f840e..dbf7ea2d916d 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -88,6 +88,7 @@ F: devtools/check-doc-vs-code.sh F: devtools/check-dup-includes.sh F: devtools/check-maintainers.sh F: devtools/check-forbidden-tokens.awk +F: devtools/check-inclusive-naming.py F: devtools/check-git-log.sh F: devtools/check-spdx-tag.sh F: devtools/check-symbol-change.sh diff --git a/devtools/check-inclusive-naming.py b/devtools/check-inclusive-naming.py new file mode 100755 index 000000000000..e8989c3c9b79 --- /dev/null +++ b/devtools/check-inclusive-naming.py @@ -0,0 +1,135 @@ +#!/usr/bin/env python3 +# SPDX-License-Identifier: BSD-3-Clause +# Copyright 2023 Stephen Hemminger +# +# This script scans the source tree and creates list of files +# containing words that are recommended to be avoided by the +# Inclusive Naming Initiative. +# See: https://inclusivenaming.org/word-lists/ + +import argparse +import subprocess +from urllib.request import urlopen + +# Need JSON5 to be able to handle extra comma +import json5 + +DEFAULT_URL = 'https://inclusivenaming.org/word-lists/index.json' + +# These give false positives +skip_files = [ + 'doc/guides/rel_notes/', 'doc/guides/contributing/coding_style.rst', + 'doc/guides/prog_guide/glossary.rst' +] + +# These are allowed for now +allow_words = ['abort'] + + +def args_parse(): + "parse arguments and return the argument object back to main" + + parser = argparse.ArgumentParser( + description="Identify word usage not aligned with inclusive naming") + parser.add_argument('-c', + '--count', + help="Show the number of lines that match", + action='store_true') + parser.add_argument('-d', + '--debug', + default=False, + help="Debug this script", + action='store_true') + parser.add_argument('-l', + '--files-with-matches', + help="Show only names of files with hits", + action='store_true') + # note: tier 0 is "ok to use" + parser.add_argument('-t', + '--tier', + type=int, + choices=range(0, 4), + action='append', + help="Show non-conforming words of particular tier") + parser.add_argument('-x', + '--exclude', + default=skip_files, + action='append', + help="Exclude path from scan") + parser.add_argument('-a', + '--allow', + default=allow_words, + action='append', + help="Ignore these words") + parser.add_argument('--url', + default=DEFAULT_URL, + help="URL for the non-inclusive naming word list") + parser.add_argument('paths', nargs='*', help='files and directory to scan') + + return parser.parse_args() + + +def fetch_wordlist(url, tiers): + "Read list of words from inclusivenaming.org" + + # The wordlist is returned as JSON like: + # { + # "data" : + # [ + # { + # "term": "abort", + # "tier" : "1", + # "recommendation": "Replace when possible.", + # ... + with urlopen(url) as response: + entries = json5.loads(response.read())['data'] + + wordlist = [] + for item in entries: + tier = int(item['tier']) + if tiers.count(tier) > 0: + # convert minus sign to minus or space regex + pattern = item['term'].replace('-', '[- ]') + if not pattern in allow_words: + wordlist.append(pattern.lower()) + + return wordlist + + +def process(args): + "Find matching words" + + # Default to Tier 1, 2 and 3. + if args.tier: + tiers = args.tier + else: + tiers = list(range(1, 4)) + + wordlist = fetch_wordlist(args.url, tiers) + if args.debug: + print(f'Matching on {len(wordlist)} words') + + cmd = ['git', 'grep', '-i'] + if args.files_with_matches: + cmd.append('-l') + if args.count: + cmd.append('-c') + for word in wordlist: + cmd.append('-e') + cmd.append(word) + cmd.append('--') + for path in skip_files: + cmd.append(f':^{path}') + cmd += args.paths + if args.debug: + print(cmd) + subprocess.run(cmd, check=False) + + +def main(): + '''program main function''' + process(args_parse()) + + +if __name__ == "__main__": + main() From patchwork Mon Feb 5 17:43:30 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136389 X-Patchwork-Delegate: thomas@monjalon.net 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 E838843A3D; Mon, 5 Feb 2024 19:03:53 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id E9D4340E0F; Mon, 5 Feb 2024 19:03:42 +0100 (CET) Received: from mail-pl1-f170.google.com (mail-pl1-f170.google.com [209.85.214.170]) by mails.dpdk.org (Postfix) with ESMTP id 1B7A7402CC for ; Mon, 5 Feb 2024 19:03:40 +0100 (CET) Received: by mail-pl1-f170.google.com with SMTP id d9443c01a7336-1d95d67ff45so30540565ad.2 for ; Mon, 05 Feb 2024 10:03:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156220; x=1707761020; darn=dpdk.org; 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=xwm8Nd49JcW5dn3uPYlDxJ4M4mYTM/ah5yBAj8FDDgs=; b=orjZFUepkmZ8k9/OxTTuO3pl/shO3Ym54p+x2Wg31CPc6ffSvytGuqLu8mvF9Q/2M2 Y3fLXNgZ33uZXozMoPguXJhMC3eoKv5MKxIvoISHZJBnCElW0IY0Bs406l9f1ncovXCb vqTOheGQdhwEOZY+1T0HHn6YX2gdhNiX2v4pzfKyFRRMzQLvXfVEykUyWDaZAa/pmZCn 2wPK+1fxN71XW0A79ZBd59KkHQWMiDUg3iCILXDkG6gxONdClKLkFm1I2E0VD32cj+0f m7p5cvBeFn7ht6fOEHbF4WJCMt53PmRmnU+MNlBMBWqn8VZ4/B6j+QQmW3AF8iwg8iYv 9ILg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156220; x=1707761020; 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=xwm8Nd49JcW5dn3uPYlDxJ4M4mYTM/ah5yBAj8FDDgs=; b=IurmREAAUWc0+hdn2stJIqJ5i4KQrCWn716XDPnKSV2oPLbwE0PpUQS8NRUiX2ffsA iDyC+2QVmr9XYtyQpT6K+cewkWm780fQ+fP1kNKpMnC8TLH099/jdCOGVKxE5c42x0te aPqCOISZ5CXoPn6J1mSFF65MkhEXS/8xVcqZYNL0P36xPd4QqNvfwnPVJRN4Lf/49pu2 myxFPiwTaLC2crCE2r2aRfywekZL/2ojm9O5GNKHIRH7s2sshQhxR0gl02oF/7Q36z4P KOvqm9yd7zRZJbGaOHvV1/JXxhnPA0ksNhKg+2tlLoo/BPOs8/qxmzrcSFzN5iSJHoY5 XeZw== X-Gm-Message-State: AOJu0YyyorhKxAfr0ZAJCqpA/pjYqbcdCYmicnajHf3XZIlyf8glt/UQ WbOj/DNy7/z1uzhB/XWNhuiCA6hZ8CuHWHT5gMNV7rNYYzAPG9a5u9IkojsfSNhJK+t/lsxTklq Ildc= X-Google-Smtp-Source: AGHT+IGi7CHaKUAaioau96YbID+VxfKDOaZZ2KtE/So7VSZ6ZtqeuSN9ItNZ3sAxdDMW36JSafH2gg== X-Received: by 2002:a17:902:e54a:b0:1d9:c367:2032 with SMTP id n10-20020a170902e54a00b001d9c3672032mr392513plf.10.1707156220021; Mon, 05 Feb 2024 10:03:40 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCVqc7TvVW72MmeOgM50O+DSh4IaNd/0ZiXFCPTkW8tU9EM52g4n7cfv/QTv3/feviQMxCrbG24jcCHiogvKTmhgc/hkrcCt0HXxrSvtdAPfT2QvmbqpIKnQk6pXAkwO9sOs4rZES1F6ALiiqcOf5xky83sm/WVaen+yH+vkuk46LBr6vMTRpM9Mm2bBCV3cd1HxTWY3WoXFJDtpB0bkJ/NdUD1ZmZIHq50MumJUlNJaYzqfmDiyb1Y= Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:39 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Tyler Retzlaff , Yipeng Wang , Sameh Gobriel , Bruce Richardson , Vladimir Medvedkin Subject: [PATCH v9 02/23] test: replace use of term segregate Date: Mon, 5 Feb 2024 09:43:30 -0800 Message-ID: <20240205180328.131019-3-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Change comment based on inclusive naming recommendations. https://inclusivenaming.org/word-lists/tier-3/ Signed-off-by: Stephen Hemminger Acked-by: Tyler Retzlaff --- app/test/test_hash_readwrite_lf_perf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/test/test_hash_readwrite_lf_perf.c b/app/test/test_hash_readwrite_lf_perf.c index 5d18850e199a..817754e9889f 100644 --- a/app/test/test_hash_readwrite_lf_perf.c +++ b/app/test/test_hash_readwrite_lf_perf.c @@ -397,7 +397,7 @@ generate_keys(void) for (i = 2; i < TOTAL_INSERT; i++) keys[i] = keys[i-1] + keys[i-2]; - /* Segregate keys into keys_no_ks and keys_ks */ + /* Separate keys into keys_no_ks and keys_ks */ for (i = 0; i < TOTAL_INSERT; i++) { /* Check if primary bucket has space.*/ sig = rte_hash_hash(tbl_rwc_test_param.h, From patchwork Mon Feb 5 17:43:31 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136390 X-Patchwork-Delegate: thomas@monjalon.net 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 BE22743A3D; Mon, 5 Feb 2024 19:04:02 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1B3FF40E36; Mon, 5 Feb 2024 19:03:44 +0100 (CET) Received: from mail-pl1-f176.google.com (mail-pl1-f176.google.com [209.85.214.176]) by mails.dpdk.org (Postfix) with ESMTP id 1A18C40DF6 for ; Mon, 5 Feb 2024 19:03:42 +0100 (CET) Received: by mail-pl1-f176.google.com with SMTP id d9443c01a7336-1d72f71f222so37528365ad.1 for ; Mon, 05 Feb 2024 10:03:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156221; x=1707761021; darn=dpdk.org; 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=VXOXU50fnLBP+CKN25gJEh16oaeo3cGMEGYk3PHbOHc=; b=aSymfJVEoALWCGaRQLoZ4EWuQDIS8owVR6J5vHdZTe3VJCZZ+XYk1Ky+EK9e7TAeXv rw56JUxgkD68j3105n0VxlatnXgRppApM47cY9KouuxM2xcA51nnBU1y7uL5pSJZaePe 8TRwpfJJB4UbU6ZD9ejgR8EY7pi24TFYySU4DLSJhYqKx1TTM5RhIMLxCYBQoXmNjTK4 jEWx07O7olzyhGOyjeNlgBM8T1djRWIWyWHp+jcgjtbPW0wpKye6JYK+cNKUJrnziFiR hctgcsvb5BkUzWAE9JBPoJ+jRdTOLiTt4+s5Q7SvyOiNQpcR82dLcI+ui7lBItLfaRFT 9XfQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156221; x=1707761021; 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=VXOXU50fnLBP+CKN25gJEh16oaeo3cGMEGYk3PHbOHc=; b=TBopVCOjQIVHTr2za9M+wtw2gn/Yt3UyvAjsfcW91SI/rBpdYj7X5ZE2ND4slk2jDS Tee0d7VauIC4B+T1jD8xnaVSXOyUAwy3Bsee7E2NAPObOUMPcbZYEodOvlgbib3IeG4u tTsFjV4oQ5dLbHoMiCm78qcVo+FQMRCLy4PvbusKdaeFK0BI/6EMxU23Rvg8mmDyZiJH jb4CpgF+PUXskWxgYZmo3EB+qQiOrs6UfUQznhi6Tyw9vj/kwMWCb3P7N0Dl7SP0+bJn r/+2qzUCpw8BD+s4gG+4XMZhr7QljalP2Xic05VSgPByaFZcj+9Zy7GQKXQ0Yurd/FW1 fYoA== X-Gm-Message-State: AOJu0Yx3k4rWy4CBG9MfYEhiKw0CONHqKMv6LkMk3zCI7+F3K5vrhldE 9mm+39IT7+H9PFZfHVbIBqu/ogZkpsadqrx6aZfauDcFGYWG1pieFhi2Eh8n9OJDi88Qun16AR7 rLuQ= X-Google-Smtp-Source: AGHT+IHmfRJh5jixOYf+ccKMqv4xNWDISXEJgo4fw3Ee7EOW2hnT574+4Exvfu7dinf8DnHujTBmNA== X-Received: by 2002:a17:903:189:b0:1d9:a780:393d with SMTP id z9-20020a170903018900b001d9a780393dmr367297plg.8.1707156221201; Mon, 05 Feb 2024 10:03:41 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCVX/NLAMgkoUE6M+vFdCU8SjRAABSNpVbB0fG/rEmHWxe/Tg3BzYBggiK6Buo5rOiopD1a2Wh2N+SsdGFrc/tBbzS4KRyg5a6IYvoS9+bdlLrIZJoHQvpmHuLHfZg7oQ+5Dbuo8GZcajqdf1BO61qsqhovP5+2JAZ53+oqaJnbw5oIfDX4= Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:40 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Somnath Kotur , Ajit Khaparde , Kirill Rybalchenko Subject: [PATCH v9 03/23] examples/ptp: replace terms master and slave Date: Mon, 5 Feb 2024 09:43:31 -0800 Message-ID: <20240205180328.131019-4-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 IEEE 1588 standard has been updated to remove the use of master-slave terminolgy. Use the terms recommended by IEEE 1588g-2022 amendment: In place of the term “master”, use the term “timeTransmitter”. In place of the term “slave”, use the term “timeReceiver”. Signed-off-by: Stephen Hemminger Reviewed-by: Somnath Kotur --- doc/guides/nics/bnxt.rst | 8 ++-- doc/guides/sample_app_ug/img/ptpclient.svg | 4 +- doc/guides/sample_app_ug/intro.rst | 4 +- doc/guides/sample_app_ug/ptpclient.rst | 29 ++++++------ examples/ptpclient/ptpclient.c | 54 +++++++++++----------- 5 files changed, 51 insertions(+), 48 deletions(-) diff --git a/doc/guides/nics/bnxt.rst b/doc/guides/nics/bnxt.rst index 6db880d632f8..8b9fcd25583d 100644 --- a/doc/guides/nics/bnxt.rst +++ b/doc/guides/nics/bnxt.rst @@ -538,10 +538,12 @@ Time Synchronization ~~~~~~~~~~~~~~~~~~~~ System operators may run a PTP (Precision Time Protocol) client application to -synchronize the time on the NIC (and optionally, on the system) to a PTP master. +synchronize the time on the NIC (and optionally, on the system) to a +PTP timeTransmitter. -The BNXT PMD supports a PTP client application to communicate with a PTP master -clock using DPDK IEEE1588 APIs. Note that the PTP client application needs to +The BNXT PMD supports a PTP client application to communicate with a +PTP timeTransmitter using DPDK IEEE1588 APIs. +Note that the PTP client application needs to run on PF and vector mode needs to be disabled. .. code-block:: console diff --git a/doc/guides/sample_app_ug/img/ptpclient.svg b/doc/guides/sample_app_ug/img/ptpclient.svg index fd78ef839b91..41869bc4c959 100644 --- a/doc/guides/sample_app_ug/img/ptpclient.svg +++ b/doc/guides/sample_app_ug/img/ptpclient.svg @@ -488,7 +488,7 @@ sodipodi:role="line" id="tspan7096" x="38.764343" - y="590.47479">master + y="590.47479">timeTransmitter slave + y="593.71478">timeReceiver `: The PTP client is another minimal implementation of a real world application. In this case the application is a PTP client that communicates with a PTP - master clock to synchronize time on a Network Interface Card (NIC) using the - IEEE1588 protocol. + timeTransmitter to synchronize time on a Network Interface Card (NIC) + using the IEEE1588 protocol. * :doc:`Quality of Service (QoS) Scheduler`: The QoS Scheduler application demonstrates the use of DPDK to provide QoS scheduling. diff --git a/doc/guides/sample_app_ug/ptpclient.rst b/doc/guides/sample_app_ug/ptpclient.rst index d47e942738d0..242c9628eaff 100644 --- a/doc/guides/sample_app_ug/ptpclient.rst +++ b/doc/guides/sample_app_ug/ptpclient.rst @@ -5,8 +5,9 @@ PTP Client Sample Application ============================= The PTP (Precision Time Protocol) client sample application is a simple -example of using the DPDK IEEE1588 API to communicate with a PTP master clock -to synchronize the time on the NIC and, optionally, on the Linux system. +example of using the DPDK IEEE1588 API to communicate with a PTP +timeTransmitter to synchronize the time on the NIC and, optionally, +on the Linux system. Note, PTP is a time syncing protocol and cannot be used within DPDK as a time-stamping mechanism. See the following for an explanation of the protocol: @@ -21,10 +22,10 @@ The PTP sample application is intended as a simple reference implementation of a PTP client using the DPDK IEEE1588 API. In order to keep the application simple the following assumptions are made: -* The first discovered master is the main for the session. +* The first discovered timeTransmitter is the main for the session. * Only L2 PTP packets are supported. * Only the PTP v2 protocol is supported. -* Only the slave clock is implemented. +* Only the timeReceiver clock is implemented. How the Application Works @@ -38,12 +39,12 @@ How the Application Works The PTP synchronization in the sample application works as follows: -* Master sends *Sync* message - the slave saves it as T2. -* Master sends *Follow Up* message and sends time of T1. -* Slave sends *Delay Request* frame to PTP Master and stores T3. -* Master sends *Delay Response* T4 time which is time of received T3. +* TimeTransmitter sends *Sync* message - the TimeReceiver saves it as T2. +* TimeTransmitter sends *Follow Up* message and sends time of T1. +* TimeReceiver sends *Delay Request* frame to PTP TimeTransmitter and stores T3. +* TimeTransmitter sends *Delay Response* T4 time which is time of received T3. -The adjustment for slave can be represented as: +The adjustment for timeReceiver can be represented as: adj = -[(T2-T1)-(T4 - T3)]/2 @@ -71,8 +72,8 @@ Refer to *DPDK Getting Started Guide* for general information on running applications and the Environment Abstraction Layer (EAL) options. * ``-p portmask``: Hexadecimal portmask. -* ``-T 0``: Update only the PTP slave clock. -* ``-T 1``: Update the PTP slave clock and synchronize the Linux Kernel to the PTP clock. +* ``-T 0``: Update only the PTP timeReceiver clock. +* ``-T 1``: Update the PTP timeReceiver clock and synchronize the Linux Kernel to the PTP clock. Code Explanation @@ -178,7 +179,7 @@ The forwarding loop can be interrupted and the application closed using PTP parsing ~~~~~~~~~~~ -The ``parse_ptp_frames()`` function processes PTP packets, implementing slave +The ``parse_ptp_frames()`` function processes PTP packets, implementing timeReceiver PTP IEEE1588 L2 functionality. .. literalinclude:: ../../../examples/ptpclient/ptpclient.c @@ -187,11 +188,11 @@ PTP IEEE1588 L2 functionality. :end-before: >8 End of function processes PTP packets. There are 3 types of packets on the RX path which we must parse to create a minimal -implementation of the PTP slave client: +implementation of the PTP timeReceiver client: * SYNC packet. * FOLLOW UP packet * DELAY RESPONSE packet. When we parse the *FOLLOW UP* packet we also create and send a *DELAY_REQUEST* packet. -Also when we parse the *DELAY RESPONSE* packet, and all conditions are met we adjust the PTP slave clock. +Also when we parse the *DELAY RESPONSE* packet, and all conditions are met we adjust the PTP timeReceiver clock. diff --git a/examples/ptpclient/ptpclient.c b/examples/ptpclient/ptpclient.c index 2535d848a1e9..7ba7a3cb02c0 100644 --- a/examples/ptpclient/ptpclient.c +++ b/examples/ptpclient/ptpclient.c @@ -116,14 +116,14 @@ struct ptp_message { } __rte_packed; }; -struct ptpv2_data_slave_ordinary { +struct ptpv2_timeReceiver_ordinary { struct rte_mbuf *m; struct timespec tstamp1; struct timespec tstamp2; struct timespec tstamp3; struct timespec tstamp4; struct clock_id client_clock_id; - struct clock_id master_clock_id; + struct clock_id transmitter_clock_id; struct timeval new_adj; int64_t delta; uint16_t portid; @@ -134,7 +134,7 @@ struct ptpv2_data_slave_ordinary { uint16_t current_ptp_port; }; -static struct ptpv2_data_slave_ordinary ptp_data; +static struct ptpv2_timeReceiver_ordinary ptp_data; static inline uint64_t timespec64_to_ns(const struct timespec *ts) { @@ -263,38 +263,38 @@ port_init(uint16_t port, struct rte_mempool *mbuf_pool) } static void -print_clock_info(struct ptpv2_data_slave_ordinary *ptp_data) +print_clock_info(struct ptpv2_timeReceiver_ordinary *ptp_data) { int64_t nsec; struct timespec net_time, sys_time; - printf("Master Clock id: %02x:%02x:%02x:%02x:%02x:%02x:%02x:%02x", - ptp_data->master_clock_id.id[0], - ptp_data->master_clock_id.id[1], - ptp_data->master_clock_id.id[2], - ptp_data->master_clock_id.id[3], - ptp_data->master_clock_id.id[4], - ptp_data->master_clock_id.id[5], - ptp_data->master_clock_id.id[6], - ptp_data->master_clock_id.id[7]); - - printf("\nT2 - Slave Clock. %lds %ldns", + printf("TimeTransmitter Clock id: %02x:%02x:%02x:%02x:%02x:%02x:%02x:%02x", + ptp_data->transmitter_clock_id.id[0], + ptp_data->transmitter_clock_id.id[1], + ptp_data->transmitter_clock_id.id[2], + ptp_data->transmitter_clock_id.id[3], + ptp_data->transmitter_clock_id.id[4], + ptp_data->transmitter_clock_id.id[5], + ptp_data->transmitter_clock_id.id[6], + ptp_data->transmitter_clock_id.id[7]); + + printf("\nT2 - TimeReceiver Clock. %lds %ldns", (ptp_data->tstamp2.tv_sec), (ptp_data->tstamp2.tv_nsec)); - printf("\nT1 - Master Clock. %lds %ldns ", + printf("\nT1 - TimeTransmitter Clock. %lds %ldns ", ptp_data->tstamp1.tv_sec, (ptp_data->tstamp1.tv_nsec)); - printf("\nT3 - Slave Clock. %lds %ldns", + printf("\nT3 - TimeReceiver Clock. %lds %ldns", ptp_data->tstamp3.tv_sec, (ptp_data->tstamp3.tv_nsec)); - printf("\nT4 - Master Clock. %lds %ldns ", + printf("\nT4 - TimeTransmitter Clock. %lds %ldns ", ptp_data->tstamp4.tv_sec, (ptp_data->tstamp4.tv_nsec)); - printf("\nDelta between master and slave clocks:%"PRId64"ns\n", + printf("\nDelta between timeTransmitter and timeReceiver clocks:%"PRId64"ns\n", ptp_data->delta); clock_gettime(CLOCK_REALTIME, &sys_time); @@ -328,7 +328,7 @@ print_clock_info(struct ptpv2_data_slave_ordinary *ptp_data) } static int64_t -delta_eval(struct ptpv2_data_slave_ordinary *ptp_data) +delta_eval(struct ptpv2_timeReceiver_ordinary *ptp_data) { int64_t delta; uint64_t t1 = 0; @@ -350,7 +350,7 @@ delta_eval(struct ptpv2_data_slave_ordinary *ptp_data) * Parse the PTP SYNC message. */ static void -parse_sync(struct ptpv2_data_slave_ordinary *ptp_data, uint16_t rx_tstamp_idx) +parse_sync(struct ptpv2_timeReceiver_ordinary *ptp_data, uint16_t rx_tstamp_idx) { struct ptp_header *ptp_hdr; @@ -359,7 +359,7 @@ parse_sync(struct ptpv2_data_slave_ordinary *ptp_data, uint16_t rx_tstamp_idx) ptp_data->seqID_SYNC = rte_be_to_cpu_16(ptp_hdr->seq_id); if (ptp_data->ptpset == 0) { - rte_memcpy(&ptp_data->master_clock_id, + rte_memcpy(&ptp_data->transmitter_clock_id, &ptp_hdr->source_port_id.clock_id, sizeof(struct clock_id)); ptp_data->ptpset = 1; @@ -380,7 +380,7 @@ parse_sync(struct ptpv2_data_slave_ordinary *ptp_data, uint16_t rx_tstamp_idx) * Parse the PTP FOLLOWUP message and send DELAY_REQ to the main clock. */ static void -parse_fup(struct ptpv2_data_slave_ordinary *ptp_data) +parse_fup(struct ptpv2_timeReceiver_ordinary *ptp_data) { struct rte_ether_hdr *eth_hdr; struct rte_ether_addr eth_addr; @@ -399,7 +399,7 @@ parse_fup(struct ptpv2_data_slave_ordinary *ptp_data) eth_hdr = rte_pktmbuf_mtod(m, struct rte_ether_hdr *); ptp_hdr = rte_pktmbuf_mtod_offset(m, struct ptp_header *, sizeof(struct rte_ether_hdr)); - if (memcmp(&ptp_data->master_clock_id, + if (memcmp(&ptp_data->transmitter_clock_id, &ptp_hdr->source_port_id.clock_id, sizeof(struct clock_id)) != 0) return; @@ -530,7 +530,7 @@ update_kernel_time(void) * Parse the DELAY_RESP message. */ static void -parse_drsp(struct ptpv2_data_slave_ordinary *ptp_data) +parse_drsp(struct ptpv2_timeReceiver_ordinary *ptp_data) { struct rte_mbuf *m = ptp_data->m; struct ptp_message *ptp_msg; @@ -568,7 +568,7 @@ parse_drsp(struct ptpv2_data_slave_ordinary *ptp_data) } } -/* This function processes PTP packets, implementing slave PTP IEEE1588 L2 +/* This function processes PTP packets, implementing timeReceiver PTP IEEE1588 L2 * functionality. */ @@ -753,7 +753,7 @@ main(int argc, char *argv[]) rte_exit(EXIT_FAILURE, "Error with EAL initialization\n"); /* >8 End of initialization of EAL. */ - memset(&ptp_data, '\0', sizeof(struct ptpv2_data_slave_ordinary)); + memset(&ptp_data, 0, sizeof(struct ptpv2_timeReceiver_ordinary)); /* Parse specific arguments. 8< */ argc -= ret; From patchwork Mon Feb 5 17:43:32 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136391 X-Patchwork-Delegate: thomas@monjalon.net 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 0D13A43A3D; Mon, 5 Feb 2024 19:04:12 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 42FBD40E54; Mon, 5 Feb 2024 19:03:45 +0100 (CET) Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) by mails.dpdk.org (Postfix) with ESMTP id E497540E09 for ; Mon, 5 Feb 2024 19:03:42 +0100 (CET) Received: by mail-pl1-f173.google.com with SMTP id d9443c01a7336-1d9b2400910so7527825ad.0 for ; Mon, 05 Feb 2024 10:03:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156222; x=1707761022; darn=dpdk.org; 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=hS10/593C6UDIr0RrUgvd0Cv/5PDXEq5zhDdiZu6wOc=; b=2GCNdYzkiht8spMMpMgWwtVfoT7c2a9pgrUVYpAu1/+cPfCKCmtyKOiz8C2Xxjzhsu x93OfgS08ws2Yg2BTpzW8Wk9RAbXldZWK9+e0qn+9GrSAH1QP3GR9b9KWBOlhqSwmvlG WIMazfU9lJpLFSgCGcnjdUUt4vcTEIbQUnLCmWdv1S+xLilcda4JsGjB4YmTkMnHjXZN X4qFssoMlIzxAXl7t0lM9sBkI0VhWOLk8iSwPPHq3BZQQQuueIT/LyL/8DBkwc5C6aYy GWiTG/uTxSPb11YVMgrH39ZXmq4i1oTQPHUfao+/HyyQz+LHA/EVA7OCYBAh4ljg2aS6 OcCA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156222; x=1707761022; 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=hS10/593C6UDIr0RrUgvd0Cv/5PDXEq5zhDdiZu6wOc=; b=ps1TLCZRlcjy+c6V2hUiA2/q6w/t4j0c+gZTmb3QNukaJC9mMCV9MsqabFQkpq8rYM AqpdEd2zfn5XKrTXky03kdC+3t1vrsEYwe1HN9N9CFvB67vVm7hCn3zHN38lT8Ax7SaW qi7ZfR/KMC2YOUgBeZXOhcv8Qtjo8yGLkIFvZP5J1Jq4xGJ/G63StskN95S33n2iAewI CytBDk2M3FrynepLI1uyKnEO5Aluoci7JbRmpZDs1swSIOKYrIVaUeZJXPeqYDNoTdTN Bm2I/4CvE3VIT7xZGH5BfHVT0gQVcY9z3ZU51/Mpvvgzcr1GIvU/ZjQqvOvUKd2LWhXw gCow== X-Gm-Message-State: AOJu0Ywa2TJbpt7jD87HmXH/883FbfGrVOFcukfuDt8cV2AgZtiWYJpB FdV0kZKCgj76M2xhUQJsR21LUWiTS1tjwsrlepDoxAfN3xQfp+D5gWZYBJx9WiBxOqWAJA/+W3T BLDw= X-Google-Smtp-Source: AGHT+IE8iLQGLIZg14l3gfERVzIXXpQtq/JQUXrTSn1DGC6uBvXxzY93jP4IyUuzLYVInEctcPL08g== X-Received: by 2002:a17:902:e80e:b0:1d6:f17b:ecfc with SMTP id u14-20020a170902e80e00b001d6f17becfcmr556125plg.15.1707156222111; Mon, 05 Feb 2024 10:03:42 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCVXEf7d7HZMWpKDuz6kVupX77bIgLj52PvwUrh3ZWCjpBYpDf81nHFnQpBlTw7nIWGNLtkaXEp+DWa0wOM8uWF5b81AkPufy6hPvpSS16ML/O5LMRWmpNximlhCdc2vQX18h2zIpaMQvORzXg== Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:41 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Tyler Retzlaff , Cristian Dumitrescu Subject: [PATCH v9 04/23] test: remove use of word master in test_red Date: Mon, 5 Feb 2024 09:43:32 -0800 Message-ID: <20240205180328.131019-5-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 No need to use term master here. Signed-off-by: Stephen Hemminger Acked-by: Tyler Retzlaff --- app/test/test_red.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/test/test_red.c b/app/test/test_red.c index aa7538d51a35..6bf58ad4dd9a 100644 --- a/app/test/test_red.c +++ b/app/test/test_red.c @@ -85,7 +85,7 @@ struct test_var { /**< Test variables used for testing RTE_RED uint32_t *enqueued; /**< Test operations enqueued */ }; -struct test_config { /**< Master test structure for RTE_RED */ +struct test_config { /**< Test structure for RTE_RED */ const char *ifname; /**< Interface name */ const char *msg; /**< Test message for display */ const char *htxt; /**< Header txt display for result output */ From patchwork Mon Feb 5 17:43:33 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136392 X-Patchwork-Delegate: thomas@monjalon.net 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 DF19843A3D; Mon, 5 Feb 2024 19:04:22 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A71C340E64; Mon, 5 Feb 2024 19:03:46 +0100 (CET) Received: from mail-pl1-f180.google.com (mail-pl1-f180.google.com [209.85.214.180]) by mails.dpdk.org (Postfix) with ESMTP id 0B9C740E2D for ; Mon, 5 Feb 2024 19:03:44 +0100 (CET) Received: by mail-pl1-f180.google.com with SMTP id d9443c01a7336-1d51ba18e1bso42430685ad.0 for ; Mon, 05 Feb 2024 10:03:43 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156223; x=1707761023; darn=dpdk.org; 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=+ldwT6WnrCF5j1VIzTOM/PAMUpCXe7FMiGsRdqD3Gv8=; b=LHk/kSPVkSm7GmK4mqrRKL2Mt08a0Ha9hjumNw8Ee/kbR7DlXYLbk6diCXRbRxzvtr 6jLPbq/fNZYSM44wNUcYf3/27B9NL8ZMnmYxKp42Em2B7UyA2pEbQ9aR2uY/38nR8ma7 yugIM8VJOIZXAvTRLfXnmus1G22PEC28dNMJNqWWrol+u7tFsrOe8E5UHk1nDho1CESU Xy6tLR7Ak0AwQWlyauo4oNJP7IfdgYm6fFMqxos6E5putdOp1schvNcaZe+SW7Fcai2y ouxJj/R5LgFqyDhclDwYcYcEpwv3LF9EeH8kzyZZObJx+qWnT642AoJT+E1SUCv6m1Wl NyfA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156223; x=1707761023; 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=+ldwT6WnrCF5j1VIzTOM/PAMUpCXe7FMiGsRdqD3Gv8=; b=U1cZ+11iYbTRFcRguUyDyjKstQDyvGR6bM9swAhN6QuKjcxTzHbxGUCNY6TLATkF4Q t+Fm7iRDf8bOr3KJjuUWJkepT8842TVR2SmR1sQ5pFTRz+kvChsFZoE0GC7fdRBVko19 uLHK142BCVQV4EW7rdICL9/3FOFiw9MuvKLkbyeIMpYqU5166ZKm20tnS/fJBZmnJPt0 boXBQk6w68ufvgfbe1ge91Ny369WqMriDXXzCRQiWzPSJVLJk8XxqUJien5ooB6fyg4n v1UvYc9Q7eBaVrfQEYsKOusH+xcWfxSTamLD80RPC6zB+u+ak2eR0iQCFNe+/C1z5WGr p19A== X-Gm-Message-State: AOJu0Yzi7E0ZUv78zV8UlC5cKRtAmDeDAHbzIzvcWhBi1+JsP6xt6NY/ e0j5SAn+JYQNeqGJgezCmiSLLIFI0OyZKDINGHyf9CTt+06k1pQ4s7KGtlaf+UsndfaD2zZBBsK YxN8= X-Google-Smtp-Source: AGHT+IE6TTtSC/NhiXS4ppelXO1pmnAowjmcjmCEZSH5Hs0TlSmSWSQHEi8R8DcezXw+QFub5lPAMA== X-Received: by 2002:a17:90b:916:b0:296:521c:4caf with SMTP id bo22-20020a17090b091600b00296521c4cafmr221258pjb.8.1707156223097; Mon, 05 Feb 2024 10:03:43 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCUoAuVki7VJ6qdzCvqHVqfg646Is8Hj5gzkwclBhO5a9fOds0cPPYdI0Ck66ixvEKsQ4oYaUHwSicj6yJLqDn75kmnQPPWrVoUzgov2n8Ks31sTJ5CzwAxb8KSHEgMros3CM1rPueJq8D+Rh0JEk8rflL035iM1vlEJByCCdU/ovQgDR+E09ZqOF/4HjkMGE94aqSikX/sOcq9D7Yw9vR91 Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:42 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Andrew Rybchenko , =?utf-8?q?Morten_Br=C3=B8?= =?utf-8?q?rup?= , Steven Webster , Matt Peters Subject: [PATCH v9 05/23] mbuf: replace term sanity check Date: Mon, 5 Feb 2024 09:43:33 -0800 Message-ID: <20240205180328.131019-6-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Replace rte_mbuf_sanity_check() with rte_mbuf_verify() to match the similar macro RTE_VERIFY() in rte_debug.h Good wording from discussion english.stackexchange.com: The phrase "sanity check" is ableist language as it implies that there is something wrong with people who have mental illnesses and the word "sanity" has been used to discriminate against such people. Therefore, it should should be avoided. Signed-off-by: Stephen Hemminger Acked-by: Andrew Rybchenko Acked-by: Morten Brørup --- app/test/test_mbuf.c | 28 +++++------ doc/guides/prog_guide/mbuf_lib.rst | 4 +- doc/guides/rel_notes/deprecation.rst | 3 ++ drivers/net/avp/avp_ethdev.c | 18 +++---- drivers/net/sfc/sfc_ef100_rx.c | 6 +-- drivers/net/sfc/sfc_ef10_essb_rx.c | 4 +- drivers/net/sfc/sfc_ef10_rx.c | 4 +- drivers/net/sfc/sfc_rx.c | 2 +- examples/ipv4_multicast/main.c | 2 +- lib/mbuf/rte_mbuf.c | 23 +++++---- lib/mbuf/rte_mbuf.h | 71 +++++++++++++++------------- lib/mbuf/version.map | 1 + 12 files changed, 90 insertions(+), 76 deletions(-) diff --git a/app/test/test_mbuf.c b/app/test/test_mbuf.c index d7393df7eb5d..261c6e5d71e9 100644 --- a/app/test/test_mbuf.c +++ b/app/test/test_mbuf.c @@ -261,8 +261,8 @@ test_one_pktmbuf(struct rte_mempool *pktmbuf_pool) GOTO_FAIL("Buffer should be continuous"); memset(hdr, 0x55, MBUF_TEST_HDR2_LEN); - rte_mbuf_sanity_check(m, 1); - rte_mbuf_sanity_check(m, 0); + rte_mbuf_verify(m, 1); + rte_mbuf_verify(m, 0); rte_pktmbuf_dump(stdout, m, 0); /* this prepend should fail */ @@ -1161,7 +1161,7 @@ test_refcnt_mbuf(void) #ifdef RTE_EXEC_ENV_WINDOWS static int -test_failing_mbuf_sanity_check(struct rte_mempool *pktmbuf_pool) +test_failing_mbuf_verify(struct rte_mempool *pktmbuf_pool) { RTE_SET_USED(pktmbuf_pool); return TEST_SKIPPED; @@ -1180,12 +1180,12 @@ mbuf_check_pass(struct rte_mbuf *buf) } static int -test_failing_mbuf_sanity_check(struct rte_mempool *pktmbuf_pool) +test_failing_mbuf_verify(struct rte_mempool *pktmbuf_pool) { struct rte_mbuf *buf; struct rte_mbuf badbuf; - printf("Checking rte_mbuf_sanity_check for failure conditions\n"); + printf("Checking rte_mbuf_verify for failure conditions\n"); /* get a good mbuf to use to make copies */ buf = rte_pktmbuf_alloc(pktmbuf_pool); @@ -1707,7 +1707,7 @@ test_mbuf_validate_tx_offload(const char *test_name, GOTO_FAIL("%s: mbuf allocation failed!\n", __func__); if (rte_pktmbuf_pkt_len(m) != 0) GOTO_FAIL("%s: Bad packet length\n", __func__); - rte_mbuf_sanity_check(m, 0); + rte_mbuf_verify(m, 0); m->ol_flags = ol_flags; m->tso_segsz = segsize; ret = rte_validate_tx_offload(m); @@ -1914,7 +1914,7 @@ test_pktmbuf_read(struct rte_mempool *pktmbuf_pool) GOTO_FAIL("%s: mbuf allocation failed!\n", __func__); if (rte_pktmbuf_pkt_len(m) != 0) GOTO_FAIL("%s: Bad packet length\n", __func__); - rte_mbuf_sanity_check(m, 0); + rte_mbuf_verify(m, 0); data = rte_pktmbuf_append(m, MBUF_TEST_DATA_LEN2); if (data == NULL) @@ -1963,7 +1963,7 @@ test_pktmbuf_read_from_offset(struct rte_mempool *pktmbuf_pool) if (rte_pktmbuf_pkt_len(m) != 0) GOTO_FAIL("%s: Bad packet length\n", __func__); - rte_mbuf_sanity_check(m, 0); + rte_mbuf_verify(m, 0); /* prepend an ethernet header */ hdr = (struct ether_hdr *)rte_pktmbuf_prepend(m, hdr_len); @@ -2108,7 +2108,7 @@ create_packet(struct rte_mempool *pktmbuf_pool, GOTO_FAIL("%s: mbuf allocation failed!\n", __func__); if (rte_pktmbuf_pkt_len(pkt_seg) != 0) GOTO_FAIL("%s: Bad packet length\n", __func__); - rte_mbuf_sanity_check(pkt_seg, 0); + rte_mbuf_verify(pkt_seg, 0); /* Add header only for the first segment */ if (test_data->flags == MBUF_HEADER && seg == 0) { hdr_len = sizeof(struct rte_ether_hdr); @@ -2320,7 +2320,7 @@ test_pktmbuf_ext_shinfo_init_helper(struct rte_mempool *pktmbuf_pool) GOTO_FAIL("%s: mbuf allocation failed!\n", __func__); if (rte_pktmbuf_pkt_len(m) != 0) GOTO_FAIL("%s: Bad packet length\n", __func__); - rte_mbuf_sanity_check(m, 0); + rte_mbuf_verify(m, 0); ext_buf_addr = rte_malloc("External buffer", buf_len, RTE_CACHE_LINE_SIZE); @@ -2484,8 +2484,8 @@ test_pktmbuf_ext_pinned_buffer(struct rte_mempool *std_pool) GOTO_FAIL("%s: test_pktmbuf_copy(pinned) failed\n", __func__); - if (test_failing_mbuf_sanity_check(pinned_pool) < 0) - GOTO_FAIL("%s: test_failing_mbuf_sanity_check(pinned)" + if (test_failing_mbuf_verify(pinned_pool) < 0) + GOTO_FAIL("%s: test_failing_mbuf_verify(pinned)" " failed\n", __func__); if (test_mbuf_linearize_check(pinned_pool) < 0) @@ -2859,8 +2859,8 @@ test_mbuf(void) goto err; } - if (test_failing_mbuf_sanity_check(pktmbuf_pool) < 0) { - printf("test_failing_mbuf_sanity_check() failed\n"); + if (test_failing_mbuf_verify(pktmbuf_pool) < 0) { + printf("test_failing_mbuf_verify() failed\n"); goto err; } diff --git a/doc/guides/prog_guide/mbuf_lib.rst b/doc/guides/prog_guide/mbuf_lib.rst index 049357c75563..0accb51a98c7 100644 --- a/doc/guides/prog_guide/mbuf_lib.rst +++ b/doc/guides/prog_guide/mbuf_lib.rst @@ -266,8 +266,8 @@ can be found in several of the sample applications, for example, the IPv4 Multic Debug ----- -In debug mode, the functions of the mbuf library perform sanity checks before any operation (such as, buffer corruption, -bad type, and so on). +In debug mode, the functions of the mbuf library perform consistency checks +before any operation (such as, buffer corruption, bad type, and so on). Use Cases --------- diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index 81b93515cbd9..1e1544b5b644 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/guides/rel_notes/deprecation.rst @@ -146,3 +146,6 @@ Deprecation Notices will be deprecated and subsequently removed in DPDK 24.11 release. Before this, the new port library API (functions rte_swx_port_*) will gradually transition from experimental to stable status. + +* mbuf: The function ``rte_mbuf_sanity_check`` is deprecated. + Use the new function ``rte_mbuf_verify`` instead. diff --git a/drivers/net/avp/avp_ethdev.c b/drivers/net/avp/avp_ethdev.c index 53d9e38c939b..ae76fad84948 100644 --- a/drivers/net/avp/avp_ethdev.c +++ b/drivers/net/avp/avp_ethdev.c @@ -1231,7 +1231,7 @@ _avp_mac_filter(struct avp_dev *avp, struct rte_mbuf *m) #ifdef RTE_LIBRTE_AVP_DEBUG_BUFFERS static inline void -__avp_dev_buffer_sanity_check(struct avp_dev *avp, struct rte_avp_desc *buf) +__avp_dev_buffer_check(struct avp_dev *avp, struct rte_avp_desc *buf) { struct rte_avp_desc *first_buf; struct rte_avp_desc *pkt_buf; @@ -1272,12 +1272,12 @@ __avp_dev_buffer_sanity_check(struct avp_dev *avp, struct rte_avp_desc *buf) first_buf->pkt_len, pkt_len); } -#define avp_dev_buffer_sanity_check(a, b) \ - __avp_dev_buffer_sanity_check((a), (b)) +#define avp_dev_buffer_check(a, b) \ + __avp_dev_buffer_check((a), (b)) #else /* RTE_LIBRTE_AVP_DEBUG_BUFFERS */ -#define avp_dev_buffer_sanity_check(a, b) do {} while (0) +#define avp_dev_buffer_check(a, b) do {} while (0) #endif @@ -1302,7 +1302,7 @@ avp_dev_copy_from_buffers(struct avp_dev *avp, void *pkt_data; unsigned int i; - avp_dev_buffer_sanity_check(avp, buf); + avp_dev_buffer_check(avp, buf); /* setup the first source buffer */ pkt_buf = avp_dev_translate_buffer(avp, buf); @@ -1370,7 +1370,7 @@ avp_dev_copy_from_buffers(struct avp_dev *avp, rte_pktmbuf_pkt_len(m) = total_length; m->vlan_tci = vlan_tci; - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); return m; } @@ -1614,7 +1614,7 @@ avp_dev_copy_to_buffers(struct avp_dev *avp, char *pkt_data; unsigned int i; - __rte_mbuf_sanity_check(mbuf, 1); + __rte_mbuf_verify(mbuf, 1); m = mbuf; src_offset = 0; @@ -1680,7 +1680,7 @@ avp_dev_copy_to_buffers(struct avp_dev *avp, first_buf->vlan_tci = mbuf->vlan_tci; } - avp_dev_buffer_sanity_check(avp, buffers[0]); + avp_dev_buffer_check(avp, buffers[0]); return total_length; } @@ -1798,7 +1798,7 @@ avp_xmit_scattered_pkts(void *tx_queue, #ifdef RTE_LIBRTE_AVP_DEBUG_BUFFERS for (i = 0; i < nb_pkts; i++) - avp_dev_buffer_sanity_check(avp, tx_bufs[i]); + avp_dev_buffer_check(avp, tx_bufs[i]); #endif /* send the packets */ diff --git a/drivers/net/sfc/sfc_ef100_rx.c b/drivers/net/sfc/sfc_ef100_rx.c index 2677003da326..8199b56f2740 100644 --- a/drivers/net/sfc/sfc_ef100_rx.c +++ b/drivers/net/sfc/sfc_ef100_rx.c @@ -179,7 +179,7 @@ sfc_ef100_rx_qrefill(struct sfc_ef100_rxq *rxq) struct sfc_ef100_rx_sw_desc *rxd; rte_iova_t dma_addr; - __rte_mbuf_raw_sanity_check(m); + __rte_mbuf_raw_verify(m); dma_addr = rte_mbuf_data_iova_default(m); if (rxq->flags & SFC_EF100_RXQ_NIC_DMA_MAP) { @@ -551,7 +551,7 @@ sfc_ef100_rx_process_ready_pkts(struct sfc_ef100_rxq *rxq, rxq->ready_pkts--; pkt = sfc_ef100_rx_next_mbuf(rxq); - __rte_mbuf_raw_sanity_check(pkt); + __rte_mbuf_raw_verify(pkt); RTE_BUILD_BUG_ON(sizeof(pkt->rearm_data[0]) != sizeof(rxq->rearm_data)); @@ -575,7 +575,7 @@ sfc_ef100_rx_process_ready_pkts(struct sfc_ef100_rxq *rxq, struct rte_mbuf *seg; seg = sfc_ef100_rx_next_mbuf(rxq); - __rte_mbuf_raw_sanity_check(seg); + __rte_mbuf_raw_verify(seg); seg->data_off = RTE_PKTMBUF_HEADROOM; diff --git a/drivers/net/sfc/sfc_ef10_essb_rx.c b/drivers/net/sfc/sfc_ef10_essb_rx.c index 78bd430363b1..74647e2792b1 100644 --- a/drivers/net/sfc/sfc_ef10_essb_rx.c +++ b/drivers/net/sfc/sfc_ef10_essb_rx.c @@ -125,7 +125,7 @@ sfc_ef10_essb_next_mbuf(const struct sfc_ef10_essb_rxq *rxq, struct rte_mbuf *m; m = (struct rte_mbuf *)((uintptr_t)mbuf + rxq->buf_stride); - __rte_mbuf_raw_sanity_check(m); + __rte_mbuf_raw_verify(m); return m; } @@ -136,7 +136,7 @@ sfc_ef10_essb_mbuf_by_index(const struct sfc_ef10_essb_rxq *rxq, struct rte_mbuf *m; m = (struct rte_mbuf *)((uintptr_t)mbuf + idx * rxq->buf_stride); - __rte_mbuf_raw_sanity_check(m); + __rte_mbuf_raw_verify(m); return m; } diff --git a/drivers/net/sfc/sfc_ef10_rx.c b/drivers/net/sfc/sfc_ef10_rx.c index 30a320d0791c..72b03b3bba7a 100644 --- a/drivers/net/sfc/sfc_ef10_rx.c +++ b/drivers/net/sfc/sfc_ef10_rx.c @@ -148,7 +148,7 @@ sfc_ef10_rx_qrefill(struct sfc_ef10_rxq *rxq) struct sfc_ef10_rx_sw_desc *rxd; rte_iova_t phys_addr; - __rte_mbuf_raw_sanity_check(m); + __rte_mbuf_raw_verify(m); SFC_ASSERT((id & ~ptr_mask) == 0); rxd = &rxq->sw_ring[id]; @@ -297,7 +297,7 @@ sfc_ef10_rx_process_event(struct sfc_ef10_rxq *rxq, efx_qword_t rx_ev, rxd = &rxq->sw_ring[pending++ & ptr_mask]; m = rxd->mbuf; - __rte_mbuf_raw_sanity_check(m); + __rte_mbuf_raw_verify(m); m->data_off = RTE_PKTMBUF_HEADROOM; rte_pktmbuf_data_len(m) = seg_len; diff --git a/drivers/net/sfc/sfc_rx.c b/drivers/net/sfc/sfc_rx.c index 1dde2c111001..645c8643d1c1 100644 --- a/drivers/net/sfc/sfc_rx.c +++ b/drivers/net/sfc/sfc_rx.c @@ -120,7 +120,7 @@ sfc_efx_rx_qrefill(struct sfc_efx_rxq *rxq) ++i, id = (id + 1) & rxq->ptr_mask) { m = objs[i]; - __rte_mbuf_raw_sanity_check(m); + __rte_mbuf_raw_verify(m); rxd = &rxq->sw_desc[id]; rxd->mbuf = m; diff --git a/examples/ipv4_multicast/main.c b/examples/ipv4_multicast/main.c index 6d0a8501eff5..f39658f4e249 100644 --- a/examples/ipv4_multicast/main.c +++ b/examples/ipv4_multicast/main.c @@ -258,7 +258,7 @@ mcast_out_pkt(struct rte_mbuf *pkt, int use_clone) hdr->pkt_len = (uint16_t)(hdr->data_len + pkt->pkt_len); hdr->nb_segs = pkt->nb_segs + 1; - __rte_mbuf_sanity_check(hdr, 1); + __rte_mbuf_verify(hdr, 1); return hdr; } /* >8 End of mcast_out_kt. */ diff --git a/lib/mbuf/rte_mbuf.c b/lib/mbuf/rte_mbuf.c index 559d5ad8a71c..fc5d4ba29db1 100644 --- a/lib/mbuf/rte_mbuf.c +++ b/lib/mbuf/rte_mbuf.c @@ -367,9 +367,9 @@ rte_pktmbuf_pool_create_extbuf(const char *name, unsigned int n, return mp; } -/* do some sanity checks on a mbuf: panic if it fails */ +/* do some checks on a mbuf: panic if it fails */ void -rte_mbuf_sanity_check(const struct rte_mbuf *m, int is_header) +rte_mbuf_verify(const struct rte_mbuf *m, int is_header) { const char *reason; @@ -377,6 +377,13 @@ rte_mbuf_sanity_check(const struct rte_mbuf *m, int is_header) rte_panic("%s\n", reason); } +/* For ABI compatibility, to be removed in next release */ +void +rte_mbuf_sanity_check(const struct rte_mbuf *m, int is_header) +{ + rte_mbuf_verify(m, is_header); +} + int rte_mbuf_check(const struct rte_mbuf *m, int is_header, const char **reason) { @@ -496,7 +503,7 @@ void rte_pktmbuf_free_bulk(struct rte_mbuf **mbufs, unsigned int count) if (unlikely(m == NULL)) continue; - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); do { m_next = m->next; @@ -546,7 +553,7 @@ rte_pktmbuf_clone(struct rte_mbuf *md, struct rte_mempool *mp) return NULL; } - __rte_mbuf_sanity_check(mc, 1); + __rte_mbuf_verify(mc, 1); return mc; } @@ -596,7 +603,7 @@ rte_pktmbuf_copy(const struct rte_mbuf *m, struct rte_mempool *mp, struct rte_mbuf *mc, *m_last, **prev; /* garbage in check */ - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); /* check for request to copy at offset past end of mbuf */ if (unlikely(off >= m->pkt_len)) @@ -660,7 +667,7 @@ rte_pktmbuf_copy(const struct rte_mbuf *m, struct rte_mempool *mp, } /* garbage out check */ - __rte_mbuf_sanity_check(mc, 1); + __rte_mbuf_verify(mc, 1); return mc; } @@ -671,7 +678,7 @@ rte_pktmbuf_dump(FILE *f, const struct rte_mbuf *m, unsigned dump_len) unsigned int len; unsigned int nb_segs; - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); fprintf(f, "dump mbuf at %p, iova=%#" PRIx64 ", buf_len=%u\n", m, rte_mbuf_iova_get(m), m->buf_len); @@ -689,7 +696,7 @@ rte_pktmbuf_dump(FILE *f, const struct rte_mbuf *m, unsigned dump_len) nb_segs = m->nb_segs; while (m && nb_segs != 0) { - __rte_mbuf_sanity_check(m, 0); + __rte_mbuf_verify(m, 0); fprintf(f, " segment at %p, data=%p, len=%u, off=%u, refcnt=%u\n", m, rte_pktmbuf_mtod(m, void *), diff --git a/lib/mbuf/rte_mbuf.h b/lib/mbuf/rte_mbuf.h index 286b32b788a5..380663a0893b 100644 --- a/lib/mbuf/rte_mbuf.h +++ b/lib/mbuf/rte_mbuf.h @@ -339,13 +339,13 @@ rte_pktmbuf_priv_flags(struct rte_mempool *mp) #ifdef RTE_LIBRTE_MBUF_DEBUG -/** check mbuf type in debug mode */ -#define __rte_mbuf_sanity_check(m, is_h) rte_mbuf_sanity_check(m, is_h) +/** do mbuf type in debug mode */ +#define __rte_mbuf_verify(m, is_h) rte_mbuf_verify(m, is_h) #else /* RTE_LIBRTE_MBUF_DEBUG */ -/** check mbuf type in debug mode */ -#define __rte_mbuf_sanity_check(m, is_h) do { } while (0) +/** ignore mbuf checks if not in debug mode */ +#define __rte_mbuf_verify(m, is_h) do { } while (0) #endif /* RTE_LIBRTE_MBUF_DEBUG */ @@ -514,10 +514,9 @@ rte_mbuf_ext_refcnt_update(struct rte_mbuf_ext_shared_info *shinfo, /** - * Sanity checks on an mbuf. + * Check that the mbuf is valid and panic if corrupted. * - * Check the consistency of the given mbuf. The function will cause a - * panic if corruption is detected. + * Acts assertion that mbuf is consistent. If not it calls rte_panic(). * * @param m * The mbuf to be checked. @@ -526,13 +525,17 @@ rte_mbuf_ext_refcnt_update(struct rte_mbuf_ext_shared_info *shinfo, * of a packet (in this case, some fields like nb_segs are not checked) */ void +rte_mbuf_verify(const struct rte_mbuf *m, int is_header); + +/* Older deprecated name for rte_mbuf_verify() */ +void __rte_deprecated rte_mbuf_sanity_check(const struct rte_mbuf *m, int is_header); /** - * Sanity checks on a mbuf. + * Do consistency checks on a mbuf. * - * Almost like rte_mbuf_sanity_check(), but this function gives the reason - * if corruption is detected rather than panic. + * Check the consistency of the given mbuf and if not valid + * return the reason. * * @param m * The mbuf to be checked. @@ -551,7 +554,7 @@ int rte_mbuf_check(const struct rte_mbuf *m, int is_header, const char **reason); /** - * Sanity checks on a reinitialized mbuf in debug mode. + * Do checks on a reinitialized mbuf in debug mode. * * Check the consistency of the given reinitialized mbuf. * The function will cause a panic if corruption is detected. @@ -563,16 +566,16 @@ int rte_mbuf_check(const struct rte_mbuf *m, int is_header, * The mbuf to be checked. */ static __rte_always_inline void -__rte_mbuf_raw_sanity_check(__rte_unused const struct rte_mbuf *m) +__rte_mbuf_raw_verify(__rte_unused const struct rte_mbuf *m) { RTE_ASSERT(rte_mbuf_refcnt_read(m) == 1); RTE_ASSERT(m->next == NULL); RTE_ASSERT(m->nb_segs == 1); - __rte_mbuf_sanity_check(m, 0); + __rte_mbuf_verify(m, 0); } /** For backwards compatibility. */ -#define MBUF_RAW_ALLOC_CHECK(m) __rte_mbuf_raw_sanity_check(m) +#define MBUF_RAW_ALLOC_CHECK(m) __rte_mbuf_raw_verify(m) /** * Allocate an uninitialized mbuf from mempool *mp*. @@ -599,7 +602,7 @@ static inline struct rte_mbuf *rte_mbuf_raw_alloc(struct rte_mempool *mp) if (rte_mempool_get(mp, (void **)&m) < 0) return NULL; - __rte_mbuf_raw_sanity_check(m); + __rte_mbuf_raw_verify(m); return m; } @@ -622,7 +625,7 @@ rte_mbuf_raw_free(struct rte_mbuf *m) { RTE_ASSERT(!RTE_MBUF_CLONED(m) && (!RTE_MBUF_HAS_EXTBUF(m) || RTE_MBUF_HAS_PINNED_EXTBUF(m))); - __rte_mbuf_raw_sanity_check(m); + __rte_mbuf_raw_verify(m); rte_mempool_put(m->pool, m); } @@ -885,7 +888,7 @@ static inline void rte_pktmbuf_reset(struct rte_mbuf *m) rte_pktmbuf_reset_headroom(m); m->data_len = 0; - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); } /** @@ -941,22 +944,22 @@ static inline int rte_pktmbuf_alloc_bulk(struct rte_mempool *pool, switch (count % 4) { case 0: while (idx != count) { - __rte_mbuf_raw_sanity_check(mbufs[idx]); + __rte_mbuf_raw_verify(mbufs[idx]); rte_pktmbuf_reset(mbufs[idx]); idx++; /* fall-through */ case 3: - __rte_mbuf_raw_sanity_check(mbufs[idx]); + __rte_mbuf_raw_verify(mbufs[idx]); rte_pktmbuf_reset(mbufs[idx]); idx++; /* fall-through */ case 2: - __rte_mbuf_raw_sanity_check(mbufs[idx]); + __rte_mbuf_raw_verify(mbufs[idx]); rte_pktmbuf_reset(mbufs[idx]); idx++; /* fall-through */ case 1: - __rte_mbuf_raw_sanity_check(mbufs[idx]); + __rte_mbuf_raw_verify(mbufs[idx]); rte_pktmbuf_reset(mbufs[idx]); idx++; /* fall-through */ @@ -1184,8 +1187,8 @@ static inline void rte_pktmbuf_attach(struct rte_mbuf *mi, struct rte_mbuf *m) mi->pkt_len = mi->data_len; mi->nb_segs = 1; - __rte_mbuf_sanity_check(mi, 1); - __rte_mbuf_sanity_check(m, 0); + __rte_mbuf_verify(mi, 1); + __rte_mbuf_verify(m, 0); } /** @@ -1340,7 +1343,7 @@ static inline int __rte_pktmbuf_pinned_extbuf_decref(struct rte_mbuf *m) static __rte_always_inline struct rte_mbuf * rte_pktmbuf_prefree_seg(struct rte_mbuf *m) { - __rte_mbuf_sanity_check(m, 0); + __rte_mbuf_verify(m, 0); if (likely(rte_mbuf_refcnt_read(m) == 1)) { @@ -1411,7 +1414,7 @@ static inline void rte_pktmbuf_free(struct rte_mbuf *m) struct rte_mbuf *m_next; if (m != NULL) - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); while (m != NULL) { m_next = m->next; @@ -1492,7 +1495,7 @@ rte_pktmbuf_copy(const struct rte_mbuf *m, struct rte_mempool *mp, */ static inline void rte_pktmbuf_refcnt_update(struct rte_mbuf *m, int16_t v) { - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); do { rte_mbuf_refcnt_update(m, v); @@ -1509,7 +1512,7 @@ static inline void rte_pktmbuf_refcnt_update(struct rte_mbuf *m, int16_t v) */ static inline uint16_t rte_pktmbuf_headroom(const struct rte_mbuf *m) { - __rte_mbuf_sanity_check(m, 0); + __rte_mbuf_verify(m, 0); return m->data_off; } @@ -1523,7 +1526,7 @@ static inline uint16_t rte_pktmbuf_headroom(const struct rte_mbuf *m) */ static inline uint16_t rte_pktmbuf_tailroom(const struct rte_mbuf *m) { - __rte_mbuf_sanity_check(m, 0); + __rte_mbuf_verify(m, 0); return (uint16_t)(m->buf_len - rte_pktmbuf_headroom(m) - m->data_len); } @@ -1538,7 +1541,7 @@ static inline uint16_t rte_pktmbuf_tailroom(const struct rte_mbuf *m) */ static inline struct rte_mbuf *rte_pktmbuf_lastseg(struct rte_mbuf *m) { - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); while (m->next != NULL) m = m->next; return m; @@ -1582,7 +1585,7 @@ static inline struct rte_mbuf *rte_pktmbuf_lastseg(struct rte_mbuf *m) static inline char *rte_pktmbuf_prepend(struct rte_mbuf *m, uint16_t len) { - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); if (unlikely(len > rte_pktmbuf_headroom(m))) return NULL; @@ -1617,7 +1620,7 @@ static inline char *rte_pktmbuf_append(struct rte_mbuf *m, uint16_t len) void *tail; struct rte_mbuf *m_last; - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); m_last = rte_pktmbuf_lastseg(m); if (unlikely(len > rte_pktmbuf_tailroom(m_last))) @@ -1645,7 +1648,7 @@ static inline char *rte_pktmbuf_append(struct rte_mbuf *m, uint16_t len) */ static inline char *rte_pktmbuf_adj(struct rte_mbuf *m, uint16_t len) { - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); if (unlikely(len > m->data_len)) return NULL; @@ -1677,7 +1680,7 @@ static inline int rte_pktmbuf_trim(struct rte_mbuf *m, uint16_t len) { struct rte_mbuf *m_last; - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); m_last = rte_pktmbuf_lastseg(m); if (unlikely(len > m_last->data_len)) @@ -1699,7 +1702,7 @@ static inline int rte_pktmbuf_trim(struct rte_mbuf *m, uint16_t len) */ static inline int rte_pktmbuf_is_contiguous(const struct rte_mbuf *m) { - __rte_mbuf_sanity_check(m, 1); + __rte_mbuf_verify(m, 1); return m->nb_segs == 1; } diff --git a/lib/mbuf/version.map b/lib/mbuf/version.map index daa65e2bbdb2..c85370e430b2 100644 --- a/lib/mbuf/version.map +++ b/lib/mbuf/version.map @@ -31,6 +31,7 @@ DPDK_24 { rte_mbuf_set_platform_mempool_ops; rte_mbuf_set_user_mempool_ops; rte_mbuf_user_mempool_ops; + rte_mbuf_verify; rte_pktmbuf_clone; rte_pktmbuf_copy; rte_pktmbuf_dump; From patchwork Mon Feb 5 17:43:34 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136393 X-Patchwork-Delegate: thomas@monjalon.net 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 A72D443A3D; Mon, 5 Feb 2024 19:04:30 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BF53D40ED8; Mon, 5 Feb 2024 19:03:47 +0100 (CET) Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) by mails.dpdk.org (Postfix) with ESMTP id DC88340269 for ; Mon, 5 Feb 2024 19:03:44 +0100 (CET) Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-1d958e0d73dso27441465ad.1 for ; Mon, 05 Feb 2024 10:03:44 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156224; x=1707761024; darn=dpdk.org; 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=ApPm79+YUtT09EP+FY3CN8Jd9NvPoernSgFeoMJrzko=; b=YTY7Nb/lM8AWmgehOyuoZoLILsBPLHOfivJeCE6XvWfinqytVQ0bXQtfW8pFTUZaLM 3kDjb6nt5DZYieoPAL4Ix1Ofoq4baR7wyuCgCIU+9eEsNWyLVpeuxxTyS387xMUVI+NQ jDwAMLjKzJTnDZew1R7gMCR2iHmZFt7Ds4OcUh9lTiLmw/mN7TilvflvQ7lQEOhfRdhy lVjYqFPsEdbQuzSK0022FlvvqAaq4oc9XsntB7+C/i11RnJmF8i03OSb1mSYJ99UevFP HyeY1yUipdnPEzXpnRgG2724aZVkgBh15cDXvF9ycPRkIdphF6jS2eC1AS64ikxlbUiX ssBA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156224; x=1707761024; 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=ApPm79+YUtT09EP+FY3CN8Jd9NvPoernSgFeoMJrzko=; b=AWYFNg7FLnq9bHjnDcz3ecnRB2YpC4U1wUL+wpDzKet/+5g9Csmpn0PFXW5zacDCd0 dJqgyptJyS4s8eCIrryqQndTQrdh9HGQ2e79u67OWqcxESItF65wOJSZCp1caX/ADAVu bxVuNnbvYROI4IwbjKLZD2zV7mw7hrIG1q8ie4Jn4ONQUR/Xjxkj6s7duo1vLooH4/IB P8+OLLXbZRTsHqbcwpux5cZYtYaBlmXMDdb+bvHvUCxh6e3iDkeGJ/cj9AbkyYjCiqtq 7blMJqf6e9A1zcy1x00h1IjUf/akyIdT2n1CGdkmFcH/d3sArc4OcZtPOy47rHZFPbie zacQ== X-Gm-Message-State: AOJu0YxJt6c/yAM+RHcWvD2/nDDuNNEcRF/GrPS1uBPW6+s8vqJqWWXS dxtL5uNCU4n1mJwdz/D4TjHLKqSTichnVYI10Bdi4BwihsR6VUKJcUWppuCdRHQRmI/T+r6f9q4 b3fM= X-Google-Smtp-Source: AGHT+IFj7BcOaA2GeW2bEZq9MYJr9R0b+pHl8OajeFJvEpmbWqGVdl8IDZewrhEyYuB+PWtppROXcg== X-Received: by 2002:a17:902:fc90:b0:1d7:17e6:44fb with SMTP id mf16-20020a170902fc9000b001d717e644fbmr7334960plb.32.1707156224065; Mon, 05 Feb 2024 10:03:44 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCVyqY4nkmaF49BRyPGLnJGfAfZa17LrDGHxYH7t+E3OZCOa/0u8dbeBw3SU770iTgmbbUqkyCc0+p0NaNsrtvQFcTkx1dsxC2f39uZ+vQ3zWRuQxVgX5DIcgad6EhLrrP7RhCWrjLbjWkld8YQkSAkkLBH359euMN69dqvTOty4E0qmBsRvX80TWCoNcDecy+7+r6vKR83Xs8TlDk2uTwg5pizaLq93m+7xQU8H4YiAeTZYGc8H+FxQ3RziLM++DnWGdn8qNNd3w/sJ+gjaY+C+pLsN9y5US0ufUX4BmQ== Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:43 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Anatoly Burakov , Jerin Jacob , Sunil Kumar Kori , Bruce Richardson , Dmitry Kozlyuk , Tyler Retzlaff , Pallavi Kadam Subject: [PATCH v9 06/23] eal: replace use of sanity check in comments and messages Date: Mon, 5 Feb 2024 09:43:34 -0800 Message-ID: <20240205180328.131019-7-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Sanity check is on the Tier 2 non-inclusive list. Replace or remove it. Signed-off-by: Stephen Hemminger Acked-by: Anatoly Burakov --- lib/eal/common/eal_common_memory.c | 2 +- lib/eal/common/eal_common_proc.c | 3 ++- lib/eal/common/eal_common_trace.c | 2 +- lib/eal/common/eal_memcfg.h | 2 +- lib/eal/common/rte_malloc.c | 2 +- lib/eal/freebsd/eal.c | 2 +- lib/eal/linux/eal.c | 2 +- lib/eal/windows/eal.c | 2 +- 8 files changed, 9 insertions(+), 8 deletions(-) diff --git a/lib/eal/common/eal_common_memory.c b/lib/eal/common/eal_common_memory.c index 60ddc30580a7..9f4a8676f18f 100644 --- a/lib/eal/common/eal_common_memory.c +++ b/lib/eal/common/eal_common_memory.c @@ -562,7 +562,7 @@ check_dma_mask(uint8_t maskbits, bool thread_unsafe) uint64_t mask; int ret; - /* Sanity check. We only check width can be managed with 64 bits + /* We only check width can be managed with 64 bits * variables. Indeed any higher value is likely wrong. */ if (maskbits > MAX_DMA_MASK_BITS) { EAL_LOG(ERR, "wrong dma mask size %u (Max: %u)", diff --git a/lib/eal/common/eal_common_proc.c b/lib/eal/common/eal_common_proc.c index d24093937c1d..838602edef08 100644 --- a/lib/eal/common/eal_common_proc.c +++ b/lib/eal/common/eal_common_proc.c @@ -309,7 +309,8 @@ read_msg(int fd, struct mp_msg_internal *m, struct sockaddr_un *s) break; } } - /* sanity-check the response */ + + /* Check that the response is valid */ if (m->msg.num_fds < 0 || m->msg.num_fds > RTE_MP_MAX_FD_NUM) { EAL_LOG(ERR, "invalid number of fd's received"); return -1; diff --git a/lib/eal/common/eal_common_trace.c b/lib/eal/common/eal_common_trace.c index 6ad87fc0df5d..b2bab5c45f1e 100644 --- a/lib/eal/common/eal_common_trace.c +++ b/lib/eal/common/eal_common_trace.c @@ -468,7 +468,7 @@ __rte_trace_point_register(rte_trace_point_t *handle, const char *name, struct trace_point *tp; uint16_t sz; - /* Sanity checks of arguments */ + /* Check arguments */ if (name == NULL || register_fn == NULL || handle == NULL) { trace_err("invalid arguments"); rte_errno = EINVAL; diff --git a/lib/eal/common/eal_memcfg.h b/lib/eal/common/eal_memcfg.h index 60e2089797a2..e8a85df35585 100644 --- a/lib/eal/common/eal_memcfg.h +++ b/lib/eal/common/eal_memcfg.h @@ -18,7 +18,7 @@ * Memory configuration shared across multiple processes. */ struct rte_mem_config { - volatile uint32_t magic; /**< Magic number - sanity check. */ + volatile uint32_t magic; /**< Magic number check. */ uint32_t version; /**< Prevent secondary processes using different DPDK versions. */ diff --git a/lib/eal/common/rte_malloc.c b/lib/eal/common/rte_malloc.c index 6d3c301a2368..dac077bc8e09 100644 --- a/lib/eal/common/rte_malloc.c +++ b/lib/eal/common/rte_malloc.c @@ -654,7 +654,7 @@ rte_malloc_heap_destroy(const char *heap_name) ret = -1; goto unlock; } - /* sanity checks done, now we can destroy the heap */ + /* checks done, now we can destroy the heap */ rte_spinlock_lock(&heap->lock); ret = malloc_heap_destroy(heap); rte_spinlock_unlock(&heap->lock); diff --git a/lib/eal/freebsd/eal.c b/lib/eal/freebsd/eal.c index bab77118e967..1500778b4f06 100644 --- a/lib/eal/freebsd/eal.c +++ b/lib/eal/freebsd/eal.c @@ -501,7 +501,7 @@ eal_parse_args(int argc, char **argv) goto out; } - /* sanity checks */ + /* options checks */ if (eal_check_common_options(internal_conf) != 0) { eal_usage(prgname); ret = -1; diff --git a/lib/eal/linux/eal.c b/lib/eal/linux/eal.c index fd422f1f6236..50cebdbbd481 100644 --- a/lib/eal/linux/eal.c +++ b/lib/eal/linux/eal.c @@ -803,7 +803,7 @@ eal_parse_args(int argc, char **argv) goto out; } - /* sanity checks */ + /* options checks */ if (eal_check_common_options(internal_conf) != 0) { eal_usage(prgname); ret = -1; diff --git a/lib/eal/windows/eal.c b/lib/eal/windows/eal.c index 52f0e7462dd8..915c8e569a5d 100644 --- a/lib/eal/windows/eal.c +++ b/lib/eal/windows/eal.c @@ -194,7 +194,7 @@ eal_parse_args(int argc, char **argv) if (eal_adjust_config(internal_conf) != 0) return -1; - /* sanity checks */ + /* options checks */ if (eal_check_common_options(internal_conf) != 0) { eal_usage(prgname); return -1; From patchwork Mon Feb 5 17:43:35 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136394 X-Patchwork-Delegate: thomas@monjalon.net 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 13A7043A3D; Mon, 5 Feb 2024 19:04:40 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1E57F40FDE; Mon, 5 Feb 2024 19:03:49 +0100 (CET) Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) by mails.dpdk.org (Postfix) with ESMTP id 3700140E64 for ; Mon, 5 Feb 2024 19:03:46 +0100 (CET) Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-1d7431e702dso39098135ad.1 for ; Mon, 05 Feb 2024 10:03:46 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156225; x=1707761025; darn=dpdk.org; 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=cb6D46cPIDtlgwZOSFszjLYKu/nVkfZBGzoewkKXbPg=; b=Z4YRt7P321twXzFn/pL1KqfPh25lhD/bsi2nJriuLNeBqjs/EOzQqU6ZCph+mKwrH4 VbbuleU5Dp46Y99GCJcWLSAmcrXs5yw4gsnJPZUQcIq22xd8hpqpou0nHqwItFLtPcKK 1TEqtYO2IfyZJSEH4L8sxS6VOYY5AQfEQ5Gvn9+IKCkFtSDjRouKnjBNl18Uhlk7UCHm 07EimiTVMCU7gEXGm4gYIYIHAQTGrLhweXARY0C2oSQacf5OruoQJX5Hq3j4O7HC4UHd 6QdxOkEHLdyYc9UY4Is05p2DZUR08jKVcciN8aKVfLa7zlpBcu3N4t4KiF+8R/97hOwg iqMw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156225; x=1707761025; 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=cb6D46cPIDtlgwZOSFszjLYKu/nVkfZBGzoewkKXbPg=; b=WO6LENxtOXu05zhBZmaS1CRDNdjmxHQ2fDnqyb/NFC9fsjlLrAmhtK/gbN5FMGuQMb T+Lf9Qvzx3YiZA1JUcrVCqqs89xsT6tUuh1uRSeTA4wkvUogv46qvHBu4pf9TRkolkFy qasa4HEISQsQMlaSHWNhC9e2eXwe+nltbTxVevO63u/hjIM0fJv/J6qz8pn7bj10QjeQ kJStZHyTc7gbD7NIXgpSA96bJU2tRhnjYyTeiC0v6jeowO/K57GVgqrNzOcgdWMCJxL0 sQVlVDzgNW1n/SzWUDcL29VM5HkDL+VHLpx580w/4dUIBPg7vC7COdMYMSPsPFcjsbye ynMg== X-Gm-Message-State: AOJu0Yxnt9FO4WB8FjEPPwfUTecBWHicaWmCzNyNWnWJ/lFn/iQKOyuC 5eFfxujpkWQ+jcopKTaq5iIcRFUfDsT+HUmWPXzeHJ4iFfOX5dwPZCNEgXmlpdiFOag5AykbFI5 rsQc= X-Google-Smtp-Source: AGHT+IFCSXZmWnx+jb5jkHV5NwRURDUb6aV8P0N92zSWyLXU7muE/cIezknqC92/Go4nH9es+776Hw== X-Received: by 2002:a17:902:d34d:b0:1d9:a825:19cf with SMTP id l13-20020a170902d34d00b001d9a82519cfmr268105plk.32.1707156225302; Mon, 05 Feb 2024 10:03:45 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCUtsnieAD3Vt3OQ2radJ3Zvz66I5uxQJmQMINi31tPqo/vv1qldyPODOzwhVfmg3dpwPdx/Q3sirQ3US5iamlMxtlRmoylp0owbJKiNW/99AP63br6CLtr8DKzsFrUHzkmXLS0eYHd2kI2QvYFCvXi6DG/7vgvX9XsgIPnOvJviKavmFTOSj6tLw8qKgF/U3aE8m8wFwgmU+4G9o7xXf2n6Ynet98Hx2zE4Niv8Rrcgx9kBrUcxQA91/sh+8mrPGL4nymKYLwoX5Z6wL7jUUetV1Jd5HCMwqUUMZqUmIY/Y300meB4o/P1Ork9rUxJAOwoTwvgJ9FA9nCev6gqg+OBEugqHS093CDGQwx843GFq/GTIm/ZIWP7jJUIAoYDRfYFpox7dx3oCZ85NeEACz4vQkzVVa6p6hgY11q440PTTp/qIqPMVrenwKvrNwZfV2LH9fTTgy7sy72DOX4bEx3N2MXXL5OWIKQWMr8YfhzNEg2IbvqbAJFvN4vqJCighOmPTEhxLIVlMysfQ8yXM+D0CugiC8FWwD49fZJmGSpUL Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:44 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Cristian Dumitrescu , Konstantin Ananyev , David Hunt , Chengwen Feng , Kevin Laatz , Bruce Richardson , Yipeng Wang , Sameh Gobriel , Vladimir Medvedkin , Harman Kalra , Chas Williams , "Min Hu (Connor)" , Sachin Saxena , Hemant Agrawal , Erik Gabriel Carrillo Subject: [PATCH v9 07/23] test: replace use word sanity Date: Mon, 5 Feb 2024 09:43:35 -0800 Message-ID: <20240205180328.131019-8-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 word "sanity" is on the not-allowed inclusive naming list. Replace the unnecessary usage in tests. Signed-off-by: Stephen Hemminger --- app/test/test_bitmap.c | 4 +-- app/test/test_bpf.c | 6 ++-- app/test/test_common.c | 2 +- app/test/test_distributor.c | 46 +++++++++++++++--------------- app/test/test_dmadev.c | 2 +- app/test/test_eal_flags.c | 20 ++++++------- app/test/test_hash.c | 2 +- app/test/test_interrupts.c | 9 +++--- app/test/test_link_bonding_mode4.c | 2 +- app/test/test_mbuf.c | 2 +- app/test/test_rawdev.c | 2 +- app/test/test_timer.c | 10 +++---- 12 files changed, 54 insertions(+), 53 deletions(-) diff --git a/app/test/test_bitmap.c b/app/test/test_bitmap.c index bab11812c767..a074b64f2e98 100644 --- a/app/test/test_bitmap.c +++ b/app/test/test_bitmap.c @@ -33,7 +33,7 @@ test_bitmap_scan_operations(struct rte_bitmap *bmp) } if (slab1_magic != out_slab) { - printf("Scan operation sanity failed.\n"); + printf("Scan operation magic number check failed.\n"); return TEST_FAILED; } @@ -43,7 +43,7 @@ test_bitmap_scan_operations(struct rte_bitmap *bmp) } if (slab2_magic != out_slab) { - printf("Scan operation sanity failed.\n"); + printf("Scan operation magic number check failed.\n"); return TEST_FAILED; } diff --git a/app/test/test_bpf.c b/app/test/test_bpf.c index f83e72a9adfd..ffadcece2212 100644 --- a/app/test/test_bpf.c +++ b/app/test/test_bpf.c @@ -3329,9 +3329,9 @@ test_bpf_match(pcap_t *pcap, const char *str, return ret; } -/* Basic sanity test can we match a IP packet */ +/* Basic test can we match a IP packet */ static int -test_bpf_filter_sanity(pcap_t *pcap) +test_bpf_filter_match(pcap_t *pcap) { const uint32_t plen = 100; struct rte_mbuf mb, *m; @@ -3463,7 +3463,7 @@ test_bpf_convert(void) return -1; } - rc = test_bpf_filter_sanity(pcap); + rc = test_bpf_filter_match(pcap); for (i = 0; i < RTE_DIM(sample_filters); i++) rc |= test_bpf_filter(pcap, sample_filters[i]); diff --git a/app/test/test_common.c b/app/test/test_common.c index 21eb2285e1b2..784cdaf3d640 100644 --- a/app/test/test_common.c +++ b/app/test/test_common.c @@ -19,7 +19,7 @@ {printf(x "() test failed!\n");\ return -1;} -/* this is really a sanity check */ +/* this is really a consistency check */ static int test_macros(int __rte_unused unused_parm) { diff --git a/app/test/test_distributor.c b/app/test/test_distributor.c index 6cb27f4de111..915de3027bd0 100644 --- a/app/test/test_distributor.c +++ b/app/test/test_distributor.c @@ -80,7 +80,7 @@ clear_packet_count(void) __ATOMIC_RELAXED); } -/* this is the basic worker function for sanity test +/* this is the basic worker function * it does nothing but return packets and count them. */ static int @@ -105,7 +105,7 @@ handle_work(void *arg) return 0; } -/* do basic sanity testing of the distributor. This test tests the following: +/* do basic testing of the distributor. This test tests the following: * - send 32 packets through distributor with the same tag and ensure they * all go to the one worker * - send 32 packets through the distributor with two different tags and @@ -117,7 +117,7 @@ handle_work(void *arg) * not necessarily in the same order (as different flows). */ static int -sanity_test(struct worker_params *wp, struct rte_mempool *p) +basic_test(struct worker_params *wp, struct rte_mempool *p) { struct rte_distributor *db = wp->dist; struct rte_mbuf *bufs[BURST]; @@ -126,7 +126,7 @@ sanity_test(struct worker_params *wp, struct rte_mempool *p) unsigned int retries; unsigned int processed; - printf("=== Basic distributor sanity tests ===\n"); + printf("=== Basic distributor tests ===\n"); clear_packet_count(); if (rte_mempool_get_bulk(p, (void *)bufs, BURST) != 0) { printf("line %d: Error getting mbufs from pool\n", __LINE__); @@ -163,7 +163,7 @@ sanity_test(struct worker_params *wp, struct rte_mempool *p) printf("Worker %u handled %u packets\n", i, __atomic_load_n(&worker_stats[i].handled_packets, __ATOMIC_RELAXED)); - printf("Sanity test with all zero hashes done.\n"); + printf("Test with all zero hashes done.\n"); /* pick two flows and check they go correctly */ if (rte_lcore_count() >= 3) { @@ -191,7 +191,7 @@ sanity_test(struct worker_params *wp, struct rte_mempool *p) __atomic_load_n( &worker_stats[i].handled_packets, __ATOMIC_RELAXED)); - printf("Sanity test with two hash values done\n"); + printf("Test with two hash values done\n"); } /* give a different hash value to each packet, @@ -219,11 +219,11 @@ sanity_test(struct worker_params *wp, struct rte_mempool *p) printf("Worker %u handled %u packets\n", i, __atomic_load_n(&worker_stats[i].handled_packets, __ATOMIC_RELAXED)); - printf("Sanity test with non-zero hashes done\n"); + printf("Test with non-zero hashes done\n"); rte_mempool_put_bulk(p, (void *)bufs, BURST); - /* sanity test with BIG_BATCH packets to ensure they all arrived back + /* test with BIG_BATCH packets to ensure they all arrived back * from the returned packets function */ clear_packet_count(); struct rte_mbuf *many_bufs[BIG_BATCH], *return_bufs[BIG_BATCH]; @@ -288,7 +288,7 @@ sanity_test(struct worker_params *wp, struct rte_mempool *p) return -1; } } - printf("Sanity test of returned packets done\n"); + printf("Test of returned packets done\n"); rte_mempool_put_bulk(p, (void *)many_bufs, BIG_BATCH); @@ -326,20 +326,20 @@ handle_work_with_free_mbufs(void *arg) return 0; } -/* Perform a sanity test of the distributor with a large number of packets, +/* Perform a test of the distributor with a large number of packets, * where we allocate a new set of mbufs for each burst. The workers then * free the mbufs. This ensures that we don't have any packet leaks in the * library. */ static int -sanity_test_with_mbuf_alloc(struct worker_params *wp, struct rte_mempool *p) +basic_test_with_mbuf_alloc(struct worker_params *wp, struct rte_mempool *p) { struct rte_distributor *d = wp->dist; unsigned i; struct rte_mbuf *bufs[BURST]; unsigned int processed; - printf("=== Sanity test with mbuf alloc/free (%s) ===\n", wp->name); + printf("=== Test with mbuf alloc/free (%s) ===\n", wp->name); clear_packet_count(); for (i = 0; i < ((1<dist; @@ -449,7 +449,7 @@ sanity_test_with_worker_shutdown(struct worker_params *wp, unsigned int failed = 0; unsigned int processed = 0; - printf("=== Sanity test of worker shutdown ===\n"); + printf("=== Test of worker shutdown ===\n"); clear_packet_count(); @@ -515,7 +515,7 @@ sanity_test_with_worker_shutdown(struct worker_params *wp, if (failed) return -1; - printf("Sanity test with worker shutdown passed\n\n"); + printf("Test with worker shutdown passed\n\n"); return 0; } @@ -611,13 +611,13 @@ handle_and_mark_work(void *arg) return 0; } -/* sanity_mark_test sends packets to workers which mark them. +/* mark_test sends packets to workers which mark them. * Every packet has also encoded sequence number. * The returned packets are sorted and verified if they were handled * by proper workers. */ static int -sanity_mark_test(struct worker_params *wp, struct rte_mempool *p) +mark_test(struct worker_params *wp, struct rte_mempool *p) { const unsigned int buf_count = 24; const unsigned int burst = 8; @@ -898,13 +898,13 @@ test_distributor(void) rte_eal_mp_remote_launch(handle_work, &worker_params, SKIP_MAIN); - if (sanity_test(&worker_params, p) < 0) + if (basic_test(&worker_params, p) < 0) goto err; quit_workers(&worker_params, p); rte_eal_mp_remote_launch(handle_work_with_free_mbufs, &worker_params, SKIP_MAIN); - if (sanity_test_with_mbuf_alloc(&worker_params, p) < 0) + if (basic_test_with_mbuf_alloc(&worker_params, p) < 0) goto err; quit_workers(&worker_params, p); @@ -912,7 +912,7 @@ test_distributor(void) rte_eal_mp_remote_launch(handle_work_for_shutdown_test, &worker_params, SKIP_MAIN); - if (sanity_test_with_worker_shutdown(&worker_params, + if (basic_test_with_worker_shutdown(&worker_params, p) < 0) goto err; quit_workers(&worker_params, p); @@ -927,7 +927,7 @@ test_distributor(void) rte_eal_mp_remote_launch(handle_and_mark_work, &worker_params, SKIP_MAIN); - if (sanity_mark_test(&worker_params, p) < 0) + if (mark_test(&worker_params, p) < 0) goto err; quit_workers(&worker_params, p); diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c index 7581fc2b4cb0..899786b345ed 100644 --- a/app/test/test_dmadev.c +++ b/app/test/test_dmadev.c @@ -1087,7 +1087,7 @@ test_dma(void) parse_dma_env_var(); - /* basic sanity on dmadev infrastructure */ + /* basic validation on dmadev infrastructure */ if (test_apis() < 0) ERR_RETURN("Error performing API tests\n"); diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c index 6cb4b0675730..9c082ff91496 100644 --- a/app/test/test_eal_flags.c +++ b/app/test/test_eal_flags.c @@ -314,7 +314,7 @@ get_number_of_sockets(void) /* * Test that the app doesn't run with invalid allow option. - * Final tests ensures it does run with valid options as sanity check (one + * Final tests ensures it does run with valid options as basic test (one * test for with Domain+BDF, second for just with BDF) */ static int @@ -382,7 +382,7 @@ test_allow_flag(void) /* * Test that the app doesn't run with invalid blocklist option. - * Final test ensures it does run with valid options as sanity check + * Final test ensures it does run with valid options as basic check. */ static int test_invalid_b_flag(void) @@ -429,7 +429,7 @@ test_invalid_b_flag(void) /* * Test that the app doesn't run with invalid vdev option. - * Final test ensures it does run with valid options as sanity check + * Final test ensures it does run with valid options basic check. */ static int test_invalid_vdev_flag(void) @@ -554,7 +554,7 @@ test_missing_c_flag(void) /* bad coremask value */ const char *argv3[] = { prgname, prefix, mp_flag, "-c", "error" }; - /* sanity check of tests - valid coremask value */ + /* check of tests - valid coremask value */ const char *argv4[] = { prgname, prefix, mp_flag, "-c", "1" }; /* -l flag but no corelist value */ @@ -581,7 +581,7 @@ test_missing_c_flag(void) "-l", RTE_STR(RTE_MAX_LCORE) }; const char * const argv14[] = { prgname, prefix, mp_flag, "-l", "1-"RTE_STR(RTE_MAX_LCORE) }; - /* sanity check test - valid corelist value */ + /* check test - valid corelist value */ const char * const argv15[] = { prgname, prefix, mp_flag, "-l", "1-2,3" }; @@ -613,7 +613,7 @@ test_missing_c_flag(void) "--lcores", "(4-7)=(1,3)" }; const char * const argv28[] = { prgname, prefix, mp_flag, "--lcores", "[4-7]@(1,3)" }; - /* sanity check of tests - valid lcores value */ + /* more complex valid lcores value */ const char * const argv29[] = { prgname, prefix, mp_flag, "--lcores", "0-1,2@(5-7),(3-5)@(0,2),(0,6),7"}; @@ -744,7 +744,7 @@ test_main_lcore_flag(void) /* * Test that the app doesn't run with invalid -n flag option. - * Final test ensures it does run with valid options as sanity check + * Final test ensures it does run with valid options as basic check. * Since -n is not compulsory for MP, we instead use --no-huge and --no-shconf * flags. */ @@ -772,10 +772,10 @@ test_invalid_n_flag(void) /* zero is invalid */ const char *argv3[] = { prgname, prefix, no_huge, no_shconf, "-n", "0" }; - /* sanity test - check with good value */ + /* check with good value */ const char *argv4[] = { prgname, prefix, no_huge, no_shconf, "-n", "2" }; - /* sanity test - check with no -n flag */ + /* check with no -n flag */ const char *argv5[] = { prgname, prefix, no_huge, no_shconf}; if (launch_proc(argv1) == 0 @@ -977,7 +977,7 @@ test_misc_flags(void) * All cases, apart from the first, app should run. * No further testing of output done. */ - /* sanity check - failure with invalid option */ + /* failure with invalid option */ const char *argv0[] = {prgname, prefix, mp_flag, "--invalid-opt"}; /* With --no-pci */ diff --git a/app/test/test_hash.c b/app/test/test_hash.c index d586878a2244..54e751b5eede 100644 --- a/app/test/test_hash.c +++ b/app/test/test_hash.c @@ -1088,7 +1088,7 @@ fbk_hash_unit_test(void) .socket_id = 0, }; - /* this is a sanity check for "same name" test + /* this is a check for "same name" test * creating this hash will check if we are actually able to create * multiple hashes with different names (instead of having just one). */ diff --git a/app/test/test_interrupts.c b/app/test/test_interrupts.c index 3952f9685fd9..42b830e9368a 100644 --- a/app/test/test_interrupts.c +++ b/app/test/test_interrupts.c @@ -48,7 +48,7 @@ static union intr_pipefds pfds; * Check if the interrupt handle is valid. */ static inline int -test_interrupt_handle_sanity_check(struct rte_intr_handle *intr_handle) +test_interrupt_handle_check(struct rte_intr_handle *intr_handle) { if (!intr_handle || rte_intr_fd_get(intr_handle) < 0) return -1; @@ -164,7 +164,7 @@ test_interrupt_handle_compare(struct rte_intr_handle *intr_handle_l, #else /* to be implemented for bsd later */ static inline int -test_interrupt_handle_sanity_check(struct rte_intr_handle *intr_handle) +test_interrupt_handle_check(struct rte_intr_handle *intr_handle) { RTE_SET_USED(intr_handle); @@ -215,7 +215,7 @@ test_interrupt_callback(void *arg) return; } - if (test_interrupt_handle_sanity_check(intr_handle) < 0) { + if (test_interrupt_handle_check(intr_handle) < 0) { printf("null or invalid intr_handle for %s\n", __func__); flag = -1; return; @@ -241,7 +241,8 @@ static void test_interrupt_callback_1(void *arg) { struct rte_intr_handle *intr_handle = arg; - if (test_interrupt_handle_sanity_check(intr_handle) < 0) { + + if (test_interrupt_handle_check(intr_handle) < 0) { printf("null or invalid intr_handle for %s\n", __func__); flag = -1; return; diff --git a/app/test/test_link_bonding_mode4.c b/app/test/test_link_bonding_mode4.c index ff13dbed93f3..299b89f669ee 100644 --- a/app/test/test_link_bonding_mode4.c +++ b/app/test/test_link_bonding_mode4.c @@ -224,7 +224,7 @@ add_member(struct member_conf *member, uint8_t start) struct rte_ether_addr addr, addr_check; int retval; - /* Some sanity check */ + /* Some basic tests. */ RTE_VERIFY(test_params.member_ports <= member && member - test_params.member_ports < (int)RTE_DIM(test_params.member_ports)); RTE_VERIFY(member->bonding == 0); diff --git a/app/test/test_mbuf.c b/app/test/test_mbuf.c index 261c6e5d71e9..7c73880090d4 100644 --- a/app/test/test_mbuf.c +++ b/app/test/test_mbuf.c @@ -1701,7 +1701,7 @@ test_mbuf_validate_tx_offload(const char *test_name, struct rte_mbuf *m = NULL; int ret = 0; - /* alloc a mbuf and do sanity check */ + /* alloc a mbuf and do some basic checks */ m = rte_pktmbuf_alloc(pktmbuf_pool); if (m == NULL) GOTO_FAIL("%s: mbuf allocation failed!\n", __func__); diff --git a/app/test/test_rawdev.c b/app/test/test_rawdev.c index d34691dacf5f..0d4f0ce1fa17 100644 --- a/app/test/test_rawdev.c +++ b/app/test/test_rawdev.c @@ -47,7 +47,7 @@ test_rawdev_selftests(void) int ret = 0; int i; - /* basic sanity on rawdev infrastructure */ + /* basic check on rawdev infrastructure */ if (test_rawdev_selftest_skeleton() < 0) return -1; diff --git a/app/test/test_timer.c b/app/test/test_timer.c index cac8fc01149c..a23b5d8f2cb4 100644 --- a/app/test/test_timer.c +++ b/app/test/test_timer.c @@ -491,11 +491,11 @@ timer_basic_main_loop(__rte_unused void *arg) } static int -timer_sanity_check(void) +timer_check(void) { #ifdef RTE_LIBEAL_USE_HPET if (eal_timer_source != EAL_TIMER_HPET) { - printf("Not using HPET, can't sanity check timer sources\n"); + printf("Not using HPET, can't check timer sources\n"); return 0; } @@ -539,9 +539,9 @@ test_timer(void) return TEST_SKIPPED; } - /* sanity check our timer sources and timer config values */ - if (timer_sanity_check() < 0) { - printf("Timer sanity checks failed\n"); + /* check our timer sources and timer config values */ + if (timer_check() < 0) { + printf("Timer checks failed\n"); return TEST_FAILED; } From patchwork Mon Feb 5 17:43:36 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136395 X-Patchwork-Delegate: thomas@monjalon.net 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 95E6443A3D; Mon, 5 Feb 2024 19:04:46 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 3EFD5410D0; Mon, 5 Feb 2024 19:03:50 +0100 (CET) Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) by mails.dpdk.org (Postfix) with ESMTP id DB10840ED2 for ; Mon, 5 Feb 2024 19:03:46 +0100 (CET) Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-1d9bd8fa49eso7718045ad.1 for ; Mon, 05 Feb 2024 10:03:46 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156226; x=1707761026; darn=dpdk.org; 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=/F6I4lQJ8LKu9xHO5RYDkIkwNTnjwcVaEvuhPLm9LQw=; b=wJGzuFam8N9oOXIqtUOBy9DwD9hQMnVeEGuz70JErxMyLwXRDX2fGBf4pd2zxM5y7i kpM7aPcXMd3W8UI5obS1lS3bi3lOqd9Bsoh+qKL3Cu3ZDcoAw+OZIz9H9fVFsGBY01Pa ty1+wNWxkE1uvBIEDdDKjX6lyzzVv3nBexcXwx56dWMV4Wnj5cU27eOeGZf9X2CVbidh xT9qdC61PuT5bxZo2g/LqWi0IujaToeEAMANH5uJT5l/5Ju2myebmnFpt346mGzSVYNK JgaJl0VahklKeuirJMy6JZ6JHL9rp1kml76L2ol1ov4k8Pnbigun0QxpSt7kN1qsZv0A YU6A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156226; x=1707761026; 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=/F6I4lQJ8LKu9xHO5RYDkIkwNTnjwcVaEvuhPLm9LQw=; b=Y9/8YIW1vaygP5y0AYFPMDJ2cmT40zj9CGjlL9zbOqWgsLAkXgHHrlYkak/G/bzENb +0BjJp7pEMgEtL22z71ZzZZ9KjPhZfefBWDWE6qDrjztzjDdY67Ykhd42MnHM4jsTiK3 a1sy3G1eWHtwjZP7evnTSitN+t6NSsw61nHCieaS2atYr1gtro6LE3ktzr0cBuu+zIUk XYQq4i21PfZROcP+4aZyZ48x2XmOsu5ewqggKffy4RDvExk7nNszZPePf2FLOIZPZgHV sG4YWD05IspT9dPJk0mw15YgOkcs6P2Wbz7WGBLoco40SDdz+PO94YmnzlepQctJGdpq pxaw== X-Gm-Message-State: AOJu0Yzi4398OduY6vL3pObfOt/+U/iRnjqbhIEKOJNCdBP4gzyZrq/y RbQfiVrqrH7KP7d8JBF7P37akR+/MTbbUojO8hvkoXUA4CO+mUj5wHRHtaYADcUCdz1qMb5sLQR 0SF8= X-Google-Smtp-Source: AGHT+IHrZrbf7nJrROgAy7rja4JJhPteNef2I+iR3kQfqRQhqkrYQ2jdmoYmT/aGHklZYmpdDytRdg== X-Received: by 2002:a17:902:f745:b0:1d8:b6c8:d9e0 with SMTP id g5-20020a170902f74500b001d8b6c8d9e0mr254614plw.68.1707156226114; Mon, 05 Feb 2024 10:03:46 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCUnlAAJUON9yIo4k2RXgwkzSPxvXcr4H3UpU26XHf+gaG0VSmUGKyreEZ6/2jFFQHIG/EjSwywfxRb3CEqbdnScH+TR+pjZAUCF4/IdzMj+pL4VHM4h8q0MfkFvBeMHo7oLwNAZ5IO8s0xiMxgMHRfskBOxrslxo7RE Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:45 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Radu Nicolau , Akhil Goyal , Cristian Dumitrescu Subject: [PATCH v9 08/23] examples: remove term sanity Date: Mon, 5 Feb 2024 09:43:36 -0800 Message-ID: <20240205180328.131019-9-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Do not use non-inclusive terms. Signed-off-by: Stephen Hemminger Acked-by: Akhil Goyal --- examples/ipsec-secgw/event_helper.c | 2 +- examples/qos_sched/args.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/examples/ipsec-secgw/event_helper.c b/examples/ipsec-secgw/event_helper.c index 89fb7e62a5a3..f595b4fe01dc 100644 --- a/examples/ipsec-secgw/event_helper.c +++ b/examples/ipsec-secgw/event_helper.c @@ -2100,7 +2100,7 @@ eh_launch_worker(struct eh_conf *conf, struct eh_app_worker_params *app_wrkr, goto clean_and_exit; } - /* Verify sanity of the matched worker */ + /* Verify validity of the matched worker */ if (eh_verify_match_worker(match_wrkr) != 1) { EH_LOG_ERR("Failed to validate the matched worker"); goto clean_and_exit; diff --git a/examples/qos_sched/args.c b/examples/qos_sched/args.c index e97273152aa7..1d8ba9522586 100644 --- a/examples/qos_sched/args.c +++ b/examples/qos_sched/args.c @@ -384,7 +384,7 @@ app_parse_args(int argc, char **argv) return -1; } - /* sanity check for cores assignment */ + /* check for cores assignment */ for(i = 0; i < nb_pfc; i++) { if (qos_conf[i].rx_core >= RTE_MAX_LCORE) { RTE_LOG(ERR, APP, "pfc %u: invalid RX lcore index %u\n", i + 1, From patchwork Mon Feb 5 17:43:37 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136396 X-Patchwork-Delegate: thomas@monjalon.net 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 01EA543A73; Mon, 5 Feb 2024 19:04:54 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 84ED6410F1; Mon, 5 Feb 2024 19:03:51 +0100 (CET) Received: from mail-pl1-f169.google.com (mail-pl1-f169.google.com [209.85.214.169]) by mails.dpdk.org (Postfix) with ESMTP id 106CF40EDF for ; Mon, 5 Feb 2024 19:03:48 +0100 (CET) Received: by mail-pl1-f169.google.com with SMTP id d9443c01a7336-1d73066880eso40762425ad.3 for ; Mon, 05 Feb 2024 10:03:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156227; x=1707761027; darn=dpdk.org; 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=Z0sSlJYi03kscoZ3gDSMZlPaDhE+Va5+wqjhjq17duk=; b=TLenU0GxyDOEi2dHx2iOPnOYjV5hxpPqsJKMFxpUKCBtM4RCp1m1I+qstfU48wqXFr zWKC+ZCRdF3pSW70pfq/Cw8+GLFh5KoTY8dMmpcTDNTCJ4SKUrSQE0EiDE/hnzuL5lRN mwATZWPZS622vCiHHE4RGyM3kngy4NAodFidYrkxAKoWfgjrdBUjlAOFkhbnVdVaWEsm xqzuix8KN4i4Ws0Pnso+eXnPjA13dKGSlx/YDg0RynsaECsB+zUBFCOu5Sfg1CDKYnjx 5tnbD4zGxnLiY5/+4imt9SpOClq+j6N0XLFu7XJHxLen3FlLgYKq0nCENkgnC1xL/xf6 rr8Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156227; x=1707761027; 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=Z0sSlJYi03kscoZ3gDSMZlPaDhE+Va5+wqjhjq17duk=; b=loGTfJ8MT8ZM6Syg0tKJqnuFOlGJs6ieqO+M7TeneWc9vQRaemVerkNCxw2zBmS+ft A25fchJif8zIq/8rM7mSIBRdv5A1CgPwlhc5C3a0N/G9szOTMizgZNqTIpYAQk1+NOPh csehuiHx2YJ2HhbpbhOR4W+inX4G9uL6M7SEGodxU4XXK4gmgX52FD6BeHObYidzaO6l qa8w8AfnbiNl/a/fNiXxbDZpUAmG29nz+ZHDNYVYHvDIBVHIHv2NZROVSBwfjFjKvwtY 0o6eQ0pDok5TMyjM2Zl84FyhWgXiCFtFtuLvkMYl0UhDaax97OIyjwN6qJkQenpGA8sr V4nQ== X-Gm-Message-State: AOJu0YxxEjSb91EAzOtfeBXVGqoQGhyQ35L6oM0RQmOkOz8SR+0LoOvS os4QsptZwOOp+SyfL0O1A4vE1MVgm6XExmjSpc+NFhUC/kW1cZY+7wNNNHN9dpav8tu4Ce0Mh7H I6NI= X-Google-Smtp-Source: AGHT+IHNxJgYL1UFir7ZLGgEZM45z6Vfb31auJeLWKrUSo2mUBUYkQAtwFjlyqgoZLd8yVr15w5mRQ== X-Received: by 2002:a17:902:e549:b0:1d8:f016:cfaf with SMTP id n9-20020a170902e54900b001d8f016cfafmr225788plf.47.1707156227248; Mon, 05 Feb 2024 10:03:47 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCVUVpx6U1sXCpr180x1+LilNRiJLSYXd0Q4NJtUTaT6d0Lvs9nyPDYAtnkpWsYC3yab4tG0lAI2nlomZfaRSd7Yp3vWkBBBQZpEsuPSZzidynCtBqTVZXj1NrczpiC1miN8xOV3GmJOwDrm1/xqG5R9xXQshBfJASbM9gqvXca0vcvwV/9q9z/mgEhzE4fI8EI3pnmVm9f9HrQfHo/ZCillsfIP Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:46 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Jerin Jacob , Kiran Kumar K , Nithin Dabilpuram , Zhirun Yan , Reshma Pattan Subject: [PATCH v9 09/23] lib: replace use of sanity check in comments and messages Date: Mon, 5 Feb 2024 09:43:37 -0800 Message-ID: <20240205180328.131019-10-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Sanity check is on the Tier 2 non-inclusive list. Replace or remove it. Signed-off-by: Stephen Hemminger --- lib/graph/graph.c | 2 +- lib/graph/graph_stats.c | 2 +- lib/graph/node.c | 2 +- lib/jobstats/rte_jobstats.c | 6 +++--- lib/metrics/rte_metrics.c | 2 +- lib/pcapng/rte_pcapng.c | 2 +- 6 files changed, 8 insertions(+), 8 deletions(-) diff --git a/lib/graph/graph.c b/lib/graph/graph.c index 26f0968a978d..e51dcc30568e 100644 --- a/lib/graph/graph.c +++ b/lib/graph/graph.c @@ -344,7 +344,7 @@ rte_graph_create(const char *name, struct rte_graph_param *prm) graph_spinlock_lock(); - /* Check arguments sanity */ + /* Check argument validity */ if (prm == NULL) SET_ERR_JMP(EINVAL, fail, "Param should not be NULL"); diff --git a/lib/graph/graph_stats.c b/lib/graph/graph_stats.c index cc32245c0593..cfb30ae07497 100644 --- a/lib/graph/graph_stats.c +++ b/lib/graph/graph_stats.c @@ -317,7 +317,7 @@ rte_graph_cluster_stats_create(const struct rte_graph_cluster_stats_param *prm) const char *pattern; rte_graph_t i; - /* Sanity checks */ + /* Validate arguments */ if (!rte_graph_has_stats_feature()) SET_ERR_JMP(EINVAL, fail, "Stats feature is not enabled"); diff --git a/lib/graph/node.c b/lib/graph/node.c index 99a962277964..c238d140ed31 100644 --- a/lib/graph/node.c +++ b/lib/graph/node.c @@ -68,7 +68,7 @@ __rte_node_register(const struct rte_node_register *reg) graph_spinlock_lock(); - /* Check sanity */ + /* Check argument validity */ if (reg == NULL || reg->process == NULL) { rte_errno = EINVAL; goto fail; diff --git a/lib/jobstats/rte_jobstats.c b/lib/jobstats/rte_jobstats.c index af565a14eaf3..1c4b280c952a 100644 --- a/lib/jobstats/rte_jobstats.c +++ b/lib/jobstats/rte_jobstats.c @@ -124,7 +124,7 @@ rte_jobstats_start(struct rte_jobstats_context *ctx, struct rte_jobstats *job) { uint64_t now; - /* Some sanity check. */ + /* Check validity of arguments. */ if (unlikely(ctx == NULL || job == NULL || job->context != NULL)) return -EINVAL; @@ -144,7 +144,7 @@ rte_jobstats_abort(struct rte_jobstats *job) struct rte_jobstats_context *ctx; uint64_t now, exec_time; - /* Some sanity check. */ + /* Check that arguments are valid */ if (unlikely(job == NULL || job->context == NULL)) return -EINVAL; @@ -165,7 +165,7 @@ rte_jobstats_finish(struct rte_jobstats *job, int64_t job_value) uint64_t now, exec_time; int need_update; - /* Some sanity check. */ + /* Check arguments */ if (unlikely(job == NULL || job->context == NULL)) return -EINVAL; diff --git a/lib/metrics/rte_metrics.c b/lib/metrics/rte_metrics.c index 0ccdbabc04eb..f60d4f3f6e25 100644 --- a/lib/metrics/rte_metrics.c +++ b/lib/metrics/rte_metrics.c @@ -120,7 +120,7 @@ rte_metrics_reg_names(const char * const *names, uint16_t cnt_names) uint16_t idx_name; uint16_t idx_base; - /* Some sanity checks */ + /* Some consistency checks */ if (cnt_names < 1 || names == NULL) return -EINVAL; for (idx_name = 0; idx_name < cnt_names; idx_name++) diff --git a/lib/pcapng/rte_pcapng.c b/lib/pcapng/rte_pcapng.c index f74ec939a9f8..0de0a231693e 100644 --- a/lib/pcapng/rte_pcapng.c +++ b/lib/pcapng/rte_pcapng.c @@ -619,7 +619,7 @@ rte_pcapng_write_packets(rte_pcapng_t *self, struct pcapng_enhance_packet_block *epb; uint64_t cycles, timestamp; - /* sanity check that is really a pcapng mbuf */ + /* Check that is really a pcapng mbuf */ epb = rte_pktmbuf_mtod(m, struct pcapng_enhance_packet_block *); if (unlikely(epb->block_type != PCAPNG_ENHANCED_PACKET_BLOCK || epb->block_length != rte_pktmbuf_data_len(m))) { From patchwork Mon Feb 5 17:43:38 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136397 X-Patchwork-Delegate: thomas@monjalon.net 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 DAD4643A3D; Mon, 5 Feb 2024 19:05:00 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 9E93C410F9; Mon, 5 Feb 2024 19:03:52 +0100 (CET) Received: from mail-pl1-f172.google.com (mail-pl1-f172.google.com [209.85.214.172]) by mails.dpdk.org (Postfix) with ESMTP id CDB1140F35 for ; Mon, 5 Feb 2024 19:03:48 +0100 (CET) Received: by mail-pl1-f172.google.com with SMTP id d9443c01a7336-1d93edfa76dso39986715ad.1 for ; Mon, 05 Feb 2024 10:03:48 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156228; x=1707761028; darn=dpdk.org; 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=deqTX0VoA3I70sjPbBvqJlqaSbzFJhbFWOydqC8sopY=; b=ZO4CJ1JLQrgpZGYJZRlDcscPGREKhSMgCIlatPiwJ/a3jtdeZmOM/yABcOSq1GmVuu h/udM67NbDHpdhy+1ugmoVReOHiX4GbQJ+G3ENE2NfONNLWer0VQQrbhgQfUtAyUz5zx 3P6s8MnId7YkhrbsFhynQMwoWst6ccULcp7U83u6y/cFaJhwy6/iQd1SVIQAbKtpucR5 SjIPkLqfPnuSZe/ZtVzLKpai+aaUcGl8lXpwW60KFgEeroNBteRA05B9zc5mRKnZN73C 47EdFJVUlZ0sXqqyjIe2AufrLfUlgOVqnnYY0h8xnxchNO5OqEDiOFqqvy96nTRCSx+v J1Cw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156228; x=1707761028; 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=deqTX0VoA3I70sjPbBvqJlqaSbzFJhbFWOydqC8sopY=; b=wRQYFXoBux4mxZkiLOjSCzhVXYGvkzzcFdGB9VYQOiBZFbckFOmreAty/c2ODsJeP1 C5ul3nxF1UJ6utgtAImXe98X3hDy7yLbBPjV7dQYAomjYjpIC90n+hbLP+AslFXzJxHL 8mXE8TNCeVvDtKWEcKsLmt8MN8212kIHLK70sBRD83Pk9a5aINzUxg7zFCCa49tHihr+ 3qydcyPNdsmP4/C1jeslXOPXFPUNqFSQAOaCiiyXfy+P6b4Xm8lY3G99aIo8z4tpHqLH u+QfZH7iCJha1sL90bsyy3LDVBGyv7wSFqiqOSLTP5Ic/7bzVJlN92J/Ibhoig3paSWl sKjg== X-Gm-Message-State: AOJu0Yxm7O6anpTCx/r0flEcLca5V+ozLNx/gBSZXlwoNIssc7eAOmX+ JhUb5Rps6GdpNwgDiOkSfkuZDjsfXhB+NVeuQhQLF0IDuCugEEBfwbNLEPih9dKruypE/haGFCx Yzhc= X-Google-Smtp-Source: AGHT+IEtXvOfYGsWNl2z7w3+EN1NVuqME+fkUj7bMBsPoJ8VQ35Wmsltzw/4CzcwndvQ3OWj/MnzWw== X-Received: by 2002:a17:903:2302:b0:1d9:b3ea:25ef with SMTP id d2-20020a170903230200b001d9b3ea25efmr322096plh.10.1707156228018; Mon, 05 Feb 2024 10:03:48 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCWgqlvaVZGGMdioVsJ+GzHDFp2F2T4OOJn/WOCqNSHiscO0CNFenkNvddwd0F7q7Y2rauu9qyLh+1E10QuDsKo4qxIA64Go7k5AxOynNX1i/5wq1Pr9iEKjn4K22RtImtn/tr+Ad3KdhRhyi5R2pbfiAuz2z7rY5cx25To= Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:47 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Elena Agostini , Harry van Haaren , Nicolas Chautru Subject: [PATCH v9 10/23] doc/eventdev_pipeline: remove sanity Date: Mon, 5 Feb 2024 09:43:38 -0800 Message-ID: <20240205180328.131019-11-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 word sanity check is unnecessary here. Signed-off-by: Stephen Hemminger --- doc/guides/gpus/cuda.rst | 2 +- doc/guides/sample_app_ug/eventdev_pipeline.rst | 2 +- doc/guides/tools/testbbdev.rst | 8 ++++---- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/doc/guides/gpus/cuda.rst b/doc/guides/gpus/cuda.rst index 6520c17c3ea4..c523da1f3e9d 100644 --- a/doc/guides/gpus/cuda.rst +++ b/doc/guides/gpus/cuda.rst @@ -80,7 +80,7 @@ Similarly to CUDA shared library, if the **libgdrapi.so** shared library is not installed in default locations (e.g. /usr/local/lib), you can use the variable ``GDRCOPY_PATH_L``. -As an example, to enable the CPU map feature sanity check, +As an example, to enable the CPU map feature check, run the ``app/test-gpudev`` application with: .. code-block:: console diff --git a/doc/guides/sample_app_ug/eventdev_pipeline.rst b/doc/guides/sample_app_ug/eventdev_pipeline.rst index 19ff53803ede..e7787fb26609 100644 --- a/doc/guides/sample_app_ug/eventdev_pipeline.rst +++ b/doc/guides/sample_app_ug/eventdev_pipeline.rst @@ -50,7 +50,7 @@ these settings is shown below: .//examples/dpdk-eventdev_pipeline -l 0,2,8-15 --vdev event_sw0 \ -- -r1 -t1 -e4 -w FF00 -s4 -n0 -c32 -W1000 -D -The application has some sanity checking built-in, so if there is a function +The application has some internal validation, so if there is a function (e.g.; the RX core) which doesn't have a cpu core mask assigned, the application will print an error message: diff --git a/doc/guides/tools/testbbdev.rst b/doc/guides/tools/testbbdev.rst index ddb8d787be6f..baaf9d115dda 100644 --- a/doc/guides/tools/testbbdev.rst +++ b/doc/guides/tools/testbbdev.rst @@ -9,7 +9,7 @@ dpdk-test-bbdev Application The ``dpdk-test-bbdev`` tool is a Data Plane Development Kit (DPDK) utility that allows measuring performance parameters of PMDs available in the bbdev framework. Tests available for execution are: latency, throughput, validation, -bler and sanity tests. Execution of tests can be customized using various +bler and basic functionality. Execution of tests can be customized using various parameters passed to a python running script. @@ -97,8 +97,8 @@ Test Cases There are 7 main test cases that can be executed using testbbdev tool: -* Sanity checks [-c unittest] - - Performs sanity checks on BBDEV interface, validating basic functionality +* Functional checks [-c unittest] + - Performs checks on BBDEV interface, validating basic functionality * Validation tests [-c validation] - Performs full operation of enqueue and dequeue @@ -232,7 +232,7 @@ vector name refer more explicitly processing specificity such as is purely read/written for external DDR, lbrm when limited buffer rate matching is expected, or crc_fail when a CRC failure is expected. They are chosen to have a good coverage across sizes and processing -parameters while still keeping their number limited as part of sanity +parameters while still keeping their number limited as part of functional regression. Shortened tree of isg_cid-wireless_dpdk_ae with dpdk compiled and output From patchwork Mon Feb 5 17:43:39 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136398 X-Patchwork-Delegate: thomas@monjalon.net 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 C7D6F43A3D; Mon, 5 Feb 2024 19:05:07 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 9799F410FC; Mon, 5 Feb 2024 19:03:53 +0100 (CET) Received: from mail-pl1-f179.google.com (mail-pl1-f179.google.com [209.85.214.179]) by mails.dpdk.org (Postfix) with ESMTP id 7D99640E7C for ; Mon, 5 Feb 2024 19:03:49 +0100 (CET) Received: by mail-pl1-f179.google.com with SMTP id d9443c01a7336-1d746856d85so35117795ad.0 for ; Mon, 05 Feb 2024 10:03:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156229; x=1707761029; darn=dpdk.org; 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=HE2SUGyOs3ZpiLiWyOpzagSFc7cwYc1MQ4IEOKUS/u0=; b=ZVxyahCT1M7ecsDXAH9FSparRwhCDI9U0nH7/0JzRLd5hnaA+zB6u3H3aU6PjVuvWw yEnaxvuMiTgZertlFzu2Zbtc5foITHQootj8tHsGH9mRpJnAZKe3A04UHypt0VS+s0gi o2FsOZi4l/i92AmaF0Cq+QsKB8QsXPvTuuUdXirmtDnhlx9YU1vaCUEaUEOVJmk9rVpr mhtnIvA5D//bGQHD/gzo2IFDeRVh9n3c8TA6aamc1QEKXIBBYoR1mEt1tGhNlgUUS4jc WzPwsm51euVsV8YrtRB2udfXnwJ4nN9h5oSwx/iBBI6dn7aFEIBQVn7lCfFShFsa1VD7 fJ3g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156229; x=1707761029; 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=HE2SUGyOs3ZpiLiWyOpzagSFc7cwYc1MQ4IEOKUS/u0=; b=lbluZ047NcToWyQNDcWSIXpaWsHa8GQOK6YBdSVBh0CJo5H0uDKgyAwMV3RTO8fqz6 FjJxz5GrkxPgjcCwaCHXqKoXt86pQGrE6ay55K3KT7vsZxM+dCBEoV0Sg/IHlET5BSKr 8XF543hfradJ8h3S6D76Mc2+aM9w6u06mDfhto90yaTe0H1HWIuRLOqqLYW5egn/4Hcp wE13u9+EIYNKve+RelAbcoNhQ7q2bNOLjIor0ZEttaIqGHwGCT89TjNI+9IAaHsrTTjz S9vtXy6SPND4XUvDWhjpa0uBlATl79AlR4wLXWWr8WL10LhHQLQ/6LinUSpV3xbKLN2F qmJA== X-Gm-Message-State: AOJu0YwWpvHax8Sy7I5y6WmFyFHG15bmsWDi4k/5J07UmGTHE5h38vCd FC2CRZxZ9xNtLa4nMTsCjrNtF0ZrLOfY25C8EA78y4vof31c/BPg3efxEJaOYW5URMyfhQO0Eg1 7rVU= X-Google-Smtp-Source: AGHT+IEJ36oUfE4zbYqZe3iRMPUZOGwDcrd7AL6AwOSMazpbxLV1C1apXjUJtHEhTXNilIjQP0tl7g== X-Received: by 2002:a17:902:d4c5:b0:1d9:95c5:2975 with SMTP id o5-20020a170902d4c500b001d995c52975mr285940plg.26.1707156228763; Mon, 05 Feb 2024 10:03:48 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCX/cjDNlqvtCJ850+iiWAJagnrTv94jf0FCCSzjc4p+VaOL5n9f8SeTjFDlzbSmjrzOpW2g2a9fMcsgnlcoEbtxUe9UedqJP8gZmA== Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:48 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Bruce Richardson Subject: [PATCH v9 11/23] net/ring: replace use of sanity Date: Mon, 5 Feb 2024 09:43:39 -0800 Message-ID: <20240205180328.131019-12-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Don't use term sanity check Signed-off-by: Stephen Hemminger --- drivers/net/ring/rte_eth_ring.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/net/ring/rte_eth_ring.c b/drivers/net/ring/rte_eth_ring.c index 48953dd7a059..710fd9ebe526 100644 --- a/drivers/net/ring/rte_eth_ring.c +++ b/drivers/net/ring/rte_eth_ring.c @@ -34,7 +34,7 @@ struct ring_internal_args { struct rte_ring * const *tx_queues; const unsigned int nb_tx_queues; const unsigned int numa_node; - void *addr; /* self addr for sanity check */ + void *addr; /* self addr for verification */ }; enum dev_action { @@ -606,10 +606,7 @@ static int parse_kvlist(const char *key __rte_unused, *action = '\0'; action++; - /* - * Need to do some sanity checking here - */ - + /* Check the command argument is valid action */ if (strcmp(action, ETH_RING_ACTION_ATTACH) == 0) info->list[info->count].action = DEV_ATTACH; else if (strcmp(action, ETH_RING_ACTION_CREATE) == 0) From patchwork Mon Feb 5 17:43:40 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136399 X-Patchwork-Delegate: thomas@monjalon.net 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 1F52343A3D; Mon, 5 Feb 2024 19:05:14 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A584141109; Mon, 5 Feb 2024 19:03:54 +0100 (CET) Received: from mail-pl1-f176.google.com (mail-pl1-f176.google.com [209.85.214.176]) by mails.dpdk.org (Postfix) with ESMTP id 5F654410D5 for ; Mon, 5 Feb 2024 19:03:50 +0100 (CET) Received: by mail-pl1-f176.google.com with SMTP id d9443c01a7336-1d8da50bffaso21490475ad.2 for ; Mon, 05 Feb 2024 10:03:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156229; x=1707761029; darn=dpdk.org; 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=zz3tDcheU2yYlmgbDe3+gRqcQeGPzvC2YvqPJk/WnKA=; b=DsMFcBEKtT0xGPKLmMvVdY0u3MdNnxFJili5NqoS30f4HeXI3QbVILl0/aXmJgvgSn 8mYF/j2RJb9wEvm/cRCBsuxGdClo/FGzpaV/vB2LdLS4ZdIuR1fMaVlqMD93MmjXx5Vs sQi95p5Zgp3AbtmSwEJ50Lbhf5qic9CNBLvc0vwh/WKwnWsqd+5RzUoc09crSDX9NQTv ctCIydBxOWE2tUYmGW5hiH2tnHPSKj1cJrYK6J+URkEEOwQtffojn162EUstbeRMcpfq D+AmCB3ynQloMQqmsvc1Qva4GJR6inprcJg8jBRb5e2WjpACxITkyBPvgrCGFgIvetgh 4XyQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156229; x=1707761029; 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=zz3tDcheU2yYlmgbDe3+gRqcQeGPzvC2YvqPJk/WnKA=; b=bfCxWeSehAf/S8nKJfHYNW+zgfze46mP1aNzXmiUEymIv+W78zC1hsHoVyKeTAikeu W3jLHe82lC7Ro6L+XFISAAUZgf4AXY+a2KRSG4gi7KDkPF7PTYeiNoRh7n0b37p9DQbN gYaua7LZq/KPlrIjWlLU/6y04HyKSowB6/bl0tJP04iHWPApHHAGGQmWOvud22nYa5nP fN8CuRIJrrHuB5cD6Lf+kd+oRLP+4RAhStJYHMlfwkOhdESkLUR4Y0wIXJlLVoe2Xf7V IIbIBRvv4jmFBeTmrgiRH4EG+fJR2X9B6ozRpZRiuvS6etx171sgTSw/B+L/eWl30jnY vSVw== X-Gm-Message-State: AOJu0YyWp4xjZCZWc0/AkhbqViztAMc20J2Z+MNcYvO+UFBPJuvY8Bn7 ao+nRbxegeBt7KsnRAOetWeJGcxRsx7FzMB7t1yvOW5IAZ0pmEWoMz9DbPJEzUbWezS9oKpBeMS bgtM= X-Google-Smtp-Source: AGHT+IHspkyodY0dTJsQy1UbZxMJqbzVJ1pYJO5EbwKfGQGlBKCknf3vFcyEkbMnUf2Pu0Vo6zz+gw== X-Received: by 2002:a17:903:32d0:b0:1d9:b8cf:795 with SMTP id i16-20020a17090332d000b001d9b8cf0795mr340721plr.58.1707156229583; Mon, 05 Feb 2024 10:03:49 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCVUP+NZyFuO9sX3H6Etc0tUSsox6z7WmN41szHoMs1/7+q+FNYvpHX1J7rLtqh47HllJWjbniSqqlJz6upv89GbMojz19c= Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:49 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Xiao Wang Subject: [PATCH v9 12/23] net/fm10k, net/ixgbe: remove word sanity Date: Mon, 5 Feb 2024 09:43:40 -0800 Message-ID: <20240205180328.131019-13-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Remove non-inclusive term sanity from these Intel drivers. Signed-off-by: Stephen Hemminger --- drivers/net/fm10k/fm10k_ethdev.c | 2 +- drivers/net/fm10k/fm10k_rxtx.c | 2 +- drivers/net/ixgbe/ixgbe_fdir.c | 2 +- drivers/net/ixgbe/ixgbe_ipsec.c | 2 +- drivers/net/ixgbe/ixgbe_rxtx.c | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/net/fm10k/fm10k_ethdev.c b/drivers/net/fm10k/fm10k_ethdev.c index 4d3c4c10cfa4..894cf8c8a6b5 100644 --- a/drivers/net/fm10k/fm10k_ethdev.c +++ b/drivers/net/fm10k/fm10k_ethdev.c @@ -1757,7 +1757,7 @@ mempool_element_size_valid(struct rte_mempool *mp) /* account for up to 512B of alignment */ min_size -= FM10K_RX_DATABUF_ALIGN; - /* sanity check for overflow */ + /* check for overflow */ if (min_size > mp->elt_size) return 0; diff --git a/drivers/net/fm10k/fm10k_rxtx.c b/drivers/net/fm10k/fm10k_rxtx.c index c0841ab8cb34..94345299b22a 100644 --- a/drivers/net/fm10k/fm10k_rxtx.c +++ b/drivers/net/fm10k/fm10k_rxtx.c @@ -647,7 +647,7 @@ fm10k_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, if (q->nb_free < mb->nb_segs) break; - /* sanity check to make sure the mbuf is valid */ + /* check to make sure the mbuf is valid */ if ((mb->nb_segs == 0) || ((mb->nb_segs > 1) && (mb->next == NULL))) break; diff --git a/drivers/net/ixgbe/ixgbe_fdir.c b/drivers/net/ixgbe/ixgbe_fdir.c index 06d6e2126de5..51bbf577e3fe 100644 --- a/drivers/net/ixgbe/ixgbe_fdir.c +++ b/drivers/net/ixgbe/ixgbe_fdir.c @@ -1130,7 +1130,7 @@ ixgbe_fdir_filter_program(struct rte_eth_dev *dev, return -ENOTSUP; /* - * Sanity check for x550. + * Check for x550. * When adding a new filter with flow type set to IPv4, * the flow director mask should be configed before, * and the L4 protocol and ports are masked. diff --git a/drivers/net/ixgbe/ixgbe_ipsec.c b/drivers/net/ixgbe/ixgbe_ipsec.c index d3313085560d..d4aa3e9ede8e 100644 --- a/drivers/net/ixgbe/ixgbe_ipsec.c +++ b/drivers/net/ixgbe/ixgbe_ipsec.c @@ -615,7 +615,7 @@ ixgbe_crypto_enable_ipsec(struct rte_eth_dev *dev) rx_offloads = dev->data->dev_conf.rxmode.offloads; tx_offloads = dev->data->dev_conf.txmode.offloads; - /* sanity checks */ + /* input validation */ if (rx_offloads & RTE_ETH_RX_OFFLOAD_TCP_LRO) { PMD_DRV_LOG(ERR, "RSC and IPsec not supported"); return -1; diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c index 90b0a7004f50..0e53b8a73a18 100644 --- a/drivers/net/ixgbe/ixgbe_rxtx.c +++ b/drivers/net/ixgbe/ixgbe_rxtx.c @@ -4983,7 +4983,7 @@ ixgbe_set_rsc(struct rte_eth_dev *dev) uint32_t rdrxctl; uint32_t rfctl; - /* Sanity check */ + /* Offload check */ dev->dev_ops->dev_infos_get(dev, &dev_info); if (dev_info.rx_offload_capa & RTE_ETH_RX_OFFLOAD_TCP_LRO) rsc_capable = true; From patchwork Mon Feb 5 17:43:41 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136400 X-Patchwork-Delegate: thomas@monjalon.net 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 1CAE043A3D; Mon, 5 Feb 2024 19:05:20 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0F2EC41132; Mon, 5 Feb 2024 19:03:56 +0100 (CET) Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) by mails.dpdk.org (Postfix) with ESMTP id 3493D410E6 for ; Mon, 5 Feb 2024 19:03:51 +0100 (CET) Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-1d958e0d73dso27442725ad.1 for ; Mon, 05 Feb 2024 10:03:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156230; x=1707761030; darn=dpdk.org; 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=F/cFx9VR40m09NkIyvZPh/pcDuHFjxfEs1tBZYQG5io=; b=tdkH1eugxHUJ4v8rPbPMyK8D9aoJPZJ3mm3SCdwIZcAcFN7zZdzRIe7JqTyJGMEQxL HWWe1n29QXCVeiXRSgBXKrN7ufwnZP7h2tNXzU86cAcaBgOy9fXJWgqy67VoVP/T/md3 +Xbd1YWWd1egEZBV6NpdcND/3Xb0o0TzBEAYPSQbVQVNebzZTKV3Mx7QP8+8RuDQxW18 MqbQE//HSaTc3cOjDB5YeNgYzTrq3Wvj6GGNUCytEmG+sRDnY8DQguKUT4ryoGncxpfZ 73Dgu0w2YzXM/XGFCyafXQ+Tk1rZCFQm2xwbuj8Kg5s41ZLwSHs4T1oW5gfYC5Cx1+dh SnAA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156230; x=1707761030; 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=F/cFx9VR40m09NkIyvZPh/pcDuHFjxfEs1tBZYQG5io=; b=hwI4dGzVZe/LmklJO8+ttVu7T2PuQJ6gvPGdWVF0NRuwsqGyC/bqyyrukjEBMhqzFv XGChNfXmHZL3HCcYa9LhlUrw1lJDF/pDI1jPq8HGgTGZG4zQC0mlJBKaXvoSr8rRLKuN /G9ZS4Rk/RQvNpHOhSv2GA2V3E8JcKuUUMVGoGxvHHt6gjBrc8a4RfRXPL57HxRVUXvZ Ru6jYq7xHCGwUHg/KDr87F8/yVZbvbA11esSVDSD2RymzrPSC7eKa89zMo9IrtQn0gOp e6ZINfUwjzdaSHfAroK3nGk10lPCL0/piUoW9XXuWhnXilDb7rnJFEx+U8eTVRo5qvvr JhYw== X-Gm-Message-State: AOJu0YwiAIevrNvrh9oYV2gLo4yWcjFlEo7h6O7zrERsIGwVGR0Nm+0Q EB5Hl0/8VmIQm/GG8IKgAUBZzM/97O/3l0WBvGRY9NraaE6dNB/XzonARyL1e2WNhFA0qd261LK aMFw= X-Google-Smtp-Source: AGHT+IEcH6Z5TIh9Q9fG+6Di5GU9wstZS66Z7Jz33UTwhgbZhXkVDnjMFz+nBKv7a4IA2MSBgS2aHQ== X-Received: by 2002:a17:902:64c8:b0:1d9:8b6c:2167 with SMTP id y8-20020a17090264c800b001d98b6c2167mr467482pli.23.1707156230438; Mon, 05 Feb 2024 10:03:50 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCUk0fjrHdSZe1g67NhZNp1VMcG00k4bbRS1JhLSUHjxfCeuBE4l4TRgvIkED8tfubIrzlGDe1mBaUhvXmHi0cNET4vBGD1JCQa2uAVttSwNf9SsociIMH+Vk8qxcRLg2V71T059eiaaTpbIuCB3qIiDK2LxxOAohVqMEiJszCty2t+Ty38ILsSlBemvr8GkkX6vyaDY1P1f Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:50 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Dariusz Sosnowski , Viacheslav Ovsiienko , Ori Kam , Suanming Mou , Matan Azrad Subject: [PATCH v9 13/23] net/mlx[45]: remove word sanity Date: Mon, 5 Feb 2024 09:43:41 -0800 Message-ID: <20240205180328.131019-14-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 "sanity" is on non-inclusive naming list. Remove it from the Nvidia drivers. Signed-off-by: Stephen Hemminger Acked-by: Dariusz Sosnowski --- drivers/common/mlx5/linux/mlx5_common_os.c | 2 +- drivers/net/mlx4/mlx4.c | 2 +- drivers/net/mlx4/mlx4_flow.c | 6 +++--- drivers/net/mlx5/mlx5_flow_dv.c | 2 +- drivers/net/mlx5/mlx5_flow_hw.c | 6 ++---- drivers/net/mlx5/mlx5_rxq.c | 2 +- drivers/net/mlx5/mlx5_rxtx_vec.h | 2 +- 7 files changed, 10 insertions(+), 12 deletions(-) diff --git a/drivers/common/mlx5/linux/mlx5_common_os.c b/drivers/common/mlx5/linux/mlx5_common_os.c index 41345e1597a4..2caa1a1da0b6 100644 --- a/drivers/common/mlx5/linux/mlx5_common_os.c +++ b/drivers/common/mlx5/linux/mlx5_common_os.c @@ -235,7 +235,7 @@ mlx5_get_ifname_sysfs(const char *ibdev_path, char *ifname) /** * Suffix RTE_EAL_PMD_PATH with "-glue". * - * This function performs a sanity check on RTE_EAL_PMD_PATH before + * This function performs a check on RTE_EAL_PMD_PATH before * suffixing its last component. * * @param buf[out] diff --git a/drivers/net/mlx4/mlx4.c b/drivers/net/mlx4/mlx4.c index a1a7e9328848..c167d7f90f97 100644 --- a/drivers/net/mlx4/mlx4.c +++ b/drivers/net/mlx4/mlx4.c @@ -1216,7 +1216,7 @@ static struct rte_pci_driver mlx4_driver = { /** * Suffix RTE_EAL_PMD_PATH with "-glue". * - * This function performs a sanity check on RTE_EAL_PMD_PATH before + * This function performs a check on RTE_EAL_PMD_PATH before * suffixing its last component. * * @param buf[out] diff --git a/drivers/net/mlx4/mlx4_flow.c b/drivers/net/mlx4/mlx4_flow.c index 8ef9fd2db44e..25b185d8710c 100644 --- a/drivers/net/mlx4/mlx4_flow.c +++ b/drivers/net/mlx4/mlx4_flow.c @@ -514,7 +514,7 @@ mlx4_flow_merge_tcp(struct rte_flow *flow, } /** - * Perform basic sanity checks on a pattern item. + * Perform basic checks on a pattern item. * * @param[in] item * Item specification. @@ -730,7 +730,7 @@ mlx4_flow_prepare(struct mlx4_priv *priv, goto exit_item_not_supported; proc = next; /* - * Perform basic sanity checks only once, while handle is + * Perform basic checks only once, while handle is * not allocated. */ if (flow == &temp) { @@ -799,7 +799,7 @@ mlx4_flow_prepare(struct mlx4_priv *priv, rss_key = mlx4_rss_hash_key_default; rss_key_len = MLX4_RSS_HASH_KEY_SIZE; } - /* Sanity checks. */ + /* Input checks. */ for (i = 0; i < rss->queue_num; ++i) if (rss->queue[i] >= ETH_DEV(priv)->data->nb_rx_queues) diff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_dv.c index 115d7303172d..254dc521c78e 100644 --- a/drivers/net/mlx5/mlx5_flow_dv.c +++ b/drivers/net/mlx5/mlx5_flow_dv.c @@ -8820,7 +8820,7 @@ flow_dv_prepare(struct rte_eth_dev *dev, #ifdef RTE_LIBRTE_MLX5_DEBUG /** - * Sanity check for match mask and value. Similar to check_valid_spec() in + * Check for match mask and value. Similar to check_valid_spec() in * kernel driver. If unmasked bit is present in value, it returns failure. * * @param match_mask diff --git a/drivers/net/mlx5/mlx5_flow_hw.c b/drivers/net/mlx5/mlx5_flow_hw.c index da873ae2e243..69bf3c51f886 100644 --- a/drivers/net/mlx5/mlx5_flow_hw.c +++ b/drivers/net/mlx5/mlx5_flow_hw.c @@ -7453,11 +7453,10 @@ flow_hw_tx_tag_regc_mask(struct rte_eth_dev *dev) struct mlx5_priv *priv = dev->data->dev_private; uint32_t mask = priv->sh->dv_regc0_mask; - /* Mask is verified during device initialization. Sanity checking here. */ + /* Mask is verified during device initialization. */ MLX5_ASSERT(mask != 0); /* * Availability of sufficient number of bits in REG_C_0 is verified on initialization. - * Sanity checking here. */ MLX5_ASSERT(rte_popcount32(mask) >= rte_popcount32(priv->vport_meta_mask)); return mask; @@ -7469,12 +7468,11 @@ flow_hw_tx_tag_regc_value(struct rte_eth_dev *dev) struct mlx5_priv *priv = dev->data->dev_private; uint32_t tag; - /* Mask is verified during device initialization. Sanity checking here. */ + /* Mask is verified during device initialization. */ MLX5_ASSERT(priv->vport_meta_mask != 0); tag = priv->vport_meta_tag >> (rte_bsf32(priv->vport_meta_mask)); /* * Availability of sufficient number of bits in REG_C_0 is verified on initialization. - * Sanity checking here. */ MLX5_ASSERT((tag & priv->sh->dv_regc0_mask) == tag); return tag; diff --git a/drivers/net/mlx5/mlx5_rxq.c b/drivers/net/mlx5/mlx5_rxq.c index 1bb036afebb3..fbe5cfd30201 100644 --- a/drivers/net/mlx5/mlx5_rxq.c +++ b/drivers/net/mlx5/mlx5_rxq.c @@ -1442,7 +1442,7 @@ mlx5_mprq_alloc_mp(struct rte_eth_dev *dev) desc *= 4; obj_num = desc + MLX5_MPRQ_MP_CACHE_SZ * n_ibv; /* - * rte_mempool_create_empty() has sanity check to refuse large cache + * rte_mempool_create_empty() has basic check to refuse large cache * size compared to the number of elements. * CACHE_FLUSHTHRESH_MULTIPLIER is defined in a C file, so using a * constant number 2 instead. diff --git a/drivers/net/mlx5/mlx5_rxtx_vec.h b/drivers/net/mlx5/mlx5_rxtx_vec.h index 77c3f4efa019..f8376f2b55b4 100644 --- a/drivers/net/mlx5/mlx5_rxtx_vec.h +++ b/drivers/net/mlx5/mlx5_rxtx_vec.h @@ -21,7 +21,7 @@ RTE_ETH_TX_OFFLOAD_OUTER_IPV4_CKSUM) /* - * Compile time sanity check for vectorized functions. + * Compile time checks for vectorized functions. */ #define S_ASSERT_RTE_MBUF(s) \ From patchwork Mon Feb 5 17:43:42 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136401 X-Patchwork-Delegate: thomas@monjalon.net 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 E7DF443A3D; Mon, 5 Feb 2024 19:05:28 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 74562427D9; Mon, 5 Feb 2024 19:03:57 +0100 (CET) Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) by mails.dpdk.org (Postfix) with ESMTP id 765E540EE7 for ; Mon, 5 Feb 2024 19:03:52 +0100 (CET) Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-1d93ddd76adso33943735ad.2 for ; Mon, 05 Feb 2024 10:03:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156231; x=1707761031; darn=dpdk.org; 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=mWApJfkT2be5eBBqtn0iJnwC4nYqTvztCGvaprbuHyg=; b=QKFp2wMS3jRxKwzXpKcwNNBaJ/hDTcQ0YiQ5cdP990d00NEVArif6IVfEdxA1wkUkp lJS/MUMABXKmpAmjjVZmc41R1DfnADnh2OxKch3eHpQlrz44uEH8tUXQwgFhvN8nS0oS zG8IkyQMu1hk9RNe4nEQSgWILSZdiMradYbi/pvWMpYIHd5hsH5mD1JwmMYMMgxxaZPk M+mcMavoLVlt2wHuI1rPw6hDfKkkVZwgPrgpu0t9K6x7jJhlNToOcGNJL9naL9CU2m6T GbFgDR6E473iTd1mGdmsjhYun4eLXCA68VWUApV3Ue7jI1GdaiINmCqT/FdIoOJQsrSE CEAQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156231; x=1707761031; 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=mWApJfkT2be5eBBqtn0iJnwC4nYqTvztCGvaprbuHyg=; b=pZ8JWv5nC/GqdIEzsgpIJzErrM+Xfqtod+hXNJDMnJHH0EJygd4PgB26qIMKNsZEvS 08ggyarhNoanA29Yw52ZJ6ob+aTHnPOVbNXWNOApvTh0CnvBgZjhFXt5cJogFzWERjJA NU3GY2JA52xqmdzhCgFLIu13u69QSNRd8ptD+AI+rUKKR23Kd/fOQCMgTwEucWoueuBX CWp9+t3bctVh7lQpmXZazPqQRmN8soTVvXboGNhDhfuyMyFbVKtYu2NUQ+AKH5zHcthB W5i5iwm8duAR7NYwNi4IGoa8+rZ+xDAyoUpz19UDt3S9yha+S6LVhKlfJEAAAzIB/eAb dehQ== X-Gm-Message-State: AOJu0YyMij4Pd7I+kkts78nr1CslOLsBHPvCO6hvgdSkNhsXA7SndXku mh3SCwiCkGBoouLwEQXD8kac9MxUHNuEqoFrwMskH5SThq0FIdK1VJa/nS1H0NzJoxfZHNFAHHf Z/y0= X-Google-Smtp-Source: AGHT+IG2jey0AQ+LFmWszZNnhDl7U96PcFHr11X0qPinZRz2YX4v/TWtbo96OSebzUUE8YXs/fNibA== X-Received: by 2002:a17:902:74ca:b0:1d9:591b:227a with SMTP id f10-20020a17090274ca00b001d9591b227amr287634plt.48.1707156231583; Mon, 05 Feb 2024 10:03:51 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCVvpIJezofG5WEK1r+UCjd5+tbH/h70aLd8q4xTM/Rp0Dz3m5IS57PuiyosXHLX9tx3ptx1Sxbg/kA8IAwKwkYR9vzCVyI0Jf5m7Fczmg== Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:50 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Andrew Rybchenko Subject: [PATCH v9 14/23] net/sfc: remove term "sanity check" Date: Mon, 5 Feb 2024 09:43:42 -0800 Message-ID: <20240205180328.131019-15-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Remove term sanity check in comment. Signed-off-by: Stephen Hemminger Acked-by: Andrew Rybchenko --- drivers/net/sfc/sfc_dp_rx.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/sfc/sfc_dp_rx.h b/drivers/net/sfc/sfc_dp_rx.h index 9f9bf2898881..dfb4fb98844b 100644 --- a/drivers/net/sfc/sfc_dp_rx.h +++ b/drivers/net/sfc/sfc_dp_rx.h @@ -59,7 +59,7 @@ struct sfc_dp_rx_qcreate_info { /** * Maximum number of Rx descriptors completed in one Rx event. - * Just for sanity checks if datapath would like to do. + * Checks if datapath would like to do. */ unsigned int batch_max; From patchwork Mon Feb 5 17:43:43 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136402 X-Patchwork-Delegate: thomas@monjalon.net 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 F206F43A3D; Mon, 5 Feb 2024 19:05:34 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7AC43427DB; Mon, 5 Feb 2024 19:03:58 +0100 (CET) Received: from mail-pl1-f170.google.com (mail-pl1-f170.google.com [209.85.214.170]) by mails.dpdk.org (Postfix) with ESMTP id 7C53440EF1 for ; Mon, 5 Feb 2024 19:03:53 +0100 (CET) Received: by mail-pl1-f170.google.com with SMTP id d9443c01a7336-1d51ba18e1bso42432215ad.0 for ; Mon, 05 Feb 2024 10:03:53 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156233; x=1707761033; darn=dpdk.org; 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=GPM0E+2aId52d1p9K/29wTQ3MJ9okC0u3+Y5FafgK+E=; b=NFR3A+TzSPuCquPnR45SJeTiQY2tY24PKjDbY5S6z61aMBhxaIpODwa7MrWSCN8XPl D4DpaTj21M+3FJDOueeQRGGgSQhs+oRtq/uzxUKt43N+0NWTP40pcSi+acLFGRpzlsbI mGynv6WHWzYdKj7a0GCnEHkdaGENFifluDbwVSKyOm6V6AHnhfvVUiP7Y59YC9+RdV4M PdGAZioGPDZvQ/UbnD1uBOXt576U+wPB0Q2BtJlTSa3hZ0ZszsQO1Ge4cYecTH3KiCT7 xvzSCdqrnILK91yfzbUaMriYECeEoSBUHZoCu5AVA+dk/kUVwy/75La5hwtY336A7rbb Vv+w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156233; x=1707761033; 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=GPM0E+2aId52d1p9K/29wTQ3MJ9okC0u3+Y5FafgK+E=; b=M6h2oRC1g0/Utukcg7cPd5GEMisc/F5FefR6ezHJF4dOYDP/5muozfXCW5ybtojeD/ f4TaUJFNSBCf3tsT45Ly72qxWTc58WMnpt7ifY1Wa0Opi1vT3v0CkrCvbDmNLy3Rm3FE p7RSS+X64GCiqNwvK9fReV77VqLxYXjKtxuJhMpV4rGYO7yql/B0G4RlmJqsYYDaHStk wlTqukE6Tle/OlyPIEqAZtt17Z6kUgFss5yelYzBNpPhW/vgwMHPpxpD7kVBFX9yM03p v4mgR9TQL8XB0wQViX54+geqL2ejZi6Kyg++PLHc3EhYFHNldFkl7LVhkxOSjryVxEGv Aqsw== X-Gm-Message-State: AOJu0Yx0tqVuY2q91hoXcVM22/qicB1kgvq7G15syrAVsjPNOyRD1n98 a+363JowFrhczf9YU8gifQXREvHyity82kqwP5Vzas4KZdpQ1tuwVXCcqq3xEH8lHEIkZV48GSN 9Hzo= X-Google-Smtp-Source: AGHT+IEqmCOjo1lNBXgCiL6I8+Knn7w1V/kAAjVxZqSVWpIcQl0TFU5RuB8pk4sC00jFibUOsuKYXA== X-Received: by 2002:a17:902:da81:b0:1d8:8aba:4b07 with SMTP id j1-20020a170902da8100b001d88aba4b07mr222813plx.60.1707156232401; Mon, 05 Feb 2024 10:03:52 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCU341SMs/eLgHkJJzLRWeLMJKINxtno77F6T/DaAY4o6OdNO5GWYLclFn4dgRzuuPvDIt6KFMgcsd9tQ5XO3VvQ4dKFpsoB5uDbFFmx5ixG8R9Cyo0OHhS9dkCMkRt47mdHqQO6TJVkbP91NBtY1Ab7GlJkccITYUnteFFF0T5+/QVumRo7 Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:52 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Shepard Siegel , Ed Czeck , John Miller Subject: [PATCH v9 15/23] net/ark: replace use of term sanity Date: Mon, 5 Feb 2024 09:43:43 -0800 Message-ID: <20240205180328.131019-16-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Do not use non inclusive terms. Signed-off-by: Stephen Hemminger Acked-by: Ed Czeck --- drivers/net/ark/ark_ethdev.c | 8 ++++---- drivers/net/ark/ark_ethdev_rx.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/net/ark/ark_ethdev.c b/drivers/net/ark/ark_ethdev.c index 0ffd4b9e9e4a..68edb542db43 100644 --- a/drivers/net/ark/ark_ethdev.c +++ b/drivers/net/ark/ark_ethdev.c @@ -346,10 +346,10 @@ eth_ark_dev_init(struct rte_eth_dev *dev) ARK_PMD_LOG(NOTICE, "Arkville HW Commit_ID: %08x\n", rte_be_to_cpu_32(ark->sysctrl.t32[0x20 / 4])); - /* If HW sanity test fails, return an error */ + /* If HW test fails, return an error */ if (ark->sysctrl.t32[4] != 0xcafef00d) { ARK_PMD_LOG(ERR, - "HW Sanity test has failed, expected constant" + "HW test has failed, expected constant" " 0x%x, read 0x%x (%s)\n", 0xcafef00d, ark->sysctrl.t32[4], __func__); @@ -357,7 +357,7 @@ eth_ark_dev_init(struct rte_eth_dev *dev) } ARK_PMD_LOG(DEBUG, - "HW Sanity test has PASSED, expected constant" + "HW test has PASSED, expected constant" " 0x%x, read 0x%x (%s)\n", 0xcafef00d, ark->sysctrl.t32[4], __func__); @@ -587,7 +587,7 @@ eth_ark_dev_start(struct rte_eth_dev *dev) rte_thread_t thread; /* Delay packet generator start allow the hardware to be ready - * This is only used for sanity checking with internal generator + * This is only used for checking with internal generator */ char tname[RTE_THREAD_INTERNAL_NAME_SIZE]; snprintf(tname, sizeof(tname), "ark-pg%d", dev->data->port_id); diff --git a/drivers/net/ark/ark_ethdev_rx.c b/drivers/net/ark/ark_ethdev_rx.c index 24f1c65690eb..8ac9d904e795 100644 --- a/drivers/net/ark/ark_ethdev_rx.c +++ b/drivers/net/ark/ark_ethdev_rx.c @@ -283,7 +283,7 @@ eth_ark_recv_pkts(void *rx_queue, mbuf->pkt_len = meta->pkt_len; mbuf->data_len = meta->pkt_len; - if (ARK_DEBUG_CORE) { /* debug sanity checks */ + if (ARK_DEBUG_CORE) { /* debug checks */ if ((meta->pkt_len > (1024 * 16)) || (meta->pkt_len == 0)) { From patchwork Mon Feb 5 17:43:44 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136403 X-Patchwork-Delegate: thomas@monjalon.net 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 5E90543A3D; Mon, 5 Feb 2024 19:05:41 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 78420427E4; Mon, 5 Feb 2024 19:03:59 +0100 (CET) Received: from mail-pl1-f172.google.com (mail-pl1-f172.google.com [209.85.214.172]) by mails.dpdk.org (Postfix) with ESMTP id 08623410FD for ; Mon, 5 Feb 2024 19:03:54 +0100 (CET) Received: by mail-pl1-f172.google.com with SMTP id d9443c01a7336-1d93f2c3701so24177075ad.3 for ; Mon, 05 Feb 2024 10:03:53 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156233; x=1707761033; darn=dpdk.org; 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=f8t+3TQp80Zr1JUJsxVAMQVqu2dEjh/W25YIO4CA8oo=; b=FaR/I8U6eeMTiA1GfiB0Egqbgm6urtdhEHYHzDdUTYqL1NhlOVbZ7A7DGDhE4C/EdZ ENEijJCohXvzw945F79OF8p5G/cOysYF78aKaO/ADyFCTtfU2AOfPHS+jXwEga9Mw9Ta qaEneqmjlqY+UGH9x6qpjRTJSqcQzoqrGVP6daqbiKvEmdALFE9fnCQqLDmn4WejavO/ d65RmzRNZgK9oIDtCiJ/gXWcNvL/QCoPBEuSwjI77hPyHzkfK9hp4fOZ5/Wjg7EEv+36 IWZqFirzdxC+k1p9XyFBtf1wOnITPc0p+CrBMZpeCWWc2M/ANAuvPUpxIk76sJhALO4a CHYQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156233; x=1707761033; 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=f8t+3TQp80Zr1JUJsxVAMQVqu2dEjh/W25YIO4CA8oo=; b=acfh0WRK/wt8iQbGSfzsVR9IZELFjvo5t0ap9dhaQUcECQtXBe3KZN9l9ngnbx2nis AfkK5JCfptV5wSwR3aygozIUlTKwOvbKCGuOqYfZFqvzqsIYx7w1X1u8fgiQ3M9B8zKD Tft1quXf3oZbRHqP7vQ16pRniVOKDrxbW6LtJunZKYuKc4foQEUnpVlT5HejgbMSWZ2X apGK1C6XBP638pEjfdnftVneBsWJXgGmwUzdxihDIj3d1UIqbJvo0tsEGXYFIn+sd1BP XJAzM+R1iBEy8i0BtxaH7rZva+bbdA5Do1TX4I8rpsifag5IZmcACYsIqGaz5FqpFJ4c dgjA== X-Gm-Message-State: AOJu0YwO21DhXC9UhEWKQ8lKr/EaLaR+4lSez5O7+7LOvLWMXwO5KFCq Hs9c0zyv8YTeFKWKGfKCNdqDIx9aFlaQdIdUKXGYTUj6soemyuOArEAdUVBRAsESeutqTmUYCuy ZoYw= X-Google-Smtp-Source: AGHT+IEAIpnND0Tu4IHaKenVH816xXCvOzxyU8VkZ3xh7WjqdUIEC1AJLzrZapv9N50sPZVJNRZPvg== X-Received: by 2002:a17:902:ea11:b0:1d9:a7fc:4c7 with SMTP id s17-20020a170902ea1100b001d9a7fc04c7mr294598plg.49.1707156233213; Mon, 05 Feb 2024 10:03:53 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCVrKhsLxk2Zmq2H9LxbgtVbgJlU02zykRste8/kBdOa1ZZaCneshxdW5AzkPk8/dkBqrT1SJybCdvx/hpPkDh6qtlDIbt++pdJ7ta9dcViO0y53fiEq6phwkl3tuaWj8rFxy74tHBQ= Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:52 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Ajit Khaparde , Somnath Kotur Subject: [PATCH v9 16/23] net/bnxt: replace use of term sanity Date: Mon, 5 Feb 2024 09:43:44 -0800 Message-ID: <20240205180328.131019-17-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Do not use non-inclusive terms. Signed-off-by: Stephen Hemminger --- drivers/net/bnxt/bnxt_hwrm.c | 2 +- drivers/net/bnxt/hsi_struct_def_dpdk.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c index 06f196760fa8..e1254375b650 100644 --- a/drivers/net/bnxt/bnxt_hwrm.c +++ b/drivers/net/bnxt/bnxt_hwrm.c @@ -262,7 +262,7 @@ static int bnxt_hwrm_send_message(struct bnxt *bp, void *msg, done = bnxt_check_cq_hwrm_done(cpr, is_tx, is_rx, i == timeout - 1); - /* Sanity check on the resp->resp_len */ + /* Input check on the resp->resp_len */ rte_io_rmb(); if (resp->resp_len && resp->resp_len <= bp->max_resp_len) { /* Last byte of resp contains the valid key */ diff --git a/drivers/net/bnxt/hsi_struct_def_dpdk.h b/drivers/net/bnxt/hsi_struct_def_dpdk.h index 9afdd056ce7a..a8076a6fc223 100644 --- a/drivers/net/bnxt/hsi_struct_def_dpdk.h +++ b/drivers/net/bnxt/hsi_struct_def_dpdk.h @@ -55482,7 +55482,7 @@ struct hwrm_tfc_ident_alloc_output { * Requests the firmware to free a TFC resource identifier. * A resource subtype and session id are passed in. * An identifier (previously allocated) corresponding to all these is - * freed, only after various sanity checks are completed. + * freed, only after various checks are completed. */ /* hwrm_tfc_ident_free_input (size:192b/24B) */ struct hwrm_tfc_ident_free_input { From patchwork Mon Feb 5 17:43:45 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136404 X-Patchwork-Delegate: thomas@monjalon.net 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 C738B43A3D; Mon, 5 Feb 2024 19:05:47 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8EC84427E6; Mon, 5 Feb 2024 19:04:00 +0100 (CET) Received: from mail-pl1-f180.google.com (mail-pl1-f180.google.com [209.85.214.180]) by mails.dpdk.org (Postfix) with ESMTP id BC7984111C for ; Mon, 5 Feb 2024 19:03:54 +0100 (CET) Received: by mail-pl1-f180.google.com with SMTP id d9443c01a7336-1d91397bd22so35994295ad.0 for ; Mon, 05 Feb 2024 10:03:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156234; x=1707761034; darn=dpdk.org; 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=P2oFxkSJHPebrZfcV+POmP1qNUsGTBu4dYoB/rm+LX8=; b=XgvUzA3Kr1lViU19ZBkGv5damDxO8V4iEHAAGZFf3q5Kmql90/uyFs7CH8iu8ZwIaP OiFwIt4vUOg79MngqfLVBw053fB0vdW9WDwuLp5isPB957yVlaIhB6o1Y7224CBlqUrg KkAjSDFrmRNz6ufzqx/LI5q+u538kjjb+p8X9MlZ47VgboycRnjTwxQk7A7oXcmqeXuk FoPw2INKLQsWGvNA38rduOhaygDFjXWC3NJ+7F3+HRwRujoRL5yQo2PieiSmXJi6My2L S2+oP1XIaSAO06fExI5TP4WhU+RXHf5S6dO0T4Wtt0p9kmzDui3+wA1emY0DG4+MflTf NEDw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156234; x=1707761034; 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=P2oFxkSJHPebrZfcV+POmP1qNUsGTBu4dYoB/rm+LX8=; b=VdxlK/tW4y1jYBi2Es6QMiD8KlJ+4hb5S+EwBlI3FRxyFLKKc0LGOm2OujhnmHO1wa IyYXM06Y47eIkYtn71mpGOCHa2LaXXBHLv6C9fTx3Qc8mxN2mmpenXSeyw6LURkJa7Yf kNo+Q5FS4ldCQvnGcIr+i5lpsXfL/6wE5sZaUKCrXs2RplnrfjRdqwdkeTcugdq6qiDB mdJmK4zndwDrA8Or1VO7QyQPV6//HlsnXHLbG26SelFkZhDNLy/m4qZ0OuDa0Q46mGtD Zr1RiBQV6GJvBsuoZuxCgqq8eJB66pwy9zigmMHDuJKc7nYEN0MlUtslp8aSj23ng0bS IcTA== X-Gm-Message-State: AOJu0YxGrRncI17zJzUJ7uBL0iH1PetgxUlx+IGsLLcSC4AQxgxxc+wZ w/Sg1nMKd2zUkYTp338UkQuoiyKl7tyHZym7WweZ4AUsMEhOzKd7gzXUGR/UnEwrPS40QZPtqt7 LrOY= X-Google-Smtp-Source: AGHT+IHBQbage7hXnRTqBNfUhY9Zq/amnEfAAGEoy6hqzOChS/DFPIU9vSZNBWa41CecpUb6Oe8JuA== X-Received: by 2002:a17:902:7584:b0:1d9:a589:aa6f with SMTP id j4-20020a170902758400b001d9a589aa6fmr252730pll.67.1707156233994; Mon, 05 Feb 2024 10:03:53 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCUySJkk9o3cbnfChBAC5i0yCp11gtguKBihsQ6yxQTuUgkXDVwjIfruq+V00XFj0W6BkZvPZql2e8yDsULKRNutOIvgPQ== Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:53 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Julien Aube Subject: [PATCH v9 17/23] net/bnx2x: remove reference to sanity Date: Mon, 5 Feb 2024 09:43:45 -0800 Message-ID: <20240205180328.131019-18-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Replace "sanity check" Signed-off-by: Stephen Hemminger --- drivers/net/bnx2x/bnx2x.c | 2 +- drivers/net/bnx2x/bnx2x_stats.c | 8 ++++---- drivers/net/bnx2x/ecore_sp.c | 2 +- drivers/net/bnx2x/elink.c | 4 ++-- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/net/bnx2x/bnx2x.c b/drivers/net/bnx2x/bnx2x.c index c3283c94f38f..e345f293f25c 100644 --- a/drivers/net/bnx2x/bnx2x.c +++ b/drivers/net/bnx2x/bnx2x.c @@ -7820,7 +7820,7 @@ static int bnx2x_check_valid_mf_cfg(struct bnx2x_softc *sc) uint32_t ovlan2; uint8_t i, j; - /* various MF mode sanity checks... */ + /* various MF mode checks... */ if (mf_info->mf_config[SC_VN(sc)] & FUNC_MF_CFG_FUNC_HIDE) { PMD_DRV_LOG(NOTICE, sc, diff --git a/drivers/net/bnx2x/bnx2x_stats.c b/drivers/net/bnx2x/bnx2x_stats.c index c07b01510a06..86d90679ac0a 100644 --- a/drivers/net/bnx2x/bnx2x_stats.c +++ b/drivers/net/bnx2x/bnx2x_stats.c @@ -187,7 +187,7 @@ bnx2x_stats_pmf_update(struct bnx2x_softc *sc) */ return; } - /* sanity */ + /* input valid? */ if (!sc->port.pmf || !sc->port.port_stx) { PMD_DRV_LOG(ERR, sc, "BUG!"); return; @@ -237,7 +237,7 @@ bnx2x_port_stats_init(struct bnx2x_softc *sc) uint32_t mac_addr; uint32_t *stats_comp = BNX2X_SP(sc, stats_comp); - /* sanity */ + /* input valid? */ if (!sc->link_vars.link_up || !sc->port.pmf) { PMD_DRV_LOG(ERR, sc, "BUG!"); return; @@ -461,7 +461,7 @@ bnx2x_func_stats_init(struct bnx2x_softc *sc) struct dmae_command *dmae = &sc->stats_dmae; uint32_t *stats_comp = BNX2X_SP(sc, stats_comp); - /* sanity */ + /* input valid? */ if (!sc->func_stx) { PMD_DRV_LOG(ERR, sc, "BUG!"); return; @@ -1301,7 +1301,7 @@ bnx2x_port_stats_base_init(struct bnx2x_softc *sc) struct dmae_command *dmae; uint32_t *stats_comp = BNX2X_SP(sc, stats_comp); - /* sanity */ + /* input valid? */ if (!sc->port.pmf || !sc->port.port_stx) { PMD_DRV_LOG(ERR, sc, "BUG!"); return; diff --git a/drivers/net/bnx2x/ecore_sp.c b/drivers/net/bnx2x/ecore_sp.c index c6c3857778f7..158205266f62 100644 --- a/drivers/net/bnx2x/ecore_sp.c +++ b/drivers/net/bnx2x/ecore_sp.c @@ -207,7 +207,7 @@ static int ecore_exe_queue_step(struct bnx2x_softc *sc, break; } - /* Sanity check */ + /* Argument check */ if (!cur_len) return ECORE_SUCCESS; diff --git a/drivers/net/bnx2x/elink.c b/drivers/net/bnx2x/elink.c index 2c81f85b9605..6ebd47d927f8 100644 --- a/drivers/net/bnx2x/elink.c +++ b/drivers/net/bnx2x/elink.c @@ -1088,7 +1088,7 @@ static void elink_get_epio(struct bnx2x_softc *sc, uint32_t epio_pin, { uint32_t epio_mask, gp_oenable; *en = 0; - /* Sanity check */ + /* Input check */ if (epio_pin > 31) { ELINK_DEBUG_P1(sc, "Invalid EPIO pin %d to get", epio_pin); return; @@ -1105,7 +1105,7 @@ static void elink_set_epio(struct bnx2x_softc *sc, uint32_t epio_pin, uint32_t e { uint32_t epio_mask, gp_output, gp_oenable; - /* Sanity check */ + /* Input check */ if (epio_pin > 31) { ELINK_DEBUG_P1(sc, "Invalid EPIO pin %d to set", epio_pin); return; From patchwork Mon Feb 5 17:43:46 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136405 X-Patchwork-Delegate: thomas@monjalon.net 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 0BA4C43A3D; Mon, 5 Feb 2024 19:05:57 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 186A6427E8; Mon, 5 Feb 2024 19:04:02 +0100 (CET) Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) by mails.dpdk.org (Postfix) with ESMTP id 8C33D41132 for ; Mon, 5 Feb 2024 19:03:55 +0100 (CET) Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-1d72f71f222so37530245ad.1 for ; Mon, 05 Feb 2024 10:03:55 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156235; x=1707761035; darn=dpdk.org; 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=venPue0YKmgFqmTkb3A+szF8IVhNfbQaBZV5vfYyiy4=; b=wZAv5sh+Gd4YvwuyEpUUmT3CtLcyllD9hCTrVPnoHfAJ5S2l57ZPomO+QC7Cdi/JXf iZBDZWoal8SZx7Oz2QreClqOZUmB/rw/3yvQQRx3zdxSgYwN8H2xZ33j2YTwuktdYw/A QBcA0hezq22XgILe2y1QUrHIdKtRgp3TWIBSihNG7/WcvHEkatbQgSFAkVMDm+q7RxUO ZBY8pJFfhDIq5GQrMQP8wik+DipYWd6lnLMAJADeuhRWmMN35gqjsBvSYzYctk3TYbOD BVL+6kzCrQS62MlH+WkBDmUTSZxzgvVyLku8ubo3DGu6PuImgZMoE9xXV4CxBzwy+9gT u2DA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156235; x=1707761035; 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=venPue0YKmgFqmTkb3A+szF8IVhNfbQaBZV5vfYyiy4=; b=EWCm/6YHXAemDwteKw8IDPsgKvcSi31un1ocg7aphyKDLFMPWGRhf8x+8zighbwNdV zoracP7gR0LDpzf43fx1xt6saGKU3O5fF+xNzHBn+SrsWKDIjFIjsnlhDlHM+avjSVuQ Dx/dcLx3Sf39Hb//yUevxJC6I08FD2TNjJXmwMzcWNSoDHDuF7SduSQvQliCradyiOIC RIit/EvKHeH+BDVhbNDl5+xX6Pw1uZsOHckFsVEd0Pe+9HiKXNJC3m+eXEtv+cVudQic Iwe3jfoslYdbHIqalRovKGIw+1mRPYvpizvkIi8dONONRunpK5vjiVfycilhQjLKySQV uSLw== X-Gm-Message-State: AOJu0YzzuSnaBYLEbNl+d481k6WqF1+ML/gK98XDT29ut/peZTd+IUfp KhxgMwvrVMCQz8Pzl1u4Sm13nM6hiqw3QLR+Xm4T0ZaWuX2Mw9IC6pxZYa5rY5rKsMcZPyhV5o0 AkIQ= X-Google-Smtp-Source: AGHT+IHEgyoedR3OMk42gXPTUT1d1uzyErdegPMT31igivisZpdDKyfVYn9bf6087l9MUK8ENp8htw== X-Received: by 2002:a17:903:2782:b0:1d9:a674:aa77 with SMTP id jw2-20020a170903278200b001d9a674aa77mr242862plb.24.1707156234783; Mon, 05 Feb 2024 10:03:54 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCWf2uwE0rDa7I8+ds7efKnlKv/jOJRfPv2Efb4xXhJmvimq97XkF7j+NkEHzqQ+kNHD04e/pTcEu8jIqVjk84LaOp0QlAa0bMzuXdyzuOhbrva9HRpZ3LDdgvr/ffOjEAez1Rl/pNqjPUEorowvSWkOUxpofFjQHhiR+rOGR0BZ+aI5P6uoDbg0bq6E Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:54 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Nithin Dabilpuram , Kiran Kumar K , Sunil Kumar Kori , Satha Rao Subject: [PATCH v9 18/23] cnxk: replace term sanity Date: Mon, 5 Feb 2024 09:43:46 -0800 Message-ID: <20240205180328.131019-19-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Do not use non-inclusive naming. Signed-off-by: Stephen Hemminger --- drivers/common/cnxk/roc_nix_tm_ops.c | 2 +- drivers/common/cnxk/roc_npa.c | 4 ++-- drivers/net/cnxk/cnxk_ethdev.c | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/common/cnxk/roc_nix_tm_ops.c b/drivers/common/cnxk/roc_nix_tm_ops.c index e1cef7a670b6..6060dba0d799 100644 --- a/drivers/common/cnxk/roc_nix_tm_ops.c +++ b/drivers/common/cnxk/roc_nix_tm_ops.c @@ -527,7 +527,7 @@ roc_nix_tm_hierarchy_disable(struct roc_nix *roc_nix) node->flags &= ~NIX_TM_NODE_ENABLED; } - /* Verify sanity of all tx queues */ + /* Verify all tx queues */ for (i = 0; i < sq_cnt; i++) { sq = nix->sqs[i]; if (!sq) diff --git a/drivers/common/cnxk/roc_npa.c b/drivers/common/cnxk/roc_npa.c index 6c14c499019a..5e38ecf0414e 100644 --- a/drivers/common/cnxk/roc_npa.c +++ b/drivers/common/cnxk/roc_npa.c @@ -458,7 +458,7 @@ npa_aura_pool_pair_alloc(struct npa_lf *lf, const uint32_t block_size, char name[PLT_MEMZONE_NAMESIZE]; const struct plt_memzone *mz; - /* Sanity check */ + /* argument checks */ if (!lf || !block_size || !block_count || !pool || !aura || !aura_handle) return NPA_ERR_PARAM; @@ -1096,7 +1096,7 @@ npa_dev_init(struct npa_lf *lf, uintptr_t base, struct mbox *mbox) uint8_t aura_sz; int rc; - /* Sanity checks */ + /* Input checks */ if (!lf || !base || !mbox) return NPA_ERR_PARAM; diff --git a/drivers/net/cnxk/cnxk_ethdev.c b/drivers/net/cnxk/cnxk_ethdev.c index 5e11bbb017ea..798567592efe 100644 --- a/drivers/net/cnxk/cnxk_ethdev.c +++ b/drivers/net/cnxk/cnxk_ethdev.c @@ -636,7 +636,7 @@ cnxk_nix_rx_queue_setup(struct rte_eth_dev *eth_dev, uint16_t qid, struct rte_mempool *lpb_pool = mp; struct rte_mempool *spb_pool = NULL; - /* Sanity checks */ + /* input checks */ if (rx_conf->rx_deferred_start == 1) { plt_err("Deferred Rx start is not supported"); goto fail; @@ -1205,7 +1205,7 @@ cnxk_nix_configure(struct rte_eth_dev *eth_dev) rc = -EINVAL; - /* Sanity checks */ + /* baseline checks */ if (rte_eal_has_hugepages() == 0) { plt_err("Huge page is not configured"); goto fail_configure; From patchwork Mon Feb 5 17:43:47 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136406 X-Patchwork-Delegate: thomas@monjalon.net 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 CC08D43A3D; Mon, 5 Feb 2024 19:06:01 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 2131A42D66; Mon, 5 Feb 2024 19:04:03 +0100 (CET) Received: from mail-pl1-f171.google.com (mail-pl1-f171.google.com [209.85.214.171]) by mails.dpdk.org (Postfix) with ESMTP id 7D2614113D for ; Mon, 5 Feb 2024 19:03:56 +0100 (CET) Received: by mail-pl1-f171.google.com with SMTP id d9443c01a7336-1d944e8f367so36683735ad.0 for ; Mon, 05 Feb 2024 10:03:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156236; x=1707761036; darn=dpdk.org; 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=ujWWwdq2QX2nV306iarfVcMWhf8ykUwwoCc8Glf1X18=; b=wWMh+yxTtMm8Pqly6IhnYDBoBDFVvIAGY4Te1q3ZLQHh0z4aNa2VCMos3mCcfC1ybt 91jicu2+vdDCgYZXFdIvG69I6tmcV5fMT1VgCb0NxKf4w+pSLw9HNwbSopdZe7g5EDEH c00ev1SzCtDG0BLCClzp+fhtQYBbd0e3DY/ORDbdu52v6tIx9Ny715F2bcQ61Iv8+9Or /ITaWDbqBTIwH+Br/ZYaQ14S9VCiWCWazq/7e4AVAG+s4myLKDhU3zAsyQgEyAL4763h EeAprxO9DCe+xBhqNajvBQnaEFUBiSPG0Z3D8Nwf6QjivqqIe5ZiURUvY4bUxl5vFE1S XlCw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156236; x=1707761036; 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=ujWWwdq2QX2nV306iarfVcMWhf8ykUwwoCc8Glf1X18=; b=oHtUdWbge88KV3oRhC4kpoR5Gik/OQWOoYDlJo55QwDUl9lNp/4/IyGkqB3ddN4AmQ fyS/3aGAgP3k27mbnyR19WrcXuWbhOQ0zbMzjNW/IjzqRcrll5J54FNCVgtlexYw4MbJ d9CtpabkHNL7lQLUnc+NuFio/yrcxaV8hKhSz2SmyLDob9Udr1/ThpDWmI8iQ0G2LgRL BeTyHDQiB7BrNx1PaP5zUPZiDi8tlgSTLYQ6xhrN6dP1C0ybKQna5ywPRkLXN5VwVw13 8Y0Rf/i/75JfP3/3UwZaxeaZlB1tLcmt0xo9r7hwcqsoqTL+HVQ3emVIbmpUiBjxa+Nt 0QVw== X-Gm-Message-State: AOJu0YxylvpSqPTOrQcaD8qqE8up8GuEGpzaU7cU1k/ao+FB/S08LTRb nbnEpEhvMgPaR7jXY1qntc1KseAQt1Fuzcn0ChL8KmH0CqgcKA6yNPPLyWtXBKIiH34Uqi/zNMu HN/U= X-Google-Smtp-Source: AGHT+IEX34X2EnhG9nkvQAK6if4PW4Ncfz9y14mgmEv/ZYStsNqf7w6g8Tni9GQeyCKc54J1XbGQiw== X-Received: by 2002:a17:903:1104:b0:1d9:657c:2769 with SMTP id n4-20020a170903110400b001d9657c2769mr371923plh.19.1707156235729; Mon, 05 Feb 2024 10:03:55 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCUnWrikwyfZNUKsx8XW+EvxSHW3A776jvXtzQOKXr1+llVdQPL5E/ebaomKIk+/PyIyxFEV8vSxp6V2zIfwAp5fiVO9mkf8tt7pywi4lKruPwUTJfeLFZ0EHsReUiK+ Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:55 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Liang Ma , Peter Mccarthy Subject: [PATCH v9 19/23] event/opdl: remove term sanity Date: Mon, 5 Feb 2024 09:43:47 -0800 Message-ID: <20240205180328.131019-20-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Don't use non-inclusive term. Signed-off-by: Stephen Hemminger --- drivers/event/opdl/opdl_evdev.c | 2 +- drivers/event/opdl/opdl_evdev_init.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/event/opdl/opdl_evdev.c b/drivers/event/opdl/opdl_evdev.c index 0cccaf7e9731..39cd87b8f944 100644 --- a/drivers/event/opdl/opdl_evdev.c +++ b/drivers/event/opdl/opdl_evdev.c @@ -241,7 +241,7 @@ opdl_queue_setup(struct rte_eventdev *dev, struct opdl_evdev *device = opdl_pmd_priv(dev); - /* Extra sanity check, probably not needed */ + /* Extra check, probably not needed */ if (queue_id == OPDL_INVALID_QID) { PMD_DRV_LOG(ERR, "DEV_ID:[%02d] : " "Invalid queue id %u requested\n", diff --git a/drivers/event/opdl/opdl_evdev_init.c b/drivers/event/opdl/opdl_evdev_init.c index be3437fc2f40..12eb2695f5de 100644 --- a/drivers/event/opdl/opdl_evdev_init.c +++ b/drivers/event/opdl/opdl_evdev_init.c @@ -300,7 +300,7 @@ static int opdl_add_deps(struct opdl_evdev *device, struct opdl_queue *queue_deps = &device->queue[deps_q_id]; struct opdl_stage *dep_stages[OPDL_PORTS_MAX]; - /* sanity check that all stages are for same opdl ring */ + /* check that all stages are for same opdl ring */ for (i = 0; i < queue->nb_ports; i++) { struct opdl_ring *r = opdl_stage_get_opdl_ring(stage_for_port(queue, i)); From patchwork Mon Feb 5 17:43:48 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136407 X-Patchwork-Delegate: thomas@monjalon.net 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 BCC5243A3D; Mon, 5 Feb 2024 19:06:08 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1E03042D72; Mon, 5 Feb 2024 19:04:04 +0100 (CET) Received: from mail-pl1-f182.google.com (mail-pl1-f182.google.com [209.85.214.182]) by mails.dpdk.org (Postfix) with ESMTP id 61FB44161A for ; Mon, 5 Feb 2024 19:03:57 +0100 (CET) Received: by mail-pl1-f182.google.com with SMTP id d9443c01a7336-1d73066880eso40763845ad.3 for ; Mon, 05 Feb 2024 10:03:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156236; x=1707761036; darn=dpdk.org; 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=lT/4FwNf0uq2QP0fvygzZtYOTCeo5EK/ZA8tz8D24Dw=; b=jXQPE/HWXfiotpBT6hnovlLMcpBnUUxUcaOa9kaLBcPBKp6MJ+ZVI421PBheYdyPzx WhFh5B7qjk5z3yVvnRz5f/A8RA++aYFZZ5YCUS+DcjCpfBOWnRCg5+gkrfhnjQQL12li xpHcOoU1oiNOLFucjkoX8NDqfuAop8PrL5HTwrBEC5dEVOkFFtGckQag3ksGPS/sD/5m pn/CabCIlPL+XrBIqy7zXYk9g9maAEIviYCEsuxNfYX3R4DB4USjvmfA9Qpf/8G1hvAN hUVB/5P5JpRF7ecZv65N6/zOBUiq0tD1HNk72Lanq7qgLbUpGU7wPsc7hI04bK06B3QC mAfQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156236; x=1707761036; 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=lT/4FwNf0uq2QP0fvygzZtYOTCeo5EK/ZA8tz8D24Dw=; b=YmB5/czdU2C5LNABsG9jRTXOi7leYVtVd+of30H/oWjBR1Ydq1yJIFehq0HmpUcfBD xnC8Rk1LuiEdtPaOjkc2evjogSRF5jH6HwGQtyBb3GT6GsDNcHMYjqjc2V1qpv0vOQT2 70nlIW9+uuz5P8pdayq7fJWrl+FsWrIOmSW0ypj3KWaVqMAviuwWDEmDMiHYAugAggre SsqOG5vLGkyZH5RRGRj48nT1riQTCcTvzYxvHFP3aaTRjovWOLFssVCH8PEV4HEBluoU gl0Ed0WKtJ0nEceQZFEja8wVbyoJXW2ukdYbrkf2RYib9yF4qNN/NpTgKmmKAVPFXK22 TiHA== X-Gm-Message-State: AOJu0YzF5cPNsJekw8VKmZLl4TVQty5k6uWI2KtrEJqfHTdiI4UWsN8h YTR5A/Vd6eIuokt+QjQyhNLgG4IL+/Y3Lepq96lotUsMc248F8wopiziQsCL8h1c3hd2JNvzqsN 35q8= X-Google-Smtp-Source: AGHT+IGZsFmsrmFfq5T2hcG2Xi9JewkKUyCXIYqX0Ws+fqcptwBXJ37Wi5YeexbB39p4M0pI/WpMIA== X-Received: by 2002:a17:902:e88f:b0:1d9:1359:75ec with SMTP id w15-20020a170902e88f00b001d9135975ecmr259519plg.30.1707156236674; Mon, 05 Feb 2024 10:03:56 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCVczmsQavlEwXh2JiShXhVP+91seoRGjTkSSXImW5x6PomYw2Ef0zwwtKengb789HDHetLMv0b2EVgbr9/El4WDQk0NupdbkaXqtaGIPvk0oZCCjPWROwXkCjz6z+6hFn4= Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:56 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Jiawen Wu , Jian Wang Subject: [PATCH v9 20/23] net/txgbe: replace term sanity Date: Mon, 5 Feb 2024 09:43:48 -0800 Message-ID: <20240205180328.131019-21-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Don't use term sanity. Signed-off-by: Stephen Hemminger --- drivers/net/txgbe/txgbe_ipsec.c | 2 +- drivers/net/txgbe/txgbe_rxtx.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/txgbe/txgbe_ipsec.c b/drivers/net/txgbe/txgbe_ipsec.c index f9f8108fb894..cadb0943c82d 100644 --- a/drivers/net/txgbe/txgbe_ipsec.c +++ b/drivers/net/txgbe/txgbe_ipsec.c @@ -595,7 +595,7 @@ txgbe_crypto_enable_ipsec(struct rte_eth_dev *dev) rx_offloads = dev->data->dev_conf.rxmode.offloads; tx_offloads = dev->data->dev_conf.txmode.offloads; - /* sanity checks */ + /* offload checks */ if (rx_offloads & RTE_ETH_RX_OFFLOAD_TCP_LRO) { PMD_DRV_LOG(ERR, "RSC and IPsec not supported"); return -1; diff --git a/drivers/net/txgbe/txgbe_rxtx.c b/drivers/net/txgbe/txgbe_rxtx.c index 1cd4b25965b5..5d45d8763dc9 100644 --- a/drivers/net/txgbe/txgbe_rxtx.c +++ b/drivers/net/txgbe/txgbe_rxtx.c @@ -4115,7 +4115,7 @@ txgbe_set_rsc(struct rte_eth_dev *dev) uint32_t rdrxctl; uint32_t rfctl; - /* Sanity check */ + /* Offload check */ dev->dev_ops->dev_infos_get(dev, &dev_info); if (dev_info.rx_offload_capa & RTE_ETH_RX_OFFLOAD_TCP_LRO) rsc_capable = true; From patchwork Mon Feb 5 17:43:49 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136408 X-Patchwork-Delegate: thomas@monjalon.net 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 A058043A3D; Mon, 5 Feb 2024 19:06:14 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0237542831; Mon, 5 Feb 2024 19:04:05 +0100 (CET) Received: from mail-pg1-f169.google.com (mail-pg1-f169.google.com [209.85.215.169]) by mails.dpdk.org (Postfix) with ESMTP id 23AD6427DB for ; Mon, 5 Feb 2024 19:03:58 +0100 (CET) Received: by mail-pg1-f169.google.com with SMTP id 41be03b00d2f7-5c229dabbb6so3040089a12.0 for ; Mon, 05 Feb 2024 10:03:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156237; x=1707761037; darn=dpdk.org; 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=6YqyIHEczoDsNiea26GNzlsErxCIo0MJI7s9o1RXckA=; b=j7QxTvpo7Gb+ht1mc3rVyXksV7Upkb/LV8D/X00RAfFTQzSA/yU8BTE1+u8OALVwMQ gfNcZ8xapNQqX26Hw6+fKrcQJq4AwbFY3pWv2DI6DAH/jHzNc8VydPinITjIfgexaf37 KcXw7unNlXuSyHD9TBrfkM0bQJF0+x4Bp+MRQQZslnnaowIZqlUIyKV7Gg36dQkxB0yG hqwJ+hkFN4smQhpAvm4tGpLMxAVBdZUuxlEfnnCOneYAgmW6i6aqMrT55cqvg1a5jpL1 xzEaXosHPLlbC/Ft12TpIpv8a4l2bBwSSvXcD7gTE5G4g8vYv45K9J6AgfapN51GDlqo 2cyw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156237; x=1707761037; 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=6YqyIHEczoDsNiea26GNzlsErxCIo0MJI7s9o1RXckA=; b=rrpzDzU55PpXx8KMKLbqOoctBWw5y54DWDY3JGInMn1VvvKACjfLoOLTWl7CTOH1z0 Gusjg1KKKU3p7cYR74RgC4H0WuB7I7mBP7kWPGOmG8iy/bKEAPdxRPm7eH4oN4SLSVvv 7PUO7hzezWWe8X2qCWZbWt2pLrCPUlmb6g8H/SKSCeNZrcetH0DYaX4M2qPycZwTbPa3 vd7CxxBbnPYwhh3QWts8QX7RjcK36tR972LGgs81p6xfFzP5Nbdr+hyr4HH1gActnKFB dZH0R2rVjPRJ3M4GUJ7adPlsc77swQEePmnCw/sXVB2d7IoBHvXF+UDkBk713hydl1QP k8Hw== X-Gm-Message-State: AOJu0Yx5es3TBiHudYK1gcWKAGZlEK8PTM9XRYjbyVITr2WbKvvT6mQW BlwblhFjyyfBT2GRE41x2LSbPN9lFR6jbQ3hXXIEDkCFkBHTLwUw9MjGV+j/2jdXySkMfB7ddHJ 2mlM= X-Google-Smtp-Source: AGHT+IGFna99CNUhNJwEIJ5wYJfX4m6bPZmD8PVAx4Ugl/dvRG9SWYDjWz7qzGmtFly2I06mgTShgA== X-Received: by 2002:a17:90b:1052:b0:296:3804:c380 with SMTP id gq18-20020a17090b105200b002963804c380mr168359pjb.46.1707156237377; Mon, 05 Feb 2024 10:03:57 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCUi5S/PxvcI7TblNOxwDaR1jB5umDb7PHcfq9COLDbqreYaErUPO18APgVOX5Cqr1T5V0qgAUbKZbrEKR3qyGjT43B4axIzEn7TVdwr Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:57 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Rahul Lakkireddy Subject: [PATCH v9 21/23] net/cxgbe: remove use of term sanity Date: Mon, 5 Feb 2024 09:43:49 -0800 Message-ID: <20240205180328.131019-22-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Remove non-inclusive terminolgy. Signed-off-by: Stephen Hemminger --- drivers/net/cxgbe/cxgbe_ethdev.c | 10 ++-------- drivers/net/cxgbe/cxgbevf_main.c | 4 ++-- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/drivers/net/cxgbe/cxgbe_ethdev.c b/drivers/net/cxgbe/cxgbe_ethdev.c index 8cc3d9f257a0..c7c745fee495 100644 --- a/drivers/net/cxgbe/cxgbe_ethdev.c +++ b/drivers/net/cxgbe/cxgbe_ethdev.c @@ -524,10 +524,7 @@ int cxgbe_dev_tx_queue_setup(struct rte_eth_dev *eth_dev, eth_dev->data->tx_queues[queue_idx] = (void *)txq; - /* Sanity Checking - * - * nb_desc should be > 1023 and <= CXGBE_MAX_RING_DESC_SIZE - */ + /* nb_desc should be > 1023 and <= CXGBE_MAX_RING_DESC_SIZE */ temp_nb_desc = nb_desc; if (nb_desc < CXGBE_MIN_RING_DESC_SIZE) { dev_warn(adapter, "%s: number of descriptors must be >= %d. Using default [%d]\n", @@ -648,10 +645,7 @@ int cxgbe_dev_rx_queue_setup(struct rte_eth_dev *eth_dev, eth_dev->data->rx_queues[queue_idx] = (void *)rxq; - /* Sanity Checking - * - * nb_desc should be > 0 and <= CXGBE_MAX_RING_DESC_SIZE - */ + /* nb_desc should be > 0 and <= CXGBE_MAX_RING_DESC_SIZE */ temp_nb_desc = nb_desc; if (nb_desc < CXGBE_MIN_RING_DESC_SIZE) { dev_warn(adapter, "%s: number of descriptors must be >= %d. Using default [%d]\n", diff --git a/drivers/net/cxgbe/cxgbevf_main.c b/drivers/net/cxgbe/cxgbevf_main.c index d0c93f8ac3a5..dfb12b4ed5c9 100644 --- a/drivers/net/cxgbe/cxgbevf_main.c +++ b/drivers/net/cxgbe/cxgbevf_main.c @@ -137,7 +137,7 @@ static int adap_init0vf(struct adapter *adapter) /* * Grab our Virtual Interface resource allocation, extract the - * features that we're interested in and do a bit of sanity testing on + * features that we're interested in and do a bit of testing on * what we discover. */ err = t4vf_get_vfres(adapter); @@ -148,7 +148,7 @@ static int adap_init0vf(struct adapter *adapter) } /* - * Check for various parameter sanity issues. + * Check for various parameter issues. */ if (adapter->params.vfres.pmask == 0) { dev_err(adapter->pdev_dev, "no port access configured\n" From patchwork Mon Feb 5 17:43:50 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136409 X-Patchwork-Delegate: thomas@monjalon.net 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 02E6343A3D; Mon, 5 Feb 2024 19:06:21 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id DA79142D87; Mon, 5 Feb 2024 19:04:05 +0100 (CET) Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) by mails.dpdk.org (Postfix) with ESMTP id 563BA427DF for ; Mon, 5 Feb 2024 19:03:59 +0100 (CET) Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-1d7881b1843so41531965ad.3 for ; Mon, 05 Feb 2024 10:03:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156238; x=1707761038; darn=dpdk.org; 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=4TOBL+h2UZrP4REYTyUj9I4YMTvzfdogvnH8ToEgRqY=; b=BENz9UU0kL97qo4AtRnC+Za2F6DkgOY894GJpzXfEWWFhwa9MKNwernVxlLQwLaO1e sKLUdTxAzvJezRsI0Qxo6e1wVd3V8oZAjEx42n33jLx21jabr4fDikFZALrntH9MtAht qfAPSCTCrrSRbctCV5/vlVqSAzqL0073+aTreKVybt1uj6Da0tsPFuZ9b2eBuzo+z7bh S3fAvI29GAbT5BLvNabn5UYRtCu0indLnsB3Ijq8HtfMKGFMKyww5tDHIPklxtDHmMxD YiJ8xA1b1OqpkbQ1KSnnxV0d4W/eMEu1NWRwAmyXakisv7jMUlgDFP8qNFklwrw7lizH w8PA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156238; x=1707761038; 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=4TOBL+h2UZrP4REYTyUj9I4YMTvzfdogvnH8ToEgRqY=; b=r9mUo4Fp6QusVkN+gR/ONyxo96BCWrRGH7Pci9OSBEc/xt9xwfPR6SQh8EeJ+xCj8o /dzSU99c7tjnD5xRRJrJHnkMh6FIvRimJxnuGFihQlVhcH55vj7txlnHogcmvorgJ4P/ lTVUHENFQWlFkM0zpZvyxZF0Gs1cR/ctDWidjK+wJkAa5GDmFNRr9Jz3qwsk73yTRJ5s DEHWxwXzsDQ13RCMqfp920UIegakT0Yl0KSf+KGTdq6JIDir9oPY/B1vJbBSsj0pDRW9 PS5SjMTIfqwXptO7WsfWpuZgj2x4hLkufuVP77ds/GjUvxYqPiGuebbndyW2VSlAro3J TAyQ== X-Gm-Message-State: AOJu0YyITgheoq8W3JBRm//JWkh1zCT6j6cBKOAn+UAISEZN3vt8PYt5 X3gTl/KmtKMs9j6CHbB9RT5/u5f8Knek6wCmXtN9SjHWbqcgJtjMi6RVzXICSCVLvv8qCwQCSdX cXcM= X-Google-Smtp-Source: AGHT+IGYXR/vtbWk49xjkDMIkstyaC6v4FzTRXOCieucFka6ZIJ7CKBvuObWuEsmCSWn/A1tgyNwDA== X-Received: by 2002:a17:902:da81:b0:1d8:8aba:4b07 with SMTP id j1-20020a170902da8100b001d88aba4b07mr223138plx.60.1707156238140; Mon, 05 Feb 2024 10:03:58 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCWpKCsqxwMJnGo7p0zykJ1tSg5t3DYQzDNHuhbc8KSCC727u4yXzf2P9ng8sidYhrEteQWC3riIYj1ukCAfGNpbE8nd1hciVaPi/Q2kV/yBCphUjPic3oTH0Epv7RhLYIDlBjLvzZIMR82fBO/A/kWpflgBI526YnelRX2wvkqvXY/acZYEQBU= Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:57 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Ajit Khaparde , Raveendra Padasalagi , Vikas Gupta Subject: [PATCH v9 22/23] crypto/bcmfs: replace term sanity check Date: Mon, 5 Feb 2024 09:43:50 -0800 Message-ID: <20240205180328.131019-23-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Do not use non-inclusive naming here. Signed-off-by: Stephen Hemminger --- drivers/crypto/bcmfs/hw/bcmfs4_rm.c | 6 +++--- drivers/crypto/bcmfs/hw/bcmfs5_rm.c | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/crypto/bcmfs/hw/bcmfs4_rm.c b/drivers/crypto/bcmfs/hw/bcmfs4_rm.c index 9a30c654dab8..051bbea9cc81 100644 --- a/drivers/crypto/bcmfs/hw/bcmfs4_rm.c +++ b/drivers/crypto/bcmfs/hw/bcmfs4_rm.c @@ -317,7 +317,7 @@ bcmfs4_mdst_desc(uint64_t addr, unsigned int length_div_16) } static bool -bcmfs4_sanity_check(struct bcmfs_qp_message *msg) +bcmfs4_message_check(struct bcmfs_qp_message *msg) { unsigned int i = 0; @@ -458,8 +458,8 @@ bcmfs4_enqueue_single_request_qp(struct bcmfs_qp *qp, void *op) struct bcmfs_queue *txq = &qp->tx_q; struct bcmfs_qp_message *msg = (struct bcmfs_qp_message *)op; - /* Do sanity check on message */ - if (!bcmfs4_sanity_check(msg)) { + /* Do check on message */ + if (!bcmfs4_message_check(msg)) { BCMFS_DP_LOG(ERR, "Invalid msg on queue %d", qp->qpair_id); return -EIO; } diff --git a/drivers/crypto/bcmfs/hw/bcmfs5_rm.c b/drivers/crypto/bcmfs/hw/bcmfs5_rm.c index cbfe42cb4713..2df10a302841 100644 --- a/drivers/crypto/bcmfs/hw/bcmfs5_rm.c +++ b/drivers/crypto/bcmfs/hw/bcmfs5_rm.c @@ -293,7 +293,7 @@ bcmfs5_mdst_desc(uint64_t addr, unsigned int len_div_16) } static bool -bcmfs5_sanity_check(struct bcmfs_qp_message *msg) +bcmfs5_message_check(struct bcmfs_qp_message *msg) { unsigned int i = 0; @@ -389,8 +389,8 @@ bcmfs5_enqueue_single_request_qp(struct bcmfs_qp *qp, void *op) struct bcmfs_queue *txq = &qp->tx_q; struct bcmfs_qp_message *msg = (struct bcmfs_qp_message *)op; - /* Do sanity check on message */ - if (!bcmfs5_sanity_check(msg)) { + /* Do check on message */ + if (!bcmfs5_message_check(msg)) { BCMFS_DP_LOG(ERR, "Invalid msg on queue %d", qp->qpair_id); return -EIO; } From patchwork Mon Feb 5 17:43:51 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 136410 X-Patchwork-Delegate: thomas@monjalon.net 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 E776743A3D; Mon, 5 Feb 2024 19:06:27 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id B878142DC9; Mon, 5 Feb 2024 19:04:06 +0100 (CET) Received: from mail-pl1-f177.google.com (mail-pl1-f177.google.com [209.85.214.177]) by mails.dpdk.org (Postfix) with ESMTP id A594A427E8 for ; Mon, 5 Feb 2024 19:04:00 +0100 (CET) Received: by mail-pl1-f177.google.com with SMTP id d9443c01a7336-1d93b525959so37921905ad.0 for ; Mon, 05 Feb 2024 10:04:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1707156240; x=1707761040; darn=dpdk.org; 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=BTxIIJtDDAub74TaBkECgdiDvPJkkLOw8MTt5rTvcgk=; b=q+lTtgvzlG5SBCEE+by/1uSF3Zegvs9MAB7uhK3wEDVQa8JuIgWTCfVy5eYxiSYE9N O6/Qth95Gfx0iVK5N2jEVp2vpEFgpFBA6NUNzwJpskkCfSTs0M4ihMHPjrDflbObwHqA RoypxpqpJ13LdtzSIVmtgypzWmIdxarozn1Hz1RI9lLd/Bx2ecLpypv7RCfVuK79VzAM HLhFq884vq+0DBlm4Jys1lhp8NYrxe/9LQ3Xfnm3ZNG0FairSum8IGBnoqMyM3Vj/K2S RscjenJUTInTbW74LogHKC24YiP3ISwo+WEu/2bzDa+iwL4c+bFSreL26LsO1OQFSHwy dfzg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707156240; x=1707761040; 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=BTxIIJtDDAub74TaBkECgdiDvPJkkLOw8MTt5rTvcgk=; b=QYqoGIjBGWfVegbBKIPSXZvvtqjKUFI6yAKTm/BUpv6kE1BctS+4Hot5nxgUwqaaah pjH7KZoSLGuqQytHbyDpiXV2OwMT3UMPPwJLtIGTaefNaCcRVFystol7jchDq/A4HLKS eE5vmRc/CaL34VWTlAUa51mmyEFD6quYjE89q/MjVrJpV8Q+wRy56pyieQ59w11spL5w CKqYfv9KLT9mhp/yBAFsR9HYph3SIQxWI1doMHbFTd1O9lnLQTaSEYZni4A3XDRIzxVc WWVbwVaHIMtu2021FkolhhruJskxugWtuJ3BQvRWrIKN/SQZZz8hWPgl3OWsO5u3bNvl yzlQ== X-Gm-Message-State: AOJu0YzVIYHXvcEIKnVnbQf/5GQ0Vk2k3cqNbxGOs5wjXhCWLWSCR5q/ 01eOizy5Z8xo5wmzIVycJD4k1s1iMu6kJm+7w1YvzYVr8/2RNWLv3kB2FTT3DJxgUDJQh6n7Yjr ahPQ= X-Google-Smtp-Source: AGHT+IFTK9AbTv4B64ml4SG55rBIhTS+7ZO62H7iKLqww9dEWFZ3FUJA7rITyJKz9XCrjM7GdVkZkQ== X-Received: by 2002:a17:903:646:b0:1d7:836d:7b3f with SMTP id kh6-20020a170903064600b001d7836d7b3fmr489029plb.9.1707156239649; Mon, 05 Feb 2024 10:03:59 -0800 (PST) X-Forwarded-Encrypted: i=0; AJvYcCXv4f1QS8EXimI/0QX6jfEnxf5oCAO12i6Rzuo5q9IPu5IVWc8sqC7Jg4Z9xqVcJ84GAj3YWe2CI4X3SNqpO3aWqyBJWpUdtdIxz7YssUFCS5z4xfR8jVietUib8moWZICcKeJnbkrMaBmPees0t6Y1CiMIb5/wojDueT4vX1ZNWbWMghK3b0GaGdBY33V5AxEz9t9pWcguxaWG05vs3knI4pID9SWqpHfER5/rujxV9i9S6FubqrwQNbYBolAad9AEYOZ6Tx4fFBwvzQH5BRmnlcg+m4iCDzo4ERZ1cYMVz6kS+4KSXpQjARFRexfwCuei1MGG6uz9AjCv3NdOzA1/rvUn8AuclC2EoqNDvOfpTg1Iyr4RwKUkOA06UobCKLnwgjMoV5vlh5kKdBBCbeKtEpVD/r3izJ5XB07/ULwT83gv9DMGWgta43PPw+d+34yb20/GqAinNmj70mZiTRqb9oksyWMOT/Z/mL5OqGbDQ9+UxG+9A7BhhbApLwH7F2FX5TsBZ+XIT50= Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id x8-20020a170902b40800b001d9b0a15bbfsm144867plr.262.2024.02.05.10.03.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Feb 2024 10:03:59 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Hemant Agrawal , Sachin Saxena , Nithin Dabilpuram , Kiran Kumar K , Sunil Kumar Kori , Satha Rao , Andrew Rybchenko , Bruce Richardson , Kevin Laatz , Chas Williams , "Min Hu (Connor)" , Liron Himi , Chaoyong He , Devendra Singh Rawat , Alok Prasad Subject: [PATCH v9 23/23] drivers: remove use of term sanity check Date: Mon, 5 Feb 2024 09:43:51 -0800 Message-ID: <20240205180328.131019-24-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240205180328.131019-1-stephen@networkplumber.org> References: <0230331200824.195294-1-stephen@networkplumber.org> <20240205180328.131019-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 Replace non-inclusive language with better comments. Covers multiple drivers Signed-off-by: Stephen Hemminger Acked-by: Hemant Agrawal --- drivers/bus/fslmc/qbman/qbman_sys_decl.h | 2 +- drivers/common/cnxk/roc_npa.c | 2 +- drivers/common/dpaax/caamflib/desc.h | 2 +- drivers/common/sfc_efx/base/ef10_nvram.c | 4 ++-- drivers/common/sfc_efx/base/efx_rx.c | 2 +- drivers/dma/idxd/idxd_pci.c | 2 +- drivers/net/bonding/rte_eth_bond_8023ad.c | 2 +- drivers/net/mvpp2/mrvl_qos.c | 2 +- drivers/net/nfp/nfp_flow.c | 2 +- drivers/net/qede/qede_rxtx.c | 6 +++--- 10 files changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/bus/fslmc/qbman/qbman_sys_decl.h b/drivers/bus/fslmc/qbman/qbman_sys_decl.h index caaae417773c..2b6162d0ceaf 100644 --- a/drivers/bus/fslmc/qbman/qbman_sys_decl.h +++ b/drivers/bus/fslmc/qbman/qbman_sys_decl.h @@ -9,7 +9,7 @@ #include #include -/* Sanity check */ +/* Byte order check */ #if (__BYTE_ORDER__ != __ORDER_BIG_ENDIAN__) && \ (__BYTE_ORDER__ != __ORDER_LITTLE_ENDIAN__) #error "Unknown endianness!" diff --git a/drivers/common/cnxk/roc_npa.c b/drivers/common/cnxk/roc_npa.c index 5e38ecf0414e..bd8ab1b39584 100644 --- a/drivers/common/cnxk/roc_npa.c +++ b/drivers/common/cnxk/roc_npa.c @@ -622,7 +622,7 @@ npa_aura_alloc(struct npa_lf *lf, const uint32_t block_count, int pool_id, { int rc, aura_id; - /* Sanity check */ + /* parameter check */ if (!lf || !aura || !aura_handle) return NPA_ERR_PARAM; diff --git a/drivers/common/dpaax/caamflib/desc.h b/drivers/common/dpaax/caamflib/desc.h index 4a1285c4d488..37c87eebd5b4 100644 --- a/drivers/common/dpaax/caamflib/desc.h +++ b/drivers/common/dpaax/caamflib/desc.h @@ -107,7 +107,7 @@ extern enum rta_sec_era rta_sec_era; /* * ONE - should always be set. Combination of ONE (always - * set) and ZRO (always clear) forms an endianness sanity check + * set) and ZRO (always clear) forms an endianness check */ #define HDR_ONE BIT(23) #define HDR_ZRO BIT(15) diff --git a/drivers/common/sfc_efx/base/ef10_nvram.c b/drivers/common/sfc_efx/base/ef10_nvram.c index ce8357fa94ce..fad9f0f507f3 100644 --- a/drivers/common/sfc_efx/base/ef10_nvram.c +++ b/drivers/common/sfc_efx/base/ef10_nvram.c @@ -595,13 +595,13 @@ tlv_update_partition_len_and_cks( /* * We just modified the partition, so the total length may not be - * valid. Don't use tlv_find(), which performs some sanity checks + * valid. Don't use tlv_find(), which performs some checks * that may fail here. */ partition.data = cursor->block; memcpy(&partition.tlv_cursor, cursor, sizeof (*cursor)); header = (struct tlv_partition_header *)partition.data; - /* Sanity check. */ + /* Header check */ if (__LE_TO_CPU_32(header->tag) != TLV_TAG_PARTITION_HEADER) { rc = EFAULT; goto fail1; diff --git a/drivers/common/sfc_efx/base/efx_rx.c b/drivers/common/sfc_efx/base/efx_rx.c index dce9ada55d28..28676c5a5b99 100644 --- a/drivers/common/sfc_efx/base/efx_rx.c +++ b/drivers/common/sfc_efx/base/efx_rx.c @@ -930,7 +930,7 @@ efx_rx_qcreate_internal( ndescs, id, flags, eep, erp)) != 0) goto fail4; - /* Sanity check queue creation result */ + /* Check queue creation result */ if (flags & EFX_RXQ_FLAG_RSS_HASH) { const efx_rx_prefix_layout_t *erplp = &erp->er_prefix_layout; const efx_rx_prefix_field_info_t *rss_hash_field; diff --git a/drivers/dma/idxd/idxd_pci.c b/drivers/dma/idxd/idxd_pci.c index a78889a7efa0..ba36a96f085d 100644 --- a/drivers/dma/idxd/idxd_pci.c +++ b/drivers/dma/idxd/idxd_pci.c @@ -204,7 +204,7 @@ init_pci_device(struct rte_pci_device *dev, struct idxd_dmadev *idxd, goto err; } - /* sanity check device status */ + /* check device status */ if (pci->regs->gensts & GENSTS_DEV_STATE_MASK) { /* need function-level-reset (FLR) or is enabled */ IDXD_PMD_ERR("Device status is not disabled, cannot init"); diff --git a/drivers/net/bonding/rte_eth_bond_8023ad.c b/drivers/net/bonding/rte_eth_bond_8023ad.c index 79f1b3f1a0f2..57a67bd029ed 100644 --- a/drivers/net/bonding/rte_eth_bond_8023ad.c +++ b/drivers/net/bonding/rte_eth_bond_8023ad.c @@ -1476,7 +1476,7 @@ bond_8023ad_setup_validate(uint16_t port_id, return -EINVAL; if (conf != NULL) { - /* Basic sanity check */ + /* Basic input check */ if (conf->slow_periodic_ms == 0 || conf->fast_periodic_ms >= conf->slow_periodic_ms || conf->long_timeout_ms == 0 || diff --git a/drivers/net/mvpp2/mrvl_qos.c b/drivers/net/mvpp2/mrvl_qos.c index f43217bc589c..db5052a9ab6f 100644 --- a/drivers/net/mvpp2/mrvl_qos.c +++ b/drivers/net/mvpp2/mrvl_qos.c @@ -260,7 +260,7 @@ get_entry_values(const char *entry, uint8_t *tab, if (nb_rng_tokens != 2) return -3; - /* Range and sanity checks. */ + /* Range and input checks. */ if (get_val_securely(rng_tokens[0], &token_val) < 0) return -4; beg = (char)token_val; diff --git a/drivers/net/nfp/nfp_flow.c b/drivers/net/nfp/nfp_flow.c index f832b52d8996..c867697e05c9 100644 --- a/drivers/net/nfp/nfp_flow.c +++ b/drivers/net/nfp/nfp_flow.c @@ -2071,7 +2071,7 @@ nfp_flow_compile_item_proc(struct nfp_flower_representor *repr, break; } - /* Perform basic sanity checks */ + /* Perform basic checks */ ret = nfp_flow_item_check(item, proc); if (ret != 0) { PMD_DRV_LOG(ERR, "nfp flow item %d check failed", item->type); diff --git a/drivers/net/qede/qede_rxtx.c b/drivers/net/qede/qede_rxtx.c index c35585f5fdd6..8b89841e03b1 100644 --- a/drivers/net/qede/qede_rxtx.c +++ b/drivers/net/qede/qede_rxtx.c @@ -1383,7 +1383,7 @@ qede_rx_process_tpa_end_cqe(struct qede_dev *qdev, cqe->len_list[0]); /* Update total length and frags based on end TPA */ rx_mb = rxq->tpa_info[cqe->tpa_agg_index].tpa_head; - /* TODO: Add Sanity Checks */ + /* TODO: Add more checks */ rx_mb->nb_segs = cqe->num_of_bds; rx_mb->pkt_len = cqe->total_packet_len; @@ -2211,7 +2211,7 @@ qede_xmit_prep_pkts(__rte_unused void *p_txq, struct rte_mbuf **tx_pkts, #ifdef RTE_LIBRTE_QEDE_DEBUG_TX static inline void -qede_mpls_tunn_tx_sanity_check(struct rte_mbuf *mbuf, +qede_mpls_tunn_tx_basic_check(struct rte_mbuf *mbuf, struct qede_tx_queue *txq) { if (((mbuf->outer_l2_len + mbuf->outer_l3_len) / 2) > 0xff) @@ -2473,7 +2473,7 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts) RTE_MBUF_F_TX_TUNNEL_MPLSINUDP) { mplsoudp_flg = true; #ifdef RTE_LIBRTE_QEDE_DEBUG_TX - qede_mpls_tunn_tx_sanity_check(mbuf, txq); + qede_mpls_tunn_tx_basic_check(mbuf, txq); #endif /* Outer L4 offset in two byte words */ tunn_l4_hdr_start_offset =