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