On 11/1/24 21:09, Pavel Begunkov wrote:
On 11/1/24 20:06, Mina Almasry wrote: ...+__maybe_unused +static const struct memory_provider_ops io_uring_pp_zc_ops; + +static inline struct io_zcrx_area *io_zcrx_iov_to_area(const struct net_iov *niov) +{ + struct net_iov_area *owner = net_iov_owner(niov); + + return container_of(owner, struct io_zcrx_area, nia); +} +We discussed this before I disappeared on vacation but I'm not too convinced to be honest, sorry.
To expand on this one, a few weeks ago I outlined how you can employ the compiler and verify correctness, and I don't really see a way to convince you unless you're willing to check your claim that it can go wrong. Turning it the other way around, if you see a path where it could go wrong, please do let me know, and it'll certainly get fixed, but until then I don't believe it's anyhow a blocker for the series. -- Pavel Begunkov
