From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fabio Fantoni Subject: Re: [Xen-devel] [PATCH v3 2/4] GlobalProperty: Display warning about unused -global Date: Sat, 19 Apr 2014 12:56:16 +0200 Message-ID: <53525650.9030201@m2r.biz> References: <1395705336-22528-1-git-send-email-dslutz@verizon.com> <1395705336-22528-3-git-send-email-dslutz@verizon.com> <53514312.5060705@suse.de> <53514BFC.700@suse.de> <535158BF.1030703@m2r.biz> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <535158BF.1030703@m2r.biz> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org To: =?ISO-8859-1?Q?Andreas_F=E4rber?= Cc: xen-devel , Stefano Stabellini , "Michael S. Tsirkin" , "qemu-devel@nongnu.org" , Don Slutz , Anthony Liguori List-Id: xen-devel@lists.xenproject.org Il 18/04/2014 18:54, Fabio Fantoni ha scritto: > > Il 18/04/2014 17:59, Andreas Färber ha scritto: >> Am 18.04.2014 17:36, schrieb Fabio Fantoni: >>> 2014-04-18 17:21 GMT+02:00 Andreas Färber >> >: >>> >>> Hi Don, >>> >>> Am 25.03.2014 00 :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. > > Thanks for your reply. > Can you tell me if also -device cirrus-vga,vram_size_mb=N is correct > and working? I probably found the correct values settable: in in hw/display/cirrus_vga.c: > 2988 static Property pci_vga_cirrus_properties[] = { > 2989 DEFINE_PROP_UINT32("vgamem_mb", struct PCICirrusVGAState, > 2990 cirrus_vga.vga.vram_size_mb, 8), > 2991 DEFINE_PROP_END_OF_LIST(), > 2992 }; Than I "-device cirrus-vga,vgamem_mb=N", not show errors and should be correct, right? in hw/display/vga-pci.c: > 182 static Property vga_pci_properties[] = { > 183 DEFINE_PROP_UINT32("vgamem_mb", PCIVGAState, vga.vram_size_mb, > 16), > 184 DEFINE_PROP_BIT("mmio", PCIVGAState, flags, > PCI_VGA_FLAG_ENABLE_MMIO, true), > 185 DEFINE_PROP_END_OF_LIST(), > 186 }; I tried time ago the videoram setting of stdvga on xen but seems was not worked (no error but performance remain bad on medium/large resolution, trying kvm with same parameters is better), I not know if is vgabios problem or low level xen changes are needed. The mmio option seems new to me, what it is in practice, may need to disable it in xen? Thanks for any reply and sorry for my bad english. > > Thanks for any reply. >> >> Regards, >> Andreas >> >> P.S. Please remember to use text format mails. >> >