Re: [Qemu-devel] [Qemu-ppc] [PATCH v2 0/5] QOM'ify Power Architecture CPU

2012-04-09 Thread David Gibson
On Fri, Apr 06, 2012 at 06:17:07PM +0200, Andreas Färber wrote: > Hello, > > This series follows up on my PowerPC QOM'ification patches from the > qom-cpu-others.v1 RFC series and splits it into steps easier to review. > The finalizer is actually filled with life now. Subclasses are postponed. >

Re: [Qemu-devel] [patch V5 3/5] FreeSCALE i.MX31 support: Timers

2012-04-09 Thread Peter Chubb
> "Peter" == Peter Chubb writes: Peter> I'm not sure how to make qdev properties work. The timers need Peter> to be able to get at the CCM-generated frequencies, which of Peter> course are private to the CCM. I've just split the problematic function into two: iomx_timerg_create() and imx_ti

Re: [Qemu-devel] [PATCH V2 10/10] hw/sd.c: introduce SD card "image" property and allow SD hot-insert

2012-04-09 Thread Paolo Bonzini
Il 09/04/2012 14:42, Luiz Capitulino ha scritto: >> > >> > There is a problem here however. QOM commands are HMP only, and >> > drive_add is QMP only. > I think you meant QOM commands are QMP only, and drive_add is HMP only? Yes. >> > I think that blocking drive_add in QMP is >> > perfect bein

Re: [Qemu-devel] [PATCH V2 10/10] hw/sd.c: introduce SD card "image" property and allow SD hot-insert

