Re: [PATCH 1/7] vsprintf: Add %pTN to print task name

2024-12-13 Thread Andy Shevchenko
On Fri, Dec 13, 2024 at 02:27:09PM +0100, Borislav Petkov wrote: > On Fri, Dec 13, 2024 at 10:35:03AM +0200, Kalle Valo wrote: > > I agree, it makes the code harder to read for someone who is not > > familiar with all the %p magic we have (like me). > +1 And me too. In case one thinks of unprinta

Re: [PATCH 1/7] vsprintf: Add %pTN to print task name

2024-12-13 Thread Borislav Petkov
On Fri, Dec 13, 2024 at 10:35:03AM +0200, Kalle Valo wrote: > I agree, it makes the code harder to read for someone who is not > familiar with all the %p magic we have (like me). +1 -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette _

Re: [PATCH 1/7] vsprintf: Add %pTN to print task name

2024-12-13 Thread Yafang Shao
On Fri, Dec 13, 2024 at 4:05 PM Petr Mladek wrote: > > On Fri 2024-12-13 13:46:04, Yafang Shao wrote: > > Since the task->comm is guaranteed to be NUL-ternimated, we can print it > > directly. Add a new vsnprintf format specifier "%pTN" to print task comm, > > where 'p' represents the task Pointer

Re: [PATCH 1/7] vsprintf: Add %pTN to print task name

2024-12-13 Thread Kalle Valo
Petr Mladek writes: > On Fri 2024-12-13 13:46:04, Yafang Shao wrote: >> Since the task->comm is guaranteed to be NUL-ternimated, we can print it >> directly. Add a new vsnprintf format specifier "%pTN" to print task comm, >> where 'p' represents the task Pointer, 'T' stands for Task, and 'N' deno

Re: [PATCH 1/7] vsprintf: Add %pTN to print task name

2024-12-13 Thread Petr Mladek
On Fri 2024-12-13 13:46:04, Yafang Shao wrote: > Since the task->comm is guaranteed to be NUL-ternimated, we can print it > directly. Add a new vsnprintf format specifier "%pTN" to print task comm, > where 'p' represents the task Pointer, 'T' stands for Task, and 'N' denots > Name. With this abstra

[PATCH 1/7] vsprintf: Add %pTN to print task name

2024-12-12 Thread Yafang Shao
Since the task->comm is guaranteed to be NUL-ternimated, we can print it directly. Add a new vsnprintf format specifier "%pTN" to print task comm, where 'p' represents the task Pointer, 'T' stands for Task, and 'N' denots Name. With this abstraction, the user no longer needs to care about retrievin