Re: [PATCH v3] page_alloc: assert IRQs are enabled in heap alloc/free

2022-04-25 Thread Jan Beulich
On 24.04.2022 17:52, Julien Grall wrote: >> Changes in v3: >> - Use num_online_cpus() in assert. With this ... >> --- a/xen/common/page_alloc.c >> +++ b/xen/common/page_alloc.c >> @@ -162,6 +162,13 @@ >> static char __initdata opt_badpage[100] = ""; >> string_param("badpage", opt_badpage); >>

Re: [PATCH v3] page_alloc: assert IRQs are enabled in heap alloc/free

2022-04-24 Thread Julien Grall
Hi David, On 22/04/2022 16:36, David Vrabel wrote: From: David Vrabel Heap pages can only be safely allocated and freed with interuupts typo: s/interuupts/interrupts/ enabled as they may require a TLB flush which will send IPIs. We don't have such requirements on Arm. Given this is commo

[PATCH v3] page_alloc: assert IRQs are enabled in heap alloc/free

2022-04-22 Thread David Vrabel
From: David Vrabel Heap pages can only be safely allocated and freed with interuupts enabled as they may require a TLB flush which will send IPIs. Normally spinlock debugging would catch calls from the incorrect context, but not from stop_machine_run() action functions as these are called with s