Re: [PATCH v4 01/23] lib/scatterlist: cleanup macros into static inline functions

From: Christoph Hellwig
Date: Tue Dec 21 2021 - 04:00:14 EST


On Wed, Nov 17, 2021 at 02:53:48PM -0700, Logan Gunthorpe wrote:
> Convert the sg_is_chain(), sg_is_last() and sg_chain_ptr() macros
> into static inline functions. There's no reason for these to be macros
> and static inline are generally preferred these days.
>
> Also introduce the SG_PAGE_LINK_MASK define so the P2PDMA work, which is
> adding another bit to this mask, can do so more easily.
>
> Suggested-by: Jason Gunthorpe <jgg@xxxxxxxxxx>
> Signed-off-by: Logan Gunthorpe <logang@xxxxxxxxxxxx>

Looks fine:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

scatterlist.h doesn't have a real maintainer, do you want me to pick
this up through the DMA tree?