[CCing Vadim and Yan]
On Thu, Jul 04, 2013 at 07:00:49AM +, Libaiqing wrote:
> Hi asias,
> 1 Window can not boot from vhost-scsi device;
> 2 A non-bootable vhost-scsi device can not be driven by windows guest.(I
> used the driver virtio-win-0.1-59 version.)
>
> Do you have an
Il 04/07/2013 23:07, Peter Lieven ha scritto:
>
> Am 04.07.2013 um 14:37 schrieb Paolo Bonzini :
>
>> Il 03/07/2013 23:23, Peter Lieven ha scritto:
>>> BDC is not used. I had an implementation that sent multiple descriptors
>>> out, but
>>> at least for my storage the maximum unmap counts not fo
On 06/13/2013 02:14 PM, Stefan Hajnoczi wrote:
On Thu, Jun 13, 2013 at 10:31:04AM +0530, chandrashekar shastri wrote:
We are testing the upstream KVM with :
Kernel, Qemu, Libvirt, Virt-Manager is built from the source (git).
kernel version : 3.9.0+
qemu version : QEMU emulator version 1.5.0
li
** Attachment added: "Destination guest logs"
https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1192499/+attachment/3725329/+files/dest.tar.bz2
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1
** Attachment added: "Destination Libvirtd logs"
https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1192499/+attachment/3725331/+files/destination_libvirtd.logs
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.laun
** Attachment added: "Source Libvirtd logs"
https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1192499/+attachment/3725330/+files/source_libvirtd.logs
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/
** Attachment added: "source guest logs"
https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1192499/+attachment/3725328/+files/source.log.tar.bz2
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/
Currently, unrealized is triggered in device_unparent(). But
unrealized normally involves the reclaim of resource occupied by
DeviceState. To obey the idiom that reclaiming resource when
refcnt reach zero, move it on the path of object_finalize().
As for device_unparent(), it would be the place to
On Thu, 07/04 16:02, Stefan Hajnoczi wrote:
> On Tue, Jul 02, 2013 at 01:59:46PM +0800, Fam Zheng wrote:
> > Signed-off-by: Fam Zheng
> > ---
> > block.c | 71
> > ++---
> > 1 file changed, 11 insertions(+), 60 deletions(-)
>
> Way too
On 07/05/2013 05:15 AM, Laszlo Ersek wrote:
> On 07/04/13 21:49, Paolo Bonzini wrote:
>> Il 04/07/2013 11:53, Wanlong Gao ha scritto:
>>> As you know, QEMU can't direct it's memory allocation now, this may cause
>>> guest cross node access performance regression.
>>> And, the worse thing is that if
On 07/05/2013 03:49 AM, Paolo Bonzini wrote:
> Il 04/07/2013 11:53, Wanlong Gao ha scritto:
>> As you know, QEMU can't direct it's memory allocation now, this may cause
>> guest cross node access performance regression.
>> And, the worse thing is that if PCI-passthrough is used,
>> direct-attached-
On Thu, 07/04 23:10, Peter Lieven wrote:
>
> Am 04.07.2013 um 10:13 schrieb Paolo Bonzini :
>
> > Il 03/07/2013 23:04, Peter Lieven ha scritto:
> Define the return value of get_block_status. Bits 0, 1, 2 and 8-62
> are valid; bit 63 (the sign bit) is reserved for errors. Bits 3-7
> >>
On Thu, Jul 04, 2013 at 07:00:49AM +, Libaiqing wrote:
> Hi asias,
> 1 Window can not boot from vhost-scsi device;
> 2 A non-bootable vhost-scsi device can not be driven by windows guest.(I
> used the driver virtio-win-0.1-59 version.)
>
> Do you have any plan about these?
Th
On 04.07.2013, at 17:01, Markus Armbruster wrote:
> Alexander Graf writes:
>
>> On 04.07.2013, at 15:09, Markus Armbruster wrote:
>>
>>> Make it QEMU_OPT_NUMBER, so it gets parsed by generic code, which
>>> actually bothers to check for errors, rather than its user, which
>>> doesn't.
>>>
>>>
On 29/06/2013 22:29, Olivier Danet wrote:
On 28/06/2013 23:44, Mark Cave-Ayland wrote:
On 28/06/13 03:08, Rob Landley wrote:
Commit 467b34689d27 upgraded the openbios image, and ever since my
linux
system images hang about the time they try to initialize interrupts.
http://landley.net/aborig
On 07/04/13 21:49, Paolo Bonzini wrote:
> Il 04/07/2013 11:53, Wanlong Gao ha scritto:
>> As you know, QEMU can't direct it's memory allocation now, this may cause
>> guest cross node access performance regression.
>> And, the worse thing is that if PCI-passthrough is used,
>> direct-attached-devic
Am 04.07.2013 um 10:13 schrieb Paolo Bonzini :
> Il 03/07/2013 23:04, Peter Lieven ha scritto:
Define the return value of get_block_status. Bits 0, 1, 2 and 8-62
are valid; bit 63 (the sign bit) is reserved for errors. Bits 3-7
are left for future extensions.
>> Is Bit 8 not also
Am 04.07.2013 um 14:37 schrieb Paolo Bonzini :
> Il 03/07/2013 23:23, Peter Lieven ha scritto:
>> BDC is not used. I had an implementation that sent multiple descriptors out,
>> but
>> at least for my storage the maximum unmap counts not for each descriptors,
>> but for all
>> together. So in t
I don't think the debugger actually looks at this for anything,
using the correct .debug_frame contents, but might as well get
it all correct.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
tcg/tcg.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tcg/tc
These will necessarily be the same layout for all hosts. This limits
the amount of boilerplate required to implement jit debug for a host.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
tcg/hppa/tcg-target.c | 35 ++-
tcg/i386/tcg-target.c | 3
With this we can generate armv7 insns even when the OS compiles for a
lower common denominator. The macros are arranged so that when we do
compile for a given ISA, all of the runtime checks for that ISA are
optimized away.
Signed-off-by: Richard Henderson
---
tcg/arm/tcg-target.c | 20 +
We can now detect and use divide instructions at runtime, rather than
having to restrict their availability to compile-time.
Signed-off-by: Richard Henderson
---
tcg/arm/tcg-target.c | 16 ++--
tcg/arm/tcg-target.h | 14 --
2 files changed, 22 insertions(+), 8 deletions(-
As it really controls the availability of a thumb interworking
instruction on armv5t.
Signed-off-by: Richard Henderson
---
tcg/arm/tcg-target.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/tcg/arm/tcg-target.c b/tcg/arm/tcg-target.c
index 648137f..cfb9afb 1006
This allows TCG_TARGET_HAS_* to be a variable rather than a constant,
which allows easier support for differing ISA levels for the host.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
tcg/tcg-opc.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tcg/tcg-opc.
Reviewed-by: Andreas Färber
Signed-off-by: Richard Henderson
---
tcg/ppc64/tcg-target.c | 26 --
tcg/ppc64/tcg-target.h | 4 ++--
2 files changed, 2 insertions(+), 28 deletions(-)
diff --git a/tcg/ppc64/tcg-target.c b/tcg/ppc64/tcg-target.c
index 606b73d..0678de2 100644
GCC 4.8 defines a handy __ARM_ARCH symbol that we can use, which
will make us nicely forward compatible with ARMv8 AArch32.
Signed-off-by: Richard Henderson
---
tcg/arm/tcg-target.c | 62 +++-
1 file changed, 23 insertions(+), 39 deletions(-)
diff
Reviewed-by: Andreas Färber
Signed-off-by: Richard Henderson
---
tcg/ppc/tcg-target.c | 14 --
tcg/ppc/tcg-target.h | 2 +-
2 files changed, 1 insertion(+), 15 deletions(-)
diff --git a/tcg/ppc/tcg-target.c b/tcg/ppc/tcg-target.c
index 29ca934..453ab6b 100644
--- a/tcg/ppc/tcg-targ
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
tcg/arm/tcg-target.c | 14 --
tcg/arm/tcg-target.h | 3 +--
2 files changed, 1 insertion(+), 16 deletions(-)
diff --git a/tcg/arm/tcg-target.c b/tcg/arm/tcg-target.c
index 6be736b..8321f80 100644
--- a/tcg/arm/tcg-targ
Allows unwinding past the code_gen_buffer.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
tcg/arm/tcg-target.c | 76 +---
1 file changed, 67 insertions(+), 9 deletions(-)
diff --git a/tcg/arm/tcg-target.c b/tcg/arm/tcg-target.c
i
There are several hosts with only a "div" insn. Remainder is computed
manually from the quotient and inputs. We can do this generically.
Signed-off-by: Richard Henderson
---
tcg/aarch64/tcg-target.h | 2 ++
tcg/arm/tcg-target.h | 2 ++
tcg/hppa/tcg-target.h| 1 +
tcg/ia64/tcg-target
We can check the condition at compile time, rather than run time.
Reviewed-by: Andreas Färber
Signed-off-by: Richard Henderson
---
tcg/i386/tcg-target.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c
index ae3de09..87eeab3
Acked-by: Peter Maydell
Signed-off-by: Richard Henderson
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index ad9c860..7847e60 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -772,6 +772,7 @@ Tiny Code Generator (TCG)
-
Commo
Expand the definition of "not present" to include "should not be present".
This means we can simplify the logic surrounding the generic tcg opcodes
for which the host backend ought not be providing definitions.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
tcg/tcg-opc.h | 26 +
Changes v3-v4:
* Feedback from Peter Maydell:
+ Use only HWCAP_ARM_IDIVA to detect division
+ New patch 9 to rename variable from armv5 to armv5t
+ Adjust ARM ISA logic patch for identifying armv5t
I didn't go whole hog wrt detecting armv5t w/ hwcap, as I didn't
really want to have to t
Il 04/07/2013 19:17, Andreas Färber ha scritto:
> Also if at some point you try to use, e.g., q35 rather than i440fx
> machine, it might have EHCI by default and the snippet would be
> instantiating a duplicate one. I.e., the three levels of dependency
> could be separated more clearly: PCI, USB, r
Previous realization doesn't consider flags in the status register.
Add DS and INTR bits of HST_STS register set after transaction execution.
Update bits resetting in HST_STS register. Update error processing:
if DEV_ERR bit set transaction isn't execution.
Signed-off-by: MRatnikov
---
hw/i2c/p
Il 04/07/2013 11:53, Wanlong Gao ha scritto:
> As you know, QEMU can't direct it's memory allocation now, this may cause
> guest cross node access performance regression.
> And, the worse thing is that if PCI-passthrough is used,
> direct-attached-device uses DMA transfer between device and qemu pr
On 4 July 2013 18:44, Richard Henderson wrote:
> On 07/04/2013 04:22 AM, Peter Maydell wrote:
>>
>> This change means we now set use_armv5_instructions
>> for __ARCH_ARCH_5__ and __ARM_ARCH_5E__, which we didn't
>> before. However one of the things that bool is gating is
>> whether we use the 'blx
On Thu, 4 Jul 2013 16:01:23 +0300
Timo Teräs wrote:
> PIE images are ET_DYN images. Check first for pinterp_name to make
> sure the main executable always is loaded to correct place.
>
> See below for current behaviour of PIE executables:
>
> Reserved 0x7f00 bytes of guest address space
>
On 07/04/2013 04:22 AM, Peter Maydell wrote:
>
> This change means we now set use_armv5_instructions
> for __ARCH_ARCH_5__ and __ARM_ARCH_5E__, which we didn't
> before. However one of the things that bool is gating is
> whether we use the 'blx' insn, which is ARMv5T and above only.
> So this will
On 4 July 2013 18:33, Richard Henderson wrote:
> On 07/04/2013 04:02 AM, Peter Maydell wrote:
>>> +unsigned long hwcap = getauxval(AT_HWCAP);
>>> +use_idiv_instructions = hwcap & (HWCAP_ARM_IDIVA |
>>> HWCAP_ARM_IDIVT);
>>
>> Doesn't this mean we'll try to use the ARM division
>>
On 07/04/2013 04:02 AM, Peter Maydell wrote:
>
> My ARM system doesn't have a sys/auxv.h, which renders most of this patch
> a bit moot (and certainly untestable :-)). Do newer glibc have this?
glibc 2.16 has this.
I've also got another pending patch set that implements getauxval inside
qemu if
On 07/04/2013 03:53 AM, Peter Maydell wrote:
> On 3 July 2013 22:29, Richard Henderson wrote:
>> This allows TCG_TARGET_HAS_* to be a variable rather than a constant,
>> which allows easier support for differing ISA levels for the host.
>
> The effect of this is that TCG_OPF_NOT_PRESENT means "if
Am 04.07.2013 14:46, schrieb Stefano Stabellini:
> On Thu, 4 Jul 2013, Fabio Fantoni wrote:
>> Il 04/07/2013 12:32, Wei Liu ha scritto:
>>
>> On Thu, Jul 04, 2013 at 12:16:43PM +0200, Fabio Fantoni wrote:
>>
>> Il 04/07/2013 12:12, Wei Liu ha scritto:
>>
>> On Thu, Jul 04, 2013 at 12:05:59PM +0200,
From: Jan Kiszka
In case the latter may vanish one day, make sure the vmport read handler
type will remain unaffected. This is also conceptually cleaner.
Signed-off-by: Jan Kiszka
Signed-off-by: Paolo Bonzini
---
hw/misc/vmport.c | 4 ++--
include/hw/i386/pc.h | 6 --
2 files changed,
You created a qcow2 file but told libvirt it's type='raw'. Therefore
the guest sees a raw file and not a qcow2 file. The qcow2 file is only
196 KB on disk (qemu-img output shows this), this matches with the
blocksize --getsize64 output inside the guest.
Change type='raw' to type='qcow2' in yo
Peter Maydell writes:
> On 4 July 2013 16:58, Markus Armbruster wrote:
>> Peter Maydell writes:
>>
>>> On 4 July 2013 14:09, Markus Armbruster wrote:
Multiple -machine options with the same ID are merged. All but the
one without an ID are to be silently ignored.
>>>
>>> I think it w
From: Jan Kiszka
Convert over to memory regions to obsolete register_ioport*.
Signed-off-by: Jan Kiszka
Signed-off-by: Paolo Bonzini
---
hw/watchdog/wdt_ib700.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/hw/watchdog/wdt_ib700.c b/hw/watchdog/wdt_ib700.c
Guest output:
---
[root@redhat ~]# ls -lrt /dev/sd
sda sda1 sda2 sdb
[root@redhat ~]# ls -lrt /dev/sd
sda sda1 sda2 sdb
[root@redhat ~]# ls -lrt /dev/sdb
brw-rw. 1 root disk 8, 16 Jul 4 21:58 /dev/sdb
[root@redhat ~]# blockdev --getsize64 /dev/sdb
197120
[root@r
From: Liu Ping Fan
Sections like phys_section_unassigned always have fixed address
in phys_sections. Declared as macro, so we can use them
when having more than one phys_sections array.
Signed-off-by: Liu Ping Fan
Signed-off-by: Liu Ping Fan
Signed-off-by: Paolo Bonzini
---
exec.c | 34
On 4 July 2013 17:11, Markus Armbruster wrote:
> Peter Maydell writes:
>
>> On 4 July 2013 16:03, Markus Armbruster wrote:
>>> Peter Maydell writes:
This looks a bit odd -- why are we creating new
options in a function that claims to only be querying
them?
>>>
>>> So we never ret
Signed-off-by: Paolo Bonzini
---
hw/timer/arm_mptimer.c | 4 ++--
hw/timer/arm_timer.c| 6 --
hw/timer/cadence_ttc.c | 3 ++-
hw/timer/etraxfs_timer.c| 3 ++-
hw/timer/exynos4210_mct.c | 4 ++--
hw/timer/exynos4210_pwm.c | 4 ++--
hw/timer/exynos4210_rtc.c | 4 ++--
On Jul 4, 2013, at 10:51 AM, Stefan Hajnoczi wrote:
> On Thu, Jul 4, 2013 at 4:45 PM, Alexander Graf wrote:
>>
>> On 04.07.2013, at 16:40, Programmingkid wrote:
>>
>>> We have made a lot of progress in the last month with making Mac OS X run
>>> in QEMU. A lot of people are to thank for this
On Jul 4, 2013, at 10:45 AM, Alexander Graf wrote:
>
> On 04.07.2013, at 16:40, Programmingkid wrote:
>
>> We have made a lot of progress in the last month with making Mac OS X run in
>> QEMU. A lot of people are to thank for this milestone. To everyone involved,
>> thank you.
>>
>> There is
This is the first step towards converting as->current_map to
RCU-style updates, where the FlatView updates run concurrently
with uses of an old FlatView.
Reviewed-by: Anthony Liguori
Signed-off-by: Paolo Bonzini
---
memory.c | 34 ++
1 file changed, 18 insertions
Il 04/07/2013 18:27, Jan Kiszka ha scritto:
> This workaround causes troubles here with gcc-4.5.1:
>
> In file included from /data/qemu/include/exec/memory.h:29:0,
> from /data/qemu/include/exec/ioport.h:29,
> from /data/qemu/include/hw/hw.h:11,
>
Signed-off-by: Paolo Bonzini
---
hw/pci-host/pam.c | 16 +---
hw/pci-host/piix.c| 4 ++--
hw/pci-host/q35.c | 4 ++--
include/hw/pci-host/pam.h | 4 ++--
4 files changed, 15 insertions(+), 13 deletions(-)
diff --git a/hw/pci-host/pam.c b/hw/pci-host/pam.c
i
Il 04/07/2013 12:32, Wei Liu ha scritto:
On Thu, Jul 04, 2013 at 12:16:43PM +0200, Fabio Fantoni wrote:
Il 04/07/2013 12:12, Wei Liu ha scritto:
On Thu, Jul 04, 2013 at 12:05:59PM +0200, Fabio Fantoni wrote:
Usage: spiceusbredirection=1|0 (default=0)
Enables spice usbredirection. The Spice usb
Hi Andreas,
Thank you for taking a look.
I have incorporated your feedback into a new patch, attached herewith.
Regards,
Prerna
Subject: [PATCH] target-ppc: Add POWER8 v1.0 CPU model
This patch adds CPU PVR definition for POWER8,
and enables QEMU to launch guests on POWER8 hardware.
Signed-off
On 03.07.2013 13:43, Daniel P. Berrange wrote:
> On Tue, Jul 02, 2013 at 01:25:21PM +0200, poma wrote:
>> Hello people,
>>
>> libvirtd (libvirt) 1.0.5.2
>> virsh 1.0.5.2
>> virt-manager 0.10.0
>>
>> Host:
>> Linux localhost 3.9.8-300.fc19.x86_64 #1 SMP Thu Jun 27 19:24:23 UTC
>> 2013 x86_64 x86_64
You have been subscribed to a public bug:
virsh migration copy-storage-all fails with "Unable to read from
monitor: Connection reset by peer" and shut downs the guest on the
source host.
Kernel Version: 3.10.0-rc5+
Libvirt Version: 1.0.6
Qemu Version: 1.5.50
Steps to reproduce the issue:
---
On 2013-07-01 12:14, Paolo Bonzini wrote:
> Il 01/07/2013 11:54, Peter Maydell ha scritto:
>> On 1 July 2013 10:35, Stefan Hajnoczi wrote:
>>> From: Paolo Bonzini
>>>
>>> Fast TLS is not available on some platforms, but it is always nice to
>>> use it. This wrapper implementation falls back to p
Signed-off-by: Paolo Bonzini
---
hw/pci-bridge/dec.c| 4 ++--
hw/pci-bridge/pci_bridge_dev.c | 2 +-
hw/pci-host/apb.c | 12 ++--
hw/pci-host/bonito.c | 10 +-
hw/pci-host/grackle.c | 8
hw/pci-host/piix.c | 13 ++
Signed-off-by: Paolo Bonzini
---
hw/display/cirrus_vga.c | 2 +-
hw/display/exynos4210_fimd.c | 2 +-
hw/display/jazz_led.c| 2 +-
hw/display/milkymist-tmu2.c | 2 +-
hw/display/milkymist-vgafb.c | 2 +-
hw/display/pl110.c | 2 +-
hw/display/qxl.c | 15 ++
From: Jan Kiszka
Before switching to the memory core dispatcher, we need to make sure
that this pv-device will continue to receive unaligned portio accesses.
Signed-off-by: Jan Kiszka
Signed-off-by: Paolo Bonzini
---
hw/xen/xen_platform.c | 4
1 file changed, 4 insertions(+)
diff --git
phys_sections_clear is invoked after the dispatch tree has been
destroyed. This leaves a window where phys_sections_nb > 0 but the
subpages are not valid anymore, which is a recipe for use-after-free
bugs.
Move the destruction of subpages in phys_sections_clear. We will
still destroy the subpage
On 2013-07-04 18:00, Paolo Bonzini wrote:
> Il 04/07/2013 17:51, Jan Kiszka ha scritto:
diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c
index 16d6356..65e6680 100644
--- a/hw/usb/hcd-ehci.c
+++ b/hw/usb/hcd-ehci.c
@@ -1245,7 +1245,7 @@ static int ehci_init_transfer(EHCIP
We are using the same struct name for two devices. 8250 is widespread
enough that this causes some confusion, rename the other instance.
Reviewed-by: Andreas Faerber
Signed-off-by: Paolo Bonzini
---
hw/char/escc.c | 34 +-
1 file changed, 17 insertions(+), 17 de
Peter Maydell writes:
> On 4 July 2013 16:03, Markus Armbruster wrote:
>> Peter Maydell writes:
>>> This looks a bit odd -- why are we creating new
>>> options in a function that claims to only be querying
>>> them?
>>
>> So we never return null. If it bothers you, I can initialize the
>> opti
Il 04/07/2013 17:52, Paolo Bonzini ha scritto:
>> > Anthony,
>> >
>> > the following changes since commit
>> > 1acd5a373905ddb28957842256a038956941f332:
>> >
>> > Merge remote-tracking branch 'agraf/ppc-for-upstream' into staging
>> > (2013-07-01 09:03:04 -0500)
>> >
>> > are available in th
On Thu, Jul 4, 2013 at 9:07 AM, chandrashekar shastri
wrote:
> Public bug reported:
>
> qcow2 [virtio-scsi] devices when mapped to the guest shows as 0MB
> irrespective of the volume size.
qemu-system-x86_64 -enable-kvm -m 1024 -cpu host,+x2apic \
-device virtio-scsi-pci,id=virtio-scsi0 \
Signed-off-by: Paolo Bonzini
---
hw/gpio/omap_gpio.c | 6 +++---
hw/gpio/pl061.c | 2 +-
hw/gpio/puv3_gpio.c | 2 +-
hw/gpio/zaurus.c| 2 +-
4 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/hw/gpio/omap_gpio.c b/hw/gpio/omap_gpio.c
index 13399f9..855afae 100644
--- a/hw/gpio
On 4 July 2013 16:58, Markus Armbruster wrote:
> Peter Maydell writes:
>
>> On 4 July 2013 14:09, Markus Armbruster wrote:
>>> Multiple -machine options with the same ID are merged. All but the
>>> one without an ID are to be silently ignored.
>>
>> I think it would make more sense just to say
Il 04/07/2013 17:51, Jan Kiszka ha scritto:
>> > diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c
>> > index 16d6356..65e6680 100644
>> > --- a/hw/usb/hcd-ehci.c
>> > +++ b/hw/usb/hcd-ehci.c
>> > @@ -1245,7 +1245,7 @@ static int ehci_init_transfer(EHCIPacket *p)
>> > cpage = get_field(p->qt
On 2013-07-04 17:51, Jan Kiszka wrote:
> On 2013-07-04 17:12, Paolo Bonzini wrote:
>> Signed-off-by: Paolo Bonzini
>> ---
>> dma-helpers.c | 6 +-
>> hw/ide/ahci.c | 3 ++-
>> hw/ide/macio.c| 4 ++--
>> hw/scsi/virtio-scsi.c | 10 ++
>> hw/usb/hcd-ehci.c
Peter Maydell writes:
> On 4 July 2013 14:09, Markus Armbruster wrote:
>> Multiple -machine options with the same ID are merged. All but the
>> one without an ID are to be silently ignored.
>
> I think it would make more sense just to say that specifying
> id= for -machine (or any other merge_l
On 4 July 2013 16:28, Markus Armbruster wrote:
> Peter Maydell writes:
>> On the other hand I don't think the old code's behaviour
>> was really right either. I think part of the problem here
>> is that it really makes no sense to specify id= for a
>> QemuOptsList with merge_lists=true -- id= is
Reviewed-by: Jan Kiszka
Signed-off-by: Paolo Bonzini
---
include/exec/memory.h | 30 ++
memory.c | 14 ++
2 files changed, 44 insertions(+)
diff --git a/include/exec/memory.h b/include/exec/memory.h
index 2f3e8e4..d2466a7 100644
--- a/include
Il 04/07/2013 17:12, Paolo Bonzini ha scritto:
> Anthony,
>
> the following changes since commit 1acd5a373905ddb28957842256a038956941f332:
>
> Merge remote-tracking branch 'agraf/ppc-for-upstream' into staging
> (2013-07-01 09:03:04 -0500)
>
> are available in the git repository at:
>
>
>
On 2013-07-04 17:12, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini
> ---
> dma-helpers.c | 6 +-
> hw/ide/ahci.c | 3 ++-
> hw/ide/macio.c| 4 ++--
> hw/scsi/virtio-scsi.c | 10 ++
> hw/usb/hcd-ehci.c | 4 ++--
> include/hw/pci/pci.h | 2 +-
> i
Signed-off-by: Paolo Bonzini
---
hw/usb/hcd-ehci-sysbus.c | 2 +-
hw/usb/hcd-ehci.c | 8
hw/usb/hcd-ohci.c | 3 ++-
hw/usb/hcd-uhci.c | 4 +++-
hw/usb/hcd-xhci.c | 12 ++--
hw/virtio/virtio-pci.c | 4 ++--
hw/watchdog/wdt_i6300esb.c
After this patch, AddressSpaceDispatch holds a constistent tuple of
(phys_map, nodes, sections). This will be important when updates
of the topology will run concurrently with reads.
cur_map is not used anymore except for freeing it at the end of the
topology update.
Signed-off-by: Paolo Bonzini
cur_map is not used anymore; instead, each AddressSpaceDispatch
has its own nodes/sections pair. The priorities of the
MemoryListeners, and in the future RCU, guarantee that the
nodes/sections are not freed while they are still in use.
(In fact, next_map itself is not needed except to free the da
From: Jan Kiszka
Object reference counts will soon be changed outside the BQL. So we need
to use atomics in object_ref/unref.
Based on a patch by Liu Ping Fan.
Signed-off-by: Liu Ping Fan
Signed-off-by: Jan Kiszka
Signed-off-by: Paolo Bonzini
---
qom/object.c | 5 ++---
1 file changed, 2 in
This will help having two copies of AddressSpaceDispatch during the
recreation of the radix tree (one being built, and one that is complete
and will be protected by RCU). We do not want to have to unregister and
re-register the listener.
Reviewed-by: Jan Kiszka
Signed-off-by: Paolo Bonzini
---
Signed-off-by: Paolo Bonzini
---
hw/net/cadence_gem.c| 3 ++-
hw/net/e1000.c | 6 +++---
hw/net/eepro100.c | 12 ++--
hw/net/etraxfs_eth.c| 3 ++-
hw/net/lan9118.c| 3 ++-
hw/net/lance.c | 3 ++-
hw/net/milkymist-minimac
This same treatment previously done to phys_node_map and phys_sections
is now applied to the dispatch field of AddressSpace. Topology updates
use as->next_dispatch while accesses use as->dispatch.
Reviewed-by: Jan Kiszka
Signed-off-by: Paolo Bonzini
---
exec.c| 23 +
With this change, a FlatView can be used even after a concurrent
update has replaced it. Because we do not yet have RCU, we use a
mutex to protect the small critical sections that read/write the
as->current_map pointer. Accesses to the FlatView can be done
outside the mutex.
If a MemoryRegion wi
Signed-off-by: Paolo Bonzini
---
hw/mips/gt64xxx_pci.c | 2 +-
hw/misc/a9scu.c | 3 ++-
hw/misc/applesmc.c| 4 ++--
hw/misc/arm_l2x0.c| 3 ++-
hw/misc/arm_sysctl.c | 3 ++-
hw/misc/debugexit.c | 2 +-
hw/misc/eccmemctl.c | 4 ++--
hw/misc/exyn
Reviewed-by: Jan Kiszka
Signed-off-by: Paolo Bonzini
---
include/exec/memory.h | 66 +--
1 file changed, 33 insertions(+), 33 deletions(-)
diff --git a/include/exec/memory.h b/include/exec/memory.h
index c8d0bf4..8355bdb 100644
--- a/include/exec/
Peter Maydell writes:
> On 4 July 2013 14:09, Markus Armbruster wrote:
>> Commit 4f6dd9a changed the initialization of opts in opts_parse() to
>> this:
>>
>> if (defaults) {
>> if (!id && !QTAILQ_EMPTY(&list->head)) {
>> opts = qemu_opts_find(list, NULL);
>> } els
We will soon require accesses to as->current_map to be placed under
a lock (with reference counting so as to keep the critical section
small). To simplify this change, always fetch as->current_map into
a local variable and access it through that variable.
Reviewed-by: Anthony Liguori
Signed-off-
Reviewed-by: Jan Kiszka
Signed-off-by: Paolo Bonzini
---
exec.c | 11 +--
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/exec.c b/exec.c
index 7a6d969..03fdf7e 100644
--- a/exec.c
+++ b/exec.c
@@ -230,11 +230,10 @@ bool memory_region_is_unassigned(MemoryRegion *mr)
Signed-off-by: Paolo Bonzini
---
hw/block/fdc.c | 7 ---
hw/block/nvme.c | 3 ++-
hw/block/onenand.c | 10 ++
hw/block/pflash_cfi01.c | 2 +-
hw/block/pflash_cfi02.c | 8
5 files changed, 17 insertions(+), 13 deletions(-)
diff --git a/hw/block/fdc.c
Currently, phys_node_map and phys_sections are shared by all
of the AddressSpaceDispatch. When updating mem topology, all
AddressSpaceDispatch will rebuild dispatch tables sequentially
on them. In order to prepare for RCU access, leave the old
memory map alive while the next one is being accessed
Signed-off-by: Paolo Bonzini
---
hw/ppc/spapr_iommu.c | 4 ++--
hw/ppc/spapr_pci.c | 3 ++-
hw/ppc/spapr_vio.c | 2 +-
include/hw/ppc/spapr.h | 3 ++-
4 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/hw/ppc/spapr_iommu.c b/hw/ppc/spapr_iommu.c
index 3976af6..89b33a5 100644
Signed-off-by: Paolo Bonzini
---
hw/i2c/bitbang_i2c.c | 2 +-
hw/i2c/exynos4210_i2c.c | 4 ++--
hw/i2c/omap_i2c.c | 2 +-
hw/i2c/pm_smbus.c | 3 ++-
hw/i2c/versatile_i2c.c| 2 +-
hw/i386/kvm/pci-assign.c | 28 +++-
h
Signed-off-by: Paolo Bonzini
---
hw/acpi/core.c | 9 ++---
hw/acpi/ich9.c | 10 +-
hw/acpi/piix4.c| 14 +++---
hw/alpha/typhoon.c | 21 -
hw/arm/armv7m.c| 4 ++--
hw/arm/highbank.c | 4 ++--
We're already using them in several places, but __sync builtins are just
too ugly to type, and do not provide seqcst load/store operations.
Reviewed-by: Richard Henderson
Signed-off-by: Paolo Bonzini
---
docs/atomics.txt | 352 +++
hw/display/
Signed-off-by: Paolo Bonzini
---
hw/display/vga-isa-mm.c | 2 +-
hw/display/vga-isa.c| 2 +-
hw/display/vga-pci.c| 2 +-
hw/display/vga.c| 4 ++--
hw/display/vga_int.h| 2 +-
hw/display/vmware_vga.c | 2 +-
6 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/hw/disp
1 - 100 of 254 matches
Mail list logo