CC: [email protected]
CC: [email protected]
TO: Alistair Popple <[email protected]>
CC: Huang Ying <[email protected]>
CC: Andrew Morton <[email protected]>
CC: Linux Memory Management List <[email protected]>

tree:   https://github.com/hying-caritas/linux.git mmotm-2021-12-06
head:   a57b74d47d00793cb27918be5147ede2ece3a37b
commit: 3c4d0562835e755206ada3026d28b0b6b108fa78 [129/268] mm/migrate.c: rework 
migration_entry_wait() to not take a pageref
:::::: branch date: 2 days ago
:::::: commit date: 2 days ago
config: i386-randconfig-m021-20211209 
(https://download.01.org/0day-ci/archive/20211211/[email protected]/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>
Reported-by: Dan Carpenter <[email protected]>

smatch warnings:
mm/migrate.c:311 __migration_entry_wait() warn: inconsistent returns 'ptl'.

vim +/ptl +311 mm/migrate.c

04e62a29bf157c Christoph Lameter 2006-06-23  283  
0697212a411c1d Christoph Lameter 2006-06-23  284  /*
0697212a411c1d Christoph Lameter 2006-06-23  285   * Something used the pte of 
a page under migration. We need to
0697212a411c1d Christoph Lameter 2006-06-23  286   * get to the page and wait 
until migration is finished.
0697212a411c1d Christoph Lameter 2006-06-23  287   * When we return from this 
function the fault will be retried.
0697212a411c1d Christoph Lameter 2006-06-23  288   */
e66f17ff71772b Naoya Horiguchi   2015-02-11  289  void 
__migration_entry_wait(struct mm_struct *mm, pte_t *ptep,
30dad30922ccc7 Naoya Horiguchi   2013-06-12  290                                
spinlock_t *ptl)
0697212a411c1d Christoph Lameter 2006-06-23  291  {
30dad30922ccc7 Naoya Horiguchi   2013-06-12  292        pte_t pte;
0697212a411c1d Christoph Lameter 2006-06-23  293        swp_entry_t entry;
0697212a411c1d Christoph Lameter 2006-06-23  294        struct page *page;
0697212a411c1d Christoph Lameter 2006-06-23  295  
30dad30922ccc7 Naoya Horiguchi   2013-06-12  296        spin_lock(ptl);
0697212a411c1d Christoph Lameter 2006-06-23  297        pte = *ptep;
0697212a411c1d Christoph Lameter 2006-06-23  298        if (!is_swap_pte(pte))
0697212a411c1d Christoph Lameter 2006-06-23  299                goto out;
0697212a411c1d Christoph Lameter 2006-06-23  300  
0697212a411c1d Christoph Lameter 2006-06-23  301        entry = 
pte_to_swp_entry(pte);
0697212a411c1d Christoph Lameter 2006-06-23  302        if 
(!is_migration_entry(entry))
0697212a411c1d Christoph Lameter 2006-06-23  303                goto out;
0697212a411c1d Christoph Lameter 2006-06-23  304  
af5cdaf82238fb Alistair Popple   2021-06-30  305        page = 
pfn_swap_entry_to_page(entry);
ffc90cbb2970ab Xu Yu             2021-06-15  306        page = 
compound_head(page);
0697212a411c1d Christoph Lameter 2006-06-23  307  
3c4d0562835e75 Alistair Popple   2021-12-09  308        
migration_entry_wait_on_locked(page_folio(page), ptep, ptl);
0697212a411c1d Christoph Lameter 2006-06-23  309        return;
0697212a411c1d Christoph Lameter 2006-06-23  310  out:
0697212a411c1d Christoph Lameter 2006-06-23 @311        pte_unmap_unlock(ptep, 
ptl);
0697212a411c1d Christoph Lameter 2006-06-23  312  }
0697212a411c1d Christoph Lameter 2006-06-23  313  

:::::: The code at line 311 was first introduced by commit
:::::: 0697212a411c1dae03c27845f2de2f3adb32c331 [PATCH] Swapless page 
migration: add R/W migration entries

:::::: TO: Christoph Lameter <[email protected]>
:::::: CC: Linus Torvalds <[email protected]>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to