[PATCH v2] media: pci/ivtv: Replace GPLv2 boilerplate text with SPDX

2025-08-01 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in 59 Temple Place anymore, so we should not ask the people to write to that address in case they need a copy of the GPL. Anyway, all other files in this directory already had their boilerplate text replaced by a proper SPDX tag in the earlier commit

[PATCH] drivers/media/pci/ivtv: Replace GPLv2 boilerplate text with SPDX

2025-08-01 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in 59 Temple Place anymore, so we should not ask the people to write to that address in case they need a copy of the GPL. Anyway, all other files in this directory already had their boilerplate text replaced by a proper SPDX tag in the earlier commit

Re: [PATCH v2] arch/x86/kvm/ioapic: Remove license boilerplate with bad FSF address

2025-08-01 Thread Thomas Huth
On 28/07/2025 17.50, Greg Kroah-Hartman wrote: On Mon, Jul 28, 2025 at 05:36:47PM +0200, Greg Kroah-Hartman wrote: On Mon, Jul 28, 2025 at 05:28:43PM +0200, Thomas Huth wrote: From: Thomas Huth The Free Software Foundation does not reside in "59 Temple Place" anymore, so we

[PATCH] x86/mtrr: Remove license boilerplate text with bad FSF address

2025-08-01 Thread Thomas Huth
From: Thomas Huth The Free Software Foundation does not reside in "675 Mass Ave, Cambridge" anymore, so we should not mention that address in the source code here. But instead of updating the address to their current location, let's rather drop the license boilerplate text here a

Re: [PATCH v2] arch/x86/kvm/ioapic: Remove license boilerplate with bad FSF address

2025-07-28 Thread Thomas Huth
On 28/07/2025 17.36, Greg Kroah-Hartman wrote: On Mon, Jul 28, 2025 at 05:28:43PM +0200, Thomas Huth wrote: From: Thomas Huth The Free Software Foundation does not reside in "59 Temple Place" anymore, so we should not mention that address in the source code here. But instead of up

[PATCH v2] arch/x86/kvm/ioapic: Remove license boilerplate with bad FSF address

2025-07-28 Thread Thomas Huth
From: Thomas Huth The Free Software Foundation does not reside in "59 Temple Place" anymore, so we should not mention that address in the source code here. But instead of updating the address to their current location, let's rather drop the license boilerplate text here and us

Re: [PATCH] arch/x86/kvm/ioapic: Remove license boilerplate with bad FSF address

2025-07-28 Thread Thomas Huth
On 28/07/2025 16.59, Marc-Etienne Vargenau (Nokia) wrote: Hello, That should be: // SPDX-License-Identifier: LGPL-2.1-or-later not // SPDX-License-Identifier: LGPL-2.1+ « LGPL-2.1+ » is deprecated Ah, stupid me, I copied it from the wrong file! Thanks for the hint, I'll send a v2! Thom

[PATCH] arch/x86/kvm/ioapic: Remove license boilerplate with bad FSF address

2025-07-28 Thread Thomas Huth
From: Thomas Huth The Free Software Foundation does not reside in "59 Temple Place" anymore, so we should not mention that address in the source code here. But instead of updating the address to their current location, let's rather drop the license boilerplate text here and us

[PATCH v2 6/6] LICENSES: Replace the obsolete address of the FSF in the GFDL-1.2

2025-07-21 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in the Franklin street anymore. Let's update the address with the link to their website, as suggested in the latest revisions of their GFDL-1.2 license: https://www.gnu.org/licenses/old-licenses/fdl-1.2.txt Signed-off-by: Thomas Huth --- LIC

[PATCH v2 4/6] LICENSES: Replace the obsolete address of the FSF in the LGPL-2.1

2025-07-21 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in the Franklin street anymore. Let's update the address with the link to their website, as suggested in the latest revisions of their LGPL-2.1 license: https://www.gnu.org/licenses/old-licenses/lgpl-2.1.txt Signed-off-by: Thomas Huth --- LIC

[PATCH v2 5/6] LICENSES: Replace the obsolete address of the FSF in the GFDL-1.1

2025-07-21 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in the Franklin street anymore. Let's update the address with the link to their website, as suggested in the latest revisions of their GFDL-1.1 license: https://www.gnu.org/licenses/old-licenses/fdl-1.1.txt Signed-off-by: Thomas Huth --- LIC

[PATCH v2 3/6] LICENSES: Replace the obsolete address of the FSF in the LGPL-2.0

2025-07-21 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in the Franklin street anymore. Let's update the address with the link to their website, as suggested in the latest revisions of their LGPL-2.0 license: https://www.gnu.org/licenses/old-licenses/lgpl-2.0.txt Signed-off-by: Thomas Huth --- LIC

[PATCH v2 1/6] LICENSES: Replace the obsolete address of the FSF in the GPL-1.0

2025-07-21 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in the Mass Ave anymore. Let's update the address with the link to their website, as suggested in the latest revisions of their GPL-1.0 license: https://www.gnu.org/licenses/old-licenses/gpl-1.0.txt Signed-off-by: Thomas Huth --- LICENSES/depre

[PATCH v2 2/6] LICENSES: Replace the obsolete address of the FSF in the GPL-2.0

