qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] qemu-doc: Clarify that -vga std is now the default
@ 2017-01-27  9:41 Alberto Garcia
  2017-02-07 10:13 ` Alberto Garcia
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Alberto Garcia @ 2017-01-27  9:41 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-trivial, Alberto Garcia

The QEMU manual page states that Cirrus Logic is the default video
card if the user doesn't specify any. However this is not true since
QEMU 2.2.

Signed-off-by: Alberto Garcia <berto@igalia.com>
---
 qemu-options.hx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/qemu-options.hx b/qemu-options.hx
index 80df52651a..66ee562821 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -1195,12 +1195,12 @@ Select type of VGA card to emulate. Valid values for @var{type} are
 Cirrus Logic GD5446 Video card. All Windows versions starting from
 Windows 95 should recognize and use this graphic card. For optimal
 performances, use 16 bit color depth in the guest and the host OS.
-(This one is the default)
+(This card was the default before QEMU 2.2)
 @item std
 Standard VGA card with Bochs VBE extensions.  If your guest OS
 supports the VESA 2.0 VBE extensions (e.g. Windows XP) and if you want
 to use high resolution modes (>= 1280x1024x16) then you should use
-this option.
+this option. (This card is the default since QEMU 2.2)
 @item vmware
 VMWare SVGA-II compatible adapter. Use it if you have sufficiently
 recent XFree86/XOrg server or Windows guest with a driver for this
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-doc: Clarify that -vga std is now the default
  2017-01-27  9:41 [Qemu-devel] [PATCH] qemu-doc: Clarify that -vga std is now the default Alberto Garcia
@ 2017-02-07 10:13 ` Alberto Garcia
  2017-02-07 15:11 ` Markus Armbruster
  2017-02-07 15:28 ` Paolo Bonzini
  2 siblings, 0 replies; 5+ messages in thread
From: Alberto Garcia @ 2017-02-07 10:13 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-trivial

On Fri, Jan 27, 2017 at 11:41:54AM +0200, Alberto Garcia wrote:
> The QEMU manual page states that Cirrus Logic is the default video
> card if the user doesn't specify any. However this is not true since
> QEMU 2.2.
> 
> Signed-off-by: Alberto Garcia <berto@igalia.com>

Ping

Berto

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-doc: Clarify that -vga std is now the default
  2017-01-27  9:41 [Qemu-devel] [PATCH] qemu-doc: Clarify that -vga std is now the default Alberto Garcia
  2017-02-07 10:13 ` Alberto Garcia
@ 2017-02-07 15:11 ` Markus Armbruster
  2017-02-07 15:16   ` Alberto Garcia
  2017-02-07 15:28 ` Paolo Bonzini
  2 siblings, 1 reply; 5+ messages in thread
From: Markus Armbruster @ 2017-02-07 15:11 UTC (permalink / raw)
  To: Alberto Garcia; +Cc: qemu-devel, qemu-trivial

Alberto Garcia <berto@igalia.com> writes:

> The QEMU manual page states that Cirrus Logic is the default video
> card if the user doesn't specify any. However this is not true since
> QEMU 2.2.
>
> Signed-off-by: Alberto Garcia <berto@igalia.com>
> ---
>  qemu-options.hx | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 80df52651a..66ee562821 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -1195,12 +1195,12 @@ Select type of VGA card to emulate. Valid values for @var{type} are
>  Cirrus Logic GD5446 Video card. All Windows versions starting from
>  Windows 95 should recognize and use this graphic card. For optimal
>  performances, use 16 bit color depth in the guest and the host OS.
> -(This one is the default)
> +(This card was the default before QEMU 2.2)
>  @item std
>  Standard VGA card with Bochs VBE extensions.  If your guest OS
>  supports the VESA 2.0 VBE extensions (e.g. Windows XP) and if you want
>  to use high resolution modes (>= 1280x1024x16) then you should use
> -this option.
> +this option. (This card is the default since QEMU 2.2)
>  @item vmware
>  VMWare SVGA-II compatible adapter. Use it if you have sufficiently
>  recent XFree86/XOrg server or Windows guest with a driver for this

Do we want to cover history in the user manual?  Or should we simply
move the "(This one is the default)" phrase?

Hmm, the default seems to depend on the machine type:

    if (default_vga) {
        if (machine_class->default_display) {
            vga_model = machine_class->default_display;
        } else if (vga_interface_available(VGA_CIRRUS)) {
            vga_model = "cirrus";
        } else if (vga_interface_available(VGA_STD)) {
            vga_model = "std";
        }
    }

