Re: [Qemu-devel] [PATCHv2 00/13] unicore32: add softmmu support and puv3 machine

2012-06-17 Thread Wei-Ren Chen
> Yes. > But how to do that. I can't find the upload location or someone who > receives the image. Perhaps someone can open a wiki account for you, so that you can upload an image. Regards, chenwj -- Wei-Ren Chen (陳韋任) Computer Systems Lab, Institute of Information Science, Academia Sinica, T

Re: [Qemu-devel] How to measure guest memory access (qemu_ld/qemu_st) time?

2012-06-17 Thread Wei-Ren Chen
> The idea looks nice, but instead of different TLB functions selected > at configure time, the optimization should be enabled by default. > > Maybe a 'call' instruction could be used to jump to the slow path, > that way the slow path could be shared. I don't understand what "maybe a 'call' ins

[Qemu-devel] [Bug 538908] Re: qemu-system-cris crashes after a few seconds

2012-06-17 Thread Ken Sharp
Fixed in Natty+ ** Changed in: qemu (Ubuntu) Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/538908 Title: qemu-system-cris crashes after a few seconds Status in

Re: [Qemu-devel] [PATCH] Fix for qemu crash on assertion error when adding PCI passthru device.

2012-06-17 Thread Jan Kiszka
On 2012-06-17 16:28, Anthony Liguori wrote: > On 06/17/2012 03:34 AM, Michael S. Tsirkin wrote: >> On Sun, Jun 17, 2012 at 06:26:33AM +, Ma, Stephen B. wrote: >>> >>> Michael, >>> >>> Thanks for the review. I added the unparent to the qdev_free. >>> >>> >>> --- >>> hw/qdev.c |1 + >>> 1

Re: [Qemu-devel] How to measure guest memory access (qemu_ld/qemu_st) time?

2012-06-17 Thread YeongKyoon Lee
> The idea looks nice, but instead of different TLB functions selected > at configure time, the optimization should be enabled by default. > > Maybe a 'call' instruction could be used to jump to the slow path, > that way the slow path could be shared. I had considered the approach of sharing slow

Re: [Qemu-devel] [PATCH v1 3/3] qom: Converged dynamic cast for interfaces & objs

2012-06-17 Thread Peter Crosthwaite
On Sat, Jun 16, 2012 at 8:39 PM, Peter A. G. Crosthwaite wrote: > Interfaces and Object for some reason have seperate code paths for dynamic > casting. AFAICT, Anthonys new interface specific cast is a functional superset > of the object cast, so this patch renames the interface cast to object cas

Re: [Qemu-devel] hw/esp.c mishandles commands without dma

2012-06-17 Thread Martin Husemann
On Sun, Jun 17, 2012 at 11:55:20PM +0200, Martin Husemann wrote: > The NetBSD driver sometimes uses commands without DMA (for example a simple > TEST_UNIT_READY). On esp hardware, the command has a DMA bit (if dma is to > be used), and when writing to the command register, the s->dma status is > up

[Qemu-devel] hw/esp.c mishandles commands without dma

2012-06-17 Thread Martin Husemann
The NetBSD driver sometimes uses commands without DMA (for example a simple TEST_UNIT_READY). On esp hardware, the command has a DMA bit (if dma is to be used), and when writing to the command register, the s->dma status is updated accordingly. When commands with dma are used and dma is disabled,

Re: [Qemu-devel] [PATCHv2 1/1] Add usb option in machine options to enable/disable usb

2012-06-17 Thread Li Zhang
On Fri, Jun 15, 2012 at 10:34 PM, Markus Armbruster wrote: > Li Zhang writes: > >> On Fri, Jun 15, 2012 at 8:04 PM, Markus Armbruster wrote: >>> Li Zhang writes: >>> For pseries machine, it needs to enable usb to add keyboard or usb mouse. -usb option won't be used in the future,

Re: [Qemu-devel] [Qemu-ppc] [PATCHv3 1/1] Add usb option in machine options to enable/disable usb

2012-06-17 Thread Li Zhang
On Sat, Jun 16, 2012 at 6:51 AM, Benjamin Herrenschmidt wrote: > On Fri, 2012-06-15 at 22:40 -0400, Li Zhang wrote: >> For pseries machine, it needs to enable usb to add >> keyboard or usb mouse. -usb option won't be used in >> the future, and machine options is a better way to >> enable usb. > >

Re: [Qemu-devel] [PATCHv2 02/13] unicore32-softmmu: Add coprocessor 0(sysctrl) and 1(ocd) instruction support

