[PATCH v2] MAINTAINERS: adjust file entries after crypto vmx file movement

2024-02-08 Thread Lukas Bulwahn
Commit 109303336a0c ("crypto: vmx - Move to arch/powerpc/crypto") moves the crypto vmx files to arch/powerpc, but misses to adjust the file entries for IBM Power VMX Cryptographic instructions and LINUX FOR POWERPC. Hence, ./scripts/get_maintainer.pl --self-test=patterns complains about broken ref

Re: [PATCH] MAINTAINERS: adjust file entries after crypto vmx file movement

2024-02-08 Thread Lukas Bulwahn
On Thu, Feb 1, 2024 at 6:51 AM Herbert Xu wrote: > > On Mon, Jan 29, 2024 at 02:17:29PM +0100, Lukas Bulwahn wrote: > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index 2fb944964be5..15bc79e80e28 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -10307,12 +10307,12 @@ M: Nayna Jain >

Re: [PATCH v4 5/5] sched: rename SD_SHARE_PKG_RESOURCES to SD_SHARE_LLC

2024-02-08 Thread Valentin Schneider
On 07/02/24 11:58, al...@kernel.org wrote: > From: Alex Shi > > SD_CLUSTER shares the CPU resources like llc tags or l2 cache, that's > easy confuse with SD_SHARE_PKG_RESOURCES. So let's specifical point > what the latter shares: LLC. That would reduce some confusing. > > Suggested-by: Valentin Sc

Re: [PATCH v2 1/4] PCI/AER: Store more information in aer_err_info

2024-02-08 Thread Wang, Qingshun
On Tue, Feb 06, 2024 at 11:23:35AM -0600, Bjorn Helgaas wrote: > On Wed, Feb 07, 2024 at 12:41:41AM +0800, Wang, Qingshun wrote: > > On Mon, Feb 05, 2024 at 05:12:31PM -0600, Bjorn Helgaas wrote: > > > On Thu, Jan 25, 2024 at 02:27:59PM +0800, Wang, Qingshun wrote: > > > > When Advisory Non-Fatal e

Re: [PATCH v5 00/25] Transparent Contiguous PTEs for User Mappings

2024-02-08 Thread Mark Rutland
On Fri, Feb 02, 2024 at 08:07:31AM +, Ryan Roberts wrote: > Hi All, Hi Ryan, I assume this is the same as your 'features/granule_perf/contpte-lkml_v' branch on https://gitlab.arm.com/linux-arm/linux-rr/ I've taken a quick look, and I have a few initial/superficial comments before digging int

[PATCH v2 2/5] sched/vtime: get rid of generic vtime_task_switch() implementation

2024-02-08 Thread Alexander Gordeev
The generic vtime_task_switch() implementation gets built only if __ARCH_HAS_VTIME_TASK_SWITCH is not defined, but requires an architecture to implement arch_vtime_task_switch() callback at the same time, which is confusing. Further, arch_vtime_task_switch() is implemented for 32-bit PowerPC archi

[PATCH v2 4/5] s390/irq,nmi: include header directly

2024-02-08 Thread Alexander Gordeev
update_timer_sys() and update_timer_mcck() are inlines used for CPU time accounting from the interrupt and machine-check handlers. These routines are specific to s390 architecture, but included via header implicitly. Avoid the extra loop and include header directly. Acked-by: Heiko Carstens Sig

[PATCH v2 0/5] sched/vtime: vtime.h headers cleanup

2024-02-08 Thread Alexander Gordeev
Hi All, I kept all tags on reveiwed patches. v2: - patch 4: commit message reworded (Heiko) - patch 5: vtime.h is removed from Kbuild scripts (PowerPC only) (Heiko) v1: Please find a small cleanup to vtime_task_switch() wiring. I split it into smaller patches to allow separate PowerPC vs s390

[PATCH v2 3/5] s390/vtime: remove unused __ARCH_HAS_VTIME_TASK_SWITCH leftover

