Paolo,
here is a fix for kvm/master targetting 3.15
The following changes since commit 5367742ad5321dd38058420adb4750ed9c7ead1e:
Merge tag 'signed-for-3.15' of git://github.com/agraf/linux-2.6 into
kvm-master (2014-05-13 18:15:16 +0200)
are available in the git repository at:
On 13/05/14 16:58, Michael Mueller wrote:
The proposed patch set implements S390 cpu model support in kvm. A cpu
model is defined by a triple comprizing the cpu type , the cpu facility
set and instruction blocking control. A consumer requests a feasible
combination of that that triple before
of the
S390 KVM code as well.
Signed-off-by: Thomas Huth th...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/kvm-s390.h | 1 +
arch/s390/kvm/priv.c | 5 -
2 files changed, 5 insertions
with the subset of instructions that may cause a SIE intercept.
Signed-off-by: Alexander Yarygin yary...@linux.vnet.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390
...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/kvm_host.h | 4 +++-
arch/s390/kvm/diag.c | 2 +-
arch/s390/kvm/gaccess.h | 8
arch/s390/kvm/kvm-s390.c | 10
...@linux.vnet.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/intercept.c | 25 +
1 file changed, 9 insertions(+), 16 deletions(-)
diff --git
...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/sclp.h | 1 +
drivers/s390/char/sclp_early.c | 12 +++-
2 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/arch/s390/include/asm/sclp.h b/arch/s390/include/asm/sclp.h
index 943d434..1aba89b
that it prevents any host faults from
taking place which does have implications on keys vs PGSTE.
Signed-off-by: Martin Schwidefsky schwidef...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/mm/pgtable.c | 22 +++---
1 file changed, 7 insertions
interpretion]
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/kvm_host.h | 4 +++-
arch/s390/kvm/intercept.c| 4
arch/s390/kvm/interrupt.c| 30 ++
arch/s390/kvm/kvm-s390.c | 2 +-
arch/s390/kvm/kvm-s390.h
.
Signed-off-by: Thomas Huth th...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/intercept.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/arch/s390/kvm
From: Alexander Yarygin yary...@linux.vnet.ibm.com
Use the symbolic translation tables from sie.h for decoding diag, sigp
and sie exit codes.
Signed-off-by: Alexander Yarygin yary...@linux.vnet.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: Cornelia Huck cornelia.h
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/trace.h | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/arch/s390/kvm/trace.h b/arch/s390
Paolo,
the following changes since commit d9f89b88f5102ce235b75a5907838e3c7ed84b97:
KVM: x86: Fix CR3 reserved bits check in long mode (2014-05-12 20:04:01 +0200)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/kvms390/linux.git
...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/uapi/asm/sie.h | 212 +++
1 file changed, 212 insertions(+)
create mode 100644 arch/s390/include/uapi/asm/sie.h
diff --git
undetected error conditions before).
Signed-off-by: Thomas Huth th...@linux.vnet.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/kvm-s390.c | 32
On 22/05/14 10:53, Paolo Bonzini wrote:
Il 22/05/2014 10:23, Michael Mueller ha scritto:
On Wed, 21 May 2014 15:22:35 +0200
Alexander Graf ag...@suse.de wrote:
I have seen the slides from Eduardo which he presented during this years
DevConf in Brno and made my comments according the s390x
On 21/05/14 23:43, Juan Quintela wrote:
Hi
Please, send any topic that you are interested in covering.
(mostly) Michael Mueller
-
CPU models on s390 and other platforms. Interfaces/requirements/Contraints etc.
Thanks, Juan.
Call details:
15:00 CEST
From: Thomas Huth th...@linux.vnet.ibm.com
The TPROT instruction can be used to check the accessability of storage
for any kind of logical addresses. So far, our handler only supported
real addresses. This patch now also enables support for addresses that
have to be translated via DAT first. And
Paolo,
The following changes since commit 1f854112553a1d65363ab27d4ee3dfb4b27075fb:
KVM: vmx: DR7 masking on task switch emulation is wrong (2014-05-22 17:47:18
+0200)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/kvms390/linux.git
-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
---
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 e519860..06d1888 100644
--- a/arch/s390/kvm/kvm
From: Thomas Huth th...@linux.vnet.ibm.com
This patch adds a function for translating logical guest addresses into
physical guest addresses without touching the memory at the given location.
Signed-off-by: Thomas Huth th...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
.
Signed-off-by: Nick Wang jfw...@us.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Tested-by: Matthew Rosato mjros...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm
anyway.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/interrupt.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm
From: Jens Freimann jf...@linux.vnet.ibm.com
We have all the logic to inject interrupts available in
kvm_s390_inject_vcpu(), so let's use it instead of
injecting irqs manually to the list in sigp code.
SIGP stop is special because we have to check the
action_flags before injecting the interrupt.
setting the cpu state to STOPPED while
holding the local irq lock.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
Paolo,
here is a preview of a rework of CPU state on s390x. Since we extend
MP_STATE I wanted to send an RFC upfront. I will wait for some feedback
and send a proper pull request in the next week:
-
This series enables the KVM_SET_MP_STATE ioctl on s390 to make the cpu
state settable by
-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/kvm-s390.c | 7 +--
arch/s390/kvm/kvm-s390.h | 4 ++--
2 files
This is the qemu part of kernel series Let user space control the
cpu states
Christian Borntraeger (1):
update linux headers with with cpustate changes
David Hildenbrand (4):
s390x/kvm: introduce proper states for s390 cpus
s390x/kvm: proper use of the cpu states OPERATING and STOPPED
for a user space driven VCPU state change
(to avoid race conditions).
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
of KVM_(S|G)ET_MP_STATE by s390.
Fix a spelling error (KVM_SET_MP_STATE vs. KVM_SET_MPSTATE) on the way.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian
...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
[folded Jason's patch into David's patch to avoid add/remove same lines]
---
hw/s390x/s390-virtio.c | 32
.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
target-s390x/cpu.c | 6 ++
target-s390x/kvm.c | 5 +
2 files
synchronization with the kvm part, we need to track the real
logical state of a cpu.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae
...@linux.vnet.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
target-s390x/cpu.c | 3 +++
target-s390x/cpu.h | 5 +
target-s390x/kvm.c | 33
the ioctl KVM_SET_MP_STATE, we can
disable
manual changing of the VCPU state and trust user space to do the right thing.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off
Will do a proper headers sync when ready
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
linux-headers/linux/kvm.h | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/linux-headers/linux/kvm.h b/linux-headers/linux/kvm.h
index f5d2c38..9b584da 100644
Paolo,
The following changes since commit 9f6226a762c7ae02f6a23a3d4fc552dafa57ea23:
arch: x86: kvm: x86.c: Cleaning up variable is set more than once (2014-06-30
16:52:04 +0200)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/kvms390/linux.git
for a user space driven VCPU state change
(to avoid race conditions).
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
of KVM_(S|G)ET_MP_STATE by s390.
Fix a spelling error (KVM_SET_MP_STATE vs. KVM_SET_MPSTATE) on the way.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian
-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/kvm-s390.c | 7 +--
arch/s390/kvm/kvm-s390.h | 4 ++--
2 files
setting the cpu state to STOPPED while
holding the local irq lock.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
the ioctl KVM_SET_MP_STATE, we can
disable
manual changing of the VCPU state and trust user space to do the right thing.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off
On 18/07/14 16:49, Paolo Bonzini wrote:
Il 15/07/2014 15:27, Christian Borntraeger ha scritto:
Paolo,
The following changes since commit 9f6226a762c7ae02f6a23a3d4fc552dafa57ea23:
arch: x86: kvm: x86.c: Cleaning up variable is set more than once
(2014-06-30 16:52:04 +0200)
are available
IPTE intercept can happen, let's decode that.
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
Acked-by: Cornelia Huck cornelia.h...@de.ibm.com
---
arch/s390/include/uapi/asm/sie.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/s390/include/uapi/asm/sie.h b/arch/s390/include
From: Cornelia Huck cornelia.h...@de.ibm.com
Let's document that this is a capability that may be enabled per-vm.
Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: David Hildenbrand d...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
Documentation/virtual/kvm/api.txt | 18 +++---
1 file changed, 15 insertions(+), 3 deletions(-)
diff --git a/Documentation/virtual/kvm/api.txt
b/Documentation/virtual/kvm/api.txt
index a41465b..7ab41e9
uses these functions, so no checks are lost.
The flag timer_due can be removed - kvm_cpu_has_pending_timer() tests whether
the timer is pending, thus the vcpu is correctly woken up.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
From: David Hildenbrand d...@linux.vnet.ibm.com
We can get rid of the tasklet used for waking up a VCPU in the hrtimer
code but wakeup the VCPU directly.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Acked-by: Cornelia Huck
From: David Hildenbrand d...@linux.vnet.ibm.com
local_int.lock is not used in a bottom-half handler anymore, therefore we can
turn it into an ordinary spin_lock at all occurrences.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
From: Cornelia Huck cornelia.h...@de.ibm.com
We should advertise all capabilities, including those that can
be enabled.
Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
From: David Hildenbrand d...@linux.vnet.ibm.com
The start_stop_lock is no longer acquired when in atomic context, therefore we
can convert it into an ordinary spin_lock.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Acked
Christian Borntraeger (1):
KVM: s390: add ipte to trace event decoding
Cornelia Huck (3):
KVM: document target of capability enablement
KVM: s390: document KVM_CAP_S390_IRQCHIP
KVM: s390: advertise KVM_CAP_S390_IRQCHIP
David Hildenbrand (5):
KVM: s390
in immediate reschedules after schedule()
in some scenarios.
The wakeup code doesn't require the local_int.lock to be held.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Acked-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off
On 22/07/14 10:32, Paolo Bonzini wrote:
Il 21/07/2014 21:14, Christian Borntraeger ha scritto:
From: Cornelia Huck cornelia.h...@de.ibm.com
We should advertise all capabilities, including those that can
be enabled.
Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian
On 28/07/14 16:22, Alexander Graf wrote:
On 28.07.2014, at 16:16, David Hildenbrand d...@linux.vnet.ibm.com wrote:
On 10.07.14 15:10, Christian Borntraeger wrote:
From: David Hildenbrand d...@linux.vnet.ibm.com
If a cpu is stopped, it must never be allowed to run and no interrupt may
-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Acked-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
[some additional patch description]
---
arch/s390/kvm/interrupt.c | 3 +++
1 file changed, 3
commit 7dfc63cf977447e09b1072911c22564f900fc578
(KVM: s390: allow only one SIGP STOP (AND STORE STATUS) at a time)
introduced a memory leak if a sigp stop is already pending. Free
the allocated inti structure.
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: David
) with Linux as a guest.
So no need for cc stable
Christian Borntraeger (1):
KVM: s390: Fix memory leak on busy SIGP stop
David Hildenbrand (1):
KVM: s390: rework broken SIGP STOP interrupt handling
arch/s390/kvm/interrupt.c | 3
-by: Christian Borntraeger borntrae...@de.ibm.com
---
tools/perf/builtin-kvm.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/perf/builtin-kvm.c b/tools/perf/builtin-kvm.c
index 43367eb..fe92dfd 100644
--- a/tools/perf/builtin-kvm.c
+++ b/tools/perf/builtin-kvm.c
On my s390 kvm system most of the kvm exits are in the range
of 0 or 1 microseconds. Can we increase the accuracy by 2
additional digits?
Opinions?
Christian Borntraeger (1):
KVM: perf/stat: Properly show submicrosecond times
tools/perf/builtin-kvm.c | 8
1 file changed, 4
of yielding, so
let's update the pid only in the KVM_RUN ioctl.
In addition, don't do a synchronize_rcu on startup (pid == 0).
This speeds up guest boot time on s390 noticably for some configs, e.g.
HZ=100, no full state tracking, 64 guest cpus 32 host cpus.
Signed-off-by: Christian Borntraeger
On 07/08/14 10:21, Raghavendra K T wrote:
On 08/05/2014 08:14 PM, Christian Borntraeger wrote:
We currently track the pid of the task that runs the VCPU in
vcpu_load. Since we call vcpu_load for all kind of ioctls on a
CPU, this causes hickups due to synchronize_rcu if one CPU is
modified
On 07/08/14 15:40, Paolo Bonzini wrote:
Il 07/08/2014 11:59, Christian Borntraeger ha scritto:
Paolo,
are you willing to apply to kvm/queue?
I asked a question, but anyway... not until the end of the merge window
and my small vacation. :)
Paolo
Absolutely, was on vacation myself
On 07/08/14 15:39, Paolo Bonzini wrote:
Il 05/08/2014 16:44, Christian Borntraeger ha scritto:
We currently track the pid of the task that runs the VCPU in
vcpu_load. Since we call vcpu_load for all kind of ioctls on a
CPU, this causes hickups due to synchronize_rcu if one CPU is
modified
On 19/08/14 11:27, Paolo Bonzini wrote:
Il 19/08/2014 10:38, Christian Borntraeger ha scritto:
The patch may be okay, but I'm worried that it might be hiding a bug in
QEMU.
On s390 we call KVM_S390_INITIAL_RESET from several reset functions, e.g.
during
CPU creation. This is the first
On 19/08/14 11:53, Paolo Bonzini wrote:
Il 19/08/2014 11:47, Christian Borntraeger ha scritto:
On 19/08/14 11:27, Paolo Bonzini wrote:
Il 19/08/2014 10:38, Christian Borntraeger ha scritto:
The patch may be okay, but I'm worried that it might be
hiding a bug in QEMU.
On s390 we call
On 19/08/14 12:03, Paolo Bonzini wrote:
Il 19/08/2014 11:59, Christian Borntraeger ha scritto:
Its only for the interrupts that are cpu local (like pending IPIs).
In addition, we would do that only for the reset case (with an
interface that can be used for migration). Right now
On 19/08/14 12:31, Paolo Bonzini wrote:
Il 19/08/2014 12:09, Christian Borntraeger ha scritto:
I'm not sure, this does seem like a workaround for another
limitation after all... Gleb?
Yes. We want to get rid of KVM_S390_INITIAL_RESET in QEMU. This comes
from a time, when we had another
On 18/08/14 07:02, Wanpeng Li wrote:
Hi Christian,
On Tue, Aug 05, 2014 at 04:44:14PM +0200, Christian Borntraeger wrote:
We currently track the pid of the task that runs the VCPU in
vcpu_load. Since we call vcpu_load for all kind of ioctls on a
CPU, this causes hickups due to synchronize_rcu
On 19/08/14 14:14, David Hildenbrand wrote:
Il 19/08/2014 13:28, David Hildenbrand ha scritto:
Looking at the code, kvm_cpu_synchronize_state() seems to do these ioctls in
the vcpu thread (e.g. comming from cpu_synchronize_all_states()), any
reasons
why kvm_cpu_synchronize_post_reset()
On 19/08/14 16:23, David Hildenbrand wrote:
On 19/08/14 14:14, David Hildenbrand wrote:
Il 19/08/2014 13:28, David Hildenbrand ha scritto:
Looking at the code, kvm_cpu_synchronize_state() seems to do these ioctls
in
the vcpu thread (e.g. comming from cpu_synchronize_all_states()), any
We dont have to wait for a grace period if there is no oldpid that
we are going to free. putpid also checks for NULL, so this patch
only fences synchronize_rcu.
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
virt/kvm/kvm_main.c | 3 ++-
1 file changed, 2 insertions(+), 1
On 20/08/14 01:22, Wanpeng Li wrote:
On Tue, Aug 19, 2014 at 04:04:03PM +0200, Christian Borntraeger wrote:
On 18/08/14 07:02, Wanpeng Li wrote:
Hi Christian,
On Tue, Aug 05, 2014 at 04:44:14PM +0200, Christian Borntraeger wrote:
We currently track the pid of the task that runs the VCPU
On 19/08/14 22:35, Radim Krčmář wrote:
--- a/virt/kvm/kvm_main.c
+++ b/virt/kvm/kvm_main.c
@@ -3123,6 +3123,8 @@ static void kvm_sched_in(struct preempt_notifier *pn,
int cpu)
if (vcpu-preempted)
vcpu-preempted = false;
+ kvm_arch_sched_in(vcpu, cpu);
+
On 10/08/14 10:30, Razya Ladelsky wrote:
From: Razya Ladelsky ra...@il.ibm.com
Date: Thu, 31 Jul 2014 09:47:20 +0300
Subject: [PATCH] vhost: Add polling mode
When vhost is waiting for buffers from the guest driver (e.g., more packets to
send in vhost-net's transmit queue), it normally goes
On 19/08/14 16:45, Christian Borntraeger wrote:
We dont have to wait for a grace period if there is no oldpid that
we are going to free. putpid also checks for NULL, so this patch
only fences synchronize_rcu.
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
virt/kvm
On 25/08/14 10:24, Christian Borntraeger wrote:
On 19/08/14 16:45, Christian Borntraeger wrote:
We dont have to wait for a grace period if there is no oldpid that
we are going to free. putpid also checks for NULL, so this patch
only fences synchronize_rcu.
Signed-off-by: Christian
The PFMF instruction handler blindly wrote the storage key even if
the page was mapped R/O in the host. Lets try a COW before continuing
and bail out in case of errors.
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: Dominik Dingel din...@linux.vnet.ibm.com
Cc: sta
. a malicious user to change the storage key of read-only pages
Christian Borntraeger (2):
KVM: s390: Fix user triggerable bug in dead code
KVM: s390/mm: try a cow on read only pages for key ops
arch/s390/kvm/kvm-s390.c | 13
structure. Unfortunately, this
allows userspace to trigger a kernel BUG. Let's just remove
this switch statement.
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: David Hildenbrand d...@linux.vnet.ibm.com
Cc: sta
On 25/08/14 15:42, Paolo Bonzini wrote:
Il 25/08/2014 15:10, Christian Borntraeger ha scritto:
Paolo,
the following changes since commit 7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9:
Linux 3.17-rc1 (2014-08-16 10:40:26 -0600)
are available in the git repository at:
git://git.kernel.org
-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/kvm-s390.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
index 81b0e11..f00d0b0 100644
--- a/arch/s390/kvm/kvm-s390.c
+++ b/arch/s390/kvm/kvm-s390.c
tables
with less than 5 levels.
Christian Borntraeger (1):
KVM: s390: no special machine check delivery
David Hildenbrand (4):
KVM: clarify the idea of kvm_dirty_regs
KVM: s390: clear kvm_dirty_regs when dropping
From: Jens Freimann jf...@linux.vnet.ibm.com
Let's make this a reusable function.
Signed-off-by: Jens Freimann jf...@linux.vnet.ibm.com
Acked-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae
From: Martin Schwidefsky schwidef...@de.ibm.com
Revert git commit 1b7fd6952063 (remove unecessary parameter from
pgste_ipte_notify)
Signed-off-by: Martin Schwidefsky schwidef...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/pgtable.h | 17
From: Martin Schwidefsky schwidef...@de.ibm.com
The radix tree rework removed all code that uses the gmap_rmap
and gmap_pgtable data structures. Remove these outdated definitions.
Signed-off-by: Martin Schwidefsky schwidef...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/interrupt.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c
index e2f6240..ba89bbb
helper routines.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Reviewed-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/uapi/asm/kvm.h | 10
out the segment invalidation in
the gmap shadow page table. As the radix tree can only store one
pointer, each host segment may only be mapped to exactly one
guest location.
Signed-off-by: Martin Schwidefsky schwidef...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch
The load PSW handler does not have to inject pending machine checks.
This can wait until the CPU runs the generic interrupt injection code.
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
---
arch/s390/kvm/interrupt.c | 56
-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
Documentation/virtual/kvm/api.txt | 4
1 file changed, 4 insertions(+)
diff --git a/Documentation/virtual/kvm/api.txt
b/Documentation/virtual/kvm/api.txt
index beae3fd..6485750 100644
-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/kvm-s390.c | 10 --
arch/s390/kvm/kvm-s390.h | 2 +-
2 files
-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/interrupt.c | 40 ++--
arch/s390/kvm/kvm-s390.c | 7 +--
arch/s390/kvm/kvm-s390.h | 2 +-
3 files changed, 24 insertions(+), 25 deletions(-)
diff --git a/arch/s390/kvm/interrupt.c b/arch
...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/pgtable.h | 3 +-
arch/s390/kvm/kvm-s390.c| 4 +-
arch/s390/mm/pgtable.c | 85 ++---
3 files changed, 59 insertions(+), 33 deletions(-)
diff --git
From: Jens Freimann jf...@linux.vnet.ibm.com
Get rid of open coded values for pfault init.
Signed-off-by: Jens Freimann jf...@linux.vnet.ibm.com
Acked-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/interrupt.c | 6
and vmaddr for
a user address.
Signed-off-by: Martin Schwidefsky schwidef...@de.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/pgtable.h | 14 ++---
arch/s390/kvm/diag.c| 8 +--
arch/s390/kvm
From: Martin Schwidefsky schwidef...@de.ibm.com
Revert git commit c3a23b9874c1 (remove unnecessary parameter from
gmap_do_ipte_notify).
Signed-off-by: Martin Schwidefsky schwidef...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/pgtable.h | 4
On 26/08/14 13:40, Paolo Bonzini wrote:
Il 26/08/2014 10:28, Christian Borntraeger ha scritto:
2. We use KVM_REQ_TLB_FLUSH instead of open coding tlb flushes
Why is this needed? It seems slower than what you are replacing.
Supporting KVM_REQ_TLB_FLUSH is useful (the first hunk
On 27/08/14 05:06, Ben Hutchings wrote:
On Mon, 2014-08-25 at 15:10 +0200, Christian Borntraeger wrote:
The PFMF instruction handler blindly wrote the storage key even if
the page was mapped R/O in the host. Lets try a COW before continuing
and bail out in case of errors.
Signed-off
commit ab3f285f227f (KVM: s390/mm: try a cow on read only pages for
key ops)' misaligned a code block. Let's fixup the indentation.
Reported-by: Ben Hutchings b...@decadent.org.uk
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/mm/pgtable.c | 12 ++--
1 file
501 - 600 of 1075 matches
Mail list logo