Hi, On 12/26/23 13:23, Tanzir Hasan wrote: > asm-generic/barrier.h and asm/bitops.h are already brought into the > header and the file can still be built with their removal. Brought into which header? Does this conflict with Rule #1 in Documentation/process/submit-checklist.rst ? > Suggested-by: Al Viro <viro@xxxxxxxxxxxxxxxxxx> > Signed-off-by: Tanzir Hasan <tanzirh@xxxxxxxxxx> > --- > net/sunrpc/xprtrdma/verbs.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/net/sunrpc/xprtrdma/verbs.c b/net/sunrpc/xprtrdma/verbs.c > index 28c0771c4e8c..5436560dda85 100644 > --- a/net/sunrpc/xprtrdma/verbs.c > +++ b/net/sunrpc/xprtrdma/verbs.c > @@ -55,9 +55,6 @@ > #include <linux/sunrpc/svc_rdma.h> > #include <linux/log2.h> > > -#include <asm-generic/barrier.h> > -#include <asm/bitops.h> > - > #include <rdma/ib_cm.h> > > #include "xprt_rdma.h" > > --- > base-commit: fbafc3e621c3f4ded43720fdb1d6ce1728ec664e > change-id: 20231226-verbs-30800631d3f1 > > Best regards, -- #Randy