All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: Fabio Fantoni <fabio.fantoni@m2r.biz>
Cc: xen-devel <xen-devel@lists.xensource.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Don Slutz <dslutz@verizon.com>,
	Anthony Liguori <aliguori@amazon.com>
Subject: Re: [Xen-devel] [PATCH v3 2/4] GlobalProperty: Display warning about unused -global
Date: Fri, 18 Apr 2014 17:59:56 +0200	[thread overview]
Message-ID: <53514BFC.700@suse.de> (raw)
In-Reply-To: <CABMPFzieCkRcgbB8V9TifEhiakauLg7U9DZ34JyNG5pAqD=_Zw@mail.gmail.com>

Am 18.04.2014 17:36, schrieb Fabio Fantoni:
> 2014-04-18 17:21 GMT+02:00 Andreas Färber <afaerber@suse.de
> <mailto:afaerber@suse.de>>:
> 
>     Hi Don,
> 
>     Am 25.03.2014 00 <tel:25.03.2014%2000>:55, schrieb Don Slutz:
>     > This can help a user understand why -global was ignored.
>     >
>     > For example: with "-vga cirrus"; "-global vga.vgamem_mb=16" is just
>     > ignored when "-global cirrus-vga.vgamem_mb=16" is not.
>     >
>     > This is currently clear when the wrong property is provided:
>     >
>     > out/x86_64-softmmu/qemu-system-x86_64 -global
>     cirrus-vga.vram_size_mb=16 -monitor pty -vga cirrus
>     > char device redirected to /dev/pts/20 (label compat_monitor0)
>     > qemu-system-x86_64: Property '.vram_size_mb' not found
>     > Aborted (core dumped)
>     >
>     > vs
>     >
>     > out/x86_64-softmmu/qemu-system-x86_64 -global vga.vram_size_mb=16
>     -monitor pty -vga cirrus
>     > char device redirected to /dev/pts/20 (label compat_monitor0)
>     > VNC server running on `::1:5900'
>     > ^Cqemu: terminating on signal 2
> 
> 
> I added the cirrus video memory setting in libxl time ago (using -global
> vga.vram_size_mb), testing it with qemu 1.3 and also on qemu 1.6 when I
> changed from -vga to -device if I remember good.
> Has been changed in recent versions or something was not right even
> though it seemed right to me?

There are multiple graphics cards to choose from. When using -vga std or
-device vga, then -global vga.foo=bar gets used; if -vga cirrus or
-device cirrus-vga then it needs to be -global cirrus-vga.foo=bar and
any -global vga.foo=bar gets ignored - unless you manage to add it as
secondary (PCI) graphics card.

Regards,
Andreas

P.S. Please remember to use text format mails.

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

  reply	other threads:[~2014-04-18 15:59 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-24 23:55 [Qemu-devel] [PATCH v3 0/4] Add max-ram-below-4g (was Add pci_hole_min_size machine option) Don Slutz
2014-03-24 23:55 ` Don Slutz
2014-03-24 23:55 ` [Qemu-devel] [PATCH v3 1/4] xen-all: Fix xen_hvm_init() to adjust pc memory layout Don Slutz
2014-03-24 23:55   ` Don Slutz
2014-04-18 14:23   ` Andreas Färber
2014-04-22 16:29     ` Don Slutz
2014-03-24 23:55 ` [Qemu-devel] [PATCH v3 2/4] GlobalProperty: Display warning about unused -global Don Slutz
2014-03-24 23:55   ` Don Slutz
2014-04-18 15:21   ` Andreas Färber
2014-04-18 15:36     ` [Xen-devel] " Fabio Fantoni
2014-04-18 15:59       ` Andreas Färber [this message]
2014-04-18 16:54         ` Fabio Fantoni
2014-04-19 10:56           ` Fabio Fantoni
2014-04-22 18:44             ` Don Slutz
2014-04-19 20:54     ` Paolo Bonzini
2014-04-22 23:13       ` Don Slutz
2014-04-23  0:28         ` Paolo Bonzini
2014-04-23 12:58           ` Don Slutz
2014-04-23 13:33         ` Andreas Färber
2014-04-22 20:23     ` Don Slutz
2014-04-23  0:28       ` Paolo Bonzini
2014-04-23 13:25         ` Don Slutz
2014-03-24 23:55 ` [Qemu-devel] [PATCH v3 3/4] pc & q35: Add new object pc-memory-layout Don Slutz
2014-03-24 23:55   ` Don Slutz
2014-04-18 15:45   ` Andreas Färber
2014-04-22 23:54     ` Don Slutz
2014-04-21 12:27   ` Paolo Bonzini
2014-04-23  0:13     ` Don Slutz
2014-04-23  3:27       ` Paolo Bonzini
2014-04-23  6:51       ` Marcel Apfelbaum
2014-03-24 23:55 ` [Qemu-devel] [PATCH v3 4/4] xen-all: Pass max_ram_below_4g to xen_hvm_init Don Slutz
2014-03-24 23:55   ` Don Slutz
2014-03-25 11:17   ` [Qemu-devel] " Stefano Stabellini
2014-03-25 11:17     ` Stefano Stabellini
2014-04-18 16:19   ` Andreas Färber
2014-04-22 18:27     ` Don Slutz
2014-03-25  9:08 ` [Qemu-devel] [PATCH v3 0/4] Add max-ram-below-4g (was Add pci_hole_min_size machine option) Michael S. Tsirkin
2014-03-25  9:08   ` Michael S. Tsirkin
2014-04-17 18:27   ` PING " Don Slutz
2014-04-18 14:10   ` Andreas Färber
2014-04-22 16:31     ` Don Slutz
2014-05-05  8:12       ` [Qemu-devel] " Michael S. Tsirkin
2014-05-05 16:16         ` Don Slutz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53514BFC.700@suse.de \
    --to=afaerber@suse.de \
    --cc=aliguori@amazon.com \
    --cc=dslutz@verizon.com \
    --cc=fabio.fantoni@m2r.biz \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.