From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qv1-f42.google.com (mail-qv1-f42.google.com [209.85.219.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 760BA43C7AB for ; Tue, 7 Jul 2026 18:30:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783449004; cv=none; b=W+0TDU5LVnwQHQ7qCAOTHsetf/Jd2Z+wop3D74nJYaVW21Fu90NbO1Xd6KDqjNXiSg5JKnrIgl+Y5abyv9DhZtn/xMDdi4SLOyv3gNU8AqfUKVf+1uMw/UDE0PCs5uk3YUW49OrxlyUXH15oD1HqYQTM8uS4qoaLsZ5MoiAJC00= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783449004; c=relaxed/simple; bh=BC4hsNu1Ymw7Ru2UgY5LJ4zKQ7w97jP2orYIw5ekKU4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=KmhJyso/01AmxYZVoyt3xo44kraHq14zhiThd6wRAMma/FMy/Lwn7lu4T+Y0NsE+xAW/2srLfZP1UpGQUAYfAO08vhiuVxarssW32rpIxT4Z8B7AAgc8STlWYXdZComrH5PMKjnliAbI2GFD1R89hjH2lJUsV2/ps90wJ4mWBc4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=dIYUo1Fj; arc=none smtp.client-ip=209.85.219.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="dIYUo1Fj" Received: by mail-qv1-f42.google.com with SMTP id 6a1803df08f44-8f3cbaef818so29478516d6.3 for ; Tue, 07 Jul 2026 11:30:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783449001; x=1784053801; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=CyPIHfNf3UfKvOML8iyzORC7dRkgEzIxIf93zJCMmMk=; b=dIYUo1FjPh6U0U4vtqQ2emtre9QaFLdfVI8ni25pnfWxLGiBtJrTgBSo642/iL10fb U6lw2EyKq7e6rKhKGs0G+mjtzUAcLI8FTi6O8STJjzxROh+LxkLCVJr0Vm1jEvK3nB8/ wdVyNS+ggniWX9wfTWMte2bg59UaAemeXXU/yYYKaBYUTuO80Rd8QVRtZphumMohrJdt sggksQkW6Ayl8cLfkKTbVmfhfNlOpeYpTcHC9ukUsy+0j5KwBqdzw+qM9vsJQ5N7Mu5U JYD6WkRo+LvThDfBnDfr/K1RwbXNm+idgzH9CyX7X9S9cJu0v58feCw1OMJNW0HkgIif HrtQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783449001; x=1784053801; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=CyPIHfNf3UfKvOML8iyzORC7dRkgEzIxIf93zJCMmMk=; b=jCd7gA29s3nbbufs6UZ0VKEAjuVmI6XLwEKqZdGF70t97Wiqbx/G/txZm2/McStVoR BOHTnbBbcP4ohcxhAGkhERpLK428fAUP5mCD0+POIWMRv+qExXRRBMWAb3wPrpY8gOnV NTQD6f4VB3blFTIZHCR8MrKHMYSAjEiPzbdJAFUF6tOKo/i/l3W0xeCV/fszvZs93nlj hBozXTqLdMgVpASLyxQmj8qvTO5gO43fuTk5AX41Q+7dfHNtjTB63elmqzb3f/1vTjra yhlYaY5sqVXAX1WdffOlLr/YP4OWCxZ+JzR5krSiSLHJiHJrKXbGlcjvfnwgWMetP33e +CWQ== X-Forwarded-Encrypted: i=1; AHgh+Rr9drM4wVnpNf2MqMhDilrfd8qwSX9z5MQuwkQmcqkTSe8IdxyzAi6SAsZmaUOGd/VG0reU8XvNDb2Bo30=@vger.kernel.org X-Gm-Message-State: AOJu0Yzh0bnNCiFcvbzP/MUNjWnyzKGTOYFl+JLt5KxdcJ9Ngvij26l0 3+RszHto/+eDRf6M72iE/Ljohp0NV/O3gLQPCEMa+8S54dzcjVtm+q9L X-Gm-Gg: AfdE7clu5deHF4V324v2WJyWyOHPoVyGLZPw168jx1b8eXPybLoTglFKkgR+6Q6ud86 zLIpDl3RpTGsQzpvZHOpPToFsbYtj75AQTdsLTyZUKr0wqNNOZRovUzMujFoBLwNCpJuIH6sKM5 9czaekR8NQkIRAD61aSXzoquHoHVh4hsYeqxdMF9ATtBaHb0owWc3i51SOzfCDtQNClU/vDCFRl KvjyxEEvkXBUXGGm5KhPQgMKH2OG9Kc7Cu8acA/n8OZrGCBKIlX9kgarFPEKBTuj7t31hGuMcjQ gXDM+JVUHHJsMtpjxquqwX/U/YnH8xDoMGz5TU5SMNzuZi0HWwrQQMmaBDjCeJWjz3Dcjp7/33A zjYNm3XOoEnKZqX77C36B8/UF5B3INaY8Gstvt6rcmII6gz6KKLwzhiqNXU0JNgdvSu0a1Bgf3C kNR9rzN/GVJZLcZky5uc7sXasxyr7NQSEcYZy9Q+PkbD4= X-Received: by 2002:a05:6214:e4b:b0:8f2:f32c:6bbb with SMTP id 6a1803df08f44-8fcb628f719mr77752796d6.51.1783449001234; Tue, 07 Jul 2026 11:30:01 -0700 (PDT) Received: from ?IPV6:2a03:83e0:1145:4:d844:20d:e589:1800? ([2620:10d:c091:500::3:f3b7]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-8f46aecc908sm168889176d6.0.2026.07.07.11.29.59 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 07 Jul 2026 11:30:00 -0700 (PDT) Message-ID: <0dfe5f6b-dbc8-4104-8883-e88e8e59ab58@gmail.com> Date: Tue, 7 Jul 2026 14:29:59 -0400 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next 00/15] net/mlx5e: PSP cleanups and improvements To: Tariq Toukan , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , netdev@vger.kernel.org, Paolo Abeni Cc: Aleksandr Loktionov , Boris Pismenny , Chris Mi , Cosmin Ratiu , Dragos Tatulea , Gal Pressman , Jacob Keller , Jianbo Liu , Lama Kayal , Leon Romanovsky , linux-kernel@vger.kernel.org, linux-rdma@vger.kernel.org, Mark Bloch , Raed Salem , Rahul Rameshbabu , Saeed Mahameed , Stanislav Fomichev , Stanislav Fomichev , Willem de Bruijn References: <20260707130858.969928-1-tariqt@nvidia.com> Content-Language: en-US From: Daniel Zahka In-Reply-To: <20260707130858.969928-1-tariqt@nvidia.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 7/7/26 9:08 AM, Tariq Toukan wrote: > Hi, > > This series by Cosmin refactors mlx5 PSP support in preparation for > HW-GRO support. > There are almost no functionality changes in all but the last two > patches, which address a long-standing TODO in mlx5e_psp_set_config(). > > Regards, > Tariq > > Cosmin Ratiu (15): > net/mlx5e: psp: Rename the saved psp_dev to 'psd' > net/mlx5e: psp: Remove PSP steering mutexes > net/mlx5e: psp: Remove unneeded ref counting for PSP steering > net/mlx5e: psp: Merge rx_err rule add/delete with ft create/delete > net/mlx5e: psp: Use helpers for steering object manipulation > net/mlx5e: psp: Factor out drop rule creation code > net/mlx5e: psp: Remove unused PSP syndrome copy action > net/mlx5e: psp: Rename and consolidate steering functions > net/mlx5e: psp: Adjust rx_check FT size and use a drop_group > net/mlx5e: psp: Add an RX steering table > net/mlx5e: psp: Use a single rx_check table > net/mlx5e: psp: Flatten steering structures > net/mlx5e: psp: Make PSP steering config dynamic > net/mlx5e: Return errors from profile->enable > net/mlx5e: psp: Report PSP dev registration errors > > drivers/net/ethernet/mellanox/mlx5/core/en.h | 2 +- > .../net/ethernet/mellanox/mlx5/core/en/fs.h | 7 +- > .../mellanox/mlx5/core/en_accel/en_accel.h | 19 +- > .../mellanox/mlx5/core/en_accel/psp.c | 1007 ++++++++--------- > .../mellanox/mlx5/core/en_accel/psp.h | 18 +- > .../mellanox/mlx5/core/en_accel/psp_rxtx.c | 13 +- > .../mellanox/mlx5/core/en_accel/psp_rxtx.h | 3 +- > .../net/ethernet/mellanox/mlx5/core/en_main.c | 23 +- > .../net/ethernet/mellanox/mlx5/core/en_rep.c | 8 +- > 9 files changed, 516 insertions(+), 584 deletions(-) > > > base-commit: 31816fc5d9acf8cdf226cdd0dc296e8cf15cc033 Thanks. Excited about the support for mlx5e_psp_set_config(). Jakub and I had a test case for psp_dev_ops::set_config() that we were waiting to upstream. I just rebased it onto net-next here: https://github.com/danieldzahka/linux/commit/b58e9a99573cf6b884e5fe3227c9af7a1f0d80b0 I ran it with the series but am seeing an error trying to catch undecrypted PSP-UDP packets after disabling all versions with set_config() TAP version 13 1..30 ok 1 psp.data_basic_send.v0_ip4 # SKIP Test requires IPv4 connectivity ok 2 psp.data_basic_send.v0_ip6 ok 3 psp.data_basic_send.v1_ip4 # SKIP Test requires IPv4 connectivity ok 4 psp.data_basic_send.v1_ip6 ok 5 psp.data_basic_send.v2_ip4 # SKIP Test requires IPv4 connectivity ok 6 psp.data_basic_send.v2_ip6 # SKIP ('PSP version not supported', 'hdr0-aes-gmac-128') ok 7 psp.data_basic_send.v3_ip4 # SKIP Test requires IPv4 connectivity ok 8 psp.data_basic_send.v3_ip6 # SKIP ('PSP version not supported', 'hdr0-aes-gmac-256') ok 9 psp.data_mss_adjust.ip4 # SKIP Test requires IPv4 connectivity ok 10 psp.data_mss_adjust.ip6 ok 11 psp.data_send_off.ip4 # SKIP Test requires IPv4 connectivity # Exception| Traceback (most recent call last): # Exception|   File "/root/ksft-psp-set-config/net/lib/py/ksft.py", line 420, in ksft_run # Exception|     func(*args) # Exception|   File "/root/./ksft-psp-set-config/drivers/net/psp.py", line 608, in data_send_off # Exception|     udps.recv(8192, socket.MSG_DONTWAIT) # Exception| BlockingIOError: [Errno 11] Resource temporarily unavailable # Exception| not ok 12 psp.data_send_off.ip6 ok 13 psp.dev_list_devices ok 14 psp.dev_get_device ok 15 psp.dev_get_device_bad ok 16 psp.dev_rotate ok 17 psp.dev_rotate_spi ok 18 psp.assoc_basic ok 19 psp.assoc_bad_dev ok 20 psp.assoc_sk_only_conn ok 21 psp.assoc_sk_only_mismatch ok 22 psp.assoc_sk_only_mismatch_tx ok 23 psp.assoc_sk_only_unconn ok 24 psp.assoc_version_mismatch ok 25 psp.assoc_twice ok 26 psp.data_send_bad_key ok 27 psp.data_send_disconnect ok 28 psp.data_stale_key ok 29 psp.removal_device_rx # XFAIL Test only works on netdevsim ok 30 psp.removal_device_bi # XFAIL Test only works on netdevsim # Totals: pass:19 fail:1 xfail:2 xpass:0 skip:8 error:0 # # Responder logs (0): # STDERR: # #  Set PSP enable on device 1 to 0x3 # #  Set PSP enable on device 1 to 0x0 I recall this working on an earlier prototype of this feature for mlx5. Are the steering rules setup to drop PSP-UDP packets when the corresponding psp version is disabled?