| abd8d4a | Anthony Liguori | 01 February 2013, 21:10:33 UTC | Update version for 1.4.0-rc0 Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 02 February 2013, 00:05:30 UTC |
| 02cd809 | Anthony Liguori | 02 February 2013, 00:02:50 UTC | tap: unbreak -netdev tap,fd=X The multiqueue patch series broke -netdev tap,fd=X which manifests as libvirt not being able to start a guest. This was because it passed NULL for the netdev name which results in an anonymous netdev device regardless of what the user specified. Cc: Jason Wang <jasowang@redhat.com> Cc: Bruce Rogers <brogers@suse.com> Reported-by: Bruce Rogers <brogers@suse.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 02 February 2013, 00:05:30 UTC |
| b76facc | Paolo Bonzini | 25 January 2013, 13:12:39 UTC | qom: remove object_delete This is now unused. Document the initial reference count of an object and when it will be freed/finalized. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:11 UTC |
| 5c09953 | Paolo Bonzini | 25 January 2013, 13:12:38 UTC | cpu: do not use object_delete CPUs are never added to the composition tree, so delete is achieved simply by removing the last references to them. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:11 UTC |
| b09995a | Paolo Bonzini | 25 January 2013, 13:12:37 UTC | qdev: drop extra references at creation time qdev_free and qbus_free have to do unparent+unref, because nobody else drops the initial reference (the one included by object_initialize) before them. For device_init_func and do_device_add, this is trivially correct, since the DeviceState goes out of scope. For qdev_create, qdev_try_create and qbus_init, it is a bit more tricky. What we are doing here is just assuming that the caller knows what it's doing, and won't call qdev_free/qbus_free while the device is still there. This is a pretty reasonable assumption and (behind the scenes) is also what GObject/GTK does. GTK actually has a "floating reference" that goes away as soon as the caller does gtk_container_add or something like that, but in the end qbus_init and qdev_try_create are already adding the new object to its qdev parent! So in the end the two solutions are the same. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:11 UTC |
| dc7389b | Paolo Bonzini | 25 January 2013, 13:12:36 UTC | qdev: inline object_delete into qbus_free/qdev_free We want object_delete to disappear, and we will do this one class at a time. Inline it for the qdev case, which we will tackle first. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:10 UTC |
| 62d7ba6 | Paolo Bonzini | 25 January 2013, 13:12:35 UTC | qdev: add reference for the bus while it is referred to by the DeviceState Now that the unparent callbacks are complete, we can correctly account more missing references. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:10 UTC |
| 06f7f2b | Paolo Bonzini | 25 January 2013, 13:12:34 UTC | qdev: move unrealization of devices from finalize to unparent Similarly, a bus holds a reference back to the device, and this will prevent the device from going away as soon as this reference is counted properly. To avoid this, move the unrealization of devices to the unparent callback. This includes recursively unparenting all the buses and (after the previous patch) the devices on those buses, which ensures that the web of references completely disappears for all devices that reside (in the qdev tree) below the one being unplugged. After this patch, the qdev tree and the bus<->child relationship is defined as "A is above B, iff unplugging A will automatically unplug B". Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:10 UTC |
| 6853d27 | Paolo Bonzini | 25 January 2013, 13:12:33 UTC | qdev: move deletion of children from finalize to unparent A device will never be finalized as long as it has a reference from other devices that sit on its buses. To ensure that the references go away, deassociate a bus from its children in the unparent callback for the bus. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:10 UTC |
| 9d12782 | Paolo Bonzini | 25 January 2013, 13:12:32 UTC | qdev: add reference count to a device for the BusChild Each device has a reference through the BusChild. This reference was not accounted for, add it now. Reviewed-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:10 UTC |
| 6c232d2 | Paolo Bonzini | 25 January 2013, 13:12:31 UTC | qom: document reference counting of link properties Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:10 UTC |
| 52e636c | Paolo Bonzini | 25 January 2013, 13:12:30 UTC | qom: preserve object while unparenting it Avoid that the object disappears after it's deleted from the QOM composition tree, in case that was the only reference to it. Acked-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:09 UTC |
| 4fec640 | Paolo Bonzini | 25 January 2013, 13:12:29 UTC | pci: use qbus_create in pci_bus_new Remove knowledge of QOM innards. The common part of pci_bus_new and pci_bus_new_inplace is moved to a new function pci_bus_init. Acked-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:09 UTC |
| 39355c3 | Paolo Bonzini | 25 January 2013, 13:12:28 UTC | qdev: change first argument of qbus_create_inplace to void * Make it clear that no BUS() macro is needed in the callers (in fact it wouldn't work because the object has not been initialized yet with the right class). Suggested-by: Andreas Faerber <afaerber@suse.de> Acked-by: Andreas F=E4rber <afaerber@suse.de> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:09 UTC |
| 013e118 | Paolo Bonzini | 25 January 2013, 13:12:27 UTC | qdev: remove duplication between qbus_create and qbus_create_inplace Move the common part to qbus_realize. Acked-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:53:09 UTC |
| d5286af | liguang | 24 January 2013, 05:03:27 UTC | accel: change {xen, kvm, tcg, qtest}_allowed from int to bool Signed-off-by: liguang <lig.fnst@cn.fujitsu.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:08:02 UTC |
| 217e21b | liguang | 24 January 2013, 05:03:26 UTC | vl: correct error message when fail to init kvm command: qemu-system-x86_64 -hda disk.img -smp 32 --enable-kvm error: Number of SMP cpus requested (32) exceeds max cpus supported by KVM (16) failed to initialize KVM: Invalid argument No accelerator found! well, it did find kvm, but failed to init, so message "No accelerator found!" is confusing, this commit remove the confusing error message. Signed-off-by: liguang <lig.fnst@cn.fujitsu.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:07:44 UTC |
| e3c66d9 | liguang | 24 January 2013, 05:03:25 UTC | vl: skip init accelerator if it's not available Signed-off-by: liguang <lig.fnst@cn.fujitsu.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 21:07:44 UTC |
| baeddde | Anthony Liguori | 29 January 2013, 21:42:45 UTC | sparc: disable qtest in make check We've seen this repeatedly in buildbot but I can now reliably reproduce it myself too. With a few hundred runs of 'make check', qemu-system-sparc will hang consuming 100% CPU. I've attached GDB to the hung process and unfortunately, I can't get anything useful out of GDB (RIP is not a valid simple and there is nothing else on the stack). At any rate, since this only manifests in qemu-system-sparc and it doesn't appear to be a qtest specific problem, I think we should disable it until the problem is resolved. Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 20:45:41 UTC |
| 77a5f4f | Anthony Liguori | 01 February 2013, 20:40:05 UTC | Merge remote-tracking branch 'stefanha/block' into staging # By Kevin Wolf (7) and others # Via Stefan Hajnoczi * stefanha/block: block/raw-posix: Build fix for O_ASYNC vmdk: Allow space in file name parallels: Fix bdrv_open() error handling dmg: Use g_free instead of free dmg: Fix bdrv_open() error handling vpc: Fix bdrv_open() error handling cloop: Fix bdrv_open() error handling bochs: Fix bdrv_open() error handling sheepdog: pass vdi_id to sheep daemon for sd_close() vmdk: Allow selecting SCSI adapter in image creation block: Adds mirroring tests for resized images block: Fix is_allocated_above with resized files qemu-iotests: Add regression test for b7ab0fea | 01 February 2013, 20:40:05 UTC |
| 3e3648b | Anthony Liguori | 01 February 2013, 18:55:56 UTC | xen: fix build problem introduced from per-queue peers Reported-by Andreas Färber <afaerber@suse.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 19:00:33 UTC |
| a9c87c5 | Jason Wang | 30 January 2013, 11:12:41 UTC | virtio-net: compat multiqueue support Disable multiqueue support for pre 1.4. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:03 UTC |
| 5f80080 | Jason Wang | 30 January 2013, 11:12:40 UTC | virtio-net: migration support for multiqueue This patch add migration support for multiqueue virtio-net. Instead of bumping the version, we conditionally send the info of multiqueue only when the device support more than one queue to maintain the backward compatibility. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:03 UTC |
| fed699f | Jason Wang | 30 January 2013, 11:12:39 UTC | virtio-net: multiqueue support This patch implements both userspace and vhost support for multiple queue virtio-net (VIRTIO_NET_F_MQ). This is done by introducing an array of VirtIONetQueue to VirtIONet. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:03 UTC |
| 0c87e93 | Jason Wang | 30 January 2013, 11:12:38 UTC | virtio-net: separate virtqueue from VirtIONet To support multiqueue virtio-net, the first step is to separate the virtqueue related fields from VirtIONet to a new structure VirtIONetQueue. The following patches will add an array of VirtIONetQueue to VirtIONet based on this patch. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:03 UTC |
| e78a2b4 | Jason Wang | 30 January 2013, 11:12:37 UTC | virtio: add a queue_index to VirtQueue Add a queue_index to VirtQueue and a helper to fetch it, this could be used by multiqueue supported device. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:03 UTC |
| f23fd81 | Jason Wang | 30 January 2013, 11:12:36 UTC | virtio: introduce virtio_del_queue() Some device (such as virtio-net) needs the ability to destroy or re-order the virtqueues, this patch adds a helper to do this. Signed-off-by: Jason Wang <jasowang> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:02 UTC |
| a9f98bb | Jason Wang | 30 January 2013, 11:12:35 UTC | vhost: multiqueue support This patch lets vhost support multiqueue. The idea is simple, just launching multiple threads of vhost and let each of vhost thread processing a subset of the virtqueues of the device. After this change each emulated device can have multiple vhost threads as its backend. To do this, a virtqueue index were introduced to record to first virtqueue that will be handled by this vhost_net device. Based on this and nvqs, vhost could calculate its relative index to setup vhost_net device. Since we may have many vhost/net devices for a virtio-net device. The setting of guest notifiers were moved out of the starting/stopping of a specific vhost thread. The vhost_net_{start|stop}() were renamed to vhost_net_{start|stop}_one(), and a new vhost_net_{start|stop}() were introduced to configure the guest notifiers and start/stop all vhost/vhost_net devices. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:02 UTC |
| 264986e | Jason Wang | 30 January 2013, 11:12:34 UTC | tap: multiqueue support Recently, linux support multiqueue tap which could let userspace call TUNSETIFF for a signle device many times to create multiple file descriptors as independent queues. User could also enable/disabe a specific queue through TUNSETQUEUE. The patch adds the generic infrastructure to create multiqueue taps. To achieve this a new parameter "queues" were introduced to specify how many queues were expected to be created for tap by qemu itself. Alternatively, management could also pass multiple pre-created tap file descriptors separated with ':' through a new parameter fds like -netdev tap,id=hn0,fds="X:Y:..:Z". Multiple vhost file descriptors could also be passed in this way. Each TAPState were still associated to a tap fd, which mean multiple TAPStates were created when user needs multiqueue taps. Since each TAPState contains one NetClientState, with the multiqueue nic support, an N peers of NetClientState were built up. A new parameter, mq_required were introduce in tap_open() to create multiqueue tap fds. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:02 UTC |
| e5dc0b4 | Jason Wang | 30 January 2013, 11:12:33 UTC | tap: introduce a helper to get the name of an interface This patch introduces a helper tap_get_ifname() to get the device name of tap device. This is needed when ifname is unspecified in the command line and qemu were asked to create tap device by itself. In this situation, the name were allocated by kernel, so if multiqueue is asked, we need to fetch its name after creating the first queue. Only linux has this support since it's the only platform that supports multiqueue tap. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:02 UTC |
| 16dbaf9 | Jason Wang | 30 January 2013, 11:12:32 UTC | tap: support enabling or disabling a queue This patch introduce a new bit - enabled in TAPState which tracks whether a specific queue/fd is enabled. The tap/fd is enabled during initialization and could be enabled/disabled by tap_enalbe() and tap_disable() which calls platform specific helpers to do the real work. Polling of a tap fd can only done when the tap was enabled. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:02 UTC |
| 94fdc6d | Jason Wang | 30 January 2013, 11:12:31 UTC | tap: add Linux multiqueue support This patch add basic multiqueue support for Linux. When multiqueue is needed, we will first check whether kernel support multiqueue tap before creating more queues. Two new functions tap_fd_enable() and tap_fd_disable() were introduced to enable and disable a specific queue. Since the multiqueue is only supported in Linux, return error on other platforms. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:01 UTC |
| 5193e5f | Jason Wang | 30 January 2013, 11:12:30 UTC | tap: factor out common tap initialization This patch factors out the common initialization of tap into a new helper net_init_tap_one(). This will be used by multiqueue tap patches. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:01 UTC |
| 4e4f9ae | Jason Wang | 30 January 2013, 11:12:29 UTC | tap: import linux multiqueue constants Import multiqueue constants from if_tun.h from 3.8-rc3. A new ifr flag IFF_MULTI_QUEUE were introduced to create a multiqueue backend by calling TUNSETIFF with the this flag and with the same interface name many times. A new ioctl TUNSETQUEUE were introduced. When doing this ioctl with IFF_DETACH_QUEUE, the queue were disabled in the linux kernel. When doing this ioctl with IFF_ATTACH_QUEUE, the queue were enabled in the linux kernel. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:01 UTC |
| 1ceef9f | Jason Wang | 30 January 2013, 11:12:28 UTC | net: multiqueue support This patch adds basic multiqueue support for qemu. The idea is simple, an array of NetClientStates were introduced in NICState, parse_netdev() were extended to find and match all NetClientStates belongs to the backend and place their pointers in NICConf. Then qemu_new_nic can setup a N:N mapping between NICStates that belongs to a nic and NICStates belongs to the netdev. And a queue_index were introduced in NetClientState to track its index. After this, each peers of a NICState were abstracted as a queue. After this change, all NetClientState that belongs to the same backend/nic has the same id. When use want to change the link status, all NetClientStates that belongs to the same backend/nic will be also changed. When user want to delete a device or netdev, all NetClientStates that belongs to the same backend/nic will be deleted also. Changing or deleting an specific queue is not allowed. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:01 UTC |
| f786045 | Jason Wang | 30 January 2013, 11:12:27 UTC | net: introduce NetClientState destructor To allow allocating an array of NetClientState and free it once, this patch introduces destructor of NetClientState. Which could do type specific free, which could be used by multiqueue to free the array once. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:01 UTC |
| 18a1541 | Jason Wang | 30 January 2013, 11:12:26 UTC | net: introduce qemu_net_client_setup() This patch separates the setup of NetClientState from its allocation, this will allow allocating an arrays of NetClientState and does the initialization one by one which is what multiqueue needs. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:01 UTC |
| 6c51ae7 | Jason Wang | 30 January 2013, 11:12:25 UTC | net: introduce qemu_find_net_clients_except() In multiqueue, all NetClientState that belongs to the same netdev or nic has the same id. So this patches introduces an helper qemu_find_net_clients_except() which finds all NetClientState with the same id. This will be used by multiqueue networking. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:01 UTC |
| 948ecf2 | Jason Wang | 30 January 2013, 11:12:24 UTC | net: intorduce qemu_del_nic() To support multiqueue nic, this patch separate the nic destructor from qemu_del_net_client() to a new helper qemu_del_nic() since the mapping bettween NiCState and NetClientState were not 1:1 in multiqueue. The following patches would refactor this function to support multiqueue nic. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:01 UTC |
| cc1f0f4 | Jason Wang | 30 January 2013, 11:12:23 UTC | net: introduce qemu_get_nic() To support multiqueue, this patch introduces a helper qemu_get_nic() to get NICState from a NetClientState. The following patches would refactor this helper to support multiqueue. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:03:00 UTC |
| b356f76 | Jason Wang | 30 January 2013, 11:12:22 UTC | net: introduce qemu_get_queue() To support multiqueue, the patch introduce a helper qemu_get_queue() which is used to get the NetClientState of a device. The following patches would refactor this helper to support multiqueue. Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 17:02:55 UTC |
| 28a6589 | Jason Wang | 30 January 2013, 11:12:21 UTC | net: tap: use abort() instead of assert(0) Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 16:50:59 UTC |
| ec45f08 | Jason Wang | 30 January 2013, 11:12:20 UTC | net: tap: using bool instead of bitfield Signed-off-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 16:50:59 UTC |
| 350ed2f | Andreas Färber | 31 January 2013, 13:49:44 UTC | target-ppc: Fix target_ulong vs. hwaddr format mismatches Since HWADDR_PRIx is always the same now, use %016 for TARGET_PPC64 and %08 for common code. This may slightly change the ppc64 debug output. Signed-off-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 16:43:41 UTC |
| 6bf0076 | Eduardo Habkost | 28 January 2013, 18:15:16 UTC | libqtest: Wait for the right child PID after killing QEMU When running "make check" with gcov enabled, we get the following message: hw/tmp105.gcda:cannot open data file, assuming not executed The problem happens because: * tmp105-test exits before QEMU exits, because waitpid() at qtest_quit() fails; * waitpid() fails because there's another process already waiting for the QEMU process; * The process that is already waiting for QEMU is the child created by qtest_init() to run system(); * qtest_quit() is incorrectly waiting for the QEMU PID directly instead of the child created by qtest_init(). This fixes the problem by sending SIGTERM to QEMU, but waiting for the child process created by qtest_init() (that exits immediately after QEMU exits). Reported-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 16:43:34 UTC |
| 3e7b8f4 | Andreas Färber | 20 January 2013, 17:56:18 UTC | isa: QOM'ify isa_bus_from_device() DeviceState::parent_bus is document as private and should be accessed through qdev_get_parent_bus(). Use a DEVICE() cast instead of accessing ISADevice's qdev field directly. Use ISA_BUS() in place of DO_UPCAST(). Signed-off-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 16:43:28 UTC |
| 8aae84a | Andreas Färber | 25 January 2013, 08:12:54 UTC | i2c: Drop I2C_SLAVE_FROM_QDEV() macro It is not being used in hot paths and is obsoleted by I2C_SLAVE() QOM cast macro. Clean it up using a scripted conversion, so that it doesn't get used in new code. Some of its callers were combining it with FROM_I2C_SLAVE() macro, which is equally obsolete but needs to be replaced in a type-specific way. Signed-off-by: Andreas Färber <afaerber@suse.de> Cc: Kuo-Jung Su <dantesu@faraday-tech.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 16:43:24 UTC |
| 6c8fec8 | Andreas Färber | 27 January 2013, 15:16:19 UTC | configure: Keep -Werror enabled for Release Candidates The automatic drop of -Werror during the RC phases has in the past led to warnings creeping into submaintainer trees. Last QEMU Summit it was concluded that -Werror should stay on and enabled only as part of the release process. To relieve our release manager, instead of always enabling -Werror or doing some number magic, let's enable it depending on whether a .git/ directory exists in the source tree. Signed-off-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 16:43:20 UTC |
| 70ddd9f | Michael Roth | 31 January 2013, 23:43:51 UTC | Revert "e1000: no need auto-negotiation if link was down" This reverts commit 84dd2120247a7d25ff1bb337de21c0e76816ad2d. I'm not sure what issue the original commit was meant to fix, or if the logic is actually wrong, but it causes e1000 to stop working after a guest issues a reset. >From what I can tell a guest with an e1000 nic has no way of changing the link status, as far as it's NetClient peer is concerned, except in the auto-negotiation path, so with this patch in place there's no recovery after a reset, since the link goes down and stays that way. Revert this patch now to fix the bigger problem, and handle any lingering issues with a follow-up. Reproduced/tested with qemu-jeos and Ubuntu 12.10. Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> | 01 February 2013, 16:43:13 UTC |
| 5dea9a6 | Anthony Liguori | 01 February 2013, 15:08:14 UTC | Merge remote-tracking branch 'quintela/migration.next' into staging # By Orit Wasserman # Via Juan Quintela * quintela/migration.next: Fix error message in migrate_set_capability HMP command Allow XBZRLE decoding without enabling the capability Fix example for query-migrate-capabilities Add XBZRLE testing Move XBZRLE encoding code to a separate file to allow testing | 01 February 2013, 15:08:14 UTC |
| e205842 | Anthony Liguori | 01 February 2013, 15:06:23 UTC | Merge remote-tracking branch 'agraf/s390-for-upstream' into staging # By Andreas Färber # Via Alexander Graf * agraf/s390-for-upstream: target-s390x: Pass S390CPU to s390_{add, del}_running_cpu() target-s390x: Clean up cpu_inject_*() signatures target-s390x: Fix debug output target-s390x: Fix debug output (continued) | 01 February 2013, 15:06:23 UTC |
| fabb604 | Anthony Liguori | 01 February 2013, 15:02:09 UTC | Merge remote-tracking branch 'agraf/ppc-for-upstream' into staging # By Andreas Färber (2) and Alexander Graf (1) # Via Alexander Graf * agraf/ppc-for-upstream: target-ppc: Fix build for PPC_DEBUG_DISAS target-ppc: Fix unused variable warning for FLUSH_ALL_TLBS PPC: Unify dcbzl code path | 01 February 2013, 15:02:09 UTC |
| fdf263f | Andreas Färber | 31 January 2013, 14:40:14 UTC | block/raw-posix: Build fix for O_ASYNC Commit eeb6b45d48800e96f67ef2a5c80332557fd45ddb (block: raw-posix image file reopen) broke the build on OpenIndiana. illumos has no O_ASYNC. Exclude it from flags to be compared and instead assert that it is not set where defined. Cf. e61ab1da7e98357da47c54d8f893b9bd6ff2f7f9 for qemu-ga. Cc: qemu-stable@nongnu.org (1.3.x) Cc: Jeff Cody <jcody@redhat.com> Suggested-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Andreas Färber <andreas.faerber@web.de> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 14:11:12 UTC |
| cd92347 | Philipp Hahn | 29 January 2013, 21:50:31 UTC | vmdk: Allow space in file name The previous scanf() format string stopped parsing the file name on the first white white space, which seems to be allowed at least by VMware Workstation. Change the format string to collect everything between the first and second quote as the file name, disallowing line breaks. Signed-off-by: Philipp Hahn <hahn@univention.de> Reviewed-by: Markus Armbruster <armbru@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:29 UTC |
| 4653623 | Kevin Wolf | 25 January 2013, 16:07:32 UTC | parallels: Fix bdrv_open() error handling Return -errno instead of -1 on errors. Hey, no memory leak to fix here while we're touching it! Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:29 UTC |
| 4f8aa2e | Kevin Wolf | 25 January 2013, 16:07:31 UTC | dmg: Use g_free instead of free The buffers are allocated with g_(re)alloc, so use g_free to free them. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:29 UTC |
| 69d34a3 | Kevin Wolf | 25 January 2013, 16:07:30 UTC | dmg: Fix bdrv_open() error handling Return -errno instead of -1 on errors and add error checks in some places that didn't have one. Passing things by reference requires more correct typing, replaced a few off_ts therefore - with a 32-bit off_t this is even a fix for truncation bugs. While touching the code, fix even some more memory leaks than in the other drivers... Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:29 UTC |
| 59294e4 | Kevin Wolf | 25 January 2013, 16:07:29 UTC | vpc: Fix bdrv_open() error handling Return -errno instead of -1 on errors. While touching the code, fix a memory leak. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:28 UTC |
| 1a60657 | Kevin Wolf | 25 January 2013, 16:07:28 UTC | cloop: Fix bdrv_open() error handling Return -errno instead of -1 on errors. While touching the code, fix a memory leak. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:28 UTC |
| 5b7d7df | Kevin Wolf | 25 January 2013, 16:07:27 UTC | bochs: Fix bdrv_open() error handling Return -errno instead of -1 on errors. While touching the code, fix a memory leak. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:28 UTC |
| 6f74c26 | Liu Yuan | 29 January 2013, 09:14:16 UTC | sheepdog: pass vdi_id to sheep daemon for sd_close() Sheep daemon needs vdi_id to identify which vdi is closed to release resources such as object cache. Cc: MORITA Kazutaka <morita.kazutaka@lab.ntt.co.jp> Cc: Kevin Wolf <kwolf@redhat.com> Cc: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Liu Yuan <tailai.ly@taobao.com> Reviewed-by: MORITA Kazutaka <morita.kazutaka@lab.ntt.co.jp> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:28 UTC |
| 7f2039f | Othmar Pasteka | 29 January 2013, 23:26:52 UTC | vmdk: Allow selecting SCSI adapter in image creation Introduce a new option "adapter_type" when converting to vmdk images. It can be one of the following: ide (default), buslogic, lsilogic or legacyESX (according to the vmdk spec from vmware). In case of a non-ide adapter, heads is set to 255 instead of the 16. The latter is used for "ide". Also see LP#545089 Signed-off-by: Othmar Pasteka <pasteka@kabsi.at> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:28 UTC |
| a04eca1 | Vishvananda Ishaya | 25 January 2013, 18:57:20 UTC | block: Adds mirroring tests for resized images This test verifies two mirroring issues are fixed with resized images: * sync='top' creates an image that is the proper size * sync='full' doesn't cause an assertion failure and crash qemu Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:28 UTC |
| 63ba17d | Vishvananda Ishaya | 24 January 2013, 18:02:08 UTC | block: Fix is_allocated_above with resized files In an image chain, if the base image is smaller than the current image, we need to make sure to use the current images count of unallocated blocks once we get to the end of the base image. Without this change the code will return 0 blocks when it gets to the end of the base image and mirror_run will fail its assertion. Signed-off-by: Vishvananda Ishaya <vishvananda@gmail.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:28 UTC |
| b93d6d2 | Kevin Wolf | 23 January 2013, 15:52:49 UTC | qemu-iotests: Add regression test for b7ab0fea It turned out that the change in b7ab0fea was actually a real qcow2 corruption fix. This is a reproducer for the bug. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> | 01 February 2013, 13:58:28 UTC |
| a31ca01 | Orit Wasserman | 31 January 2013, 07:12:19 UTC | Fix error message in migrate_set_capability HMP command Signed-off-by: Orit Wasserman <owasserm@redhat.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Juan Quintela <quintela@redhat.com> | 01 February 2013, 07:32:21 UTC |
| 1b1fdfe | Orit Wasserman | 31 January 2013, 07:12:18 UTC | Allow XBZRLE decoding without enabling the capability Before this fix we couldn't load a guest from XBZRLE compressed file. For example: The user activated the XBZRLE capability The user run migrate -d "exec:gzip -c > vm.gz" The user won't be able to load vm.gz and get an error. Signed-off-by: Orit Wasserman <owasserm@redhat.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Juan Quintela <quintela@redhat.com> | 01 February 2013, 07:32:21 UTC |
| dbca1b3 | Orit Wasserman | 31 January 2013, 07:12:17 UTC | Fix example for query-migrate-capabilities Signed-off-by: Orit Wasserman <owasserm@redhat.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Juan Quintela <quintela@redhat.com> | 01 February 2013, 07:32:21 UTC |
| 21e3cd2 | Orit Wasserman | 31 January 2013, 07:12:16 UTC | Add XBZRLE testing Signed-off-by: Orit Wasserman <owasserm@redhat.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Juan Quintela <quintela@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> | 01 February 2013, 07:32:20 UTC |
| ba2e28e | Orit Wasserman | 31 January 2013, 07:12:15 UTC | Move XBZRLE encoding code to a separate file to allow testing Signed-off-by: Orit Wasserman <owasserm@redhat.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Juan Quintela <quintela@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> | 01 February 2013, 07:32:20 UTC |
| 8a55ebf | Anthony Liguori | 01 February 2013, 01:37:33 UTC | Merge remote-tracking branch 'afaerber/qom-cpu' into staging # By Andreas Färber # Via Andreas Färber * afaerber/qom-cpu: linux-user: bsd-user: Don't reset X86CPU twice target-i386: Pass X86CPU to cpu_x86_set_a20() target-unicore32: Rename CPU subtypes target-openrisc: Rename CPU subtypes target-openrisc: TYPE_OPENRISC_CPU should be abstract target-m68k: Rename CPU subtypes target-m68k: Mark as unmigratable target-s390x: Mark as unmigratable target-sh4: Mark as unmigratable target-xtensa: Mark as unmigratable target-microblaze: Mark as unmigratable target-unicore32: Mark as unmigratable ide/mmio: QOM'ify MMIO IDE for R2D | 01 February 2013, 01:37:33 UTC |
| 2854c54 | Anthony Liguori | 01 February 2013, 01:37:23 UTC | Merge remote-tracking branch 'afaerber-or/cocoa-for-upstream' into staging # By Henry Harrington (1) and Stefan Weil (1) # Via Andreas Färber * afaerber-or/cocoa-for-upstream: cocoa: Replace non-portable asprintf() by g_strdup_printf() cocoa: Fix VBE function Set Display Start | 01 February 2013, 01:37:23 UTC |
| c628d74 | Anthony Liguori | 01 February 2013, 01:35:24 UTC | Merge remote-tracking branch 'mst/tags/for_anthony' into staging virtio,make,pci,e1000,vfio,piix This includes my timestamp generation cleanup, Amos's and my work on virtio net commands, pci,e1000,vfio and piix fixes. Signed-off-by: Michael S. Tsirkin <mst@redhat.com> # gpg: Signature made Thu 31 Jan 2013 06:20:27 AM CST using RSA key ID D28D5469 # gpg: Can't check signature: public key not found # By Michael S. Tsirkin (6) and others # Via Michael S. Tsirkin * mst/tags/for_anthony: vfio-pci: Enable PCIe extended config space PIIX3: reset the VM when the Reset Control Register's RCPU bit gets set ich9: add support for pci assignment virtio-net: rename ctrl rx commands virtio-net: introduce a new macaddr control virtio-net: remove layout assumptions for ctrl vq virtio-net: revert mac on reset rules/mak: make clean should blow away timestamp files Makefile: clean timestamp generation rule rules.mak: cleanup config generation rules e1000: document ICS read behaviour | 01 February 2013, 01:35:24 UTC |
| 49e1587 | Andreas Färber | 30 January 2013, 12:48:25 UTC | target-s390x: Pass S390CPU to s390_{add, del}_running_cpu() This prepares for moving the halted field to CPUState. Most call sites can already supply S390CPU, for some env becomes unused. Signed-off-by: Andreas Färber <afaerber@suse.de> Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com> Signed-off-by: Alexander Graf <agraf@suse.de> | 01 February 2013, 01:02:20 UTC |
| f946673 | Andreas Färber | 30 January 2013, 12:48:24 UTC | target-s390x: Clean up cpu_inject_*() signatures Despite cautioning that S390CPU is needed for upcoming CPUState refactorings, commit 5d69c547d947798cba92d836d06f6e017ba2b19d (s390: I/O interrupt and machine check injection.) added functions cpu_inject_io() and cpu_inject_crw_mchk() with CPUS390XState argument, claiming consistency with cpu_inject_ext(). This complicates making cpu_interrupt() take a CPUState even more and it required to pass &cpu->env from some S390CPU-aware call sites already, creating inconsistency elsewhere. Address that. This also eliminates the need for CPUS390XState in s390_virtio_irq(). Signed-off-by: Andreas Färber <afaerber@suse.de> Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com> Signed-off-by: Alexander Graf <agraf@suse.de> | 01 February 2013, 01:02:20 UTC |
| 07cc7d1 | Andreas Färber | 27 January 2013, 03:32:03 UTC | target-s390x: Fix debug output Commit 71e470886fb6092504503a5fe41092ace71c096c (target-s390x: fix style) renamed the cpu_s390x_handle_mmu_fault() argument from _vaddr to orig_vaddr. Update the debug output code. Signed-off-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Alexander Graf <agraf@suse.de> | 01 February 2013, 01:02:20 UTC |
| a6f921b | Andreas Färber | 27 January 2013, 03:32:04 UTC | target-s390x: Fix debug output (continued) Since its introduction in d5a439645a5a70fed5431318c3bce9dc2caa950f (s390x: helper functions for system emulation) the variable name was raddr. Fix this. Signed-off-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Alexander Graf <agraf@suse.de> | 01 February 2013, 01:02:20 UTC |
| 476b6d1 | Andreas Färber | 27 January 2013, 03:32:02 UTC | target-ppc: Fix build for PPC_DEBUG_DISAS In r5949 / 76db3ba44ee8db671f804755f13b016eefd13288 (target-ppc: memory load/store rework) variable little_endian was replaced with ctx.le_mode. Update the debug code. Signed-off-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Alexander Graf <agraf@suse.de> | 01 February 2013, 00:52:03 UTC |
| cca48a9 | Andreas Färber | 27 January 2013, 03:32:01 UTC | target-ppc: Fix unused variable warning for FLUSH_ALL_TLBS Signed-off-by: Andreas Färber <afaerber@suse.de> Signed-off-by: Alexander Graf <agraf@suse.de> | 01 February 2013, 00:52:02 UTC |
| 8e33944 | Alexander Graf | 29 January 2013, 12:36:02 UTC | PPC: Unify dcbzl code path The bit that makes a dcbz instruction a dcbzl instruction was declared as reserved in ppc32 ISAs. However, hardware simply ignores the bit, making code valid if it simply invokes dcbzl instead of dcbz even on 750 and G4. Thus, mark the bit as unreserved so that we properly emulate a simple dcbz in case we're running on non-G5s. While at it, also refactor the code to check the 970 special case during runtime. This way we don't need to differenciate between a 970 dcbz and any other dcbz anymore. We also allow for future improvements to add e500mc dcbz handling. Reported-by: Amadeusz Sławiński <amade@asmblr.net> Signed-off-by: Alexander Graf <agraf@suse.de> | 01 February 2013, 00:52:02 UTC |
| 7786812 | Andreas Färber | 20 January 2013, 04:34:10 UTC | linux-user: bsd-user: Don't reset X86CPU twice Since commit 65dee38052597b6285eb208125369f01b29ba6c1 (target-i386: move cpu_reset and reset callback to cpu.c) the x86 CPU is reset through cpu_init() but was still reset immediately after in linux-user and bsd-user. Clean this up. Similarly in linux-user/syscall.c it is also reset after cpu_copy(). But that's a bug of its own, fixing which poses a semantic change. Signed-off-by: Andreas Färber <afaerber@suse.de> Reviewed-by: Igor Mammedov <imammedo@redhat.com> | 01 February 2013, 00:35:43 UTC |
| cc36a7a | Andreas Färber | 18 January 2013, 14:19:06 UTC | target-i386: Pass X86CPU to cpu_x86_set_a20() Prepares for cpu_interrupt() changing argument to CPUState. While touching it, rename to x86_cpu_...() now that it takes an X86CPU. Signed-off-by: Andreas Färber <afaerber@suse.de> Reviewed-by: Eduardo Habkost <ehabkost@redhat.com> | 01 February 2013, 00:35:43 UTC |
| eeb266d | Andreas Färber | 27 January 2013, 22:25:25 UTC | target-unicore32: Rename CPU subtypes In the initial conversion of CPU models to QOM types, model names were mapped 1:1 to type names. As a side effect this gained us a type "any", which is now a device. To avoid "-device any" silliness and to pave the way for compiling multiple targets into one executable, adopt a <name>-<arch>-cpu scheme. No functional changes for -cpu arguments. Signed-off-by: Andreas Färber <afaerber@suse.de> | 01 February 2013, 00:35:43 UTC |
| 478032a | Andreas Färber | 27 January 2013, 21:50:35 UTC | target-openrisc: Rename CPU subtypes Model names were mapped 1:1 to type names. As a side effect this registered a type "any", which is now a device. To avoid "-device any" silliness and to pave the way for compiling multiple targets into one executable, adopt a <name>-<arch>-cpu scheme. No functional changes for -cpu arguments or -cpu ? output. Signed-off-by: Andreas Färber <afaerber@suse.de> | 01 February 2013, 00:35:43 UTC |
| bc755a0 | Andreas Färber | 27 January 2013, 21:27:17 UTC | target-openrisc: TYPE_OPENRISC_CPU should be abstract A basic assumption of CPU subtypes is that only specific models get instantiated. A user is not supposed to instantiate an <arch>-cpu. Suppress it via abstract = true, which also drops or32-cpu from -cpu ? output. Cc: qemu-stable@nongnu.org Cc: Jia Liu <proljc@gmail.com> Signed-off-by: Andreas Färber <afaerber@suse.de> | 01 February 2013, 00:35:43 UTC |
| 7a9f812 | Andreas Färber | 27 January 2013, 19:16:17 UTC | target-m68k: Rename CPU subtypes In the initial conversion of CPU models to QOM types, model names were mapped 1:1 to type names. As a side effect this gained us a type "any", which is now a device. To avoid "-device any" silliness and to pave the way for compiling multiple targets into one executable, adopt a <name>-<arch>-cpu scheme. No functional changes for -cpu arguments or -cpu ? output. Signed-off-by: Andreas Färber <afaerber@suse.de> | 01 February 2013, 00:35:22 UTC |
| 087fe4f | Andreas Färber | 20 January 2013, 18:53:28 UTC | target-m68k: Mark as unmigratable It neither defined CPU_SAVE_VERSION nor implemented cpu_{save,load}(). Mark M68kCPU as unmigratable at device level. Signed-off-by: Andreas Färber <afaerber@suse.de> Reviewed-by: Juan Quintela <quintela@redhat.com> | 01 February 2013, 00:35:22 UTC |
| c7396bb | Andreas Färber | 20 January 2013, 18:41:06 UTC | target-s390x: Mark as unmigratable CPU_SAVE_VERSION was undefined, so "cpu_common" VMState and cpu_{save,load}() were not registered. They were no-ops. Therefore there is no backwards compatibility to keep, so we can mark S390CPU as unmigratable at device level. Signed-off-by: Andreas Färber <afaerber@suse.de> Acked-by: Alexander Graf <agraf@suse.de> Reviewed-by: Juan Quintela <quintela@redhat.com> | 01 February 2013, 00:35:22 UTC |
| 1e45d31 | Andreas Färber | 20 January 2013, 18:32:33 UTC | target-sh4: Mark as unmigratable It neither defined CPU_SAVE_VERSION nor implemented cpu{save,load}(). Mark it as unmigratable at device level. Signed-off-by: Andreas Färber <afaerber@suse.de> Reviewed-by: Juan Quintela <quintela@redhat.com> | 01 February 2013, 00:35:22 UTC |
| 004a569 | Andreas Färber | 20 January 2013, 18:22:41 UTC | target-xtensa: Mark as unmigratable There was no CPU_SAVE_VERSION defined, so neither "cpu_common" VMState nor cpu_{save,load}() were registered. Their implementation was no-op. Therefore there is no backwards compatibility to keep, so mark XtensaCPU as unmigratable at device level. Signed-off-by: Andreas Färber <afaerber@suse.de> Reviewed-by: Juan Quintela <quintela@redhat.com> | 01 February 2013, 00:35:21 UTC |
| 3ce8b2b | Andreas Färber | 20 January 2013, 18:03:32 UTC | target-microblaze: Mark as unmigratable cpu_{save,load} were no-ops, so de facto it is unmigratable and no backwards compatibility to keep. Therefore mark the MicroBlazeCPU as unmigratable at device level the QOM way and suppress "cpu_common" VMState registration by dropping CPU_SAVE_VERSION. Signed-off-by: Juan Quintela <quintela@redhat.com> Signed-off-by: Andreas Färber <afaerber@suse.de> Reviewed-by: Juan Quintela <quintela@redhat.com> | 01 February 2013, 00:35:21 UTC |
| 88e2851 | Andreas Färber | 20 January 2013, 10:43:30 UTC | target-unicore32: Mark as unmigratable CPU_SAVE_VERSION 2 was bogus as both save and load would just throw a hw_error(). Therefore we can without problems suppress registration of "cpu_common" VMState by dropping CPU_SAVE_VERSION define and provide an unmigratable "cpu" VMStateDescription for UniCore32CPU at device level instead, where we can attach this the QOM way. Signed-off-by: Juan Quintela <quintela@redhat.com> Signed-off-by: Andreas Färber <afaerber@suse.de> Reviewed-by: Juan Quintela <quintela@redhat.com> | 01 February 2013, 00:35:21 UTC |
| 6b2578d | Andreas Färber | 31 January 2013, 23:13:41 UTC | ide/mmio: QOM'ify MMIO IDE for R2D It was not qdev'ified before, so turn it into a SysBusDevice. Keep mmio_ide_init_drives() around to attach the hard drive. Signed-off-by: Andreas Färberr <afaerber@suse.de> Cc: Markus Armbruster <armbru@redhat.com> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net> | 31 January 2013, 23:13:41 UTC |
| e62a214 | Petar Jovanovic | 23 January 2013, 03:17:41 UTC | target-mips: fix incorrect test for MTHLIP The pos field in the DSPControl register is not correctly initialized. Per documentation, the result of MTHLIP is unpredictable if the value of the pos field before the execution is greater than 32. Signed-off-by: Petar Jovanovic <petarj@mips.com> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net> | 31 January 2013, 22:42:04 UTC |
| 6f0af30 | Petar Jovanovic | 23 January 2013, 02:57:02 UTC | target-mips: enable access to DSP ASE if implemented compute_hflags() will reset DSP h-flags, so MX bit should be initially set for usermode in cpu_state_reset() if DSP ASE is implemented. This change will bring back user-mode support for DSP ASE, since one of the recent changes broke it. Signed-off-by: Petar Jovanovic <petarj@mips.com> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net> | 31 January 2013, 22:42:03 UTC |
| f54c35d | Richard Sandiford | 22 January 2013, 17:16:00 UTC | target-mips: Unfuse {,N}M{ADD,SUB}.fmt Turn MADD.fmt, MSUB.fmt, NMADD.fmt and NMSUB.fmt from fused to unfused operations, so that they behave in the same way as a separate multiplication and addition. The instructions were only fused in early MIPS IV processors. Signed-off-by: Richard Sandiford <rdsandiford@googlemail.com> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net> | 31 January 2013, 22:42:02 UTC |
| c728154 | Richard Sandiford | 20 January 2013, 19:30:54 UTC | target-mips: Sign-extend the result of LWR Sign-extend the result of LWR, as is already done for LWL. This is necessary in the case where LWR loads the full word (i.e. the address is actually aligned). In the other cases, it is implementation defined whether the upper 32 bits of the result are unchanged or a copy of bit 31. The latter seems easier to implement. Previously the code used: (oldval & (0xfffffffe << (31 - bitshift))) | (newval >> bitshift) which zeroed the upper bits of the register, losing any previous sign extension in the unaligned cases. Signed-off-by: Richard Sandiford <rdsandiford@googlemail.com> Reviewed-by: Richard Henderson <rth@twiddle.net> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net> | 31 January 2013, 22:40:52 UTC |
| 17e8fef | Richard Sandiford | 20 January 2013, 19:28:48 UTC | target-mips: Fix signedness of loads in MIPS16 RESTOREs Make RESTORE use sign-extending rather than zero-extending loads. Signed-off-by: Richard Sandiford <rdsandiford@googlemail.com> Reviewed-by: Richard Henderson <rth@twiddle.net> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net> | 31 January 2013, 22:38:37 UTC |
| 810ded1 | Aurelien Jarno | 31 January 2013, 22:33:14 UTC | Merge branch 'target-arm.next' of git://git.linaro.org/people/pmaydell/qemu-arm * 'target-arm.next' of git://git.linaro.org/people/pmaydell/qemu-arm: target-arm: Rename CPU types target-arm: Fix TCG temp leaks for WI and UNDEF VFP sysreg writes | 31 January 2013, 22:33:14 UTC |
| df6126a | Aurelien Jarno | 01 January 2013, 17:02:24 UTC | target-mips: implement DSP (d)append sub-class with TCG DSP instruction from the (d)append sub-class can be implemented with TCG. Use a different function for these instructions are they are quite different from compare-pick sub-class. Fix BALIGN instruction for negative value, where the value should be zero-extended before being shift to the right. Reviewed-by: Richard Henderson <rth@twiddle.net> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net> | 31 January 2013, 22:30:19 UTC |
