On May 11, 2018 1:03:04 AM GMT+03:00, Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote: >On 05/10/2018 01:48 PM, Andrew Morton wrote: >> On Thu, 10 May 2018 09:37:51 -0700 Randy Dunlap ><rdunlap@xxxxxxxxxxxxx> wrote: >> >>> On 05/10/2018 12:28 AM, Stephen Rothwell wrote: >>>> Hi all, >>>> >>>> Changes since 20180509: >>>> >>> >>> on i386: >>> >>> ../mm/ksm.c: In function 'try_to_merge_one_page': >>> ../mm/ksm.c:1244:4: error: implicit declaration of function >'set_page_stable_node' [-Werror=implicit-function-declaration] >>> set_page_stable_node(page, NULL); Oops, missed that, sorry. >> Thanks. >> >> From: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> >> Subject: mm-ksm-move-page_stable_node-from-ksmh-to-ksmc-fix >> >> fix SYSFS=n build >> >> Cc: Andrea Arcangeli <aarcange@xxxxxxxxxx> >> Cc: Mike Rapoport <rppt@xxxxxxxxxxxxxxxxxx> >> Cc: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> > >Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> >Reported-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> >Tested-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> Acked-by: Mike Rapoport <rppt@xxxxxxxxxxxxxxxxxx> >> Cc: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> >> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> >> --- >> >> mm/ksm.c | 9 ++++----- >> 1 file changed, 4 insertions(+), 5 deletions(-) >> >> diff -puN >include/linux/ksm.h~mm-ksm-move-page_stable_node-from-ksmh-to-ksmc-fix >include/linux/ksm.h >> diff -puN mm/ksm.c~mm-ksm-move-page_stable_node-from-ksmh-to-ksmc-fix >mm/ksm.c >> --- a/mm/ksm.c~mm-ksm-move-page_stable_node-from-ksmh-to-ksmc-fix >> +++ a/mm/ksm.c >> @@ -823,11 +823,6 @@ static int unmerge_ksm_pages(struct vm_a >> return err; >> } >> >> -#ifdef CONFIG_SYSFS >> -/* >> - * Only called through the sysfs control interface: >> - */ >> - >> static inline struct stable_node *page_stable_node(struct page >*page) >> { >> return PageKsm(page) ? page_rmapping(page) : NULL; >> @@ -839,6 +834,10 @@ static inline void set_page_stable_node( >> page->mapping = (void *)((unsigned long)stable_node | >PAGE_MAPPING_KSM); >> } >> >> +#ifdef CONFIG_SYSFS >> +/* >> + * Only called through the sysfs control interface: >> + */ >> static int remove_stable_node(struct stable_node *stable_node) >> { >> struct page *page; >> _