2012-04-09 Thread Paolo Bonzini
Il 09/04/2012 16:30, Igor Mitsyanko ha scritto: >>> + >>> +if (error_is_set(errp)) { >>> +return; >>> +} >>> + >>> +if (sd->bdrv) { >>> +qmp_change_blockdev(bdrv_get_device_name(sd->bdrv), new_image, >>> +false, NULL, errp); >>> +} else { >>> +

[Qemu-devel] [PATCH] petalogix_s3adsp1800: deleted bad FIXME comment

2012-04-09 Thread Peter A. G. Crosthwaite
This FIXME has already been actioned. Deleted comment. Signed-off-by: Peter A. G. Crosthwaite --- hw/petalogix_s3adsp1800_mmu.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/hw/petalogix_s3adsp1800_mmu.c b/hw/petalogix_s3adsp1800_mmu.c index ff154c7..8b37336 100644 ---

Re: [Qemu-devel] Qemu as a library?

2012-04-09 Thread Jun Koi
On Tue, Apr 10, 2012 at 8:09 AM, Mikael wrote: > Dear list, > > I understood that earlier on there was a "libqemu" library, using which an > ordinary userspace application could allocate a qemu virtual machine and get > it in a pointer/handle representation, run it as long as it wanted to, and > i

Re: [Qemu-devel] [patch V5 3/5] FreeSCALE i.MX31 support: Timers

2012-04-09 Thread Peter Chubb
> "Peter" == Peter Maydell writes: Peter> On 3 April 2012 02:55, Peter Chubb Peter> wrote: >+/* >+ * Artificially limit tick rate to something >+ * achievable under QEMU. Otherwise, QEMU spends all >+ * its time generating timer interrupts, and there >+

Re: [Qemu-devel] [patch V5 2/5] Implement i.MX31 Clock Control Module

2012-04-09 Thread Peter Chubb
> "Peter" == Peter Maydell writes: Peter> On 3 April 2012 02:55, Peter Chubb Peter> wrote: > + > +uint32_t imx_timer_frequency(DeviceState *s, IMXClk clock); > +void imx_timer_create(const char * const name, > + const target_phys_addr_t addr, > + qem

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

2012-04-09 Thread Peter Crosthwaite
On Mon, Apr 9, 2012 at 10:42 PM, Igor Mitsyanko wrote: > On 03/30/2012 11:58 AM, Peter A. G. Crosthwaite wrote: > > Hi Peter, a few comments on your patch which would hopefully help it to be > pushed faster: > > Thanks for you review Igor, I will look into these changes for v3 which I expect to

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

2012-04-09 Thread Peter Crosthwaite
On Thu, Apr 5, 2012 at 5:07 PM, Andreas Färber wrote: > Am 30.03.2012 09:58, schrieb 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 v1: >> GPLv2 license >> some code

Re: [Qemu-devel] [PATCH v2 0/2] SDHCI for Xilinx Zynq

2012-04-09 Thread Peter Crosthwaite
Ping! This new revision of our SDHCI series addressed Igors review of V1. Igor is currently iterating through revisions of his SDHC refactorings, is that acting as a blocker on this one? Regards, Peter On Tue, Apr 3, 2012 at 4:17 PM, Peter A. G. Crosthwaite wrote: > These two patched add a devi

[Qemu-devel] Qemu as a library?

2012-04-09 Thread Mikael
Dear list, I understood that earlier on there was a "libqemu" library, using which an ordinary userspace application could allocate a qemu virtual machine and get it in a pointer/handle representation, run it as long as it wanted to, and intercept any activity from or to it in detail. This is a *

Re: [Qemu-devel] [PATCH] versatilepb: add i2c support

2012-04-09 Thread Peter Crosthwaite
On Tue, Apr 10, 2012 at 5:56 AM, Oskar Andero wrote: > On 18:45 Thu 05 Apr     , Peter Maydell wrote: >> On 2 April 2012 21:17, Oskar Andero wrote: >> > Signed-off-by: Oskar Andero >> > --- >> >  hw/versatilepb.c |   80 >> > ++ >> >  1 files c

Re: [Qemu-devel] [Bug 977391] Re: BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*]

2012-04-09 Thread Rahul
Serge, May be this screenshot (attached) will help. -Rahul N. On Tue, Apr 10, 2012 at 4:37 AM, Rahul Nair wrote: > Serge, > > These are the logs at the time when event occurred. > But unfortunately nothing helpful was there in the logs. > I am not sure about how can I provide you the proper lo

[Qemu-devel] [Bug 977391] Re: BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*] in lucid

2012-04-09 Thread Rahul
Please check the attached screenshot that I took during the issue. I cant find anything useful in the logs. -Rahul N. On Tue, Apr 10, 2012 at 3:31 AM, Christopher M. Penalver < christopher.penal...@gmx.com> wrote: > rahul, thank you for reporting this and helping make Ubuntu better. > Could you

[Qemu-devel] [Bug 977391] Re: BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*] in lucid

2012-04-09 Thread Christopher M. Penalver
rahul, thank you for reporting this and helping make Ubuntu better. Could you please capture the oops following https://wiki.ubuntu.com/KernelTeam/KernelTeamBugPolicies#Capturing_OOPs ? ** Tags added: lucid ** Changed in: linux (Ubuntu) Status: Confirmed => Incomplete -- You received thi

Re: [Qemu-devel] [Bug 977391] Re: BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*]

2012-04-09 Thread Rahul
Serge, These are the logs at the time when event occurred. But unfortunately nothing helpful was there in the logs. I am not sure about how can I provide you the proper logs even if this happened again. -Rahul N. On Tue, Apr 10, 2012 at 3:23 AM, Serge Hallyn <977...@bugs.launchpad.net>wrote: >

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2 1/5] target-ppc: Drop cpu_ppc_close()

2012-04-09 Thread David Gibson
On Fri, Apr 06, 2012 at 06:17:08PM +0200, Andreas Färber wrote: > It is unused, so avoid QOM'ifying it unneededly. > > Signed-off-by: Andreas Färber Acked-by: David Gibson -- David Gibson| I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist

[Qemu-devel] [PATCH 3/4] target-xtensa: add tests for LBEG/LEND invalidation

2012-04-09 Thread Max Filippov
Signed-off-by: Max Filippov --- tests/tcg/xtensa/test_loop.S | 50 ++ 1 files changed, 50 insertions(+), 0 deletions(-) diff --git a/tests/tcg/xtensa/test_loop.S b/tests/tcg/xtensa/test_loop.S index a5ea933..5cead47 100644 --- a/tests/tcg/xtensa/test_loo

[Qemu-devel] [PATCH 4/4] target-xtensa: add test for IBREAK invalidation

2012-04-09 Thread Max Filippov
Signed-off-by: Max Filippov --- tests/tcg/xtensa/test_break.S | 34 ++ 1 files changed, 34 insertions(+), 0 deletions(-) diff --git a/tests/tcg/xtensa/test_break.S b/tests/tcg/xtensa/test_break.S index 8a8db80..7574cbe 100644 --- a/tests/tcg/xtensa/test_break.S

[Qemu-devel] [PATCH 1/4] exec: provide tb_invalidate_phys_addr function

2012-04-09 Thread Max Filippov
Allow TB invalidation by its physical address, extract implementation from the breakpoint_invalidate function. Signed-off-by: Max Filippov --- exec-all.h |1 + exec.c |9 ++--- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/exec-all.h b/exec-all.h index fa7bdfe..3

[Qemu-devel] [PATCH 2/4] target-xtensa: fix tb invalidation for IBREAK and LOOP

2012-04-09 Thread Max Filippov
Instruction breakpoint/zero overhead loop handling code is built into TBs pointed to by IBREAKA/LEND SRs. When these or related SRs get changed TBs at virtual addresses corresponding to their old and their new values must be invalidated. Virtual address range is passed to the tb_invalidate_phys_pa

[Qemu-devel] [PATCH 0/4] Fix incorrect use of tb_invalidate_phys_page_range by target-xtensa

2012-04-09 Thread Max Filippov
Avi reported [1] that target-xtensa incorrectly supply virtual addresses to tb_invalidate_phys_page_range. Currently there's no way to invalidate all cached TBs at the given virtual address in softmmu mode. As a shortcut solution I translate virtual address to physical address using guest TLB and r

Re: [Qemu-devel] qemu-ga: possible race while suspending the guest

2012-04-09 Thread Michael Roth
On Mon, Apr 09, 2012 at 04:57:48PM -0300, Luiz Capitulino wrote: > Hi Michael, > > There's a possible race condition in the bios_supports_mode() function used > by all suspend commands in qemu-ga: if the parent process receives a SIGCHLD > while executing: > > close(pipefds[1]); > g_free(

[Qemu-devel] [Bug 977391] Re: BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*]

2012-04-09 Thread Serge Hallyn
Thanks - unfortunately they seem to be from after the event. If it happens again, please do post the new logs. And if you know how to trigger this at will, please let us know. ** Summary changed: - BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*] + BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*

[Qemu-devel] [Bug 977391] Re: BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*]

2012-04-09 Thread Rahul
** Changed in: qemu Status: New => Confirmed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/977391 Title: BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*] Status in QEMU: Confirmed Status i

Re: [Qemu-devel] [Bug 977391] Re: BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*]

2012-04-09 Thread Rahul
Serge, I attaching the logs along with this mail. -Rahul N. On Tue, Apr 10, 2012 at 1:39 AM, Serge Hallyn <977...@bugs.launchpad.net>wrote: > Quoting Rahul (rahul.n...@finicity.com): > > Is there any alternate way I can provide you the information? > > I am actually not comfortable in executing

Re: [Qemu-devel] [Bug 977391] Re: BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*]

2012-04-09 Thread Serge Hallyn
Quoting Rahul (rahul.n...@finicity.com): > Is there any alternate way I can provide you the information? > I am actually not comfortable in executing this command on the system. The most important information given the nature of the bug would be the configuration (in particular what is sitting und

[Qemu-devel] [Bug 977391] Re: BUG: soft lockup - CPU#8 stuck for 61s! [kvm:*]

2012-04-09 Thread Rahul
** Changed in: qemu-kvm (Ubuntu) Status: Incomplete => Confirmed ** Changed in: linux (Ubuntu) Status: Incomplete => Confirmed ** Also affects: qemu Importance: Undecided Status: New -- You received this bug notification because you are a member of qemu- devel-ml, which

Re: [Qemu-devel] [PATCH] versatilepb: add i2c support

2012-04-09 Thread Oskar Andero
On 18:45 Thu 05 Apr , Peter Maydell wrote: > On 2 April 2012 21:17, Oskar Andero wrote: > > Signed-off-by: Oskar Andero > > --- > >  hw/versatilepb.c |   80 > > ++ > >  1 files changed, 80 insertions(+), 0 deletions(-) > > This is just a c

[Qemu-devel] qemu-ga: possible race while suspending the guest

2012-04-09 Thread Luiz Capitulino
Hi Michael, There's a possible race condition in the bios_supports_mode() function used by all suspend commands in qemu-ga: if the parent process receives a SIGCHLD while executing: close(pipefds[1]); g_free(pmutils_path); Or: ret = read(pipefds[0], &status, sizeof(status)); Then b

Re: [Qemu-devel] [PATCH 1/4] softmmu: move target alignment definition to configure stage

2012-04-09 Thread Blue Swirl
On Mon, Apr 9, 2012 at 19:46, Blue Swirl wrote: > Signed-off-by: Blue Swirl > --- >  configure                  |   11 +++ >  exec.c                     |    6 ++ >  softmmu_template.h         |   12 ++-- >  target-alpha/mem_helper.c  |    1 - >  target-mips/op_helper.c    |  

[Qemu-devel] [PATCH 4/4] softmmu: add a faster helper for TCG

2012-04-09 Thread Blue Swirl
Don't recheck the TLB miss which we know is true, instead call directly the miss path. Signed-off-by: Blue Swirl --- softmmu_template.h | 40 tcg/arm/tcg-target.c | 19 ++- tcg/hppa/tcg-target.c | 17 + tcg/i38

[Qemu-devel] [PATCH 3/4] softmmu: move TCG memory access helpers to TCG targets

2012-04-09 Thread Blue Swirl
Signed-off-by: Blue Swirl --- target-alpha/mem_helper.c | 14 -- target-sparc/ldst_helper.c | 13 - tcg/arm/tcg-target.c | 14 ++ tcg/hppa/tcg-target.c | 14 ++ tcg/i386/tcg-target.c | 14 ++ tcg/ia64/tcg-targe

[Qemu-devel] [PATCH 2/4] softmmu: make unaligned access helper global

2012-04-09 Thread Blue Swirl
Make the unaligned access functions global, take a parameter for CPUArchState and use QEMU_NORETURN for all architectures except Xtensa where it may return. Introduce wrappers for targets which use AREG0. Signed-off-by: Blue Swirl --- exec-all.h |9 + softmmu_template

[Qemu-devel] [PATCH 1/4] softmmu: move target alignment definition to configure stage

2012-04-09 Thread Blue Swirl
Signed-off-by: Blue Swirl --- configure | 11 +++ exec.c |6 ++ softmmu_template.h | 12 ++-- target-alpha/mem_helper.c |1 - target-mips/op_helper.c|1 - target-sparc/ldst_helper.c |1 - target-xtensa/op_he

[Qemu-devel] [PATCH, RFC/FYI 0/4] TCG memory access helper changes

2012-04-09 Thread Blue Swirl
The "faster helper" in the last patch is not so impressive. GCC does not want to inline the TCG specific helper but performs a tail call from it to common helper. This results in useless shuffling of registers and stack, also stack protector gets to kick in the second time. I tried also to modify t

[Qemu-devel] [PATCH 3/3] memory: move RAM test functions to memory

2012-04-09 Thread Blue Swirl
Rename functions with memory_region_ prefix to avoid future clashes. Change the RAM/ROM/ROMD test functions to take MemoryRegion instead of MemoryRegionSection. Adjust callers. Signed-off-by: Blue Swirl --- cputlb.c | 10 +- cputlb.h | 18 - exec.c | 67

[Qemu-devel] [PATCH 0/3] TLB patches

2012-04-09 Thread Blue Swirl
The main motivation for patches 2 and 3 was to move TLB handling closer to TCG. I'll send a different patch set which does this but the improvement didn't look very promising. Blue Swirl (3): Use uintptr_t for various op related functions cputlb: move TLB handling to a separate file memory:

[Qemu-devel] [PATCH 1/2] qdev: qdev_unplug(): use error_set()

2012-04-09 Thread Luiz Capitulino
It currently uses qerror_report(), but next commit will convert the drive_del command to the QAPI and this requires using error_set(). One particularity of qerror_report() is that it knows when it's running on monitor context or command-line context and prints the error message accordingly. error_

[Qemu-devel] [PATCH 2/2] qapi: convert device_del

2012-04-09 Thread Luiz Capitulino
Signed-off-by: Anthony Liguori Signed-off-by: Luiz Capitulino Reviewed-by: Stefan Hajnoczi --- hmp-commands.hx |3 +-- hmp.c |9 + hmp.h |1 + hw/qdev-monitor.c | 18 +- qapi-schema.json | 20 qmp-commands.h

[Qemu-devel] [PULL 0/2]: QMP queue

2012-04-09 Thread Luiz Capitulino
Contains only the device_del conversion to the QAPI series. The changes (since 7914cb3c738a03a5d5f7cb32c3768bc62eb1e944) are available in the following repository: git://repo.or.cz/qemu/qmp-unstable.git queue/qmp Luiz Capitulino (2): qdev: qdev_unplug(): use error_set() qapi: con

[Qemu-devel] hpet problems with unaccelerated qemu

2012-04-09 Thread Serge E. Hallyn
Hi, at https://bugs.launchpad.net/debian/+source/qemu-kvm/+bug/975240 there is reported a problem in 1.0.0 with running unaccelerated qemu with hpet. This is fixed upstream as of commit ce967e2f33861b0e17753f97fa4527b5943c94b6. However, that one seems very depending on many of the preceding ~thous

Re: [Qemu-devel] [PATCH V2 10/10] hw/sd.c: introduce SD card "image" property and allow SD hot-insert

2012-04-09 Thread Igor Mitsyanko
On 04/05/2012 09:02 PM, Paolo Bonzini wrote: Il 05/04/2012 17:48, Igor Mitsyanko ha scritto: New SD card "image" property can be used to: - change image file attached to virtual SD card - hot-insert new image file into newly initialized BlockDriverState (this was not possible before). Exampl

[Qemu-devel] How use Qemu as an application library

2012-04-09 Thread Mikael
Dear list, I understood that earlier on there was a "libqemu" library, using which an ordinary userspace application could allocate a qemu virtual machine and get it in a pointer/handle representation, run it as long as it wanted to, and intercept any activity from or to it in detail. This is a *

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

2012-04-09 Thread Igor Mitsyanko
On 03/30/2012 11:58 AM, Peter A. G. Crosthwaite wrote: Hi Peter, a few comments on your patch which would hopefully help it to be pushed faster: Device model for Primecell PL330 dma controller. Signed-off-by: Peter A. G. Crosthwaite Signed-off-by: Kirill Batuzov --- changed from v1: GPLv2 li

Re: [Qemu-devel] [PATCH V2 10/10] hw/sd.c: introduce SD card "image" property and allow SD hot-insert

2012-04-09 Thread Luiz Capitulino
On Thu, 05 Apr 2012 19:02:23 +0200 Paolo Bonzini wrote: > Il 05/04/2012 17:48, Igor Mitsyanko ha scritto: > > New SD card "image" property can be used to: > > - change image file attached to virtual SD card > > - hot-insert new image file into newly initialized BlockDriverState (this > > was not

[Qemu-devel] KVM call agenda for April, Tuesday 10

2012-04-09 Thread Juan Quintela
Hi Please send in any agenda items you are interested in covering. Cheers, Juan.

[Qemu-devel] [PATCH] avoid repeating contain header file

2012-04-09 Thread Wanpeng Li
Signed-off-by: Wanpeng Li --- hw/ps2.h | 29 + 1 files changed, 29 insertions(+), 0 deletions(-) diff --git a/hw/ps2.h b/hw/ps2.h index 32a4231..d19c226 100644 --- a/hw/ps2.h +++ b/hw/ps2.h @@ -1,3 +1,30 @@ +/* + * QEMU PS/2 keyboard/mouse emulation + * + * Copyrigh

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-09 Thread Michael S. Tsirkin
On Mon, Apr 09, 2012 at 11:07:03AM +0300, Gleb Natapov wrote: > On Sun, Apr 08, 2012 at 10:48:02PM +0300, Michael S. Tsirkin wrote: > > On Sun, Apr 08, 2012 at 03:45:38PM -0300, Marcelo Tosatti wrote: > > > On Sun, Apr 08, 2012 at 10:15:26AM +0300, Michael S. Tsirkin wrote: > > > > On Sun, Apr 08,

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-09 Thread Gleb Natapov
On Sun, Apr 08, 2012 at 10:48:02PM +0300, Michael S. Tsirkin wrote: > On Sun, Apr 08, 2012 at 03:45:38PM -0300, Marcelo Tosatti wrote: > > On Sun, Apr 08, 2012 at 10:15:26AM +0300, Michael S. Tsirkin wrote: > > > On Sun, Apr 08, 2012 at 02:13:00AM -0300, Marcelo Tosatti wrote: > > > > On Wed, Apr 0