Re: [PATCH] KVM: x86: remove vcpu's CPL check in host invoked vcpu's xcr set process

2013-06-18 Thread Paolo Bonzini
Il 14/06/2013 09:36, Zhanghaoyu (A) ha scritto: > __kvm_set_xcr function does the CPL check when set xcr. __kvm_set_xcr is > called in two flows, > one is invoked by guest, call stack shown as below, > handle_xsetbv(or xsetbv_interception) > kvm_set_xcr > __kvm_set_xcr > the other one is inv

Re: [PATCH] kvmclock: clock should count only if vm is running

2013-06-18 Thread Paolo Bonzini
Hi Marcelo, sorry for the late review. Il 08/06/2013 04:00, Marcelo Tosatti ha scritto: > kvmclock should not count while vm is paused, because: > > 1) if the vm is paused for long periods, timekeeping > math can overflow while converting the (large) clocksource > delta to nanoseconds. > > 2)

Re: [PATCH qom-cpu v2 02/29] kvm: Change cpu_synchronize_state() argument to CPUState

2013-06-18 Thread Igor Mammedov
On Sun, 16 Jun 2013 17:57:22 +0200 Andreas Färber wrote: > Change Monitor::mon_cpu to CPUState as well. > In cpu_synchronize_all_states() use qemu_for_each_cpu() now. > > Reviewed-by: liguang > Signed-off-by: Andreas Färber > --- > cpus.c | 8 > gdbstub.c

Re: [PATCH v3 03/13] nEPT: Add EPT tables support to paging_tmpl.h

2013-06-18 Thread Gleb Natapov
On Mon, Jun 17, 2013 at 08:11:03PM +0800, Xiao Guangrong wrote: > On 06/11/2013 07:32 PM, Gleb Natapov wrote: > > On Tue, May 21, 2013 at 03:52:12PM +0800, Xiao Guangrong wrote: > >> On 05/19/2013 12:52 PM, Jun Nakajima wrote: > >>> From: Nadav Har'El > >>> > >>> This is the first patch in a serie

Re: [PATCH] KVM: x86: remove vcpu's CPL check in host invoked vcpu's xcr set process

2013-06-18 Thread Zhanghaoyu (A)
>> __kvm_set_xcr function does the CPL check when set xcr. __kvm_set_xcr >> is called in two flows, one is invoked by guest, call stack shown as >> below, handle_xsetbv(or xsetbv_interception) >> kvm_set_xcr >> __kvm_set_xcr >> the other one is invoked by host(QEMU), call stack shown as bel

Re: [PATCH 1/2] kvm-unit-tests: Add a func to run instruction in emulator

2013-06-18 Thread Gleb Natapov
On Thu, Jun 13, 2013 at 05:30:03PM +0800, 李春奇 wrote: > Hi Gleb, > I'm trying to solve these problems in the past days and meet many > difficulties. You want to save all the general registers in calling > insn_page, so registers should be saved to (save) in insn_page. > Because all the instructions

Re: [PATCH v3 03/13] nEPT: Add EPT tables support to paging_tmpl.h

2013-06-18 Thread Xiao Guangrong
On 06/18/2013 06:57 PM, Gleb Natapov wrote: > On Mon, Jun 17, 2013 at 08:11:03PM +0800, Xiao Guangrong wrote: >> On 06/11/2013 07:32 PM, Gleb Natapov wrote: >>> On Tue, May 21, 2013 at 03:52:12PM +0800, Xiao Guangrong wrote: On 05/19/2013 12:52 PM, Jun Nakajima wrote: > From: Nadav Har'El

Re: [PATCH v3 03/13] nEPT: Add EPT tables support to paging_tmpl.h

