Re: [PATCH v2 3/3] mm: Add build time sanity chcek for struct page size

2018-09-11 Thread Baoquan He
On 09/10/18 at 09:41pm, kbuild test robot wrote: >include/linux/build_bug.h:69:2: note: in expansion of macro > 'BUILD_BUG_ON_MSG' > BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed: " #condition) > ^~~~ > >> mm/page_alloc.c:6852:2: note: in expansion of macro 'BUILD_BUG_

Re: [PATCH v2 3/3] mm: Add build time sanity chcek for struct page size

2018-09-10 Thread kbuild test robot
Hi Baoquan, I love your patch! Yet something to improve: [auto build test ERROR on tip/auto-latest] [also build test ERROR on v4.19-rc2 next-20180906] [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/com

[PATCH v2 3/3] mm: Add build time sanity chcek for struct page size

2018-09-09 Thread Baoquan He
Size of struct page might be larger than 64 bytes if debug options enabled, or fields added for debugging intentionally. Yet an upper limit need be added at build time to trigger an alert in case the size is too big to boot up system, warning people to check if it's be done on purpose in advance.