Re: [PATCH 4/7] maccess: Use a scoped guard to re-enable page faults
From: David Hildenbrand (Arm)
Date: Thu Oct 01 2026 - 07:35:32 EST
On 8/24/26 18:04, Muhammad Usama Anjum wrote:
> Kernel nofault copy and string paths open-code page-fault disable and
> enable around label-based loops, duplicating cleanup on success and
> failure.
>
> Use a page-fault scope guard instead. Leaving the scope now re-enables
> page faults on both paths without separate cleanup at the fault label.
>
> No functional change.
>
> Signed-off-by: Muhammad Usama Anjum <usama.anjum@xxxxxxx>
> ---
Patch itself is fine, but the guard name is completely misleading.
Acked-by: David Hildenbrand (Arm) <david@xxxxxxxxxx>
--
Cheers,
David