Re: [PATCH v4 2/2] nfsd: Implement large extent array support in pNFS

From: Christoph Hellwig
Date: Mon Jun 23 2025 - 03:41:27 EST


On Sat, Jun 21, 2025 at 07:52:45PM +0300, Sergey Bashirov wrote:
> + memcpy(&rqstp->rq_arg, &lcp->lc_up_layout, sizeof(struct xdr_buf));

Nit: using a struct assignment like:

rqstp->rq_arg = lcp->lc_up_layout;

might be a tad bettet due to type safety here.

> + memcpy(&rqstp->rq_arg, &lcp->lc_up_layout, sizeof(struct xdr_buf));

Same here.

Otherwise this looks good to me (but I'm not the best reviwer for XDR
code :)):

Reviewed-by: Christoph Hellwig <hch@xxxxxx>