CC: [email protected]
CC: [email protected]
TO: Huang Ying <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/vishal/tiering.git 
tiering-0.8
head:   d58c7b0e1a99a2ec17f2910a310835bafc50b4d1
commit: d36de4ed8b4e6880e52ab063a39cf00498de4685 [2/44] NUMA balancing: 
optimize page placement for memory tiering system
:::::: branch date: 28 hours ago
:::::: commit date: 10 weeks ago
config: x86_64-randconfig-m001 
(https://download.01.org/0day-ci/archive/20220114/[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/vmscan.c:3795 pgdat_balanced() warn: bitwise AND condition is false here

vim +3795 mm/vmscan.c

d36de4ed8b4e68 Huang Ying      2019-06-04  3773  
e716f2eb24defb Mel Gorman      2017-05-03  3774  /*
e716f2eb24defb Mel Gorman      2017-05-03  3775   * Returns true if there is an 
eligible zone balanced for the request order
97a225e69a1f88 Joonsoo Kim     2020-06-03  3776   * and highest_zoneidx
e716f2eb24defb Mel Gorman      2017-05-03  3777   */
97a225e69a1f88 Joonsoo Kim     2020-06-03  3778  static bool 
pgdat_balanced(pg_data_t *pgdat, int order, int highest_zoneidx)
60cefed485a02b Johannes Weiner 2012-11-29  3779  {
e716f2eb24defb Mel Gorman      2017-05-03  3780         int i;
e716f2eb24defb Mel Gorman      2017-05-03  3781         unsigned long mark = -1;
e716f2eb24defb Mel Gorman      2017-05-03  3782         struct zone *zone;
60cefed485a02b Johannes Weiner 2012-11-29  3783  
1c30844d2dfe27 Mel Gorman      2018-12-28  3784         /*
1c30844d2dfe27 Mel Gorman      2018-12-28  3785          * Check watermarks 
bottom-up as lower zones are more likely to
1c30844d2dfe27 Mel Gorman      2018-12-28  3786          * meet watermarks.
1c30844d2dfe27 Mel Gorman      2018-12-28  3787          */
97a225e69a1f88 Joonsoo Kim     2020-06-03  3788         for (i = 0; i <= 
highest_zoneidx; i++) {
e716f2eb24defb Mel Gorman      2017-05-03  3789                 zone = 
pgdat->node_zones + i;
e716f2eb24defb Mel Gorman      2017-05-03  3790  
e716f2eb24defb Mel Gorman      2017-05-03  3791                 if 
(!managed_zone(zone))
e716f2eb24defb Mel Gorman      2017-05-03  3792                         
continue;
e716f2eb24defb Mel Gorman      2017-05-03  3793  
e716f2eb24defb Mel Gorman      2017-05-03  3794                 mark = 
high_wmark_pages(zone);
d36de4ed8b4e68 Huang Ying      2019-06-04 @3795                 if 
(sysctl_numa_balancing_mode & NUMA_BALANCING_MEMORY_TIERING &&
d36de4ed8b4e68 Huang Ying      2019-06-04  3796                     
numa_demotion_enabled &&
d36de4ed8b4e68 Huang Ying      2019-06-04  3797                     
next_demotion_node(pgdat->node_id) != NUMA_NO_NODE) {
d36de4ed8b4e68 Huang Ying      2019-06-04  3798                         
unsigned long promote_mark;
d36de4ed8b4e68 Huang Ying      2019-06-04  3799  
d36de4ed8b4e68 Huang Ying      2019-06-04  3800                         
promote_mark = min(NUMA_BALANCING_PROMOTE_WATERMARK,
d36de4ed8b4e68 Huang Ying      2019-06-04  3801                                 
           pgdat->node_present_pages >> 6);
d36de4ed8b4e68 Huang Ying      2019-06-04  3802                         mark += 
promote_mark;
d36de4ed8b4e68 Huang Ying      2019-06-04  3803                 }
97a225e69a1f88 Joonsoo Kim     2020-06-03  3804                 if 
(zone_watermark_ok_safe(zone, order, mark, highest_zoneidx))
e716f2eb24defb Mel Gorman      2017-05-03  3805                         return 
true;
e716f2eb24defb Mel Gorman      2017-05-03  3806         }
6256c6b499a168 Mel Gorman      2016-07-28  3807  
e716f2eb24defb Mel Gorman      2017-05-03  3808         /*
97a225e69a1f88 Joonsoo Kim     2020-06-03  3809          * If a node has no 
populated zone within highest_zoneidx, it does not
e716f2eb24defb Mel Gorman      2017-05-03  3810          * need balancing by 
definition. This can happen if a zone-restricted
e716f2eb24defb Mel Gorman      2017-05-03  3811          * allocation tries to 
wake a remote kswapd.
e716f2eb24defb Mel Gorman      2017-05-03  3812          */
e716f2eb24defb Mel Gorman      2017-05-03  3813         if (mark == -1)
6256c6b499a168 Mel Gorman      2016-07-28  3814                 return true;
e716f2eb24defb Mel Gorman      2017-05-03  3815  
e716f2eb24defb Mel Gorman      2017-05-03  3816         return false;
60cefed485a02b Johannes Weiner 2012-11-29  3817  }
60cefed485a02b Johannes Weiner 2012-11-29  3818  

---
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