2012-06-17 Thread Guan Xuetao
On Fri, 2012-06-15 at 13:13 +0200, Andreas Färber wrote: > Am 15.06.2012 11:47, schrieb Guan Xuetao: > > Coprocessor 0 is system control coprocessor, and we need get/set its > > contents. > > Also, all cache/tlb ops shoule be implemented here, but just ignored with > > no harm. > > > > Coprocess

Re: [Qemu-devel] [PATCH 3/6] add-cow file format

2012-06-17 Thread Dong Xu Wang
On Thu, Jun 14, 2012 at 7:13 PM, Paolo Bonzini wrote: > I just took a quick look at the flush code. > > Il 13/06/2012 16:36, Dong Xu Wang ha scritto: >> >> +bool add_cow_cache_set_writethrough(BlockDriverState *bs, AddCowCache *c, >> +    bool enable) >> +{ >> +    bool old = c->writethrough; >> +

[Qemu-devel] [PATCH v5 12/16] target-or32: Add system instructions

2012-06-17 Thread Jia Liu
Add OpenRISC system instruction support. Signed-off-by: Jia Liu --- target-openrisc/Makefile.objs |3 +- target-openrisc/helper.h |4 + target-openrisc/sys_helper.c | 233 + target-openrisc/translate.c | 20 4 files changed, 259 ins

Re: [Qemu-devel] [PATCH 1/6 v10] docs: spec for add-cow file format

2012-06-17 Thread Dong Xu Wang
On Thu, Jun 14, 2012 at 6:47 PM, Kevin Wolf wrote: > Am 14.06.2012 05:06, schrieb Dong Xu Wang: >> On Wed, Jun 13, 2012 at 11:10 PM, Eric Blake wrote: >>> On 06/13/2012 08:36 AM, Dong Xu Wang wrote: Introduce a new file format:add-cow. The usage can be found at this patch. Signed-o

[Qemu-devel] [PATCH v5 04/16] target-or32: Add interrupt support

2012-06-17 Thread Jia Liu
Add OpenRISC interrupt support. Signed-off-by: Jia Liu --- cpu-exec.c | 17 + target-openrisc/Makefile.objs |2 +- target-openrisc/cpu.h |9 ++- target-openrisc/helper.h| 25 +++ target-openrisc/intrpt.c

[Qemu-devel] [PATCH v5 14/16] target-or32: Add linux syscall, signal and termbits

2012-06-17 Thread Jia Liu
Add linux syscall, signal and termbits. Signed-off-by: Jia Liu --- linux-user/openrisc/syscall.h | 24 ++ linux-user/openrisc/syscall_nr.h| 506 +++ linux-user/openrisc/target_signal.h | 26 ++ linux-user/openrisc/termbits.h | 294

Re: [Qemu-devel] [PATCHv2 00/13] unicore32: add softmmu support and puv3 machine

2012-06-17 Thread Guan Xuetao
On Fri, 2012-06-15 at 18:17 +0800, 陳韋任 (Wei-Ren Chen) wrote: > On Fri, Jun 15, 2012 at 05:47:33PM +0800, Guan Xuetao wrote: > > These patches implement softmmu support on unicore32 architecture. > > > > UniCore32 CPU is embedded in PKUnity-3 SoC, so we add necessary puv3 > > devices simulation cod

[Qemu-devel] [PATCH v5 10/16] target-or32: Add timer support

2012-06-17 Thread Jia Liu
Add OpenRISC timer support. Signed-off-by: Jia Liu --- hw/openrisc_timer.c | 130 +++ 1 file changed, 130 insertions(+) diff --git a/hw/openrisc_timer.c b/hw/openrisc_timer.c index df384f6..25cde1a 100644 --- a/hw/openrisc_timer.c +++ b/hw/openri

Re: [Qemu-devel] [PATCHv2 05/13] unicore32-softmmu: Make sure that kernel can access user space

2012-06-17 Thread Guan Xuetao
On Fri, 2012-06-15 at 18:01 +0800, 陳韋任 (Wei-Ren Chen) wrote: > Hi Guan, > > > /* These instructions trap after executing, so defer them until after the > > conditional executions state has been updated. */ > > @@ -1551,12 +1558,12 @@ static void do_misc(CPUUniCore32State *env, > > DisasCont

[Qemu-devel] [PATCH v5 03/16] target-or32: Add MMU support

2012-06-17 Thread Jia Liu
Add OpenRISC MMU support. Signed-off-by: Jia Liu --- target-openrisc/cpu.h| 75 +++- target-openrisc/mmu.c| 199 +- target-openrisc/mmu_helper.c | 20 + 3 files changed, 292 insertions(+), 2 deletions(-) diff --git a/t

[Qemu-devel] [PATCH v5 16/16] target-or32: Add testcases

2012-06-17 Thread Jia Liu
Add testcases. Signed-off-by: Jia Liu --- tests/tcg/openrisc/Makefile | 71 +++ tests/tcg/openrisc/test_add.c| 34 + tests/tcg/openrisc/test_addc.c | 39 +++ tests/tcg/openrisc/test_addi.c | 31 tests/tcg/openrisc/

[Qemu-devel] [PATCH v5 15/16] target-or32: Add linux user support

2012-06-17 Thread Jia Liu
Add OpenRISC linux user support. Signed-off-by: Jia Liu --- configure |1 + default-configs/or32-linux-user.mak |2 + linux-user/elfload.c| 41 +++ linux-user/main.c | 100 +++ linux-user/signal.c

[Qemu-devel] [PATCH v5 09/16] target-or32: Add PIC support

2012-06-17 Thread Jia Liu
Add OpenRISC Programmable Interrupt Controller. Signed-off-by: Jia Liu --- hw/openrisc_pic.c | 48 1 file changed, 48 insertions(+) diff --git a/hw/openrisc_pic.c b/hw/openrisc_pic.c index 0d14bbe..76bd792 100644 --- a/hw/openrisc_pic.c +++ b/h

[Qemu-devel] [PATCH v5 11/16] target-or32: Add a IIS dummy board

2012-06-17 Thread Jia Liu
Add a dummy board for IIS. Signed-off-by: Jia Liu --- hw/openrisc/Makefile.objs |2 +- hw/openrisc_sim.c | 145 + 2 files changed, 146 insertions(+), 1 deletion(-) create mode 100644 hw/openrisc_sim.c diff --git a/hw/openrisc/Makefile.ob

Re: [Qemu-devel] please remind me if you have ARM patches awaiting review

2012-06-17 Thread Peter Crosthwaite
My little arm-boot series which I think you reviewed just before you left for holiday has been resent as well, Regards, Peter

[Qemu-devel] [PATCH v2 2/2] arm_boot: Conditionalised DTB command line update

2012-06-17 Thread Peter A. G. Crosthwaite
The DTB command line should only be overwritten if the user provides a command line with -apend. Otherwise whatever command line was in the DTB should stay unchanged. Signed-off-by: Peter A. G. Crosthwaite --- changed since v1: checked cmd line string in binfo rather than machine opt hw/arm_boo

[Qemu-devel] [PATCH v2 1/2] arm_boot: Assume Linux boot flow when -dtb given

2012-06-17 Thread Peter A. G. Crosthwaite
If the user boots with a -dtb assume the Linux boot flow, even when handling an elf. Signed-off-by: Peter A. G. Crosthwaite --- hw/arm_boot.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/hw/arm_boot.c b/hw/arm_boot.c index 7447f5c..f0fa23c 100644 --- a/hw/arm_boot

[Qemu-devel] [PATCH v2 0/2] These two patches add some flexibilities to the arm bootloader.

2012-06-17 Thread Peter A. G. Crosthwaite
Set up the the bootloader so that if -dtb is specified, then the Linux bootflow is assumed. Conditionalised dtb command line update on there being a -append argument. Means you can use a dtb with a command line already present and boot without QEMU nuking your command line. The patches are ind

[Qemu-devel] [PATCH v5 07/16] target-or32: Add float instruction helpers

2012-06-17 Thread Jia Liu
Add OpenRISC float instruction helpers. Signed-off-by: Jia Liu --- target-openrisc/Makefile.objs |2 +- target-openrisc/fpu_helper.c | 275 + target-openrisc/helper.h | 33 + 3 files changed, 309 insertions(+), 1 deletion(-) create mode 1

[Qemu-devel] [PATCH v5 13/16] target-or32: Add gdb stub support

2012-06-17 Thread Jia Liu
Add QEMU OpenRISC gdb stub support. Signed-off-by: Jia Liu --- gdbstub.c | 64 + 1 file changed, 64 insertions(+) diff --git a/gdbstub.c b/gdbstub.c index 08cf864..5d37dd9 100644 --- a/gdbstub.c +++ b/gdbstub.c @@ -1155,6 +1155,68 @@

[Qemu-devel] [PATCH v5 06/16] target-or32: Add int instruction helpers

2012-06-17 Thread Jia Liu
Add OpenRISC int instruction helpers. Signed-off-by: Jia Liu --- target-openrisc/Makefile.objs |2 +- target-openrisc/helper.h |4 target-openrisc/int_helper.c | 51 + 3 files changed, 56 insertions(+), 1 deletion(-) create mode 10064

[Qemu-devel] [PATCH v5 05/16] target-or32: Add exception support

2012-06-17 Thread Jia Liu
Add OpenRISC exception support. Signed-off-by: Jia Liu --- target-openrisc/Makefile.objs |4 ++-- target-openrisc/excp.c| 27 +++ target-openrisc/excp.h| 28 target-openrisc/excp_helper.c | 27

[Qemu-devel] [PATCH v5 01/16] target-or32: Add target stubs and cpu support

2012-06-17 Thread Jia Liu
Add OpenRISC target stubs and basic cpu support. Signed-off-by: Jia Liu --- arch_init.c |2 + arch_init.h |1 + configure| 14 +- cpu-exec.c |2 + default-configs/or32-softmmu.mak |6 + elf.

[Qemu-devel] [PATCH v5 02/16] target-or32: Add target machine

2012-06-17 Thread Jia Liu
Add OpenRISC target machine. Signed-off-by: Jia Liu --- target-openrisc/cpu.h | 67 - target-openrisc/machine.c | 22 ++- 2 files changed, 87 insertions(+), 2 deletions(-) diff --git a/target-openrisc/cpu.h b/target-openrisc/cpu.h

[Qemu-devel] [PATCH v5 00/16] QEMU OpenRISC support

2012-06-17 Thread Jia Liu
This is the OpenCores OpenRISC 1200 support for QEMU. Full implementation of the system-model and linux-user-model support. OpenRISC 1200 is a OpenCores open source CPU, its architecture manual can be found at http://opencores.org/svnget,or1k?file=/trunk/docs/openrisc_arch.pdf A OpenRISC Linux ke

[Qemu-devel] [PATCH] arm_gic: Send dbg msgs to stderr not stdout

2012-06-17 Thread Peter A. G. Crosthwaite
Signed-off-by: Peter A. G. Crosthwaite --- hw/arm_gic.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/arm_gic.c b/hw/arm_gic.c index 72298b4..c78d58e 100644 --- a/hw/arm_gic.c +++ b/hw/arm_gic.c @@ -35,7 +35,7 @@ #ifdef DEBUG_GIC #define DPRINTF(fmt, ...) \ -do

[Qemu-devel] [PATCH v4 1/2] pl330: initial version

2012-06-17 Thread Peter A. G. Crosthwaite
Device model for Primecell PL330 dma controller. Signed-off-by: Peter A. G. Crosthwaite Signed-off-by: Kirill Batuzov --- changed from v3: rebased against Makefile refactor changed from v2 (in order of diff appearance): GPL version changed to v2 or later Register and field names changed to match

[Qemu-devel] [PATCH v4 2/2] xilinx_zynq: added pl330 to machine model

2012-06-17 Thread Peter A. G. Crosthwaite
Signed-off-by: Peter A. G. Crosthwaite --- hw/xilinx_zynq.c | 18 ++ 1 files changed, 18 insertions(+), 0 deletions(-) diff --git a/hw/xilinx_zynq.c b/hw/xilinx_zynq.c index 7e6c273..c20a896 100644 --- a/hw/xilinx_zynq.c +++ b/hw/xilinx_zynq.c @@ -61,6 +61,9 @@ static void zynq

Re: [Qemu-devel] please remind me if you have ARM patches awaiting review

2012-06-17 Thread Peter Crosthwaite
On Mon, Jun 18, 2012 at 12:30 AM, Peter Maydell wrote: > On 17 June 2012 15:23, Andreas Färber wrote: >> Am 17.06.2012 14:39, schrieb Peter Maydell: >>> Hi. Due to a combination of the QEMU release freeze and my taking >>> a nice long holiday, I have rather lost track of the current status >>> of

Re: [Qemu-devel] [PATCH] make: automatically include dependencies in recursive subdir rules (v2)

2012-06-17 Thread Andreas Färber
Am 18.06.2012 02:01, schrieb Anthony Liguori: > I think I understand enough of what's going on in these rules to ensure this > is > right. But I could certainly use a second or third opinion... > > Signed-off-by: Anthony Liguori > --- > v1 -> v2 > - Remove unnecessary includes (Andreas) > - A

[Qemu-devel] [PATCH v4 0/2] Xilinx Zynq PL330 support

2012-06-17 Thread Peter A. G. Crosthwaite
These patches add support for the Primcell PL330 DMA controller and add it to the Xilinx Zynq machine model. Patch 1 is the device model. Patch 2 is the machine model update. The Device model was originally contributed by Kirill Batuzov / Samsung, as indicated by the (C) notice in hw/pl330.c.

Re: [Qemu-devel] [PATCH] make: automatically include dependencies in recursive subdir rules

2012-06-17 Thread Anthony Liguori
On 06/17/2012 02:53 PM, Andreas Färber wrote: Am 17.06.2012 20:46, schrieb Anthony Liguori: On 06/17/2012 12:00 PM, Andreas Färber wrote: If using hw/kvm/*.d instead, it actually works. ;) We can also just add a hw/kvm/Makefile.objs and then add kvm/ to the subdir rules. Seems to fix the pro

[Qemu-devel] [PATCH] make: automatically include dependencies in recursive subdir rules (v2)

2012-06-17 Thread Anthony Liguori
I think I understand enough of what's going on in these rules to ensure this is right. But I could certainly use a second or third opinion... Signed-off-by: Anthony Liguori --- v1 -> v2 - Remove unnecessary includes (Andreas) - Add a sub makefile for hw/kvm (Andreas) --- Makefile

[Qemu-devel] [Bug 1014099] Re: hw/esp.c does not properly deal with TEST_UNIT_READY in NetBSD/sparc

2012-06-17 Thread Martin Husemann
Patch removed, as it was bogus and your workflow is weird, so I'll post a better patch to the devel list ** Patch removed: "esp.c.patch" https://bugs.launchpad.net/qemu/+bug/1014099/+attachment/3192942/+files/esp.c.patch -- You received this bug notification because you are a member of qemu-

Re: [Qemu-devel] [PATCH v4 00/16] QEMU OpenRISC support

2012-06-17 Thread Max Filippov
On 06/11/2012 10:31 AM, Jia Liu wrote: This is the OpenCores OpenRISC 1200 support for QEMU. Full implementation of the system-model and linux-user-model support. Hi Jia. When I configure the tree with openrisc patches for the debug build: $ configure --target-list='or32-softmmu,or32-linux-us

[Qemu-devel] [PATCH] Fixes related to processing of qemu's -numa option

2012-06-17 Thread Chegu Vinod
The -numa option to qemu is used to create [fake] numa nodes and expose them to the guest OS instance. There are a couple of issues with the -numa option: a) Max VCPU's that can be specified for a guest while using the qemu's -numa option is 64. Due to a typecasting issue when the number of

Re: [Qemu-devel] [PATCH] make: automatically include dependencies in recursive subdir rules

2012-06-17 Thread Andreas Färber
Am 17.06.2012 20:46, schrieb Anthony Liguori: > On 06/17/2012 12:00 PM, Andreas Färber wrote: >> Am 17.06.2012 18:23, schrieb Andreas Färber: >>> I've tested with just >>> +$(eval -include $(wildcard $1/*.d)) >>> and did >>> $ touch include/qemu/object.h >>> >>> From what I see, using Anthony's or

Re: [Qemu-devel] [PATCH] make: automatically include dependencies in recursive subdir rules

2012-06-17 Thread Anthony Liguori
On 06/17/2012 12:00 PM, Andreas Färber wrote: Am 17.06.2012 18:23, schrieb Andreas Färber: I've tested with just +$(eval -include $(wildcard $1/*.d)) and did $ touch include/qemu/object.h From what I see, using Anthony's original patch plus the modification above (not the other ones), {i386,x8

Re: [Qemu-devel] [PATCH] make: automatically include dependencies in recursive subdir rules

2012-06-17 Thread Andreas Färber
Am 17.06.2012 18:23, schrieb Andreas Färber: > I've tested with just > +$(eval -include $(wildcard $1/*.d)) > and did > $ touch include/qemu/object.h > > From what I see, using Anthony's original patch plus the modification > above (not the other ones), {i386,x86_64}-softmmu/hw/kvm/apic.o is still

Re: [Qemu-devel] [PATCH] make: automatically include dependencies in recursive subdir rules

2012-06-17 Thread Andreas Färber
Am 17.06.2012 17:20, schrieb Blue Swirl: > On Sun, Jun 17, 2012 at 2:57 PM, Anthony Liguori wrote: >> I think I understand enough of what's going on in these rules to ensure this >> is >> right. But I could certainly use a second or third opinion... >> >> Signed-off-by: Anthony Liguori >> --- >

Re: [Qemu-devel] [PATCH] make: automatically include dependencies in recursive subdir rules

2012-06-17 Thread Andreas Färber
Am 17.06.2012 16:57, schrieb Anthony Liguori: > I think I understand enough of what's going on in these rules to ensure this > is > right. But I could certainly use a second or third opinion... > > Signed-off-by: Anthony Liguori > --- > Makefile |4 ++-- > rules.mak |1 + > 2 files ch

Re: [Qemu-devel] [PATCH] make: automatically include dependencies in recursive subdir rules

2012-06-17 Thread Blue Swirl
On Sun, Jun 17, 2012 at 2:57 PM, Anthony Liguori wrote: > I think I understand enough of what's going on in these rules to ensure this > is > right.  But I could certainly use a second or third opinion... > > Signed-off-by: Anthony Liguori > --- >  Makefile  |    4 ++-- >  rules.mak |    1 + >  

Re: [Qemu-devel] [qom-next] Bisecting virtio-scsi issue

2012-06-17 Thread Andreas Färber
Am 17.06.2012 16:57, schrieb Avi Kivity: > On 06/17/2012 05:33 PM, Anthony Liguori wrote: > >>> (gdb) qemu mtree >>> -7ffe system (@ 0x564627b0) >>> - io (@ 0x56462880) >>>007e-007f kvmvapic (I/O) (@ 0x

Re: [Qemu-devel] [qom-next] Bisecting virtio-scsi issue

2012-06-17 Thread Avi Kivity
On 06/17/2012 05:33 PM, Anthony Liguori wrote: >> (gdb) qemu mtree >> -7ffe system (@ 0x564627b0) >> - io (@ 0x56462880) >>007e-007f kvmvapic (I/O) (@ 0x5647c838) > > FWIW: > > > commit 55016e6ab

[Qemu-devel] [PATCH] make: automatically include dependencies in recursive subdir rules

2012-06-17 Thread Anthony Liguori
I think I understand enough of what's going on in these rules to ensure this is right. But I could certainly use a second or third opinion... Signed-off-by: Anthony Liguori --- Makefile |4 ++-- rules.mak |1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/Makefile b/M

Re: [Qemu-devel] [qom-next] Bisecting virtio-scsi issue

2012-06-17 Thread Anthony Liguori
On 06/17/2012 09:28 AM, Andreas Färber wrote: Am 17.06.2012 16:16, schrieb Avi Kivity: On 06/17/2012 05:08 PM, Andreas Färber wrote: Am 17.06.2012 10:37, schrieb Avi Kivity: On 06/16/2012 03:41 AM, Andreas Färber wrote: After multiple runs of not-so-successful bisecting, it appears as if the

Re: [Qemu-devel] please remind me if you have ARM patches awaiting review

2012-06-17 Thread Peter Maydell
On 17 June 2012 15:23, Andreas Färber wrote: > Am 17.06.2012 14:39, schrieb Peter Maydell: >> Hi. Due to a combination of the QEMU release freeze and my taking >> a nice long holiday, I have rather lost track of the current status >> of pending ARM patches. > > A short summary of recent activity b

Re: [Qemu-devel] [qom-next] Bisecting virtio-scsi issue

2012-06-17 Thread Andreas Färber
Am 17.06.2012 16:16, schrieb Avi Kivity: > On 06/17/2012 05:08 PM, Andreas Färber wrote: >> Am 17.06.2012 10:37, schrieb Avi Kivity: >>> On 06/16/2012 03:41 AM, Andreas Färber wrote: After multiple runs of not-so-successful bisecting, it appears as if the qom-next commit "qbus: Make child

Re: [Qemu-devel] [PATCH] Fix for qemu crash on assertion error when adding PCI passthru device.

2012-06-17 Thread Anthony Liguori
On 06/17/2012 03:34 AM, Michael S. Tsirkin wrote: On Sun, Jun 17, 2012 at 06:26:33AM +, Ma, Stephen B. wrote: Michael, Thanks for the review. I added the unparent to the qdev_free. --- hw/qdev.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/hw/qdev.c b/hw/q

Re: [Qemu-devel] please remind me if you have ARM patches awaiting review

2012-06-17 Thread Andreas Färber
Am 17.06.2012 14:39, schrieb Peter Maydell: > Hi. Due to a combination of the QEMU release freeze and my taking > a nice long holiday, I have rather lost track of the current status > of pending ARM patches. A short summary of recent activity before you get bombarded with links: * QOM CPUState cp

Re: [Qemu-devel] [qom-next] Bisecting virtio-scsi issue

2012-06-17 Thread Avi Kivity
On 06/17/2012 05:08 PM, Andreas Färber wrote: > Am 17.06.2012 10:37, schrieb Avi Kivity: >> On 06/16/2012 03:41 AM, Andreas Färber wrote: >>> Hi, >>> >>> After multiple runs of not-so-successful bisecting, it appears as if the >>> qom-next commit "qbus: Make child devices links" [1] is causing >>>

Re: [Qemu-devel] [qom-next] Bisecting virtio-scsi issue

2012-06-17 Thread Andreas Färber
Am 17.06.2012 10:37, schrieb Avi Kivity: > On 06/16/2012 03:41 AM, Andreas Färber wrote: >> Hi, >> >> After multiple runs of not-so-successful bisecting, it appears as if the >> qom-next commit "qbus: Make child devices links" [1] is causing >> assertions for both virtio-scsi and ahci but not for q

Re: [Qemu-devel] [PATCH v3 00/25] x86 AREG0 conversion

2012-06-17 Thread Andreas Färber
Am 17.06.2012 12:29, schrieb Blue Swirl: > On Sat, Jun 16, 2012 at 1:15 PM, Andreas Färber wrote: >> Am 16.06.2012 08:51, schrieb Blue Swirl: >>> On Sat, Jun 16, 2012 at 12:59 AM, Andreas Färber wrote: Am 09.06.2012 18:18, schrieb Blue Swirl: > v3: Rebased due to Makefile changes. If the

Re: [Qemu-devel] [PATCH] kvm: align ram_size to page boundary

2012-06-17 Thread Avi Kivity
On 06/17/2012 04:06 PM, Blue Swirl wrote: >> strtosz() is much too general. We could do it in vl.c without trouble. >> However, it takes away our ability to emulate a "640k should be enough >> for everyone" machine. > > Then how about current max of target page sizes: 8k? No machine should > wa

Re: [Qemu-devel] [PATCH] kvm: align ram_size to page boundary

2012-06-17 Thread Blue Swirl
On Sun, Jun 17, 2012 at 12:54 PM, Avi Kivity wrote: > On 06/17/2012 03:43 PM, Blue Swirl wrote: >> On Sun, Jun 17, 2012 at 11:51 AM, Avi Kivity wrote: >>> On 06/17/2012 02:47 PM, Jan Kiszka wrote: >> >> I think this should rather go into generic code. > > To be honest, I put this

Re: [Qemu-devel] [PATCH v3 00/25] x86 AREG0 conversion

2012-06-17 Thread Blue Swirl
On Sun, Jun 17, 2012 at 10:29 AM, Blue Swirl wrote: > On Sat, Jun 16, 2012 at 1:15 PM, Andreas Färber wrote: >> Am 16.06.2012 08:51, schrieb Blue Swirl: >>> On Sat, Jun 16, 2012 at 12:59 AM, Andreas Färber wrote: Am 09.06.2012 18:18, schrieb Blue Swirl: > v3: Rebased due to Makefile cha

Re: [Qemu-devel] [PATCH] kvm: align ram_size to page boundary

2012-06-17 Thread Avi Kivity
On 06/17/2012 03:43 PM, Blue Swirl wrote: > On Sun, Jun 17, 2012 at 11:51 AM, Avi Kivity wrote: >> On 06/17/2012 02:47 PM, Jan Kiszka wrote: > > I think this should rather go into generic code. To be honest, I put this in kvm-specific code because vl.c doesn't have TARGET_PA

Re: [Qemu-devel] [PATCH] kvm: align ram_size to page boundary

2012-06-17 Thread Peter Maydell
On 17 June 2012 13:43, Blue Swirl wrote: > The boards should make sure that the amount of RAM is feasible with > the board memory slots. It's not possible to put 256kb SIMMs to a slot > that expects 1GB DIMMs. We can allow some flexibility there though, > I'm not sure if the current chipsets would

Re: [Qemu-devel] [PATCH] kvm: align ram_size to page boundary

2012-06-17 Thread Blue Swirl
On Sun, Jun 17, 2012 at 11:51 AM, Avi Kivity wrote: > On 06/17/2012 02:47 PM, Jan Kiszka wrote: I think this should rather go into generic code. >>> >>> To be honest, I put this in kvm-specific code because vl.c doesn't have >>> TARGET_PAGE_ALIGN.  Maybe we should have machine->page_size

[Qemu-devel] please remind me if you have ARM patches awaiting review

2012-06-17 Thread Peter Maydell
Hi. Due to a combination of the QEMU release freeze and my taking a nice long holiday, I have rather lost track of the current status of pending ARM patches. I am therefore declaring "patch bankruptcy": if you have a patchset you want me to review or commit to target-arm.next/arm-devs.next, please

Re: [Qemu-devel] [PATCH] kvm: align ram_size to page boundary

2012-06-17 Thread Avi Kivity
On 06/17/2012 02:47 PM, Jan Kiszka wrote: >>> >>> I think this should rather go into generic code. >> >> To be honest, I put this in kvm-specific code because vl.c doesn't have >> TARGET_PAGE_ALIGN. Maybe we should have machine->page_size or >> machine->ram_alignment. >> >>> What sense does it

Re: [Qemu-devel] [PATCH] kvm: align ram_size to page boundary

2012-06-17 Thread Jan Kiszka
On 2012-06-17 13:30, Avi Kivity wrote: > On 06/17/2012 02:03 PM, Jan Kiszka wrote: >> On 2012-06-17 10:47, Avi Kivity wrote: >>> kvm is not able to execute out of partial pages; align the RAM size >>> so partial pages aren't present. >>> >>> Reported-by: Michael Tokarev >>> Signed-off-by: Avi Kivi

Re: [Qemu-devel] [PATCH] kvm: align ram_size to page boundary

2012-06-17 Thread Avi Kivity
On 06/17/2012 02:03 PM, Jan Kiszka wrote: > On 2012-06-17 10:47, Avi Kivity wrote: >> kvm is not able to execute out of partial pages; align the RAM size >> so partial pages aren't present. >> >> Reported-by: Michael Tokarev >> Signed-off-by: Avi Kivity >> --- >> kvm-all.c |2 ++ >> 1 file

Re: [Qemu-devel] [PATCH] kvm: align ram_size to page boundary

2012-06-17 Thread Jan Kiszka
On 2012-06-17 10:47, Avi Kivity wrote: > kvm is not able to execute out of partial pages; align the RAM size > so partial pages aren't present. > > Reported-by: Michael Tokarev > Signed-off-by: Avi Kivity > --- > kvm-all.c |2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/kvm-all.c

Re: [Qemu-devel] [PATCH v3 00/25] x86 AREG0 conversion

2012-06-17 Thread Blue Swirl
On Sat, Jun 16, 2012 at 1:15 PM, Andreas Färber wrote: > Am 16.06.2012 08:51, schrieb Blue Swirl: >> On Sat, Jun 16, 2012 at 12:59 AM, Andreas Färber wrote: >>> Am 09.06.2012 18:18, schrieb Blue Swirl: v3: Rebased due to Makefile changes. If there are no objections, I'll apply this next

Re: [Qemu-devel] [Bug 1014099] Re: hw/esp.c does not properly deal with TEST_UNIT_READY in NetBSD/sparc

2012-06-17 Thread Stefan Hajnoczi
On Sat, Jun 16, 2012 at 5:50 PM, Martin Husemann wrote: > ** Patch added: "esp.c.patch" >   > https://bugs.launchpad.net/bugs/1014099/+attachment/3192643/+files/esp.c.patch Please see this on how to contribute patches to QEMU: http://wiki.qemu.org/Contribute/SubmitAPatch Stefan

[Qemu-devel] [PATCH] kvm: align ram_size to page boundary

2012-06-17 Thread Avi Kivity
kvm is not able to execute out of partial pages; align the RAM size so partial pages aren't present. Reported-by: Michael Tokarev Signed-off-by: Avi Kivity --- kvm-all.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/kvm-all.c b/kvm-all.c index 4ea7d85..482768f 100644 --- a/kvm-all.c

Re: [Qemu-devel] [PATCH v4 12/16] target-or32: Add system instructions

2012-06-17 Thread Jia Liu
HI Blue, Thank you very much. On Thu, Jun 14, 2012 at 2:35 AM, Blue Swirl wrote: > On Mon, Jun 11, 2012 at 6:32 AM, Jia Liu wrote: >> Add OpenRISC system instruction helpers and translation. >> >> Signed-off-by: Jia Liu >> --- >>  target-openrisc/Makefile.objs |    3 +- >>  target-openrisc/hel

Re: [Qemu-devel] [qom-next] Bisecting virtio-scsi issue

2012-06-17 Thread Avi Kivity
On 06/16/2012 03:41 AM, Andreas Färber wrote: > Hi, > > After multiple runs of not-so-successful bisecting, it appears as if the > qom-next commit "qbus: Make child devices links" [1] is causing > assertions for both virtio-scsi and ahci but not for qemu-system-x86_64 > without parameters: > > $

Re: [Qemu-devel] [PATCH] Fix for qemu crash on assertion error when adding PCI passthru device.

2012-06-17 Thread Michael S. Tsirkin
On Sun, Jun 17, 2012 at 06:26:33AM +, Ma, Stephen B. wrote: > > Michael, > > Thanks for the review. I added the unparent to the qdev_free. > > > --- > hw/qdev.c |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/hw/qdev.c b/hw/qdev.c > index d2dc28b..ed1328d 10

Re: [Qemu-devel] KVM internal error with some amounts of guest memory

2012-06-17 Thread Avi Kivity
On 06/14/2012 11:06 PM, Michael Tokarev wrote: > On 14.06.2012 23:45, Michael Tokarev wrote: >> On 14.06.2012 23:22, Michael Tokarev wrote: >>> Now that's something else. Reported by a debian user, but >>> trivially reproducible. >>> >>> $ kvm -m 1.4g >>> KVM internal error. Suberror: 1 >>> emulat

Re: [Qemu-devel] q35 chipset support

2012-06-17 Thread Michael S. Tsirkin
On Fri, Jun 15, 2012 at 12:58:33PM -0500, Anthony Liguori wrote: > The Q35 is much more sophisticated. The PCI-e complex itself can > present interesting topologies and the legacy PCI bus sits within > the PCI-e complex. Ah, so we can mix in PCI as well? Cool. How does such a mixed topology look?