* Re: [PATCH V2] mm, compaction: don't use ALLOC_CMA in long term GUP flow
[not found] <1734157420-31110-1-git-send-email-yangge1116@126.com>
@ 2024-12-15 5:32 ` kernel test robot
0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2024-12-15 5:32 UTC (permalink / raw)
To: yangge1116, akpm
Cc: llvm, oe-kbuild-all, linux-mm, linux-kernel, stable, 21cnbao,
david, baolin.wang, vbabka, liuzixing, yangge
Hi,
kernel test robot noticed the following build errors:
[auto build test ERROR on akpm-mm/mm-everything]
url: https://github.com/intel-lab-lkp/linux/commits/yangge1116-126-com/mm-compaction-don-t-use-ALLOC_CMA-in-long-term-GUP-flow/20241214-142453
base: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-everything
patch link: https://lore.kernel.org/r/1734157420-31110-1-git-send-email-yangge1116%40126.com
patch subject: [PATCH V2] mm, compaction: don't use ALLOC_CMA in long term GUP flow
config: arm-randconfig-001-20241215 (https://download.01.org/0day-ci/archive/20241215/202412151325.svvh8EAB-lkp@intel.com/config)
compiler: clang version 20.0.0git (https://github.com/llvm/llvm-project 2dc22615fd46ab2566d0f26d5ba234ab12dc4bf8)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241215/202412151325.svvh8EAB-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/202412151325.svvh8EAB-lkp@intel.com/
All errors (new ones prefixed by >>):
In file included from mm/vmscan.c:30:
include/linux/mm_inline.h:47:41: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
47 | __mod_lruvec_state(lruvec, NR_LRU_BASE + lru, nr_pages);
| ~~~~~~~~~~~ ^ ~~~
include/linux/mm_inline.h:49:22: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
49 | NR_ZONE_LRU_BASE + lru, nr_pages);
| ~~~~~~~~~~~~~~~~ ^ ~~~
mm/vmscan.c:409:51: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
409 | size += zone_page_state(zone, NR_ZONE_LRU_BASE + lru);
| ~~~~~~~~~~~~~~~~ ^ ~~~
mm/vmscan.c:1773:4: warning: arithmetic between different enumeration types ('enum vm_event_item' and 'enum zone_type') [-Wenum-enum-conversion]
1773 | __count_zid_vm_events(PGSCAN_SKIP, zid, nr_skipped[zid]);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/vmstat.h:139:34: note: expanded from macro '__count_zid_vm_events'
139 | __count_vm_events(item##_NORMAL - ZONE_NORMAL + zid, delta)
| ~~~~~~~~~~~~~ ^ ~~~~~~~~~~~
mm/vmscan.c:2279:51: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
2279 | inactive = lruvec_page_state(lruvec, NR_LRU_BASE + inactive_lru);
| ~~~~~~~~~~~ ^ ~~~~~~~~~~~~
mm/vmscan.c:2280:49: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
2280 | active = lruvec_page_state(lruvec, NR_LRU_BASE + active_lru);
| ~~~~~~~~~~~ ^ ~~~~~~~~~~
>> mm/vmscan.c:5822:61: error: too many arguments to function call, expected 3, have 4
5822 | if (compaction_suitable(zone, sc->order, sc->reclaim_idx, 0))
| ~~~~~~~~~~~~~~~~~~~ ^
include/linux/compaction.h:111:20: note: 'compaction_suitable' declared here
111 | static inline bool compaction_suitable(struct zone *zone, int order,
| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
112 | int highest_zoneidx)
| ~~~~~~~~~~~~~~~~~~~
mm/vmscan.c:6050:61: error: too many arguments to function call, expected 3, have 4
6050 | if (!compaction_suitable(zone, sc->order, sc->reclaim_idx, 0))
| ~~~~~~~~~~~~~~~~~~~ ^
include/linux/compaction.h:111:20: note: 'compaction_suitable' declared here
111 | static inline bool compaction_suitable(struct zone *zone, int order,
| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
112 | int highest_zoneidx)
| ~~~~~~~~~~~~~~~~~~~
mm/vmscan.c:6239:3: warning: arithmetic between different enumeration types ('enum vm_event_item' and 'enum zone_type') [-Wenum-enum-conversion]
6239 | __count_zid_vm_events(ALLOCSTALL, sc->reclaim_idx, 1);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/vmstat.h:139:34: note: expanded from macro '__count_zid_vm_events'
139 | __count_vm_events(item##_NORMAL - ZONE_NORMAL + zid, delta)
| ~~~~~~~~~~~~~ ^ ~~~~~~~~~~~
7 warnings and 2 errors generated.
vim +5822 mm/vmscan.c
5778
5779 /*
5780 * Reclaim/compaction is used for high-order allocation requests. It reclaims
5781 * order-0 pages before compacting the zone. should_continue_reclaim() returns
5782 * true if more pages should be reclaimed such that when the page allocator
5783 * calls try_to_compact_pages() that it will have enough free pages to succeed.
5784 * It will give up earlier than that if there is difficulty reclaiming pages.
5785 */
5786 static inline bool should_continue_reclaim(struct pglist_data *pgdat,
5787 unsigned long nr_reclaimed,
5788 struct scan_control *sc)
5789 {
5790 unsigned long pages_for_compaction;
5791 unsigned long inactive_lru_pages;
5792 int z;
5793
5794 /* If not in reclaim/compaction mode, stop */
5795 if (!in_reclaim_compaction(sc))
5796 return false;
5797
5798 /*
5799 * Stop if we failed to reclaim any pages from the last SWAP_CLUSTER_MAX
5800 * number of pages that were scanned. This will return to the caller
5801 * with the risk reclaim/compaction and the resulting allocation attempt
5802 * fails. In the past we have tried harder for __GFP_RETRY_MAYFAIL
5803 * allocations through requiring that the full LRU list has been scanned
5804 * first, by assuming that zero delta of sc->nr_scanned means full LRU
5805 * scan, but that approximation was wrong, and there were corner cases
5806 * where always a non-zero amount of pages were scanned.
5807 */
5808 if (!nr_reclaimed)
5809 return false;
5810
5811 /* If compaction would go ahead or the allocation would succeed, stop */
5812 for (z = 0; z <= sc->reclaim_idx; z++) {
5813 struct zone *zone = &pgdat->node_zones[z];
5814 if (!managed_zone(zone))
5815 continue;
5816
5817 /* Allocation can already succeed, nothing to do */
5818 if (zone_watermark_ok(zone, sc->order, min_wmark_pages(zone),
5819 sc->reclaim_idx, 0))
5820 return false;
5821
> 5822 if (compaction_suitable(zone, sc->order, sc->reclaim_idx, 0))
5823 return false;
5824 }
5825
5826 /*
5827 * If we have not reclaimed enough pages for compaction and the
5828 * inactive lists are large enough, continue reclaiming
5829 */
5830 pages_for_compaction = compact_gap(sc->order);
5831 inactive_lru_pages = node_page_state(pgdat, NR_INACTIVE_FILE);
5832 if (can_reclaim_anon_pages(NULL, pgdat->node_id, sc))
5833 inactive_lru_pages += node_page_state(pgdat, NR_INACTIVE_ANON);
5834
5835 return inactive_lru_pages > pages_for_compaction;
5836 }
5837
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] only message in thread