Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Avi Kivity
On 08/08/2011 08:43 PM, Peter Maydell wrote: On 8 August 2011 18:06, Avi Kivitya...@redhat.com wrote: Somewhat clumsy since it needs a variable sized region. @@ -119,7 +120,7 @@ void omap_sdrc_reset(struct omap_sdrc_s *s); struct omap_gpmc_s; struct omap_gpmc_s

Re: [Qemu-devel] [PATCH v4 26/39] pcnet: convert to memory API

2011-08-09 Thread Bob Breuer
Avi Kivity wrote: Also related chips. Reviewed-by: Richard Henderson r...@twiddle.net Reviewed-by: Anthony Liguori aligu...@us.ibm.com Signed-off-by: Avi Kivity a...@redhat.com --- hw/lance.c | 31 ++- hw/pcnet-pci.c | 74

Re: [Qemu-devel] [PATCH v4 26/39] pcnet: convert to memory API

2011-08-09 Thread Avi Kivity
On 08/09/2011 09:55 AM, Bob Breuer wrote: static void lance_cleanup(VLANClientState *nc) @@ -117,13 +116,11 @@ static int lance_init(SysBusDevice *dev) SysBusPCNetState *d = FROM_SYSBUS(SysBusPCNetState, dev); PCNetState *s =d-state; -s-mmio_index = -

[PATCH] lance: unbreak after memory API conversion

2011-08-09 Thread Avi Kivity
The conversion passed the wrong opaque pointer, causing a crash on first use. Pass the correct opaque. Signed-off-by: Avi Kivity a...@redhat.com --- hw/lance.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/lance.c b/hw/lance.c index 8e20360..d83e7f5 100644 ---

Re: [Qemu-devel] [PATCH] lance: unbreak after memory API conversion

2011-08-09 Thread Edgar E. Iglesias
On Tue, Aug 09, 2011 at 09:54:22AM +0300, Avi Kivity wrote: The conversion passed the wrong opaque pointer, causing a crash on first use. Pass the correct opaque. I've applied this, thanks. Signed-off-by: Avi Kivity a...@redhat.com --- hw/lance.c |2 +- 1 files changed, 1

Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Peter Maydell
On 9 August 2011 07:34, Avi Kivity a...@redhat.com wrote: Also, my patchset focuses on mechanical transformations.  It is already risky enough in terms of regressions, I'm not going to rewrite/improve all of qemu; if you want those callbacks removed, you will have to remove them yourself.

Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Avi Kivity
On 08/09/2011 10:37 AM, Peter Maydell wrote: On 9 August 2011 07:34, Avi Kivitya...@redhat.com wrote: Also, my patchset focuses on mechanical transformations. It is already risky enough in terms of regressions, I'm not going to rewrite/improve all of qemu; if you want those callbacks

Re: [Qemu-devel] [PATCH 08/24] tusb6010: move declarations to new file tusb6010.h

2011-08-09 Thread Peter Maydell
On 8 August 2011 18:06, Avi Kivity a...@redhat.com wrote: diff --git a/hw/tusb6010.h b/hw/tusb6010.h new file mode 100644 index 000..6faa94d --- /dev/null +++ b/hw/tusb6010.h @@ -0,0 +1,10 @@ +#ifndef TUSB6010_H +#define TUSB6010_H + +typedef struct TUSBState TUSBState; +TUSBState

Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Peter Maydell
On 9 August 2011 08:41, Avi Kivity a...@redhat.com wrote: On 08/09/2011 10:37 AM, Peter Maydell wrote: On 9 August 2011 07:34, Avi Kivitya...@redhat.com  wrote:  Also, my patchset focuses on mechanical transformations.  It is already  risky enough in terms of regressions, I'm not going to

Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Avi Kivity
On 08/09/2011 11:07 AM, Peter Maydell wrote: On 9 August 2011 08:41, Avi Kivitya...@redhat.com wrote: On 08/09/2011 10:37 AM, Peter Maydell wrote: On 9 August 2011 07:34, Avi Kivitya...@redhat.comwrote: Also, my patchset focuses on mechanical transformations. It is already

Re: [PATCH v5 2/4] block: add the block queue support

2011-08-09 Thread Ram Pai
On Tue, Aug 09, 2011 at 12:17:50PM +0800, Zhi Yong Wu wrote: The patch introduce one block queue for QEMU block layer. Signed-off-by: Zhi Yong Wu wu...@linux.vnet.ibm.com --- block/blk-queue.c | 141 + block/blk-queue.h | 73

Re: [Qemu-devel] [PATCH v5 2/4] block: add the block queue support

2011-08-09 Thread Zhi Yong Wu
On Tue, Aug 9, 2011 at 4:46 PM, Ram Pai linux...@us.ibm.com wrote: On Tue, Aug 09, 2011 at 12:17:50PM +0800, Zhi Yong Wu wrote: The patch introduce one block queue for QEMU block layer. Signed-off-by: Zhi Yong Wu wu...@linux.vnet.ibm.com ---  block/blk-queue.c |  141

Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Avi Kivity
On 08/09/2011 11:44 AM, Avi Kivity wrote: ...isn't this just throwing away the warnings on bad-width accesses? It is; will fix. Reading the original code, it seems broken: uint32_t omap_badwidth_read32(void *opaque, target_phys_addr_t addr) { uint32_t ret; OMAP_32B_REG(addr);

