linux-kernel - Re: linux-next: Tree for Feb 15 (CXL on i386)

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, 15 Feb 2021 09:35:38 -0800
From:   Randy Dunlap <rdunlap@...radead.org>
To:     Stephen Rothwell <sfr@...b.auug.org.au>,
        Linux Next Mailing List <linux-next@...r.kernel.org>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Ben Widawsky <ben.widawsky@...el.com>,
        Dan Williams <dan.j.williams@...el.com>
Subject: Re: linux-next: Tree for Feb 15 (CXL on i386)

On 2/15/21 1:34 AM, Stephen Rothwell wrote:
> Hi all,
> 
> Changes since 20210212:
> 


CXL on i386 has these issue:

../drivers/cxl/mem.c:335:2: error: implicit declaration of function ‘writeq’; did you mean ‘writel’? [-Werror=implicit-function-declaration]
  writeq(cmd_reg, cxlm->mbox_regs + CXLDEV_MBOX_CMD_OFFSET);
  ^~~~~~

../drivers/cxl/mem.c:350:15: error: implicit declaration of function ‘readq’; did you mean ‘readl’? [-Werror=implicit-function-declaration]
  status_reg = readq(cxlm->mbox_regs + CXLDEV_MBOX_STATUS_OFFSET);


and lots of these (just one example here):

../drivers/cxl/cxl.h:29:47: note: in expansion of macro ‘GENMASK’
 #define   CXLDEV_MBOX_CMD_PAYLOAD_LENGTH_MASK GENMASK(36, 16)
                                               ^~~~~~~
../drivers/cxl/mem.c:329:25: note: in expansion of macro ‘CXLDEV_MBOX_CMD_PAYLOAD_LENGTH_MASK’
   cmd_reg |= FIELD_PREP(CXLDEV_MBOX_CMD_PAYLOAD_LENGTH_MASK,
                         ^
../include/linux/bits.h:36:11: warning: right shift count is negative [-Wshift-count-negative]
   (~UL(0) >> (BITS_PER_LONG - 1 - (h))))
           ^
../include/linux/bitfield.h:95:49: note: in definition of macro ‘FIELD_PREP’
   ((typeof(_mask))(_val) << __bf_shf(_mask)) & (_mask); \
                                                 ^~~~~
../include/linux/bits.h:38:31: note: in expansion of macro ‘__GENMASK’
  (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
                               ^~~~~~~~~


-- 
~Randy
Reported-by: Randy Dunlap <rdunlap@...radead.org>