All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org,
	Paolo Bonzini <pbonzini@redhat.com>
Subject: arch/x86/kernel/cpu/sgx/virt.c:59:13: sparse: sparse: incorrect type in assignment (different base types)
Date: Sun, 20 Oct 2024 17:41:59 +0800	[thread overview]
Message-ID: <202410201716.n8f4T911-lkp@intel.com> (raw)

Hi Steven,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   715ca9dd687f89ddaac8ec8ccb3b5e5a30311a99
commit: 59cbd4eea48fdbc68fc17a29ad71188fea74b28b KVM: Remove HIGH_RES_TIMERS dependency
date:   6 weeks ago
config: x86_64-randconfig-123-20241020 (https://download.01.org/0day-ci/archive/20241020/202410201716.n8f4T911-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241020/202410201716.n8f4T911-lkp@intel.com/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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202410201716.n8f4T911-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
>> arch/x86/kernel/cpu/sgx/virt.c:59:13: sparse: sparse: incorrect type in assignment (different base types) @@     expected int [assigned] ret @@     got restricted vm_fault_t @@
   arch/x86/kernel/cpu/sgx/virt.c:59:13: sparse:     expected int [assigned] ret
   arch/x86/kernel/cpu/sgx/virt.c:59:13: sparse:     got restricted vm_fault_t
>> arch/x86/kernel/cpu/sgx/virt.c:60:20: sparse: sparse: restricted vm_fault_t degrades to integer
>> arch/x86/kernel/cpu/sgx/virt.c:354:36: sparse: sparse: cast removes address space '__user' of expression
   arch/x86/kernel/cpu/sgx/virt.c:385:24: sparse: sparse: cast removes address space '__user' of expression
   arch/x86/kernel/cpu/sgx/virt.c:385:43: sparse: sparse: cast removes address space '__user' of expression
   arch/x86/kernel/cpu/sgx/virt.c:385:58: sparse: sparse: cast removes address space '__user' of expression
   arch/x86/kernel/cpu/sgx/virt.c: note: in included file (through include/linux/miscdevice.h):
   include/linux/list.h:83:21: sparse: sparse: self-comparison always evaluates to true
   include/linux/list.h:83:21: sparse: sparse: self-comparison always evaluates to true

vim +59 arch/x86/kernel/cpu/sgx/virt.c

540745ddbc70ea Sean Christopherson 2021-03-19  32  
540745ddbc70ea Sean Christopherson 2021-03-19  33  static int __sgx_vepc_fault(struct sgx_vepc *vepc,
540745ddbc70ea Sean Christopherson 2021-03-19  34  			    struct vm_area_struct *vma, unsigned long addr)
540745ddbc70ea Sean Christopherson 2021-03-19  35  {
540745ddbc70ea Sean Christopherson 2021-03-19  36  	struct sgx_epc_page *epc_page;
540745ddbc70ea Sean Christopherson 2021-03-19  37  	unsigned long index, pfn;
540745ddbc70ea Sean Christopherson 2021-03-19  38  	int ret;
540745ddbc70ea Sean Christopherson 2021-03-19  39  
540745ddbc70ea Sean Christopherson 2021-03-19  40  	WARN_ON(!mutex_is_locked(&vepc->lock));
540745ddbc70ea Sean Christopherson 2021-03-19  41  
540745ddbc70ea Sean Christopherson 2021-03-19  42  	/* Calculate index of EPC page in virtual EPC's page_array */
540745ddbc70ea Sean Christopherson 2021-03-19  43  	index = vma->vm_pgoff + PFN_DOWN(addr - vma->vm_start);
540745ddbc70ea Sean Christopherson 2021-03-19  44  
540745ddbc70ea Sean Christopherson 2021-03-19  45  	epc_page = xa_load(&vepc->page_array, index);
540745ddbc70ea Sean Christopherson 2021-03-19  46  	if (epc_page)
540745ddbc70ea Sean Christopherson 2021-03-19  47  		return 0;
540745ddbc70ea Sean Christopherson 2021-03-19  48  
540745ddbc70ea Sean Christopherson 2021-03-19  49  	epc_page = sgx_alloc_epc_page(vepc, false);
540745ddbc70ea Sean Christopherson 2021-03-19  50  	if (IS_ERR(epc_page))
540745ddbc70ea Sean Christopherson 2021-03-19  51  		return PTR_ERR(epc_page);
540745ddbc70ea Sean Christopherson 2021-03-19  52  
540745ddbc70ea Sean Christopherson 2021-03-19  53  	ret = xa_err(xa_store(&vepc->page_array, index, epc_page, GFP_KERNEL));
540745ddbc70ea Sean Christopherson 2021-03-19  54  	if (ret)
540745ddbc70ea Sean Christopherson 2021-03-19  55  		goto err_free;
540745ddbc70ea Sean Christopherson 2021-03-19  56  
540745ddbc70ea Sean Christopherson 2021-03-19  57  	pfn = PFN_DOWN(sgx_get_epc_phys_addr(epc_page));
540745ddbc70ea Sean Christopherson 2021-03-19  58  
540745ddbc70ea Sean Christopherson 2021-03-19 @59  	ret = vmf_insert_pfn(vma, addr, pfn);
540745ddbc70ea Sean Christopherson 2021-03-19 @60  	if (ret != VM_FAULT_NOPAGE) {
540745ddbc70ea Sean Christopherson 2021-03-19  61  		ret = -EFAULT;
540745ddbc70ea Sean Christopherson 2021-03-19  62  		goto err_delete;
540745ddbc70ea Sean Christopherson 2021-03-19  63  	}
540745ddbc70ea Sean Christopherson 2021-03-19  64  
540745ddbc70ea Sean Christopherson 2021-03-19  65  	return 0;
540745ddbc70ea Sean Christopherson 2021-03-19  66  
540745ddbc70ea Sean Christopherson 2021-03-19  67  err_delete:
540745ddbc70ea Sean Christopherson 2021-03-19  68  	xa_erase(&vepc->page_array, index);
540745ddbc70ea Sean Christopherson 2021-03-19  69  err_free:
540745ddbc70ea Sean Christopherson 2021-03-19  70  	sgx_free_epc_page(epc_page);
540745ddbc70ea Sean Christopherson 2021-03-19  71  	return ret;
540745ddbc70ea Sean Christopherson 2021-03-19  72  }
540745ddbc70ea Sean Christopherson 2021-03-19  73  

:::::: The code at line 59 was first introduced by commit
:::::: 540745ddbc70eabdc7dbd3fcc00fe4fb17cd59ba x86/sgx: Introduce virtual EPC for use by KVM guests

:::::: TO: Sean Christopherson <sean.j.christopherson@intel.com>
:::::: CC: Borislav Petkov <bp@suse.de>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

             reply	other threads:[~2024-10-20  9:42 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-20  9:41 kernel test robot [this message]
2024-10-22 17:39 ` arch/x86/kernel/cpu/sgx/virt.c:59:13: sparse: sparse: incorrect type in assignment (different base types) Sean Christopherson
  -- strict thread matches above, loose matches on Subject: below --
2024-11-20  5:59 kernel test robot
2024-11-20 15:05 ` Steven Rostedt
2024-11-21 12:19   ` Philip Li
2023-08-12 10:57 kernel test robot
2022-06-04  7:43 kernel test robot
2022-02-21 21:28 kernel test robot
2021-12-04 19:34 kernel test robot
2021-12-04 19:34 ` kernel test robot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202410201716.n8f4T911-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=pbonzini@redhat.com \
    --cc=rostedt@goodmis.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.