2013-06-18 Thread Gleb Natapov
On Tue, Jun 18, 2013 at 08:51:25PM +0800, Xiao Guangrong wrote: > On 06/18/2013 06:57 PM, Gleb Natapov wrote: > > On Mon, Jun 17, 2013 at 08:11:03PM +0800, Xiao Guangrong wrote: > >> On 06/11/2013 07:32 PM, Gleb Natapov wrote: > >>> On Tue, May 21, 2013 at 03:52:12PM +0800, Xiao Guangrong wrote: >

Re: [PATCH 1/7] KVM: arm-vgic: Support KVM_CREATE_DEVICE for VGIC

2013-06-18 Thread Alexander Graf
Am 11.06.2013 um 06:51 schrieb Christoffer Dall : > Support creating the ARM VGIC device through the KVM_CREATE_DEVICE > ioctl, which can then later be leveraged to use the > KVM_{GET/SET}_DEVICE_ATTR, which is useful both for setting addresses in > a more generic API than the ARM-specific one a

Re: [PATCH 2/7] KVM: arm-vgic: Set base addr through device API

2013-06-18 Thread Alexander Graf
Am 11.06.2013 um 06:51 schrieb Christoffer Dall : > Support setting the distributor and cpu interface base addresses in the > VM physical address space through the KVM_{SET,GET}_DEVICE_ATTR API > in addition the ARM specific API. > > This has the added benefit of being able to share more code i

Re: [PATCH] KVM: x86: remove vcpu's CPL check in host invoked vcpu's xcr set process

2013-06-18 Thread Paolo Bonzini
Il 18/06/2013 14:39, Zhanghaoyu (A) ha scritto: >> Applied to master, but check your setup because I had to do so manually. >> Your patch has 2 tabs + 1 space for each indentation level (??), the file >> only has one tab. >> >> This time the patch was pretty simple and may even be worth having in

Re: [PATCH 1/2] kvm-unit-tests: Add a func to run instruction in emulator

2013-06-18 Thread 李春奇
On Tue, Jun 18, 2013 at 8:45 PM, Gleb Natapov wrote: > On Thu, Jun 13, 2013 at 05:30:03PM +0800, 李春奇 wrote: >> Hi Gleb, >> I'm trying to solve these problems in the past days and meet many >> difficulties. You want to save all the general registers in calling >> insn_page, so registers should be

Re: [PATCH v2 2/2] kvm: x86: emulate MSR_PLATFORM_INFO