2024-02-08 Thread Alexander Gordeev
__ARCH_HAS_VTIME_TASK_SWITCH macro is not used anymore. Reviewed-by: Frederic Weisbecker Acked-by: Heiko Carstens Signed-off-by: Alexander Gordeev --- arch/s390/include/asm/vtime.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/s390/include/asm/vtime.h b/arch/s390/include/asm/vtime.

[PATCH v2 5/5] sched/vtime: do not include header

2024-02-08 Thread Alexander Gordeev
There is no architecture-specific code or data left that generic needs to know about. Thus, avoid the inclusion of header. Reviewed-by: Frederic Weisbecker Signed-off-by: Alexander Gordeev --- arch/powerpc/include/asm/Kbuild | 1 - include/asm-generic/vtime.h | 1 - include/linux/vtime.h

[PATCH v2 1/5] sched/vtime: remove confusing arch_vtime_task_switch() declaration

2024-02-08 Thread Alexander Gordeev
Callback arch_vtime_task_switch() is only defined when CONFIG_VIRT_CPU_ACCOUNTING_NATIVE is selected. Yet, the function prototype forward declaration is present for CONFIG_VIRT_CPU_ACCOUNTING_GEN variant. Remove it. Reviewed-by: Frederic Weisbecker Signed-off-by: Alexander Gordeev --- include/l

Re: [revert commit 83b3836bf83f09beea5f592b126cfdd1bc921e48] [mainline] [6.8.0-rc3]kernel BUG at arch/powerpc/platforms/pseries/iommu.c:100!

2024-02-08 Thread Venkat Rao Bagalkote
Adding more people On 08/02/24 8:26 am, Venkat Rao Bagalkote wrote: Resending as earlier one bounced. Greetings!!! [revert commit 83b3836bf83f09beea5f592b126cfdd1bc921e48] [mainline] [6.8.0-rc3]kernel BUG at arch/powerpc/platforms/pseries/iommu.c:100! By Reverting below commit id, issue

Re: [PATCH v2 2/4] eventfd: simplify eventfd_signal()

2024-02-08 Thread Christian Brauner
On Wed, Feb 07, 2024 at 03:34:59PM +0100, Paolo Bonzini wrote: > On Wed, Nov 22, 2023 at 1:49 PM Christian Brauner wrote: > > > > Ever since the evenfd type was introduced back in 2007 in commit > > e1ad7468c77d ("signal/timer/event: eventfd core") the eventfd_signal() > > function only ever passe

Re: [PATCH v4 5/5] sched: rename SD_SHARE_PKG_RESOURCES to SD_SHARE_LLC

2024-02-08 Thread Ricardo Neri
On Wed, Feb 07, 2024 at 11:58:40AM +0800, al...@kernel.org wrote: > From: Alex Shi > > SD_CLUSTER shares the CPU resources like llc tags or l2 cache, that's > easy confuse with SD_SHARE_PKG_RESOURCES. So let's specifical point > what the latter shares: LLC. That would reduce some confusing. > >

Re: [PATCH v2 1/5] sched/vtime: remove confusing arch_vtime_task_switch() declaration

2024-02-08 Thread Nicholas Piggin
On Fri Feb 9, 2024 at 6:15 AM AEST, Alexander Gordeev wrote: > Callback arch_vtime_task_switch() is only defined when > CONFIG_VIRT_CPU_ACCOUNTING_NATIVE is selected. Yet, the > function prototype forward declaration is present for > CONFIG_VIRT_CPU_ACCOUNTING_GEN variant. Remove it. > And powerpc

Re: [PATCH v2 2/5] sched/vtime: get rid of generic vtime_task_switch() implementation

2024-02-08 Thread Nicholas Piggin
On Fri Feb 9, 2024 at 6:15 AM AEST, Alexander Gordeev wrote: > The generic vtime_task_switch() implementation gets built only > if __ARCH_HAS_VTIME_TASK_SWITCH is not defined, but requires an > architecture to implement arch_vtime_task_switch() callback at > the same time, which is confusing. > > F

Re: [PATCH v2 3/5] s390/vtime: remove unused __ARCH_HAS_VTIME_TASK_SWITCH leftover

