[PATCH/RFC 06/21] KVM: s390: base hrtimer on a monotonic clock

2015-01-15 Thread Christian Borntraeger
Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c index 14cdf1c..37ef06c 100644 --- a/arch/s390/kvm/kvm-s390.c +++ b/arch/s390/kvm/kvm-s390.c @@ -735,7 +735,7

[PATCH/RFC 04/21] KVM: s390: Allow userspace to limit guest memory size

2015-01-15 Thread Christian Borntraeger
ned-off-by: Dominik Dingel Acked-by: Cornelia Huck Signed-off-by: Christian Borntraeger --- Documentation/virtual/kvm/devices/vm.txt | 14 +++ arch/s390/include/uapi/asm/kvm.h | 1 + arch/s390/kvm/kvm-s390.c | 65 ++-- 3 files changed, 77

[PATCH/RFC 08/21] KVM: s390: new parameter for SIGP STOP irqs

2015-01-15 Thread Christian Borntraeger
introduced. These flags further define details of the requested STOP and can be easily migrated. Reviewed-by: Thomas Huth Acked-by: Cornelia Huck Signed-off-by: David Hildenbrand Signed-off-by: Christian Borntraeger --- Documentation/virtual/kvm/api.txt | 2 +- arch/s390/include/asm/kvm_host.h | 2

[PATCH/RFC 02/21] KVM: s390: make local function static

2015-01-15 Thread Christian Borntraeger
sparse rightfully complains about warning: symbol '__inject_extcall' was not declared. Should it be static? Signed-off-by: Christian Borntraeger --- arch/s390/kvm/interrupt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/s390/kvm/interrupt.c b/arc

[PATCH/RFC 09/21] KVM: s390: handle stop irqs without action_bits

2015-01-15 Thread Christian Borntraeger
ff-by: David Hildenbrand Acked-by: Cornelia Huck Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 5 --- arch/s390/kvm/intercept.c| 15 ++--- arch/s390/kvm/interrupt.c| 71 +--- arch/s390/kvm/kvm-s390.c

[PATCH/RFC 20/21] KVM: s390: Provide guest TOD Clock Get/Set Controls

2015-01-15 Thread Christian Borntraeger
high order expansion of the tod clock value after it overflows the 64-bits of the TOD. Signed-off-by: Jason J. Herne Reviewed-by: David Hildenbrand Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 1 + arch/s390/include/uapi/asm/kvm.h | 5 ++ arch/s390/kvm/

[PATCH/RFC] KVM: fix sparse warning in include/trace/events/kvm.h

2015-01-15 Thread Christian Borntraeger
are undefined behaviour as of C99 6.10.3p11. Lets use an indirection to fix this. Signed-off-by: Christian Borntraeger --- include/trace/events/kvm.h | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/include/trace/events/kvm.h b/include/trace/events/kvm.h

Re: [PATCH 4/8] x86/spinlock: Leftover conversion ACCESS_ONCE->READ_ONCE

