From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qv1-f43.google.com (mail-qv1-f43.google.com [209.85.219.43]) (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 CBD5F2853FD for ; Tue, 8 Sep 2026 23:31:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788910279; cv=none; b=InVlS+l6/kY3IjbLu5f3c4YjIueoOeRdG9pAmFQI/8QkY+4tHBtOOb5yojahtfEuiPvSN7FLvGI/Q/WXRv74yrvGhmw4zEgdkNZTHondFKN70tIXxRetWz036W0jQsqP0MvZzuvgJfn82bjL0C69bHPMj/BWpMHhQ9g/sKbZ/VM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788910279; c=relaxed/simple; bh=lgK52YWEbmOHlLC5XhevTS6kX/pGRh0yPlV7xZbQ1f0=; h=Mime-Version:Content-Type:Date:Message-Id:Cc:Subject:From:To: References:In-Reply-To; b=r8MQWH6kGOuxY12W6pBh6g/3S3VhF+7212sPmQNb0yQlhIiAYcjaoFD2Vlg4vLVnBTiv+RJPXr21uYE68MD6suN176To/NibDoUQC/sVVd8jtNwG2cgn/jy+UDGp5Tn0XjShSAyiL1WzjXyuwjAEMCsVVDlP3JyZLK86tSFLDcw= 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=eMUqpufU; arc=none smtp.client-ip=209.85.219.43 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="eMUqpufU" Received: by mail-qv1-f43.google.com with SMTP id 6a1803df08f44-9103f5b813aso42300686d6.3 for ; Tue, 08 Sep 2026 16:31:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1788910277; x=1789515077; darn=vger.kernel.org; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-type:content-transfer-encoding:mime-version:from:to:cc :subject:date:message-id:reply-to:content-type; bh=Yq7lXkoWC5jfEWJMNBS6tBwhDj/bbJjyiLJq/Z7eNiQ=; b=eMUqpufUtakeZcaUo0NM8W4QbYk15mZl5vy0UDj4R2McNaJEoUur8T/m7FSwYfQe4o HVr2MZKCzKGgIfhlzQV2K+Tsycof34HGcUv7XI+zeTrq7+ahMvF316oUcCDtgr6JG20A SpGUNVge2hripIhw90sWNWu2HkKXxiwJYRAqn8znRt/eVBhn6c8wlZ/49AsmA3H3cl/b KJXCE39fBlAIf/jdkxt2sNTiva1RHEgQnk5XedHZsg5q1AnsYE/DzLPzJT2Z9I6HROKd xtXRd4o6Pi8Zww0vvKVBqzGrE0/6aQW6ZnEXxwrZZkYWAxhMpdAjqYahzK0i12evC7BT 5AJQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788910277; x=1789515077; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-type:content-transfer-encoding:mime-version:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=Yq7lXkoWC5jfEWJMNBS6tBwhDj/bbJjyiLJq/Z7eNiQ=; b=dnYqCkTCKOdRSUBaEVJTIedNpa1KJKZPc/muQdsUSbR+RjmhmRRZxU0A2+/GoaWvNi NdqBe+dk8sjrK7gJ32BinQOVfdmHX5jvExGbwb+nl9VJ53Xl9K893zKOnYX57Bg+WCRu MZjOyiDLfLg9a+4L6morfT/49q083/ohHplxm9GX/siFca+Xsx8WnpRnhsQf4ZYWgFA1 Zc6/ywzi5QfcoNO+uNjK/NzL0SyH1fGtj2Hoyd9STz/co/pcrEFZTwkBMVeTfRZmwOLM 7MlChqJJSSoI79b6j9628t0/ALKr5O22chuv8tPy23gisnZN4K6NoBjTOLqwHRlKbTvk dBxw== X-Forwarded-Encrypted: i=1; AKwUvBwPtJSlMiYucsxqgYnrDgYMI9kVUWqwftjjt7Q47gdMubPHqmRs7g3GzWikV5gIGBTVThn0tJdZpZcsg9R6pns=@vger.kernel.org X-Gm-Message-State: AFuF++kiN2j/BW9P0LmVGOhgv4Y8EwfUQHSl1QesrDwHG4Y/aGo1GVln MySq8dVt68MLPNXnrX9auH+gJnYL8kR03eBJtCPBQFiwz5dIE53AdCMO X-Gm-Gg: AYBFou1PqMLXmPlJh09hY5Gxr6/2Hxeuq1OmBStlfYwDb9WW+DItnc7VsQszUkVzHrx 1Dz692ztQvHhXJgjwktynqYrrJJ4vXkxE/bHQUFV7CD8ibEVbqGBz1J2P9h3DK9naS4tWEI7rtc BOl6Ke08dkstdj1VJiH8NGbB+QRAlmOmYwG02VyYwcoi0ca7IrYeIP/yCFD67uQthgauQuxRcQP F7IU6YKjyMbIUozOCu8BAfsmXwvpuiIpHOBoS5Xr/wJ+OEYu4nNGKrVfvZGrgp5lVzBv2M+wmle MBT9ckB14TjrBe8U+VvObhuUFTyyYeDy6OzZxkMrejTdJBb1k1CIdLfrkSzUgiOEwX72+0fJsN9 gU3t/rIzf0VqVxelY/TuO7gVfeSvds1EB65NllyhgJx/DMBBWEDUwAYCJS6gzXacLtmlx17F8DQ c305Lo7rTjmXnLlgiMBkwA8t4pWuTLTgNRXZ7i6eQ2pVyKU5U2azZFDoi5yCRY3Hlpmg== X-Received: by 2002:ad4:5ce2:0:b0:908:958f:4db with SMTP id 6a1803df08f44-9103eefd24fmr369191086d6.3.1788910276570; Tue, 08 Sep 2026 16:31:16 -0700 (PDT) Received: from localhost ([2600:4040:9399:4000:e553:72e5:7d37:c7ef]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-91040694c35sm129077696d6.36.2026.09.08.16.31.15 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 08 Sep 2026 16:31:16 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-kselftest@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Tue, 08 Sep 2026 19:31:14 -0400 Message-Id: Cc: "Bobby Eshleman" , "Boris Pismenny" , "Carolina Jubran" , "Cosmin Ratiu" , "Daniel Zahka" , "Doruk Tan Ozturk" , "Dragos Tatulea" , "Gal Pressman" , "Jianbo Liu" , "Kees Cook" , "Leon Romanovsky" , "open list" , , , "Mark Bloch" , "Petr Machata" , "Raed Salem" , "Rahul Rameshbabu" , "Saeed Mahameed" , "Shuah Khan" , "Shuah Khan" , "Simon Horman" , "Willem de Bruijn" , "Willem de Bruijn" Subject: Re: [PATCH net-next V4 09/14] net/mlx5e: psp: Add an rx_decap steering table From: "Daniel Zahka" To: "Tariq Toukan" , "Andrew Lunn" , "David S. Miller" , "Eric Dumazet" , "Jakub Kicinski" , , "Paolo Abeni" , "Sabrina Dubroca" X-Mailer: aerc 0.21.0-threadmapfix References: <20260908065244.3799142-1-tariqt@nvidia.com> <20260908065244.3799142-10-tariqt@nvidia.com> In-Reply-To: <20260908065244.3799142-10-tariqt@nvidia.com> On Tue Sep 8, 2026 at 2:52 AM EDT, Tariq Toukan wrote: > From: Cosmin Ratiu > =20 > +static void accel_psp_fs_rx_reconfigure(struct mlx5e_psp_fs *fs, > + bool decap_wanted) > +{ > + bool decap_supported =3D > + MLX5_CAP_FLOWTABLE(fs->mdev, > + flow_table_properties_nic_receive.reformat_del_psp_transport); not for this series, but this makes me think having some way of letting the user know whether or not psp with hw gro is enabled would be nice, so that they don't wonder why stuff isn't coalescing when they have old fw like I do :) I could see a similar thing being useful for gso features if the psp device can't segment something that it otherwise can for non-psp. > + bool decap_enable =3D decap_wanted && decap_supported; > + struct mlx5_flow_destination dest =3D {}; > + int err; > + > + if (decap_enable =3D=3D fs->decap_enabled) > + return; > + > + /* Create the decap table if needed. */ > + if (decap_enable && !fs->decap.ft) { > + err =3D accel_psp_fs_rx_decap_ft_create(fs, &fs->decap); > + if (err) > + goto out_err; > + } > + > + /* Redirect traffic to the correct table. */ > + dest.type =3D MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE; > + dest.ft =3D decap_enable ? fs->decap.ft : fs->rx.ft; > + err =3D mlx5_modify_rule_destination(fs->check.rule, &dest, NULL); > + if (err) > + goto out_err; > + > + fs->decap_enabled =3D decap_enable; > + return; > + > +out_err: > + mlx5_core_warn(fs->mdev, > + "Failed to create/modify PSP decapsulation rules (err %d), HW G= RO for PSP unavailable\n", > + err); > +}