On Mon, Jul 24, 2023 at 10:58:50AM +0100, David Howells wrote: > Chuck Lever <cel@xxxxxxxxxx> wrote: > > > + buf = page_frag_alloc(&svsk->sk_frag_cache, sizeof(marker), > > + GFP_KERNEL); > > Is this SMP-safe? page_frag_alloc() does no locking. Note that svc_tcp_sendto() takes xprt->xpt_mutex. There can be only one thread (per svsk) running in svc_tcp_sendmsg() at a time. -- Chuck Lever
