diff mbox

[1.0] disable automatic loading of sgabios when -nographic

Message ID 1320332422-2457-1-git-send-email-pbonzini@redhat.com
State New
Headers show

Commit Message

Paolo Bonzini Nov. 3, 2011, 3 p.m. UTC
sgabios hasn't gotten a lot of coverage since it was not shipped.  For 1.0,
let's disable the automatic loading of the option ROM in -nographic
mode.  We can put it back for 1.1.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
	Requested by Anthony on IRC.

 hw/pc.c |    9 ---------
 1 files changed, 0 insertions(+), 9 deletions(-)

Comments

Anthony Liguori Nov. 8, 2011, 5:23 p.m. UTC | #1
On 11/03/2011 10:00 AM, Paolo Bonzini wrote:
> sgabios hasn't gotten a lot of coverage since it was not shipped.  For 1.0,
> let's disable the automatic loading of the option ROM in -nographic
> mode.  We can put it back for 1.1.
>
> Signed-off-by: Paolo Bonzini<pbonzini@redhat.com>

Applied.  Thanks.

Regards,

Anthony Liguori

> ---
> 	Requested by Anthony on IRC.
>
>   hw/pc.c |    9 ---------
>   1 files changed, 0 insertions(+), 9 deletions(-)
>
> diff --git a/hw/pc.c b/hw/pc.c
> index eb4c2d8..1024465 100644
> --- a/hw/pc.c
> +++ b/hw/pc.c
> @@ -1104,15 +1104,6 @@ void pc_vga_init(PCIBus *pci_bus)
>               isa_vga_init();
>           }
>       }
> -
> -    /*
> -     * sga does not suppress normal vga output. So a machine can have both a
> -     * vga card and sga manually enabled. Output will be seen on both.
> -     * For nographic case, sga is enabled at all times
> -     */
> -    if (display_type == DT_NOGRAPHIC) {
> -        isa_create_simple("sga");
> -    }
>   }
>
>   static void cpu_request_exit(void *opaque, int irq, int level)
Paolo Bonzini Dec. 2, 2011, 4:33 p.m. UTC | #2
On 11/08/2011 06:23 PM, Anthony Liguori wrote:
> On 11/03/2011 10:00 AM, Paolo Bonzini wrote:
>> sgabios hasn't gotten a lot of coverage since it was not shipped. For
>> 1.0,
>> let's disable the automatic loading of the option ROM in -nographic
>> mode. We can put it back for 1.1.
>>
>> Signed-off-by: Paolo Bonzini<pbonzini@redhat.com>
>
> Applied. Thanks.

Can you please revert it for 1.1?  Thanks!

Paolo

>> ---
>> Requested by Anthony on IRC.
>>
>> hw/pc.c | 9 ---------
>> 1 files changed, 0 insertions(+), 9 deletions(-)
>>
>> diff --git a/hw/pc.c b/hw/pc.c
>> index eb4c2d8..1024465 100644
>> --- a/hw/pc.c
>> +++ b/hw/pc.c
>> @@ -1104,15 +1104,6 @@ void pc_vga_init(PCIBus *pci_bus)
>> isa_vga_init();
>> }
>> }
>> -
>> - /*
>> - * sga does not suppress normal vga output. So a machine can have both a
>> - * vga card and sga manually enabled. Output will be seen on both.
>> - * For nographic case, sga is enabled at all times
>> - */
>> - if (display_type == DT_NOGRAPHIC) {
>> - isa_create_simple("sga");
>> - }
>> }
>>
>> static void cpu_request_exit(void *opaque, int irq, int level)
>
>
>
Anthony Liguori Dec. 2, 2011, 4:34 p.m. UTC | #3
On 12/02/2011 10:33 AM, Paolo Bonzini wrote:
> On 11/08/2011 06:23 PM, Anthony Liguori wrote:
>> On 11/03/2011 10:00 AM, Paolo Bonzini wrote:
>>> sgabios hasn't gotten a lot of coverage since it was not shipped. For
>>> 1.0,
>>> let's disable the automatic loading of the option ROM in -nographic
>>> mode. We can put it back for 1.1.
>>>
>>> Signed-off-by: Paolo Bonzini<pbonzini@redhat.com>
>>
>> Applied. Thanks.
>
> Can you please revert it for 1.1? Thanks!

Can you send a patch?

Regards,

Anthony Liguori

>
> Paolo
>
>>> ---
>>> Requested by Anthony on IRC.
>>>
>>> hw/pc.c | 9 ---------
>>> 1 files changed, 0 insertions(+), 9 deletions(-)
>>>
>>> diff --git a/hw/pc.c b/hw/pc.c
>>> index eb4c2d8..1024465 100644
>>> --- a/hw/pc.c
>>> +++ b/hw/pc.c
>>> @@ -1104,15 +1104,6 @@ void pc_vga_init(PCIBus *pci_bus)
>>> isa_vga_init();
>>> }
>>> }
>>> -
>>> - /*
>>> - * sga does not suppress normal vga output. So a machine can have both a
>>> - * vga card and sga manually enabled. Output will be seen on both.
>>> - * For nographic case, sga is enabled at all times
>>> - */
>>> - if (display_type == DT_NOGRAPHIC) {
>>> - isa_create_simple("sga");
>>> - }
>>> }
>>>
>>> static void cpu_request_exit(void *opaque, int irq, int level)
>>
>>
>>
>
diff mbox

Patch

diff --git a/hw/pc.c b/hw/pc.c
index eb4c2d8..1024465 100644
--- a/hw/pc.c
+++ b/hw/pc.c
@@ -1104,15 +1104,6 @@  void pc_vga_init(PCIBus *pci_bus)
             isa_vga_init();
         }
     }
-
-    /*
-     * sga does not suppress normal vga output. So a machine can have both a
-     * vga card and sga manually enabled. Output will be seen on both.
-     * For nographic case, sga is enabled at all times
-     */
-    if (display_type == DT_NOGRAPHIC) {
-        isa_create_simple("sga");
-    }
 }
 
 static void cpu_request_exit(void *opaque, int irq, int level)