2025-07-21 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in the Franklin street anymore. Let's update the address with the link to their website, as suggested in the latest revisions of their GPL-2.0 license: https://www.gnu.org/licenses/old-licenses/gpl-2.0.txt Signed-off-by: Thomas Huth --- LIC

[PATCH v2 0/6] LICENSES: Replace the obsolete address of the FSF

2025-07-21 Thread Thomas Huth
. v2: - Split into individual patches per license - Add patches for the GFDL licenses, too Thomas Huth (6): LICENSES: Replace the obsolete address of the FSF in the GPL-1.0 LICENSES: Replace the obsolete address of the FSF in the GPL-2.0 LICENSES: Replace the obsolete address of the FSF i

Re: [PATCH] LICENSES: Replace the obsolete address of the FSF

2025-07-16 Thread Thomas Huth
On 16/07/2025 13.38, Greg Kroah-Hartman wrote: On Fri, Jul 11, 2025 at 07:45:48AM +0200, Thomas Huth wrote: From: Thomas Huth The FSF does not reside in the Franklin street anymore. Let's update the address with the link to their website, as suggested in the latest revisions of their lic

[PATCH v3] powerpc: Drop GPL boilerplate text with obsolete FSF address

2025-07-11 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in the Franklin street anymore, so we should not request the people to write to this address. Fortunately, these header files already contain a proper SPDX license identifier, so it should be fine to simply drop all of this license boilerplate code here

Re: [PATCH v2] powerpc: Replace the obsolete address of the FSF

2025-07-11 Thread Thomas Huth
On 11/07/2025 07.52, Greg Kroah-Hartman wrote: On Fri, Jul 11, 2025 at 07:35:09AM +0200, Thomas Huth wrote: From: Thomas Huth The FSF does not reside in the Franklin street anymore. Let's update the address with the link to their website, as suggested in the latest revision of the GP

[PATCH] LICENSES: Replace the obsolete address of the FSF

2025-07-10 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in the Franklin street anymore. Let's update the address with the link to their website, as suggested in the latest revisions of their licenses. (See https://www.gnu.org/licenses/old-licenses/gpl-2.0.txt for example) Signed-off-by: Thomas

[PATCH v2] powerpc: Replace the obsolete address of the FSF

2025-07-10 Thread Thomas Huth
From: Thomas Huth The FSF does not reside in the Franklin street anymore. Let's update the address with the link to their website, as suggested in the latest revision of the GPL-2.0 license. (See https://www.gnu.org/licenses/old-licenses/gpl-2.0.txt for example) Acked-by: Segher Boesse

Re: [PATCH v1] s390/virtio_ccw: don't allocate/assign airqs for non-existing queues

2025-04-03 Thread Thomas Huth
k existing setups. Let's fix it without affecting existing setups for now by properly ignoring the non-existing queues, so the indicator bits will match the queue indexes. [1] https://lore.kernel.org/all/cover.1720611677.git@redhat.com/ Fixes: a229989d975e ("virtio: don't al

[PATCH 22/41] openrisc: Replace __ASSEMBLY__ with __ASSEMBLER__ in uapi headers

2025-03-14 Thread Thomas Huth
patch (done with a simple "sed -i" statement). Cc: Jonas Bonn Cc: Stefan Kristiansson Cc: Stafford Horne Cc: linux-openr...@vger.kernel.org Signed-off-by: Thomas Huth --- arch/openrisc/include/uapi/asm/ptrace.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 23/41] openrisc: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers

2025-03-14 Thread Thomas Huth
Cc: linux-openr...@vger.kernel.org Signed-off-by: Thomas Huth --- arch/openrisc/include/asm/mmu.h | 2 +- arch/openrisc/include/asm/page.h| 8 arch/openrisc/include/asm/pgtable.h | 4 ++-- arch/openrisc/include/asm/processor.h | 4 ++-- arch/openrisc/include/asm/pt

Re: [PATCH 1/1] s390/virtio_ccw: fix config change notifications

2024-06-12 Thread Thomas Huth
suggetions? Closes: https://issues.redhat.com/browse/RHEL-39983 ? Anyway, I've tested the patch and it indeed fixes the problem with virtio-balloon and the link state for me: Tested-by: Thomas Huth

Re: [PATCH v1 1/2] s390/kvm: split kvm_s390_real_to_abs

2021-03-19 Thread Thomas Huth
_kvm_s390_real_to_abs(u32 prefix, unsigned long gra) Just a matter of taste, but maybe this could be named differently? kvm_s390_real2abs_prefix() ? kvm_s390_prefix_real_to_abs()? Anyway: Reviewed-by: Thomas Huth

Re: [kvm-unit-tests PATCH] x86: Add tests for PKS

2021-01-18 Thread Thomas Huth
On 05/11/2020 09.18, Chenyi Qiang wrote: This unit-test is intended to test the KVM support for Protection Keys for Supervisor Pages (PKS). If CR4.PKS is set in long mode, supervisor pkeys are checked in addition to normal paging protections and Access or Write can be disabled via a MSR update wi

Re: [PATCH 3/6] KVM: selftests: Convert iterations to int in dirty_log_perf_test

2021-01-12 Thread Thomas Huth
On 12/01/2021 22.42, Ben Gardon wrote: In order to add an iteration -1 to indicate that the memory population phase has not yet completed, convert the interations counters to ints. No functional change intended. Reviewed-by: Jacob Xu Signed-off-by: Ben Gardon --- .../selftests/kvm/dirty_lo

