From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1257C47F3D1 for ; Mon, 5 Oct 2026 12:04:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791201848; cv=none; b=BsN+q9xs5s5Oy1nPzFlbikdQtX8g0q7aK/jjozeUWTgTb0BnfOFH5DEykCC1NYe5W4oOygDgkxwZakcxq0z2N9D6mDOwhieXaHBCNgAz1m3OQ4Kc1K52ZkgyYx4Uzk42RG61dTSS8fxmP+yFtKFhbMBqijZsxyr6otzOkfKgFIk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791201848; c=relaxed/simple; bh=haygrbUGZc/yuuLx+pcROb+Rr2PyaZWebOApPN6gcno=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=jhZyrzEUsCrJjGrvrzT/y0rLQPKyRgrO0MfY5Y1s7sZs02nDMj+oHonEoNPpdz4wg3ek9Be271X7hYyvje/jrB6wwQNFrpr1lrY8mPnaKr/0rtz6CYa5KVAIy1Z1h3L+FKN7tkwDJ4qfLSwUQUpLs5cDK61W2/ftDnL8GbkSoKE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=S0OHY3rB; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b=eGHlgj7i; arc=none smtp.client-ip=170.10.133.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="S0OHY3rB"; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b="eGHlgj7i" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1791201845; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=vhQoza2mV1Z+1STVDq8chd2IZUgXS0fGFoZOu7nkEJQ=; b=S0OHY3rBOE6prANN2M6BlCGYzoq+Qu61U108K3fjLTvKaqfeK09dgb2IKMs0azvV6dAgII Go2f1hFXrGs2Sc1Plai7NknxM4OxURBCbqZcFpX2QZdMjmpp0SDvE/CbKjVUCMoT4q3pmC bKQBm3KCcEwnwoGYscSnb42n/NH78CQ= Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-594-l92VDrINMAWlnXV_UxANUg-1; Mon, 05 Oct 2026 08:04:02 -0400 X-MC-Unique: l92VDrINMAWlnXV_UxANUg-1 X-Mimecast-MFC-AGG-ID: l92VDrINMAWlnXV_UxANUg_1791201841 Received: by mail-wm1-f71.google.com with SMTP id 5b1f17b1804b1-49cdc4080ddso14389215e9.1 for ; Mon, 05 Oct 2026 05:04:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=google; t=1791201841; x=1791806641; darn=vger.kernel.org; h=content-transfer-encoding:content-type: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 :content-type; bh=vhQoza2mV1Z+1STVDq8chd2IZUgXS0fGFoZOu7nkEJQ=; b=eGHlgj7iAGg8QoFPi3hL21AEBUaXj8j5UL8fR46PNU96K64Xc+kTajcDh3XTZVrUoS sK6+ozLvi3vPdK3N4vEl8Ri9x8272xNdyFXbv5cieiPvLBR8D/uGfxX5CbGe/racmyhV UZyJm469DX0Uvm8RHDrRu145dNUt66IgR2lxBYFPMwN5E4U/MZ5zj+3MAnbxBOq20Q96 5ZaKoYTfJKDiVah6i8UYwh7RVxoAocKV/fbLziXKnKrenpi8+iXtHBYzZ4vbLgLhhx5x //Co8MS7xL8anUoGSINmKFVl6PBg46eu/6Of1jHJtFO+ITgAYqEfc9+RGrrN3JxqhYPN xeBQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1791201841; x=1791806641; h=content-transfer-encoding:content-type: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:content-type; bh=vhQoza2mV1Z+1STVDq8chd2IZUgXS0fGFoZOu7nkEJQ=; b=afKqiBHS4kurJjTXVwN+cc89yg1o7iCoiY0W90QEqNcQJx7676HGhE7S7rYLgoExH+ tFvnkdzpquixqm16gsEOqx4qKRCgHcnyVTl2/Qp0L9V2p6/EvjsQHwgz2jiL2u4mH6A8 3lNXLrpvyMGe9jHl68n7ZAf0xnzD8N5F+qJsPyvabjJe2RaE2+/agsWIJS26AuSxjqG4 WwHyyu85IOeAmKx+2bePTbT9kHSjaFHm2H5Ht4lGYoCmp2ak4iIOVC0qjvlSCrjZ5QBs wqdEEqR81PQmLwpNElvr3CN4IsQmNdN+HVuMWp8JJkfiQHCMkQi/M1IdydIIi70xv9h6 NIAQ== X-Forwarded-Encrypted: i=1; AKwUvBztC0x2WF2DClLJIzeRAUFAeY/2I1W3mqh62d5ZsZU3QdA+BwFV/BLf1JHMd3Zmf92pxdCxB0Q0zJPpq50=@vger.kernel.org X-Gm-Message-State: AFuF++nn8gg0P62uNlnQ54dtKsjellKl7/jRNXG6dbVkKbM7Ee7lL7d9 wt+t6Vemi6ukSDPBrerlLTRkiYmL4JAe267EdBTCA70L57PsXXgAnkPQnKBsRhRNIF//VzepSOw yiOQD5fGLKRE+MWKJopjuMgCgzDoBfOgwLd8SdcQslci+8ONHNFVaX3pJgPGAYInILw== X-Gm-Gg: AYBFou3sCW8QA1whBp2yo6YJVh7BfoAaMYHBLSwg2GlZIVwB8FhJaokn5ZKtktg4qty qlOHeT2x5LWZlzDb5O5JYbsUk2q91NDFduDtgfRvIsr1dSioxgrNDXWr87KzFDfjCl1J85sWq6v 97n+4Qsxf6LeWe5x+d1xPJGZF66yTnxfWqsF8XVTLKhmNrQ5q19YAtTFqzFhWDPewwVJ/FNNhM+ ANTG29n5y+vogg6LAfkmg6GfZcHXeEa3Uagpqfo2bIFE879pN3noSUCMEDcic6w3BSDLd2ZLu/O s9z2zzsKKiFyDofZhUMTPD5RWkH2sMDPy0b9JgjDGB+6inQIR8IjMR3h5Q1hu8hjFqiG8oHHzQ= = X-Received: by 2002:a05:600c:2192:b0:4a0:20b4:ba93 with SMTP id 5b1f17b1804b1-4a027569390mr123921215e9.16.1791201841031; Mon, 05 Oct 2026 05:04:01 -0700 (PDT) X-Received: by 2002:a05:600c:2192:b0:4a0:20b4:ba93 with SMTP id 5b1f17b1804b1-4a027569390mr123920585e9.16.1791201840418; Mon, 05 Oct 2026 05:04:00 -0700 (PDT) Received: from [192.168.2.83] ([46.175.183.46]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4a16bcb9a21sm310240445e9.10.2026.10.05.05.03.52 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 05 Oct 2026 05:03:55 -0700 (PDT) Message-ID: Date: Mon, 5 Oct 2026 14:03:49 +0200 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 iwl-net 03/10] ice: stop the LAN Tx queues when ice_vsi_open() fails To: Ivan Vecera , intel-wired-lan@osuosl.org, netdev@vger.kernel.org Cc: Tony Nguyen , Przemek Kitszel , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Alexander Lobakin , Alexei Starovoitov , Daniel Borkmann , Jesper Dangaard Brouer , John Fastabend , Stanislav Fomichev , Henry Tieman , Anirudh Venkataramanan , Michal Swiatkowski , Jesse Brandeburg , Preethi Banala , Kiran Patil , Dan Nowlin , Stephen Hemminger , intel-wired-lan@lists.osuosl.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org References: <20261002130752.2597259-1-poros@redhat.com> <20261002130752.2597259-4-poros@redhat.com> <1465AD22-8132-4286-A2AF-ED94E64E12C8@redhat.com> Content-Language: en-US From: Petr Oros In-Reply-To: <1465AD22-8132-4286-A2AF-ED94E64E12C8@redhat.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 10/3/26 11:54, Ivan Vecera wrote: > On October 2, 2026 3:07:45 PM GMT+02:00, Petr Oros wrote: >> ice_vsi_cfg_lan() adds the LAN (and XDP) Tx queues to the scheduler and >> enables them in hardware. If anything after it fails in ice_vsi_open(), >> for example requesting the MSI-X vectors, the error path only frees the >> ring memory and leaves the queues configured. Every following open then >> fails to add the same queues again and the netdev can not be brought up >> until the driver is reloaded: >> >> ice 0000:04:00.2: Failed to set LAN Tx queue context, error: -5 >> ice 0000:04:00.2 enp4s0f2np2: Failed to open VSI 0x0010 on switch 0x0002 >> >> Stop the Tx queues on the error paths that run after ice_vsi_cfg_lan(). >> The ice_up_complete() failure path already does it through ice_down(), >> so let it skip the new step. >> >> There is no i40e counterpart of this fix, i40e does not add its Tx >> queues through the admin queue. It was found while verifying the >> previous patch. With the IRQ unwind fixed the warnings were gone, but >> the netdev still could not be opened again after the forced >> request_irq failure. >> >> Fixes: cdedef59deb0 ("ice: Configure VSIs for Tx/Rx") >> Assisted-by: LLM > Which one? It was Claude Code with Claude Fable 5.1 Thanks for the review. Petr > > Otherwise... > > Reviewed-by: Ivan Vecera > > >> Signed-off-by: Petr Oros >> --- >> drivers/net/ethernet/intel/ice/ice_main.c | 10 ++++++++-- >> 1 file changed, 8 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/net/ethernet/intel/ice/ice_main.c b/drivers/net/ethernet/intel/ice/ice_main.c >> index d246cde36ae726..e62a8f544345a3 100644 >> --- a/drivers/net/ethernet/intel/ice/ice_main.c >> +++ b/drivers/net/ethernet/intel/ice/ice_main.c >> @@ -7561,13 +7561,13 @@ int ice_vsi_open(struct ice_vsi *vsi) >> >> err = ice_vsi_cfg_lan(vsi); >> if (err) >> - goto err_setup_rx; >> + goto err_stop_tx; >> >> snprintf(int_name, sizeof(int_name) - 1, "%s-%s", >> dev_driver_string(ice_pf_to_dev(pf)), vsi->netdev->name); >> err = ice_vsi_req_irq_msix(vsi, int_name); >> if (err) >> - goto err_setup_rx; >> + goto err_stop_tx; >> >> if (bitmap_empty(pf->txtime_txqs, pf->max_pf_txqs)) >> ice_vsi_cfg_netdev_tc(vsi, vsi->tc_cfg.ena_tc); >> @@ -7593,8 +7593,14 @@ int ice_vsi_open(struct ice_vsi *vsi) >> >> err_up_complete: >> ice_down(vsi); >> + ice_vsi_free_irq(vsi); >> + goto err_setup_rx; >> err_set_qs: >> ice_vsi_free_irq(vsi); >> +err_stop_tx: >> + ice_vsi_stop_lan_tx_rings(vsi, ICE_NO_RESET, 0); >> + if (vsi->xdp_rings) >> + ice_vsi_stop_xdp_tx_rings(vsi); >> err_setup_rx: >> ice_vsi_free_rx_rings(vsi); >> err_setup_tx: