On Mon, Sep 1, 2025 at 4:25 PM Lorenzo Stoakes <lorenzo.stoakes@xxxxxxxxxx> wrote: > 1. (most useful) Const pointer (const <type> *<param>) means that the dereffed > value is const, so *<param> = <val> or <param>-><field> = <val> are prohibited. Only this was what my initial patch was about. > 2. (less useful) We can't modify the actual pointer value either, so > e.g. <param> = <new param> is prohibited. This wasn't my idea, it was Andrew Morton's idea, supported by Yuanchu Xie: https://lore.kernel.org/lkml/CAJj2-QHVC0QW_4X95LLAnM=1g6apH==-OXZu65SVeBj0tSUcBg@xxxxxxxxxxxxxx/ You know that because you participated in the discussion. In that thread, nobody objected, so I took the time and adjusted all of my patches. There is some value, but of course it's very small. Note that I added the value-level "const" only to the implementation, never to prototypes, because it would have no effect there.
