Re: [PATCH v5 11/12] mm: constify assert/test functions in mm.h
From: Max Kellermann
Date: Mon Sep 01 2025 - 11:37:40 EST
On Mon, Sep 1, 2025 at 4:07 PM David Hildenbrand <david@xxxxxxxxxx> wrote:
> > -static inline void assert_fault_locked(struct vm_fault *vmf)
> > +static inline void assert_fault_locked(const struct vm_fault *vmf)
> > {
>
> This confused me a bit: in the upper variant it's "*const" and here it's
> "const *".
That was indeed a mistake. Both should be "const*const".
> There are multiple such cases here, which might imply that it is not
> "relatively trivial to const-ify them". :)
I double-checked this patch and couldn't find any other such mistake.
Or do you mean the function vs prototype thing on parameter values?