summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* make: Remove duplicate use of GLIB_CFLAGSStefan Weil2012-02-095-8/+0
| | | | | | | | | | | Makefile, Makefile.hw, Makefile.target and libcacard/Makefile added GLIB_CFLAGS to QEMU_CFLAGS. Makefile.objs does this, too, and is included by all other Makefiles, so GLIB_CFLAGS were added twice (reported by malc). Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: malc <av1474@comtv.ru>
* ARM devboards: Set arm_sysctl properties before init, not afterPeter Maydell2012-02-093-3/+3
| | | | | | | | | The ARM devboard models (vexpress-a9, realview, versatilepb, etc) were accidentally trying to set one of the arm_sysctl properties after device init. This has now become a fatal error; set the property before device init where it should be done instead. Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
* Merge remote-tracking branch 'bonzini/qdev-props-for-anthony' into stagingAnthony Liguori2012-02-0814-404/+823
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * bonzini/qdev-props-for-anthony: (25 commits) qdev: remove unused fields from PropertyInfo qdev: initialize properties via QOM qdev: inline qdev_prop_set into qdev_prop_set_ptr qdev: access properties via QOM qdev: fix off-by-one qdev: let QOM free properties qdev: remove parse/print methods for pointer properties qdev: make the non-legacy pci address property accept an integer qdev: remove parse/print methods for mac properties qdev: remove print/parse methods from LostTickPolicy properties qdev: remove parse method for string properties qdev: allow reusing get/set for legacy property qdev: remove direct calls to print/parse qom: add property get/set wrappers for links qom: fix canonical paths vs. interfaces qom: use object_resolve_path_type for links qom: add object_resolve_path_type qom: fix off-by-one qom: add property get/set wrappers for C types qom: add QObject-based property get/set wrappers ...
| * qdev: remove unused fields from PropertyInfoPaolo Bonzini2012-02-073-60/+1
| | | | | | | | | | Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: initialize properties via QOMPaolo Bonzini2012-02-073-21/+23
| | | | | | | | | | | | | | | | | | Similarly, use the object properties also to set the default values of the qdev properties. This requires reordering registration and initialization. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: inline qdev_prop_set into qdev_prop_set_ptrPaolo Bonzini2012-02-071-19/+7
| | | | | | | | | | | | | | | | qdev_prop_set is not needed anymore except for hacks, simplify it and inline it. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: access properties via QOMPaolo Bonzini2012-02-073-28/+59
| | | | | | | | | | | | | | | | Do not poke anymore in the struct when accessing qdev properties. Instead, ask the object to set the right value. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: fix off-by-onePaolo Bonzini2012-02-071-3/+3
| | | | | | | | | | | | | | Integer properties did not work. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: let QOM free propertiesPaolo Bonzini2012-02-073-15/+14
| | | | | | | | | | | | | | | | Drop the special free callback. Instead, register a "regular" release method in the non-legacy property. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: remove parse/print methods for pointer propertiesPaolo Bonzini2012-02-071-56/+72
| | | | | | | | | | | | | | | | | | Pointer properties (except for PROP_PTR of course) should not need a legacy counterpart. In the future, relative paths will ensure that QEMU will support the same syntax as now for drives etc.. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: make the non-legacy pci address property accept an integerPaolo Bonzini2012-02-071-19/+9
| | | | | | | | | | | | | | | | | | PCI addresses are set with qdev_prop_uint32. Thus we make the QOM property accept a device and function encoded in an 8-bit integer, instead of the magic dd.f hex string. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: remove parse/print methods for mac propertiesPaolo Bonzini2012-02-071-19/+42
| | | | | | | | | | Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: remove print/parse methods from LostTickPolicy propertiesPaolo Bonzini2012-02-073-31/+33
| | | | | | | | | | | | | | | | Also generalize the code so that we can have more enum properties in the future. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: remove parse method for string propertiesPaolo Bonzini2012-02-071-11/+0
| | | | | | | | | | | | | | We need the print method to put double quotes, but parsing is not special. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: allow reusing get/set for legacy propertyPaolo Bonzini2012-02-071-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | In some cases, a legacy property does need a special print method but not a special parse method. In this case, we can reuse the get/set from the static (non-legacy) property. If neither parse nor print is needed, though, do not register the legacy property at all. The previous patch ensures that the right fallback will be used. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qdev: remove direct calls to print/parsePaolo Bonzini2012-02-073-29/+36
| | | | | | | | | | | | | | | | | | | | | | | | There's no need to call into ->parse and ->print manually. The QOM legacy properties do that for us. Furthermore, in some cases legacy and static properties have exactly the same behavior, and we could drop the legacy properties right away. Add an appropriate fallback to prepare for this. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: add property get/set wrappers for linksPaolo Bonzini2012-02-072-0/+48
| | | | | | | | | | | | | | | | These can set a link to any object, as long as it is included in the composition tree. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: fix canonical paths vs. interfacesPaolo Bonzini2012-02-071-0/+10
| | | | | | | | | | Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: use object_resolve_path_type for linksPaolo Bonzini2012-02-073-14/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows to restrict partial matches to objects of the expected type. It will let people use bare names to reference drives even though their name might be the same as a device's (e.g. -drive id=hd0,if=none,... -device ...,drive=hd0,id=hd0). As a useful byproduct, this fixes a problem with links of interface type. When a link property's type is an interface, the code expects the implementation object (not the parent object) to be stored in the variable. The parent object does not contain the right vtable. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: add object_resolve_path_typePaolo Bonzini2012-02-072-10/+41
| | | | | | | | | | Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: fix off-by-onePaolo Bonzini2012-02-071-5/+2
| | | | | | | | Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: add property get/set wrappers for C typesPaolo Bonzini2012-02-072-9/+180
| | | | | | | | | | | | | | Add wrappers that let you get/set properties using normal C data types. Reviewed-by: Anthony Liguori <anthony@aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: add QObject-based property get/set wrappersPaolo Bonzini2012-02-074-16/+90
| | | | | | | | | | | | | | | | | | | | | | | | Move the creation of QmpInputVisitor and QmpOutputVisitor from qmp.c to qom/object.c, since it's the only practical way to access object properties. Keep this isolated such that it's easy to remove. At some point, we need to remove all usage of QObject in the tree and replace it with GVariant. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: do not include qdev header filePaolo Bonzini2012-02-071-2/+0
| | | | | | | | | | Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: avoid useless conversions from string to typePaolo Bonzini2012-02-071-15/+12
| | | | | | | | Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: clean up/optimize object_dynamic_castPaolo Bonzini2012-02-071-24/+38
| | | | | | | | | | | | | | | | | | | | The interface loop can be performed only on the parent object. It does not need to be done on each interface. Similarly, we can simplify the code by switching early from the implementation object to the parent object. Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: more documentation on subclassingPaolo Bonzini2012-02-071-3/+73
| | | | | | | | | | Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| * qom: clean up cast macrosPaolo Bonzini2012-02-071-7/+15
| | | | | | | | | | Reviewed-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
* | Merge remote-tracking branch 'aneesh/for-upstream' into stagingAnthony Liguori2012-02-088-34/+393
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | * aneesh/for-upstream: hw/9pfs: Remove O_NOATIME flag from 9pfs open() calls in readonly mode hw/9pfs: Update MAINTAINERS file fsdev: Fix parameter parsing for proxy helper hw/9pfs: Fix crash when mounting with synthfs hw/9pfs: Preserve S_ISGID hw/9pfs: Add new security model mapped-file.
| * | hw/9pfs: Remove O_NOATIME flag from 9pfs open() calls in readonly modeDaniel P. Berrange2012-01-301-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When 2c74c2cb4bedddbfa67628fbd5f9273b4e0e9903 added support for the 'readonly' flag against 9p filesystems, it also made QEMU add the O_NOATIME flag as a side-effect. The O_NOATIME flag, however, may only be set by the file owner, or a user with CAP_FOWNER capability. QEMU cannot assume that this is the case for filesytems exported to QEMU. eg, run QEMU as non-root, and attempt to pass the host OS filesystem through to the guest OS with readonly enable. The result is that the guest OS cannot open any files at all. If O_NOATIME is really required, it should be optionally enabled via a separate QEMU command line flag. * hw/9pfs/virtio-9p.c: Remove O_NOATIME Acked-by: M. Mohan Kumar <mohan@in.ibm.com> Signed-off-by: Daniel P. Berrange <berrange@redhat.com> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
| * | hw/9pfs: Update MAINTAINERS fileAneesh Kumar K.V2012-01-301-2/+4
| | | | | | | | | | | | | | | | | | Acked-by: Venkateswararao Jujjuri <jvrao@linux.vnet.ibm.com> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
| * | fsdev: Fix parameter parsing for proxy helperM. Mohan Kumar2012-01-301-2/+8
| | | | | | | | | | | | | | | | | | | | | This fixes a crash when using sockfd with proxy FsDriver Signed-off-by: M. Mohan Kumar <mohan@in.ibm.com> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
| * | hw/9pfs: Fix crash when mounting with synthfsAneesh Kumar K.V2012-01-301-1/+1
| | | | | | | | | | | | | | | | | | | | | Some Fsdriver backend don't have fs_root. So check for that in migrate message. Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
| * | hw/9pfs: Preserve S_ISGIDM. Mohan Kumar2012-01-302-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In passthrough security model in local fs driver, after a file creation chown and chmod are done to set the file credentials and mode as requested by 9p client. But if there was a request to create a file with S_ISGID bit, doing chown on that file resets the S_ISGID bit. So first call chown and then invoking chmod with proper mode bit retains the S_ISGID (if present/requested) This resulted in LTP mknod02, mknod03, mknod05, open10 test case failures. This patch fixes this issue. man 2 chown When the owner or group of an executable file are changed by an unprivileged user the S_ISUID and S_ISGID mode bits are cleared. POSIX does not specify whether this also should happen when root does the chown(); the Linux behavior depends on the kernel version. Signed-off-by: M. Mohan Kumar <mohan@in.ibm.com> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
| * | hw/9pfs: Add new security model mapped-file.Aneesh Kumar K.V2012-01-195-25/+378
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This enable us to do passthrough equivalent security model on NFS directory. NFS server mostly do root squashing and don't support xattr. Hence we cannot use 'passthrough' or 'mapped' security model Also added "mapped-xattr" security to indicate earlier "mapped" security model Older name is still supported. POSIX rules regarding ctime update on chmod are not followed by this security model. Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
* | | Merge remote-tracking branch 'sweil/w32' into stagingAnthony Liguori2012-02-082-3/+22
|\ \ \ | | | | | | | | | | | | | | | | | | | | * sweil/w32: w32: Initialise critical section before starting thread (fix #922131) w32: Build windows and console executables
| * | | w32: Initialise critical section before starting thread (fix #922131)Stefan Weil2012-02-071-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch was contributed by Bogdan Harjoc. I added some assertions. Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Stefan Weil <sw@weilnetz.de>
| * | | w32: Build windows and console executablesStefan Weil2012-02-071-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | System emulation executables with SDL are typically windows executables. Sometimes console executables are more useful, so create both variants if linker option -mwindows was detected. v2: This version uses QEMU_PROGW / QEMU_PROG instead of QEMU_PROG / QEMU_PROGC. Signed-off-by: Stefan Weil <sw@weilnetz.de>
* | | | Restore consistent formattingmalc2012-02-076-129/+129
|/ / / | | | | | | | | | Signed-off-by: malc <av1474@comtv.ru>
* | | s390x: fix qom-ification fall-outAnthony Liguori2012-02-062-1/+6
| | | | | | | | | | | | | | | Tested-by: Andreas Faerber <afaerber@suse.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* | | KVM: Fix breakages of QOM conversionJan Kiszka2012-02-062-0/+2
| | | | | | | | | | | | | | | | | | | | | KVM APIC and PIC require instance sizes. Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* | | usb-redir: convert to QOMAnthony Liguori2012-02-061-10/+14
| | | | | | | | | | | | | | | | | | | | | | | | This was missed due to the fact that it's in the top level and it uses 'struct DeviceInfo' instead of 'DeviceInfo' for some strange reason. Tested-by: Benoît Canet <benoit.canet@gmail.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* | | Fix previous commitBlue Swirl2012-02-041-1/+1
| | | | | | | | | | | | Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | | usb-bsd: convert to QOMAnthony Liguori2012-02-041-5/+6
| | | | | | | | | | | | | | | | | | | | | Simple enough. Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | | vga: improve documentationBlue Swirl2012-02-041-0/+12
| | | | | | | | | | | | | | | | | | Add links to chipset docs and FreeVGA site. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | | vga: use constants from vga.hBlue Swirl2012-02-043-175/+198
| | | | | | | | | | | | Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | | vga.h: remove unused stuff and reformatBlue Swirl2012-02-041-425/+103
| | | | | | | | | | | | Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | | Add vga.h unmodified from LinuxBlue Swirl2012-02-041-0/+481
| | | | | | | | | | | | Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | | vga: move Cirrus VGA template to its own fileBlue Swirl2012-02-044-79/+111
| | | | | | | | | | | | | | | | | | | | | Standard VGA does not use vga_draw_cursor_line_* functions. Move the template to cirrus_vga_template.h. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | | memory: change dirty getting API to take a sizeBlue Swirl2012-02-049-49/+46
| | | | | | | | | | | | | | | | | | | | | Instead of each device knowing or guessing the guest page size, just pass the desired size of dirtied memory area. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
OpenPOWER on IntegriCloud