[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 05/11] Delete write only variables
From: |
Blue Swirl |
Subject: |
[Qemu-devel] Re: [PATCH 05/11] Delete write only variables |
Date: |
Fri, 8 Oct 2010 18:06:43 +0000 |
On Fri, Oct 8, 2010 at 5:30 PM, Alexander Graf <address@hidden> wrote:
>
> On 08.10.2010, at 19:28, Paolo Bonzini wrote:
>
>> On 10/07/2010 07:24 PM, Blue Swirl wrote:
>>>> > On 10/06/2010 11:32 PM, Blue Swirl wrote:
>>>>> >>
>>>>> >> @@ -91,8 +90,6 @@ static int vga_osi_call (CPUState *env)
>>>>> >> env->gpr[3] = 0;
>>>>> >> break;
>>>>> >> case 39: /* video_ctrl */
>>>>> >> - if (env->gpr[6] == 0 || env->gpr[6] == 1)
>>>>> >> - vga_vbl_enabled = env->gpr[6];
>>>>> >> env->gpr[3] = 0;
>>>>> >> break;
>>>>> >> case 47:
>>>> >
>>>> > NACK, this smells like an incomplete emulation.
>>> Why would we care about unfinished temporary Mac-on-Linux hack for
>>> video.x? Perhaps the correct fix is to remove vga_osi_call function
>>> and env->osi_call field entirely.
>>
>> CCing the person who could answer.
>
> I wanted to write the same thing, but canceled the mail because I wasn't sure
> if we need it. We do use video.x from MOL, don't we?
>
> Please just try to remove the whole thing altogether and see if graphics
> still work. If so, yeah, remove it :).
OpenBIOS boots in graphical mode, I can see the penguin during Linux
boot. This is with the attached patch that removes video.x completely.
0001-ppc-remove-video.x.patch
Description: application/mbox
- [Qemu-devel] [PATCH 05/11] Delete write only variables, Blue Swirl, 2010/10/06
- [Qemu-devel] Re: [PATCH 05/11] Delete write only variables, Paolo Bonzini, 2010/10/07
- [Qemu-devel] Re: [PATCH 05/11] Delete write only variables, Blue Swirl, 2010/10/07
- [Qemu-devel] Re: [PATCH 05/11] Delete write only variables, Paolo Bonzini, 2010/10/08
- [Qemu-devel] Re: [PATCH 05/11] Delete write only variables, Alexander Graf, 2010/10/08
- [Qemu-devel] Re: [PATCH 05/11] Delete write only variables,
Blue Swirl <=
- [Qemu-devel] Re: [PATCH 05/11] Delete write only variables, Alexander Graf, 2010/10/08
- Re: [Qemu-devel] Re: [PATCH 05/11] Delete write only variables, Andreas Färber, 2010/10/09
- Re: [Qemu-devel] Re: [PATCH 05/11] Delete write only variables, Alexander Graf, 2010/10/09