Re: [PATCH v5 3/4] block: add block timer and block throttling algorithm

2011-08-09 Thread Ram Pai
On Tue, Aug 09, 2011 at 12:17:51PM +0800, Zhi Yong Wu wrote: Note: 1.) When bps/iops limits are specified to a small value such as 511 bytes/s, this VM will hang up. We are considering how to handle this senario. 2.) When dd command is issued in guest, if its option bs is set to a

[PATCH v1.1 08/24] tusb6010: move declarations to new file tusb6010.h

2011-08-09 Thread Avi Kivity
Avoid #include hell. Signed-off-by: Avi Kivity a...@redhat.com --- v1.1: add copyright/license blurb hw/devices.h |7 --- hw/nseries.c |1 + hw/tusb6010.c |2 +- hw/tusb6010.h | 25 + 4 files changed, 27 insertions(+), 8 deletions(-) create mode

[PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Avi Kivity
Somewhat clumsy since it needs a variable sized region. Signed-off-by: Avi Kivity a...@redhat.com --- v1.1: set access size validity for omap gpmc region to allow only 32-bit accesses hw/omap.h |3 ++- hw/omap_gpmc.c | 56

Re: [Qemu-devel] [PATCH v5 3/4] block: add block timer and block throttling algorithm

2011-08-09 Thread Zhi Yong Wu
On Tue, Aug 9, 2011 at 4:57 PM, Ram Pai linux...@us.ibm.com wrote: On Tue, Aug 09, 2011 at 12:17:51PM +0800, Zhi Yong Wu wrote: Note:       1.) When bps/iops limits are specified to a small value such as 511 bytes/s, this VM will hang up. We are considering how to handle this senario.      

Re: KVM cpu limitations

2011-08-09 Thread Avi Kivity
On 08/08/2011 10:18 PM, John Paul Walters wrote: On Jul 21, 2011, at 2:10 AM, Avi Kivity wrote: On 07/21/2011 02:20 AM, John Paul Walters wrote: Hi, We have a 256 core SGI Ultraviolet machine running RHEL 6.1 with qemu-kvm 0.13, and we'd like to be able to start large guest VMs of up to

Re: kvm linux guest hanging for minutes at a time

2011-08-09 Thread Avi Kivity
On 08/07/2011 05:06 PM, Thomas Fjellstrom wrote: Occasionally when there's heavy cpu and/or io load, a kvm guest will lock up for minutes at a time, last occurrence was for about 12 minutes or so, and the guest itself reported: [1992982.639514] Clocksource tsc unstable (delta = -747307707123

Re: [PATCH v4 02/10] ARM: KVM: Hypervisor identity mapping

2011-08-09 Thread Avi Kivity
On 08/06/2011 01:39 PM, Christoffer Dall wrote: Adds support in the identity mapping feature that allows KVM to setup identity mapping for the Hyp mode with the AP[1] bit set as required by the specification and also supports freeing created sub pmd's after finished use. These two functions:

KVM call agenda for August 8

2011-08-09 Thread Juan Quintela
Hi Please send in any agenda items you are interested in covering. Later, Juan. PD. I was on vacation, and I forget to send the call for agenda before. Sorry. -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo

