diff mbox series

arm/traps.c: Adjust HPFAR_EL2 representation

Message ID 1566923614-12503-1-git-send-email-andrii.anisov@gmail.com (mailing list archive)
State New, archived
Headers show
Series arm/traps.c: Adjust HPFAR_EL2 representation | expand

Commit Message

Andrii Anisov Aug. 27, 2019, 4:33 p.m. UTC 
From: Andrii Anisov <andrii_anisov@epam.com>

In the commit af156ff0859c8d362a5706640614c9d10f62adf2, it
was left unattended HPFAR_EL2 register output. Now it is printed
with 1608 digits, what is way too wide even for the biggest
monitors. So cleanup excessive paddings.

Signed-off-by: Andrii Anisov <andrii_anisov@epam.com>
---
 xen/arch/arm/traps.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Julien Grall Aug. 27, 2019, 5:18 p.m. UTC | #1 
Hi,

On 27/08/2019 18:33, Andrii Anisov wrote:
> From: Andrii Anisov <andrii_anisov@epam.com>
> 
> In the commit af156ff0859c8d362a5706640614c9d10f62adf2, it
> was left unattended HPFAR_EL2 register output. Now it is printed
> with 1608 digits, what is way too wide even for the biggest
> monitors. So cleanup excessive paddings.

A more complete patch (fix another place) has already been sent on the 
mailing list (see [1]). It is waiting on Stefano's ack at the moment...

Cheers,

[1] 
https://lists.xenproject.org/archives/html/xen-devel/2019-08/msg01439.html
Stefano Stabellini Aug. 27, 2019, 8:16 p.m. UTC | #2 
On Tue, 27 Aug 2019, Julien Grall wrote:
> Hi,
> 
> On 27/08/2019 18:33, Andrii Anisov wrote:
> > From: Andrii Anisov <andrii_anisov@epam.com>
> > 
> > In the commit af156ff0859c8d362a5706640614c9d10f62adf2, it
> > was left unattended HPFAR_EL2 register output. Now it is printed
> > with 1608 digits, what is way too wide even for the biggest
> > monitors. So cleanup excessive paddings.
> 
> A more complete patch (fix another place) has already been sent on the 
> mailing list (see [1]). It is waiting on Stefano's ack at the moment...
> 
> Cheers,
> 
> [1] 
> https://lists.xenproject.org/archives/html/xen-devel/2019-08/msg01439.html

The patch is in staging now.
Andrii Anisov Aug. 28, 2019, 7:49 a.m. UTC | #3 
On 27.08.19 20:18, Julien Grall wrote:

> A more complete patch (fix another place) has already been sent on the
> mailing list (see [1]). It is waiting on Stefano's ack at the moment...
> 
> Cheers,
> 
> [1]
> https://lists.xenproject.org/archives/html/xen-devel/2019-08/msg01439.html

Ah, yes. Missed it.
diff mbox series

Patch

diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c
index f013297..addccb9 100644
--- a/xen/arch/arm/traps.c
+++ b/xen/arch/arm/traps.c
@@ -938,7 +938,7 @@  static void _show_registers(const struct cpu_user_regs *regs,
     printk(" TTBR0_EL2: %016"PRIx64"\n", READ_SYSREG64(TTBR0_EL2));
     printk("\n");
     printk("   ESR_EL2: %08"PRIx32"\n", regs->hsr);
-    printk(" HPFAR_EL2: %016"PRIregister"\n", READ_SYSREG(HPFAR_EL2));
+    printk(" HPFAR_EL2: %"PRIregister"\n", READ_SYSREG(HPFAR_EL2));
 
 #ifdef CONFIG_ARM_32
     printk("     HDFAR: %08"PRIx32"\n", READ_CP32(HDFAR));