linux-kernel - RE: 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: <SA0PR15MB39191553A6C95FB96FAB7146994E9@SA0PR15MB3919.namprd15.prod.outlook.com>
Date:   Thu, 22 Sep 2022 19:02:54 +0000
From:   Bernard Metzler <BMT@...ich.ibm.com>
To:     Jason Gunthorpe <jgg@...dia.com>
CC:     jianghaoran <jianghaoran@...inos.cn>,
        "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: 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:48
> To: Bernard Metzler <BMT@...ich.ibm.com>
> Cc: jianghaoran <jianghaoran@...inos.cn>; leon@...nel.org; linux-
> rdma@...r.kernel.org; linux-kernel@...r.kernel.org; Linus Walleij
> <linus.walleij@...aro.org>
> Subject: [EXTERNAL] Re: Re: [PATCH] RDMA/siw: Solve the error of compiling
> the 32BIT mips kernel when enable CONFIG_RDMA_SIW
> 
> On Thu, Sep 22, 2022 at 05:17:24PM +0000, Bernard Metzler wrote:
> 
> > > 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.
> 
> That isn't what I saw in merged patches:
> 
> INVALID URI REMOVED
> 3A__lore.kernel.org_all_20220913140416.280860971-
> 40linuxfoundation.org_&d=DwIBAg&c=jf_iaSHvJObTbx-siA1ZOg&r=2TaYXQ0T-
> r8ZO1PP1alNwU_QJcRRLfmYTAgd3QCvqSc&m=Rvcp-
> EvvdOCENMF2s1l1j5ky3WQqSiJ0_by31NFDnrQ_iDO6zegGgLPy4OiQmkCO&s=FZOBbAPo0wCJJ
> R0_062_zc6iE6WyRszu3Q9eUwvA6o0&e=
> 

Looks good. It's the right patch from Linus W. which got
merged, as your lore link points at. You seem to have looked
at Jianghaoran's patch for the same issue, which came
in almost same time, but was not taken.

What was applied is Linus'
[PATCH v3] RDMA/siw: Pass a pointer to virt_to_page()

Best,
Bernard.

...
@@ -533,13 +533,23 @@ static int siw_tx_hdt(struct siw_iwarp_tx *c_tx, struct socket *s)
 					kunmap_local(kaddr);
 				}
 			} else {
-				u64 va = sge->laddr + sge_off;
+				/*
+				 * Cast to an uintptr_t to preserve all 64 bits
+				 * in sge->laddr.
+				 */
+				uintptr_t va = (uintptr_t)(sge->laddr + sge_off);
...


> I'm confused??
> 
> Jason