[edk2,v2,0/1] ArmPkg/TimerDxe: Add ISB for timer compare value reload

Message ID	1520901090-96694-1-git-send-email-heyi.guo@linaro.org
Headers	show Delivered-To: patch@linaro.org Received-SPF: pass (google.com: best guess record for domain of edk2-devel-bounces@lists.01.org designates 198.145.21.10 as permitted sender) client-ip=198.145.21.10; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:400e:c01::236; helo=mail-pl0-x236.google.com; envelope-from=heyi.guo@linaro.org; receiver=edk2-devel@lists.01.org From: Heyi Guo <heyi.guo@linaro.org> To: edk2-devel@lists.01.org Date: Tue, 13 Mar 2018 08:31:29 +0800 Message-Id: <1520901090-96694-1-git-send-email-heyi.guo@linaro.org> Subject: [edk2] [PATCH v2 0/1] ArmPkg/TimerDxe: Add ISB for timer compare value reload Precedence: list Cc: Marc Zyngier <marc.zyngier@arm.com>, Heyi Guo <heyi.guo@linaro.org>, Leif Lindholm <leif.lindholm@linaro.org>, Ard Biesheuvel <ard.biesheuvel@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" <edk2-devel-bounces@lists.01.org>
Series	ArmPkg/TimerDxe: Add ISB for timer compare value reload \| expand [edk2,v2,0/1] ArmPkg/TimerDxe: Add ISB for timer compare value reload [edk2,v2,1/1] ArmPkg/TimerDxe: Add ISB for timer compare value reload

Message ID

1520901090-96694-1-git-send-email-heyi.guo@linaro.org

Headers

Received-SPF: pass (google.com: best guess record for domain of
	edk2-devel-bounces@lists.01.org designates 198.145.21.10 as
	permitted sender) client-ip=198.145.21.10; 
Received-SPF: Pass (sender SPF authorized) identity=mailfrom;
	client-ip=2607:f8b0:400e:c01::236; helo=mail-pl0-x236.google.com;
	envelope-from=heyi.guo@linaro.org; receiver=edk2-devel@lists.01.org 
From: Heyi Guo <heyi.guo@linaro.org>
To: edk2-devel@lists.01.org
Date: Tue, 13 Mar 2018 08:31:29 +0800
Message-Id: <1520901090-96694-1-git-send-email-heyi.guo@linaro.org>
Subject: [edk2] [PATCH v2 0/1] ArmPkg/TimerDxe: Add ISB for timer compare
	value reload
Precedence: list
Cc: Marc Zyngier <marc.zyngier@arm.com>, Heyi Guo <heyi.guo@linaro.org>,
	Leif Lindholm <leif.lindholm@linaro.org>,
	Ard Biesheuvel <ard.biesheuvel@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: edk2-devel-bounces@lists.01.org
Sender: "edk2-devel" <edk2-devel-bounces@lists.01.org>

Series

ArmPkg/TimerDxe: Add ISB for timer compare value reload | expand

Message

gary guo March 13, 2018, 12:31 a.m. UTC

After rebasing to edk2 commit 5e3719a, we found D05 would hang after printing a
lot of "Spurious interrupt" messages. The issue would gone away if we restored
the removal of "enable interrupt source":
  gInterrupt->EnableInterruptSource (gInterrupt, Source);

It can also be fixed if we add a "ISB" after reloading timer compare value, and
we agree that it makes sense to do that.

Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Marc Zyngier <marc.zyngier@arm.com>

Heyi Guo (1):
  ArmPkg/TimerDxe: Add ISB for timer compare value reload

 ArmPkg/Drivers/TimerDxe/TimerDxe.c | 1 +
 1 file changed, 1 insertion(+)

-- 
2.7.4

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel