Re: [PATCH 6/6] drm/i915/gvt: use gfn_to_pfn's page instead of pfn

From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH 6/6] drm/i915/gvt: use gfn_to_pfn's page instead of pfn
Date: Thu, 24 Jun 2021 18:38:03 +0800	[thread overview]
Message-ID: <202106241830.1EkvqAMp-lkp@intel.com> (raw)
In-Reply-To: <20210624035749.4054934-7-stevensd@google.com>

[-- Attachment #1: Type: text/plain, Size: 3410 bytes --]

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-struct-page-from-x86-and-arm64-MMU/20210624-120152
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/834ba0a17bd373a384ac2361165f6a27499a61df
        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@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(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 49484 bytes --]