On 12/9/24 12:07 PM, Christoph Schlameuss wrote:
Prevent null pointer dereference when processing the
KVM_DEV_FLIC_APF_ENABLE and KVM_DEV_FLIC_APF_DISABLE_WAIT ioctls in the
interrupt controller.
Reported-by: Claudio Imbrenda
Signed-off-by: Christoph Schlameuss
Please have a look into the do
On 12/9/24 12:07 PM, Christoph Schlameuss wrote:
Prevent null pointer dereference when processing the
KVM_DEV_FLIC_APF_ENABLE and KVM_DEV_FLIC_APF_DISABLE_WAIT ioctls in the
interrupt controller.
Reported-by: Claudio Imbrenda
Signed-off-by: Christoph Schlameuss
Please add a fixes tag to this
.
Signed-off-by: Christoph Schlameuss
---
Acked-by: Janosch Frank
.
Signed-off-by: Christoph Schlameuss
Acked-by: Janosch Frank
On 10/15/24 10:37 AM, Christoph Schlameuss wrote:
Add a test case manipulating s390 storage keys from within the ucontrol
VM.
Storage key instruction (ISKE, SSKE and RRBE) intercepts and
Keyless-subset facility are disabled on first use, where the skeys are
setup by KVM in non ucontrol VMs.
[
On 10/8/24 2:56 PM, Janosch Frank wrote:
On 10/8/24 9:42 AM, Christoph Schlameuss wrote:
Add a test case verifying KVM_SET_USER_MEMORY_REGION and
KVM_SET_USER_MEMORY_REGION2 cannot be executed on ucontrol VMs.
Executing this test case on not patched kernels will cause a null
pointer
On 10/8/24 9:42 AM, Christoph Schlameuss wrote:
Add a test case verifying KVM_SET_USER_MEMORY_REGION and
KVM_SET_USER_MEMORY_REGION2 cannot be executed on ucontrol VMs.
Executing this test case on not patched kernels will cause a null
pointer dereference in the host kernel.
This is fixed with co
On 10/8/24 9:42 AM, Christoph Schlameuss wrote:
Checkpatch thinks that we're doing a multiplication but we're obviously
not. Fix 4 instances where we adhered to wrong checkpatch advice.
Signed-off-by: Christoph Schlameuss
Reviewed-by: Janosch Frank
On 9/17/24 8:44 AM, Christoph Schlameuss wrote:
Fix pointer function arguments whitespaces.
Signed-off-by: Christoph Schlameuss
Generally we don't use the fixup prefix in upstream development AFAIK.
selftests: kvm: s390: Fix whitespace confusion in ucontrol test
Checkpatch thinks that we'r
On 9/17/24 8:44 AM, Christoph Schlameuss wrote:
Add a test case manipulating s390 storage keys from within the ucontrol
VM.
Signed-off-by: Christoph Schlameuss
With the nit fixed:
Reviewed-by: Janosch Frank
+ /* SSKE */
+ sync_regs->gprs[1] = skeyvalue;
+
translation handling to map memory on access.
Signed-off-by: Christoph Schlameuss
Reviewed-by: Janosch Frank
On 9/16/24 8:25 AM, Christoph Schlameuss wrote:
On Fri Sep 13, 2024 at 6:48 PM CEST, Claudio Imbrenda wrote:
On Fri, 13 Sep 2024 13:52:46 +0200
Christoph Schlameuss wrote:
Add a test case verifying basic running and interaction of ucontrol VMs.
Fill the segment and page tables for allocated m
On 9/13/24 1:52 PM, Christoph Schlameuss wrote:
Add a test case manipulating s390 storage keys from within the ucontrol
VM.
Signed-off-by: Christoph Schlameuss
Except for the two nits:
Reviewed-by: Janosch Frank
I'll think about what to do with the nits next week.
---
.../self
translation handling to map memory on access.
Signed-off-by: Christoph Schlameuss
Reviewed-by: Janosch Frank
On 8/23/24 3:03 PM, Christoph Schlameuss wrote:
On Fri Aug 23, 2024 at 10:02 AM CEST, Janosch Frank wrote:
On 8/19/24 6:03 PM, Christoph Schlameuss wrote:
On Fri Aug 16, 2024 at 4:29 PM CEST, Janosch Frank wrote:
On 8/15/24 5:45 PM, Christoph Schlameuss wrote:
Add a test case verifying basic
On 8/19/24 6:03 PM, Christoph Schlameuss wrote:
On Fri Aug 16, 2024 at 4:29 PM CEST, Janosch Frank wrote:
On 8/15/24 5:45 PM, Christoph Schlameuss wrote:
Add a test case verifying basic running and interaction of ucontrol VMs.
Fill the segment and page tables for allocated memory and map
On 8/19/24 6:00 PM, Christoph Schlameuss wrote:
On Fri Aug 16, 2024 at 4:36 PM CEST, Janosch Frank wrote:
On 8/15/24 5:45 PM, Christoph Schlameuss wrote:
[...]
+TEST_F(uc_kvm, uc_skey)
+{
+ u64 test_vaddr = self->base_gpa + VM_MEM_SIZE - (SZ_1M / 2);
+ struct kvm_sync_r
On 8/15/24 5:45 PM, Christoph Schlameuss wrote:
Add a test case manipulating s390 storage keys from within the ucontrol
VM.
Signed-off-by: Christoph Schlameuss
---
.../selftests/kvm/s390x/ucontrol_test.c | 76 +++
1 file changed, 76 insertions(+)
diff --git a/tools/tes
On 8/15/24 5:45 PM, Christoph Schlameuss wrote:
Add a test case verifying basic running and interaction of ucontrol VMs.
Fill the segment and page tables for allocated memory and map memory on
first access.
* uc_map_unmap
Store and load data to mapped and unmapped memory and use pic segment
On 8/7/24 5:45 PM, Christoph Schlameuss wrote:
This patch series adds a selftest suite to validate the s390x
architecture specific ucontrol KVM interface.
When creating a VM on s390x it is possible to create it as userspace
controlled VM or in short ucontrol VM.
These VMs delegates the managemen
On 8/7/24 5:45 PM, Christoph Schlameuss wrote:
Add a test case verifying basic running and interaction of ucontrol VMs.
Fill the segment and page tables for allocated memory and map memory on
first access.
* uc_map_unmap
Store and load data to mapped and unmapped memory and use pic segment
On 8/2/24 5:59 PM, Christoph Schlameuss wrote:
Add a test case manipulating s390 storage keys from within the ucontrol
VM.
Signed-off-by: Christoph Schlameuss
---
.../selftests/kvm/s390x/ucontrol_test.c | 78 +++
1 file changed, 78 insertions(+)
[...]
+TEST_F(uc_kv
On 8/2/24 5:59 PM, Christoph Schlameuss wrote:
Add a test case verifying basic running and interaction of ucontrol VMs.
Fill the segment and page tables for allocated memory and map memory on
first access.
* uc_map_unmap
Store and load data to mapped and unmapped memory and use pic segment
Schlameuss
Reviewed-by: Janosch Frank
/configs/debug_defconfig
index ea63a7342f5f..0c989caed19a 100644
--- a/arch/s390/configs/debug_defconfig
+++ b/arch/s390/configs/debug_defconfig
@@ -59,6 +59,7 @@ CONFIG_CMM=m
CONFIG_APPLDATA_BASE=y
CONFIG_S390_HYPFS_FS=y
CONFIG_KVM=m
+CONFIG_KVM_S390_UCONTROL=y
Reviewed-by: Janosch Frank
commit:
commit 7816e58967d0 ("kvm: s390: Reject memory region operations for ucontrol
VMs")
Reviewed-by: Janosch Frank
On 7/30/24 9:24 AM, Christoph Schlameuss wrote:
Add a test case verifying basic running and interaction of ucontrol VMs.
Fill the segment and page tables for allocated memory and map memory on
first access.
* uc_map_unmap
Store and load data to mapped and unmapped memory and use pic segment
On 7/30/24 9:24 AM, Christoph Schlameuss wrote:
[...]
+#endif /* SELFTEST_KVM_DEBUG_PRINT_H */
diff --git a/tools/testing/selftests/kvm/s390x/ucontrol_test.c
b/tools/testing/selftests/kvm/s390x/ucontrol_test.c
index d103a92e7495..029233374465 100644
--- a/tools/testing/selftests/kvm/s390x/ucont
On 7/30/24 9:24 AM, Christoph Schlameuss wrote:
Add test case running code interacting with registers within a
ucontrol VM.
* Add uc_gprs test case
The test uses the same VM setup using the fixture and debug macros
introduced in earlier patches in this series.
Signed-off-by: Christoph Schlameu
interface the kernel needs a
non-default config containing CONFIG_KVM_S390_UCONTROL.
This config needs to be set to built-in (y) as this cannot be built as
module.
Signed-off-by: Christoph Schlameuss
Reviewed-by: Janosch Frank
Reviewed-by: Janosch Frank
On 7/23/24 11:31, Christoph Schlameuss wrote:
Add test case running code interacting with registers within a
ucontrol VM.
* Add uc_gprs test case
The test uses the same VM setup using the fixture and debug macros
introduced in earlier patches in this series.
Signed-off-by: Christoph Schlameuss
On 7/23/24 11:31, Christoph Schlameuss wrote:
Add test suite to validate the s390x architecture specific ucontrol KVM
interface.
Make use of the selftest test harness.
* uc_cap_hpage testcase verifies that a ucontrol VM cannot be run with
hugepages.
To allow testing of the ucontrol interfac
On 7/23/24 11:31, Christoph Schlameuss wrote:
Subsequent tests do require direct manipulation of the SIE instruction.
This commit introduces the SIE definition for the selftests.
AFAIK ucontrol doesn't allow to manipulate the instruction, it allows
manipulation of the *SIE control block* that'
Schlameuss
Reviewed-by: Claudio Imbrenda
Reviewed-by: Janosch Frank
On 5/18/24 02:18, Kees Cook wrote:
Instead of mixing selftest harness and ksft helpers, perform SKIP
testing from the FIXTURE_SETUPs. This also means TEST_HARNESS_MAIN does
not need to be open-coded.
Signed-off-by: Kees Cook
---
Adding the author in CC.
This changes the skip behavior from o
memop: Add cmpxchg tests")
Signed-off-by: Nina Schoetterl-Glausch
Acked-by: Janosch Frank
On 2/20/24 22:12, Eric Farman wrote:
Hi Janosch,
Here is a new (final?) version for the AR/MEM_OP issue I'm attempting to
address. Hopefully they can be picked up to whatever tree makes sense.
I've got good and bad news for you :)
You need to re-base this patch set on Heiko's feature branch
On 2/20/24 15:15, Christian Borntraeger wrote:
Am 16.02.24 um 22:36 schrieb Eric Farman:
The routine ar_translation() can be reached by both the instruction
intercept path (where the access registers had been loaded with the
guest register contents), and the MEM_OP ioctls (which hadn't).
Since
39 matches
Mail list logo