Re: [PATCH 1/1] kexec: Use %pe format specifier for error pointer printing

["Yanjun.Zhu" <yanjun.zhu@linux.dev>]

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


On 10/31/25 1:18 PM, Pasha Tatashin wrote:
> Sure. Will include it

Thanks a lot. Appreciate your helps.

Yanjun.Zhu


>
> On Fri, Oct 31, 2025, 4:09 PM Yanjun.Zhu <yanjun.zhu@linux.dev> wrote:
>
>     Hi, Pasha
>
>     I noticed that you are moving |kexec_handover.c| from the
>     |kernel/| directory to |kernel/liveupdate/|.
>
>     Several developers suggested that I resend my patch after your
>     patch series is merged upstream.
>     Therefore, I’d like to ask if you could include my patch in your
>     series. This would help avoid extra work on my side to resend the
>     commit later.
>
>     Thanks,
>
>     Yanjun.Zhu
>
>     On 10/21/25 2:41 AM, Simon Horman wrote:
>>     On Thu, Oct 16, 2025 at 08:03:20PM +0000, Zhu Yanjun wrote:
>>>     Make pr_xxx() call to use the %pe format specifier instead of %d.
>>>     The %pe specifier prints a symbolic error string (e.g., -ENOMEM,
>>>     -EINVAL) when given an error pointer created with ERR_PTR(err).
>>>
>>>     This change enhances the clarity and diagnostic value of the error
>>>     message by showing a descriptive error name rather than a numeric
>>>     error code.
>>>
>>>     Signed-off-by: Zhu Yanjun<yanjun.zhu@linux.dev> <mailto:yanjun.zhu@linux.dev>
>>     Thanks,
>>
>>     This looks good to me, even though I note that this file
>>     has moved in linux-next.
>>
>>     Reviewed-by: Simon Horman<horms@kernel.org> <mailto:horms@kernel.org>
>

[-- Attachment #2: Type: text/html, Size: 2915 bytes --]