lkml.org 
[lkml]   [2016]   [Aug]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] mm: kmemleak: Avoid using __va() on addresses that don't have a lowmem mapping
    Hi Catalin,

    [auto build test ERROR on mmotm/master]
    [also build test ERROR on v4.8-rc2 next-20160816]
    [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

    url: https://github.com/0day-ci/linux/commits/Catalin-Marinas/mm-kmemleak-Avoid-using-__va-on-addresses-that-don-t-have-a-lowmem-mapping/20160816-232733
    base: git://git.cmpxchg.org/linux-mmotm.git master
    config: mn10300-asb2364_defconfig (attached as .config)
    compiler: am33_2.0-linux-gcc (GCC) 4.9.0
    reproduce:
    wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
    chmod +x ~/bin/make.cross
    # save the attached .config to linux build tree
    make.cross ARCH=mn10300

    All error/warnings (new ones prefixed by >>):

    In file included from include/linux/kmemleak.h:24:0,
    from include/linux/slab.h:117,
    from arch/mn10300/include/asm/pgtable.h:33,
    from mm/init-mm.c:9:
    include/linux/mm.h: In function 'is_vmalloc_addr':
    >> include/linux/mm.h:486:17: error: 'VMALLOC_START' undeclared (first use in this function)
    return addr >= VMALLOC_START && addr < VMALLOC_END;
    ^
    include/linux/mm.h:486:17: note: each undeclared identifier is reported only once for each function it appears in
    >> include/linux/mm.h:486:41: error: 'VMALLOC_END' undeclared (first use in this function)
    return addr >= VMALLOC_START && addr < VMALLOC_END;
    ^
    include/linux/mm.h: In function 'maybe_mkwrite':
    >> include/linux/mm.h:624:3: error: implicit declaration of function 'pte_mkwrite' [-Werror=implicit-function-declaration]
    pte = pte_mkwrite(pte);
    ^
    >> include/linux/mm.h:624:7: error: incompatible types when assigning to type 'pte_t' from type 'int'
    pte = pte_mkwrite(pte);
    ^
    In file included from include/linux/kmemleak.h:24:0,
    from include/linux/slab.h:117,
    from arch/mn10300/include/asm/pgtable.h:33,
    from mm/init-mm.c:9:
    include/linux/mm.h: In function 'pgtable_init':
    >> include/linux/mm.h:1690:2: error: implicit declaration of function 'pgtable_cache_init' [-Werror=implicit-function-declaration]
    pgtable_cache_init();
    ^
    In file included from mm/init-mm.c:9:0:
    arch/mn10300/include/asm/pgtable.h: At top level:
    >> arch/mn10300/include/asm/pgtable.h:47:13: warning: conflicting types for 'pgtable_cache_init'
    extern void pgtable_cache_init(void);
    ^
    In file included from include/linux/kmemleak.h:24:0,
    from include/linux/slab.h:117,
    from arch/mn10300/include/asm/pgtable.h:33,
    from mm/init-mm.c:9:
    include/linux/mm.h:1690:2: note: previous implicit declaration of 'pgtable_cache_init' was here
    pgtable_cache_init();
    ^
    In file included from mm/init-mm.c:9:0:
    >> arch/mn10300/include/asm/pgtable.h:272:21: error: conflicting types for 'pte_mkwrite'
    static inline pte_t pte_mkwrite(pte_t pte)
    ^
    In file included from include/linux/kmemleak.h:24:0,
    from include/linux/slab.h:117,
    from arch/mn10300/include/asm/pgtable.h:33,
    from mm/init-mm.c:9:
    include/linux/mm.h:624:9: note: previous implicit declaration of 'pte_mkwrite' was here
    pte = pte_mkwrite(pte);
    ^
    cc1: some warnings being treated as errors

    vim +/VMALLOC_START +486 include/linux/mm.h

    0738c4bb8 Paul Mundt 2008-03-12 480 */
    bb00a789e Yaowei Bai 2016-05-19 481 static inline bool is_vmalloc_addr(const void *x)
    9e2779fa2 Christoph Lameter 2008-02-04 482 {
    0738c4bb8 Paul Mundt 2008-03-12 483 #ifdef CONFIG_MMU
    9e2779fa2 Christoph Lameter 2008-02-04 484 unsigned long addr = (unsigned long)x;
    9e2779fa2 Christoph Lameter 2008-02-04 485
    9e2779fa2 Christoph Lameter 2008-02-04 @486 return addr >= VMALLOC_START && addr < VMALLOC_END;
    0738c4bb8 Paul Mundt 2008-03-12 487 #else
    bb00a789e Yaowei Bai 2016-05-19 488 return false;
    8ca3ed87d David Howells 2008-02-23 489 #endif
    0738c4bb8 Paul Mundt 2008-03-12 490 }
    81ac3ad90 KAMEZAWA Hiroyuki 2009-09-22 491 #ifdef CONFIG_MMU
    81ac3ad90 KAMEZAWA Hiroyuki 2009-09-22 492 extern int is_vmalloc_or_module_addr(const void *x);
    81ac3ad90 KAMEZAWA Hiroyuki 2009-09-22 493 #else
    934831d06 David Howells 2009-09-24 494 static inline int is_vmalloc_or_module_addr(const void *x)
    81ac3ad90 KAMEZAWA Hiroyuki 2009-09-22 495 {
    81ac3ad90 KAMEZAWA Hiroyuki 2009-09-22 496 return 0;
    81ac3ad90 KAMEZAWA Hiroyuki 2009-09-22 497 }
    81ac3ad90 KAMEZAWA Hiroyuki 2009-09-22 498 #endif
    9e2779fa2 Christoph Lameter 2008-02-04 499
    39f1f78d5 Al Viro 2014-05-06 500 extern void kvfree(const void *addr);
    39f1f78d5 Al Viro 2014-05-06 501
    53f9263ba Kirill A. Shutemov 2016-01-15 502 static inline atomic_t *compound_mapcount_ptr(struct page *page)
    53f9263ba Kirill A. Shutemov 2016-01-15 503 {
    53f9263ba Kirill A. Shutemov 2016-01-15 504 return &page[1].compound_mapcount;
    53f9263ba Kirill A. Shutemov 2016-01-15 505 }
    53f9263ba Kirill A. Shutemov 2016-01-15 506
    53f9263ba Kirill A. Shutemov 2016-01-15 507 static inline int compound_mapcount(struct page *page)
    53f9263ba Kirill A. Shutemov 2016-01-15 508 {
    5f527c2b3 Andrea Arcangeli 2016-05-20 509 VM_BUG_ON_PAGE(!PageCompound(page), page);
    53f9263ba Kirill A. Shutemov 2016-01-15 510 page = compound_head(page);
    53f9263ba Kirill A. Shutemov 2016-01-15 511 return atomic_read(compound_mapcount_ptr(page)) + 1;
    53f9263ba Kirill A. Shutemov 2016-01-15 512 }
    53f9263ba Kirill A. Shutemov 2016-01-15 513
    ccaafd7fd Joonsoo Kim 2015-02-10 514 /*
    70b50f94f Andrea Arcangeli 2011-11-02 515 * The atomic page->_mapcount, starts from -1: so that transitions
    70b50f94f Andrea Arcangeli 2011-11-02 516 * both from it and to it can be tracked, using atomic_inc_and_test
    70b50f94f Andrea Arcangeli 2011-11-02 517 * and atomic_add_negative(-1).
    70b50f94f Andrea Arcangeli 2011-11-02 518 */
    22b751c3d Mel Gorman 2013-02-22 519 static inline void page_mapcount_reset(struct page *page)
    70b50f94f Andrea Arcangeli 2011-11-02 520 {
    70b50f94f Andrea Arcangeli 2011-11-02 521 atomic_set(&(page)->_mapcount, -1);
    70b50f94f Andrea Arcangeli 2011-11-02 522 }
    70b50f94f Andrea Arcangeli 2011-11-02 523
    b20ce5e03 Kirill A. Shutemov 2016-01-15 524 int __page_mapcount(struct page *page);
    b20ce5e03 Kirill A. Shutemov 2016-01-15 525
    70b50f94f Andrea Arcangeli 2011-11-02 526 static inline int page_mapcount(struct page *page)
    70b50f94f Andrea Arcangeli 2011-11-02 527 {
    1d148e218 Wang, Yalin 2015-02-11 528 VM_BUG_ON_PAGE(PageSlab(page), page);
    53f9263ba Kirill A. Shutemov 2016-01-15 529
    b20ce5e03 Kirill A. Shutemov 2016-01-15 530 if (unlikely(PageCompound(page)))
    b20ce5e03 Kirill A. Shutemov 2016-01-15 531 return __page_mapcount(page);
    b20ce5e03 Kirill A. Shutemov 2016-01-15 532 return atomic_read(&page->_mapcount) + 1;
    53f9263ba Kirill A. Shutemov 2016-01-15 533 }
    b20ce5e03 Kirill A. Shutemov 2016-01-15 534
    b20ce5e03 Kirill A. Shutemov 2016-01-15 535 #ifdef CONFIG_TRANSPARENT_HUGEPAGE
    b20ce5e03 Kirill A. Shutemov 2016-01-15 536 int total_mapcount(struct page *page);
    6d0a07edd Andrea Arcangeli 2016-05-12 537 int page_trans_huge_mapcount(struct page *page, int *total_mapcount);
    b20ce5e03 Kirill A. Shutemov 2016-01-15 538 #else
    b20ce5e03 Kirill A. Shutemov 2016-01-15 539 static inline int total_mapcount(struct page *page)
    b20ce5e03 Kirill A. Shutemov 2016-01-15 540 {
    b20ce5e03 Kirill A. Shutemov 2016-01-15 541 return page_mapcount(page);
    70b50f94f Andrea Arcangeli 2011-11-02 542 }
    6d0a07edd Andrea Arcangeli 2016-05-12 543 static inline int page_trans_huge_mapcount(struct page *page,
    6d0a07edd Andrea Arcangeli 2016-05-12 544 int *total_mapcount)
    6d0a07edd Andrea Arcangeli 2016-05-12 545 {
    6d0a07edd Andrea Arcangeli 2016-05-12 546 int mapcount = page_mapcount(page);
    6d0a07edd Andrea Arcangeli 2016-05-12 547 if (total_mapcount)
    6d0a07edd Andrea Arcangeli 2016-05-12 548 *total_mapcount = mapcount;
    6d0a07edd Andrea Arcangeli 2016-05-12 549 return mapcount;
    6d0a07edd Andrea Arcangeli 2016-05-12 550 }
    b20ce5e03 Kirill A. Shutemov 2016-01-15 551 #endif
    70b50f94f Andrea Arcangeli 2011-11-02 552
    b49af68ff Christoph Lameter 2007-05-06 553 static inline struct page *virt_to_head_page(const void *x)
    b49af68ff Christoph Lameter 2007-05-06 554 {
    b49af68ff Christoph Lameter 2007-05-06 555 struct page *page = virt_to_page(x);
    ccaafd7fd Joonsoo Kim 2015-02-10 556
    1d798ca3f Kirill A. Shutemov 2015-11-06 557 return compound_head(page);
    b49af68ff Christoph Lameter 2007-05-06 558 }
    b49af68ff Christoph Lameter 2007-05-06 559
    ddc58f27f Kirill A. Shutemov 2016-01-15 560 void __put_page(struct page *page);
    ddc58f27f Kirill A. Shutemov 2016-01-15 561
    1d7ea7324 Alexander Zarochentsev 2006-08-13 562 void put_pages_list(struct list_head *pages);
    ^1da177e4 Linus Torvalds 2005-04-16 563
    8dfcc9ba2 Nick Piggin 2006-03-22 564 void split_page(struct page *page, unsigned int order);
    8dfcc9ba2 Nick Piggin 2006-03-22 565
    ^1da177e4 Linus Torvalds 2005-04-16 566 /*
    33f2ef89f Andy Whitcroft 2006-12-06 567 * Compound pages have a destructor function. Provide a
    33f2ef89f Andy Whitcroft 2006-12-06 568 * prototype for that function and accessor functions.
    f1e61557f Kirill A. Shutemov 2015-11-06 569 * These are _only_ valid on the head of a compound page.
    33f2ef89f Andy Whitcroft 2006-12-06 570 */
    f1e61557f Kirill A. Shutemov 2015-11-06 571 typedef void compound_page_dtor(struct page *);
    f1e61557f Kirill A. Shutemov 2015-11-06 572
    f1e61557f Kirill A. Shutemov 2015-11-06 573 /* Keep the enum in sync with compound_page_dtors array in mm/page_alloc.c */
    f1e61557f Kirill A. Shutemov 2015-11-06 574 enum compound_dtor_id {
    f1e61557f Kirill A. Shutemov 2015-11-06 575 NULL_COMPOUND_DTOR,
    f1e61557f Kirill A. Shutemov 2015-11-06 576 COMPOUND_PAGE_DTOR,
    f1e61557f Kirill A. Shutemov 2015-11-06 577 #ifdef CONFIG_HUGETLB_PAGE
    f1e61557f Kirill A. Shutemov 2015-11-06 578 HUGETLB_PAGE_DTOR,
    f1e61557f Kirill A. Shutemov 2015-11-06 579 #endif
    9a982250f Kirill A. Shutemov 2016-01-15 580 #ifdef CONFIG_TRANSPARENT_HUGEPAGE
    9a982250f Kirill A. Shutemov 2016-01-15 581 TRANSHUGE_PAGE_DTOR,
    9a982250f Kirill A. Shutemov 2016-01-15 582 #endif
    f1e61557f Kirill A. Shutemov 2015-11-06 583 NR_COMPOUND_DTORS,
    f1e61557f Kirill A. Shutemov 2015-11-06 584 };
    f1e61557f Kirill A. Shutemov 2015-11-06 585 extern compound_page_dtor * const compound_page_dtors[];
    33f2ef89f Andy Whitcroft 2006-12-06 586
    33f2ef89f Andy Whitcroft 2006-12-06 587 static inline void set_compound_page_dtor(struct page *page,
    f1e61557f Kirill A. Shutemov 2015-11-06 588 enum compound_dtor_id compound_dtor)
    33f2ef89f Andy Whitcroft 2006-12-06 589 {
    f1e61557f Kirill A. Shutemov 2015-11-06 590 VM_BUG_ON_PAGE(compound_dtor >= NR_COMPOUND_DTORS, page);
    f1e61557f Kirill A. Shutemov 2015-11-06 591 page[1].compound_dtor = compound_dtor;
    33f2ef89f Andy Whitcroft 2006-12-06 592 }
    33f2ef89f Andy Whitcroft 2006-12-06 593
    33f2ef89f Andy Whitcroft 2006-12-06 594 static inline compound_page_dtor *get_compound_page_dtor(struct page *page)
    33f2ef89f Andy Whitcroft 2006-12-06 595 {
    f1e61557f Kirill A. Shutemov 2015-11-06 596 VM_BUG_ON_PAGE(page[1].compound_dtor >= NR_COMPOUND_DTORS, page);
    f1e61557f Kirill A. Shutemov 2015-11-06 597 return compound_page_dtors[page[1].compound_dtor];
    33f2ef89f Andy Whitcroft 2006-12-06 598 }
    33f2ef89f Andy Whitcroft 2006-12-06 599
    d00181b96 Kirill A. Shutemov 2015-11-06 600 static inline unsigned int compound_order(struct page *page)
    d85f33855 Christoph Lameter 2007-05-06 601 {
    6d7779538 Christoph Lameter 2007-05-06 602 if (!PageHead(page))
    d85f33855 Christoph Lameter 2007-05-06 603 return 0;
    e4b294c2d Kirill A. Shutemov 2015-02-11 604 return page[1].compound_order;
    d85f33855 Christoph Lameter 2007-05-06 605 }
    d85f33855 Christoph Lameter 2007-05-06 606
    f1e61557f Kirill A. Shutemov 2015-11-06 607 static inline void set_compound_order(struct page *page, unsigned int order)
    d85f33855 Christoph Lameter 2007-05-06 608 {
    e4b294c2d Kirill A. Shutemov 2015-02-11 609 page[1].compound_order = order;
    d85f33855 Christoph Lameter 2007-05-06 610 }
    d85f33855 Christoph Lameter 2007-05-06 611
    9a982250f Kirill A. Shutemov 2016-01-15 612 void free_compound_page(struct page *page);
    9a982250f Kirill A. Shutemov 2016-01-15 613
    3dece370e Michal Simek 2011-01-21 614 #ifdef CONFIG_MMU
    33f2ef89f Andy Whitcroft 2006-12-06 615 /*
    14fd403f2 Andrea Arcangeli 2011-01-13 616 * Do pte_mkwrite, but only if the vma says VM_WRITE. We do this when
    14fd403f2 Andrea Arcangeli 2011-01-13 617 * servicing faults for write access. In the normal case, do always want
    14fd403f2 Andrea Arcangeli 2011-01-13 618 * pte_mkwrite. But get_user_pages can cause write faults for mappings
    14fd403f2 Andrea Arcangeli 2011-01-13 619 * that do not have writing enabled, when used by access_process_vm.
    14fd403f2 Andrea Arcangeli 2011-01-13 620 */
    14fd403f2 Andrea Arcangeli 2011-01-13 621 static inline pte_t maybe_mkwrite(pte_t pte, struct vm_area_struct *vma)
    14fd403f2 Andrea Arcangeli 2011-01-13 622 {
    14fd403f2 Andrea Arcangeli 2011-01-13 623 if (likely(vma->vm_flags & VM_WRITE))
    14fd403f2 Andrea Arcangeli 2011-01-13 @624 pte = pte_mkwrite(pte);
    14fd403f2 Andrea Arcangeli 2011-01-13 625 return pte;
    14fd403f2 Andrea Arcangeli 2011-01-13 626 }
    8c6e50b02 Kirill A. Shutemov 2014-04-07 627

    :::::: The code at line 486 was first introduced by commit
    :::::: 9e2779fa281cfda13ac060753d674bbcaa23367e is_vmalloc_addr(): Check if an address is within the vmalloc boundaries

    :::::: TO: Christoph Lameter <clameter@sgi.com>
    :::::: CC: Linus Torvalds <torvalds@woody.linux-foundation.org>

    ---
    0-DAY kernel test infrastructure Open Source Technology Center
    https://lists.01.org/pipermail/kbuild-all Intel Corporation
    [unhandled content-type:application/octet-stream]
    \
     
     \ /
      Last update: 2016-09-17 09:57    [W:3.482 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site