linux-kernel - RE: Re: [PATCH] RDMA/siw: Solve the error of compiling the 32BIT mips kernel when enable CONFIG_RDMA

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <SA0PR15MB3919C9E8260E7DF5FB36A9D9994E9@SA0PR15MB3919.namprd15.prod.outlook.com>
Date:   Thu, 22 Sep 2022 17:17:24 +0000
From:   Bernard Metzler <BMT@...ich.ibm.com>
To:     Jason Gunthorpe <jgg@...dia.com>,
        jianghaoran <jianghaoran@...inos.cn>
CC:     "leon@...nel.org" <leon@...nel.org>,
        "linux-rdma@...r.kernel.org" <linux-rdma@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Linus Walleij <linus.walleij@...aro.org>
Subject: RE: Re: [PATCH] RDMA/siw: Solve the error of compiling the 32BIT mips
 kernel when enable CONFIG_RDMA_SIW



> -----Original Message-----
> From: Jason Gunthorpe <jgg@...dia.com>
> Sent: Thursday, 22 September 2022 19:03
> To: jianghaoran <jianghaoran@...inos.cn>
> Cc: Bernard Metzler <BMT@...ich.ibm.com>; leon@...nel.org; linux-
> rdma@...r.kernel.org; linux-kernel@...r.kernel.org
> Subject: [EXTERNAL] Re: [PATCH] RDMA/siw: Solve the error of compiling the
> 32BIT mips kernel when enable CONFIG_RDMA_SIW
> 
> On Thu, Sep 01, 2022 at 01:51:38PM +0800, jianghaoran wrote:
> > cross-compilation environment：
> > ubuntu 20.04
> > mips-linux-gnu-gcc (Ubuntu 10.3.0-1ubuntu1~20.04) 10.3.0
> >
> > generate a configuration file by make randconfig:
> > CONFIG_32BIT=y
> > CONFIG_RDMA_SIW=y
> >
> > the error message as follows：
> > In file included from ../arch/mips/include/asm/page.h:270,
> >                  from ../arch/mips/include/asm/io.h:29,
> >                  from ../arch/mips/include/asm/mmiowb.h:5,
> >                  from ../include/linux/spinlock.h:64,
> >                  from ../include/linux/wait.h:9,
> >                  from ../include/linux/net.h:19,
> >                  from ../drivers/infiniband/sw/siw/siw_qp_tx.c:8:
> > ../drivers/infiniband/sw/siw/siw_qp_tx.c: In function ‘siw_tx_hdt’:
> > ../arch/mips/include/asm/page.h:255:53: error: cast to pointer from
> integer of different size [-Werror=int-to-pointer-cast]
> >   255 | #define virt_to_pfn(kaddr)    PFN_DOWN(virt_to_phys((void
> *)(kaddr)))
> >       |                                                     ^
> > ../include/asm-generic/memory_model.h:18:41: note: in definition of macro
> ‘__pfn_to_page’
> >    18 | #define __pfn_to_page(pfn) (mem_map + ((pfn) - ARCH_PFN_OFFSET))
> >       |                                         ^~~
> > ../arch/mips/include/asm/page.h:255:31: note: in expansion of macro
> ‘PFN_DOWN’
> >   255 | #define virt_to_pfn(kaddr)    PFN_DOWN(virt_to_phys((void
> *)(kaddr)))
> >       |                               ^~~~~~~~
> > ../arch/mips/include/asm/page.h:256:41: note: in expansion of macro
> ‘virt_to_pfn’
> >   256 | #define virt_to_page(kaddr) pfn_to_page(virt_to_pfn(kaddr))
> >       |                                         ^~~~~~~~~~~
> > ../drivers/infiniband/sw/siw/siw_qp_tx.c:538:23: note: in expansion of
> macro ‘virt_to_page’
> >   538 |     page_array[seg] = virt_to_page(va & PAGE_MASK);
> >       |                       ^~~~~~~~~~~~
> > cc1: all warnings being treated as errors
> > make[5]: *** [../scripts/Makefile.build:249:
> drivers/infiniband/sw/siw/siw_qp_tx.o] Error 1
> > make[4]: *** [../scripts/Makefile.build:465: drivers/infiniband/sw/siw]
> Error 2
> > make[3]: *** [../scripts/Makefile.build:465: drivers/infiniband/sw] Error
> 2
> > make[3]: *** Waiting for unfinished jobs....
> >
> > Reported-by: k2ci <kernel-bot@...inos.cn>
> > Signed-off-by: jianghaoran <jianghaoran@...inos.cn>
> > ---
> >  drivers/infiniband/sw/siw/siw_qp_tx.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/infiniband/sw/siw/siw_qp_tx.c
> b/drivers/infiniband/sw/siw/siw_qp_tx.c
> > index 1f4e60257700..55ed0c27f449 100644
> > --- a/drivers/infiniband/sw/siw/siw_qp_tx.c
> > +++ b/drivers/infiniband/sw/siw/siw_qp_tx.c
> > @@ -533,7 +533,7 @@ static int siw_tx_hdt(struct siw_iwarp_tx *c_tx,
> struct socket *s)
> >  					kunmap_local(kaddr);
> >  				}
> >  			} else {
> > -				u64 va = sge->laddr + sge_off;
> > +				unsigned long va = sge->laddr + sge_off;
> 
> The compiler is saying it should be a void * not an unsigned long.
> 

Linus' [Patch v3] was moving it to uintptr_t  which I think is
the right solution. We went for that afaik.