Received-SPF: pass (google.com: domain of linux-kernel+bounces-194968-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
From: "Rout, ChandanX" <chandanx.rout@intel.com>
To: "Zaremba, Larysa" <larysa.zaremba@intel.com>,
	"intel-wired-lan@lists.osuosl.org" <intel-wired-lan@lists.osuosl.org>,
	"Keller, Jacob E" <jacob.e.keller@intel.com>
CC: "Fijalkowski, Maciej" <maciej.fijalkowski@intel.com>, "Jesper Dangaard
 Brouer" <hawk@kernel.org>, Daniel Borkmann <daniel@iogearbox.net>, "Kitszel,
 Przemyslaw" <przemyslaw.kitszel@intel.com>, John Fastabend
	<john.fastabend@gmail.com>, Alexei Starovoitov <ast@kernel.org>, "David S.
 Miller" <davem@davemloft.net>, Eric Dumazet <edumazet@google.com>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>, Jakub Kicinski
	<kuba@kernel.org>, "bpf@vger.kernel.org" <bpf@vger.kernel.org>, Paolo Abeni
	<pabeni@redhat.com>, Magnus Karlsson <magnus.karlsson@gmail.com>, "Bagnucki,
 Igor" <igor.bagnucki@intel.com>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>, "Kuruvinakunnel, George"
	<george.kuruvinakunnel@intel.com>, "Pandey, Atul" <atul.pandey@intel.com>,
	"Nagraj, Shravan" <shravan.nagraj@intel.com>
Subject: RE: [Intel-wired-lan] [PATCH iwl-net 3/3] ice: map XDP queues to
 vectors in ice_vsi_map_rings_to_vectors()
Thread-Topic: [Intel-wired-lan] [PATCH iwl-net 3/3] ice: map XDP queues to
 vectors in ice_vsi_map_rings_to_vectors()
Thread-Index: AQHapuJhTByGvJMbykuLo11n4VFJRLGvXlKA
Date: Thu, 30 May 2024 05:57:16 +0000
Message-ID: <CH3PR11MB83136BED89135F5F702B2756EAF32@CH3PR11MB8313.namprd11.prod.outlook.com>
References: <20240515160246.5181-1-larysa.zaremba@intel.com>
 <20240515160246.5181-4-larysa.zaremba@intel.com>
In-Reply-To: <20240515160246.5181-4-larysa.zaremba@intel.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?kuwIQ3APXBGxn6L606Bvv7b+jI2X+rEp7bDJAswNLYxIh7DWqNfQkv1IC+zk?=
 =?us-ascii?Q?5cShLFYbaK73+UtFrsB2twjgDeY67JH2LrMA1Q0UaxiZofQoRlEYDytpu78B?=
 =?us-ascii?Q?Kjj8s9SiszZeT/bnfuTDw2F3kq0DVE514tFc12lTNKwGQCDLOzKXjWlsSdQM?=
 =?us-ascii?Q?uABFkcj3uvNKDaxB1/PDxSXFJFj9U6z/AJva7VLfDOyIBEeWI331rr43Pd8p?=
 =?us-ascii?Q?ZME3khZ0W3tufU1jE6mg6yWl+2YbDkUSC1mMWczFHxCjiloGb/ssM431zXUE?=
 =?us-ascii?Q?zPKq3LF3fP+m4gFgnwTKmyxy63QbNouZVxd6NU9YSKpH6lxi6SFe4Hh7vKE9?=
 =?us-ascii?Q?Rpsv3cGEVBWb+YgnDvrgSLh3O2FALd3Qr3+d2VvOpfZurSxq4B1Me5kvhj8M?=
 =?us-ascii?Q?vYuQb4RK6i+xSOfuD+4CgOrftYpoTa6lxWICXe5CYx+QIYykblI976ujpyon?=
 =?us-ascii?Q?t1QiPLX+bjNWPBqbuiBxBwh4B7hJQdzJOEax0pjzKTkKiijZCxbG69MydG98?=
 =?us-ascii?Q?ftIj00Y/ildbHDPgpfJTHu/MDgHXT9Nm5bYbMlnOER2atfWold32rgx6CXva?=
 =?us-ascii?Q?v8VbomSP0QA7S7jbrtW3xyAQQqkMwu9PFKBJa0xt+qHTH/MmnBwxArauqva+?=
 =?us-ascii?Q?zgHqmZ6STLd4ZOZUvmcc7RKsvW2YHogcmD5IfQGK9mHrrLHUMPp6uvLemBtx?=
 =?us-ascii?Q?CK5mhCm66lB7lJmV0oHO3YYXuj4sgSkl4uRnwvNYR+yj1dN1j2lIxyzOz1fG?=
 =?us-ascii?Q?9nF6h/oiAacViCLLcgYeg6rhZYdpFqHiKy9DvvH3874yKIaBMgL6cdIR/JsL?=
 =?us-ascii?Q?puT/ui3IItkLGVO8e9/rRpy7iGbnP0ghFrmaXcLu/+bk7C9gkrj12QfKLdiF?=
 =?us-ascii?Q?D4FoCuxPDmLKQuK/ka0RTvCKPlcRq+OK6qI+wA+E1Thfm7VQu4O4duWtnqDQ?=
 =?us-ascii?Q?OgAejbMkL83VnT7o+Wz7qzhf+nZRfEguA1GVsDjFmet013W+6jxyeIYSjIVY?=
 =?us-ascii?Q?5sOJwbBHYTg1We6AP9ym3GbkL9Wr3QSIy0mmwel9xTD3TrxPjYUOK3Ct6blS?=
 =?us-ascii?Q?76ePmoWbx8vo/YzUhVPTnArGt9CnKCOPGrNrW9/vddVysMFSi/gK3QS8Flao?=
 =?us-ascii?Q?+XWDWPD4eKcTRaq3/J7kKsI+m7qPWt0HMjuZDNE06d/T7HTPd6QbFHIZzI3a?=
 =?us-ascii?Q?tvNBb74/hEE9IqGDwlK3qI/SXWXG+E5MbYFtddH1uhOWcBe+ckh24F+uRqRM?=
 =?us-ascii?Q?IUoQ5eTRMrBtty+HahKNmzxxhtA4HN0BA60Hc0ENQv6NA6MdiFixF8NtJftZ?=
 =?us-ascii?Q?XT0WTU4U/t3GokEckmDP9W7qdDtcof/QS85toY7afJxK/1iQTncW9l5ISGLh?=
 =?us-ascii?Q?DdcOS/2RkErPsiiyMKfwndCln6eauKy8CmQ07vTETVlEEgUjUCFkGXwrQiwU?=
 =?us-ascii?Q?dtZlyXtCYvKyTvj5lQ+64fHGq5ptSjiIfQ1peVEIIeVewkYiYacOeMZs7izj?=
 =?us-ascii?Q?Kz/exCeKV8dy9B8893FNuYJq4coJ1EKpVdHl+F9vGICY7MgOxJuQw3FoRhCR?=
 =?us-ascii?Q?XRC0k2bU6MNVZTcVWd+ef/hKkzJhAMHFIBQ+jO9H?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CH3PR11MB8313.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c1ea5151-0171-4887-dc12-08dc806d5c6e
X-MS-Exchange-CrossTenant-originalarrivaltime: 30 May 2024 05:57:16.9971
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: THcErkv4XhZxMF4JmYY0h2PV/0QdY9cgRonD0X0LqMqLc4dOF0JClkZbfBzOBDbjxi5ZRJFnBsLjiu8kzuuF+Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR11MB7204
X-OriginatorOrg: intel.com


>-----Original Message-----
>From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
>Zaremba, Larysa
>Sent: Wednesday, May 15, 2024 9:32 PM
>To: intel-wired-lan@lists.osuosl.org; Keller, Jacob E <jacob.e.keller@inte=
l.com>
>Cc: Fijalkowski, Maciej <maciej.fijalkowski@intel.com>; Jesper Dangaard Br=
ouer
><hawk@kernel.org>; Daniel Borkmann <daniel@iogearbox.net>; Zaremba,
>Larysa <larysa.zaremba@intel.com>; Kitszel, Przemyslaw
><przemyslaw.kitszel@intel.com>; John Fastabend
><john.fastabend@gmail.com>; Alexei Starovoitov <ast@kernel.org>; David S.
>Miller <davem@davemloft.net>; Eric Dumazet <edumazet@google.com>;
>netdev@vger.kernel.org; Jakub Kicinski <kuba@kernel.org>;
>bpf@vger.kernel.org; Paolo Abeni <pabeni@redhat.com>; Magnus Karlsson
><magnus.karlsson@gmail.com>; Bagnucki, Igor <igor.bagnucki@intel.com>;
>linux-kernel@vger.kernel.org
>Subject: [Intel-wired-lan] [PATCH iwl-net 3/3] ice: map XDP queues to vect=
ors
>in ice_vsi_map_rings_to_vectors()
>
>ice_pf_dcb_recfg() re-maps queues to vectors with
>ice_vsi_map_rings_to_vectors(), which does not restore the previous state =
for
>XDP queues. This leads to no AF_XDP traffic after rebuild.
>
>Map XDP queues to vectors in ice_vsi_map_rings_to_vectors().
>Also, move the code around, so XDP queues are mapped independently only
>through .ndo_bpf().
>
>Fixes: 6624e780a577 ("ice: split ice_vsi_setup into smaller functions")
>Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com>
>Signed-off-by: Larysa Zaremba <larysa.zaremba@intel.com>
>---
> drivers/net/ethernet/intel/ice/ice.h      |  1 +
> drivers/net/ethernet/intel/ice/ice_base.c |  3 +
>drivers/net/ethernet/intel/ice/ice_lib.c  | 14 ++--
>drivers/net/ethernet/intel/ice/ice_main.c | 96 ++++++++++++++---------
> 4 files changed, 68 insertions(+), 46 deletions(-)
>

Tested-by: Chandan Kumar Rout <chandanx.rout@intel.com> (A Contingent Worke=
r at Intel)