From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 1891245AA8; Fri, 4 Oct 2024 06:14:23 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D9174402DD; Fri, 4 Oct 2024 06:14:22 +0200 (CEST) Received: from NAM10-MW2-obe.outbound.protection.outlook.com (mail-mw2nam10on2085.outbound.protection.outlook.com [40.107.94.85]) by mails.dpdk.org (Postfix) with ESMTP id D595B402C9 for ; Fri, 4 Oct 2024 06:14:20 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=xme6re59W4Y7BQNwBxBNd5H/AtV5Jq0d9myNdB1vVNUNf6hXDXyl2KpbihbRgETLUEhXT8b8B7HTeadZCKI32lVB78cB7GT2nQFAJtJQSio7/xZxigfNaabxbO41quV7dxSCdpBED3ET1ePzzuSXDo5zFrABtOulYztwJOlTlcsSoSBewBzhieWvhTXuKyMXerHVLQPABdhO/a3mBxm4HZyYuD+qbY87hDMqSbZH6lIE06Dh/0+uv1pzw1kzuWTacbIOo08AFTg6K4P7fEpVhaubSVEtajhqBuqr5jR2JPZHeOUVudY3LmIXQDKUqbSn+jvSB7pqsQoXob5TtFh2RQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=FeMs4MtUJkWGAhlgoN0lAP3cuIuvDMlmZcWcGnxpQAc=; b=y2+FrnIYCZlakStracI/7+jbwpoxFUtqXJ5/MZfRqHWCL5pVdomxcCRdXYIKEwvDhQhLUiDK5/76ok5/0sZz1xIiWy++08pK3OEfoiMo9Usj/pPQXS0v3HCi9gOBonCf22GNJ4bf1GUHlsUmWlf8OGqa9RiWGpo0A0vjHxPt4jrqJ8ol+muJD60A+AOA/nAXtKyY5NRWqAu7CjdP6tFj6kC6fDqFj2W4Th/Uj6VQeuLeUOHlYNO2UY2lIyjloZl7gH1w9xe/bHIL7BVoiZ93vaCNpxwzRVFyxjZ+JYDuSaTHpByL88lJqB3EN1smOkCY6gfD1BB5xwJXB80pqYvyBg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=monjalon.net smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=FeMs4MtUJkWGAhlgoN0lAP3cuIuvDMlmZcWcGnxpQAc=; b=yq6BKBZPkQfVec7BQtWqwi0soXz+ilR7myjiq35wsqPK+wPRWc12iEc9w8e901lLP7HZq1VH0TGKc1WCAT+q5PvCreN5PzbKtFhid8ydjPZ2sxGIYGdtk6+yeMn5eRDZCvl2T23+QJE8TzuxhI8zWRs9+bZPmjVfP4rstt8chPc= Received: from CH2PR10CA0006.namprd10.prod.outlook.com (2603:10b6:610:4c::16) by MW4PR12MB7264.namprd12.prod.outlook.com (2603:10b6:303:22e::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8026.18; Fri, 4 Oct 2024 04:14:14 +0000 Received: from CH2PEPF00000140.namprd02.prod.outlook.com (2603:10b6:610:4c:cafe::7d) by CH2PR10CA0006.outlook.office365.com (2603:10b6:610:4c::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8026.18 via Frontend Transport; Fri, 4 Oct 2024 04:14:14 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by CH2PEPF00000140.mail.protection.outlook.com (10.167.244.72) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.8026.11 via Frontend Transport; Fri, 4 Oct 2024 04:14:14 +0000 Received: from dpdk-rc.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Thu, 3 Oct 2024 23:14:12 -0500 From: Ferruh Yigit To: Thomas Monjalon , Ori Kam , Aman Singh , Konstantin Ananyev , Reshma Pattan , Stephen Hemminger , Akhil Goyal , Anoob Joseph , Potnuri Bharat Teja , Hemant Agrawal , Sachin Saxena , Matan Azrad , Viacheslav Ovsiienko , Dariusz Sosnowski , Bing Zhao , Suanming Mou , Chaoyong He , Andrew Rybchenko CC: , =?UTF-8?q?Morten=20Br=C3=B8rup?= , Bruce Richardson Subject: [PATCH] ethdev: convert string initialization Date: Thu, 3 Oct 2024 21:13:34 -0700 Message-ID: <20241004041335.2916435-1-ferruh.yigit@amd.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240801092722.3732917-1-ferruh.yigit@amd.com> References: <20240801092722.3732917-1-ferruh.yigit@amd.com> MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8bit X-Originating-IP: [10.180.168.240] X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PEPF00000140:EE_|MW4PR12MB7264:EE_ X-MS-Office365-Filtering-Correlation-Id: 08516270-4625-4b67-5128-08dce42b01d7 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|82310400026|1800799024|36860700013|7416014|376014|921020; X-Microsoft-Antispam-Message-Info: =?utf-8?B?bFFyRUpHZXNlL3dNTDdXQ05JenBXWlVTVUQzK2F5alFQbjRLdzkwMEIwWHVI?= =?utf-8?B?ZnJYZ212cjVvZGhRUWVpb3dkZFd0Z3lsVnllYktmKzFwVzlwZStVSUVnNmd0?= =?utf-8?B?YytTMXhDWitEdHRaVUNzUU44QTFHMFJLYkIyVC9TWXNNcENNN1duQWFIR3RQ?= =?utf-8?B?UitHdVlZdC9ybzIzb1l1a3VGVnhVV0lUM3hhR0hoMHhLdjg4ZmkyZVJNYUpR?= =?utf-8?B?R1gwbEluWmlMQ0haTk1jUHhLMmZUUFAzYXFXTjRmbjYyZzhoZWxYTnI4T09x?= =?utf-8?B?ZHI5MnlLRDc5THRZT25zaU1HcFR0d0dEOVZDdW1RdHNtZXoxdEwyUFdqOVFB?= =?utf-8?B?Nlc2ZjB1TzVkWWZGeU5CRHFYV0o2Y3dqSXJzOHRXKzRiTzBWT2I1L2p2d1Z6?= =?utf-8?B?MzVyZEo0V1cwejl6c0pGRnZ3N3lodFgvRVdpWmR1K0VwdHltblF4SEh4SGZG?= =?utf-8?B?MkVPSVlXUTV1SEVId0M1YVlLYnY1YkxmaWQvaUYxZzUyQVlaekhPbjJVbXZD?= =?utf-8?B?RUtPWVFKbjFwaXNzRnFXQ0xNaW1PRzBoRUROY0dXV3g2WXMzY1pKNXdDVURm?= =?utf-8?B?SFlkMWJFTHZ1SkFteWRoKzVzbzJlMUVxOUJoODBIbFBhWHMydjRvdGFNWS9q?= =?utf-8?B?WTFpOGQveTJCWXQxZEIwV1FHNXBRSUVFNGYrRlpDM3NyUUxXdVFVQWNXS3A0?= =?utf-8?B?QVpjM0VXZE4rSS9YbHBhRlViNmdmUFowL282akFzck0xMmZHcGl5UmZxdk5w?= =?utf-8?B?R2JCU2RxRGRMZDRDajlaRlBmS2VVeFErZjBvOXgrcUlkRVRTMWRrNXpieTdz?= =?utf-8?B?UGV3a2xTenBZcWs0NHpKWElxeC82M29hN0hjSlEwTldjZlhEVHZVNHRNZEls?= =?utf-8?B?M3M2T2lydUVQQnlOclhWTXFlYThUa28va04rMHMwaktnRWZKZGh5Yk5EbU1E?= =?utf-8?B?YjN4TWY5UGNJYStEVkMyK0hrendjc2JwUGUyTi90YURuTzQvVmZ3Z2d0eUV1?= =?utf-8?B?MmRHc3BpbUw4ZU56K3JNU2JwN2JBeWJMNWpFeUtLeHNCcG1IMVhxTDM2aDhZ?= =?utf-8?B?NllnczVpYW9sOHZudVlZZmcyVFN5YTJoVjJ0SmFTKzJjcmM2ZEZRcU5OUHFq?= =?utf-8?B?QzdWV1AxbjJQenBpaWx6YXhHWDBQY1l6d1dxZGRRTlMxMHQ0TEMzejhXczYv?= =?utf-8?B?ZXBhelZsZFZ2RThyMnlzTlk5bng3aVpjZjI0NWJyb1piVnRRTk5ncmM1Um1r?= =?utf-8?B?U2I2ZmJmMzFCREtjdjFycTVWa0RndXpXSWloeEN0QXh3ZDhqd3ZTSzlUTkxy?= =?utf-8?B?SXIyY1NKb0M0Tk4xZGdjYUI4YUtPZFZrU2NTRy9kY3VWYVJmamEyNjFVQVpB?= =?utf-8?B?NFNRVFg0Zll2VThYbndtRzZqa3FBQzdQUGJxK25TWUFHbm5lM0NwU2NGOW8z?= =?utf-8?B?YUhTZndPS244cERXNlJ5Wk5IOUFYa0NkS2U0MUxNY1ZOOVJzNVJibW9VMUNq?= =?utf-8?B?SUNnTGRaYm5vUXBmU1dVZjN5TXdtc1g3TFRyRlBhSG4vYWF5ZkNremwxb0Zq?= =?utf-8?B?Y29yQ0pHMnBNNExIdFVWb0hzSGtrczMzREpSZEtGVFdBTUp1bVcrRkNrZ1o5?= =?utf-8?B?OHhSbXhiMzZ6VXIzb295ejV5TkJTakJPbS9Vb1JvYnlxakhpbW1UcXE4Nisr?= =?utf-8?B?a1k3MGR6Tjd0VkQ1NTh0eFN5bnJQOGdKOWtDSnJsUU5RSHhQMTFoYm1VRUU2?= =?utf-8?B?bXFLZHU5MFNkUTJ4R0NvT2oza2d0cDh0VllWRWZuTGEwaE1Kdk41VGRlL3dm?= =?utf-8?B?Z0ZFenZCcFF6bmI2QS9TSDNBOWRIeWR5Zmx2c0hBcmdaekxqbUNzZ0hXVjR2?= =?utf-8?Q?d3Kv8BttN5DML?= X-Forefront-Antispam-Report: CIP:165.204.84.17; CTRY:US; LANG:en; SCL:1; SRV:; IPV:CAL; SFV:NSPM; H:SATLEXMB04.amd.com; PTR:InfoDomainNonexistent; CAT:NONE; SFS:(13230040)(82310400026)(1800799024)(36860700013)(7416014)(376014)(921020); DIR:OUT; SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Oct 2024 04:14:14.4601 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 08516270-4625-4b67-5128-08dce42b01d7 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17]; Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: CH2PEPF00000140.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR12MB7264 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 gcc 15 experimental [1], with -Wextra flag, gives warning in variable initialization as string [2]. The warning has a point when initialized variable is intended to use as string, since assignment is missing the required null terminator for this case. But warning is useless for our usecase. In this patch only updated a few instance to show the issue, there are many instances to fix, if we prefer to go this way. Other option is to disable warning but it can be useful for actual string usecases, so I prefer to keep it. Converted string initialization to array initialization. [1] gcc (GCC) 15.0.0 20241003 (experimental) [2] ../lib/ethdev/rte_flow.h:906:36: error: initializer-string for array of ‘unsigned char’ is too long [-Werror=unterminated-string-initialization] 906 | .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", | ^~~~~~~~~~~~~~~~~~~~~~~~~~ ../lib/ethdev/rte_flow.h:907:36: error: initializer-string for array of ‘unsigned char’ is too long [-Werror=unterminated-string-initialization] 907 | .hdr.src_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", | ^~~~~~~~~~~~~~~~~~~~~~~~~~ ../lib/ethdev/rte_flow.h:1009:25: error: initializer-string for array of ‘unsigned char’ is too long [-Werror=unterminated-string-initialization] 1009 | "\xff\xff\xff\xff\xff\xff\xff\xff" | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../lib/ethdev/rte_flow.h:1012:25: error: initializer-string for array of ‘unsigned char’ is too long [-Werror=unterminated-string-initialization] 1012 | "\xff\xff\xff\xff\xff\xff\xff\xff" | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../lib/ethdev/rte_flow.h:1135:20: error: initializer-string for array of ‘unsigned char’ is too long [-Werror=unterminated-string-initialization] 1135 | .hdr.vni = "\xff\xff\xff", | ^~~~~~~~~~~~~~ Signed-off-by: Ferruh Yigit Acked-by: Morten Brørup Acked-by: Bruce Richardson --- app/test-pmd/cmdline_flow.c | 31 ++++----- app/test/test_bpf.c | 2 +- app/test/test_pcapng.c | 2 +- app/test/test_security_inline_macsec.c | 4 +- doc/guides/prog_guide/ethdev/flow_offload.rst | 2 +- drivers/net/cxgbe/cxgbe_flow.c | 14 ++-- drivers/net/dpaa2/dpaa2_flow.c | 14 ++-- drivers/net/mlx4/mlx4_flow.c | 6 +- drivers/net/mlx5/mlx5_flow.c | 20 +++--- drivers/net/mlx5/mlx5_flow_dv.c | 26 ++++---- drivers/net/mlx5/mlx5_flow_hw.c | 66 +++++++++---------- drivers/net/mlx5/mlx5_trigger.c | 20 +++--- drivers/net/nfp/flower/nfp_flower_flow.c | 14 ++-- drivers/net/nfp/nfp_net_flow.c | 8 +-- drivers/net/tap/tap_flow.c | 30 ++++----- examples/l2fwd-macsec/main.c | 4 +- lib/ethdev/rte_flow.h | 44 ++++++------- 17 files changed, 144 insertions(+), 163 deletions(-) diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c index b7bcf18311a2..5451b3a45338 100644 --- a/app/test-pmd/cmdline_flow.c +++ b/app/test-pmd/cmdline_flow.c @@ -898,20 +898,20 @@ struct vxlan_encap_conf vxlan_encap_conf = { .select_ipv4 = 1, .select_vlan = 0, .select_tos_ttl = 0, - .vni = "\x00\x00\x00", + .vni = { 0x00, 0x00, 0x00 }, .udp_src = 0, .udp_dst = RTE_BE16(RTE_VXLAN_DEFAULT_PORT), .ipv4_src = RTE_IPV4(127, 0, 0, 1), .ipv4_dst = RTE_IPV4(255, 255, 255, 255), - .ipv6_src = "\x00\x00\x00\x00\x00\x00\x00\x00" - "\x00\x00\x00\x00\x00\x00\x00\x01", - .ipv6_dst = "\x00\x00\x00\x00\x00\x00\x00\x00" - "\x00\x00\x00\x00\x00\x00\x11\x11", + .ipv6_src = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01 }, + .ipv6_dst = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x11, 0x11 }, .vlan_tci = 0, .ip_tos = 0, .ip_ttl = 255, - .eth_src = "\x00\x00\x00\x00\x00\x00", - .eth_dst = "\xff\xff\xff\xff\xff\xff", + .eth_src = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .eth_dst = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }; /** Maximum number of items in struct rte_flow_action_vxlan_encap. */ @@ -934,16 +934,16 @@ struct action_vxlan_encap_data { struct nvgre_encap_conf nvgre_encap_conf = { .select_ipv4 = 1, .select_vlan = 0, - .tni = "\x00\x00\x00", + .tni = { 0x00, 0x00, 0x00 }, .ipv4_src = RTE_IPV4(127, 0, 0, 1), .ipv4_dst = RTE_IPV4(255, 255, 255, 255), - .ipv6_src = "\x00\x00\x00\x00\x00\x00\x00\x00" - "\x00\x00\x00\x00\x00\x00\x00\x01", - .ipv6_dst = "\x00\x00\x00\x00\x00\x00\x00\x00" - "\x00\x00\x00\x00\x00\x00\x11\x11", + .ipv6_src = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01 }, + .ipv6_dst = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x11, 0x11 }, .vlan_tci = 0, - .eth_src = "\x00\x00\x00\x00\x00\x00", - .eth_dst = "\xff\xff\xff\xff\xff\xff", + .eth_src = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .eth_dst = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }; /** Maximum number of items in struct rte_flow_action_nvgre_encap. */ @@ -8304,7 +8304,8 @@ parse_prefix(struct context *ctx, const struct token *token, void *buf, unsigned int size) { const struct arg *arg = pop_args(ctx); - static const uint8_t conv[] = "\x00\x80\xc0\xe0\xf0\xf8\xfc\xfe\xff"; + static const uint8_t conv[] = { 0x00, 0x80, 0xc0, 0xe0, 0xf0, + 0xf8, 0xfc, 0xfe, 0xff }; char *end; uintmax_t u; unsigned int bytes; diff --git a/app/test/test_bpf.c b/app/test/test_bpf.c index 7819d6aba951..90e10d7d2ccf 100644 --- a/app/test/test_bpf.c +++ b/app/test/test_bpf.c @@ -3361,7 +3361,7 @@ test_bpf_filter_sanity(pcap_t *pcap) hdr = rte_pktmbuf_mtod(m, typeof(hdr)); hdr->eth_hdr = (struct rte_ether_hdr) { - .dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .ether_type = rte_cpu_to_be_16(RTE_ETHER_TYPE_IPV4), }; hdr->ip_hdr = (struct rte_ipv4_hdr) { diff --git a/app/test/test_pcapng.c b/app/test/test_pcapng.c index 2665b08c7647..01f52cf03541 100644 --- a/app/test/test_pcapng.c +++ b/app/test/test_pcapng.c @@ -73,7 +73,7 @@ mbuf1_prepare(struct dummy_mbuf *dm, uint32_t plen) struct rte_udp_hdr udp; } pkt = { .eth = { - .dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .ether_type = rte_cpu_to_be_16(RTE_ETHER_TYPE_IPV4), }, .ip = { diff --git a/app/test/test_security_inline_macsec.c b/app/test/test_security_inline_macsec.c index f11e9da8c375..c921bf8ebbd1 100644 --- a/app/test/test_security_inline_macsec.c +++ b/app/test/test_security_inline_macsec.c @@ -318,8 +318,8 @@ create_default_flow(const struct mcs_test_vector *td, uint16_t portid, struct rte_flow *flow; struct rte_flow_item_eth eth = { .hdr.ether_type = 0, }; static const struct rte_flow_item_eth eth_mask = { - .hdr.dst_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = RTE_BE16(0x0000), }; diff --git a/doc/guides/prog_guide/ethdev/flow_offload.rst b/doc/guides/prog_guide/ethdev/flow_offload.rst index 906d6014f512..2d6187ed11b2 100644 --- a/doc/guides/prog_guide/ethdev/flow_offload.rst +++ b/doc/guides/prog_guide/ethdev/flow_offload.rst @@ -3865,7 +3865,7 @@ For example, to create a pattern template to match on the destination MAC: const struct rte_flow_pattern_template_attr attr = {.ingress = 1}; struct rte_flow_item_eth eth_m = { - .dst.addr_bytes = "\xff\xff\xff\xff\xff\xff"; + .dst.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }; }; struct rte_flow_item pattern[] = { [0] = {.type = RTE_FLOW_ITEM_TYPE_ETH, diff --git a/drivers/net/cxgbe/cxgbe_flow.c b/drivers/net/cxgbe/cxgbe_flow.c index f5787c247f1f..40d21e694409 100644 --- a/drivers/net/cxgbe/cxgbe_flow.c +++ b/drivers/net/cxgbe/cxgbe_flow.c @@ -889,8 +889,8 @@ static struct chrte_fparse parseitem[] = { [RTE_FLOW_ITEM_TYPE_ETH] = { .fptr = ch_rte_parsetype_eth, .dmask = &(const struct rte_flow_item_eth){ - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0xffff, } }, @@ -918,12 +918,10 @@ static struct chrte_fparse parseitem[] = { .fptr = ch_rte_parsetype_ipv6, .dmask = &(const struct rte_flow_item_ipv6) { .hdr = { - .src_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - .dst_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .src_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .dst_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .vtc_flow = RTE_BE32(0xff000000), }, }, diff --git a/drivers/net/dpaa2/dpaa2_flow.c b/drivers/net/dpaa2/dpaa2_flow.c index 6c7bac4d483a..d7e3f3b30206 100644 --- a/drivers/net/dpaa2/dpaa2_flow.c +++ b/drivers/net/dpaa2/dpaa2_flow.c @@ -100,8 +100,8 @@ enum rte_flow_action_type dpaa2_supported_fs_action_type[] = { #ifndef __cplusplus static const struct rte_flow_item_eth dpaa2_flow_item_eth_mask = { - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", - .hdr.src_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .hdr.src_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .hdr.ether_type = RTE_BE16(0xffff), }; @@ -117,12 +117,10 @@ static const struct rte_flow_item_ipv4 dpaa2_flow_item_ipv4_mask = { static const struct rte_flow_item_ipv6 dpaa2_flow_item_ipv6_mask = { .hdr = { - .src_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - .dst_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .src_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .dst_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .proto = 0xff }, }; diff --git a/drivers/net/mlx4/mlx4_flow.c b/drivers/net/mlx4/mlx4_flow.c index 8ef9fd2db44e..b520664d95d1 100644 --- a/drivers/net/mlx4/mlx4_flow.c +++ b/drivers/net/mlx4/mlx4_flow.c @@ -582,7 +582,7 @@ static const struct mlx4_flow_proc_item mlx4_flow_proc_item_list[] = { RTE_FLOW_ITEM_TYPE_IPV4), .mask_support = &(const struct rte_flow_item_eth){ /* Only destination MAC can be matched. */ - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }, .mask_default = &rte_flow_item_eth_mask, .mask_sz = sizeof(struct rte_flow_item_eth), @@ -1304,10 +1304,10 @@ mlx4_flow_internal(struct mlx4_priv *priv, struct rte_flow_error *error) }; struct rte_flow_item_eth eth_spec; const struct rte_flow_item_eth eth_mask = { - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }; const struct rte_flow_item_eth eth_allmulti = { - .hdr.dst_addr.addr_bytes = "\x01\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x01, 0x00, 0x00, 0x00, 0x00, 0x00 }, }; struct rte_flow_item_vlan vlan_spec; const struct rte_flow_item_vlan vlan_mask = { diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c index 72fb3a55baf4..031db8176b4d 100644 --- a/drivers/net/mlx5/mlx5_flow.c +++ b/drivers/net/mlx5/mlx5_flow.c @@ -2668,8 +2668,8 @@ mlx5_flow_validate_item_eth(const struct rte_eth_dev *dev, { const struct rte_flow_item_eth *mask = item->mask; const struct rte_flow_item_eth nic_mask = { - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", - .hdr.src_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .hdr.src_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .hdr.ether_type = RTE_BE16(0xffff), .has_vlan = ext_vlan_sup ? 1 : 0, }; @@ -2933,12 +2933,10 @@ mlx5_flow_validate_item_ipv6(const struct rte_eth_dev *dev, const struct rte_flow_item_ipv6 *spec = item->spec; const struct rte_flow_item_ipv6 nic_mask = { .hdr = { - .src_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - .dst_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .src_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .dst_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .vtc_flow = RTE_BE32(0xffffffff), .proto = 0xff, }, @@ -3163,7 +3161,7 @@ mlx5_flow_validate_item_vxlan(struct rte_eth_dev *dev, uint8_t vni[4]; } id = { .vlan_id = 0, }; const struct rte_flow_item_vxlan nic_mask = { - .hdr.vni = "\xff\xff\xff", + .hdr.vni = { 0xff, 0xff, 0xff }, .hdr.rsvd1 = 0xff, }; const struct rte_flow_item_vxlan *valid_mask; @@ -3249,7 +3247,7 @@ mlx5_flow_validate_item_vxlan_gpe(const struct rte_flow_item *item, } id = { .vlan_id = 0, }; struct rte_flow_item_vxlan_gpe nic_mask = { - .vni = "\xff\xff\xff", + .vni = { 0xff, 0xff, 0xff }, .protocol = 0xff, .flags = 0xff, }; @@ -3563,7 +3561,7 @@ mlx5_flow_validate_item_geneve(const struct rte_flow_item *item, MLX5_GENEVE_OPT_LEN_1 : MLX5_GENEVE_OPT_LEN_0; const struct rte_flow_item_geneve nic_mask = { .ver_opt_len_o_c_rsvd0 = RTE_BE16(0x3f80), - .vni = "\xff\xff\xff", + .vni = { 0xff, 0xff, 0xff }, .protocol = RTE_BE16(UINT16_MAX), }; diff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_dv.c index 89057edbcfc2..8bcdb3f99e35 100644 --- a/drivers/net/mlx5/mlx5_flow_dv.c +++ b/drivers/net/mlx5/mlx5_flow_dv.c @@ -7697,12 +7697,10 @@ const struct rte_flow_item_ipv4 nic_ipv4_mask = { const struct rte_flow_item_ipv6 nic_ipv6_mask = { .hdr = { - .src_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - .dst_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .src_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .dst_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .vtc_flow = RTE_BE32(0xffffffff), .proto = 0xff, .hop_limits = 0xff, @@ -9291,8 +9289,8 @@ flow_dv_translate_item_eth(void *key, const struct rte_flow_item *item, const struct rte_flow_item_eth *eth_m; const struct rte_flow_item_eth *eth_v; const struct rte_flow_item_eth nic_mask = { - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", - .hdr.src_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .hdr.src_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .hdr.ether_type = RTE_BE16(0xffff), .has_vlan = 0, }; @@ -9549,12 +9547,10 @@ flow_dv_translate_item_ipv6(void *key, const struct rte_flow_item *item, const struct rte_flow_item_ipv6 *ipv6_v; const struct rte_flow_item_ipv6 nic_mask = { .hdr = { - .src_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - .dst_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .src_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .dst_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .vtc_flow = RTE_BE32(0xffffffff), .proto = 0xff, .hop_limits = 0xff, @@ -10064,7 +10060,7 @@ flow_dv_translate_item_vxlan(struct rte_eth_dev *dev, int i; struct mlx5_priv *priv = dev->data->dev_private; const struct rte_flow_item_vxlan nic_mask = { - .hdr.vni = "\xff\xff\xff", + .hdr.vni = { 0xff, 0xff, 0xff }, .hdr.rsvd1 = 0xff, }; diff --git a/drivers/net/mlx5/mlx5_flow_hw.c b/drivers/net/mlx5/mlx5_flow_hw.c index d243b59b71cd..96be742b9493 100644 --- a/drivers/net/mlx5/mlx5_flow_hw.c +++ b/drivers/net/mlx5/mlx5_flow_hw.c @@ -377,67 +377,67 @@ static uint32_t mlx5_hw_act_flag[MLX5_HW_ACTION_FLAG_MAX] /* Ethernet item spec for promiscuous mode. */ static const struct rte_flow_item_eth ctrl_rx_eth_promisc_spec = { - .hdr.dst_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; /* Ethernet item mask for promiscuous mode. */ static const struct rte_flow_item_eth ctrl_rx_eth_promisc_mask = { - .hdr.dst_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; /* Ethernet item spec for all multicast mode. */ static const struct rte_flow_item_eth ctrl_rx_eth_mcast_spec = { - .hdr.dst_addr.addr_bytes = "\x01\x00\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x01, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; /* Ethernet item mask for all multicast mode. */ static const struct rte_flow_item_eth ctrl_rx_eth_mcast_mask = { - .hdr.dst_addr.addr_bytes = "\x01\x00\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x01, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; /* Ethernet item spec for IPv4 multicast traffic. */ static const struct rte_flow_item_eth ctrl_rx_eth_ipv4_mcast_spec = { - .hdr.dst_addr.addr_bytes = "\x01\x00\x5e\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x01, 0x00, 0x5e, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; /* Ethernet item mask for IPv4 multicast traffic. */ static const struct rte_flow_item_eth ctrl_rx_eth_ipv4_mcast_mask = { - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; /* Ethernet item spec for IPv6 multicast traffic. */ static const struct rte_flow_item_eth ctrl_rx_eth_ipv6_mcast_spec = { - .hdr.dst_addr.addr_bytes = "\x33\x33\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x33, 0x33, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; /* Ethernet item mask for IPv6 multicast traffic. */ static const struct rte_flow_item_eth ctrl_rx_eth_ipv6_mcast_mask = { - .hdr.dst_addr.addr_bytes = "\xff\xff\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; /* Ethernet item mask for unicast traffic. */ static const struct rte_flow_item_eth ctrl_rx_eth_dmac_mask = { - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; /* Ethernet item spec for broadcast. */ static const struct rte_flow_item_eth ctrl_rx_eth_bcast_spec = { - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; @@ -8250,12 +8250,10 @@ const struct rte_flow_item_ipv6 hws_nic_ipv6_mask = { .payload_len = RTE_BE16(0xffff), .proto = 0xff, .hop_limits = 0xff, - .src_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - .dst_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .src_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .dst_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }, .has_frag_ext = 1, }; @@ -9936,8 +9934,8 @@ flow_hw_create_tx_default_mreg_copy_pattern_template(struct rte_eth_dev *dev, .egress = 1, }; struct rte_flow_item_eth promisc = { - .hdr.dst_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; struct rte_flow_item eth_all[] = { @@ -9974,8 +9972,8 @@ flow_hw_create_lacp_rx_pattern_template(struct rte_eth_dev *dev, struct rte_flow .ingress = 1, }; struct rte_flow_item_eth lacp_mask = { - .dst.addr_bytes = "\x00\x00\x00\x00\x00\x00", - .src.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .dst.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .src.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .type = 0xFFFF, }; struct rte_flow_item eth_all[] = { @@ -15476,8 +15474,8 @@ mlx5_flow_hw_create_tx_default_mreg_copy_flow(struct rte_eth_dev *dev) { struct mlx5_priv *priv = dev->data->dev_private; struct rte_flow_item_eth promisc = { - .hdr.dst_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; struct rte_flow_item eth_all[] = { @@ -15815,7 +15813,7 @@ __flow_hw_ctrl_flows_unicast(struct rte_eth_dev *dev, .type = MLX5_HW_CTRL_FLOW_TYPE_DEFAULT_RX_RSS, }; const struct rte_ether_addr cmp = { - .addr_bytes = "\x00\x00\x00\x00\x00\x00", + .addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, }; unsigned int i; @@ -15861,7 +15859,7 @@ __flow_hw_ctrl_flows_unicast_vlan(struct rte_eth_dev *dev, .type = MLX5_HW_CTRL_FLOW_TYPE_DEFAULT_RX_RSS, }; const struct rte_ether_addr cmp = { - .addr_bytes = "\x00\x00\x00\x00\x00\x00", + .addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, }; unsigned int i; unsigned int j; diff --git a/drivers/net/mlx5/mlx5_trigger.c b/drivers/net/mlx5/mlx5_trigger.c index a65a4607318c..7654f4d2ed24 100644 --- a/drivers/net/mlx5/mlx5_trigger.c +++ b/drivers/net/mlx5/mlx5_trigger.c @@ -1568,23 +1568,23 @@ mlx5_traffic_enable(struct rte_eth_dev *dev) { struct mlx5_priv *priv = dev->data->dev_private; struct rte_flow_item_eth bcast = { - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }; struct rte_flow_item_eth ipv6_multi_spec = { - .hdr.dst_addr.addr_bytes = "\x33\x33\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x33, 0x33, 0x00, 0x00, 0x00, 0x00 }, }; struct rte_flow_item_eth ipv6_multi_mask = { - .hdr.dst_addr.addr_bytes = "\xff\xff\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0x00, 0x00, 0x00, 0x00 }, }; struct rte_flow_item_eth unicast = { - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, }; struct rte_flow_item_eth unicast_mask = { - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }; const unsigned int vlan_filter_n = priv->vlan_filter_n; const struct rte_ether_addr cmp = { - .addr_bytes = "\x00\x00\x00\x00\x00\x00", + .addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, }; unsigned int i; unsigned int j; @@ -1653,8 +1653,8 @@ mlx5_traffic_enable(struct rte_eth_dev *dev) return 0; if (dev->data->promiscuous) { struct rte_flow_item_eth promisc = { - .hdr.dst_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; @@ -1664,8 +1664,8 @@ mlx5_traffic_enable(struct rte_eth_dev *dev) } if (dev->data->all_multicast) { struct rte_flow_item_eth multicast = { - .hdr.dst_addr.addr_bytes = "\x01\x00\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x01, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = 0, }; diff --git a/drivers/net/nfp/flower/nfp_flower_flow.c b/drivers/net/nfp/flower/nfp_flower_flow.c index 20c8b4a1b189..9e16f0267b38 100644 --- a/drivers/net/nfp/flower/nfp_flower_flow.c +++ b/drivers/net/nfp/flower/nfp_flower_flow.c @@ -2504,8 +2504,8 @@ static const struct nfp_flow_item_proc nfp_flow_item_proc_list[] = { RTE_FLOW_ITEM_TYPE_IPV6), .mask_support = &(const struct rte_flow_item_eth) { .hdr = { - .dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", - .src_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .src_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .ether_type = RTE_BE16(0xffff), }, .has_vlan = 1, @@ -2557,10 +2557,10 @@ static const struct nfp_flow_item_proc nfp_flow_item_proc_list[] = { .vtc_flow = RTE_BE32(0x0ff00000), .proto = 0xff, .hop_limits = 0xff, - .src_addr = "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - .dst_addr = "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .src_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .dst_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }, .has_frag_ext = 1, }, @@ -2618,7 +2618,7 @@ static const struct nfp_flow_item_proc nfp_flow_item_proc_list[] = { [RTE_FLOW_ITEM_TYPE_GENEVE] = { .next_item = NEXT_ITEM(RTE_FLOW_ITEM_TYPE_ETH), .mask_support = &(const struct rte_flow_item_geneve) { - .vni = "\xff\xff\xff", + .vni = { 0xff, 0xff, 0xff }, }, .mask_default = &rte_flow_item_geneve_mask, .mask_sz = sizeof(struct rte_flow_item_geneve), diff --git a/drivers/net/nfp/nfp_net_flow.c b/drivers/net/nfp/nfp_net_flow.c index 2c7801ed41f4..c3b333425b4c 100644 --- a/drivers/net/nfp/nfp_net_flow.c +++ b/drivers/net/nfp/nfp_net_flow.c @@ -403,10 +403,10 @@ static const struct nfp_net_flow_item_proc nfp_net_flow_item_proc_list[] = { .mask_support = &(const struct rte_flow_item_ipv6){ .hdr = { .proto = 0xff, - .src_addr = "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - .dst_addr = "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .src_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .dst_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }, }, .mask_default = &rte_flow_item_ipv6_mask, diff --git a/drivers/net/tap/tap_flow.c b/drivers/net/tap/tap_flow.c index 0a90c0487b46..5ae1faf9165d 100644 --- a/drivers/net/tap/tap_flow.c +++ b/drivers/net/tap/tap_flow.c @@ -166,8 +166,8 @@ static const struct tap_flow_items tap_flow_items[] = { RTE_FLOW_ITEM_TYPE_IPV4, RTE_FLOW_ITEM_TYPE_IPV6), .mask = &(const struct rte_flow_item_eth){ - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", - .hdr.src_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .hdr.src_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .hdr.ether_type = -1, }, .mask_sz = sizeof(struct rte_flow_item_eth), @@ -209,14 +209,10 @@ static const struct tap_flow_items tap_flow_items[] = { RTE_FLOW_ITEM_TYPE_TCP), .mask = &(const struct rte_flow_item_ipv6){ .hdr = { - .src_addr = { - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - }, - .dst_addr = { - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - }, + .src_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .dst_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .proto = -1, }, }, @@ -299,7 +295,7 @@ static struct remote_rule implicit_rte_flows[TAP_REMOTE_MAX_IDX] = { .items[0] = { .type = RTE_FLOW_ITEM_TYPE_ETH, .mask = &(const struct rte_flow_item_eth){ - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }, }, .items[1] = { @@ -316,10 +312,10 @@ static struct remote_rule implicit_rte_flows[TAP_REMOTE_MAX_IDX] = { .items[0] = { .type = RTE_FLOW_ITEM_TYPE_ETH, .mask = &(const struct rte_flow_item_eth){ - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }, .spec = &(const struct rte_flow_item_eth){ - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }, }, .items[1] = { @@ -336,10 +332,10 @@ static struct remote_rule implicit_rte_flows[TAP_REMOTE_MAX_IDX] = { .items[0] = { .type = RTE_FLOW_ITEM_TYPE_ETH, .mask = &(const struct rte_flow_item_eth){ - .hdr.dst_addr.addr_bytes = "\x33\x33\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x33, 0x33, 0x00, 0x00, 0x00, 0x00 }, }, .spec = &(const struct rte_flow_item_eth){ - .hdr.dst_addr.addr_bytes = "\x33\x33\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x33, 0x33, 0x00, 0x00, 0x00, 0x00 }, }, }, .items[1] = { @@ -370,10 +366,10 @@ static struct remote_rule implicit_rte_flows[TAP_REMOTE_MAX_IDX] = { .items[0] = { .type = RTE_FLOW_ITEM_TYPE_ETH, .mask = &(const struct rte_flow_item_eth){ - .hdr.dst_addr.addr_bytes = "\x01\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x01, 0x00, 0x00, 0x00, 0x00, 0x00 }, }, .spec = &(const struct rte_flow_item_eth){ - .hdr.dst_addr.addr_bytes = "\x01\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x01, 0x00, 0x00, 0x00, 0x00, 0x00 }, }, }, .items[1] = { diff --git a/examples/l2fwd-macsec/main.c b/examples/l2fwd-macsec/main.c index 294ee113e308..1303b0457e38 100644 --- a/examples/l2fwd-macsec/main.c +++ b/examples/l2fwd-macsec/main.c @@ -510,8 +510,8 @@ create_default_flow(uint16_t portid) struct rte_flow *flow; struct rte_flow_item_eth eth; static const struct rte_flow_item_eth eth_mask = { - .hdr.dst_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", - .hdr.src_addr.addr_bytes = "\x00\x00\x00\x00\x00\x00", + .hdr.dst_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, + .hdr.src_addr.addr_bytes = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }, .hdr.ether_type = RTE_BE16(0xFFFF), }; int ret; diff --git a/lib/ethdev/rte_flow.h b/lib/ethdev/rte_flow.h index a2929438bf6e..22c5c147d0ea 100644 --- a/lib/ethdev/rte_flow.h +++ b/lib/ethdev/rte_flow.h @@ -903,8 +903,8 @@ struct rte_flow_item_eth { /** Default mask for RTE_FLOW_ITEM_TYPE_ETH. */ #ifndef __cplusplus static const struct rte_flow_item_eth rte_flow_item_eth_mask = { - .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", - .hdr.src_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.dst_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .hdr.src_addr.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .hdr.ether_type = RTE_BE16(0x0000), }; #endif @@ -1005,12 +1005,10 @@ struct rte_flow_item_ipv6 { #ifndef __cplusplus static const struct rte_flow_item_ipv6 rte_flow_item_ipv6_mask = { .hdr = { - .src_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", - .dst_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .src_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, + .dst_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }, }; #endif @@ -1132,7 +1130,7 @@ struct rte_flow_item_vxlan { /** Default mask for RTE_FLOW_ITEM_TYPE_VXLAN. */ #ifndef __cplusplus static const struct rte_flow_item_vxlan rte_flow_item_vxlan_mask = { - .hdr.vni = "\xff\xff\xff", + .hdr.vni = { 0xff, 0xff, 0xff }, }; #endif @@ -1185,7 +1183,7 @@ struct rte_flow_item_nvgre { /** Default mask for RTE_FLOW_ITEM_TYPE_NVGRE. */ #ifndef __cplusplus static const struct rte_flow_item_nvgre rte_flow_item_nvgre_mask = { - .tni = "\xff\xff\xff", + .tni = { 0xff, 0xff, 0xff }, }; #endif @@ -1205,7 +1203,7 @@ struct rte_flow_item_mpls { /** Default mask for RTE_FLOW_ITEM_TYPE_MPLS. */ #ifndef __cplusplus static const struct rte_flow_item_mpls rte_flow_item_mpls_mask = { - .label_tc_s = "\xff\xff\xf0", + .label_tc_s = { 0xff, 0xff, 0xf0 }, }; #endif @@ -1349,7 +1347,7 @@ struct rte_flow_item_geneve { /** Default mask for RTE_FLOW_ITEM_TYPE_GENEVE. */ #ifndef __cplusplus static const struct rte_flow_item_geneve rte_flow_item_geneve_mask = { - .vni = "\xff\xff\xff", + .vni = { 0xff, 0xff, 0xff }, }; #endif @@ -1386,7 +1384,7 @@ struct rte_flow_item_vxlan_gpe { */ #ifndef __cplusplus static const struct rte_flow_item_vxlan_gpe rte_flow_item_vxlan_gpe_mask = { - .hdr.vni = "\xff\xff\xff", + .hdr.vni = { 0xff, 0xff, 0xff }, }; #endif @@ -1420,9 +1418,9 @@ struct rte_flow_item_arp_eth_ipv4 { #ifndef __cplusplus static const struct rte_flow_item_arp_eth_ipv4 rte_flow_item_arp_eth_ipv4_mask = { - .hdr.arp_data.arp_sha.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.arp_data.arp_sha.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .hdr.arp_data.arp_sip = RTE_BE32(UINT32_MAX), - .hdr.arp_data.arp_tha.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .hdr.arp_data.arp_tha.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, .hdr.arp_data.arp_tip = RTE_BE32(UINT32_MAX), }; #endif @@ -1509,9 +1507,8 @@ struct rte_flow_item_icmp6_nd_ns { #ifndef __cplusplus static const struct rte_flow_item_icmp6_nd_ns rte_flow_item_icmp6_nd_ns_mask = { - .target_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .target_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }; #endif @@ -1536,9 +1533,8 @@ struct rte_flow_item_icmp6_nd_na { #ifndef __cplusplus static const struct rte_flow_item_icmp6_nd_na rte_flow_item_icmp6_nd_na_mask = { - .target_addr = - "\xff\xff\xff\xff\xff\xff\xff\xff" - "\xff\xff\xff\xff\xff\xff\xff\xff", + .target_addr = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }; #endif @@ -1587,7 +1583,7 @@ struct rte_flow_item_icmp6_nd_opt_sla_eth { #ifndef __cplusplus static const struct rte_flow_item_icmp6_nd_opt_sla_eth rte_flow_item_icmp6_nd_opt_sla_eth_mask = { - .sla.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .sla.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }; #endif @@ -1612,7 +1608,7 @@ struct rte_flow_item_icmp6_nd_opt_tla_eth { #ifndef __cplusplus static const struct rte_flow_item_icmp6_nd_opt_tla_eth rte_flow_item_icmp6_nd_opt_tla_eth_mask = { - .tla.addr_bytes = "\xff\xff\xff\xff\xff\xff", + .tla.addr_bytes = { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, }; #endif @@ -2075,7 +2071,7 @@ struct rte_flow_item_ib_bth { static const struct rte_flow_item_ib_bth rte_flow_item_ib_bth_mask = { .hdr = { .opcode = 0xff, - .dst_qp = "\xff\xff\xff", + .dst_qp = { 0xff, 0xff, 0xff }, }, }; #endif -- 2.43.0