Re: [PATCH] cgroup: Use %p for pointer formatting

["Michal Koutný" <mkoutny@suse.com>]

[-- Attachment #1: Type: text/plain, Size: 1023 bytes --]

On Fri, Sep 18, 2026 at 12:48:52PM +0200, Sebastian Andrzej Siewior <bigeasy@linutronix.de> wrote:
> Since commit ad67b74d2469d ("printk: hash addresses printed with
> %p") pointers are hashed by default and the behaviour can be controller
> by `hash_pointers' boot argument.
> The policy on %p is to not introduce new ones.
                %pK  should be here?

I assume it based on:

| This is because %pK checks are done at read() time rather than open()
| ...
| The correct long-term solution is to do the permission checks at
| open() time.

of Documentation/admin-guide/sysctl/kernel.rst-time


<snip>
> Removing %pK makes it possible to remove its handling from the library.

This is also a good sell.
But I see there are a few of them still here and there.

> 
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> ---
>  kernel/cgroup/debug.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>

Nevertheless,
Acked-by: Michal Koutný <mkoutny@suse.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 265 bytes --]