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.129.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 ECC1C3EBF02 for ; Sat, 3 Oct 2026 09:56:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.129.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791021368; cv=none; b=f3CGHNN1oNRpkmWgxYRN/vGGW1Ac4aXb3qlhT7kMzTld/jBngkAbOf4vb9BYXUse8n75TPD59GrN8rlb3tvJ6OkC3PbvkJVsz2tcPiIZphqeXYFJgKgt2twFwPZ8wenXDu0RkBJGZwcbXKKLO2ZMEjwulhDesiD7fclkNVPgNRI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791021368; c=relaxed/simple; bh=NYHNZxq7Oq04CRUEjnX4HIYuiGcCOaN9Z/rNxN3m02g=; h=Date:From:To:CC:Subject:In-Reply-To:References:Message-ID: MIME-Version:Content-Type; b=Z9NB1V04uT34MBvMpr3K4y4JAbJVMISWnbpiXszV60pmrEoePQc4EnIfy8r4S5MsxxkzKpbDl5xtFgyOipRmPfNMFz1UA3Qqc6SyMdz3y25KEYtWU+XAyo337HClJU79j1wlK0S1u73BAVaN4f8g0CpMDGk+uFJ0/VQ/hmpx1nM= 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=O6ruDXy5; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b=E4BmtAYW; arc=none smtp.client-ip=170.10.129.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="O6ruDXy5"; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b="E4BmtAYW" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1791021366; 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=YqGVpdSP2zcszRi0/wYGQr7kElxokpmPd3Y6IXpKB/A=; b=O6ruDXy5UcLYopPTSdWX73Si6dFAhNjLxNBclWNom07KX7B/tCke9Si56Fy5yiXJfXg2xl 36JvERbnYwJuXz/CcsnGltkGQ0GZ8Fs2r2SgNRJwdOrcDNxJyBeV9f6sj9AHm2cW7KGRRe 3vqRO8Z00Dwitvb0Z4O6/r72wuVT+/E= Received: from mail-wr1-f72.google.com (mail-wr1-f72.google.com [209.85.221.72]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-301-03b5sN0MPryksNIrtEUAJQ-1; Sat, 03 Oct 2026 05:56:03 -0400 X-MC-Unique: 03b5sN0MPryksNIrtEUAJQ-1 X-Mimecast-MFC-AGG-ID: 03b5sN0MPryksNIrtEUAJQ_1791021362 Received: by mail-wr1-f72.google.com with SMTP id ffacd0b85a97d-48b09f01f56so159525f8f.1 for ; Sat, 03 Oct 2026 02:56:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=google; t=1791021362; x=1791626162; darn=vger.kernel.org; h=content-transfer-encoding:content-type:mime-version:message-id :references:in-reply-to:user-agent:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to:content-type; bh=YqGVpdSP2zcszRi0/wYGQr7kElxokpmPd3Y6IXpKB/A=; b=E4BmtAYWR60DvHyyqiORFCVZTsV3qg+ZOq89CXIBOd0FHEhKD2s+I3KgdOCxPiAA8+ KyUM5bOOLP+9MFoE4QEXO/70jVUlV/tAcVIRJbEuFE26tZypuPsgs+q6b4MvPan+ANTx xOEfWaXUO9L0sE+bH1Iszj1qdzCPf6HupdA1rVAPqSCX4IVHgdm852msfdhuV0A0D6vy 3ixiJnHTnhTsSBqvHu8SN8NeaAlBT8YQuFAbr4ZpEb1byX4cqXnVPOMEz4gJUGXOO7+i jLYG9SJiw6Fe8Ne8rFbNOsv2KDHgCzjwtCwfhkoFbehGfJihN3oSQpv6IpzPVbm+mBlG mpZw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1791021362; x=1791626162; h=content-transfer-encoding:content-type:mime-version:message-id :references:in-reply-to:user-agent:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=YqGVpdSP2zcszRi0/wYGQr7kElxokpmPd3Y6IXpKB/A=; b=SoSZQClbuaReryzruJ8M5Rka47etcumY208Kmn407OQlbfH+CvgiN7RERcwtpTPtBm wGuMtCDgaj6JicCq1X0OOwFNpEWSdnsCMuGwI2F5E2yG366/4YrnKCMq25BPcy95kFLD /g5K6Ls1Sb0zg8MhQU9Jq2k/XXQchmDhor0bEpHI/uLzsQ49eT8gUflwYYNP5h5BVeTS mWFBdtwt88i6hnEU3qoFarfFs/xbEf1aB6NVmRuuNL9YRd+k4Lh6JxOZA7O+xXtV8hdM Lx4dmmPDJrxmQ3n7ASsLTwdOnF/VxFIBOrWgaVaOExrw4Ka8hIDLLn4BwADrbrdCi1l/ Ikhw== X-Forwarded-Encrypted: i=1; AKwUvByDvf7lchVN3SdFz6O8E5jNs+oT5N3J6Wdr5O/KDatvr+mJlRbQZOijvTnrR862ErQEakqwcJo=@vger.kernel.org X-Gm-Message-State: AFq9FYKqHNHaD+UVfuALxm/EDuiWk+MZwsEdXEEZC+T/WykJ3i13X+3Z 2zEJFYnWiLh/4bLF+Pvny0Scm0C/EwK7jcMqm2PYsUNHbhw2u2B6IyFYcJ9wju3E8Pl4t0IgJe+ E+0nNaj+ixyL+P9lWACxM0nSaE9R7cGAETS2kusTZbUdyn5Lk2SVHEkyC2w== X-Gm-Gg: AYBFou0valR0YVGv5otvWEk7Djr/u56uJ0F5hlfHSi+73U4l14JmgclBNj/lV1P87TO 4hXPcdLpQPTXch+wOs0DV7MdHYWC9z78yfnCy2tOG20D5oMquZrma/Huj1vinFOlKUUO0RkfA01 GT7Rk+gQ2yoIPWjmawQcK8CF9OJvvZhkrc3gzoDHtM6zDSfGNaioxi/awOjodv324pIaTIfx8z/ 32oL7HDPg+H0qIJdfSL1GlUVWzh6mTqJQJu9EM3hTvvtOvH/op9eiOIqG/hG5Ca+uk0XBQfeWlt X1TVg7cRLUUvDh1qXI2dUSslzoms4LX/JSOzyTdH9c6q21CjvrMpRQ46QL3KvrVvWwTNK3ZOL1I A90TDwabGatohH9+ivZaXjxrM7Xak2TGZammVzf4= X-Received: by 2002:adf:f108:0:b0:48b:e50:715a with SMTP id ffacd0b85a97d-48b1273e612mr6550010f8f.34.1791021362482; Sat, 03 Oct 2026 02:56:02 -0700 (PDT) X-Received: by 2002:adf:f108:0:b0:48b:e50:715a with SMTP id ffacd0b85a97d-48b1273e612mr6549985f8f.34.1791021362095; Sat, 03 Oct 2026 02:56:02 -0700 (PDT) Received: from ehlo.thunderbird.net ([2a00:e580:bf11:1:2666:d874:79e6:3214]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-48b380f06b3sm11631234f8f.11.2026.10.03.02.56.01 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 03 Oct 2026 02:56:01 -0700 (PDT) Date: Sat, 03 Oct 2026 11:55:59 +0200 From: Ivan Vecera To: intel-wired-lan@osuosl.org, Petr Oros , 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 Subject: =?US-ASCII?Q?Re=3A_=5BPATCH_iwl-net_08/10=5D_ice=3A_dete?= =?US-ASCII?Q?ct_a_PF_reset_that_does_not_complete?= User-Agent: Thunderbird for Android In-Reply-To: <20261002130752.2597259-9-poros@redhat.com> References: <20261002130752.2597259-1-poros@redhat.com> <20261002130752.2597259-9-poros@redhat.com> Message-ID: <07E47138-2DB2-4824-A535-03CA9CD2F73A@redhat.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On October 2, 2026 3:07:50 PM GMT+02:00, Petr Oros wro= te: >ice_pf_reset() polls PFGEN_CTRL=2EPFSWR for up to >ICE_GLOBAL_CFG_LOCK_TIMEOUT + ICE_PF_RESET_WAIT_COUNT iterations, but >the timeout check after the loop still compares the counter with >ICE_PF_RESET_WAIT_COUNT alone=2E When the reset never completes the loop >ends with cnt =3D=3D 5300, the check does not match and the function >returns success with PFSWR still set=2E A reset that completes just when >the counter reaches ICE_PF_RESET_WAIT_COUNT is reported as a failure >instead=2E > >Check the PFSWR bit read last instead of the loop counter, the same >way i40e_pf_reset() does after its poll loop=2E > >Fixes: c9a12d6d2091 ("ice: Increase timeout after PFR") >Assisted-by: LLM >Signed-off-by: Petr Oros >--- > drivers/net/ethernet/intel/ice/ice_common=2Ec | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/drivers/net/ethernet/intel/ice/ice_common=2Ec b/drivers/net/= ethernet/intel/ice/ice_common=2Ec >index 04633103e3e610=2E=2E0ffd6b5b81239e 100644 >--- a/drivers/net/ethernet/intel/ice/ice_common=2Ec >+++ b/drivers/net/ethernet/intel/ice/ice_common=2Ec >@@ -1291,7 +1291,7 @@ static int ice_pf_reset(struct ice_hw *hw) > mdelay(1); > } >=20 >- if (cnt =3D=3D ICE_PF_RESET_WAIT_COUNT) { >+ if (reg & PFGEN_CTRL_PFSWR_M) { > ice_debug(hw, ICE_DBG_INIT, "PF reset polling failed to complete=2E\n"= ); > return -EIO; > } Reviewed-by: Ivan Vecera