Re: [PATCH v2 1/4] x86: provide an inverted Kconfig control for shim-exclusive mode

2025-01-17 Thread Andrew Cooper
On 17/01/2025 10:43 pm, Stefano Stabellini wrote: > On Fri, 17 Jan 2025, Jan Beulich wrote: >> On 17.01.2025 13:24, Alejandro Vallejo wrote: >>> On Fri Jan 17, 2025 at 10:31 AM GMT, Roger Pau Monné wrote: On Thu, Jan 16, 2025 at 04:31:46PM -0800, Stefano Stabellini wrote: > On Wed, 1 Mar 2

Re: [PATCH v2 1/4] x86: provide an inverted Kconfig control for shim-exclusive mode

2025-01-17 Thread Stefano Stabellini
On Fri, 17 Jan 2025, Jan Beulich wrote: > On 17.01.2025 13:24, Alejandro Vallejo wrote: > > On Fri Jan 17, 2025 at 10:31 AM GMT, Roger Pau Monné wrote: > >> On Thu, Jan 16, 2025 at 04:31:46PM -0800, Stefano Stabellini wrote: > >>> On Wed, 1 Mar 2023, Jan Beulich wrote: > While we want certain

Re: [XEN PATCH 0/1] Bug: Hyperlinks in generated documentation may point to the wrong architecture

2025-01-17 Thread Maximilian Engelhardt
On Freitag, 10. Januar 2025 22:32:06 CET Andrew Cooper wrote: > [...] > Thanks for the patch. I'll commit it in due course. > > As an aside though, is there anything we could sensibly do in our own CI > (Gitlab) to not regress this? > > https://salsa.debian.org/reproducible-builds/reprotest look

Re: [PATCH v4 25/30] context_tracking,x86: Defer kernel text patching IPIs

