|
 |
357786 |
From 098bbd9ed847849834ff1f21766d23ea240c5bf0 Mon Sep 17 00:00:00 2001
|
|
|
9bac43 |
From: Andrew Jones <drjones@redhat.com>
|
|
|
9bac43 |
Date: Mon, 1 Aug 2016 14:27:09 +0200
|
|
|
9bac43 |
Subject: RHEL-only: hw/char/pl011: fix SBSA reset
|
|
|
9bac43 |
|
|
|
9bac43 |
RH-Author: Andrew Jones <drjones@redhat.com>
|
|
|
9bac43 |
Message-id: <1470061629-6395-1-git-send-email-drjones@redhat.com>
|
|
|
9bac43 |
Patchwork-id: 71697
|
|
|
9bac43 |
O-Subject: [AArch64 RHEL-7.3 qemu-kvm-rhev PATCH] RHEL-only: hw/char/pl011: fix SBSA reset
|
|
|
9bac43 |
Bugzilla: 1266048
|
|
|
9bac43 |
RH-Acked-by: Auger Eric <eric.auger@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Wei Huang <wei@redhat.com>
|
|
|
9bac43 |
|
|
|
9bac43 |
When booting Linux with an SBSA UART, e.g. when booting mach-virt
|
|
|
9bac43 |
with ACPI, if the user types on the console during boot, then when
|
|
|
9bac43 |
the login prompt appears she won't be able to log in. This is
|
|
|
9bac43 |
because during boot the SBSA UART needs to be reset, but the SBSA
|
|
|
9bac43 |
specification doesn't provide registers to enable/disable the FIFOs.
|
|
|
9bac43 |
This patch observes a couple registers the SBSA UART does write to
|
|
|
9bac43 |
in order to attempt to guess when a reset is needed, and then do it.
|
|
|
9bac43 |
We risk losing some characters from the FIFO if the guess is wrong,
|
|
|
9bac43 |
but the risk of that should be quite low.
|
|
|
9bac43 |
|
|
|
9bac43 |
Signed-off-by: Andrew Jones <drjones@redhat.com>
|
|
|
9bac43 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9bac43 |
(cherry picked from commit 15ee295534f654d6b6ba9499cdd380aa9c954920)
|
|
|
357786 |
(cherry picked from commit 49be481336c227fdad2f7edc02fa088f3d88c9a2)
|
|
|
357786 |
(cherry picked from commit 9fcede24f35378e2c9113440a692d2c96cc94865)
|
|
|
357786 |
(cherry picked from commit b721e7ff2bdd42b0a786b8630c1ba8b1d3560da3)
|
|
|
357786 |
(cherry picked from commit 88f3eb70f909b03ed18074aadd12f32f28ad8437)
|
|
|
9bac43 |
---
|
|
|
9bac43 |
hw/char/pl011.c | 12 ++++++++++++
|
|
|
9bac43 |
1 file changed, 12 insertions(+)
|
|
|
9bac43 |
|
|
|
9bac43 |
diff --git a/hw/char/pl011.c b/hw/char/pl011.c
|
|
|
9bac43 |
index 2aa277f..23fe047 100644
|
|
|
9bac43 |
--- a/hw/char/pl011.c
|
|
|
9bac43 |
+++ b/hw/char/pl011.c
|
|
|
9bac43 |
@@ -209,6 +209,18 @@ static void pl011_write(void *opaque, hwaddr offset,
|
|
|
9bac43 |
pl011_update(s);
|
|
|
9bac43 |
break;
|
|
|
9bac43 |
case 17: /* UARTICR */
|
|
|
9bac43 |
+ /*
|
|
|
9bac43 |
+ * RHEL-only, fixes BZ1266048
|
|
|
9bac43 |
+ *
|
|
|
9bac43 |
+ * Look for the "signature" of a driver init or shutdown in
|
|
|
9bac43 |
+ * order to know that we need to reset the SBSA UART. Yes,
|
|
|
9bac43 |
+ * this is hacky, but as SBSA drivers aren't required to write
|
|
|
9bac43 |
+ * UARTLCR_H or UARTCR, then we don't have much choice...
|
|
|
9bac43 |
+ */
|
|
|
9bac43 |
+ if (s->int_enabled == 0 && value == 0xffff) {
|
|
|
9bac43 |
+ s->read_count = 0;
|
|
|
9bac43 |
+ s->read_pos = 0;
|
|
|
9bac43 |
+ }
|
|
|
9bac43 |
s->int_level &= ~value;
|
|
|
9bac43 |
pl011_update(s);
|
|
|
9bac43 |
break;
|
|
|
9bac43 |
--
|
|
|
9bac43 |
1.8.3.1
|
|
|
9bac43 |
|