9:20 p.m.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git hyperv-next
head: 19fd7ca00201c0525452dcf5a490e4b01674ef4c
commit: 6327882f7b4a476ea902de4bee5657f1028d6859 [4/5] scsi: storvsc: Add Isolation VM
support for storvsc driver
config: x86_64-randconfig-a013-20211216
(https://download.01.org/0day-ci/archive/20211218/202112181827.o3X7GmHz-lk...)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project
4c9e31a4814592bbda7153833e46728dc7b21100)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
chmod +x ~/bin/make.cross
#
https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git/commit/?...
git remote add hyperv
https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git
git fetch --no-tags hyperv hyperv-next
git checkout 6327882f7b4a476ea902de4bee5657f1028d6859
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir
ARCH=x86_64 SHELL=/bin/bash drivers/hv/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
> drivers/hv/vmbus_drv.c:2082:29: warning: shift count >= width
of type [-Wshift-count-overflow]
static u64 vmbus_dma_mask = DMA_BIT_MASK(64);
^~~~~~~~~~~~~~~~
include/linux/dma-mapping.h:76:54: note: expanded from macro 'DMA_BIT_MASK'
#define DMA_BIT_MASK(n) (((n) == 64) ? ~0ULL : ((1ULL<<(n))-1))
^ ~~~
1 warning generated.
vim +2082 drivers/hv/vmbus_drv.c
2081
2082 static u64 vmbus_dma_mask = DMA_BIT_MASK(64);
2083 /*
2084 * vmbus_device_register - Register the child device
2085 */
2086 int vmbus_device_register(struct hv_device *child_device_obj)
2087 {
2088 struct kobject *kobj = &child_device_obj->device.kobj;
2089 int ret;
2090
2091 dev_set_name(&child_device_obj->device, "%pUl",
2092 &child_device_obj->channel->offermsg.offer.if_instance);
2093
2094 child_device_obj->device.bus = &hv_bus;
2095 child_device_obj->device.parent = &hv_acpi_dev->dev;
2096 child_device_obj->device.release = vmbus_device_release;
2097
2098 /*
2099 * Register with the LDM. This will kick off the driver/device
2100 * binding...which will eventually call vmbus_match() and vmbus_probe()
2101 */
2102 ret = device_register(&child_device_obj->device);
2103 if (ret) {
2104 pr_err("Unable to register child device\n");
2105 return ret;
2106 }
2107
2108 child_device_obj->channels_kset = kset_create_and_add("channels",
2109 NULL, kobj);
2110 if (!child_device_obj->channels_kset) {
2111 ret = -ENOMEM;
2112 goto err_dev_unregister;
2113 }
2114
2115 ret = vmbus_add_channel_kobj(child_device_obj,
2116 child_device_obj->channel);
2117 if (ret) {
2118 pr_err("Unable to register primary channeln");
2119 goto err_kset_unregister;
2120 }
2121 hv_debug_add_dev_dir(child_device_obj);
2122
2123 child_device_obj->device.dma_mask = &vmbus_dma_mask;
2124 child_device_obj->device.dma_parms = &child_device_obj->dma_parms;
2125 return 0;
2126
2127 err_kset_unregister:
2128 kset_unregister(child_device_obj->channels_kset);
2129
2130 err_dev_unregister:
2131 device_unregister(&child_device_obj->device);
2132 return ret;
2133 }
2134
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
11:29 p.m.
On Sat, Dec 18, 2021 at 06:20:43PM +0800, kernel test robot wrote:
tree:
https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git hyperv-next
head: 19fd7ca00201c0525452dcf5a490e4b01674ef4c
commit: 6327882f7b4a476ea902de4bee5657f1028d6859 [4/5] scsi: storvsc: Add Isolation VM
support for storvsc driver
config: x86_64-randconfig-a013-20211216
(https://download.01.org/0day-ci/archive/20211218/202112181827.o3X7GmHz-lk...)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project
4c9e31a4814592bbda7153833e46728dc7b21100)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
chmod +x ~/bin/make.cross
#
https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git/commit/?...
git remote add hyperv
https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git
git fetch --no-tags hyperv hyperv-next
git checkout 6327882f7b4a476ea902de4bee5657f1028d6859
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir
ARCH=x86_64 SHELL=/bin/bash drivers/hv/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/hv/vmbus_drv.c:2082:29: warning: shift count >= width of type
[-Wshift-count-overflow]
static u64 vmbus_dma_mask = DMA_BIT_MASK(64);
^~~~~~~~~~~~~~~~
include/linux/dma-mapping.h:76:54: note: expanded from macro 'DMA_BIT_MASK'
#define DMA_BIT_MASK(n) (((n) == 64) ? ~0ULL : ((1ULL<<(n))-1))
^ ~~~
I don't think there is anything wrong with the code. When n is 64 the
problematic expression is not evaluated.
Wei.
3:53 a.m.
Hi Wei,
On Sun, Dec 19, 2021 at 12:29:37PM +0000, Wei Liu wrote:
On Sat, Dec 18, 2021 at 06:20:43PM +0800, kernel test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git
hyperv-next
> head: 19fd7ca00201c0525452dcf5a490e4b01674ef4c
> commit: 6327882f7b4a476ea902de4bee5657f1028d6859 [4/5] scsi: storvsc: Add Isolation
VM support for storvsc driver
> config: x86_64-randconfig-a013-20211216
(https://download.01.org/0day-ci/archive/20211218/202112181827.o3X7GmHz-lk...)
> compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project
4c9e31a4814592bbda7153833e46728dc7b21100)
> reproduce (this is a W=1 build):
> wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
> chmod +x ~/bin/make.cross
> #
https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git/commit/?...
> git remote add hyperv
https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git
> git fetch --no-tags hyperv hyperv-next
> git checkout 6327882f7b4a476ea902de4bee5657f1028d6859
> # save the config file to linux build tree
> mkdir build_dir
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir
ARCH=x86_64 SHELL=/bin/bash drivers/hv/
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp(a)intel.com>
>
> All warnings (new ones prefixed by >>):
>
> >> drivers/hv/vmbus_drv.c:2082:29: warning: shift count >= width of type
[-Wshift-count-overflow]
> static u64 vmbus_dma_mask = DMA_BIT_MASK(64);
> ^~~~~~~~~~~~~~~~
> include/linux/dma-mapping.h:76:54: note: expanded from macro
'DMA_BIT_MASK'
> #define DMA_BIT_MASK(n) (((n) == 64) ? ~0ULL : ((1ULL<<(n))-1))
> ^ ~~~
I don't think there is anything wrong with the code. When n is 64 the
problematic expression is not evaluated.
Yes, unfortunately, this is an outstanding bug in clang where it does
not properly build a control flow graph for expressions at a global
scope:
https://github.com/ClangBuiltLinux/linux/issues/92
We should absolutely fix that but it has not come up too often so other
fires have been prioritized. Now that -Werror is a thing, that issue's
priority should probably be upgraded, as this warning will break
allmodconfig for clang.
If you were feeling generous, just changing that to ~0ULL directly would
solve the warning but I get that it is less documentation that way.
Cheers,
Nathan
4:44 a.m.
On Mon, Dec 20, 2021 at 09:53:30AM -0700, Nathan Chancellor wrote:
Hi Wei,
On Sun, Dec 19, 2021 at 12:29:37PM +0000, Wei Liu wrote:
> On Sat, Dec 18, 2021 at 06:20:43PM +0800, kernel test robot wrote:
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git
hyperv-next
> > head: 19fd7ca00201c0525452dcf5a490e4b01674ef4c
> > commit: 6327882f7b4a476ea902de4bee5657f1028d6859 [4/5] scsi: storvsc: Add
Isolation VM support for storvsc driver
> > config: x86_64-randconfig-a013-20211216
(https://download.01.org/0day-ci/archive/20211218/202112181827.o3X7GmHz-lk...)
> > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project
4c9e31a4814592bbda7153833e46728dc7b21100)
> > reproduce (this is a W=1 build):
> > wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
> > chmod +x ~/bin/make.cross
> > #
https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git/commit/?...
> > git remote add hyperv
https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git
> > git fetch --no-tags hyperv hyperv-next
> > git checkout 6327882f7b4a476ea902de4bee5657f1028d6859
> > # save the config file to linux build tree
> > mkdir build_dir
> > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1
O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/hv/
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp(a)intel.com>
> >
> > All warnings (new ones prefixed by >>):
> >
> > >> drivers/hv/vmbus_drv.c:2082:29: warning: shift count >= width of
type [-Wshift-count-overflow]
> > static u64 vmbus_dma_mask = DMA_BIT_MASK(64);
> > ^~~~~~~~~~~~~~~~
> > include/linux/dma-mapping.h:76:54: note: expanded from macro
'DMA_BIT_MASK'
> > #define DMA_BIT_MASK(n) (((n) == 64) ? ~0ULL : ((1ULL<<(n))-1))
> > ^ ~~~
>
> I don't think there is anything wrong with the code. When n is 64 the
> problematic expression is not evaluated.
Yes, unfortunately, this is an outstanding bug in clang where it does
not properly build a control flow graph for expressions at a global
scope:
https://github.com/ClangBuiltLinux/linux/issues/92
We should absolutely fix that but it has not come up too often so other
fires have been prioritized. Now that -Werror is a thing, that issue's
priority should probably be upgraded, as this warning will break
allmodconfig for clang.
Thanks for the explanation.
If you were feeling generous, just changing that to ~0ULL directly would
solve the warning but I get that it is less documentation that way.
I would actually like to keep the code as-is because it gives a clear
idea what it is doing.
Wei.
>
> Cheers,
> Nathan
274
days inactive
276
days old
3 comments
3 participants
participants (3)
-
kernel test robot -
Nathan Chancellor -
Wei Liu