linux-kernel - Re: Build regressions/improvements in v4.16-rc2 (infiniband/hw/bnxt

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-prev] [day] [month] [year] [list]

Date:   Mon, 19 Feb 2018 11:57:43 -0800
From:   Randy Dunlap <rdunlap@...radead.org>
To:     Geert Uytterhoeven <geert@...ux-m68k.org>,
        linux-kernel@...r.kernel.org
Cc:     "linux-rdma@...r.kernel.org" <linux-rdma@...r.kernel.org>,
        Selvin Xavier <selvin.xavier@...adcom.com>,
        Devesh Sharma <devesh.sharma@...adcom.com>,
        Somnath Kotur <somnath.kotur@...adcom.com>,
        Sriharsha Basavapatna <sriharsha.basavapatna@...adcom.com>
Subject: Re: Build regressions/improvements in v4.16-rc2
 (infiniband/hw/bnxt_re)

On 02/19/18 05:57, Geert Uytterhoeven wrote:
> Below is the list of build error/warning regressions/improvements in
> v4.16-rc2[1] compared to v4.15[2].

Hi,
Just FYI, this driver could use some 32-bit attention.

> *** WARNINGS ***
> 
> 1659 warning regressions:
bytes is larger than 1024 bytes [-Wframe-larger-than=]:  => 78:1
>   + /home/kisskb/slave/src/drivers/infiniband/hw/bnxt_re/ib_verbs.c: warning: left shift count >= width of type [enabled by default]:  => 3550:2, 3552:4, 149:2

Above is due to:
#define BNXT_RE_MAX_MR_SIZE_HIGH	BIT(39)
#define BNXT_RE_MAX_MR_SIZE		BNXT_RE_MAX_MR_SIZE_HIGH
as 32-bit CPU cannot shift 39 bits left.

>   + /home/kisskb/slave/src/drivers/infiniband/hw/bnxt_re/qplib_fp.c: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]:  => 569:19
>   + /home/kisskb/slave/src/drivers/infiniband/hw/bnxt_re/qplib_fp.c: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]:  => 336:12, 333:23
>   + /home/kisskb/slave/src/drivers/infiniband/hw/bnxt_re/qplib_fp.c: warning: passing argument 1 of '__fswab64' makes integer from pointer without a cast [enabled by default]:  => 569:2
thanks
-- 
~Randy