[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3] SGABIOS: fix wrong video attrs for int 10h,
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH v3] SGABIOS: fix wrong video attrs for int 10h, ah==13h |
Date: |
Mon, 24 Apr 2017 14:08:56 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.0 |
On 23/04/2017 11:03, Michael Tokarev wrote:
> 10.03.2017 18:54, Herbie Robinson wrote:
>> 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 <address@hidden>
>> ---
>> Removed incorrect code deletions from set_current_cursor.
>> Removed explanation of same from the commit message.
>>
>> BTW, I didn't see any test cases for this code, but if they exist
>> and somebody points me at them, I should be able to generate a
>> very small qcow2 file that demonstrates the problem.
>>
>> sgabios.S | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/sgabios.S b/sgabios.S
>
> Now I'm not quite sure what to do with this.
>
> It is part of sgabios, which is not qemu code but a submodule.
>
> Maybe it's better to apply it upstream?
> Or should we apply it directly?
The original project is hosted by code.google.com, that has been closed
in 2015. I doesn't appear there is a real alive repository hosting this
project now (except some multiple clones on github).
So I think you should apply it directly (and regenerate sgabios.bin)
I cc Paolo as he has added the submodule.
Thanks,
Laurent