Re: [Intel-gfx] [PATCH 1/3] i915/gvt: Introduce the mmio_table.c to support VFIO new mdev API
Hi Zhi, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [also build test WARNING on next-20220208] [cannot apply to drm-intel/for-linux-next v5.17-rc3] [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] url: https://github.com/0day-ci/linux/commits/Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 base: git://anongit.freedesktop.org/drm/drm-tip drm-tip config: x86_64-rhel-8.3-kselftests (https://download.01.org/0day-ci/archive/20220208/202202082210.bpzsju31-...@intel.com/config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce: # apt-get install sparse # sparse version: v0.6.4-dirty # https://github.com/0day-ci/linux/commit/533f92651a7a56481a053f1e04dc5a5ec024ffb9 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 git checkout 533f92651a7a56481a053f1e04dc5a5ec024ffb9 # save the config file to linux build tree mkdir build_dir make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/gpu/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot sparse warnings: (new ones prefixed by >>) >> drivers/gpu/drm/i915/gvt/handlers.c:45:6: sparse: sparse: symbol >> 'intel_gvt_match_device' was not declared. Should it be static? vim +/intel_gvt_match_device +45 drivers/gpu/drm/i915/gvt/handlers.c 12d14cc43b3470 Zhi Wang 2016-08-30 44 12d14cc43b3470 Zhi Wang 2016-08-30 @45 bool intel_gvt_match_device(struct intel_gvt *gvt, 12d14cc43b3470 Zhi Wang 2016-08-30 46 unsigned long device) 12d14cc43b3470 Zhi Wang 2016-08-30 47 { 533f92651a7a56 Zhi Wang 2022-01-27 48 return intel_gvt_get_device_type(gvt->gt->i915) & device; 12d14cc43b3470 Zhi Wang 2016-08-30 49 } 12d14cc43b3470 Zhi Wang 2016-08-30 50 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
Re: [Intel-gfx] [PATCH 1/3] i915/gvt: Introduce the mmio_table.c to support VFIO new mdev API
Hi Zhi, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [also build test WARNING on next-20220207] [cannot apply to drm-intel/for-linux-next hch-configfs/for-next v5.17-rc3] [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] url: https://github.com/0day-ci/linux/commits/Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 base: git://anongit.freedesktop.org/drm/drm-tip drm-tip config: x86_64-rhel-8.3-kselftests (https://download.01.org/0day-ci/archive/20220207/202202072226.kzm2qm5q-...@intel.com/config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce: # apt-get install sparse # sparse version: v0.6.4-dirty # https://github.com/0day-ci/linux/commit/533f92651a7a56481a053f1e04dc5a5ec024ffb9 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 git checkout 533f92651a7a56481a053f1e04dc5a5ec024ffb9 # save the config file to linux build tree mkdir build_dir make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/gpu/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot sparse warnings: (new ones prefixed by >>) >> drivers/gpu/drm/i915/gvt/handlers.c:45:6: sparse: sparse: symbol >> 'intel_gvt_match_device' was not declared. Should it be static? vim +/intel_gvt_match_device +45 drivers/gpu/drm/i915/gvt/handlers.c 12d14cc43b3470 Zhi Wang 2016-08-30 44 12d14cc43b3470 Zhi Wang 2016-08-30 @45 bool intel_gvt_match_device(struct intel_gvt *gvt, 12d14cc43b3470 Zhi Wang 2016-08-30 46 unsigned long device) 12d14cc43b3470 Zhi Wang 2016-08-30 47 { 533f92651a7a56 Zhi Wang 2022-01-27 48 return intel_gvt_get_device_type(gvt->gt->i915) & device; 12d14cc43b3470 Zhi Wang 2016-08-30 49 } 12d14cc43b3470 Zhi Wang 2016-08-30 50 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
Re: [Intel-gfx] [PATCH 1/3] i915/gvt: Introduce the mmio_table.c to support VFIO new mdev API
Hi Zhi, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [also build test WARNING on next-20220127] [cannot apply to drm-intel/for-linux-next hch-configfs/for-next v5.17-rc1] [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] url: https://github.com/0day-ci/linux/commits/Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 base: git://anongit.freedesktop.org/drm/drm-tip drm-tip config: x86_64-rhel-8.3-kselftests (https://download.01.org/0day-ci/archive/20220128/202201281209.mnhckgpp-...@intel.com/config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce: # apt-get install sparse # sparse version: v0.6.4-dirty # https://github.com/0day-ci/linux/commit/533f92651a7a56481a053f1e04dc5a5ec024ffb9 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 git checkout 533f92651a7a56481a053f1e04dc5a5ec024ffb9 # save the config file to linux build tree mkdir build_dir make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/gpu/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot sparse warnings: (new ones prefixed by >>) >> drivers/gpu/drm/i915/gvt/handlers.c:45:6: sparse: sparse: symbol >> 'intel_gvt_match_device' was not declared. Should it be static? Please review and possibly fold the followup patch. --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
Re: [Intel-gfx] [PATCH 1/3] i915/gvt: Introduce the mmio_table.c to support VFIO new mdev API
Hi Zhi, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [also build test WARNING on next-20220127] [cannot apply to drm-intel/for-linux-next v5.17-rc1] [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] url: https://github.com/0day-ci/linux/commits/Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 base: git://anongit.freedesktop.org/drm/drm-tip drm-tip config: x86_64-rhel-8.3 (https://download.01.org/0day-ci/archive/20220128/202201280218.adxamjx6-...@intel.com/config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce (this is a W=1 build): # https://github.com/0day-ci/linux/commit/533f92651a7a56481a053f1e04dc5a5ec024ffb9 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 git checkout 533f92651a7a56481a053f1e04dc5a5ec024ffb9 # save the config file to linux build tree mkdir build_dir make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/gpu/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All warnings (new ones prefixed by >>): >> drivers/gpu/drm/i915/gvt/handlers.c:45:6: warning: no previous prototype for >> 'intel_gvt_match_device' [-Wmissing-prototypes] 45 | bool intel_gvt_match_device(struct intel_gvt *gvt, | ^~ vim +/intel_gvt_match_device +45 drivers/gpu/drm/i915/gvt/handlers.c 12d14cc43b3470 Zhi Wang 2016-08-30 44 12d14cc43b3470 Zhi Wang 2016-08-30 @45 bool intel_gvt_match_device(struct intel_gvt *gvt, 12d14cc43b3470 Zhi Wang 2016-08-30 46 unsigned long device) 12d14cc43b3470 Zhi Wang 2016-08-30 47 { 533f92651a7a56 Zhi Wang 2022-01-27 48 return intel_gvt_get_device_type(gvt->gt->i915) & device; 12d14cc43b3470 Zhi Wang 2016-08-30 49 } 12d14cc43b3470 Zhi Wang 2016-08-30 50 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
Re: [Intel-gfx] [PATCH 1/3] i915/gvt: Introduce the mmio_table.c to support VFIO new mdev API
Hi Zhi, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [also build test WARNING on next-20220127] [cannot apply to drm-intel/for-linux-next hch-configfs/for-next v5.17-rc1] [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] url: https://github.com/0day-ci/linux/commits/Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 base: git://anongit.freedesktop.org/drm/drm-tip drm-tip config: x86_64-randconfig-a005 (https://download.01.org/0day-ci/archive/20220128/202201280121.yx5gbodp-...@intel.com/config) compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project f32dccb9a43b02ce4e540d6ba5dbbdb188f2dc7d) 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://github.com/0day-ci/linux/commit/533f92651a7a56481a053f1e04dc5a5ec024ffb9 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 git checkout 533f92651a7a56481a053f1e04dc5a5ec024ffb9 # 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/gpu/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All warnings (new ones prefixed by >>): >> drivers/gpu/drm/i915/gvt/handlers.c:45:6: warning: no previous prototype for >> function 'intel_gvt_match_device' [-Wmissing-prototypes] bool intel_gvt_match_device(struct intel_gvt *gvt, ^ drivers/gpu/drm/i915/gvt/handlers.c:45:1: note: declare 'static' if the function is not intended to be used outside of this translation unit bool intel_gvt_match_device(struct intel_gvt *gvt, ^ static 1 warning generated. vim +/intel_gvt_match_device +45 drivers/gpu/drm/i915/gvt/handlers.c 12d14cc43b3470 Zhi Wang 2016-08-30 44 12d14cc43b3470 Zhi Wang 2016-08-30 @45 bool intel_gvt_match_device(struct intel_gvt *gvt, 12d14cc43b3470 Zhi Wang 2016-08-30 46 unsigned long device) 12d14cc43b3470 Zhi Wang 2016-08-30 47 { 533f92651a7a56 Zhi Wang 2022-01-27 48 return intel_gvt_get_device_type(gvt->gt->i915) & device; 12d14cc43b3470 Zhi Wang 2016-08-30 49 } 12d14cc43b3470 Zhi Wang 2016-08-30 50 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
Re: [Intel-gfx] [PATCH 1/3] i915/gvt: Introduce the mmio_table.c to support VFIO new mdev API
Hi Zhi, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [also build test WARNING on next-20220127] [cannot apply to drm-intel/for-linux-next v5.17-rc1] [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] url: https://github.com/0day-ci/linux/commits/Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 base: git://anongit.freedesktop.org/drm/drm-tip drm-tip config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20220128/202201280125.vultasas-...@intel.com/config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce (this is a W=1 build): # https://github.com/0day-ci/linux/commit/533f92651a7a56481a053f1e04dc5a5ec024ffb9 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Zhi-Wang/i915-gvt-Introduce-the-mmio_table-c-to-support-VFIO-new-mdev-API/20220127-200727 git checkout 533f92651a7a56481a053f1e04dc5a5ec024ffb9 # save the config file to linux build tree mkdir build_dir make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/gpu/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All warnings (new ones prefixed by >>): >> drivers/gpu/drm/i915/gvt/mmio_table.c:37: warning: expecting prototype for >> intel_gvt_get_init_device_info(). Prototype was for >> intel_gvt_init_device_info() instead vim +37 drivers/gpu/drm/i915/gvt/mmio_table.c 28 29 /** 30 * intel_gvt_get_init_device_info - Fill a GVT device info 31 * @i915: drm i915 private data 32 * @info: GVT device info 33 * 34 * This function will be called during the initialization of a GVT device. 35 */ 36 void intel_gvt_init_device_info(struct drm_i915_private *i915, struct intel_gvt_device_info *info) > 37 { 38 struct pci_dev *pdev = to_pci_dev(i915->drm.dev); 39 40 info->max_support_vgpus = 8; 41 info->cfg_space_size = PCI_CFG_SPACE_EXP_SIZE; 42 info->mmio_size = 2 * 1024 * 1024; 43 info->mmio_bar = 0; 44 info->gtt_start_offset = 8 * 1024 * 1024; 45 info->gtt_entry_size = 8; 46 info->gtt_entry_size_shift = 3; 47 info->gmadr_bytes_in_cmd = 8; 48 info->max_surface_size = 36 * 1024 * 1024; 49 info->msi_cap_offset = pdev->msi_cap; 50 } 51 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org