Re: [Qemu-devel] KVM/QEMU: Support executing from flash device

2012-03-04 Thread Gleb Natapov
On Sat, Mar 03, 2012 at 10:56:02PM -0800, Jordan Justen wrote: > On Tue, Oct 25, 2011 at 00:47, Avi Kivity wrote: > > The core issue that kvm (the kernel part) supports two styles of memory: > > read/write RAM, and read/write MMIO.  ROM wants writes to be ignored, > > and rom/device wants reads se

Re: [Qemu-devel] BlockDriverState stack and BlockListeners

2012-03-04 Thread Paolo Bonzini
Il 04/03/2012 06:14, Ori Mamluk ha scritto: > I prefer not to do it as NBD server, mainly because NBD by definition > requires a port per volume and I think it will pose a management overhead. NBD supports multiple volumes on the same server, just not the implementation in QEMU. Also you could us

[Qemu-devel] [PATCH] make: remove generated objects from target dirs

2012-03-04 Thread Michael S. Tsirkin
I ended up with qmp-commands.h in target directories, which makes build fail as it is found before the main header. make clean fixes it, but it might get triggered again when we make some header target-independent next. It's easy to just make sure all such leftovers are removed, so let's do this.

Re: [Qemu-devel] SPARC64: unable to boot OpenBIOS from git master

2012-03-04 Thread Michael S. Tsirkin
On Sat, Mar 03, 2012 at 03:09:16PM +, Blue Swirl wrote: > On Mon, Feb 27, 2012 at 06:36, Mark Cave-Ayland > wrote: > > Hi all, > > > > I've been experimenting with SPARC64 under QEMU, and with current git master > > I am unable to boot OpenBIOS at all with the following error: > > > > OpenBIOS

[Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
commit 5caef97a16010f818ea8b950e2ee24ba876643ad introduced a regression: we do not make IO base/limit upper 16 bit registers writeable, so we should report a 16 bit IO range type, not a 32 bit one. Note that PCI_PREF_RANGE_TYPE_32 is 0x0, but PCI_IO_RANGE_TYPE_32 is 0x1. In particular, this broke

Re: [Qemu-devel] [PULL] kvm updates

2012-03-04 Thread Avi Kivity
On 03/02/2012 01:45 AM, Andreas Färber wrote: > Am 01.03.2012 13:47, schrieb Avi Kivity: > > Avi Kivity (1): > > pc-bios: update kvmvapic.bin > > > > Gleb Natapov (1): > > kvm: Synchronize cpu state in kvm_arch_stop_on_emulation_error() > > > > Jan Kiszka (10): > > kvm: Set cpu_

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: > commit 5caef97a16010f818ea8b950e2ee24ba876643ad introduced > a regression: we do not make IO base/limit upper 16 > bit registers writeable, so we should report a 16 bit > IO range type, not a 32 bit one. > Note that PCI_PREF_RANGE_TYPE_32 i

Re: [Qemu-devel] KVM/QEMU: Support executing from flash device

2012-03-04 Thread Avi Kivity
On 03/04/2012 10:12 AM, Gleb Natapov wrote: > On Sat, Mar 03, 2012 at 10:56:02PM -0800, Jordan Justen wrote: > > On Tue, Oct 25, 2011 at 00:47, Avi Kivity wrote: > > > The core issue that kvm (the kernel part) supports two styles of memory: > > > read/write RAM, and read/write MMIO. ROM wants wri

Re: [Qemu-devel] KVM/QEMU: Support executing from flash device

2012-03-04 Thread Gleb Natapov
On Sun, Mar 04, 2012 at 12:36:20PM +0200, Avi Kivity wrote: > On 03/04/2012 10:12 AM, Gleb Natapov wrote: > > On Sat, Mar 03, 2012 at 10:56:02PM -0800, Jordan Justen wrote: > > > On Tue, Oct 25, 2011 at 00:47, Avi Kivity wrote: > > > > The core issue that kvm (the kernel part) supports two styles

Re: [Qemu-devel] KVM/QEMU: Support executing from flash device

2012-03-04 Thread Avi Kivity
On 03/04/2012 08:56 AM, Jordan Justen wrote: > On Tue, Oct 25, 2011 at 00:47, Avi Kivity wrote: > > The core issue that kvm (the kernel part) supports two styles of memory: > > read/write RAM, and read/write MMIO. ROM wants writes to be ignored, > > and rom/device wants reads serviced from memory

Re: [Qemu-devel] KVM/QEMU: Support executing from flash device

2012-03-04 Thread Avi Kivity
On 03/04/2012 12:38 PM, Gleb Natapov wrote: > On Sun, Mar 04, 2012 at 12:36:20PM +0200, Avi Kivity wrote: > > On 03/04/2012 10:12 AM, Gleb Natapov wrote: > > > On Sat, Mar 03, 2012 at 10:56:02PM -0800, Jordan Justen wrote: > > > > On Tue, Oct 25, 2011 at 00:47, Avi Kivity wrote: > > > > > The core

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Avi Kivity
On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote: > On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote: > > > What if TARGET_PAGE_SIZE > getpagesize()? Or is that impossible? > > We have yet to encounter such a case. It's not currently possible on > power (some old embedded chips could do 1K

[Qemu-devel] [PATCH v1 0/5] Microblaze machine models and bootloader refactoring

2012-03-04 Thread Peter A. G. Crosthwaite
From: "Peter A. G. Crosthwaite" Overhauled the microblaze bootloader process. Factored out the common boot code between petalogix_ml605 and petalogix_s3adsp1800 machine models into a dedicated microblaze bootloader (Patch 1). It now organised in much the same way as its handled in arm machine

[Qemu-devel] [PATCH v1 1/5] microblaze: factored out common boot code

2012-03-04 Thread Peter A. G. Crosthwaite
From: "Peter A. G. Crosthwaite" factored out the copy-pasted common boot code from the two microblaze platforms into a dedicated microblaze bootloader (microblaze_boot.o). Signed-off-by: Peter A. G. Crosthwaite --- Makefile.target |1 + hw/microblaze_boot.c | 183 ++

[Qemu-devel] [PATCH v1 2/5] petalogix_s2adsp1800: moved rst logic to rst fn

2012-03-04 Thread Peter A. G. Crosthwaite
From: "Peter A. G. Crosthwaite" This belongs in the machine specific reset function Signed-off-by: Peter A. G. Crosthwaite --- hw/petalogix_s3adsp1800_mmu.c | 12 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/hw/petalogix_s3adsp1800_mmu.c b/hw/petalogix_s3adsp18

[Qemu-devel] [PATCH v1 3/5] petalogix_s2adsp1800: macro'd magic numbers

2012-03-04 Thread Peter A. G. Crosthwaite
From: "Peter A. G. Crosthwaite" defined macros for the addresses of the peripherals in machine model Signed-off-by: Peter A. G. Crosthwaite --- hw/petalogix_s3adsp1800_mmu.c | 20 +--- 1 files changed, 13 insertions(+), 7 deletions(-) diff --git a/hw/petalogix_s3adsp1800_mmu

[Qemu-devel] [PATCH v1 4/5] qemu-options.hx: allow -dtb argument for all archs

2012-03-04 Thread Peter A. G. Crosthwaite
From: "Peter A. G. Crosthwaite" Signed-off-by: Peter A. G. Crosthwaite --- qemu-options.hx |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/qemu-options.hx b/qemu-options.hx index e38799c..daefce3 100644 --- a/qemu-options.hx +++ b/qemu-options.hx @@ -2038,7 +2038,7 @@

[Qemu-devel] [PATCH v1 5/5] microblaze_boot: enabled -dtb argument

2012-03-04 Thread Peter A. G. Crosthwaite
From: "Peter A. G. Crosthwaite" Use the -dtb argument for passing is a custom dtb rather than the old hardcoded "mb.dtb" Signed-off-by: Peter A. G. Crosthwaite --- hw/microblaze_boot.c | 34 ++ 1 files changed, 14 insertions(+), 20 deletions(-) diff --git a/h

[Qemu-devel] [PATCH] usb-ehci: drop unused isoch_pause variable

2012-03-04 Thread Hans de Goede
Signed-off-by: Hans de Goede --- hw/usb-ehci.c |6 +- 1 files changed, 1 insertions(+), 5 deletions(-) diff --git a/hw/usb-ehci.c b/hw/usb-ehci.c index df742f7..f4d53ff 100644 --- a/hw/usb-ehci.c +++ b/hw/usb-ehci.c @@ -419,7 +419,6 @@ struct EHCIState { USBPacket ipacket; QE

Re: [Qemu-devel] [PULL] Patch queue

2012-03-04 Thread Blue Swirl
Thanks, pulled. On Tue, Feb 28, 2012 at 21:47, Stefan Weil wrote: > Hi, > > please pull these patches from January. > > Thanks, > > Stefan Weil > > > The following changes since commit b55c952aea6de024bf1a06357b49367fba045443: > >  Merge remote-tracking branch 'aneesh/for-upstream' into staging (

Re: [Qemu-devel] [PATCH 0/6] w64: Improve compilation with MinGW-w64

2012-03-04 Thread Blue Swirl
Thanks, applied all. On Fri, Mar 2, 2012 at 22:30, Stefan Weil wrote: > These patches are a step towards full 64 bit support for w64. > The patches 4 and 5 are optional. > > Please apply this series. > > Thanks, > Stefan Weil > > [PATCH 1/6] w64: Fix size of ram_addr_t > [PATCH 2/6] tcg: Rearrang

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Benjamin Herrenschmidt
On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote: > On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote: > > On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote: > > > > > What if TARGET_PAGE_SIZE > getpagesize()? Or is that impossible? > > > > We have yet to encounter such a case. It's not cu

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Avi Kivity
On 03/04/2012 01:53 PM, Benjamin Herrenschmidt wrote: > On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote: > > On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote: > > > On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote: > > > > > > > What if TARGET_PAGE_SIZE > getpagesize()? Or is that impos

Re: [Qemu-devel] [PATCH v8 1/4] cadence_uart: initial version of device model

2012-03-04 Thread Peter Crosthwaite
On Fri, Mar 2, 2012 at 12:09 AM, Peter Maydell wrote: > On 28 February 2012 07:40, Peter A. G. Crosthwaite > wrote: >> Implemented cadence UART serial controller >> >> Signed-off-by: Peter A. G. Crosthwaite >> Signed-off-by: John Linn >> Acked-by: Edgar E. Iglesias >> --- >> changed from v7: >

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 10:27:24AM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: > > commit 5caef97a16010f818ea8b950e2ee24ba876643ad introduced > > a regression: we do not make IO base/limit upper 16 > > bit registers writeable, so we should report a 16 bit > >

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Avi Kivity
On 03/04/2012 12:27 PM, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: > > commit 5caef97a16010f818ea8b950e2ee24ba876643ad introduced > > a regression: we do not make IO base/limit upper 16 > > bit registers writeable, so we should report a 16 bit > > IO range type, no

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 10:27:24AM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: > > commit 5caef97a16010f818ea8b950e2ee24ba876643ad introduced > > a regression: we do not make IO base/limit upper 16 > > bit registers writeable, so we should report a 16 bit > >

[Qemu-devel] questions about pci

2012-03-04 Thread WanPeng Li
Hi all: I read pci code in qemu about i440fx, pci.c and so on. I think if guest os whose mainboard is based on x86, it will use IO instructions to access PCI configuration space.If not use passthrough, qemu should emulate these operations.I find a function called kvm_handle_io who will emulate iop

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 02:33:02PM +0200, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 10:27:24AM +, Blue Swirl wrote: > > On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: > > > commit 5caef97a16010f818ea8b950e2ee24ba876643ad introduced > > > a regression: we do not make IO base/

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Avi Kivity
On 03/04/2012 02:33 PM, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 10:27:24AM +, Blue Swirl wrote: > > On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: > > > commit 5caef97a16010f818ea8b950e2ee24ba876643ad introduced > > > a regression: we do not make IO base/limit upper 16 > >

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 12:21, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 10:27:24AM +, Blue Swirl wrote: >> On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: >> > commit 5caef97a16010f818ea8b950e2ee24ba876643ad introduced >> > a regression: we do not make IO base/limit upper 16

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 12:28, Avi Kivity wrote: > On 03/04/2012 12:27 PM, Blue Swirl wrote: >> On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: >> > commit 5caef97a16010f818ea8b950e2ee24ba876643ad introduced >> > a regression: we do not make IO base/limit upper 16 >> > bit registers writea

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Avi Kivity
On 03/04/2012 02:38 PM, Blue Swirl wrote: > >> > >> This unassigned memory exception is triggered because CMD646 IDE I/O > >> registers are not accessible: > >> > >> (qemu) info pci > >> Bus 0, device 5, function 0: > >> IDE controller: PCI device 1095:0646 > >> IRQ 1. > >> BAR

[Qemu-devel] restart a coroutine?

2012-03-04 Thread Michael Tokarev
Since all block (bdrv) layer is now implemented using coroutines, I thought I'd give it a try. But immediately hit a question to which I don't know a good answer. Suppose we've some networking block device (like NBD) and want to be able to support reconnection - this is actually very useful featu

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 12:33, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 10:27:24AM +, Blue Swirl wrote: >> On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: >> > commit 5caef97a16010f818ea8b950e2ee24ba876643ad introduced >> > a regression: we do not make IO base/limit upper 16

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 12:41, Avi Kivity wrote: > On 03/04/2012 02:38 PM, Blue Swirl wrote: >> >> >> >> This unassigned memory exception is triggered because CMD646 IDE I/O >> >> registers are not accessible: >> >> >> >> (qemu) info pci >> >>   Bus  0, device   5, function 0: >> >>     IDE control

Re: [Qemu-devel] [PATCH v8 2/4] cadence_ttc: initial version of device model

2012-03-04 Thread Peter Crosthwaite
On Fri, Mar 2, 2012 at 12:58 AM, Peter Maydell wrote: > On 28 February 2012 07:40, Peter A. G. Crosthwaite > wrote: >> Implemented cadence Triple Timer Counter (TCC) >> >> Signed-off-by: Peter A. G. Crosthwaite >> Signed-off-by: John Linn >> Acked-by: Edgar E. Iglesias >> --- >> changed from v

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 12:46:23PM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 12:41, Avi Kivity wrote: > > On 03/04/2012 02:38 PM, Blue Swirl wrote: > >> >> > >> >> This unassigned memory exception is triggered because CMD646 IDE I/O > >> >> registers are not accessible: > >> >> > >> >> (qe

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 02:41:33PM +0200, Avi Kivity wrote: > On 03/04/2012 02:38 PM, Blue Swirl wrote: > > >> > > >> This unassigned memory exception is triggered because CMD646 IDE I/O > > >> registers are not accessible: > > >> > > >> (qemu) info pci > > >> Bus 0, device 5, function 0: > >

Re: [Qemu-devel] [PATCH] make: remove generated objects from target dirs

2012-03-04 Thread Peter Maydell
On 4 March 2012 09:10, Michael S. Tsirkin wrote: > I ended up with qmp-commands.h in target directories, > which makes build fail as it is found before the > main header. > make clean fixes it, but it might get triggered > again when we make some header target-independent next. > It's easy to just

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 12:37:57PM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 12:21, Michael S. Tsirkin wrote: > > On Sun, Mar 04, 2012 at 10:27:24AM +, Blue Swirl wrote: > >> On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: > >> > commit 5caef97a16010f818ea8b950e2ee24ba876643ad

Re: [Qemu-devel] [PATCH] make: remove generated objects from target dirs

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 01:25:59PM +, Peter Maydell wrote: > On 4 March 2012 09:10, Michael S. Tsirkin wrote: > > I ended up with qmp-commands.h in target directories, > > which makes build fail as it is found before the > > main header. > > make clean fixes it, but it might get triggered > >

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 13:22, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 02:41:33PM +0200, Avi Kivity wrote: >> On 03/04/2012 02:38 PM, Blue Swirl wrote: >> > >> >> > >> This unassigned memory exception is triggered because CMD646 IDE I/O >> > >> registers are not accessible: >> > >> >> >

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 12:37:57PM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 12:21, Michael S. Tsirkin wrote: > > On Sun, Mar 04, 2012 at 10:27:24AM +, Blue Swirl wrote: > >> On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin wrote: > >> > commit 5caef97a16010f818ea8b950e2ee24ba876643ad

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 13:28, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 12:37:57PM +, Blue Swirl wrote: >> On Sun, Mar 4, 2012 at 12:21, Michael S. Tsirkin wrote: >> > On Sun, Mar 04, 2012 at 10:27:24AM +, Blue Swirl wrote: >> >> On Sun, Mar 4, 2012 at 09:46, Michael S. Tsirkin

Re: [Qemu-devel] [PATCH] make: remove generated objects from target dirs

2012-03-04 Thread Peter Maydell
On 4 March 2012 13:31, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 01:25:59PM +, Peter Maydell wrote: >> In general we don't have workarounds for "something >> moved directory and this broke builds not from clean" > > Why don't we? It's cheaper than always doing > make clean after pull

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 01:33:42PM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 13:22, Michael S. Tsirkin wrote: > > On Sun, Mar 04, 2012 at 02:41:33PM +0200, Avi Kivity wrote: > >> On 03/04/2012 02:38 PM, Blue Swirl wrote: > >> > >> > >> > >> This unassigned memory exception is triggered bec

Re: [Qemu-devel] [PATCH] make: remove generated objects from target dirs

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 01:44:26PM +, Peter Maydell wrote: > On 4 March 2012 13:31, Michael S. Tsirkin wrote: > > On Sun, Mar 04, 2012 at 01:25:59PM +, Peter Maydell wrote: > >> In general we don't have workarounds for "something > >> moved directory and this broke builds not from clean" >

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 01:38:38PM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 13:28, Michael S. Tsirkin wrote: > > On Sun, Mar 04, 2012 at 12:37:57PM +, Blue Swirl wrote: > >> On Sun, Mar 4, 2012 at 12:21, Michael S. Tsirkin wrote: > >> > On Sun, Mar 04, 2012 at 10:27:24AM +, Blue

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 14:08, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 01:33:42PM +, Blue Swirl wrote: >> On Sun, Mar 4, 2012 at 13:22, Michael S. Tsirkin wrote: >> > On Sun, Mar 04, 2012 at 02:41:33PM +0200, Avi Kivity wrote: >> >> On 03/04/2012 02:38 PM, Blue Swirl wrote: >> >> >

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 14:23, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 01:38:38PM +, Blue Swirl wrote: >> On Sun, Mar 4, 2012 at 13:28, Michael S. Tsirkin wrote: >> > On Sun, Mar 04, 2012 at 12:37:57PM +, Blue Swirl wrote: >> >> On Sun, Mar 4, 2012 at 12:21, Michael S. Tsirkin

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 02:35:28PM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 14:23, Michael S. Tsirkin wrote: > > On Sun, Mar 04, 2012 at 01:38:38PM +, Blue Swirl wrote: > >> On Sun, Mar 4, 2012 at 13:28, Michael S. Tsirkin wrote: > >> > On Sun, Mar 04, 2012 at 12:37:57PM +, Blue

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 02:35:28PM +, Blue Swirl wrote: > Yes, it's the host bridge, also known as PBM. It's documented in > UltraSPARC IIi User's Manual and there it says that the device is > found in the configuration space. So it seems I can make things work if I disable is_bridge and make

Re: [Qemu-devel] [PATCH] make: remove generated objects from target dirs

2012-03-04 Thread Andreas Färber
Am 04.03.2012 10:10, schrieb Michael S. Tsirkin: > I ended up with qmp-commands.h in target directories, > which makes build fail as it is found before the > main header. > make clean fixes it, but it might get triggered > again when we make some header target-independent next. > It's easy to just

Re: [Qemu-devel] restart a coroutine?

2012-03-04 Thread Avi Kivity
On 03/04/2012 02:41 PM, Michael Tokarev wrote: > Since all block (bdrv) layer is now implemented using > coroutines, I thought I'd give it a try. But immediately > hit a question to which I don't know a good answer. > > Suppose we've some networking block device (like NBD) and > want to be able to

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 02:26:13PM +, Blue Swirl wrote: > > It seems to have to do with the host bridge. > > It's unusual to have host bridge present itself > > as a pci to pci bridge but there it is. > > It looks like the I/O base calculations in OpenBIOS are confused Where's the source for

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Andreas Färber
Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt: > On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote: >> On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote: >>> On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote: >>> What if TARGET_PAGE_SIZE > getpagesize()? Or is that impossible? >>

Re: [Qemu-devel] [PATCH] make: remove generated objects from target dirs

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 05:03:14PM +0100, Andreas Färber wrote: > Am 04.03.2012 10:10, schrieb Michael S. Tsirkin: > > I ended up with qmp-commands.h in target directories, > > which makes build fail as it is found before the > > main header. > > make clean fixes it, but it might get triggered > >

Re: [Qemu-devel] [PATCH] seabios: acpi: Add _STA for PCI hotplug slots

2012-03-04 Thread Kevin O'Connor
On Fri, Feb 24, 2012 at 04:21:17PM -0700, Alex Williamson wrote: > When a Status method is provided on a slot, the OSPM evaluates > _STA in response to the device check notify on the slot. This > allows some degree of a handshake between the platform and the > OSPM that the hotplug has been acknow

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 15:22, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 02:35:28PM +, Blue Swirl wrote: >> On Sun, Mar 4, 2012 at 14:23, Michael S. Tsirkin wrote: >> > On Sun, Mar 04, 2012 at 01:38:38PM +, Blue Swirl wrote: >> >> On Sun, Mar 4, 2012 at 13:28, Michael S. Tsirkin

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 05:07:34PM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 15:22, Michael S. Tsirkin wrote: > > On Sun, Mar 04, 2012 at 02:35:28PM +, Blue Swirl wrote: > >> On Sun, Mar 4, 2012 at 14:23, Michael S. Tsirkin wrote: > >> > On Sun, Mar 04, 2012 at 01:38:38PM +, Blue

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 16:42, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 02:26:13PM +, Blue Swirl wrote: >> > It seems to have to do with the host bridge. >> > It's unusual to have host bridge present itself >> > as a pci to pci bridge but there it is. >> >> It looks like the I/O base

[Qemu-devel] PPC: VGA broken in git master

2012-03-04 Thread Mark Cave-Ayland
Hi all, I've just done a git pull to update my local repository, and it now appears that the VGA device is broken in QEMU - rather than displaying the OpenBIOS banner in my VNC client, the framebuffer remains constantly black. A git bisect shows that the problem is caused by the following co

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Mark Cave-Ayland
On 04/03/12 17:49, Blue Swirl wrote: According to the spec it is a device, so should be ok? If I just make BAR4 writeable we get past the bios screen at least. Maybe openbios gets confused if a device has no BARs? Do things work for you with the patch below? All it does is make BAR4 writeable, a

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Alexander Graf
On 04.03.2012, at 17:46, Andreas Färber wrote: > Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt: >> On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote: >>> On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote: On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote: > What if T

Re: [Qemu-devel] [PATCH] seabios: acpi: Add _STA for PCI hotplug slots

2012-03-04 Thread Michael S. Tsirkin
On Fri, Feb 24, 2012 at 04:21:17PM -0700, Alex Williamson wrote: > When a Status method is provided on a slot, the OSPM evaluates > _STA in response to the device check notify on the slot. This > allows some degree of a handshake between the platform and the > OSPM that the hotplug has been acknow

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 05:49:04PM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 16:42, Michael S. Tsirkin wrote: > > On Sun, Mar 04, 2012 at 02:26:13PM +, Blue Swirl wrote: > >> > It seems to have to do with the host bridge. > >> > It's unusual to have host bridge present itself > >> > as

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 06:11:20PM +, Mark Cave-Ayland wrote: > On 04/03/12 17:49, Blue Swirl wrote: > > >>According to the spec it is a device, so should be ok? > >>If I just make BAR4 writeable we get past the > >>bios screen at least. > >>Maybe openbios gets confused if a device has no BARs

Re: [Qemu-devel] [Seabios] [PATCH 0/6] 64bit PCI BARs allocations (take 2)

2012-03-04 Thread Kevin O'Connor
On Thu, Mar 01, 2012 at 06:50:43PM +1300, Alexey Korolev wrote: > Hi, > > This patch series enables 64bit BAR support in seabios. > It has a bit different approach for resources accounting, We did this > because we wanted: > a) Provide 64bit bar support for PCI BARs and bridges with 64bit memory

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 06:11:20PM +, Mark Cave-Ayland wrote: > On 04/03/12 17:49, Blue Swirl wrote: > > >>According to the spec it is a device, so should be ok? > >>If I just make BAR4 writeable we get past the > >>bios screen at least. > >>Maybe openbios gets confused if a device has no BARs

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 17:35, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 05:07:34PM +, Blue Swirl wrote: >> On Sun, Mar 4, 2012 at 15:22, Michael S. Tsirkin wrote: >> > On Sun, Mar 04, 2012 at 02:35:28PM +, Blue Swirl wrote: >> >> On Sun, Mar 4, 2012 at 14:23, Michael S. Tsirkin

Re: [Qemu-devel] restart a coroutine?

2012-03-04 Thread Michael Tokarev
On 04.03.2012 20:08, Avi Kivity wrote: > On 03/04/2012 02:41 PM, Michael Tokarev wrote: >> Since all block (bdrv) layer is now implemented using >> coroutines, I thought I'd give it a try. But immediately >> hit a question to which I don't know a good answer. >> >> Suppose we've some networking bl

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 07:51:02PM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 17:35, Michael S. Tsirkin wrote: > > On Sun, Mar 04, 2012 at 05:07:34PM +, Blue Swirl wrote: > >> On Sun, Mar 4, 2012 at 15:22, Michael S. Tsirkin wrote: > >> > On Sun, Mar 04, 2012 at 02:35:28PM +, Blue

[Qemu-devel] XP install cores with SCSI LSI 53C895A disks

2012-03-04 Thread Gerhard Wiesinger
Hello, Clean XP install cores with SCSI LSI 53C89A disk when copying files. Reproduceable. Driver used is sym_hi. Details are below. Tried also old versions 1.0, 0.15.1, cores too. Any ideas? Thnx. Ciao, Gerhard -- http://www.wiesinger.com/ Image created with: qemu-img create -f qcow2 XP

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Andreas Färber
Am 04.03.2012 19:46, schrieb Alexander Graf: > > > On 04.03.2012, at 17:46, Andreas Färber wrote: > >> Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt: >>> On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote: On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote: > On Tue, 2012-02-2

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Alexander Graf
On 04.03.2012, at 21:21, Andreas Färber wrote: > Am 04.03.2012 19:46, schrieb Alexander Graf: >> >> >> On 04.03.2012, at 17:46, Andreas Färber wrote: >> >>> Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt: On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote: > On 02/28/2012 11:

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Andreas Färber
Am 04.03.2012 21:25, schrieb Alexander Graf: > > > On 04.03.2012, at 21:21, Andreas Färber wrote: > >> Am 04.03.2012 19:46, schrieb Alexander Graf: >>> >>> >>> On 04.03.2012, at 17:46, Andreas Färber wrote: >>> Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt: > On Sun, 2012-03-04

[Qemu-devel] [PATCH v4 1/3] kvmclock: Always register type

2012-03-04 Thread Andreas Färber
Currently, the "kvmclock" type is only registered when kvm_enabled(). This breaks when moving type registration to before command line parsing (so that QOM types can be used for CPU and machine). Since the QOM classes are lazy-initialized anyway and kvmclock_create() has another kvm_enabled() che

[Qemu-devel] [PATCH v4 2/3] qom: Register QOM infrastructure early

2012-03-04 Thread Andreas Färber
The constructors for QOM TYPE_INTERFACE were executed rather late in vl.c's main(). Call them very early so that QOM can safely be used for machines and CPUs. Signed-off-by: Andreas Färber Cc: Anthony Liguori --- vl.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/v

[Qemu-devel] [PATCH v4 3/3] qom: Add QOM support to user emulators

2012-03-04 Thread Andreas Färber
Link the Object base class and the module infrastructure for class registration. Introduce $(universal-obj-y) for objects that are more common than $(common-obj-y), so that those only get built once. Call QOM module init for type registration. Signed-off-by: Andreas Färber Cc: Anthony Liguori -

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 20:02, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 07:51:02PM +, Blue Swirl wrote: >> On Sun, Mar 4, 2012 at 17:35, Michael S. Tsirkin wrote: >> > On Sun, Mar 04, 2012 at 05:07:34PM +, Blue Swirl wrote: >> >> On Sun, Mar 4, 2012 at 15:22, Michael S. Tsirkin

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Alexander Graf
On 04.03.2012, at 21:31, Andreas Färber wrote: > Am 04.03.2012 21:25, schrieb Alexander Graf: >> >> >> On 04.03.2012, at 21:21, Andreas Färber wrote: >> >>> Am 04.03.2012 19:46, schrieb Alexander Graf: On 04.03.2012, at 17:46, Andreas Färber wrote: > Am 04.03.201

[Qemu-devel] [PATCH v4 0/3] Prepare QOM support for machines and CPU

2012-03-04 Thread Andreas Färber
Hello Anthony, This series prepares for QOM'ification of CPUs and machines by rearranging constructor calls and Makefile dependency rules. I've separated it out due to complaints about patch series length for conversion of all targets. Patch 1 is a RESEND that avoids an abort after patch 2. Patc

Re: [Qemu-devel] KVM/QEMU: Support executing from flash device

2012-03-04 Thread Jordan Justen
On Sun, Mar 4, 2012 at 02:46, Avi Kivity wrote: > On 03/04/2012 12:38 PM, Gleb Natapov wrote: >> On Sun, Mar 04, 2012 at 12:36:20PM +0200, Avi Kivity wrote: >> > On 03/04/2012 10:12 AM, Gleb Natapov wrote: >> > > On Sat, Mar 03, 2012 at 10:56:02PM -0800, Jordan Justen wrote: >> > > > On Tue, Oct 2

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Benjamin Herrenschmidt
On Sun, 2012-03-04 at 17:46 +0100, Andreas Färber wrote: > Except for ppcemb-softmmu (1k), which is irrelevant for KVM AFAIU. > > Maybe just add an assert and be done with it? Well, my patch should work anyway, as long as getpagesize() returns a higher power of two. The case that wouldn't work (

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Benjamin Herrenschmidt
On Sun, 2012-03-04 at 21:59 +0100, Alexander Graf wrote: > >>> g_assert(TARGET_PAGE_SIZE <= getpagesize()) > >>> > >>> Just declare the above case as unsupported and abort if we > encounter it. > >> > >> What I'm trying to tell you is that it's the default case on book3s > ppc! ;) > > > > Exactl

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Andreas Färber
Am 04.03.2012 21:59, schrieb Alexander Graf: > > > On 04.03.2012, at 21:31, Andreas Färber wrote: > >> Am 04.03.2012 21:25, schrieb Alexander Graf: >>> >>> >>> On 04.03.2012, at 21:21, Andreas Färber wrote: >>> Am 04.03.2012 19:46, schrieb Alexander Graf: > > > On 04.03.2012,

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 08:32:26PM +, Blue Swirl wrote: > On Sun, Mar 4, 2012 at 20:02, Michael S. Tsirkin wrote: > > On Sun, Mar 04, 2012 at 07:51:02PM +, Blue Swirl wrote: > >> On Sun, Mar 4, 2012 at 17:35, Michael S. Tsirkin wrote: > >> > On Sun, Mar 04, 2012 at 05:07:34PM +, Blue

Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size

2012-03-04 Thread Alexander Graf
On 04.03.2012, at 22:19, Benjamin Herrenschmidt wrote: > On Sun, 2012-03-04 at 21:59 +0100, Alexander Graf wrote: > g_assert(TARGET_PAGE_SIZE <= getpagesize()) > > Just declare the above case as unsupported and abort if we >> encounter it. What I'm trying to tell you is

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Blue Swirl
On Sun, Mar 4, 2012 at 21:28, Michael S. Tsirkin wrote: > On Sun, Mar 04, 2012 at 08:32:26PM +, Blue Swirl wrote: >> On Sun, Mar 4, 2012 at 20:02, Michael S. Tsirkin wrote: >> > On Sun, Mar 04, 2012 at 07:51:02PM +, Blue Swirl wrote: >> >> On Sun, Mar 4, 2012 at 17:35, Michael S. Tsirkin

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Mark Cave-Ayland
On 04/03/12 19:51, Blue Swirl wrote: I now know the root cause of the problem. OpenBIOS programs the BARs somewhat correctly just by accident. The initial io_base and mem_base for BARs are not correct, but because the host bridge BARs (and also 6 of which 4 are not even BARs!) are programmed fir

Re: [Qemu-devel] [PATCH] pci: fix bridge IO/BASE

2012-03-04 Thread Michael S. Tsirkin
On Sun, Mar 04, 2012 at 09:54:02PM +, Blue Swirl wrote: > >> 19.3.1.10 tells that the header type is 0, as you noted too. Still, > >> the register layout matches bridge spec instead, for example there are > >> bus number registers in place of BAR 2. > > > > Sorry I don't see this in 19.3.1 > >

Re: [Qemu-devel] [PATCH V14 2/7] Add TPM (frontend) hardware interface (TPM TIS) to Qemu

2012-03-04 Thread Michael S. Tsirkin
On Fri, Mar 02, 2012 at 07:02:21AM -0500, Stefan Berger wrote: > On 02/21/2012 06:08 PM, Michael S. Tsirkin wrote: > >On Tue, Feb 21, 2012 at 05:30:32PM -0500, Stefan Berger wrote: > >>On 02/21/2012 02:58 PM, Michael S. Tsirkin wrote: > >>>On Tue, Feb 21, 2012 at 10:05:26AM -0500, Stefan Berger wro

Re: [Qemu-devel] [PATCH] seabios: acpi: Add _STA for PCI hotplug slots

2012-03-04 Thread Alex Williamson
On Sun, 2012-03-04 at 20:53 +0200, Michael S. Tsirkin wrote: > On Fri, Feb 24, 2012 at 04:21:17PM -0700, Alex Williamson wrote: > > When a Status method is provided on a slot, the OSPM evaluates > > _STA in response to the device check notify on the slot. This > > allows some degree of a handshake

[Qemu-devel] [PATCH 1/6] rtl8139: limit transmission buffer size in c+ mode

2012-03-04 Thread Jason Wang
The tx buffer would be re-allocated for tx descriptor with big size and without LS bit set, this would make guest driver could easily let qemu to allocate unlimited. In linux host, a glib failure were easy to be triggered: GLib-ERROR **: gmem.c:176: failed to allocate 18446744071562067968 bytes

[Qemu-devel] [PATCH 2/6] rtl8139: remove unused marco

2012-03-04 Thread Jason Wang
Signed-off-by: Jason Wang --- hw/rtl8139.c |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/hw/rtl8139.c b/hw/rtl8139.c index d9e742c..ca613ae 100644 --- a/hw/rtl8139.c +++ b/hw/rtl8139.c @@ -65,9 +65,6 @@ #define PCI_FREQUENCY 3300L -/* debug RTL8139 card C+ mo

[Qemu-devel] [PATCH 4/6] net: move compute_mcast_idx() to net.h

2012-03-04 Thread Jason Wang
Reduce duplicated codes. Signed-off-by: Jason Wang --- hw/eepro100.c | 25 - hw/ne2000.c| 24 hw/opencores_eth.c | 25 - hw/rtl8139.c | 24 net.c | 23 +

[Qemu-devel] [PATCH 3/6] rtl8139: support byte read to TxStatus registers

2012-03-04 Thread Jason Wang
Some drivers (such as win7) use byte read for TxStatus registers, so we need to support this to let guest driver behave correctly. For writing, only double-word access is allowed by spec. Signed-off-by: Jason Wang --- hw/rtl8139.c | 33 + 1 files changed, 29 in

[Qemu-devel] [PATCH 5/6] rtl8139: correctly check the opmode

2012-03-04 Thread Jason Wang
According to the spec, only when opmode is "Config. Register Write Enable" could driver write to CONFIG0,1,3,4 and bits 13,12,8 of BMCR. Currently, we allow modifying to those registers also when 8139 is in "Auto-load" mode and "93C46 (93C56) Programming" mode. This patch fixes this. Signed-off-b

[Qemu-devel] [PATCH 6/6] rtl8139: do the network/host communication only in normal operating mode

2012-03-04 Thread Jason Wang
According the spec, the card works in network/host communication mode only when both EEM1 and EEM0 are unset in 93C46 Command Register (normal op mode). So this patch check these bits before trying to receive packets. As some guest driver (such as linux, see cp_init_hw() in 8139cp.c) allocate rx r

  1   2   >