# Re: \[PATCH net-next 01/17\] net: Copy slab data for sendmsg(MSG\_SPLICE\_PAGES)

- From: Simon Horman \<hidden\>
- Date: 2023-06-17 13:52:21 
- List: netdev
- Page: https://ratatoskr.run/netdev/2023/06/2237497
- Thread: https://ratatoskr.run/netdev/2023/06/2236827/t.md

```text
On Sat, Jun 17, 2023 at 07:43:15AM +0100, David Howells wrote:
> Simon Horman [ref] wrote:
> 
> > > +	cache = get_cpu_ptr(&skb_splice_frag_cache);
> ...
> > > +	put_cpu_ptr(skb_splice_frag_cache);
> > 
> > Hi David,
> > 
> > I don't think it makes any difference at run-time.
> > But to keep Sparse happy, perhaps this ought to be put_cpu_var()
> 
> Actually, the problem is a missing "&".  I think I should use put_cpu_ptr() to
> match get_cpu_ptr().  It doesn't crash because the argument is ignored.

Thanks David, I agree that is a better idea.
```