From nobody Fri Dec 19 20:14:08 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 B2B1C184424; Sun, 24 Mar 2024 23:51:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711324273; cv=none; b=GMq28/S3g/q0QArHqZpDjoTaUdywWnA2YGyR5Z3l/yEaDcS7CCWtx8zxFLIbyKrA5QchLs9T37CasmaPvh2t1QssnuA7QZ8fMPucDGOVyzlM9iEBrMaF/0/c4sCgGPVWTzWmd4hnAbnt5FrOQkrlRzl1mX3Y11fWCdJKIJAGRYI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711324273; c=relaxed/simple; bh=YUlZ7Qh9XrKOvLc5vbPfNIAQyjzjHe192nRk8wGzRjA=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ZY3HyNywwQR5G0rJyHKFle8Uu5EEmmvMP2FVET47BwT/bfRuey3ph/wQ7khLALHadWU8Km/IzfcNFToaC01wM46ODaj6nAsA67VMjIKjVDCdVY9IlC9i2NELi3BRc+uvMYvEyfV1Nlm3VIvfe4FV2ZCFwB/fgjvF6lVwzCRQaaE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=FXzGKkq0; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="FXzGKkq0" Received: by smtp.kernel.org (Postfix) with ESMTPSA id BCFD3C433C7; Sun, 24 Mar 2024 23:51:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1711324272; bh=YUlZ7Qh9XrKOvLc5vbPfNIAQyjzjHe192nRk8wGzRjA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=FXzGKkq0pyHIrxfCW21wXs9gbEJ0WcHiFi/n9q12j8J/zHm/S3haVFOvIcPNWDuvF Cp1c0GL2D97SB1oh2Uu2g9LAzOswFDQMUhHaTCKEhmwp+S5bGF+APjJHs8XMHz6J+B qHPjndDTzMDQFNaimgfqRAyhsJKPqrBjLUbhZ5Lt5FALEaEq8CB8yPjZHtVnpR4xt2 R2+CvGnr1LfKqLqHIBAoK4meC191HceLLqmzNezmV9JQo3kFgozI8X/a5g3GwL1COT qjasGA9Mgg2+rMTdSnK1iM5k7RNiDj2rwdICCHeRgNsyOLApvSFQ28K376VFHRkaAo 8eB/kxgLlK0sg== From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Vinicius Costa Gomes , Richard Cochran , Pucha Himasekhar Reddy , Tony Nguyen , Sasha Levin Subject: [PATCH 4.19 051/148] igb: Fix missing time sync events Date: Sun, 24 Mar 2024 19:48:35 -0400 Message-ID: <20240324235012.1356413-52-sashal@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240324235012.1356413-1-sashal@kernel.org> References: <20240324235012.1356413-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Vinicius Costa Gomes [ Upstream commit ee14cc9ea19ba9678177e2224a9c58cce5937c73 ] Fix "double" clearing of interrupts, which can cause external events or timestamps to be missed. The E1000_TSIRC Time Sync Interrupt Cause register can be cleared in two ways, by either reading it or by writing '1' into the specific cause bit. This is documented in section 8.16.1. The following flow was used: 1. read E1000_TSIRC into 'tsicr'; 2. handle the interrupts present into 'tsirc' and mark them in 'ack'; 3. write 'ack' into E1000_TSICR; As both (1) and (3) will clear the interrupt cause, if the same interrupt happens again between (1) and (3) it will be ignored, causing events to be missed. Remove the extra clear in (3). Fixes: 00c65578b47b ("igb: enable internal PPS for the i210") Acked-by: Richard Cochran Signed-off-by: Vinicius Costa Gomes Tested-by: Pucha Himasekhar Reddy (A Co= ntingent worker at Intel) Signed-off-by: Tony Nguyen Signed-off-by: Sasha Levin --- drivers/net/ethernet/intel/igb/igb_main.c | 23 +++++------------------ 1 file changed, 5 insertions(+), 18 deletions(-) diff --git a/drivers/net/ethernet/intel/igb/igb_main.c b/drivers/net/ethern= et/intel/igb/igb_main.c index c0e2323e50c6e..5d8d5915bc276 100644 --- a/drivers/net/ethernet/intel/igb/igb_main.c +++ b/drivers/net/ethernet/intel/igb/igb_main.c @@ -6555,44 +6555,31 @@ static void igb_extts(struct igb_adapter *adapter, = int tsintr_tt) static void igb_tsync_interrupt(struct igb_adapter *adapter) { struct e1000_hw *hw =3D &adapter->hw; - u32 ack =3D 0, tsicr =3D rd32(E1000_TSICR); + u32 tsicr =3D rd32(E1000_TSICR); struct ptp_clock_event event; =20 if (tsicr & TSINTR_SYS_WRAP) { event.type =3D PTP_CLOCK_PPS; if (adapter->ptp_caps.pps) ptp_clock_event(adapter->ptp_clock, &event); - ack |=3D TSINTR_SYS_WRAP; } =20 if (tsicr & E1000_TSICR_TXTS) { /* retrieve hardware timestamp */ schedule_work(&adapter->ptp_tx_work); - ack |=3D E1000_TSICR_TXTS; } =20 - if (tsicr & TSINTR_TT0) { + if (tsicr & TSINTR_TT0) igb_perout(adapter, 0); - ack |=3D TSINTR_TT0; - } =20 - if (tsicr & TSINTR_TT1) { + if (tsicr & TSINTR_TT1) igb_perout(adapter, 1); - ack |=3D TSINTR_TT1; - } =20 - if (tsicr & TSINTR_AUTT0) { + if (tsicr & TSINTR_AUTT0) igb_extts(adapter, 0); - ack |=3D TSINTR_AUTT0; - } =20 - if (tsicr & TSINTR_AUTT1) { + if (tsicr & TSINTR_AUTT1) igb_extts(adapter, 1); - ack |=3D TSINTR_AUTT1; - } - - /* acknowledge the interrupts */ - wr32(E1000_TSICR, ack); } =20 static irqreturn_t igb_msix_other(int irq, void *data) --=20 2.43.0