On Sat, Mar 23, 2024 at 4:32 PM Yanjun.Zhu <yanjun.zhu@xxxxxxxxx> wrote: > > From: Zhu Yanjun <yanjun.zhu@xxxxxxxxx> > > If the definition of pr_fmt is before the header file. The pr_fmt > will be overwritten by the header file. So move the definition of > pr_fmt to the below of the header file. Awesome. You fix my problem. > > Signed-off-by: Zhu Yanjun <yanjun.zhu@xxxxxxxxx> > --- > drivers/infiniband/sw/rxe/rxe.h | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/infiniband/sw/rxe/rxe.h b/drivers/infiniband/sw/rxe/rxe.h > index d8fb2c7af30a..dc2d8dd2f681 100644 > --- a/drivers/infiniband/sw/rxe/rxe.h > +++ b/drivers/infiniband/sw/rxe/rxe.h > @@ -7,11 +7,6 @@ > #ifndef RXE_H > #define RXE_H > > -#ifdef pr_fmt > -#undef pr_fmt > -#endif > -#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt > - > #include <linux/skbuff.h> > > #include <rdma/ib_verbs.h> > @@ -30,6 +25,11 @@ > #include "rxe_verbs.h" > #include "rxe_loc.h" > > +#ifdef pr_fmt > +#undef pr_fmt > +#endif > +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt > + > /* > * Version 1 and Version 2 are identical on 64 bit machines, but on 32 bit > * machines Version 2 has a different struct layout. > -- > 2.34.1 > >
