On Fri, May 16, 2025 at 12:37 PM Christoph Hellwig <h...@infradead.org> wrote: > > On Thu, May 15, 2025 at 02:16:41PM +0800, Jason Xing wrote: > > +extern size_t relay_dump(struct rchan *chan, int flags); > > Please don't add pointless externs for function prototypes.
Do you mean make it inline in include/linux/relay.h like how relay_write() works? Will do that. > > > +EXPORT_SYMBOL_GPL(relay_dump); > > This export seems unused even with the entire series applied. My initial thought was to provide a symbol for some kernel modules to use. Thanks, Jason
