From cbaee52287e5f32373181cff50a00b6c4ac9015a Mon Sep 17 00:00:00 2001 From: Herbie Robinson Date: Fri, 10 Mar 2017 10:54:55 -0500 Subject: [PATCH] SGABIOS: fix wrong video attrs for int 10h, ah==13h Fix Bug #1670509: wrong video attrs for int 10h, ah==13h The subroutine set_cursor_position is trashing %bx. Unfortunately, %bl contains the video attribute for write_string. The fix saves %bx in the function prolog and restores it in the epilog. Signed-off-by: Herbie Robinson Message-Id: <1489161295-23864-1-git-send-email-Herbie.Robinson@stratus.com> Signed-off-by: Paolo Bonzini --- sgabios.S | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sgabios.S b/sgabios.S index 275d063..96c4e23 100644 --- a/sgabios.S +++ b/sgabios.S @@ -1728,6 +1728,7 @@ write_string_tail: set_cursor_position: pushw %ax + pushw %bx pushw %ds pushw $BDA_SEG popw %ds /* ds = 0x40 */ @@ -1738,6 +1739,7 @@ set_cursor_position: movw %ax, %bx /* bx = cursor save offset */ movw %dx, (%bx) /* save new cursor value */ popw %ds + popw %bx popw %ax ret -- 2.11.0