Re: [PATCH 2/6] KVM: selftests: Avoid flooding debug log while populating memory

2021-01-12 Thread Thomas Huth
hile (READ_ONCE(vcpu_last_completed_iteration[vcpu_id]) + != iteration) + ; } ts_diff = timespec_elapsed(start); Reviewed-by: Thomas Huth

Re: [kvm-unit-tests PATCH] x86: pmu: Test full-width counter writes support

2020-06-16 Thread Thomas Huth
On 29/05/2020 09.43, Like Xu wrote: > When the full-width writes capability is set, use the alternative MSR > range to write larger sign counter values (up to GP counter width). > > Signed-off-by: Like Xu > --- > lib/x86/msr.h | 1 + > x86/pmu.c | 125 ++

Re: [PATCH trivial] KVM: PPC: Remove superfluous check for non-zero return value

2019-09-18 Thread Thomas Huth
On 18/09/2019 22.54, Greg Kurz wrote: > On Wed, 18 Sep 2019 18:44:36 +0200 > Greg Kurz wrote: > >> On Wed, 11 Sep 2019 21:52:35 +0200 >> Thomas Huth wrote: >> >>> After the kfree()s haven been removed in the previous >>> commit 9798f4ea71ea

Re: [PATCH] KVM: s390: Do not leak kernel stack data in the KVM_S390_INTERRUPT ioctl

2019-09-13 Thread Thomas Huth
On 13/09/2019 09.37, David Hildenbrand wrote: > On 13.09.19 09:34, Thomas Huth wrote: >> On 13/09/2019 09.20, Cornelia Huck wrote: >>> On Thu, 12 Sep 2019 13:23:38 +0200 >>> Thomas Huth wrote: >>> >>>> Hmm, we already talked about deprecating su

Re: [PATCH] KVM: s390: Do not leak kernel stack data in the KVM_S390_INTERRUPT ioctl