The pc-*-2.2 and newer set ->default_display = "std".

Everything else keeps it null, and therefore gets "cirrus" if compiled
in, else "std".

The easy way out is to simply delete "(This one is the default)".

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-doc: Clarify that -vga std is now the default
  2017-02-07 15:11 ` Markus Armbruster
@ 2017-02-07 15:16   ` Alberto Garcia
  0 siblings, 0 replies; 5+ messages in thread
From: Alberto Garcia @ 2017-02-07 15:16 UTC (permalink / raw)
  To: Markus Armbruster; +Cc: qemu-devel, qemu-trivial

On Tue 07 Feb 2017 04:11:51 PM CET, Markus Armbruster wrote:
>>  Cirrus Logic GD5446 Video card. All Windows versions starting from
>>  Windows 95 should recognize and use this graphic card. For optimal
>>  performances, use 16 bit color depth in the guest and the host OS.
>> -(This one is the default)
>> +(This card was the default before QEMU 2.2)
>>  @item std
>>  Standard VGA card with Bochs VBE extensions.  If your guest OS
>>  supports the VESA 2.0 VBE extensions (e.g. Windows XP) and if you want
>>  to use high resolution modes (>= 1280x1024x16) then you should use
>> -this option.
>> +this option. (This card is the default since QEMU 2.2)
>>  @item vmware
>>  VMWare SVGA-II compatible adapter. Use it if you have sufficiently
>>  recent XFree86/XOrg server or Windows guest with a driver for this
>
> Do we want to cover history in the user manual?  Or should we simply
> move the "(This one is the default)" phrase?

> Hmm, the default seems to depend on the machine type:
>
>     if (default_vga) {
>         if (machine_class->default_display) {
>             vga_model = machine_class->default_display;
>         } else if (vga_interface_available(VGA_CIRRUS)) {
>             vga_model = "cirrus";
>         } else if (vga_interface_available(VGA_STD)) {
>             vga_model = "std";
>         }
>     }
>
> The pc-*-2.2 and newer set ->default_display = "std".

Right, if you set e.g. pc-i440fx-1.7 you'll get a cirrus card, that's
why I thought it was worth mentioning that older versions had a
different default.

But I'm fine either way to be honest.

Berto

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-doc: Clarify that -vga std is now the default
  2017-01-27  9:41 [Qemu-devel] [PATCH] qemu-doc: Clarify that -vga std is now the default Alberto Garcia
  2017-02-07 10:13 ` Alberto Garcia
  2017-02-07 15:11 ` Markus Armbruster
@ 2017-02-07 15:28 ` Paolo Bonzini
  2 siblings, 0 replies; 5+ messages in thread
From: Paolo Bonzini @ 2017-02-07 15:28 UTC (permalink / raw)
  To: Alberto Garcia, qemu-devel; +Cc: qemu-trivial



On 27/01/2017 10:41, Alberto Garcia wrote:
> The QEMU manual page states that Cirrus Logic is the default video
> card if the user doesn't specify any. However this is not true since
> QEMU 2.2.
> 
> Signed-off-by: Alberto Garcia <berto@igalia.com>
> ---
>  qemu-options.hx | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 80df52651a..66ee562821 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -1195,12 +1195,12 @@ Select type of VGA card to emulate. Valid values for @var{type} are
>  Cirrus Logic GD5446 Video card. All Windows versions starting from
>  Windows 95 should recognize and use this graphic card. For optimal
>  performances, use 16 bit color depth in the guest and the host OS.
> -(This one is the default)
> +(This card was the default before QEMU 2.2)
>  @item std
>  Standard VGA card with Bochs VBE extensions.  If your guest OS
>  supports the VESA 2.0 VBE extensions (e.g. Windows XP) and if you want
>  to use high resolution modes (>= 1280x1024x16) then you should use
> -this option.
> +this option. (This card is the default since QEMU 2.2)
>  @item vmware
>  VMWare SVGA-II compatible adapter. Use it if you have sufficiently
>  recent XFree86/XOrg server or Windows guest with a driver for this
> 

Queued, thanks.

Paolo

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-02-07 15:29 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-27  9:41 [Qemu-devel] [PATCH] qemu-doc: Clarify that -vga std is now the default Alberto Garcia
2017-02-07 10:13 ` Alberto Garcia
2017-02-07 15:11 ` Markus Armbruster
2017-02-07 15:16   ` Alberto Garcia
2017-02-07 15:28 ` Paolo Bonzini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).