From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f199.google.com (mail-pg1-f199.google.com [209.85.215.199]) (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 7A7443624B3 for ; Thu, 8 Oct 2026 17:33:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.199 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791480840; cv=none; b=f27uzErkBR8uVDd4Tb0KS8dI0IV1bsG6zVJgNdTQS2o2hf57ccv5c9V5+DKBjVo+40naI7VdemNoXCzuEbZrANY+S505MCRkNbud1b/6OqaiFuK4cw2q2ZQOajonI8SELHV4Axxax3lySzYkHgwVMYufLrI3uBrWxaETG37fTyI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791480840; c=relaxed/simple; bh=2Dy8NcG3RR4epT6g2CrsOvKssEwIyXFc/uFqGfl3ubA=; h=Date:Mime-Version:Message-ID:Subject:From:To:Cc:Content-Type; b=BJdgu3tffeRVHcftGAjNT1yxMrdkqEnYp4M8mLKZTvoWRW6jladubi7+vMIY5xM1TFwI7gCgEFjFipyryFypful1I15BLGnTpLnImmW5WjQa/0PM/Knc32vbL3xmgRhFYPMgpvuR5wx6DWINn3sC8lM2iHx1B9vZCkx59r88PZ0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--brianvv.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=rZY2PYog; arc=none smtp.client-ip=209.85.215.199 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--brianvv.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="rZY2PYog" Received: by mail-pg1-f199.google.com with SMTP id 41be03b00d2f7-cc42a07d04aso3757207a12.1 for ; Thu, 08 Oct 2026 10:33:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1791480839; x=1792085639; darn=vger.kernel.org; h=content-type:cc:to:from:subject:message-id:mime-version:date:from :to:cc:subject:date:message-id:reply-to:content-type; bh=oL7DYN1cPQUBG7i6wjoO+oNk4/EoYeE3qdEtA4etZEc=; b=rZY2PYogpgv0L42I7CI2m23+b7SvhhlaJDJMcYgxBrKHEVK8zOVH7jIvXTBDGBCUgO qx4X+qNzGzt5DZyzt28SuII+hDHEk/kgRuK8QytyBQghqgA0rbZ0XMwjE8VXjlUuNoxE z3V5ky/Z8Uq718clWzyIf4wxDL9676KoZ1XPHJBEZy6ba6gmv5fDmcSjMLfP9KeYEqdW u2qP1i1/Z3ScCjuNdCX0xqnz3ybcn7KrFuGP54j6s3QUCr0Ngz6TKG2T1aLQGIq6CE0W isW6bVeq2XirfsvsTB3hLNpeoQtVonGfFmPnt3moy/DS9Cl0otH7f2sj4RXP08KBRpIW 7V3A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1791480839; x=1792085639; h=content-type:cc:to:from:subject:message-id:mime-version:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=oL7DYN1cPQUBG7i6wjoO+oNk4/EoYeE3qdEtA4etZEc=; b=IbE28El3Bh3tck2kcVz85XIMELfSmIa1kY1xaCFe5Utr4/S+u1JDcef+Ua1JBU7V1V UL1bw5Ln82kSTD6/hluKbtccUmsi1TG3LfJiRQy/+H0gbZTGlMA3dhYkqYqI9f3U0dtU 81p5Bh72bM8pmwsOInTrR8RHFld8Kg1ODEhhz5gjDTO4TGKZAMKG8oKUvT8zNNaMBV3Z NX7l/aRS5L0GbRV0QTxr1S0/5lj5fn+6VtO6op6O/Hd4RkQsBAANbIv0oPwA7vY1SIFn TmEYUkILHmM0T4myGv2Vw/zMyb3s756kqDzTRISLcyHAzEie+91qc23jxaPlI30cm5lx q64g== X-Forwarded-Encrypted: i=1; AKwUvBzvc4L8rGUbpTOCwBeZWMYHjC71FAthDzL0laRyPtsVO/0Zxq6dShx3TgdO59+hFhiUXUk2rTo=@vger.kernel.org X-Gm-Message-State: AFq9FYJbyc5akU7GbjwD4WaEk5gPvwEJnrX9TPSHOiriij5u9iz4J34Z HiegXPh9Wwy0oxRhSbqf6ojcXuLhBBGauNAXWqvWLPGV8B5HjdBty5hlbH2or5RccbpcfsqTdVf geaD/9YhAqg== X-Received: from plrs9.prod.google.com ([2002:a17:902:b189:b0:2e2:f2a4:dd99]) (user=brianvv job=prod-delivery.src-stubby-dispatcher) by 2002:a17:902:d983:b0:2e5:4604:c375 with SMTP id d9443c01a7336-2e60056530amr54905285ad.48.1791480837419; Thu, 08 Oct 2026 10:33:57 -0700 (PDT) Date: Thu, 8 Oct 2026 17:33:34 +0000 Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 X-Mailer: git-send-email 2.56.0.385.gd3acb90ef8-goog Message-ID: <20261008173334.2881475-1-brianvv@google.com> Subject: [iwl-net PATCH] idpf: wait for the reset completed state in idpf_check_reset_complete() From: Brian Vazquez To: Brian Vazquez , Tony Nguyen , Przemek Kitszel , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , intel-wired-lan@lists.osuosl.org, netdev@vger.kernel.org Cc: Madhu Chittim , Pavan Kumar Linga , Joshua Hay , Shailendra Bhatnagar , Sridhar Samudrala , linux-kernel@vger.kernel.org, David Decotigny , Li Li , Emil Tantilov , Brian Vazquez Content-Type: text/plain; charset="UTF-8" PFGEN_RSTAT[1:0] / VFGEN_RSTAT[1:0] (PFR_STATE / VFR_STATE) encode the function reset progress as a 2-bit state, not a flag: 00b - reset in progress 01b - reset completed, written by the control plane 10b - function active, written by the control plane once the function has completed VIRTCHNL2_OP_VERSION 11b - reserved idpf_check_reset_complete() tests (reg_val & rstat_m) as a boolean, with rstat_m = GENMASK(1, 0), so any non-zero state is taken as "completed", although only 01b carries that meaning. That only works if hardware moves the field to 00b as soon as the reset is triggered. On the devices tested it does not: PFSWR transitions PFR_STATE to 00b only when the field currently reads 01b. When the function is active (10b) the field keeps its value until the control plane finishes the reset and writes 01b, which takes a few hundred milliseconds. Consequently, on a hard reset of an active function (IDPF_HR_FUNC_RESET) the first poll sees 10b, returns immediately, the driver re-creates the mailbox and sends VIRTCHNL2_OP_VERSION while the reset is still being processed. The control plane tears the mailbox down underneath it, VERSION is never answered, and the driver sits in a 60 second virtchnl timeout (-ETIME) before going through a second hard reset which then succeeds. Make idpf_check_reset_complete() wait for the state to be exactly 01b. This is the stricter reading of the register definition and it does not depend on how the field got to its current value: functions that are still in progress (00b) or active (10b) keep polling until the control plane reports completion. Fixes: 8077c727561a ("idpf: add controlq init and reset checks") Signed-off-by: Brian Vazquez --- drivers/net/ethernet/intel/idpf/idpf.h | 2 ++ drivers/net/ethernet/intel/idpf/idpf_lib.c | 3 ++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/net/ethernet/intel/idpf/idpf.h b/drivers/net/ethernet/intel/idpf/idpf.h index 470bc23c844c..ce8a1bd0ec8b 100644 --- a/drivers/net/ethernet/intel/idpf/idpf.h +++ b/drivers/net/ethernet/intel/idpf/idpf.h @@ -166,6 +166,8 @@ struct idpf_netdev_priv { spinlock_t stats_lock; }; +#define IDPF_RSTAT_COMPLETE 0x01 + /** * struct idpf_reset_reg - Reset register offsets/masks * @rstat: Reset status register diff --git a/drivers/net/ethernet/intel/idpf/idpf_lib.c b/drivers/net/ethernet/intel/idpf/idpf_lib.c index 2c148377540c..a827e0fa67c8 100644 --- a/drivers/net/ethernet/intel/idpf/idpf_lib.c +++ b/drivers/net/ethernet/intel/idpf/idpf_lib.c @@ -1870,7 +1870,8 @@ static int idpf_check_reset_complete(struct idpf_adapter *adapter, * register for us yet and 0xFFFFFFFF is not a valid value for * the register, so treat that as invalid. */ - if (reg_val != 0xFFFFFFFF && (reg_val & reset_reg->rstat_m)) + if (reg_val != 0xFFFFFFFF && + (reg_val & reset_reg->rstat_m) == IDPF_RSTAT_COMPLETE) return 0; usleep_range(5000, 10000); -- 2.56.0.385.gd3acb90ef8-goog