2019-09-13 Thread Thomas Huth
On 13/09/2019 09.20, Cornelia Huck wrote: > On Thu, 12 Sep 2019 13:23:38 +0200 > Thomas Huth wrote: > >> Hmm, we already talked about deprecating support for pre-3.15 kernel >> stuff in the past (see >> https://wiki.qemu.org/ChangeLog/2.12#Future_incompatible_change

[PATCH v2] KVM: s390: Do not leak kernel stack data in the KVM_S390_INTERRUPT ioctl

2019-09-12 Thread Thomas Huth
ypes, so that we immediately get a proper error code in case we add more interrupt types to do_inject_vcpu() without updating s390int_to_s390irq() sometime in the future. Cc: sta...@vger.kernel.org Reviewed-by: David Hildenbrand Reviewed-by: Christian Borntraeger Signed-off-by: Thomas Huth --- a

Re: [PATCH] KVM: s390: Do not leak kernel stack data in the KVM_S390_INTERRUPT ioctl

2019-09-12 Thread Thomas Huth
On 12/09/2019 12.52, Christian Borntraeger wrote: > > > On 12.09.19 11:20, Thomas Huth wrote: >> On 12/09/2019 11.14, David Hildenbrand wrote: >>> On 12.09.19 11:00, Thomas Huth wrote: >>>> When the userspace program runs the KVM_S390_INTERRUPT ioctl to injec

Re: [PATCH] KVM: s390: Do not leak kernel stack data in the KVM_S390_INTERRUPT ioctl

2019-09-12 Thread Thomas Huth
On 12/09/2019 12.47, Christian Borntraeger wrote: > > > On 12.09.19 11:00, Thomas Huth wrote: >> When the userspace program runs the KVM_S390_INTERRUPT ioctl to inject >> an interrupt, we convert them from the legacy struct kvm_s390_interrupt >> to the new

Re: [PATCH] KVM: s390: Do not leak kernel stack data in the KVM_S390_INTERRUPT ioctl

2019-09-12 Thread Thomas Huth
On 12/09/2019 11.14, David Hildenbrand wrote: > On 12.09.19 11:00, Thomas Huth wrote: >> When the userspace program runs the KVM_S390_INTERRUPT ioctl to inject >> an interrupt, we convert them from the legacy struct kvm_s390_interrupt >> to the new struct kvm_s390_irq via t

[PATCH] KVM: s390: Do not leak kernel stack data in the KVM_S390_INTERRUPT ioctl

2019-09-12 Thread Thomas Huth
in s390int_to_s390irq(), too. And while we're at it, make sure that s390int_to_s390irq() now directly returns -EINVAL for unknown interrupt types, so that we do not run into this problem again in case we add more interrupt types to do_inject_vcpu() sometime in the future. Signed-off-by: Thomas

[PATCH] KVM: s390: Remove unused parameter from __inject_sigp_restart()

2019-09-12 Thread Thomas Huth
It's not required, so drop it to make it clear that this interrupt does not have any extra parameters. Signed-off-by: Thomas Huth --- arch/s390/kvm/interrupt.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c

[PATCH] KVM: PPC: Remove superfluous check for non-zero return value

2019-09-11 Thread Thomas Huth
After the kfree()s haven been removed in the previous commit 9798f4ea71ea ("fix rollback when kvmppc_xive_create fails"), the code can be simplified even more to simply always "return ret" now. Signed-off-by: Thomas Huth --- arch/powerpc/kvm/book3s_xive.c| 5 +--

Re: [PATCH v2 1/2] KVM: s390: Disallow invalid bits in kvm_valid_regs and kvm_dirty_regs

2019-09-04 Thread Thomas Huth
On 04/09/2019 11.15, David Hildenbrand wrote: > On 04.09.19 11:11, Christian Borntraeger wrote: >> >> >> On 04.09.19 11:05, David Hildenbrand wrote: >>> On 04.09.19 10:51, Thomas Huth wrote: >>>> If unknown bits are set in kvm_valid_regs or kvm_di

Re: [PATCH v2 1/2] KVM: s390: Disallow invalid bits in kvm_valid_regs and kvm_dirty_regs

2019-09-04 Thread Thomas Huth
On 04/09/2019 11.05, David Hildenbrand wrote: > On 04.09.19 10:51, Thomas Huth wrote: >> If unknown bits are set in kvm_valid_regs or kvm_dirty_regs, this >> clearly indicates that something went wrong in the KVM userspace >> application. The x86 variant of KVM already contai

[PATCH v2 0/2] KVM: s390: Check for invalid bits in kvm_valid_regs and kvm_dirty_regs

2019-09-04 Thread Thomas Huth
Avoid invalid bits in kvm_valid_regs and kvm_dirty_regs on s390x. v2: - Split the single patch from v1 into two separate patches (I've kept the Reviewed-bys from v1, but if you don't agree with the patch description of the 2nd patch, please complain) Thomas Huth (2):

[PATCH v2 1/2] KVM: s390: Disallow invalid bits in kvm_valid_regs and kvm_dirty_regs

2019-09-04 Thread Thomas Huth
stian Borntraeger Reviewed-by: Cornelia Huck Signed-off-by: Thomas Huth --- arch/s390/include/uapi/asm/kvm.h | 6 ++ arch/s390/kvm/kvm-s390.c | 4 2 files changed, 10 insertions(+) diff --git a/arch/s390/include/uapi/asm/kvm.h b/arch/s390/include/uapi/asm/kvm.h index 47104e5

[PATCH v2 2/2] KVM: selftests: Test invalid bits in kvm_valid_regs and kvm_dirty_regs on s390x

2019-09-04 Thread Thomas Huth
Huck Signed-off-by: Thomas Huth --- .../selftests/kvm/s390x/sync_regs_test.c | 30 +++ 1 file changed, 30 insertions(+) diff --git a/tools/testing/selftests/kvm/s390x/sync_regs_test.c b/tools/testing/selftests/kvm/s390x/sync_regs_test.c index bbc93094519b..d5290b4ad636 100644

[PATCH] KVM: s390: Disallow invalid bits in kvm_valid_regs and kvm_dirty_regs

2019-09-04 Thread Thomas Huth
, too. Signed-off-by: Thomas Huth --- arch/s390/include/uapi/asm/kvm.h | 6 arch/s390/kvm/kvm-s390.c | 4 +++ .../selftests/kvm/s390x/sync_regs_test.c | 30 +++ 3 files changed, 40 insertions(+) diff --git a/arch/s390/include/uapi/asm/kvm.h b

Re: [PATCH v3] KVM: selftests: Add a test for the KVM_S390_MEM_OP ioctl

2019-08-29 Thread Thomas Huth
On 29/08/2019 15.26, Christian Borntraeger wrote: > > > On 29.08.19 15:07, Thomas Huth wrote: >> Check that we can write and read the guest memory with this s390x >> ioctl, and that some error cases are handled correctly. >> >> Signed-off-by: Thomas Huth >>

[PATCH v3] KVM: selftests: Add a test for the KVM_S390_MEM_OP ioctl

2019-08-29 Thread Thomas Huth
Check that we can write and read the guest memory with this s390x ioctl, and that some error cases are handled correctly. Signed-off-by: Thomas Huth --- v3: - Replaced wrong copy-n-pasted report string with a proper one - Check for errno after calling the ioctl with size = 0 tools/testing

Re: [PATCH] KVM: s390: improve documentation for S390_MEM_OP

2019-08-29 Thread Thomas Huth
n to for > +KVM_S390_MEMOP_LOGICAL_READ, or where the data that should be written is > +stored for a KVM_S390_MEMOP_LOGICAL_WRITE. When KVM_S390_MEMOP_F_CHECK_ONLY > +is specified, "buf" is unused and can be NULL. "ar" designates the access > +register number to be used; the valid range is 0..15. > > The "reserved" field is meant for future extensions. It is not used by > KVM with the currently defined set of flags. > Reviewed-by: Thomas Huth

Re: [PATCH v2] KVM: selftests: Add a test for the KVM_S390_MEM_OP ioctl

2019-08-29 Thread Thomas Huth
On 29/08/2019 14.21, Janosch Frank wrote: > On 8/29/19 2:14 PM, Thomas Huth wrote: >> Check that we can write and read the guest memory with this s390x >> ioctl, and that some error cases are handled correctly. >> >> Signed-off-by: Thomas Huth >> --- >> v

[PATCH v2] KVM: s390: Test for bad access register and size at the start of S390_MEM_OP

2019-08-29 Thread Thomas Huth
and this will cause a kernel warning. Signed-off-by: Thomas Huth --- v2: Check mop->size to be non-zero 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 f329dcb3f44c..49d779ae 100644

[PATCH v2] KVM: selftests: Add a test for the KVM_S390_MEM_OP ioctl

2019-08-29 Thread Thomas Huth
Check that we can write and read the guest memory with this s390x ioctl, and that some error cases are handled correctly. Signed-off-by: Thomas Huth --- v2: Check the ioctl also with "size" set to 0 tools/testing/selftests/kvm/Makefile | 1 + tools/testing/selftests/kvm/s39

Re: [PATCH] KVM: s390: Test for bad access register at the start of S390_MEM_OP

2019-08-29 Thread Thomas Huth
On 29/08/2019 13.15, Janosch Frank wrote: [...] > By the way, I think we want to check mop->size for 0 before giving it to > vmalloc and working with it. You're right! This currently triggers a kernel warning message with a Call Trace! I'll add a check to my new memop selftest and send a patch...

Re: [PATCH] KVM: s390: Test for bad access register at the start of S390_MEM_OP

2019-08-29 Thread Thomas Huth
On 29/08/2019 13.18, Cornelia Huck wrote: [...] > > Btw: should Documentation/virt/kvm/api.txt spell out the valid range > for ar explicitly? > That certainly would not hurt. Care to send a patch, or shall I assemble one? Thomas

[PATCH] KVM: s390: Test for bad access register at the start of S390_MEM_OP

2019-08-29 Thread Thomas Huth
ng on a check somewhere deep down the calling chain, so let's add another check to kvm_s390_guest_mem_op() directly. Signed-off-by: Thomas Huth --- 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

[PATCH] KVM: selftests: Add a test for the KVM_S390_MEM_OP ioctl

2019-08-29 Thread Thomas Huth
Check that we can write and read the guest memory with this s390x ioctl, and that some error cases are handled correctly. Signed-off-by: Thomas Huth --- This test uses the ucall() interface, so this patch needs to be applied on top of my "Implement ucall() for s390x" patch (which is

[PATCH] powerpc: Replace GPL boilerplate with SPDX identifiers

2019-08-27 Thread Thomas Huth
The FSF does not reside in "675 Mass Ave, Cambridge" anymore... let's simply use proper SPDX identifiers instead. Signed-off-by: Thomas Huth --- arch/powerpc/include/uapi/asm/spu_info.h | 14 -- arch/powerpc/kernel/eeh_driver.c | 18 +

Re: [PATCH] KVM: selftests: Detect max PA width from cpuid

2019-08-26 Thread Thomas Huth
On 26/08/2019 09.57, Peter Xu wrote: > The dirty_log_test is failing on some old machines like Xeon E3-1220 > with tripple faults when writting to the tracked memory region: > > Test iterations: 32, interval: 10 (ms) > Testing guest mode: PA-bits:52, VA-bits:48, 4K pages > guest physical tes

[PATCH v3 2/3] KVM: selftests: Implement ucall() for s390x

2019-07-31 Thread Thomas Huth
On s390x, we can neither exit via PIO nor MMIO, but have to use an instruction like DIAGNOSE. Now that ucall() is implemented, we can use it in the sync_reg_test on s390x, too. Reviewed-by: Andrew Jones Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/Makefile | 2 +- tools

[PATCH v3 1/3] KVM: selftests: Split ucall.c into architecture specific files

2019-07-31 Thread Thomas Huth
de in ucall.c currently looks more complex than required. Let's split this up into architecture specific ucall.c files instead, so we can get rid of the #ifdefs and the unnecessary ucall_type_t handling. Reviewed-by: Andrew Jones Signed-off-by: Thomas Huth --- tools/testing/selftests/kv

[PATCH v3 0/3] KVM: selftests: Enable ucall and dirty_log_test on s390x

2019-07-31 Thread Thomas Huth
the dirty_log patch Thomas Huth (3): KVM: selftests: Split ucall.c into architecture specific files KVM: selftests: Implement ucall() for s390x KVM: selftests: Enable dirty_log_test on s390x tools/testing/selftests/kvm/Makefile | 9 +- tools/testing/selftests/kvm

[PATCH v3 3/3] KVM: selftests: Enable dirty_log_test on s390x

2019-07-31 Thread Thomas Huth
area. 0xc000 seems to be a good alternative that should work on x86 and aarch64, too. Acked-by: Paolo Bonzini Reviewed-by: Andrew Jones Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/Makefile | 1 + tools/testing/selftests/kvm/dirty_log_test.c | 59

[PATCH] KVM: selftests: Update gitignore file for latest changes

2019-07-31 Thread Thomas Huth
The kvm_create_max_vcpus test has been moved to the main directory, and sync_regs_test is now available on s390x, too. Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/.gitignore | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/kvm

Re: [PATCH v2 0/3] KVM: selftests: Enable ucall and dirty_log_test on s390x

2019-07-31 Thread Thomas Huth
On 31/07/2019 15.32, Thomas Huth wrote: > Implement the ucall() interface on s390x to be able to use the > dirty_log_test KVM selftest on s390x, too. > > v2: > - Split up ucall.c into architecture specific files > - Removed some #ifdef __s390x__ in the dirty_log patch &

[PATCH v2 0/3] KVM: selftests: Enable ucall and dirty_log_test on s390x

2019-07-31 Thread Thomas Huth
Implement the ucall() interface on s390x to be able to use the dirty_log_test KVM selftest on s390x, too. v2: - Split up ucall.c into architecture specific files - Removed some #ifdef __s390x__ in the dirty_log patch Thomas Huth (3): KVM: selftests: Split ucall.c into architecture specific

[PATCH v2 3/3] KVM: selftests: Enable dirty_log_test on s390x

2019-07-31 Thread Thomas Huth
area. 0xc000 seems to be a good alternative that should work on x86 and aarch64, too. Acked-by: Paolo Bonzini Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/Makefile | 1 + tools/testing/selftests/kvm/dirty_log_test.c | 59 +--- 2 files changed, 53

[PATCH v2 1/3] KVM: selftests: Split ucall.c into architecture specific files

2019-07-31 Thread Thomas Huth
de in ucall.c currently looks more complex than required. Let's split this up into architecture specific ucall.c files instead, so we can get rid of the #ifdefs and the unnecessary ucall_type_t handling. Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/Makefile | 6 +- too

[PATCH v2 2/3] KVM: selftests: Implement ucall() for s390x

2019-07-31 Thread Thomas Huth
On s390x, we can neither exit via PIO nor MMIO, but have to use an instruction like DIAGNOSE. Now that ucall() is implemented, we can use it in the sync_reg_test on s390x, too. Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/Makefile | 2 +- tools/testing/selftests/kvm/lib

Re: [PATCH 1/2] KVM: selftests: Implement ucall() for s390x

2019-07-31 Thread Thomas Huth
On 31/07/2019 12.28, Andrew Jones wrote: > On Wed, Jul 31, 2019 at 11:43:16AM +0200, Thomas Huth wrote: >> On 30/07/2019 12.48, Andrew Jones wrote: >>> On Tue, Jul 30, 2019 at 12:01:11PM +0200, Thomas Huth wrote: >>>> On s390x, we can neither exit via PIO nor

Re: [PATCH 1/2] KVM: selftests: Implement ucall() for s390x

2019-07-31 Thread Thomas Huth
On 30/07/2019 12.48, Andrew Jones wrote: > On Tue, Jul 30, 2019 at 12:01:11PM +0200, Thomas Huth wrote: >> On s390x, we can neither exit via PIO nor MMIO, but have to use >> an instruction like DIAGNOSE. While we're at it, rename UCALL_PIO >> to UCALL_DEFAULT, since PI

Re: [PATCH 2/2] KVM: selftests: Enable dirty_log_test on s390x

2019-07-31 Thread Thomas Huth
On 30/07/2019 12.57, Andrew Jones wrote: > On Tue, Jul 30, 2019 at 12:01:12PM +0200, Thomas Huth wrote: >> To run the dirty_log_test on s390x, we have to make sure that we >> access the dirty log bitmap with little endian byte ordering and >> we have to properly align the

Re: [PATCH 2/2] KVM: selftests: Enable dirty_log_test on s390x

2019-07-30 Thread Thomas Huth
On 30/07/2019 16.57, Christian Borntraeger wrote: > > > On 30.07.19 12:01, Thomas Huth wrote: >> To run the dirty_log_test on s390x, we have to make sure that we >> access the dirty log bitmap with little endian byte ordering and >> we have to properly align the memslo

[PATCH 1/2] KVM: selftests: Implement ucall() for s390x

2019-07-30 Thread Thomas Huth
implemented, we can use it in the sync_reg_test on s390x, too. Signed-off-by: Thomas Huth --- .../testing/selftests/kvm/include/kvm_util.h | 2 +- tools/testing/selftests/kvm/lib/ucall.c | 34 +++ .../selftests/kvm/s390x/sync_regs_test.c | 6 ++-- 3 files changed

[PATCH 0/2] KVM: selftests: Enable ucall and dirty_log_test on s390x

2019-07-30 Thread Thomas Huth
Implement the ucall() interface on s390x to be able to use the dirty_log_test KVM selftest on s390x, too. Thomas Huth (2): KVM: selftests: Implement ucall() for s390x KVM: selftests: Enable dirty_log_test on s390x tools/testing/selftests/kvm/Makefile | 1 + tools/testing/selftests

[PATCH 2/2] KVM: selftests: Enable dirty_log_test on s390x

2019-07-30 Thread Thomas Huth
first time, so we have to make sure that we touch all pages during the first iteration to keep the test in sync here. Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/Makefile | 1 + tools/testing/selftests/kvm/dirty_log_test.c | 70 ++-- 2 files changed, 66

[PATCH] kernel/configs: Replace GPL boilerplate code with SPDX identifier

2019-07-22 Thread Thomas Huth
The FSF does not reside in "675 Mass Ave, Cambridge" anymore... let's replace the old GPL boilerplate code with a proper SPDX identifier instead. Signed-off-by: Thomas Huth --- kernel/configs.c | 16 +--- 1 file changed, 1 insertion(+), 15 deletions(-) diff --git a/k

[RFC PATCH] KVM: PPC: Enable the kvm_create_max_vcpus selftest on ppc64

2019-07-18 Thread Thomas Huth
the vCPUs. Signed-off-by: Thomas Huth --- RFC since the stubs are a little bit ugly (does someone here like to implement them?), and since it's a little bit annoying that you have to raise the ulimit for this test in case the kernel provides more vCPUs than the default ulimit... tools/te

[PATCH] sparc: Remove redundant copy of the LGPL-2.0

2019-05-27 Thread Thomas Huth
We already provide the LGPL-2.0 text in LICENSES/preferred/LGPL-2.0, so there is no need for this additional copy here. Signed-off-by: Thomas Huth --- arch/sparc/lib/COPYING.LIB | 481 - 1 file changed, 481 deletions(-) delete mode 100644 arch/sparc/lib

Re: [Qemu-devel] Running linux on qemu omap

2019-05-26 Thread Thomas Huth
On 24/05/2019 20.59, Aaro Koskinen wrote: > Hi, > > On Fri, May 24, 2019 at 06:00:18PM +0300, Aaro Koskinen wrote: >> Please don't delete OMAP boards quite yet :) In the mainline kernel >> they are not orphaned, they frequently get tested using actual hardware, >> and QEMU would help in additional

Re: [PATCH 5/9] KVM: selftests: Align memory region addresses to 1M on s390x

2019-05-24 Thread Thomas Huth
On 24/05/2019 10.29, Christian Borntraeger wrote: > > > On 23.05.19 19:40, Andrew Jones wrote: >> On Thu, May 23, 2019 at 06:43:05PM +0200, Thomas Huth wrote: >>> On s390x, there is a constraint that memory regions have to be aligned >>> to 1M (or running t

[PATCH 7/9] KVM: selftests: Add the sync_regs test for s390x

2019-05-23 Thread Thomas Huth
usable on s390x), so it simply drops out of the VM with a diag 0x501 breakpoint instead. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + tools/testing/selftests/kvm/Makefile | 2 + .../selftests/kvm/s390x/sync_regs_test.c | 151

[PATCH 9/9] KVM: selftests: Move kvm_create_max_vcpus test to generic code

2019-05-23 Thread Thomas Huth
There is nothing x86-specific in the test apart from the VM_MODE_P52V48_4K which we can now replace with VM_MODE_DEFAULT. Thus let's move the file to the main folder and enable it for aarch64 and s390x, too. Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/Mak

[PATCH 8/9] KVM: s390: Do not report unusabled IDs via KVM_CAP_MAX_VCPU_ID

2019-05-23 Thread Thomas Huth
-off-by: Thomas Huth --- arch/mips/kvm/mips.c | 3 +++ arch/powerpc/kvm/powerpc.c | 3 +++ arch/s390/kvm/kvm-s390.c | 1 + arch/x86/kvm/x86.c | 3 +++ virt/kvm/arm/arm.c | 3 +++ virt/kvm/kvm_main.c| 2 -- 6 files changed, 13 insertions(+), 2 deletions(-) diff --git

[PATCH 5/9] KVM: selftests: Align memory region addresses to 1M on s390x

2019-05-23 Thread Thomas Huth
On s390x, there is a constraint that memory regions have to be aligned to 1M (or running the VM will fail). Introduce a new "alignment" variable in the vm_userspace_mem_region_add() function which now can be used for both, huge page and s390x alignment requirements. Signed-off-by: T

[PATCH 6/9] KVM: selftests: Add processor code for s390x

2019-05-23 Thread Thomas Huth
Code that takes care of basic CPU setup, page table walking, etc. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + tools/testing/selftests/kvm/Makefile | 1 + .../selftests/kvm/include/s390x/processor.h | 22 ++ .../selftests/kvm/lib/s390x

[PATCH 3/9] kvm: selftests: aarch64: fix default vm mode

2019-05-23 Thread Thomas Huth
7;s good to get it fixed in advance.) Reported-by: Thomas Huth Signed-off-by: Andrew Jones --- tools/testing/selftests/kvm/lib/aarch64/processor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/kvm/lib/aarch64/processor.c b/tools/testing/selftests/

[PATCH 4/9] KVM: selftests: Introduce a VM_MODE_DEFAULT macro for the default bits

2019-05-23 Thread Thomas Huth
This will be required later for tests like the kvm_create_max_vcpus test that do not use the vm_create_default() function. Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/include/kvm_util.h | 6 ++ tools/testing/selftests/kvm/lib/aarch64/processor.c | 2 +- tools/testing

[PATCH 1/9] KVM: selftests: Wrap vcpu_nested_state_get/set functions with x86 guard

2019-05-23 Thread Thomas Huth
struct kvm_nested_state is only available on x86 so far. To be able to compile the code on other architectures as well, we need to wrap the related code with #ifdefs. Reviewed-by: Andrew Jones Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/include/kvm_util.h | 2 ++ tools/testing

[PATCH 2/9] KVM: selftests: Guard struct kvm_vcpu_events with __KVM_HAVE_VCPU_EVENTS

2019-05-23 Thread Thomas Huth
The struct kvm_vcpu_events code is only available on certain architectures (arm, arm64 and x86). To be able to compile kvm_util.c also for other architectures, we have to fence the code with __KVM_HAVE_VCPU_EVENTS. Reviewed-by: David Hildenbrand Signed-off-by: Thomas Huth --- tools/testing

[PATCH v1 0/9] KVM selftests for s390x

2019-05-23 Thread Thomas Huth
rew Jones (1): kvm: selftests: aarch64: fix default vm mode Thomas Huth (8): KVM: selftests: Wrap vcpu_nested_state_get/set functions with x86 guard KVM: selftests: Guard struct kvm_vcpu_events with __KVM_HAVE_VCPU_EVENTS KVM: selftests: Introduce a VM_MODE_DEFAULT macro for the defa

Re: [Qemu-devel] Running linux on qemu omap

2019-05-23 Thread Thomas Huth
On 22/05/2019 20.19, Aaro Koskinen wrote: > Hi, > > On Wed, May 22, 2019 at 11:33:41AM +0200, Corentin Labbe wrote: >> qemu-system-arm -M help |grep OMAP >> cheetah Palm Tungsten|E aka. Cheetah PDA (OMAP310) >> n800 Nokia N800 tablet aka. RX-34 (OMAP2420) >> n810

Re: [RFC PATCH 4/4] KVM: selftests: Add the sync_regs test for s390x

2019-05-23 Thread Thomas Huth
On 23/05/2019 12.56, Andrew Jones wrote: > On Thu, May 16, 2019 at 01:12:53PM +0200, Thomas Huth wrote: >> The test is an adaption of the same test for x86. Note that there >> are some differences in the way how s390x deals with the kvm_valid_regs >> in struct kvm_run, so som

[PATCH] KVM: selftests: Wrap vcpu_nested_state_get/set functions with x86 guard

2019-05-23 Thread Thomas Huth
struct kvm_nested_state is only available on x86 so far. To be able to compile the code on other architectures as well, we need to wrap the related code with #ifdefs. Signed-off-by: Thomas Huth --- tools/testing/selftests/kvm/include/kvm_util.h | 2 ++ tools/testing/selftests/kvm/lib/kvm_util.c

Re: [PATCH] kvm: selftests: avoid type punning

2019-05-20 Thread Thomas Huth
@@ -75,7 +75,7 @@ void set_revision_id_for_vmcs12(struct kvm_nested_state > *state, > u32 vmcs12_revision) > { > /* Set revision_id in vmcs12 to vmcs12_revision. */ > - *(u32 *)(state->data) = vmcs12_revision; > + memcpy(state->data, &vmcs12_revision, sizeof(u32)); > } > > void set_default_state(struct kvm_nested_state *state) > Reviewed-by: Thomas Huth

Re: [RFC PATCH 0/4] KVM selftests for s390x

2019-05-20 Thread Thomas Huth
On 20/05/2019 13.20, Paolo Bonzini wrote: > On 16/05/19 13:12, Thomas Huth wrote: >> This patch series enables the KVM selftests for s390x. As a first >> test, the sync_regs from x86 has been adapted to s390x. >> >> Please note that the ucall() interface is not used y

[PATCH] KVM: selftests: Remove duplicated TEST_ASSERT in hyperv_cpuid.c

2019-05-20 Thread Thomas Huth
The check for entry->index == 0 is done twice. One time should be sufficient. Suggested-by: Vitaly Kuznetsov Signed-off-by: Thomas Huth --- Vitaly already noticed this in his review to the "Fix a condition in test_hv_cpuid()" patch a couple of days ago, but so far I haven'

Re: [RFC PATCH 1/4] KVM: selftests: Guard struct kvm_vcpu_events with __KVM_HAVE_VCPU_EVENTS

2019-05-20 Thread Thomas Huth
On 20/05/2019 09.12, Christian Borntraeger wrote: > > On 16.05.19 13:12, Thomas Huth wrote: >> The struct kvm_vcpu_events code is only available on certain architectures >> (arm, arm64 and x86). To be able to compile kvm_util.c also for other >> architectures, we'

Re: [PATCH v2] KVM: selftests: Compile code with warnings enabled

2019-05-17 Thread Thomas Huth
On 17/05/2019 11.41, Vitaly Kuznetsov wrote: > Peter Xu writes: > >> On Fri, May 17, 2019 at 11:04:45AM +0200, Thomas Huth wrote: >>> So far the KVM selftests are compiled without any compiler warnings >>> enabled. That's quite bad, since we miss a lot of possi

[PATCH v2] KVM: selftests: Compile code with warnings enabled

2019-05-17 Thread Thomas Huth
variables). Signed-off-by: Thomas Huth --- v2: - Rebased to kvm/queue - Fix warnings in state_test.c and evmcs_test.c, too tools/testing/selftests/kvm/Makefile | 4 +++- tools/testing/selftests/kvm/dirty_log_test.c | 6 +- tools/testing/selftest

Re: [PATCH] KVM: selftests: Compile code with warnings enabled

2019-05-17 Thread Thomas Huth
On 17/05/2019 04.45, Peter Xu wrote: > Hi, Thomas, > > On Thu, May 16, 2019 at 03:02:57PM +0200, Thomas Huth wrote: >> So far the KVM selftests are compiled without any compiler warnings >> enabled. That's quite bad, since we miss a lot of possible bugs this >> w

[PATCH] KVM: selftests: Compile code with warnings enabled

2019-05-16 Thread Thomas Huth
So far the KVM selftests are compiled without any compiler warnings enabled. That's quite bad, since we miss a lot of possible bugs this way. Let's enable at least "-Wall" and some other useful warning flags now. Signed-off-by: Thomas Huth --- This patch fixes most of the

  1   2   >