Re: [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Peter Maydell
On 9 August 2011 10:02, Avi Kivity a...@redhat.com wrote: +static const MemoryRegionOps omap_gpmc_ops = { +    .read = omap_gpmc_read, +    .write = omap_gpmc_write, +    .endianness = DEVICE_NATIVE_ENDIAN, +    .valid = { +        .min_access_size = 4, +        .max_access_size = 4, +    

Re: [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Avi Kivity
On 08/09/2011 12:23 PM, Peter Maydell wrote: On 9 August 2011 10:02, Avi Kivitya...@redhat.com wrote: +static const MemoryRegionOps omap_gpmc_ops = { +.read = omap_gpmc_read, +.write = omap_gpmc_write, +.endianness = DEVICE_NATIVE_ENDIAN, +.valid = { +

Re: [PATCH v4 02/10] ARM: KVM: Hypervisor identity mapping

2011-08-09 Thread Christoffer Dall
On Aug 9, 2011, at 11:20 AM, Avi Kivity wrote: On 08/06/2011 01:39 PM, Christoffer Dall wrote: Adds support in the identity mapping feature that allows KVM to setup identity mapping for the Hyp mode with the AP[1] bit set as required by the specification and also supports freeing created sub

[PATCH 2/2] kvm tools: Fix wrong help message for kvm pause

2011-08-09 Thread Liming Wang
From: walimis walimis...@gmail.com replace debug with pause for kvm pause. Signed-off-by: walimis walimis...@gmail.com --- tools/kvm/builtin-pause.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/kvm/builtin-pause.c b/tools/kvm/builtin-pause.c index

[PATCH 1/2] kvm tools: remove unused options

2011-08-09 Thread Liming Wang
From: walimis walimis...@gmail.com kvm --help and --version are not implemented, so remove them to avoid ambiguous. Signed-off-by: walimis walimis...@gmail.com --- tools/kvm/builtin-help.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/kvm/builtin-help.c

Re: [PATCH v4 02/10] ARM: KVM: Hypervisor identity mapping

2011-08-09 Thread Catalin Marinas
On Tue, Aug 09, 2011 at 10:20:27AM +0100, Avi Kivity wrote: On 08/06/2011 01:39 PM, Christoffer Dall wrote: Adds support in the identity mapping feature that allows KVM to setup identity mapping for the Hyp mode with the AP[1] bit set as required by the specification and also supports

Re: [Qemu-devel] [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Peter Maydell
On 9 August 2011 10:26, Avi Kivity a...@redhat.com wrote: On 08/09/2011 12:23 PM, Peter Maydell wrote: On 9 August 2011 10:02, Avi Kivitya...@redhat.com  wrote:  +static const MemoryRegionOps omap_gpmc_ops = {  +    .read = omap_gpmc_read,  +    .write = omap_gpmc_write,  +    

Re: [PATCH v4 04/10] ARM: KVM: Memory virtualization setup

2011-08-09 Thread Avi Kivity
On 08/06/2011 01:39 PM, Christoffer Dall wrote: This commit introduces the framework for guest memory management through the use of 2nd stage translation. Each VM has a pointer to a level-1 tabled (the pgd field in struct kvm_arch) which is used for the 2nd stage translations. Entries are added

Re: [PATCH v4 05/10] ARM: KVM: Inject IRQs and FIQs from userspace

2011-08-09 Thread Avi Kivity
On 08/06/2011 01:39 PM, Christoffer Dall wrote: Userspace can inject IRQs and FIQs through the KVM_IRQ_LINE VM ioctl. This ioctl is used since the sematics are in fact two lines that can be either raised or lowered on the VCPU - the IRQ and FIQ lines. KVM needs to know which VCPU it must

Re: [Qemu-devel] [PATCH v4 14/39] ac97: convert to memory API

2011-08-09 Thread malc
On Mon, 8 Aug 2011, Avi Kivity wrote: fixes BAR sizing as well. I'm fine with this version, thanks. [..snip..] -- mailto:av1...@comtv.ru -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [Qemu-devel] [PATCH v4 17/39] es1370: convert to memory API

2011-08-09 Thread malc
On Mon, 8 Aug 2011, Avi Kivity wrote: Fine with this too, thanks. [..snip..] -- mailto:av1...@comtv.ru -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH] Fix SMEP issues

2011-08-09 Thread Yang, Wei Y
This patch fix kvm-unit-tests hanging and incorrect PT_ACCESSED_MASK bit set in the case of SMEP fault. Signed-off-by: Yang, Wei wei.y.y...@intel.com --- arch/x86/kvm/paging_tmpl.h | 22 +- 1 files changed, 13 insertions(+), 9 deletions(-) diff --git

[PATCH] kvm tools: Change method of retrieving process name

2011-08-09 Thread Sasha Levin
This patch changes './kvm list' to retrieve process name from '/proc/pid/stat' instead of '/proc/pid/comm' as it appears the latter does not exist by default on several systems. Reported-by: pradeep psuri...@linux.vnet.ibm.com Signed-off-by: Sasha Levin levinsasha...@gmail.com ---

Re: [Qemu-devel] [PATCH] Introduce short names for fixed width integer types

2011-08-09 Thread malc
On Mon, 8 Aug 2011, Peter Maydell wrote: On 8 August 2011 13:56, Avi Kivity a...@redhat.com wrote: QEMU deals with a lot of fixed width integer types; their names (uint64_t etc) are clumsy to use and take up a lot of space. Following Linux, introduce shorter names, for example U64 for

Re: [Android-virt] [PATCH v4 02/10] ARM: KVM: Hypervisor identity mapping

2011-08-09 Thread Alexey Smirnov
Hi Christoffer, These are for mapping host memory, not guest memory, right? yes (or to be exact - hypervisor memory). The point is that there are special hardware requirements for translation tables used in Hyp-mode not otherwise satisfied by the normal page tables. In function

Re: [PATCH] Adds cgroup handling library

2011-08-09 Thread Jiri Zupka
ACK nice work. May be cgroup_common should be more general and placed in client/common_lib and can work general tools for manipulating with cgroups. - Original Message - [new] cgroup_common.py * library for handling cgroups Signed-off-by: Lukas Doktor ldok...@redhat.com ---

Re: [PATCH v4 06/10] ARM: KVM: World-switch implementation

2011-08-09 Thread Avi Kivity
On 08/06/2011 01:39 PM, Christoffer Dall wrote: Provides complete world-switch implementation to switch to other guests runinng in non-secure modes. Includes Hyp exception handlers that captures necessary exception information and stores the information on the VCPU and KVM structures. Switching

Re: [PATCH v4 07/10] ARM: KVM: Emulation framework and CP15 emulation

2011-08-09 Thread Avi Kivity
On 08/06/2011 01:39 PM, Christoffer Dall wrote: Adds a new important function in the main KVM/ARM code called handle_exit() which is called from kvm_arch_vcpu_ioctl_run() on returns from guest execution. This function examines the Hyp-Syndrome-Register (HSR), which contains information telling

Re: [Android-virt] [PATCH v4 02/10] ARM: KVM: Hypervisor identity mapping

2011-08-09 Thread Christoffer Dall
On Aug 9, 2011, at 12:23 PM, Alexey Smirnov wrote: Hi Christoffer, These are for mapping host memory, not guest memory, right? yes (or to be exact - hypervisor memory). The point is that there are special hardware requirements for translation tables used in Hyp-mode not otherwise

Re: [Android-virt] [PATCH v4 04/10] ARM: KVM: Memory virtualization setup

2011-08-09 Thread Christoffer Dall
On Aug 9, 2011, at 11:57 AM, Avi Kivity wrote: On 08/06/2011 01:39 PM, Christoffer Dall wrote: This commit introduces the framework for guest memory management through the use of 2nd stage translation. Each VM has a pointer to a level-1 tabled (the pgd field in struct kvm_arch) which is

Re: [PATCH v4 08/10] ARM: KVM: Handle guest faults in KVM

2011-08-09 Thread Avi Kivity
On 08/06/2011 01:39 PM, Christoffer Dall wrote: Handles the guest faults in KVM by mapping in corresponding user pages in the 2nd stage page tables. Introduces new ARM-specific kernel memory types, PAGE_KVM_GUEST and pgprot_guest variables used to map 2nd stage memory for KVM guests. +static

Re: [Android-virt] [PATCH v4 05/10] ARM: KVM: Inject IRQs and FIQs from userspace

2011-08-09 Thread Christoffer Dall
On Aug 9, 2011, at 12:07 PM, Avi Kivity wrote: On 08/06/2011 01:39 PM, Christoffer Dall wrote: Userspace can inject IRQs and FIQs through the KVM_IRQ_LINE VM ioctl. This ioctl is used since the sematics are in fact two lines that can be either raised or lowered on the VCPU - the IRQ and FIQ

[PATCH] nVMX: Document 'nested' parameter

2011-08-09 Thread Sasha Levin
Add documentation of the new 'nested' parameter to 'Documentation/kernel-parameters.txt'. Cc: Avi Kivity a...@redhat.com Cc: Marcelo Tosatti mtosa...@redhat.com Cc: Nadav Har'El n...@il.ibm.com Signed-off-by: Sasha Levin levinsasha...@gmail.com --- Documentation/kernel-parameters.txt |4

Re: [PATCH v4 06/10] ARM: KVM: World-switch implementation

2011-08-09 Thread Christoffer Dall
On Aug 9, 2011, at 1:09 PM, Avi Kivity wrote: On 08/06/2011 01:39 PM, Christoffer Dall wrote: Provides complete world-switch implementation to switch to other guests runinng in non-secure modes. Includes Hyp exception handlers that captures necessary exception information and stores the

Re: [PATCH v4 07/10] ARM: KVM: Emulation framework and CP15 emulation

2011-08-09 Thread Christoffer Dall
On Aug 9, 2011, at 1:17 PM, Avi Kivity wrote: On 08/06/2011 01:39 PM, Christoffer Dall wrote: Adds a new important function in the main KVM/ARM code called handle_exit() which is called from kvm_arch_vcpu_ioctl_run() on returns from guest execution. This function examines the

Re: [PATCH v4 08/10] ARM: KVM: Handle guest faults in KVM

2011-08-09 Thread Christoffer Dall
On Aug 9, 2011, at 1:24 PM, Avi Kivity wrote: On 08/06/2011 01:39 PM, Christoffer Dall wrote: Handles the guest faults in KVM by mapping in corresponding user pages in the 2nd stage page tables. Introduces new ARM-specific kernel memory types, PAGE_KVM_GUEST and pgprot_guest variables

Re: [PATCH v4 09/10] ARM: KVM: Handle I/O aborts

2011-08-09 Thread Avi Kivity
On 08/06/2011 01:40 PM, Christoffer Dall wrote: When the guest accesses I/O memory this will create data abort exceptions and they are handled by decoding the HSR information (physical address, read/write, length, register) and forwarding reads and writes to QEMU which performs the device

Re: [Android-virt] [PATCH v4 05/10] ARM: KVM: Inject IRQs and FIQs from userspace

2011-08-09 Thread Avi Kivity
On 08/09/2011 02:27 PM, Christoffer Dall wrote: + + if (irq_level-level) { + vcpu-arch.virt_irq |= mask; + vcpu-arch.wait_for_interrupts = 0; + } else + vcpu-arch.virt_irq= ~mask; + This seems to be non-smp-safe? Do you need atomic

Re: [PATCH v4 09/10] ARM: KVM: Handle I/O aborts

2011-08-09 Thread Christoffer Dall
On Aug 9, 2011, at 1:34 PM, Avi Kivity wrote: On 08/06/2011 01:40 PM, Christoffer Dall wrote: When the guest accesses I/O memory this will create data abort exceptions and they are handled by decoding the HSR information (physical address, read/write, length, register) and forwarding reads

Re: [PATCH v4 07/10] ARM: KVM: Emulation framework and CP15 emulation

2011-08-09 Thread Avi Kivity
On 08/09/2011 02:34 PM, Christoffer Dall wrote: + +/** + * emulate_cp15_c15_access -- emulates cp15 accesses for CRn == 15 + * @vcpu: The VCPU pointer + * @p:The coprocessor parameters struct pointer holding trap inst. details + * + * The CP15 c15 register is implementation

Re: [Android-virt] [PATCH v4 05/10] ARM: KVM: Inject IRQs and FIQs from userspace

2011-08-09 Thread Christoffer Dall
On Aug 9, 2011, at 1:37 PM, Avi Kivity wrote: On 08/09/2011 02:27 PM, Christoffer Dall wrote: + +if (irq_level-level) { +vcpu-arch.virt_irq |= mask; +vcpu-arch.wait_for_interrupts = 0; +} else +

Re: [PATCH v4 07/10] ARM: KVM: Emulation framework and CP15 emulation

2011-08-09 Thread Christoffer Dall
On Aug 9, 2011, at 1:39 PM, Avi Kivity wrote: On 08/09/2011 02:34 PM, Christoffer Dall wrote: + +/** + * emulate_cp15_c15_access -- emulates cp15 accesses for CRn == 15 + * @vcpu: The VCPU pointer + * @p:The coprocessor parameters struct pointer holding trap inst.

Re: [PATCH v4 00/10] KVM/ARM Implementation

2011-08-09 Thread Avi Kivity
On 08/06/2011 01:38 PM, Christoffer Dall wrote: The following series implements KVM support for ARM processors, specifically on the Cortex A-15 platform. The patch series applies to the arm-lpae branch of ARM Ltd's kernel tree. This is Version 4 of the patch series, but the first two versions

Re: [PATCH v4 09/10] ARM: KVM: Handle I/O aborts

2011-08-09 Thread Avi Kivity
On 08/09/2011 02:39 PM, Christoffer Dall wrote: + +static u32 ls_instr[NUM_LS_INSTR][2] = { + {0x0470, 0x0d70}, /* LDRBT */ + {0x0430, 0x0d70}, /* LDRT */ + {0x0410, 0x0c50}, /* LDR */ + {0x0450, 0x0c50}, /* LDRB */ +

Re: kvm linux guest hanging for minutes at a time

2011-08-09 Thread Thomas Fjellstrom
On August 9, 2011, Avi Kivity wrote: On 08/07/2011 05:06 PM, Thomas Fjellstrom wrote: Occasionally when there's heavy cpu and/or io load, a kvm guest will lock up for minutes at a time, last occurrence was for about 12 minutes or so, and the guest itself reported: [1992982.639514]

Re: [PATCH v5 0/4] The intro of QEMU block I/O throttling

2011-08-09 Thread Stefan Hajnoczi
On Tue, Aug 9, 2011 at 5:17 AM, Zhi Yong Wu wu...@linux.vnet.ibm.com wrote:  Makefile.objs     |    2 +-  block.c           |  347 +++--  block.h           |    6 +-  block/blk-queue.c |  141 ++  block/blk-queue.h |   73

Re: [Qemu-devel] [PATCH v5 1/4] block: add the command line support

2011-08-09 Thread Stefan Hajnoczi
On Tue, Aug 9, 2011 at 5:17 AM, Zhi Yong Wu wu...@linux.vnet.ibm.com wrote: Signed-off-by: Zhi Yong Wu wu...@linux.vnet.ibm.com ---  Makefile.objs   |    2 +-  blockdev.c      |   39 +++  qemu-config.c   |   24  qemu-option.c   |  

Re: kvm linux guest hanging for minutes at a time

2011-08-09 Thread Avi Kivity
On 08/09/2011 03:03 PM, Thomas Fjellstrom wrote: Your guest isn't using kvmclock for some reason. Is it compiled in the guest kernel? What are the contents of /sys/devices/system/clocksource/clocksource0/available_clocksource and

Re: [Qemu-devel] KVM call agenda for August 8

2011-08-09 Thread Anthony Liguori
On 08/09/2011 04:21 AM, Juan Quintela wrote: Hi Please send in any agenda items you are interested in covering. I think we can skip since many people will be in Vancouver next week and we'll have plenty to talk about then. Just a couple reminders, for KVM Forum speakers, please post your

Re: [PATCH] Introduce short names for fixed width integer types

2011-08-09 Thread Juan Quintela
Peter Maydell peter.mayd...@linaro.org wrote: On 8 August 2011 13:56, Avi Kivity a...@redhat.com wrote: QEMU deals with a lot of fixed width integer types; their names (uint64_t etc) are clumsy to use and take up a lot of space. Following Linux, introduce shorter names, for example U64 for

Re: [Qemu-devel] [PATCH v4 26/39] pcnet: convert to memory API

2011-08-09 Thread Michael S. Tsirkin
On Tue, Aug 09, 2011 at 09:52:17AM +0300, Avi Kivity wrote: On 08/09/2011 09:55 AM, Bob Breuer wrote: static void lance_cleanup(VLANClientState *nc) @@ -117,13 +116,11 @@ static int lance_init(SysBusDevice *dev) SysBusPCNetState *d = FROM_SYSBUS(SysBusPCNetState, dev);

Re: [Qemu-devel] [PATCH v4 26/39] pcnet: convert to memory API

2011-08-09 Thread Avi Kivity
On 08/09/2011 03:42 PM, Michael S. Tsirkin wrote: On Tue, Aug 09, 2011 at 09:52:17AM +0300, Avi Kivity wrote: On 08/09/2011 09:55 AM, Bob Breuer wrote: static void lance_cleanup(VLANClientState *nc) @@ -117,13 +116,11 @@ static int lance_init(SysBusDevice *dev)

Re: [Qemu-devel] [PATCH v4 26/39] pcnet: convert to memory API

2011-08-09 Thread Michael S. Tsirkin
On Tue, Aug 09, 2011 at 03:44:35PM +0300, Avi Kivity wrote: On 08/09/2011 03:42 PM, Michael S. Tsirkin wrote: On Tue, Aug 09, 2011 at 09:52:17AM +0300, Avi Kivity wrote: On 08/09/2011 09:55 AM, Bob Breuer wrote: static void lance_cleanup(VLANClientState *nc) @@ -117,13 +116,11 @@

Re: [PATCH v5 2/4] block: add the block queue support

2011-08-09 Thread Stefan Hajnoczi
On Tue, Aug 9, 2011 at 5:17 AM, Zhi Yong Wu wu...@linux.vnet.ibm.com wrote: +/* The APIs for block request queue on qemu block layer. + */ + +static void qemu_block_queue_cancel(BlockDriverAIOCB *acb) +{ +    qemu_aio_release(acb); +} + +static AIOPool block_queue_pool = { +    

Re: [Qemu-devel] [PATCH v4 26/39] pcnet: convert to memory API

2011-08-09 Thread Avi Kivity
On 08/09/2011 03:48 PM, Michael S. Tsirkin wrote: But in some cases, we can't, and the it's a pain having to wrap MemoryRegion in another structure containing an opaque. I guess, even though that wrapping structure would use a proper type, not an opaque. Yes, of course - that's what the

Re: [Qemu-devel] [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Avi Kivity
On 08/09/2011 12:41 PM, Peter Maydell wrote: On 9 August 2011 10:26, Avi Kivitya...@redhat.com wrote: On 08/09/2011 12:23 PM, Peter Maydell wrote: On 9 August 2011 10:02, Avi Kivitya...@redhat.comwrote: +static const MemoryRegionOps omap_gpmc_ops = { +.read =

[PATCH v1.2 09/24] omap_gpmc/nseries/tusb6010: convert to memory API

2011-08-09 Thread Avi Kivity
Somewhat clumsy since it needs a variable sized region. Signed-off-by: Avi Kivity a...@redhat.com --- v1.2: drop v1.1 changes, instead forward invalid size accesses to original omap_badwidth_*() functions v1.1: set access size validity for omap gpmc region to allow only 32-bit

Re: [PATCH 1/2] kvm tools: remove unused options

2011-08-09 Thread Pekka Enberg
On Tue, 9 Aug 2011, Liming Wang wrote: From: walimis walimis...@gmail.com kvm --help and --version are not implemented, so remove them to avoid ambiguous. Signed-off-by: walimis walimis...@gmail.com The patch looks good but unfortunately I have to ask you to resend with proper

Re: [PATCH 2/2] kvm tools: Fix wrong help message for kvm pause

2011-08-09 Thread Pekka Enberg
On Tue, 9 Aug 2011, Liming Wang wrote: From: walimis walimis...@gmail.com replace debug with pause for kvm pause. Signed-off-by: walimis walimis...@gmail.com --- tools/kvm/builtin-pause.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/kvm/builtin-pause.c

Re: KVM call agenda for August 8

2011-08-09 Thread Juan Quintela
Anthony Liguori anth...@codemonkey.ws wrote: On 08/09/2011 04:21 AM, Juan Quintela wrote: Hi Please send in any agenda items you are interested in covering. I think we can skip since many people will be in Vancouver next week and we'll have plenty to talk about then. Just a couple

[PATCH] Fix build error with --disable-cpu-emulation

2011-08-09 Thread Arjan Koers
Fix build error with --disable-cpu-emulation which did break in 4183a43d1405fbb9da3ccb3dcff223266a36df9f Signed-off-by: Arjan Koers 0h61vkll2...@xutrox.com --- target-i386/fake-exec.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/target-i386/fake-exec.c

[PATCH 1/2] kvm tools: remove unused options

2011-08-09 Thread walimis
kvm --help and --version are not implemented, so remove them to avoid ambiguous. Signed-off-by: walimis walimis...@gmail.com --- tools/kvm/builtin-help.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/kvm/builtin-help.c b/tools/kvm/builtin-help.c index

Re: kvm linux guest hanging for minutes at a time

2011-08-09 Thread Thomas Fjellstrom
On August 9, 2011, Avi Kivity wrote: On 08/09/2011 03:03 PM, Thomas Fjellstrom wrote: Your guest isn't using kvmclock for some reason. Is it compiled in the guest kernel? What are the contents of /sys/devices/system/clocksource/clocksource0/available_clocksource and

Re: kvm linux guest hanging for minutes at a time

2011-08-09 Thread Avi Kivity
On 08/09/2011 05:31 PM, Thomas Fjellstrom wrote: On August 9, 2011, Avi Kivity wrote: On 08/09/2011 03:03 PM, Thomas Fjellstrom wrote: Your guest isn't using kvmclock for some reason. Is it compiled in the guest kernel? What are the contents of

Re: kvm linux guest hanging for minutes at a time

2011-08-09 Thread Thomas Fjellstrom
On August 9, 2011, Avi Kivity wrote: On 08/09/2011 05:31 PM, Thomas Fjellstrom wrote: On August 9, 2011, Avi Kivity wrote: On 08/09/2011 03:03 PM, Thomas Fjellstrom wrote: Your guest isn't using kvmclock for some reason. Is it compiled in the guest kernel? What are the

Re: kvm linux guest hanging for minutes at a time

2011-08-09 Thread Avi Kivity
On 08/09/2011 05:46 PM, Thomas Fjellstrom wrote: Does it matter that I have several vms running? Is there a way to limit it to tracing the single kvm process that's been locking up? You can use trace-cmd record -F ... qemu ... but that misses out on events the run from workqueues.

Re: kvm linux guest hanging for minutes at a time

2011-08-09 Thread Thomas Fjellstrom
On August 9, 2011, Avi Kivity wrote: On 08/09/2011 05:46 PM, Thomas Fjellstrom wrote: Does it matter that I have several vms running? Is there a way to limit it to tracing the single kvm process that's been locking up? You can use trace-cmd record -F ... qemu ... but that

Re: [PATCH] kvm tools: Change method of retrieving process name

2011-08-09 Thread Ingo Molnar
* Sasha Levin levinsasha...@gmail.com wrote: This patch changes './kvm list' to retrieve process name from '/proc/pid/stat' instead of '/proc/pid/comm' as it appears the latter does not exist by default on several systems. Reported-by: pradeep psuri...@linux.vnet.ibm.com Signed-off-by:

Re: [PATCH] kvm tools: Change method of retrieving process name

2011-08-09 Thread Pekka Enberg
On Tue, Aug 9, 2011 at 5:53 PM, Ingo Molnar mi...@elte.hu wrote: * Sasha Levin levinsasha...@gmail.com wrote: This patch changes './kvm list' to retrieve process name from '/proc/pid/stat' instead of '/proc/pid/comm' as it appears the latter does not exist by default on several systems.

[PATCH] kvm tools: Use overlayfs when mounting rootfs

2011-08-09 Thread Sasha Levin
This patch uses (the not yet merged) overlayfs to allow booting complex distribution rootfs and provide a COW layer within the rootfs. We use overlayfs for two reasons: - Overwrite the /etc/fstab file so that the mounter will use the virtio-9p device as root instead of /dev/vda. - Provide a

Re: [PATCH] kvm tools: Change method of retrieving process name

2011-08-09 Thread Ingo Molnar
* Pekka Enberg penb...@kernel.org wrote: On Tue, Aug 9, 2011 at 5:53 PM, Ingo Molnar mi...@elte.hu wrote: * Sasha Levin levinsasha...@gmail.com wrote: This patch changes './kvm list' to retrieve process name from '/proc/pid/stat' instead of '/proc/pid/comm' as it appears the latter

Re: kvm linux guest hanging for minutes at a time

2011-08-09 Thread Thomas Fjellstrom
On August 9, 2011, Avi Kivity wrote: On 08/09/2011 05:46 PM, Thomas Fjellstrom wrote: Does it matter that I have several vms running? Is there a way to limit it to tracing the single kvm process that's been locking up? You can use trace-cmd record -F ... qemu ... but that

Re: [PATCH] kvm tools: Change method of retrieving process name

2011-08-09 Thread Sasha Levin
On Tue, 2011-08-09 at 17:56 +0300, Pekka Enberg wrote: On Tue, Aug 9, 2011 at 5:53 PM, Ingo Molnar mi...@elte.hu wrote: * Sasha Levin levinsasha...@gmail.com wrote: This patch changes './kvm list' to retrieve process name from '/proc/pid/stat' instead of '/proc/pid/comm' as it appears

Re: [PATCH] kvm tools: Use overlayfs when mounting rootfs

2011-08-09 Thread Pekka Enberg
On Tue, Aug 9, 2011 at 5:57 PM, Sasha Levin levinsasha...@gmail.com wrote: This patch uses (the not yet merged) overlayfs to allow booting complex distribution rootfs and provide a COW layer within the rootfs. We use overlayfs for two reasons:  - Overwrite the /etc/fstab file so that the

[Bug 40752] New: BUG: unable to handle kernel paging request at bffffc20

2011-08-09 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=40752 Summary: BUG: unable to handle kernel paging request at bc20 Product: Virtualization Version: unspecified Kernel Version: 3.0 Platform: All OS/Version: Linux

[PATCH 1/2] kvm tools: remove unused options

2011-08-09 Thread walimis
kvm --help and --version are not implemented, so remove them to avoid ambiguous. Signed-off-by: Liming Wang walimis...@gmail.com --- tools/kvm/builtin-help.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/kvm/builtin-help.c b/tools/kvm/builtin-help.c index

Re: [PATCH] kvm tools: Change method of retrieving process name

2011-08-09 Thread Pekka Enberg
On Tue, Aug 9, 2011 at 6:02 PM, Sasha Levin levinsasha...@gmail.com wrote: On Tue, 2011-08-09 at 17:56 +0300, Pekka Enberg wrote: On Tue, Aug 9, 2011 at 5:53 PM, Ingo Molnar mi...@elte.hu wrote: * Sasha Levin levinsasha...@gmail.com wrote: This patch changes './kvm list' to retrieve

Re: [PATCH v5 3/4] block: add block timer and block throttling algorithm

2011-08-09 Thread Stefan Hajnoczi
On Tue, Aug 9, 2011 at 5:17 AM, Zhi Yong Wu wu...@linux.vnet.ibm.com wrote: Note:      1.) When bps/iops limits are specified to a small value such as 511 bytes/s, this VM will hang up. We are considering how to handle this senario. If an I/O request is larger than the limit itself then I

[PATCH] kvm tools: Use correct size for VESA memory BAR

2011-08-09 Thread Sasha Levin
This patch makes BAR 1 16k, instead of BAR0 - which is the PIO bar. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/hw/vesa.c |2 +- tools/kvm/pci.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/kvm/hw/vesa.c b/tools/kvm/hw/vesa.c index

[PATCH] kvm tools: Boot to host /bin/sh via 9p by default

2011-08-09 Thread Pekka Enberg
This patch updates 'kvm run' to boot to host filesystem via 9p '/bin/sh' by default: $ ./kvm run # kvm run -k ../../arch/x86/boot/bzImage -m 320 -c 2 --name guest-3462 [0.00] Linux version 3.1.0-rc1+ (penberg@tiger) (gcc version 4.4.3 (Ubuntu 4.4.3-4ubuntu5) ) #7 SMP PREEMPT Tue

Re: [PATCH] kvm tools: Boot to host /bin/sh via 9p by default

2011-08-09 Thread Pekka Enberg
On Tue, Aug 9, 2011 at 6:33 PM, Pekka Enberg penb...@kernel.org wrote: This patch updates 'kvm run' to boot to host filesystem via 9p '/bin/sh' by default: Ingo, I think you're the only one that relied on the old behavior. Are you OK with this new default? -- To unsubscribe from this list: send

Re: [PATCH] kvm tools: Boot to host /bin/sh via 9p by default

2011-08-09 Thread Ingo Molnar
* Pekka Enberg penb...@kernel.org wrote: On Tue, Aug 9, 2011 at 6:33 PM, Pekka Enberg penb...@kernel.org wrote: This patch updates 'kvm run' to boot to host filesystem via 9p '/bin/sh' by default: Ingo, I think you're the only one that relied on the old behavior. Are you OK with this

Re: [PATCH] kvm tools: Use correct size for VESA memory BAR

2011-08-09 Thread Ingo Molnar
* Sasha Levin levinsasha...@gmail.com wrote: This patch makes BAR 1 16k, instead of BAR0 - which is the PIO bar. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/hw/vesa.c |2 +- tools/kvm/pci.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) This

Re: [PATCH 1/2] kvm tools: remove unused options

2011-08-09 Thread Ingo Molnar
* walimis walimis...@gmail.com wrote: kvm --help and --version are not implemented, so remove them to avoid ambiguous. would be nice to implement them: 'kvm --help' should probably map to 'kvm help', and 'kvm --version' should output something similar to what 'perf --version' does: $ perf

Re: [PATCH 1/2] kvm tools: remove unused options

2011-08-09 Thread Pekka Enberg
On Tue, 9 Aug 2011, Ingo Molnar wrote: kvm --help and --version are not implemented, so remove them to avoid ambiguous. would be nice to implement them: 'kvm --help' should probably map to 'kvm help', and 'kvm --version' should output something similar to what 'perf --version' does: $ perf

Re: kvm linux guest hanging for minutes at a time

2011-08-09 Thread Nick
Hi, Just joined this list, looking for leads to solve a similar-sounding problem (guest processes hanging for seconds or minutes when host IO load is high). I'll say more in a separate email, but I caught the end of this thread and wanted to ask about kvm-clock. Naively I'd have thought that

[PATCH 05/10] KVM: PPC: Read out syscall instruction on trap

2011-08-09 Thread Alexander Graf
We have a few traps where we cache the instruction that cause the trap for analysis later on. Since we now need to be able to distinguish between SC 0 and SC 1 system calls and the only way to find out which is which is by looking at the instruction, we also read out the instruction causing the

[PATCH 02/10] KVM: PPC: Add papr_enabled flag

2011-08-09 Thread Alexander Graf
When running a PAPR guest, some things change. The privilege level drops from hypervisor to supervisor, SDR1 gets treated differently and we interpret hypercalls. For bisectability sake, add the flag now, but only enable it when all the support code is there. Signed-off-by: Alexander Graf

[PATCH 10/10] KVM: PPC: Enable the PAPR CAP for Book3S

2011-08-09 Thread Alexander Graf
Now that Book3S PV mode can also run PAPR guests, we can add a PAPR cap and enable it for all Book3S targets. Enabling that CAP switches KVM into PAPR mode. Signed-off-by: Alexander Graf ag...@suse.de --- arch/powerpc/kvm/powerpc.c |5 + include/linux/kvm.h|1 + 2 files

  1   2   >