2015-01-15 Thread Christian Borntraeger
Am 15.01.2015 um 20:38 schrieb Oleg Nesterov: > On 01/15, Christian Borntraeger wrote: >> >> --- a/arch/x86/include/asm/spinlock.h >> +++ b/arch/x86/include/asm/spinlock.h >> @@ -186,7 +186,7 @@ static inline void arch_spin_unlock_wait(arch_spinlock_t >&g

Re: [PATCH 4/8] x86/spinlock: Leftover conversion ACCESS_ONCE->READ_ONCE

2015-01-15 Thread Christian Borntraeger
Am 15.01.2015 um 21:01 schrieb Oleg Nesterov: > On 01/15, Christian Borntraeger wrote: >> >> Am 15.01.2015 um 20:38 schrieb Oleg Nesterov: >>> On 01/15, Christian Borntraeger wrote: >>>> >>>> --- a/arch/x86/include/asm/spinlock.h >>>>

Re: [PATCH 1/8] ppc/kvm: Replace ACCESS_ONCE with READ_ONCE

2015-01-16 Thread Christian Borntraeger
Am 16.01.2015 um 00:09 schrieb Michael Ellerman: > On Thu, 2015-01-15 at 09:58 +0100, Christian Borntraeger wrote: >> ACCESS_ONCE does not work reliably on non-scalar types. For >> example gcc 4.6 and 4.7 might remove the volatile tag for such >> accesses during the SRA (

Re: [PATCH/RFC 14/21] KVM: s390: clear the pfault queue if user space sets the invalid token

2015-01-19 Thread Christian Borntraeger
Am 17.01.2015 um 11:50 schrieb Heiko Carstens: [...] >> +/* for KVM_SYNC_PFAULT and KVM_REG_S390_PFTOKEN */ >> +#define KVM_S390_PFAULT_TOKEN_INVALID 0xUL > > uapi is for both 64 bit and (theoretically) also 32 bit. So this > should be 0xfff...ffULL instead of UL to prevent a

Re: [PATCH/RFC] KVM: fix sparse warning in include/trace/events/kvm.h

2015-01-19 Thread Christian Borntraeger
Adding Alex and Marcelo CC as git blame gives them authorship to that piece of code. Paolo, I assume that you will pick that up without a git tree if Alex/Marcelo ack that change. Christian Am 15.01.2015 um 15:21 schrieb Christian Borntraeger: > sparse complains about > include/trace/

Re: [kvm:master 1/1] arch/x86/kvm/../../../virt/kvm/kvm_main.c:2062:25: sparse: incorrect type in assignment (different address spaces)

2015-01-19 Thread Christian Borntraeger
i Kivity2006-12-10 2053 > switch (ioctl) { > 9a2bb7f4 drivers/kvm/kvm_main.c Avi Kivity2007-02-22 2054 > case KVM_RUN: > f0fe5108 drivers/kvm/kvm_main.c Avi Kivity 2007-03-07 2055 > r = -EINVAL; > f0fe5108 drivers/kvm/kvm_main.c Avi Kivity

[GIT PULL 14/24] KVM: s390: clear the pfault queue if user space sets the invalid token

2015-01-23 Thread Christian Borntraeger
Hildenbrand Acked-by: Cornelia Huck Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 1 - arch/s390/include/uapi/asm/kvm.h | 3 +++ arch/s390/kvm/kvm-s390.c | 4 3 files changed, 7 insertions(+), 1 deletion(-) diff --git a/arch/s390/include/asm/kvm_host.h

[GIT PULL 00/24] KVM: s390: fixes and features for kvm/next (3.20)

2015-01-23 Thread Christian Borntraeger
ted key instructions. ---- Christian Borntraeger (2): KVM: s390: make local function static KVM: s390: no need to hold the kvm->mutex for floating interrupts David Hildenbrand (12): KVM: s390: prevent sleep duration underflows in handle_wait() KVM:

[GIT PULL 01/24] KVM: remove unneeded return value of vcpu_postcreate

2015-01-23 Thread Christian Borntraeger
just get rid of the unchecked return value. Signed-off-by: Dominik Dingel Acked-by: Cornelia Huck Signed-off-by: Christian Borntraeger --- arch/arm/kvm/arm.c | 3 +-- arch/mips/kvm/mips.c | 3 +-- arch/powerpc/kvm/powerpc.c | 3 +-- arch/s390/kvm/kvm-s390.c | 3 +-- arch/x86

[GIT PULL 23/24] KVM: s390: fix bug in interrupt parameter check

2015-01-23 Thread Christian Borntraeger
From: Jens Freimann When we convert interrupt data from struct kvm_s390_interrupt to struct kvm_s390_irq we need to check the data in the input parameter not the output parameter. Acked-by: Cornelia Huck Acked-by: Christian Borntraeger Signed-off-by: Jens Freimann Signed-off-by: Christian

[GIT PULL 24/24] KVM: s390: remove redundant setting of interrupt type

2015-01-23 Thread Christian Borntraeger
From: Jens Freimann Setting inti->type again is unnecessary here, so let's remove this. Acked-by: Cornelia Huck Signed-off-by: Jens Freimann Signed-off-by: Christian Borntraeger --- arch/s390/kvm/interrupt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/s390/kvm/interrupt.

[GIT PULL 22/24] KVM: s390: avoid memory leaks if __inject_vm() fails

2015-01-23 Thread Christian Borntraeger
Cc: sta...@vger.kernel.org # v3.15+ Signed-off-by: Christian Borntraeger --- arch/s390/kvm/interrupt.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c index a6cb238..6d7513e 100644 --- a/arch/s390/kvm/interrupt.c +++ b

Re: [PATCH RFC 0/1] KVM: ioctl for reading/writing guest memory

2015-02-03 Thread Christian Borntraeger
Am 03.02.2015 um 13:59 schrieb Paolo Bonzini: > > > On 03/02/2015 13:11, Thomas Huth wrote: >> The userspace (QEMU) then can simply call this ioctl when it wants >> to read or write from/to virtual guest memory. Then kernel then takes >> the IPTE-lock, walks the MMU table of the guest to find out

Re: [PATCH RFC 1/1] KVM: s390: Add MEMOP ioctl for reading/writing guest memory

2015-02-04 Thread Christian Borntraeger
Am 03.02.2015 um 16:22 schrieb Paolo Bonzini: > > > On 03/02/2015 16:16, Thomas Huth wrote: >> Actually, I'd prefer to keep the "virtual" in the defines for the type >> of operation below: When it comes to s390 storage keys, we likely might >> need some calls for reading and writing to physical m

[PATCH 1/7] KVM: s390: floating irqs: fix user triggerable endless loop

2015-02-04 Thread Christian Borntraeger
ff-by: David Hildenbrand Cc: sta...@vger.kernel.org # v3.15+ Signed-off-by: Christian Borntraeger --- arch/s390/kvm/interrupt.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c index c34e1d9..073b5f3 100644 --- a/arch/s390/kvm/interrupt.c

[PATCH 0/7] KVM: s390: fixes and features for kvm/next (3.20)

2015-02-04 Thread Christian Borntraeger
of these patches some more days for external review. Is it ok to send a pull request early next week or do you want it before 3.19? There is one non-KVM s390 patch, but it has an ACK from Heiko Carstens Christian Borntraeger (1): KVM: s390: reenable LPP facility David Hildenbrand (1): KVM

[PATCH 4/7] KVM: s390/CPACF: Choose crypto control block format

2015-02-04 Thread Christian Borntraeger
Tony Krowiak Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 2 ++ arch/s390/kvm/kvm-s390.c | 49 ++-- 2 files changed, 49 insertions(+), 2 deletions(-) diff --git a/arch/s390/include/asm/kvm_host.h b/arch/s390/includ

[PATCH 7/7] KVM: s390: Create ioctl for Getting/Setting guest storage keys

2015-02-04 Thread Christian Borntraeger
ned-off-by: Christian Borntraeger --- Documentation/virtual/kvm/api.txt | 58 ++ arch/s390/kvm/kvm-s390.c | 121 ++ include/uapi/linux/kvm.h | 14 + 3 files changed, 193 insertions(+) diff --git a/Documentation/virtual/k

[PATCH 2/7] KVM: s390: reenable LPP facility

2015-02-04 Thread Christian Borntraeger
commit 7be81a46695d ("KVM: s390/facilities: allow TOD-CLOCK steering facility bit") accidentially disabled the "load program parameter" facility bit during rebase for upstream submission (my fault). Re-add that bit. Signed-off-by: Christian Borntraeger Reviewed-by: David H

[PATCH 6/7] KVM: s390: add cpu model support

2015-02-04 Thread Christian Borntraeger
the full host capability. Userspace has to initialize the cpu model before vcpu creation. A cpu model change of running vcpus is not possible. Signed-off-by: Michael Mueller Reviewed-by: Cornelia Huck Reviewed-by: David Hildenbrand Signed-off-by: Christian Borntraeger --- Documentation

[PATCH 5/7] KVM: s390: use facilities and cpu_id per KVM

2015-02-04 Thread Christian Borntraeger
iewed-by: David Hildenbrand Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 21 + arch/s390/kvm/gaccess.c | 4 +- arch/s390/kvm/kvm-s390.c | 92 +--- arch/s390/kvm/kvm-s390.h | 13 -- arch/s39

[PATCH 3/7] s390/kernel: Update /proc/sysinfo file with Extended Name and UUID

2015-02-04 Thread Christian Borntraeger
Acked-by: Heiko Carstens Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/sysinfo.h | 10 +++--- arch/s390/kernel/sysinfo.c | 29 + 2 files changed, 36 insertions(+), 3 deletions(-) diff --git a/arch/s390/include/asm/sysinfo.h b/arch/s390/include

Re: [PATCH RFC 1/1] KVM: s390: Add MEMOP ioctl for reading/writing guest memory

2015-02-04 Thread Christian Borntraeger
Am 04.02.2015 um 11:39 schrieb Paolo Bonzini: >> Conny: >> I am asking myself, if we should explicitly add a comment in the >> virtio-ccw spec, that all accesses are assumed to be with key 0 and >> thus never cause key protection. The change/reference bit is set >> by the underlying I/O or memory

[PATCH/RFC] KVM: Disable compat ioctl for s390

2015-02-04 Thread Christian Borntraeger
We never had a 31bit QEMU/kuli running. We would need to review several ioctls to check if this creates holes, bugs or whatever to make it work. Lets just disable compat support for KVM on s390. Signed-off-by: Christian Borntraeger --- virt/kvm/Kconfig| 4 virt/kvm/kvm_main.c | 12

Re: [PATCH RFC 1/1] KVM: s390: Add MEMOP ioctl for reading/writing guest memory

2015-02-04 Thread Christian Borntraeger
Am 04.02.2015 um 12:42 schrieb Paolo Bonzini: > > > On 04/02/2015 12:25, Christian Borntraeger wrote: >> Whenever vhost or qemu or a finished aio request wrote content into a >> virtio buffer, the HW has set the storage key for that physical page, >> which m

Re: [PATCH 3/7] s390/kernel: Update /proc/sysinfo file with Extended Name and UUID

2015-02-04 Thread Christian Borntraeger
Am 04.02.2015 um 13:57 schrieb Paolo Bonzini: > > > On 04/02/2015 10:44, Christian Borntraeger wrote: >> From: Ekaterina Tumanova >> >> Read the additional data fields (Extended Name and UUID) from the >> 1KB block returned by the STSI command and reflect

Re: [PATCH 4/7] KVM: s390/CPACF: Choose crypto control block format

2015-02-04 Thread Christian Borntraeger
Am 04.02.2015 um 14:00 schrieb Paolo Bonzini: > > > On 04/02/2015 10:44, Christian Borntraeger wrote: >> +static void kvm_s390_set_crycb_format(struct kvm *kvm) >> +{ >> +kvm->arch.crypto.crycbd = (__u32)(unsigned long) kvm->arch.crypto.crycb; >>

Re: [PATCH 4/7] KVM: s390/CPACF: Choose crypto control block format

2015-02-04 Thread Christian Borntraeger
Am 04.02.2015 um 14:48 schrieb Paolo Bonzini: > > > On 04/02/2015 14:05, Christian Borntraeger wrote: >>>> >>>> Could this also be part of the "things" that KVM can choose to enable, >>>> even though it's not a facility? >

Re: [PATCH 2/7] KVM: s390: reenable LPP facility

2015-02-04 Thread Christian Borntraeger
Am 04.02.2015 um 13:57 schrieb Paolo Bonzini: > > > On 04/02/2015 10:44, Christian Borntraeger wrote: >> commit 7be81a46695d ("KVM: s390/facilities: allow TOD-CLOCK steering >> facility bit") accidentially disabled the "load program parameter" &

Re: [PATCH 3/7] s390/kernel: Update /proc/sysinfo file with Extended Name and UUID

2015-02-04 Thread Christian Borntraeger
Am 04.02.2015 um 14:03 schrieb Paolo Bonzini: > > > On 04/02/2015 14:01, Christian Borntraeger wrote: >> Am 04.02.2015 um 13:57 schrieb Paolo Bonzini: >>> >>> >>> On 04/02/2015 10:44, Christian Borntraeger wrote: >>>> From: Ekaterina Tumanov

Re: [PATCH 4/4] kvm,rcu: use RCU extended quiescent state when running KVM guest

2015-02-05 Thread Christian Borntraeger
Am 05.02.2015 um 17:35 schrieb r...@redhat.com: > From: Rik van Riel > > The host kernel is not doing anything while the CPU is executing > a KVM guest VCPU, so it can be marked as being in an extended > quiescent state, identical to that used when running user space > code. > > The only excepti

Re: [PATCH v2 0/5] rcu,nohz,kvm: use RCU extended quiescent state when running KVM guest

2015-02-06 Thread Christian Borntraeger
h NOHZ_FULL enabled on most CPUs, and a VCPU thread bound to a > CPU. In a 10 second interval, rcuos/N threads on other CPUs got woken > up several times, while the rcuos thread on the CPU running the bound > and alwasy running VCPU thread never got woken up once. > > Thanks to Chr

[GIT PULL 3/7] s390/kernel: Update /proc/sysinfo file with Extended Name and UUID

2015-02-09 Thread Christian Borntraeger
/sysinfo file accordingly. Signed-off-by: Ekaterina Tumanova Reviewed-by: David Hildenbrand Reviewed-by: Cornelia Huck Acked-by: Heiko Carstens Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/sysinfo.h | 10 +++--- arch/s390/kernel/sysinfo.c | 29

[GIT PULL 7/7] KVM: Disable compat ioctl for s390

2015-02-09 Thread Christian Borntraeger
We never had a 31bit QEMU/kuli running. We would need to review several ioctls to check if this creates holes, bugs or whatever to make it work. Lets just disable compat support for KVM on s390. Signed-off-by: Christian Borntraeger Acked-by: Paolo Bonzini --- virt/kvm/Kconfig| 4

[GIT PULL 0/7] KVM: s390: fixes and features for kvm/next (3.20)

2015-02-09 Thread Christian Borntraeger
per VM - add s390 specific vm attributes for setting values 3. crypto - toleration patch for z13 support 4. add uuid and long name to /proc/sysinfo (stsi 322) - patch Acked by Heiko Carstens (touches non-kvm s390 code) Christian

[GIT PULL 6/7] KVM: s390: add cpu model support

2015-02-09 Thread Christian Borntraeger
the full host capability. Userspace has to initialize the cpu model before vcpu creation. A cpu model change of running vcpus is not possible. Signed-off-by: Michael Mueller Reviewed-by: Cornelia Huck Reviewed-by: David Hildenbrand Signed-off-by: Christian Borntraeger --- Documentation

[GIT PULL 1/7] KVM: s390: floating irqs: fix user triggerable endless loop

2015-02-09 Thread Christian Borntraeger
ff-by: David Hildenbrand Cc: sta...@vger.kernel.org # v3.15+ Signed-off-by: Christian Borntraeger --- arch/s390/kvm/interrupt.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c index c34e1d9..073b5f3 100644 --- a/arch/s390/kvm/interrupt.c

[GIT PULL 4/7] KVM: s390/CPACF: Choose crypto control block format

2015-02-09 Thread Christian Borntraeger
only change for z13 and does not affect the guest view. Signed-off-by: Tony Krowiak Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 2 ++ arch/s390/kvm/kvm-s390.c | 49 ++-- 2 files changed, 49 insertions(+), 2 deletions(-)

[GIT PULL 5/7] KVM: s390: use facilities and cpu_id per KVM

2015-02-09 Thread Christian Borntraeger
iewed-by: David Hildenbrand Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 21 + arch/s390/kvm/gaccess.c | 4 +- arch/s390/kvm/kvm-s390.c | 92 +--- arch/s390/kvm/kvm-s390.h | 13 -- arch/s39

[GIT PULL 2/7] KVM: s390: reenable LPP facility

2015-02-09 Thread Christian Borntraeger
ruction (used by KVM itself) cc stable is not necessary see http://www-01.ibm.com/support/docview.wss?uid=isg26fcd1cc32246f4c8852574ce0044734a (SA23-2260 The Load-Program-Parameter and CPU-Measurement Facilities) for details about LPP and its usecase. Signed-off-by: Christian Borntraeger Reviewed-b

Re: [PATCH] kvm: add halt_poll_ns module parameter

2015-02-09 Thread Christian Borntraeger
Am 06.02.2015 um 13:48 schrieb Paolo Bonzini: > This patch introduces a new module parameter for the KVM module; when it > is present, KVM attempts a bit of polling on every HLT before scheduling > itself out via kvm_vcpu_block. > > This parameter helps a lot for latency-bound workloads---in parti

Re: [GIT PULL] locking fixes

2015-02-23 Thread Christian Borntraeger
Am 21.02.2015 um 02:51 schrieb Linus Torvalds: > So here's my try at fixing READ_ONCE() so that it is happy with 'const' > sources. > > It is entirely untested. Comments/testing? > > Christian, I guess I could just have forced a cast instead of the > union. I'd like you to take a look at this, b

Re: [Qemu-devel] [RFC PATCH v2 04/15] cpu-model/s390: Introduce S390 CPU models

2015-02-23 Thread Christian Borntraeger
Am 20.02.2015 um 16:22 schrieb Alexander Graf: > > > >> Am 20.02.2015 um 16:00 schrieb Michael Mueller : >> >> On Fri, 20 Feb 2015 14:54:23 +0100 >> Alexander Graf wrote: >> +/* machine related properties */ +typedef struct S390CPUMachineProps { +uint16_t class; /*

Re: [Qemu-devel] [RFC PATCH v2 04/15] cpu-model/s390: Introduce S390 CPU models

2015-02-23 Thread Christian Borntraeger
Am 23.02.2015 um 13:56 schrieb Christian Borntraeger: > Am 20.02.2015 um 16:22 schrieb Alexander Graf: >> >> >> >>> Am 20.02.2015 um 16:00 schrieb Michael Mueller : >>> >>> On Fri, 20 Feb 2015 14:54:23 +0100 >>> Alexander Gr

Re: [PATCH for stable] x86/spinlocks/paravirt: Fix memory corruption on unlock

2015-02-25 Thread Christian Borntraeger
Am 25.02.2015 um 11:08 schrieb Ingo Molnar: > > * Greg KH wrote: > It's: d6abfdb20223 x86/spinlocks/paravirt: Fix memory corruption on unlock >>> >>> Yes, This is the original patch. Please note I have taken out the >>> READ_ONCE changes from the original patch to avoid build war

Re: [PATCH] KVM/s390: Fix trivial typo in comments

2015-02-27 Thread Christian Borntraeger
Am 26.02.2015 um 23:16 schrieb Yannick Guerrini: > Change 'architecuture' to 'architecture' > > Signed-off-by: Yannick Guerrini Thanks, applied. > --- > arch/s390/kvm/gaccess.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/s390/kvm/gaccess.c b/arch/s390/kvm/ga

[PATCH] KVM: make halt_poll_ns static

2015-02-27 Thread Christian Borntraeger
halt_poll_ns is used only locally. Make it static. Signed-off-by: Christian Borntraeger --- virt/kvm/kvm_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index 34310a8..58bc2a9 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm

Re: [PATCH] KVM: make halt_poll_ns static

2015-03-03 Thread Christian Borntraeger
Am 03.03.2015 um 22:42 schrieb Bandan Das: > Christian Borntraeger writes: > >> halt_poll_ns is used only locally. Make it static. >> >> Signed-off-by: Christian Borntraeger >> --- >> virt/kvm/kvm_main.c | 2 +- >> 1 file changed, 1 insertion(+),

[GIT PULL 2/5] KVM: s390/cpacf: Fix kernel bug under z/VM

2015-03-04 Thread Christian Borntraeger
660e>] SyS_ioctl+0x9e/0xb0 [ 386.098814] [<0070476a>] system_call+0xd6/0x258 [ 386.098815] [<03fffc7400a2>] 0x3fffc7400a2 Lets add an extable entry and provide a zeroed config in that case. Reported-by: Stefan Zimmermann Signed-off-by: Christian Borntraeger Reviewe

[GIT PULL 5/5] KVM: s390: non-LPAR case obsolete during facilities mask init

2015-03-04 Thread Christian Borntraeger
From: Michael Mueller With patch "include guest facilities in kvm facility test" it is no longer necessary to have special handling for the non-LPAR case. Signed-off-by: Michael Mueller Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.c | 17 + 1 fi

[GIT PULL 0/5] KVM: s390: Fixups for changes in merge window for 4.0

2015-03-04 Thread Christian Borntraeger
4e892c34 ("KVM: s390/cpacf: Enable/disable protected key functions for kvm guest") commit 45c9b47c588 ("KVM: s390/CPACF: Choose crypto control block format") which all have been merged during the merge window for 4.0. ---------

[GIT PULL 1/5] KVM: s390/cpacf: Enable key wrapping by default

2015-03-04 Thread Christian Borntraeger
From: Tony Krowiak z/VM and LPAR enable key wrapping by default, lets do the same on KVM. Signed-off-by: Tony Krowiak Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/arch/s390/kvm/kvm-s390.c b

[GIT PULL 4/5] KVM: s390: include guest facilities in kvm facility test

2015-03-04 Thread Christian Borntraeger
. The userspace interface is not affected by this change. Signed-off-by: Michael Mueller Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 12 ++-- arch/s390/kvm/kvm-s390.c | 30 -- arch/s390/kvm/kvm-s390.h | 3 ++- arc

[GIT PULL 3/5] KVM: s390: fix in memory copy of facility lists

2015-03-04 Thread Christian Borntraeger
From: Michael Mueller The facility lists were not fully copied. Signed-off-by: Michael Mueller Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c index

Re: [GIT PULL 0/5] KVM: s390: Fixups for changes in merge window for 4.0

2015-03-04 Thread Christian Borntraeger
Am 04.03.2015 um 12:41 schrieb Christian Borntraeger: > Paolo, Marcelo, > > here is a pull request for kvm/master targetting 4.0. It is based on > 4.0-rc1 as I want all patches in an upcoming pull request against > kvm/next as well to avoid nasty merge conflicts with other changes

Re: [GIT PULL 0/5] KVM: s390: Fixups for changes in merge window for 4.0

2015-03-05 Thread Christian Borntraeger
Am 05.03.2015 um 02:07 schrieb Marcelo Tosatti: > On Wed, Mar 04, 2015 at 12:41:49PM +0100, Christian Borntraeger wrote: >> Paolo, Marcelo, >> >> here is a pull request for kvm/master targetting 4.0. It is based on >> 4.0-rc1 as I want all patches in an upcoming pull r

Re: [GIT PULL 0/5] KVM: s390: Fixups for changes in merge window for 4.0

2015-03-05 Thread Christian Borntraeger
Am 05.03.2015 um 09:41 schrieb Christian Borntraeger: >> Can you generate pull against current master? >> >> $ git pull --ff-only >> git://git.kernel.org/pub/scm/linux/kernel/git/kvms390/linux.git > > Can you use git://git.kernel.org/pub/scm/linux/kernel/git/kvms3

[GIT PULL 04/20] KVM: s390: include guest facilities in kvm facility test

2015-03-06 Thread Christian Borntraeger
. The userspace interface is not affected by this change. Signed-off-by: Michael Mueller Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 12 ++-- arch/s390/kvm/kvm-s390.c | 30 -- arch/s390/kvm/kvm-s390.h | 3 ++- arc

[GIT PULL 01/20] KVM: s390/cpacf: Enable key wrapping by default

2015-03-06 Thread Christian Borntraeger
From: Tony Krowiak z/VM and LPAR enable key wrapping by default, lets do the same on KVM. Signed-off-by: Tony Krowiak Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/arch/s390/kvm/kvm-s390.c b

[GIT PULL 12/20] KVM: s390: perform vcpu model setup in a function

2015-03-06 Thread Christian Borntraeger
kvm facility tests. There is no need to protect the cpu model setup with a lock since the attributes can't be changed anymore as soon the first vcpu is online. Signed-off-by: Michael Mueller Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.c | 17 +++-- 1

[GIT PULL 07/20] KVM: s390: fix handling of write errors in the tpi handler

2015-03-06 Thread Christian Borntraeger
ected. Therefore the dequeued interrupt is lost. This patch fixes the problem while cleaning up the function and making the cc and rc logic easier to handle. Signed-off-by: David Hildenbrand Cc: sta...@vger.kernel.org # 3.16+ Signed-off-by: Christian Borntraeger --- arch/

[GIT PULL 02/20] KVM: s390/cpacf: Fix kernel bug under z/VM

2015-03-06 Thread Christian Borntraeger
660e>] SyS_ioctl+0x9e/0xb0 [ 386.098814] [<0070476a>] system_call+0xd6/0x258 [ 386.098815] [<03fffc7400a2>] 0x3fffc7400a2 Lets add an extable entry and provide a zeroed config in that case. Reported-by: Stefan Zimmermann Signed-off-by: Christian Borntraeger Reviewe

[GIT PULL 16/20] KVM: s390: Allocate and save/restore vector registers

2015-03-06 Thread Christian Borntraeger
ric Farman Reviewed-by: Thomas Huth Acked-by: Christian Borntraeger Signed-off-by: Christian Borntraeger --- Documentation/virtual/kvm/api.txt | 10 ++ arch/s390/include/asm/kvm_host.h | 10 +- arch/s390/include/uapi/asm/kvm.h | 4 arch/s390/kvm/kvm-s390.c

[GIT PULL 11/20] KVM: s390: Use insn_length() to calculate length of instruction

2015-03-06 Thread Christian Borntraeger
Jens Freimann Reviewed-by: David Hildenbrand Acked-by: Cornelia Huck Signed-off-by: Christian Borntraeger --- arch/s390/kvm/interrupt.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c index 98a3131..9561e1d 100644

[GIT PULL 14/20] KVM: s390: fix instruction interception trace point

2015-03-06 Thread Christian Borntraeger
a warning: "kvm_s390_intercept_instruction: [FAILED TO PARSE]..." So let's add parentheses around the instruction parser macro to fix the format parsing. Acked-by: Christian Borntraeger Acked-by: Cornelia Huck Signed-off-by: David Hildenbrand Signed-off-by: Christian Borntraeger --- arch/s390/include/uapi/a

[GIT PULL 00/20] KVM: s390: Features and Fixes for 4.1 (kvm/next)

2015-03-06 Thread Christian Borntraeger
. Alexander Yarygin (1): KVM: s390: Use the read_guest_abs() in guest debug functions Christian Borntraeger (1): KVM: s390/cpacf: Fix kernel bug under z/VM David Hildenbrand (3): KVM: s390: fix handling of write errors in the tpi handler KVM: s390: reinjection of

[GIT PULL 20/20] KVM: s390: Enable vector support for capable guest

2015-03-06 Thread Christian Borntraeger
would otherwise occur when a vector instruction is issued by the guest operating system. Signed-off-by: Eric Farman Reviewed-by: Thomas Huth Signed-off-by: Christian Borntraeger --- arch/s390/include/asm/kvm_host.h | 4 +++- arch/s390/kvm/kvm-s390.c | 5 + 2 files changed, 8 inser

[GIT PULL 13/20] KVM: s390: Fix trivial typo in comments

2015-03-06 Thread Christian Borntraeger
From: Yannick Guerrini Change 'architecuture' to 'architecture' Signed-off-by: Yannick Guerrini Message-Id: <1424989004-14412-1-git-send-email-yguerr...@tomshardware.fr> Signed-off-by: Christian Borntraeger --- arch/s390/kvm/gaccess.c | 2 +- 1 file changed,

[GIT PULL 18/20] KVM: s390: Add new SIGP order to kernel counters

2015-03-06 Thread Christian Borntraeger
From: Eric Farman The new SIGP order Store Additional Status at Address is totally handled by user space, but we should still record the occurrence of this order in the kernel code. Signed-off-by: Eric Farman Reviewed-by: Thomas Huth Reviewed-by: David Hildenbrand Signed-off-by: Christian

[GIT PULL 09/20] KVM: s390: Nullify instruction for certain program exceptions

2015-03-06 Thread Christian Borntraeger
we have to rewind the PSW before writing it into the low-core. The list of nullifying exceptions can be found in the POP, chapter 6, figure 6-1 ("Interruption Action"). Signed-off-by: Thomas Huth Reviewed-by: Jens Freimann Reviewed-by: David Hildenbrand Acked-by: Christian Borntraeger

[GIT PULL 10/20] KVM: s390: Forward PSW to next instruction for addressing exceptions

2015-03-06 Thread Christian Borntraeger
this issue, we can now also enable the TPROT interpretation facility again which had been disabled because of this problem. Signed-off-by: Thomas Huth Reviewed-by: Christian Borntraeger Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.c | 35 --- 1

[GIT PULL 19/20] KVM: s390: Machine Check

2015-03-06 Thread Christian Borntraeger
: Christian Borntraeger --- arch/s390/kernel/asm-offsets.c | 1 + arch/s390/kvm/interrupt.c | 4 arch/s390/kvm/kvm-s390.c | 29 + arch/s390/kvm/kvm-s390.h | 3 +++ 4 files changed, 37 insertions(+) diff --git a/arch/s390/kernel/asm-offsets.c b/arch

[GIT PULL 03/20] KVM: s390: fix in memory copy of facility lists

2015-03-06 Thread Christian Borntraeger
From: Michael Mueller The facility lists were not fully copied. Signed-off-by: Michael Mueller Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c index

[GIT PULL 08/20] KVM: s390: reinjection of irqs can fail in the tpi handler

2015-03-06 Thread Christian Borntraeger
special case. This error is not recoverable, so let's fail hard. This can later be avoided by not dequeuing the interrupt but working directly on the locked list. Signed-off-by: David Hildenbrand Cc: sta...@vger.kernel.org # 3.16+ Signed-off-by: Christian Borntraeger --- arch/s390/kvm/interr

[GIT PULL 15/20] KVM: s390: Use the read_guest_abs() in guest debug functions

2015-03-06 Thread Christian Borntraeger
: Thomas Huth Signed-off-by: Christian Borntraeger --- arch/s390/kvm/guestdbg.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/s390/kvm/guestdbg.c b/arch/s390/kvm/guestdbg.c index 3e8d409..e97b345 100644 --- a/arch/s390/kvm/guestdbg.c +++ b/arch/s390/kvm/guestdbg.c

[GIT PULL 05/20] KVM: s390: non-LPAR case obsolete during facilities mask init

2015-03-06 Thread Christian Borntraeger
From: Michael Mueller With patch "include guest facilities in kvm facility test" it is no longer necessary to have special handling for the non-LPAR case. Signed-off-by: Michael Mueller Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.c | 17 + 1 fi

[GIT PULL 06/20] KVM: s390: Zero out current VMDB of STSI before including level3 data.

2015-03-06 Thread Christian Borntraeger
From: Ekaterina Tumanova s390 documentation requires words 0 and 10-15 to be reserved and stored as zeros. As we fill out all other fields, we can memset the full structure. Signed-off-by: Ekaterina Tumanova Cc: sta...@vger.kernel.org Reviewed-by: David Hildenbrand Signed-off-by: Christian

[GIT PULL 17/20] KVM: s390: Vector exceptions

2015-03-06 Thread Christian Borntraeger
From: Eric Farman A new exception type for vector instructions is introduced with the new processor, but is handled exactly like a Data Exception which is already handled by the system. Signed-off-by: Eric Farman Reviewed-by: David Hildenbrand Acked-by: Christian Borntraeger Signed-off-by

Re: [PATCH] KVM: make halt_poll_ns static

2015-03-09 Thread Christian Borntraeger
Am 03.03.2015 um 22:46 schrieb Christian Borntraeger: [...] > > halt_poll_ns is used only locally. Make it static and remove > the initializer. > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > index 34310a8..58bc2a9 100644 > --- a/virt/kvm/kvm_main.c > +

Re: [PATCH trivial] KVM: s390: Spelling s/intance/instance/

2015-03-10 Thread Christian Borntraeger
Am 09.03.2015 um 21:27 schrieb Geert Uytterhoeven: > Signed-off-by: Geert Uytterhoeven > --- > arch/s390/kvm/kvm-s390.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/s390/kvm/kvm-s390.h b/arch/s390/kvm/kvm-s390.h > index fda3f3146eb636d3..83f32a147d72c0f3 100644 >

[PATCH] KVM: MAINTAINERS: add file arch/x86/kernel/kvm.c|kvmclock.c

2015-03-12 Thread Christian Borntraeger
The KVM list should be CCed on changes for arch/x86/kernel/kvm.c and arch/x86/kernel/kvmclock.c Signed-off-by: Christian Borntraeger --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index ddc5a8c..80e3488 100644 --- a/MAINTAINERS +++ b/MAINTAINERS

Re: [PATCH 3/3] KVM: For the symbols used locally only should be static type.

2015-03-13 Thread Christian Borntraeger
Am 13.03.2015 um 10:39 schrieb Xiubo Li: > This patch fix the following sparse warnings: > > for file virt/kvm/kvm_main.c: > warning: symbol 'halt_poll_ns' was not declared. Should it be static? > > Signed-off-by: Xiubo Li > --- > virt/kvm/kvm_main.c | 2 +- > 1 file changed, 1 insertion(+), 1

[PATCH/RFC 6/9] KVM: s390: Guest's memory access functions get access registers

2015-03-16 Thread Christian Borntraeger
: Christian Borntraeger --- arch/s390/kvm/diag.c | 4 +-- arch/s390/kvm/gaccess.c | 8 ++--- arch/s390/kvm/gaccess.h | 16 +- arch/s390/kvm/intercept.c | 4 +-- arch/s390/kvm/kvm-s390.c | 10 +++--- arch/s390/kvm/kvm-s390.h | 26 +--- arch/s390/kvm/priv.c | 79

[PATCH/RFC 4/9] KVM: s390: Add MEMOP ioctls for reading/writing guest memory

2015-03-16 Thread Christian Borntraeger
guest can trigger huge copy_from/to_user transfers. QEMU currently only requests up to one or two pages so far, so 16*4kB seems to be a reasonable limit here. Signed-off-by: Thomas Huth Signed-off-by: Christian Borntraeger --- Documentation/virtual/kvm/api.txt | 46 arc

[PATCH/RFC 2/9] KVM: s390: cleanup jump lables in kvm_arch_init_vm

2015-03-16 Thread Christian Borntraeger
From: Dominik Dingel As all cleanup functions can handle their respective NULL case there is no need to have more than one error jump label. Signed-off-by: Dominik Dingel Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.c | 14 +- 1 file changed, 5 insertions(+), 9

[PATCH/RFC 7/9] KVM: s390: Optimize paths where get_vcpu_asce() is invoked

2015-03-16 Thread Christian Borntraeger
Alexander Yarygin Reviewed-by: Thomas Huth Reviewed-by: David Hildenbrand Signed-off-by: Christian Borntraeger --- arch/s390/kvm/gaccess.c | 25 - 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/arch/s390/kvm/gaccess.c b/arch/s390/kvm/gaccess.c index 33b5ae

[PATCH/RFC 8/9] KVM: s390: Add access register mode

2015-03-16 Thread Christian Borntraeger
. See the "Access-Register Introduction" section of the chapter 5 "Program Execution" in "Principles of Operations" for more details. Signed-off-by: Alexander Yarygin Reviewed-by: Thomas Huth Signed-off-by: Christian Borntraeger ---

[PATCH/RFC 5/9] KVM: s390: introduce post handlers for STSI

2015-03-16 Thread Christian Borntraeger
ric Farman Reviewed-by: Christian Borntraeger Signed-off-by: Ekaterina Tumanova Signed-off-by: Christian Borntraeger --- Documentation/virtual/kvm/api.txt | 28 arch/s390/include/asm/kvm_host.h | 1 + arch/s390/kvm/kvm-s390.c | 5 + arch/s390/

[PATCH/RFC 3/9] KVM: s390: Fix low-address protection for real addresses

2015-03-16 Thread Christian Borntraeger
-by: David Hildenbrand Signed-off-by: Christian Borntraeger --- arch/s390/kvm/gaccess.c | 11 ++- arch/s390/kvm/gaccess.h | 2 +- arch/s390/kvm/priv.c| 4 ++-- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/arch/s390/kvm/gaccess.c b/arch/s390/kvm/gaccess.c index 633fe9

[PATCH/RFC 1/9] KVM: s390: Spelling s/intance/instance/

2015-03-16 Thread Christian Borntraeger
From: Geert Uytterhoeven Signed-off-by: Geert Uytterhoeven Message-Id: <1425932832-6244-1-git-send-email-geert+rene...@glider.be> Signed-off-by: Christian Borntraeger --- arch/s390/kvm/kvm-s390.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/s390/kvm/kvm-s3

[PATCH/RFC 0/9] Next bunch of KVM/s390x changes for next

2015-03-16 Thread Christian Borntraeger
Paolo, Marcelo, Alex, this is my current patch queue that I want to send for the next pull request. As it contains 3 API extensions - KVM: s390: introduce post handlers for STSI - KVM: s390: Create ioctl for Getting/Setting guest storage keys - KVM: s390: Add MEMOP ioctls for reading/writing guest

[PATCH/RFC 9/9] KVM: s390: Create ioctl for Getting/Setting guest storage keys

2015-03-16 Thread Christian Borntraeger
ned-off-by: Christian Borntraeger --- Documentation/virtual/kvm/api.txt | 58 ++ arch/s390/kvm/kvm-s390.c | 122 ++ include/uapi/linux/kvm.h | 14 + 3 files changed, 194 insertions(+) diff --git a/Documentation/virtual/k

Re: [PATCH/RFC 5/9] KVM: s390: introduce post handlers for STSI

2015-03-16 Thread Christian Borntraeger
Am 16.03.2015 um 12:55 schrieb Paolo Bonzini: > > > On 16/03/2015 09:51, Christian Borntraeger wrote: >> From: Ekaterina Tumanova >> >> The Store System Information (STSI) instruction currently collects all >> information it relays to the caller in the kernel.

<    2   3   4   5   6   7   8   9   10   11   >