...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
Documentation/virtual/kvm/api.txt| 8 ++---
Documentation/virtual/kvm/devices/vm.txt | 6
arch/s390/kvm/kvm-s390.c | 54
include/uapi/linux/kvm.h
Marcelo, Gleb, Alex,
this is a followup to the recent discussion (re config device)
To avoid a bunch of new ioctl, lets reuse the device attributes on
the VM device. All of these attributes are architecture specific.
Patch1 introduces the interface,
Patch2 makes CMMA configurable
Patch3 uses the
On 10/04/14 13:29, Alexander Graf wrote:
On 10.04.14 13:16, Christian Borntraeger wrote:
From: Dominik Dingel din...@linux.vnet.ibm.com
We sometimes need to get/set attributes specific to a virtual machine
and so need something else than ONE_REG.
Let's copy the KVM_DEVICE approach
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert handle_stidp() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/priv.c | 7
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert kvm_s390_store_status_unloaded() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch
From: Heiko Carstens heiko.carst...@de.ibm.com
Check if siif is available before setting.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/sclp.h | 7
From: Dominik Dingel din...@linux.vnet.ibm.com
To enable CMMA and to reset its state we use the vm kvm_device ioctls,
encapsulating attributes within the KVM_S390_VM_MEM_CTRL group.
Signed-off-by: Dominik Dingel din...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae
Hildenbrand 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/guestdbg.c | 2 ++
arch/s390/kvm/interrupt.c | 3 +++
2 files changed, 5 insertions(+)
diff --git a/arch/s390/kvm/guestdbg.c b/arch/s390
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert handle_prog() to new guest access functions.
Also make the code a bit more readable and look at the return code
of write_guest_lc() which was missing before.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Signed-off-by: Christian
-off-by: David Hildenbrand 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 | 38 --
1 file changed, 20 insertions(+), 18 deletions(-)
diff --git
On hard exits (abort, sigkill) we have have some kvm_s390_interrupt_info
structures hanging around. Delete those on exit to avoid memory leaks.
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
CC: sta...@vger.kernel.org
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
---
arch/s390
Freimann jf...@linux.vnet.ibm.com
Acked-by: Heiko Carstens heiko.carst...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/lowcore.h | 6 --
arch/s390/kernel/asm-offsets.c | 5 +++--
arch/s390/kernel/entry.S| 4 ++--
arch/s390/kernel
From: Michael Mueller m...@linux.vnet.ibm.com
The generically assembled low core labels already contain the
address for the TDB.
Signed-off-by: Michael Mueller m...@linux.vnet.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
From: David Hildenbrand d...@linux.vnet.ibm.com
Introduce the methods to emulate the stctl and stctg instruction. Added tracing
code.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/kvm_host.h | 2
From: David Hildenbrand d...@linux.vnet.ibm.com
Added documentation for diag 501, stating that no subfunctions are provided and
no parameters are used.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian
From: Michael Mueller m...@linux.vnet.ibm.com
Factor out the new function handle_itdb(), which copies the ITDB into
guest lowcore to fully handle a TX abort.
Signed-off-by: Michael Mueller m...@linux.vnet.ibm.com
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian
debugging it.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/kvm_host.h | 53 -
arch/s390/include/uapi/asm/kvm.h | 1 +
arch/s390/kvm/Makefile | 2 +-
arch/s390/kvm/guestdbg.c
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert handle_stsi() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/priv.c | 9
...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kernel/asm-offsets.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/arch/s390/kernel/asm-offsets.c b/arch/s390/kernel/asm-offsets.c
index 32a284d..5e55d2a 100644
--- a/arch/s390/kernel/asm-offsets.c
+++ b/arch
in addition to any program interrupt.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/kvm_host.h | 18 +--
arch/s390/kvm/intercept.c| 65 +++-
2 files
...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/lowcore.h | 4 ++--
arch/s390/kernel/asm-offsets.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/s390/include/asm/lowcore.h b/arch/s390/include/asm/lowcore.h
index bbf8141
From: David Hildenbrand d...@linux.vnet.ibm.com
This patch adds the structs to the kernel headers needed to pass information
from/to userspace in order to debug a guest on s390 with hardware support.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger
can be injected in addition to any program
interrupt and uses the correct instruction length code (depending on the
interception code) for the injection of program interrupts.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
a page boundary.
If a protection exception happens while accessing any of the pages, the
instruction is suppressed and may not have modified any memory contents.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert handle_store_prefix() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm
...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/interrupt.c | 6 ++
arch/s390/kvm/kvm-s390.h | 2 ++
arch/s390/kvm/priv.c | 10 +-
3 files changed, 17 insertions(+), 1 deletion(-)
diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert handle lctl[g]() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/priv.c
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert handle_set_clock() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/priv.c
From: Heiko Carstens heiko.carst...@de.ibm.com
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/gaccess.h | 81
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert handle_test_block() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm
-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/gaccess.h | 24
1 file changed, 16 insertions(+), 8 deletions(-)
diff --git a/arch/s390/kvm/gaccess.h b/arch/s390/kvm/gaccess.h
index ed4749e..1079c8f 100644
--- a/arch/s390/kvm/gaccess.h
+++ b/arch/s390/kvm
From: Jens Freimann jf...@linux.vnet.ibm.com
Convert local irqs in __do_deliver_interrupt() to new guest
access functions.
Signed-off-by: Jens Freimann jf...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/interrupt.c | 81
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert handle_lpsw[e]() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/priv.c
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert __sigp_set_prefix() and handle_set_prefix() to new guest
access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert handle_stfl() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/priv.c | 6
From: Jens Freimann jf...@linux.vnet.ibm.com
Add a new data structure and function that allows to inject
all kinds of interrupt as defined in the PoP
Signed-off-by: Jens Freimann jf...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm
...@de.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
---
arch/s390/kvm/gaccess.h | 95 +++--
1 file changed, 93 insertions(+), 2 deletions
From: Heiko Carstens heiko.carst...@de.ibm.com
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/kvm_host.h | 12 -
arch/s390/kvm/gaccess.c
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert pfault code to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Dominik Dingel din...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/diag.c
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert __do_deliver_interrupt() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm
. These functions currently do
not perform any access checks, since there is no use case (yet?).
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/Makefile | 4
From: Heiko Carstens heiko.carst...@de.ibm.com
Convert handle_store_cpu_address() to new guest access functions.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Reviewed-by: Thomas Huth th...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390
(psw).t = 1; /* set dat bit */
Signed-off-by: Heiko Carstens heiko.carst...@de.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
---
arch/s390/include/asm/ptrace.h | 44
cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/kvm-s390.c | 2 +-
arch/s390/kvm/kvm-s390.h | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
index fe2396c..4b1df68 100644
() which can inject
vcpu's last exception if needed.
Signed-off-by: Heiko Carstens heiko.carst...@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/kvm_host.h | 1 +
arch/s390/kvm/kvm-s390.h | 33
...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Alexander Graf ag...@suse.de
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
Documentation/virtual/kvm/api.txt| 8 ++---
Documentation/virtual/kvm/devices/vm.txt | 6
arch/s390/kvm/kvm-s390
Carstens heiko.carst...@de.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
---
arch/s390/include/asm/ctl_reg.h | 14 ++
1 file changed, 14 insertions(+)
diff --git
vCPU creation. It can't be switched off
once enabled. All subsequently created vCPUs will be enabled for
CMMA according to the CMMA state of the VM.
Signed-off-by: Dominik Dingel din...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
[remove now unnecessary calls
mode).
Depending on the vcpu's addressing mode the upper 40 bits (24 bit addressing
mode), 33 bits (31 bit addressing mode) or no bits (64 bit addressing mode)
will be zeroed and the remaining bits will be returned.
Signed-off-by: Heiko Carstens heiko.carst...@de.ibm.com
Signed-off-by: Christian
base KVM patch KVM: add kvm_is_error_gpa() helper
Christian
Christian Borntraeger (1):
KVM: s390: Drop pending interrupts on guest exit
Cornelia Huck (1):
KVM: s390: reinject io interrupt on tpi failure
David Hildenbrand (8):
KVM: s390: extract irq parameters of intercepted program irqs
-off-by: Heiko Carstens heiko.carst...@de.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
---
include/linux/kvm_host.h | 7 +++
1 file changed, 7 insertions(+)
diff --git
On 09/04/14 16:21, Christian Borntraeger wrote:
Ingo, Arnaldo,
two fixes indentified during kvm on s390 development are available
since commit 538592ff0b008237ae88f5ce5fb1247127dc3ce5:
Merge tag 'perf-core-for-mingo' of
git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux
On 16/04/14 10:13, Marc Zyngier wrote:
Hi Mario,
On 16/04/14 02:24, Mario Smarduch wrote:
Hi Eric, Mark -
what repository should I use to pick up Eric patches?
The initial posting was there:
https://lists.cs.columbia.edu/pipermail/kvmarm/2014-April/008791.html
Reading Eric's
On 16/04/14 15:00, Jiri Olsa wrote:
On Wed, Apr 09, 2014 at 04:21:58PM +0200, Christian Borntraeger wrote:
From: Alexander Yarygin yary...@linux.vnet.ibm.com
Trace events potentially can have a '-' in their trace system name,
e.g. kvm on s390 defines kvm-s390:* tracepoints.
tools/perf could
On 16/04/14 15:28, Christian Borntraeger wrote:
On 16/04/14 15:00, Jiri Olsa wrote:
On Wed, Apr 09, 2014 at 04:21:58PM +0200, Christian Borntraeger wrote:
From: Alexander Yarygin yary...@linux.vnet.ibm.com
Trace events potentially can have a '-' in their trace system name,
e.g. kvm on s390
On 17/04/14 13:32, Jiri Olsa wrote:
On Tue, Mar 25, 2014 at 11:15:29AM +0100, Paolo Bonzini wrote:
Il 24/03/2014 21:49, Christian Borntraeger ha scritto:
event_legacy_tracepoint:
+PE_NAME '-' PE_NAME ':' PE_NAME
+{
+ struct parse_events_evlist *data = _data;
+ struct list_head *list
While preparing/testing some KVM on s390 patches for the next merge window
(target is kvm/next which is based on 3.15-rc1) I faced a very severe
performance hickup on guest paging (all anonymous memory).
All memory bound guests are in D state now and the system is barely unusable.
Reverting
From: Dominik Dingel din...@linux.vnet.ibm.com
To enable CMMA and to reset its state we use the vm kvm_device ioctls,
encapsulating attributes within the KVM_S390_VM_MEM_CTRL group.
Signed-off-by: Dominik Dingel din...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger borntrae
...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Acked-by: Alexander Graf ag...@suse.de
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
Documentation/virtual/kvm/api.txt| 8 ++---
Documentation/virtual/kvm/devices/vm.txt | 10 ++
arch/s390/kvm/kvm
introduces the notion of VM device attributes
Christian Borntraeger (1):
KVM: s390: Drop pending interrupts on guest exit
Cornelia Huck (1):
KVM: s390: reinject io interrupt on tpi failure
David Hildenbrand (8):
KVM: s390
On 22/04/14 12:55, Christian Borntraeger wrote:
While preparing/testing some KVM on s390 patches for the next merge window
(target is kvm/next which is based on 3.15-rc1) I faced a very severe
performance hickup on guest paging (all anonymous memory).
All memory bound guests are in D state
On 22/04/14 17:06, Johannes Weiner wrote:
Hi Christian,
On Tue, Apr 22, 2014 at 12:55:37PM +0200, Christian Borntraeger wrote:
While preparing/testing some KVM on s390 patches for the next merge window
(target is kvm/next which is based on 3.15-rc1) I faced a very severe
performance
On 21/04/14 15:25, Oleg Nesterov wrote:
async_pf_execute() has no reasons to adopt apf-mm, gup(current, mm)
should work just fine even if current has another or NULL -mm.
Recently kvm_async_page_present_sync() was added insedie the use_mm
section, but it seems that it doesn't need current-mm
On 21/04/14 15:26, Oleg Nesterov wrote:
get_user_pages(mm) is simply wrong if mm-mm_users == 0 and exit_mmap/etc
was already called (or is in progress), mm-mm_count can only pin mm-pgd
and mm_struct itself.
Change kvm_setup_async_pf/async_pf_execute to inc/dec mm-mm_users.
On 22/04/14 22:15, Christian Borntraeger wrote:
On 21/04/14 15:25, Oleg Nesterov wrote:
async_pf_execute() has no reasons to adopt apf-mm, gup(current, mm)
should work just fine even if current has another or NULL -mm.
Recently kvm_async_page_present_sync() was added insedie the use_mm
On 17/04/14 16:40, Jiri Olsa wrote:
On Mon, Mar 24, 2014 at 09:49:00PM +0100, Christian Borntraeger wrote:
From: Alexander Yarygin yary...@linux.vnet.ibm.com
Trace events potentially can have a '-' in their trace system name,
e.g. kvm on s390 defines kvm-s390:* tracepoints.
tools/perf could
On 23/04/14 13:45, Jiri Olsa wrote:
On Mon, Apr 21, 2014 at 07:43:50PM +0400, Alexander Yarygin wrote:
[...]
And a bit of offtopic :)
Apparently, s390 doesn't have syscalls:*, so some of the tests
don't work properly (or maybe I missed something? I set
CONFIG_FTRACE_SYSCALLS
to 'y' in my
, seems fine. I think patch2 really
does fix a bug. So if Paolo, Marcelo, Gleb agree (maybe do a test on x86 for
async_pf) both patches are good to go. Given that somebody tests this on x86:
Acked-by: Christian Borntraeger borntrae...@de.ibm.com
--
To unsubscribe from this list: send the line
-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
tools/perf/arch/x86/Makefile | 1 +
tools/perf/builtin-kvm.c | 6 +++---
tools/perf/config/Makefile | 4
3 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/tools/perf/arch
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
The goal of this series is to
- enable perf kvm stat on s390
- get rid of a perf warning insn_to_mnemonic not found when it tries to
decode s390 perf trace events
Review feedback is welcome.
In addition, The patch set touches s390/kvm code and perf code.
So any feedback on which tree to use for
From: Alexander Yarygin yary...@linux.vnet.ibm.com
This patch defines CPUINFO_PROC for s390 and implements get_cpuid().
Signed-off-by: Alexander Yarygin yary...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
be used to init
arch-depended things.
Signed-off-by: Alexander Yarygin yary...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
tools/perf/builtin-kvm.c | 37 +
1 file changed
literals.
Signed-off-by: Alexander Yarygin yary...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/x86/include/uapi/asm/kvm.h | 8
tools/perf/builtin-kvm.c| 10 --
2 files changed
Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/uapi/asm/kvm.h | 7 ++
tools/perf/MANIFEST | 3 +
tools/perf/arch/s390/Makefile| 1 +
tools/perf/builtin-kvm.c | 151 ++-
4
...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
tools/perf/builtin-kvm.c | 27 ---
1 file changed, 12 insertions(+), 15 deletions(-)
diff --git a/tools/perf/builtin-kvm.c b/tools/perf/builtin-kvm.c
index 202b9b0..9b97737 100644
--- a/tools/perf
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
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
From: Alexander Yarygin yary...@linux.vnet.ibm.com
This makes it possible for other architectures to decode to different
string lengths.
Needed by follow-up patch perf kvm: add stat support on s390.
Signed-off-by: Alexander Yarygin yary...@linux.vnet.ibm.com
Signed-off-by: Christian Borntraeger
...@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
: 'kvm-s390:*'
This patch adds an extra rule to event_legacy_tracepoint which handles
those cases. Without the patch, perf will not accept such tracepoints in
the -e option.
Signed-off-by: Alexander Yarygin yary...@linux.vnet.ibm.com
Tested-by: Christian Borntraeger borntrae...@de.ibm.com
Acked
On 28/04/14 18:39, Paolo Bonzini wrote:
From: Christian Borntraeger borntrae...@de.ibm.com
When starting lots of dataplane devices the bootup takes very long on
Christian's s390 with irqfd patches. With larger setups he is even
able to trigger some timeouts in some components. Turns out
Paolo, Gleb,
please consider belows pull request for 3.16.
Thanks
Christian
The following changes since commit 198c74f43f0f5473f99967aead30ddc622804bc1:
KVM: MMU: flush tlb out of mmu lock when write-protect the sptes (2014-04-23
17:49:52 -0300)
are available in the git repository at:
-cpu guests.
Signed-off-by: David Hildenbrand d...@linux.vnet.ibm.com
Reviewed-by: Dominik Dingel din...@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/asm/kvm_host.h | 2 +
arch/s390/kvm/kvm
of protection, it also
sets up the exception data accordingly.
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/gaccess.c | 28
arch/s390/kvm
From: Thomas Huth th...@linux.vnet.ibm.com
TEST BLOCK is also subject to the low-address protection, so we need
to check the destination address in our handler.
Signed-off-by: Thomas Huth th...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian
user pages.
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 | 55 ++-
1 file changed, 54 insertions(+), 1
Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/priv.c | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/arch/s390/kvm/priv.c b/arch/s390/kvm/priv.c
index 27f9051..a47157b 100644
--- a/arch/s390/kvm
not be touched outside of these functions.
Signed-off-by: David Hildenbrand 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/diag.c | 2 +-
arch/s390/kvm/intercept.c | 3 +--
arch/s390/kvm
On 30/04/14 10:07, Heiko Carstens wrote:
On Tue, Apr 29, 2014 at 03:36:43PM +0200, Christian Borntraeger wrote:
+static int handle_mvpg_pei(struct kvm_vcpu *vcpu)
+{
+unsigned long hostaddr, srcaddr, dstaddr;
+psw_t *psw = vcpu-arch.sie_block-gpsw;
+struct mm_struct *mm = current
On 03/05/14 22:18, Dan Carpenter wrote:
When copy_from_user() fails, this code returns the number of bytes
remaining instead of a negative error code. The positive number is
returned to the user but otherwise it is harmless.
Signed-off-by: Dan Carpenter dan.carpen...@oracle.com
Thanks.
David,
thanks for the review.
Are you ok with this change as well? The alternative is to shorten our
descriptions (in 1/11 s390: add sie exit reasons tables), which would make the
trace output less comprehensible, though.
Christian
On 25/04/14 11:12, Christian Borntraeger wrote:
From
On 02/05/14 20:14, David Ahern wrote:
On 5/2/14, 3:16 AM, Jiri Olsa wrote:
[...]
CC-ing David Ahern
I don't have the original emails, but looking at
https://lkml.org/lkml/2014/4/25/331
[PATCH 01/11] s390: add sie exit reasons tables
[PATCH 02/11] KVM: s390: Use trace tables from sie.h
On 25/04/14 11:12, Christian Borntraeger wrote:
+#if defined(__i386__) || defined(__x86_64__)
else if (!strcmp(kvm-report_event, mmio))
kvm-events_ops = mmio_events;
else if (!strcmp(kvm-report_event, ioport))
kvm-events_ops = ioport_events;
+#endif
On 28/04/14 18:39, Paolo Bonzini wrote:
From: Christian Borntraeger borntrae...@de.ibm.com
Given all your work, What about From: Paolo Bonzini pbonz...@redhat.com
plus
Based on an inital patch from Christian Borntraeger
When starting lots of dataplane devices the bootup takes very long
...@linux.vnet.ibm.com
Reviewed-by: Cornelia Huck cornelia.h...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
Documentation/virtual/kvm/api.txt | 2 ++
arch/s390/kvm/interrupt.c | 32
include/uapi/linux/kvm.h | 2 ++
3 files changed, 36
Paolo, Gleb,
the s390 queue has 3 fixes scheduled for 3.16.
The EFAULT fix fixes a bug that is only in kvm/next, and the
other fixes are long standing non-critical problems so no
cc stable.
Please apply.
The following changes since commit 57b5981cd38cbca3554c5e663b2361d9adea70c2:
Merge tag
-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/include/asm/kvm_host.h | 5 +++-
arch/s390/kvm/intercept.c| 49
2 files changed, 49 insertions(+), 5 deletions(-)
diff --git a/arch/s390/include/asm/kvm_host.h b/arch/s390/include/asm
-by: Christian Borntraeger borntrae...@de.ibm.com
---
arch/s390/kvm/guestdbg.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/arch/s390/kvm/guestdbg.c b/arch/s390/kvm/guestdbg.c
index 757ccef..3e8d409 100644
--- a/arch/s390/kvm/guestdbg.c
+++ b/arch/s390/kvm/guestdbg.c
Borntraeger borntrae...@de.ibm.com
Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com
---
Documentation/virtual/kvm/api.txt | 2 +-
arch/s390/kvm/kvm-s390.c | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/Documentation/virtual/kvm/api.txt
b/Documentation
401 - 500 of 1075 matches
Mail list logo