2025-01-17 Thread Sean Christopherson
On Fri, Jan 17, 2025, Valentin Schneider wrote: > On 14/01/25 13:13, Sean Christopherson wrote: > > On Tue, Jan 14, 2025, Valentin Schneider wrote: > >> +/** > >> + * is_kernel_noinstr_text - checks if the pointer address is located in > >> the > >> + *.noinstr section > >> + *

Re: [PATCH v4 29/30] x86/mm, mm/vmalloc: Defer flush_tlb_kernel_range() targeting NOHZ_FULL CPUs

2025-01-17 Thread Valentin Schneider
On 17/01/25 17:11, Uladzislau Rezki wrote: > On Fri, Jan 17, 2025 at 04:25:45PM +0100, Valentin Schneider wrote: >> On 14/01/25 19:16, Jann Horn wrote: >> > On Tue, Jan 14, 2025 at 6:51 PM Valentin Schneider >> > wrote: >> >> vunmap()'s issued from housekeeping CPUs are a relatively common source

Re: [PATCH v4 29/30] x86/mm, mm/vmalloc: Defer flush_tlb_kernel_range() targeting NOHZ_FULL CPUs

2025-01-17 Thread Valentin Schneider
On 17/01/25 16:52, Jann Horn wrote: > On Fri, Jan 17, 2025 at 4:25 PM Valentin Schneider > wrote: >> On 14/01/25 19:16, Jann Horn wrote: >> > On Tue, Jan 14, 2025 at 6:51 PM Valentin Schneider >> > wrote: >> >> vunmap()'s issued from housekeeping CPUs are a relatively common source of >> >> int

Re: [PATCH v4 29/30] x86/mm, mm/vmalloc: Defer flush_tlb_kernel_range() targeting NOHZ_FULL CPUs

2025-01-17 Thread Uladzislau Rezki
On Fri, Jan 17, 2025 at 04:25:45PM +0100, Valentin Schneider wrote: > On 14/01/25 19:16, Jann Horn wrote: > > On Tue, Jan 14, 2025 at 6:51 PM Valentin Schneider > > wrote: > >> vunmap()'s issued from housekeeping CPUs are a relatively common source of > >> interference for isolated NOHZ_FULL CPUs

Re: [PATCH v4 29/30] x86/mm, mm/vmalloc: Defer flush_tlb_kernel_range() targeting NOHZ_FULL CPUs

2025-01-17 Thread Jann Horn
On Fri, Jan 17, 2025 at 4:25 PM Valentin Schneider wrote: > On 14/01/25 19:16, Jann Horn wrote: > > On Tue, Jan 14, 2025 at 6:51 PM Valentin Schneider > > wrote: > >> vunmap()'s issued from housekeeping CPUs are a relatively common source of > >> interference for isolated NOHZ_FULL CPUs, as they

Re: [PATCH v4 29/30] x86/mm, mm/vmalloc: Defer flush_tlb_kernel_range() targeting NOHZ_FULL CPUs

2025-01-17 Thread Valentin Schneider
On 14/01/25 19:16, Jann Horn wrote: > On Tue, Jan 14, 2025 at 6:51 PM Valentin Schneider > wrote: >> vunmap()'s issued from housekeeping CPUs are a relatively common source of >> interference for isolated NOHZ_FULL CPUs, as they are hit by the >> flush_tlb_kernel_range() IPIs. >> >> Given that CP

Re: [PATCH v2 02/18] x86/domain: limit window where curr_vcpu != current on context switch

2025-01-17 Thread Roger Pau Monné
On Tue, Jan 14, 2025 at 04:02:01PM +0100, Jan Beulich wrote: > On 09.01.2025 18:33, Roger Pau Monné wrote: > > On Thu, Jan 09, 2025 at 09:59:58AM +0100, Jan Beulich wrote: > >> On 08.01.2025 15:26, Roger Pau Monne wrote: > >>> } > >>> else > >>> { > >>> -__context_switch(); >

Re: [PATCH for-4.20 v2 2/2] automation/cirrus-ci: introduce FreeBSD randconfig builds

2025-01-17 Thread Oleksii Kurochko
On 1/16/25 3:13 PM, Roger Pau Monné wrote: On Thu, Jan 16, 2025 at 02:02:38PM +, Andrew Cooper wrote: On 16/01/2025 1:59 pm, Roger Pau Monne wrote: Add a new randconfig job for each FreeBSD version. This requires some rework of the template so common parts can be shared between the full a

Re: [PATCH v2 00/18] x86: adventures in Address Space Isolation

2025-01-17 Thread Roger Pau Monné
On Tue, Jan 14, 2025 at 05:20:04PM +0100, Jan Beulich wrote: > On 08.01.2025 15:26, Roger Pau Monne wrote: > > Hello, > > > > The aim of this series is to introduce the functionality required to > > create linear mappings visible to a single pCPU. > > > > Doing so requires having a per-vCPU root

Re: [PATCH v4 26/30] x86,tlb: Make __flush_tlb_global() noinstr-compliant

2025-01-17 Thread Valentin Schneider
On 14/01/25 13:45, Dave Hansen wrote: > On 1/14/25 09:51, Valentin Schneider wrote: >> +cr4 = this_cpu_read(cpu_tlbstate.cr4); >> +asm volatile("mov %0,%%cr4": : "r" (cr4 ^ X86_CR4_PGE) : "memory"); >> +asm volatile("mov %0,%%cr4": : "r" (cr4) : "memory"); >> +/* >> + * In lieu

Re: [PATCH v2 1/4] x86: provide an inverted Kconfig control for shim-exclusive mode

2025-01-17 Thread Jan Beulich
On 17.01.2025 13:24, Alejandro Vallejo wrote: > On Fri Jan 17, 2025 at 10:31 AM GMT, Roger Pau Monné wrote: >> On Thu, Jan 16, 2025 at 04:31:46PM -0800, Stefano Stabellini wrote: >>> On Wed, 1 Mar 2023, Jan Beulich wrote: While we want certain things turned off in shim-exclusive mode, doing >>

Re: [PATCH v2 1/4] x86: provide an inverted Kconfig control for shim-exclusive mode

2025-01-17 Thread Alejandro Vallejo
On Fri Jan 17, 2025 at 10:31 AM GMT, Roger Pau Monné wrote: > On Thu, Jan 16, 2025 at 04:31:46PM -0800, Stefano Stabellini wrote: > > On Wed, 1 Mar 2023, Jan Beulich wrote: > > > While we want certain things turned off in shim-exclusive mode, doing > > > so via "depends on !PV_SHIM_EXCLUSIVE" badly

Config space access to Mediatek MT7922 doesn't work after device reset in Xen PV dom0 (regression, Linux 6.12)

2025-01-17 Thread Marek Marczykowski-Górecki
Hi, After updating PV dom0 to Linux 6.12, The Mediatek MT7922 device reports all 0xff when accessing its config space. This happens only after device reset (which is also triggered when binding the device to the xen-pciback driver). Reproducer: # lspci -xs 01:00.0 01:00.0 Network control

Re: Problems in PV dom0 on recent x86 hardware

2025-01-17 Thread Roger Pau Monné
On Tue, Jul 09, 2024 at 09:08:11AM -0400, Jason Andryuk wrote: > On 2024-07-09 06:56, Jürgen Groß wrote: > > On 09.07.24 09:01, Jan Beulich wrote: > > > On 09.07.2024 08:36, Jürgen Groß wrote: > > > > On 09.07.24 08:24, Jan Beulich wrote: > > > > > On 08.07.2024 23:30, Jason Andryuk wrote: > > > >

Re: [PATCH v2 1/4] x86: provide an inverted Kconfig control for shim-exclusive mode

2025-01-17 Thread Roger Pau Monné
On Thu, Jan 16, 2025 at 04:31:46PM -0800, Stefano Stabellini wrote: > On Wed, 1 Mar 2023, Jan Beulich wrote: > > While we want certain things turned off in shim-exclusive mode, doing > > so via "depends on !PV_SHIM_EXCLUSIVE" badly affects allyesconfig: Since > > that will turn on PV_SHIM_EXCLUSIVE

Re: [PATCH v4 25/30] context_tracking,x86: Defer kernel text patching IPIs

2025-01-17 Thread Valentin Schneider
On 14/01/25 13:48, Sean Christopherson wrote: > On Tue, Jan 14, 2025, Sean Christopherson wrote: >> On Tue, Jan 14, 2025, Valentin Schneider wrote: >> > +/** >> > + * is_kernel_noinstr_text - checks if the pointer address is located in >> > the >> > + *.noinstr section >> > + *

Re: [PATCH v4 18/30] x86/kvm/vmx: Mark vmx_l1d_should flush and vmx_l1d_flush_cond keys as allowed in .noinstr

2025-01-17 Thread Valentin Schneider
On 14/01/25 13:19, Sean Christopherson wrote: > Please use "KVM: VMX:" for the scope. > > On Tue, Jan 14, 2025, Valentin Schneider wrote: >> Later commits will cause objtool to warn about static keys being used in >> .noinstr sections in order to safely defer instruction patching IPIs >> targeted a

Re: [PATCH v4 25/30] context_tracking,x86: Defer kernel text patching IPIs

2025-01-17 Thread Valentin Schneider
On 14/01/25 13:13, Sean Christopherson wrote: > On Tue, Jan 14, 2025, Valentin Schneider wrote: >> text_poke_bp_batch() sends IPIs to all online CPUs to synchronize >> them vs the newly patched instruction. CPUs that are executing in userspace >> do not need this synchronization to happen immediate