Re: [PATCH] xprtrdma: removed unnecessary headers from verbs.c

From: Randy Dunlap
Date: Tue Dec 26 2023 - 19:11:16 EST




On 12/26/23 16:04, Tanzir Hasan wrote:
> Hi Randy,
>
>> Where can I find your current working list of what/how to #include?
> Here is my current working list of what to #include.
>
> #include <linux/bitops.h>
> #include <linux/interrupt.h>
> #include <linux/slab.h>
> #include <linux/sunrpc/addr.h>
> #include <linux/sunrpc/svc_rdma.h>
> #include <linux/log2.h>
>
> #include <asm/barrier.h>
>
> #include <rdma/ib_cm.h>
>
> #include "xprt_rdma.h"
> #include <trace/events/rpcrdma.h>
>
> There was a discussion here about when to include asm/asm-generics:
> https://lore.kernel.org/llvm/20231215210344.GA3096493@ZenIV/
>
> If I misunderstood your question please let me know.

Yes, more the latter link for general info rather than the specific
info for this one source file.

Thanks.

--
#Randy