forge/linux-uconsole
1
0
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions

ANDROID: mm: compaction: fix isolate_and_split_free_page() redefinition

Browse source 
Guard isolate_and_split_free_page() with CONFIG_COMPACTION. This fixes
the follwoing build error as the function collides with its inline stub
from the header file:

mm/compaction.c:766:15: error: redefinition of ‘isolate_and_split_free_page’
  766 | unsigned long isolate_and_split_free_page(struct page *page,
      |               ^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from mm/compaction.c:14:
./include/linux/compaction.h:241:29: note: previous definition of ‘isolate_and_split_free_page’ was here
  241 | static inline unsigned long isolate_and_split_free_page(struct page *page,
      |                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~

Bug: 186243330
Fixes: 8cd9aa93b7 ("ANDROID: implement wrapper for reverse migration")
Reported-by: kernelci.org bot <bot@kernelci.org>
Signed-off-by: Carlos Llamas <cmllamas@google.com>
Change-Id: Ie8f3fedcc9d4af5cfdcfd5829377671745ab77d6
This commit is contained in:
Carlos Llamas 2021-04-23 19:20:54 +00:00
commit 48b3f756ed
1 changed files with 2 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
mm/compaction.c
View file

@ -763,6 +763,7 @@ isolate_freepages_range(struct compact_control *cc,
return pfn;
}
#ifdef CONFIG_COMPACTION
unsigned long isolate_and_split_free_page(struct page *page,
struct list_head *list)
{
@ -785,6 +786,7 @@ unsigned long isolate_and_split_free_page(struct page *page,
return isolated;
}
EXPORT_SYMBOL_GPL(isolate_and_split_free_page);
#endif
/* Similar to reclaim, but different enough that they don't share logic */
static bool too_many_isolated(pg_data_t *pgdat)