2024-02-08 Thread Nicholas Piggin
On Fri Feb 9, 2024 at 6:15 AM AEST, Alexander Gordeev wrote: > __ARCH_HAS_VTIME_TASK_SWITCH macro is not used anymore. ... but for benefit of patchwork if you decide to keep them apart Acked-by: Nicholas Piggin > > Reviewed-by: Frederic Weisbecker > Acked-by: Heiko Carstens > Signed-off-by: A

Re: [PATCH v2 5/5] sched/vtime: do not include header

2024-02-08 Thread Nicholas Piggin
On Fri Feb 9, 2024 at 6:15 AM AEST, Alexander Gordeev wrote: > There is no architecture-specific code or data left > that generic needs to know about. > Thus, avoid the inclusion of header. Nice cleanup! Acked-by: Nicholas Piggin > > Reviewed-by: Frederic Weisbecker > Signed-off-by: Alexande

[PATCH v3 1/2] powerpc: Add Power11 architected and raw mode

2024-02-08 Thread Madhavan Srinivasan
reg.h is updated with Power11 pvr. pvr_mask value of 0x0F07 means we are arch v3.1 compliant. This is used by phyp and kvm when booting as a pseries guest to detect and enable the appropriate hwcap, facility bits and PMU related fields. Copied most of fields from Power10 table entry. Signed-of

[PATCH v3 2/2] powerpc/perf: Power11 Performance Monitoring support

2024-02-08 Thread Madhavan Srinivasan
Base enablement patch to register performance monitoring hardware support for Power11. Most of fields are copied from power10_pmu struct for power11_pmu struct. Signed-off-by: Madhavan Srinivasan --- Changelog V2: - No change Changelog v1: - Copied power10 struct for power11 with name change a

Re: [kvm-unit-tests PATCH v2 2/9] arch-run: Clean up temporary files properly

2024-02-08 Thread Nicholas Piggin
On Wed Feb 7, 2024 at 5:58 PM AEST, Thomas Huth wrote: > On 02/02/2024 07.57, Nicholas Piggin wrote: > > Migration files weren't being removed when tests were interrupted. > > This improves the situation. > > > > Signed-off-by: Nicholas Piggin > > --- > > scripts/arch-run.bash | 12 +++-

Re: [PATCH v2] drivers/ps3: select VIDEO to provide cmdline functions

