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 <[email protected]> > --- > 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 <[email protected]>