2013-06-18 Thread Paolo Bonzini
Il 04/06/2013 18:02, Bandan Das ha scritto: > +static u64 kvm_get_platform_info(struct kvm_vcpu *vcpu) > +{ > + u8 cpumodel; > + u32 bclk; > + > + /* > + * Programmable Ratio Limit for Turbo Mode (bit 28): 0 > + * Programmable TDC-TDP Limit for Turbo Mode (bit 29): 0 > +

Re: [PATCH v2 0/2] kvm: x86: Emulate MSR_PLATFORM_INFO

2013-06-18 Thread Paolo Bonzini
Il 05/06/2013 10:42, Gleb Natapov ha scritto: >> > These patches add an emulated MSR_PLATFORM_INFO that kvm guests >> > can read as described in section 14.3.2.4 of the Intel SDM. >> > The relevant changes and details are in [2/2]; [1/2] makes vendor_intel >> > generic. There are atleat two known

Re: [PATCH] KVM: x86: fix missed memory synchronization when patch hypercall

2013-06-18 Thread Paolo Bonzini
Il 09/06/2013 14:27, Gleb Natapov ha scritto: > On Sun, Jun 09, 2013 at 08:17:19PM +0800, Xiao Guangrong wrote: >> On 06/09/2013 07:56 PM, Gleb Natapov wrote: >>> On Sun, Jun 09, 2013 at 07:44:03PM +0800, Xiao Guangrong wrote: On 06/09/2013 07:36 PM, Gleb Natapov wrote: > On Sun, Jun 09, 2

Re: [PATCH 1/2] kvm-unit-tests: Add a func to run instruction in emulator

2013-06-18 Thread 李春奇
On Tue, Jun 18, 2013 at 8:45 PM, Gleb Natapov wrote: > On Thu, Jun 13, 2013 at 05:30:03PM +0800, 李春奇 wrote: >> Hi Gleb, >> I'm trying to solve these problems in the past days and meet many >> difficulties. You want to save all the general registers in calling >> insn_page, so registers should be

Re: [PATCH v3 0/6] KVM: MMU: fast invalidate all mmio sptes

2013-06-18 Thread Paolo Bonzini
Il 07/06/2013 10:51, Xiao Guangrong ha scritto: > Changelog: > V3: > All of these changes are from Gleb's review: > 1) rename RET_MMIO_PF_EMU to RET_MMIO_PF_EMULATE. > 2) smartly adjust kvm generation number in kvm_current_mmio_generatio() > to avoid kvm_memslots->generation overflow. >

Re: [PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-06-18 Thread Alex Williamson
On Tue, 2013-06-18 at 14:38 +1000, Benjamin Herrenschmidt wrote: > On Mon, 2013-06-17 at 20:32 -0600, Alex Williamson wrote: > > > Right, we don't want to create dependencies across modules. I don't > > have a vision for how this should work. This is effectively a complete > > side-band to vfio,

Re: [PATCH v2 2/2] kvm: x86: emulate MSR_PLATFORM_INFO

2013-06-18 Thread Bandan Das
Paolo Bonzini writes: > Il 04/06/2013 18:02, Bandan Das ha scritto: >> +static u64 kvm_get_platform_info(struct kvm_vcpu *vcpu) >> +{ >> +u8 cpumodel; >> +u32 bclk; >> + >> +/* >> + * Programmable Ratio Limit for Turbo Mode (bit 28): 0 >> + * Programmable TDC-TDP Limit for Tur

Re: [PATCH v2 0/2] kvm: x86: Emulate MSR_PLATFORM_INFO

2013-06-18 Thread Gleb Natapov
On Tue, Jun 18, 2013 at 04:05:08PM +0200, Paolo Bonzini wrote: > Il 05/06/2013 10:42, Gleb Natapov ha scritto: > >> > These patches add an emulated MSR_PLATFORM_INFO that kvm guests > >> > can read as described in section 14.3.2.4 of the Intel SDM. > >> > The relevant changes and details are in [2

Re: [PATCH] KVM: x86: fix missed memory synchronization when patch hypercall

2013-06-18 Thread Gleb Natapov
On Tue, Jun 18, 2013 at 04:13:02PM +0200, Paolo Bonzini wrote: > Il 09/06/2013 14:27, Gleb Natapov ha scritto: > > On Sun, Jun 09, 2013 at 08:17:19PM +0800, Xiao Guangrong wrote: > >> On 06/09/2013 07:56 PM, Gleb Natapov wrote: > >>> On Sun, Jun 09, 2013 at 07:44:03PM +0800, Xiao Guangrong wrote: >

Re: [PATCH v2 0/2] kvm: x86: Emulate MSR_PLATFORM_INFO

2013-06-18 Thread Bandan Das
Gleb Natapov writes: > On Tue, Jun 18, 2013 at 04:05:08PM +0200, Paolo Bonzini wrote: >> Il 05/06/2013 10:42, Gleb Natapov ha scritto: >> >> > These patches add an emulated MSR_PLATFORM_INFO that kvm guests >> >> > can read as described in section 14.3.2.4 of the Intel SDM. >> >> > The relevant

Re: [PATCH v2 0/2] kvm: x86: Emulate MSR_PLATFORM_INFO

2013-06-18 Thread Gleb Natapov
On Tue, Jun 18, 2013 at 11:29:27AM -0400, Bandan Das wrote: > Gleb Natapov writes: > > > On Tue, Jun 18, 2013 at 04:05:08PM +0200, Paolo Bonzini wrote: > >> Il 05/06/2013 10:42, Gleb Natapov ha scritto: > >> >> > These patches add an emulated MSR_PLATFORM_INFO that kvm guests > >> >> > can read a

Re: [PATCH 1/7] KVM: arm-vgic: Support KVM_CREATE_DEVICE for VGIC

2013-06-18 Thread Christoffer Dall
On Tue, Jun 18, 2013 at 03:21:38PM +0200, Alexander Graf wrote: > > > Am 11.06.2013 um 06:51 schrieb Christoffer Dall : > > > Support creating the ARM VGIC device through the KVM_CREATE_DEVICE > > ioctl, which can then later be leveraged to use the > > KVM_{GET/SET}_DEVICE_ATTR, which is useful

Re: [PATCH 2/7] KVM: arm-vgic: Set base addr through device API

2013-06-18 Thread Christoffer Dall
On Tue, Jun 18, 2013 at 03:25:04PM +0200, Alexander Graf wrote: > > > Am 11.06.2013 um 06:51 schrieb Christoffer Dall : > > > Support setting the distributor and cpu interface base addresses in the > > VM physical address space through the KVM_{SET,GET}_DEVICE_ATTR API > > in addition the ARM sp

Re: [PATCH 1/2] kvm-unit-tests: Add a func to run instruction in emulator

2013-06-18 Thread Gleb Natapov
On Tue, Jun 18, 2013 at 10:28:59PM +0800, �??�?���? wrote: > On Tue, Jun 18, 2013 at 8:45 PM, Gleb Natapov wrote: > > On Thu, Jun 13, 2013 at 05:30:03PM +0800, 李春奇 wrote: > >> Hi Gleb, > >> I'm trying to solve these problems in the past days and meet many > >> difficulties. You want to save all

Re: [PATCH 2/7] KVM: arm-vgic: Set base addr through device API

2013-06-18 Thread Alexander Graf
On 18.06.2013, at 17:46, Christoffer Dall wrote: > On Tue, Jun 18, 2013 at 03:25:04PM +0200, Alexander Graf wrote: >> >> >> Am 11.06.2013 um 06:51 schrieb Christoffer Dall >> : >> >>> Support setting the distributor and cpu interface base addresses in the >>> VM physical address space through

Re: [PATCH 1/2] kvm-unit-tests: Add a func to run instruction in emulator

2013-06-18 Thread 李春奇
On Tue, Jun 18, 2013 at 11:47 PM, Gleb Natapov wrote: > On Tue, Jun 18, 2013 at 10:28:59PM +0800, Ê??Ê?•Â•? wrote: >> On Tue, Jun 18, 2013 at 8:45 PM, Gleb Natapov wrote: >> > On Thu, Jun 13, 2013 at 05:30:03PM +0800, ÊùéÊò•Â•á >> > wrote: >> >> Hi Gleb, >> >> I'm trying to solve these problem

Re: [PATCH 1/2] kvm-unit-tests: Add a func to run instruction in emulator

2013-06-18 Thread Gleb Natapov
On Tue, Jun 18, 2013 at 11:56:24PM +0800, 李春奇 wrote: > On Tue, Jun 18, 2013 at 11:47 PM, Gleb Natapov wrote: > > On Tue, Jun 18, 2013 at 10:28:59PM +0800, Ê??Ê?•Â•? > > wrote: > >> On Tue, Jun 18, 2013 at 8:45 PM, Gleb Natapov wrote: > >> > On Thu, Jun 13, 2013 at 05:30:03PM +0800, ÊùéÊò•Â•á

Re: [PATCH 1/2] kvm-unit-tests: Add a func to run instruction in emulator

2013-06-18 Thread 李春奇
extern u8 insn_page[], insn_page_end[]; extern u8 test_insn[], test_insn_end[]; extern u8 alt_insn_page[]; asm( ".align 4096\n\t" ".global insn_page\n\t" ".global insn_page_end\n\t" ".global test_insn\n\t" ".global test_insn_end\n\t" "insn_page:\n\t" "ret \n\t" "push %rax; push %rbx\n\t" "push %

Re: [PATCH qom-cpu v2 01/29] kvm: Change kvm_cpu_synchronize_state() argument to CPUState

2013-06-18 Thread Andreas Färber
Am 17.06.2013 18:15, schrieb Paolo Bonzini: > Il 16/06/2013 17:57, Andreas Färber ha scritto: >> It no longer relies on CPUArchState since 20d695a. >> >> Reviewed-by: liguang >> Signed-off-by: Andreas Färber >> --- >> hw/ppc/spapr_rtas.c | 2 +- >> include/sysemu/kvm.h | 4 ++-- >> kvm-all.c

Re: Bug#707257: linux-image-3.8-1-686-pae: KVM crashes with "entry failed, hardware error 0x80000021"

2013-06-18 Thread Stefan Pietsch
On 17.06.2013 18:07, Paolo Bonzini wrote: > Il 16/06/2013 02:25, Stefan Pietsch ha scritto: >> Bisecting leads to >> >> git bisect bad 378a8b099fc207ddcb91b19a8c1457667e0af398 >> git bisect good 007a3b547512d69f67ceb9641796d64552bd337e >> git bisect good 1f3141e80b149e7215313dff29e9a0c47811b1d1 >>

Re: [PATCH 1/2] kvm-unit-tests: Add a func to run instruction in emulator

2013-06-18 Thread Gleb Natapov
Send code in a form of a patch. On Wed, Jun 19, 2013 at 12:14:13AM +0800, 李春奇 wrote: > extern u8 insn_page[], insn_page_end[]; > extern u8 test_insn[], test_insn_end[]; > extern u8 alt_insn_page[]; > > asm( > ".align 4096\n\t" > ".global insn_page\n\t" > ".global insn_page_end\n\t" > ".global te

Re: [PATCH] pci: Enable overrides for missing ACS capabilities

2013-06-18 Thread Bjorn Helgaas
On Thu, May 30, 2013 at 12:40:19PM -0600, Alex Williamson wrote: > PCIe ACS (Access Control Services) is the PCIe 2.0+ feature that > allows us to control whether transactions are allowed to be redirected > in various subnodes of a PCIe topology. For instance, if two > endpoints are below a root p

Re: [PATCH v2 0/2] kvm: x86: Emulate MSR_PLATFORM_INFO

2013-06-18 Thread Nakajima, Jun
On Tue, Jun 18, 2013 at 8:16 AM, Gleb Natapov wrote: > On Tue, Jun 18, 2013 at 04:05:08PM +0200, Paolo Bonzini wrote: >> Il 05/06/2013 10:42, Gleb Natapov ha scritto: >> >> > These patches add an emulated MSR_PLATFORM_INFO that kvm guests >> >> > can read as described in section 14.3.2.4 of the In

Re: [PATCH] pci: Enable overrides for missing ACS capabilities

2013-06-18 Thread Alex Williamson
On Tue, 2013-06-18 at 11:28 -0600, Bjorn Helgaas wrote: > On Thu, May 30, 2013 at 12:40:19PM -0600, Alex Williamson wrote: > > PCIe ACS (Access Control Services) is the PCIe 2.0+ feature that > > allows us to control whether transactions are allowed to be redirected > > in various subnodes of a PCI

Unreliable QueryPerformanceCounter on Windows 2008 RC2 guest

2013-06-18 Thread pedro pinto
Hi there, I am using a Ubuntu 12.04 x64 host to run a Windows 2008 RC2 guest and I have noticed very puzzling behaviour inside the guest when invoking QueryPerformanceCounter. Specifically the program below will periodically show a 43 second offset between loop iterations despite the fact that lo

Re: [PATCH] pci: Enable overrides for missing ACS capabilities

2013-06-18 Thread Bjorn Helgaas
On Tue, Jun 18, 2013 at 12:20 PM, Alex Williamson wrote: > On Tue, 2013-06-18 at 11:28 -0600, Bjorn Helgaas wrote: >> On Thu, May 30, 2013 at 12:40:19PM -0600, Alex Williamson wrote: >> > PCIe ACS (Access Control Services) is the PCIe 2.0+ feature that >> > allows us to control whether transaction

Re: [PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-06-18 Thread Benjamin Herrenschmidt
On Tue, 2013-06-18 at 08:48 -0600, Alex Williamson wrote: > On Tue, 2013-06-18 at 14:38 +1000, Benjamin Herrenschmidt wrote: > > On Mon, 2013-06-17 at 20:32 -0600, Alex Williamson wrote: > > > > > Right, we don't want to create dependencies across modules. I don't > > > have a vision for how this

Re: [PATCHv1] kvm guest: fix uninitialized kvmclock read by KVM guest

2013-06-18 Thread Marcelo Tosatti
On Sat, Jun 15, 2013 at 10:01:45PM +0400, Eugene Batalov wrote: > Due to unintialized kvmclock read KVM guest is hanging on SMP boot stage. > If unintialized memory contains fatal garbage then hang reproduction is 100%. > Unintialized memory is allocated by memblock_alloc. So the garbage values > d

Re: [PATCH v3 0/6] KVM: MMU: fast invalidate all mmio sptes

2013-06-18 Thread Marcelo Tosatti
On Mon, Jun 17, 2013 at 07:59:15PM +0800, Xiao Guangrong wrote: > Sorry for the delay reply since i was on vacation. > > On 06/15/2013 10:22 AM, Takuya Yoshikawa wrote: > > On Thu, 13 Jun 2013 21:08:21 -0300 > > Marcelo Tosatti wrote: > > > >> On Fri, Jun 07, 2013 at 04:51:22PM +0800, Xiao Guang

Re: [PATCH] pci: Enable overrides for missing ACS capabilities

2013-06-18 Thread Alex Williamson
On Tue, 2013-06-18 at 15:31 -0600, Bjorn Helgaas wrote: > On Tue, Jun 18, 2013 at 12:20 PM, Alex Williamson > wrote: > > On Tue, 2013-06-18 at 11:28 -0600, Bjorn Helgaas wrote: > >> On Thu, May 30, 2013 at 12:40:19PM -0600, Alex Williamson wrote: > >> > PCIe ACS (Access Control Services) is the PC

Re: [PATCH] pci: Enable overrides for missing ACS capabilities

2013-06-18 Thread Don Dutile
On 06/18/2013 06:22 PM, Alex Williamson wrote: On Tue, 2013-06-18 at 15:31 -0600, Bjorn Helgaas wrote: On Tue, Jun 18, 2013 at 12:20 PM, Alex Williamson wrote: On Tue, 2013-06-18 at 11:28 -0600, Bjorn Helgaas wrote: On Thu, May 30, 2013 at 12:40:19PM -0600, Alex Williamson wrote: PCIe ACS (

[PATCH] kvmclock: clock should count only if vm is running (v2)

2013-06-18 Thread Marcelo Tosatti
v2: remove unnecessary runstate_is_running() usage (Paolo) -- kvmclock should not count while vm is paused, because: 1) if the vm is paused for long periods, timekeeping math can overflow while converting the (large) clocksource delta to nanoseconds. 2) Users rely on CLOCK_MONOTONIC to count r

Re: [PATCH] kvmclock: clock should count only if vm is running

2013-06-18 Thread Marcelo Tosatti
On Tue, Jun 18, 2013 at 11:02:27AM +0200, Paolo Bonzini wrote: > Hi Marcelo, sorry for the late review. > > Il 08/06/2013 04:00, Marcelo Tosatti ha scritto: > > kvmclock should not count while vm is paused, because: > > > > 1) if the vm is paused for long periods, timekeeping > > math can overfl

Re: [PATCH 1/2] kvm-unit-tests: Add a func to run instruction in emulator

2013-06-18 Thread 李春奇
On Wed, Jun 19, 2013 at 12:44 AM, Gleb Natapov wrote: > Send code in a form of a patch. > > On Wed, Jun 19, 2013 at 12:14:13AM +0800, 李春奇 wrote: >> extern u8 insn_page[], insn_page_end[]; >> extern u8 test_insn[], test_insn_end[]; >> extern u8 alt_insn_page[]; >> >> asm( >> ".align 4096\n\t" >> "

[PATCH] kvm api doc: fix section numbers

2013-06-18 Thread Alexey Kardashevskiy
Signed-off-by: Alexey Kardashevskiy --- Documentation/virtual/kvm/api.txt |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Documentation/virtual/kvm/api.txt b/Documentation/virtual/kvm/api.txt index 5f91eda..6365fef 100644 --- a/Documentation/virtual/kvm/api.txt +++

Re: [PATCH v3 0/6] KVM: MMU: fast invalidate all mmio sptes

2013-06-18 Thread Xiao Guangrong
On 06/18/2013 10:26 PM, Paolo Bonzini wrote: > Il 07/06/2013 10:51, Xiao Guangrong ha scritto: >> Changelog: >> V3: >> All of these changes are from Gleb's review: >> 1) rename RET_MMIO_PF_EMU to RET_MMIO_PF_EMULATE. >> 2) smartly adjust kvm generation number in kvm_current_mmio_generatio() >

Re: [PATCH] pci: Enable overrides for missing ACS capabilities

2013-06-18 Thread Bjorn Helgaas
On Tue, Jun 18, 2013 at 5:03 PM, Don Dutile wrote: > On 06/18/2013 06:22 PM, Alex Williamson wrote: >> >> On Tue, 2013-06-18 at 15:31 -0600, Bjorn Helgaas wrote: >>> >>> On Tue, Jun 18, 2013 at 12:20 PM, Alex Williamson >>> wrote: On Tue, 2013-06-18 at 11:28 -0600, Bjorn Helgaas wrote:

Re: [PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-06-18 Thread Alexey Kardashevskiy
On 06/16/2013 02:39 PM, Benjamin Herrenschmidt wrote: >> static pte_t kvmppc_lookup_pte(pgd_t *pgdir, unsigned long hva, bool >> writing, >> -unsigned long *pte_sizep) >> +unsigned long *pte_sizep, bool do_get_page) >> { >> pte_t *ptep; >> unsign

Re: [PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-06-18 Thread Rusty Russell
Alex Williamson writes: > On Mon, 2013-06-17 at 13:56 +1000, Benjamin Herrenschmidt wrote: >> On Sun, 2013-06-16 at 21:13 -0600, Alex Williamson wrote: >> >> > IOMMU groups themselves don't provide security, they're accessed by >> > interfaces like VFIO, which provide the security. Given a brief

Re: [PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-06-18 Thread Benjamin Herrenschmidt
On Wed, 2013-06-19 at 13:05 +0930, Rusty Russell wrote: > symbol_get() won't try to load a module; it'll just fail. This is what > you want, since they must have vfio in the kernel to get a valid fd... Ok, cool. I suppose what we want here Alexey is slightly higher level, something like:

[Bug 59891] New: Unrecoverable Kernel Panic, in Virtio Net in memcpy(), Complete system lockup.

2013-06-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=59891 Summary: Unrecoverable Kernel Panic, in Virtio Net in memcpy(), Complete system lockup. Product: Virtualization Version: unspecified Kernel Version: 3.9+ Platform: All OS/Ve

[Bug 59891] Unrecoverable Kernel Panic, in Virtio Net in memcpy(), Complete system lockup.

2013-06-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=59891 --- Comment #1 from Matthew Grant 2013-06-19 05:51:59 --- Created an attachment (id=105301) --> (https://bugzilla.kernel.org/attachment.cgi?id=105301) JPEG picture of screen dump Screen dump picture. Can't get serial console output, or ca