2024-02-08 Thread Michael Ellerman
Thomas Zimmermann writes: > Am 07.02.24 um 17:13 schrieb Randy Dunlap: >> When VIDEO is not set, there is a build error. Fix that by selecting >> VIDEO for PS3_PS3AV. >> >> ERROR: modpost: ".video_get_options" [drivers/ps3/ps3av_mod.ko] undefined! >> >> Fixes: dae7fbf43fd0 ("driver/ps3: Include f

[PATCH] mm/hugetlb: Move page order check inside hugetlb_cma_reserve()

2024-02-08 Thread Anshuman Khandual
All platforms could benefit from page order check against MAX_PAGE_ORDER before allocating a CMA area for gigantic hugetlb pages. Let's move this check from individual platforms to generic hugetlb. Cc: Catalin Marinas Cc: Will Deacon Cc: Michael Ellerman Cc: Nicholas Piggin Cc: linux-arm-ker..

[kvm-unit-tests PATCH v3 0/8] Multi-migration support

2024-02-08 Thread Nicholas Piggin
Since v2: - Rebase on riscv port and auxvinfo fix was merged. - Clean up initrd cleanup moves more commands into the new cleanup function from the trap handler comands (suggested by Thomas). - "arch-run: Clean up temporary files properly" patch is now renamed to "arch-run: Fix TRAP handler..."

[kvm-unit-tests PATCH v3 1/8] arch-run: Fix TRAP handler recursion to remove temporary files properly

2024-02-08 Thread Nicholas Piggin
Migration files were not being removed when the QEMU process is interrupted (e.g., with ^C). This is becaus the SIGINT propagates to the bash TRAP handler, which recursively TRAPs due to the 'kill 0' in the handler. This eventually crashes bash. This can be observed by interrupting a long-running

[kvm-unit-tests PATCH v3 2/8] arch-run: Clean up initrd cleanup

2024-02-08 Thread Nicholas Piggin
Rather than put a big script into the trap handler, have it call a function. Signed-off-by: Nicholas Piggin --- scripts/arch-run.bash | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/scripts/arch-run.bash b/scripts/arch-run.bash index 11d47a85..1e903e83 100644 --

[kvm-unit-tests PATCH v3 3/8] migration: use a more robust way to wait for background job

2024-02-08 Thread Nicholas Piggin
Starting a pipeline of jobs in the background does not seem to have a simple way to reliably find the pid of a particular process in the pipeline (because not all processes are started when the shell continues to execute). The way PID of QEMU is derived can result in a failure waiting on a PID tha

[kvm-unit-tests PATCH v3 4/8] migration: Support multiple migrations

2024-02-08 Thread Nicholas Piggin
Support multiple migrations by flipping dest file/socket variables to source after the migration is complete, ready to start again. A new destination is created if the test outputs the migrate line again. Test cases may now switch to calling migrate() one or more times. Signed-off-by: Nicholas Pig

[kvm-unit-tests PATCH v3 5/8] arch-run: rename migration variables

2024-02-08 Thread Nicholas Piggin
Using 1 and 2 for source and destination is confusing, particularly now with multiple migrations that flip between them. Do a rename pass to tidy things up. Signed-off-by: Nicholas Piggin --- scripts/arch-run.bash | 115 +- 1 file changed, 58 insertions(+)

[kvm-unit-tests PATCH v3 6/8] migration: Add quiet migration support

2024-02-08 Thread Nicholas Piggin
Console output required to support migration becomes quite noisy when doing lots of migrations. Provide a migrate_quiet() call that suppresses console output and doesn't log a message. Signed-off-by: Nicholas Piggin --- lib/migrate.c | 12 lib/migrate.h | 1 + scrip

[kvm-unit-tests PATCH v3 7/8] Add common/ directory for architecture-independent tests

2024-02-08 Thread Nicholas Piggin
x86/sieve.c is used by s390x, arm, and riscv via symbolic link. Make a new directory common/ for architecture-independent tests and move sieve.c here. Signed-off-by: Nicholas Piggin --- arm/sieve.c| 2 +- common/sieve.c | 51 + riscv/sieve.c

[kvm-unit-tests PATCH v3 8/8] migration: add a migration selftest

2024-02-08 Thread Nicholas Piggin
Add a selftest for migration support in guest library and test harness code. It performs migrations a tight loop to irritate races and bugs in the test harness code. Include the test in arm, s390, powerpc. Acked-by: Claudio Imbrenda (s390x) Signed-off-by: Nicholas Piggin --- This has flushed o

Re: [kvm-unit-tests PATCH v3 1/8] arch-run: Fix TRAP handler recursion to remove temporary files properly

2024-02-08 Thread Thomas Huth
On 09/02/2024 08.01, Nicholas Piggin wrote: Migration files were not being removed when the QEMU process is interrupted (e.g., with ^C). This is becaus the SIGINT propagates to the bash TRAP handler, which recursively TRAPs due to the 'kill 0' in the handler. This eventually crashes bash. This c

Re: [kvm-unit-tests PATCH v3 2/8] arch-run: Clean up initrd cleanup

2024-02-08 Thread Thomas Huth
On 09/02/2024 08.01, Nicholas Piggin wrote: Rather than put a big script into the trap handler, have it call a function. Signed-off-by: Nicholas Piggin --- scripts/arch-run.bash | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/scripts/arch-run.bash b/scripts/

Re: [kvm-unit-tests PATCH v3 3/8] migration: use a more robust way to wait for background job

2024-02-08 Thread Thomas Huth
On 09/02/2024 08.01, Nicholas Piggin wrote: Starting a pipeline of jobs in the background does not seem to have a simple way to reliably find the pid of a particular process in the pipeline (because not all processes are started when the shell continues to execute). The way PID of QEMU is derive