On Thu, 2026-03-26 at 12:32 +0000, Brendan Jackman wrote:
> %pGg produces nice readable output and decouples the format string from
> the size of gfp_t.
>
> Signed-off-by: Brendan Jackman <jackmanb@google.com>
> ---
> net/rds/tcp_recv.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/net/rds/tcp_recv.c b/net/rds/tcp_recv.c
> index 49f96ee0c40f6..ffe843ca219c7 100644
> --- a/net/rds/tcp_recv.c
> +++ b/net/rds/tcp_recv.c
> @@ -275,7 +275,7 @@ static int rds_tcp_read_sock(struct rds_conn_path *cp, gfp_t gfp)
> desc.count = 1; /* give more than one skb per call */
>
> tcp_read_sock(sock->sk, &desc, rds_tcp_data_recv);
> - rdsdebug("tcp_read_sock for tc %p gfp 0x%x returned %d\n", tc, gfp,
> + rdsdebug("tcp_read_sock for tc %p gfp %pGg returned %d\n", tc, &gfp,
> desc.error);
>
> if (skb_queue_empty_lockless(&sock->sk->sk_receive_queue) &&
>
This looks fine to me. Thanks Brendan!
Acked-by: Allison Henderson <achender@kernel.org>