Hi David,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on v5.13-rc7]
[cannot apply to kvm/queue drm-intel/for-linux-next kvm-ppc/kvm-ppc-next next-20210623]
[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/David-Stevens/KVM-Remove-uses-of...
base: 13311e74253fe64329390df80bed3f07314ddd61
config: x86_64-randconfig-r025-20210622 (attached as .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/834ba0a17bd373a384ac2361165f6a274...
git remote add linux-review
https://github.com/0day-ci/linux
git fetch --no-tags linux-review
David-Stevens/KVM-Remove-uses-of-struct-page-from-x86-and-arm64-MMU/20210624-120152
git checkout 834ba0a17bd373a384ac2361165f6a27499a61df
# save the attached .config to linux build tree
make W=1 ARCH=x86_64
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 >>):
In file included from drivers/gpu/drm/i915/gvt/gvt.h:734,
from drivers/gpu/drm/i915/intel_gvt.c:27:
drivers/gpu/drm/i915/gvt/mpt.h: In function
'intel_gvt_hypervisor_gfn_to_mfn_page':
> drivers/gpu/drm/i915/gvt/mpt.h:227:9: warning: returning
'struct page *' from a function with return type 'long unsigned int' makes
integer from pointer without a cast [-Wint-conversion]
227 | return
intel_gvt_host.mpt->gfn_to_mfn_page(vgpu->handle, gfn);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from drivers/gpu/drm/i915/gvt/gvt.h:734,
from drivers/gpu/drm/i915/gvt/gtt.c:37:
drivers/gpu/drm/i915/gvt/mpt.h: In function
'intel_gvt_hypervisor_gfn_to_mfn_page':
> drivers/gpu/drm/i915/gvt/mpt.h:227:9: warning: returning
'struct page *' from a function with return type 'long unsigned int' makes
integer from pointer without a cast [-Wint-conversion]
227 | return
intel_gvt_host.mpt->gfn_to_mfn_page(vgpu->handle, gfn);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/gvt/gtt.c: In function 'is_2MB_gtt_possible':
> drivers/gpu/drm/i915/gvt/gtt.c:1169:7: warning: assignment to
'struct page *' from 'long unsigned int' makes pointer from integer
without a cast [-Wint-conversion]
1169 | page =
intel_gvt_hypervisor_gfn_to_mfn_page(vgpu, ops->get_pfn(entry));
| ^
vim +227 drivers/gpu/drm/i915/gvt/mpt.h
215
216 /**
217 * intel_gvt_hypervisor_gfn_to_mfn_page - translate a GFN to MFN page
218 * @vgpu: a vGPU
219 * @gpfn: guest pfn
220 *
221 * Returns:
222 * struct page* on success, NULL if failed.
223 */
224 static inline unsigned long intel_gvt_hypervisor_gfn_to_mfn_page(
225 struct intel_vgpu *vgpu, unsigned long gfn)
226 {
227 return intel_gvt_host.mpt->gfn_to_mfn_page(vgpu->handle,
gfn);
228 }
229
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org