Re: [PATCH 1/3] Drivers: hv: vmbus: Disable Suspend-to-Idle for VMBus
From: kernel test robot
Date: Fri Sep 13 2024 - 13:02:21 EST
Hi Erni,
kernel test robot noticed the following build errors:
[auto build test ERROR on hid/for-next]
[also build test ERROR on dtor-input/next dtor-input/for-linus linus/master v6.11-rc7 next-20240913]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Erni-Sri-Satya-Vennela/Drivers-hv-vmbus-Disable-Suspend-to-Idle-for-VMBus/20240913-053127
base: https://git.kernel.org/pub/scm/linux/kernel/git/hid/hid.git for-next
patch link: https://lore.kernel.org/r/1726176470-13133-2-git-send-email-ernis%40linux.microsoft.com
patch subject: [PATCH 1/3] Drivers: hv: vmbus: Disable Suspend-to-Idle for VMBus
config: x86_64-randconfig-161-20240913 (https://download.01.org/0day-ci/archive/20240914/202409140042.imFE8dSL-lkp@xxxxxxxxx/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240914/202409140042.imFE8dSL-lkp@xxxxxxxxx/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202409140042.imFE8dSL-lkp@xxxxxxxxx/
All errors (new ones prefixed by >>):
>> drivers/hv/vmbus_drv.c:985:20: error: use of undeclared identifier 'vmbus_freeze'
985 | .suspend_noirq = vmbus_freeze,
| ^
drivers/hv/vmbus_drv.c:1913:42: warning: shift count >= width of type [-Wshift-count-overflow]
1913 | dma_set_mask(&child_device_obj->device, DMA_BIT_MASK(64));
| ^~~~~~~~~~~~~~~~
include/linux/dma-mapping.h:77:54: note: expanded from macro 'DMA_BIT_MASK'
77 | #define DMA_BIT_MASK(n) (((n) == 64) ? ~0ULL : ((1ULL<<(n))-1))
| ^ ~~~
1 warning and 1 error generated.
vim +/vmbus_freeze +985 drivers/hv/vmbus_drv.c
973
974 /*
975 * Note: we must use the "noirq" ops: see the comment before vmbus_bus_pm.
976 *
977 * suspend_noirq/resume_noirq are set to NULL to support Suspend-to-Idle: we
978 * shouldn't suspend the vmbus devices upon Suspend-to-Idle, otherwise there
979 * is no way to wake up a Generation-2 VM.
980 *
981 * The other 4 ops are for hibernation.
982 */
983
984 static const struct dev_pm_ops vmbus_pm = {
> 985 .suspend_noirq = vmbus_freeze,
986 .resume_noirq = NULL,
987 .freeze_noirq = vmbus_suspend,
988 .thaw_noirq = vmbus_resume,
989 .poweroff_noirq = vmbus_suspend,
990 .restore_noirq = vmbus_resume,
991 };
992
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki