* [linux-next:master 11896/11991] include/linux/page_idle.h:49:19: warning: unused variable 'page_ext'
@ 2016-05-24 10:48 kbuild test robot
2016-05-24 19:44 ` Andrew Morton
0 siblings, 1 reply; 3+ messages in thread
From: kbuild test robot @ 2016-05-24 10:48 UTC (permalink / raw)
Cc: kbuild-all, Yang Shi, Andrew Morton, Linux Memory Management List
[-- Attachment #1: Type: text/plain, Size: 29085 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: 66c198deda3725c57939c6cdaf2c9f5375cd79ad
commit: 186ba1a848cef542bdf7c881f863783e9e7a91df [11896/11991] mm-check-the-return-value-of-lookup_page_ext-for-all-call-sites-checkpatch-fixes
config: i386-randconfig-h1-05241552 (attached as .config)
compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
reproduce:
git checkout 186ba1a848cef542bdf7c881f863783e9e7a91df
# save the attached .config to linux build tree
make ARCH=i386
All warnings (new ones prefixed by >>):
^~~~~~~~~~~~~~~~~~~~~~~~
fs/proc/task_mmu.c:408:30: warning: unused variable 'proc_pid_maps_operations' [-Wunused-variable]
const struct file_operations proc_pid_maps_operations = {
^~~~~~~~~~~~~~~~~~~~~~~~
In file included from fs/proc/task_mmu.c:22:0:
fs/proc/internal.h:299:37: warning: unused variable 'proc_pagemap_operations' [-Wunused-variable]
extern const struct file_operations proc_pagemap_operations;
^~~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:298:37: warning: unused variable 'proc_clear_refs_operations' [-Wunused-variable]
extern const struct file_operations proc_clear_refs_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:297:37: warning: unused variable 'proc_tid_smaps_operations' [-Wunused-variable]
extern const struct file_operations proc_tid_smaps_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:296:37: warning: unused variable 'proc_pid_smaps_operations' [-Wunused-variable]
extern const struct file_operations proc_pid_smaps_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:295:37: warning: unused variable 'proc_tid_numa_maps_operations' [-Wunused-variable]
extern const struct file_operations proc_tid_numa_maps_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:294:37: warning: unused variable 'proc_pid_numa_maps_operations' [-Wunused-variable]
extern const struct file_operations proc_pid_numa_maps_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:270:30: warning: unused variable 'proc_root' [-Wunused-variable]
extern struct proc_dir_entry proc_root;
^~~~~~~~~
fs/proc/internal.h:228:38: warning: unused variable 'proc_net_inode_operations' [-Wunused-variable]
extern const struct inode_operations proc_net_inode_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:227:37: warning: unused variable 'proc_net_operations' [-Wunused-variable]
extern const struct file_operations proc_net_operations;
^~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:222:37: warning: unused variable 'proc_ns_dir_operations' [-Wunused-variable]
extern const struct file_operations proc_ns_dir_operations;
^~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:221:38: warning: unused variable 'proc_ns_dir_inode_operations' [-Wunused-variable]
extern const struct inode_operations proc_ns_dir_inode_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:211:38: warning: unused variable 'proc_pid_link_inode_operations' [-Wunused-variable]
extern const struct inode_operations proc_pid_link_inode_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:209:38: warning: unused variable 'proc_link_inode_operations' [-Wunused-variable]
extern const struct inode_operations proc_link_inode_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:162:39: warning: unused variable 'pid_dentry_operations' [-Wunused-variable]
extern const struct dentry_operations pid_dentry_operations;
^~~~~~~~~~~~~~~~~~~~~
fs/proc/internal.h:148:37: warning: unused variable 'proc_tid_children_operations' [-Wunused-variable]
extern const struct file_operations proc_tid_children_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from fs/proc/internal.h:16:0,
from fs/proc/task_mmu.c:22:
include/linux/binfmts.h:105:12: warning: unused variable 'suid_dumpable' [-Wunused-variable]
extern int suid_dumpable;
^~~~~~~~~~~~~
In file included from fs/proc/internal.h:13:0,
from fs/proc/task_mmu.c:22:
include/linux/proc_ns.h:29:40: warning: unused variable 'cgroupns_operations' [-Wunused-variable]
extern const struct proc_ns_operations cgroupns_operations;
^~~~~~~~~~~~~~~~~~~
include/linux/proc_ns.h:28:40: warning: unused variable 'mntns_operations' [-Wunused-variable]
extern const struct proc_ns_operations mntns_operations;
^~~~~~~~~~~~~~~~
include/linux/proc_ns.h:27:40: warning: unused variable 'userns_operations' [-Wunused-variable]
extern const struct proc_ns_operations userns_operations;
^~~~~~~~~~~~~~~~~
include/linux/proc_ns.h:26:40: warning: unused variable 'pidns_operations' [-Wunused-variable]
extern const struct proc_ns_operations pidns_operations;
^~~~~~~~~~~~~~~~
include/linux/proc_ns.h:25:40: warning: unused variable 'ipcns_operations' [-Wunused-variable]
extern const struct proc_ns_operations ipcns_operations;
^~~~~~~~~~~~~~~~
include/linux/proc_ns.h:24:40: warning: unused variable 'utsns_operations' [-Wunused-variable]
extern const struct proc_ns_operations utsns_operations;
^~~~~~~~~~~~~~~~
include/linux/proc_ns.h:23:40: warning: unused variable 'netns_operations' [-Wunused-variable]
extern const struct proc_ns_operations netns_operations;
^~~~~~~~~~~~~~~~
In file included from fs/proc/task_mmu.c:19:0:
arch/x86/include/asm/elf.h:364:28: warning: unused variable 'va_align' [-Wunused-variable]
extern struct va_alignment va_align;
^~~~~~~~
In file included from arch/x86/include/asm/elf.h:94:0,
from fs/proc/task_mmu.c:19:
arch/x86/include/asm/desc.h:40:18: warning: unused variable 'debug_idt_table' [-Wunused-variable]
extern gate_desc debug_idt_table[];
^~~~~~~~~~~~~~~
arch/x86/include/asm/desc.h:39:24: warning: unused variable 'debug_idt_descr' [-Wunused-variable]
extern struct desc_ptr debug_idt_descr;
^~~~~~~~~~~~~~~
In file included from fs/proc/task_mmu.c:19:0:
arch/x86/include/asm/elf.h:82:21: warning: unused variable 'vdso32_enabled' [-Wunused-variable]
extern unsigned int vdso32_enabled;
^~~~~~~~~~~~~~
In file included from arch/x86/include/asm/elf.h:76:0,
from fs/proc/task_mmu.c:19:
arch/x86/include/asm/vdso.h:39:32: warning: unused variable 'vdso_image_32' [-Wunused-variable]
extern const struct vdso_image vdso_image_32;
^~~~~~~~~~~~~
In file included from fs/proc/task_mmu.c:16:0:
>> include/linux/page_idle.h:49:19: warning: unused variable 'page_ext' [-Wunused-variable]
struct page_ext *page_ext = lookup_page_ext(page);
^~~~~~~~
fs/proc/task_mmu.c:420:1: warning: no return statement in function returning non-void [-Wreturn-type]
};
^
At top level:
fs/proc/task_mmu.c:91:15: warning: 'task_statm' defined but not used [-Wunused-function]
unsigned long task_statm(struct mm_struct *mm,
^~~~~~~~~~
fs/proc/task_mmu.c:86:15: warning: 'task_vsize' defined but not used [-Wunused-function]
unsigned long task_vsize(struct mm_struct *mm)
^~~~~~~~~~
fs/proc/task_mmu.c:24:6: warning: 'task_mem' defined but not used [-Wunused-function]
void task_mem(struct seq_file *m, struct mm_struct *mm)
^~~~~~~~
cc1: some warnings being treated as errors
--
from include/trace/events/pagemap.h:87,
from mm/swap.c:41:
arch/x86/include/asm/hw_breakpoint.h:74:19: warning: unused variable 'perf_ops_bp' [-Wunused-variable]
extern struct pmu perf_ops_bp;
^~~~~~~~~~~
In file included from arch/x86/include/asm/perf_event.h:246:0,
from include/linux/perf_event.h:24,
from include/linux/trace_events.h:9,
from include/trace/trace_events.h:19,
from include/trace/define_trace.h:95,
from include/trace/events/pagemap.h:87,
from mm/swap.c:41:
arch/x86/include/asm/stacktrace.h:92:21: warning: unused variable 'code_bytes' [-Wunused-variable]
extern unsigned int code_bytes;
^~~~~~~~~~
arch/x86/include/asm/stacktrace.h:12:12: warning: unused variable 'kstack_depth_to_print' [-Wunused-variable]
extern int kstack_depth_to_print;
^~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/ptrace.h:8:0,
from arch/x86/include/asm/stacktrace.h:10,
from arch/x86/include/asm/perf_event.h:246,
from include/linux/perf_event.h:24,
from include/linux/trace_events.h:9,
from include/trace/trace_events.h:19,
from include/trace/define_trace.h:95,
from include/trace/events/pagemap.h:87,
from mm/swap.c:41:
include/linux/pid_namespace.h:50:29: warning: unused variable 'init_pid_ns' [-Wunused-variable]
extern struct pid_namespace init_pid_ns;
^~~~~~~~~~~
In file included from include/linux/ring_buffer.h:7:0,
from include/linux/trace_events.h:5,
from include/trace/trace_events.h:19,
from include/trace/define_trace.h:95,
from include/trace/events/pagemap.h:87,
from mm/swap.c:41:
include/linux/poll.h:14:25: warning: unused variable 'epoll_table' [-Wunused-variable]
extern struct ctl_table epoll_table[]; /* for sysctl */
^~~~~~~~~~~
In file included from include/trace/events/pagemap.h:7:0,
from mm/swap.c:41:
include/linux/tracepoint.h:235:20: warning: unused variable '__tracepoint_mm_lru_activate' [-Wunused-variable]
struct tracepoint __tracepoint_##name \
^
include/linux/tracepoint.h:243:2: note: in expansion of macro 'DEFINE_TRACE_FN'
DEFINE_TRACE_FN(name, NULL, NULL);
^~~~~~~~~~~~~~~
include/trace/define_trace.h:27:2: note: in expansion of macro 'DEFINE_TRACE'
DEFINE_TRACE(name)
^~~~~~~~~~~~
include/trace/events/pagemap.h:63:1: note: in expansion of macro 'TRACE_EVENT'
TRACE_EVENT(mm_lru_activate,
^~~~~~~~~~~
include/linux/tracepoint.h:235:20: warning: unused variable '__tracepoint_mm_lru_insertion' [-Wunused-variable]
struct tracepoint __tracepoint_##name \
^
include/linux/tracepoint.h:243:2: note: in expansion of macro 'DEFINE_TRACE_FN'
DEFINE_TRACE_FN(name, NULL, NULL);
^~~~~~~~~~~~~~~
include/trace/define_trace.h:27:2: note: in expansion of macro 'DEFINE_TRACE'
DEFINE_TRACE(name)
^~~~~~~~~~~~
include/trace/events/pagemap.h:27:1: note: in expansion of macro 'TRACE_EVENT'
TRACE_EVENT(mm_lru_insertion,
^~~~~~~~~~~
In file included from mm/swap.c:38:0:
mm/internal.h:485:39: warning: unused variable 'gfpflag_names' [-Wunused-variable]
extern const struct trace_print_flags gfpflag_names[];
^~~~~~~~~~~~~
mm/internal.h:484:39: warning: unused variable 'vmaflag_names' [-Wunused-variable]
extern const struct trace_print_flags vmaflag_names[];
^~~~~~~~~~~~~
mm/internal.h:483:39: warning: unused variable 'pageflag_names' [-Wunused-variable]
extern const struct trace_print_flags pageflag_names[];
^~~~~~~~~~~~~~
mm/internal.h:443:12: warning: unused variable 'hwpoison_filter_enable' [-Wunused-variable]
extern u32 hwpoison_filter_enable;
^~~~~~~~~~~~~~~~~~~~~~
mm/internal.h:442:12: warning: unused variable 'hwpoison_filter_memcg' [-Wunused-variable]
extern u64 hwpoison_filter_memcg;
^~~~~~~~~~~~~~~~~~~~~
mm/internal.h:441:12: warning: unused variable 'hwpoison_filter_flags_value' [-Wunused-variable]
extern u64 hwpoison_filter_flags_value;
^~~~~~~~~~~~~~~~~~~~~~~~~~~
mm/internal.h:440:12: warning: unused variable 'hwpoison_filter_flags_mask' [-Wunused-variable]
extern u64 hwpoison_filter_flags_mask;
^~~~~~~~~~~~~~~~~~~~~~~~~~
mm/internal.h:439:12: warning: unused variable 'hwpoison_filter_dev_minor' [-Wunused-variable]
extern u32 hwpoison_filter_dev_minor;
^~~~~~~~~~~~~~~~~~~~~~~~~
mm/internal.h:438:12: warning: unused variable 'hwpoison_filter_dev_major' [-Wunused-variable]
extern u32 hwpoison_filter_dev_major;
^~~~~~~~~~~~~~~~~~~~~~~~~
mm/internal.h:152:12: warning: unused variable 'user_min_free_kbytes' [-Wunused-variable]
extern int user_min_free_kbytes;
^~~~~~~~~~~~~~~~~~~~
mm/internal.h:71:22: warning: unused variable 'highest_memmap_pfn' [-Wunused-variable]
extern unsigned long highest_memmap_pfn;
^~~~~~~~~~~~~~~~~~
In file included from mm/swap.c:36:0:
>> include/linux/page_idle.h:49:19: warning: unused variable 'page_ext' [-Wunused-variable]
struct page_ext *page_ext = lookup_page_ext(page);
^~~~~~~~
mm/swap.c:971:1: warning: no return statement in function returning non-void [-Wreturn-type]
}
^
At top level:
mm/swap.c:952:13: warning: 'swap_setup' defined but not used [-Wunused-function]
void __init swap_setup(void)
^~~~~~~~~~
mm/swap.c:940:10: warning: 'pagevec_lookup_tag' defined but not used [-Wunused-function]
unsigned pagevec_lookup_tag(struct pagevec *pvec, struct address_space *mapping,
^~~~~~~~~~~~~~~~~~
mm/swap.c:932:10: warning: 'pagevec_lookup' defined but not used [-Wunused-function]
unsigned pagevec_lookup(struct pagevec *pvec, struct address_space *mapping,
^~~~~~~~~~~~~~
mm/swap.c:904:6: warning: 'pagevec_remove_exceptionals' defined but not used [-Wunused-function]
void pagevec_remove_exceptionals(struct pagevec *pvec)
^~~~~~~~~~~~~~~~~~~~~~~~~~~
mm/swap.c:885:10: warning: 'pagevec_lookup_entries' defined but not used [-Wunused-function]
unsigned pagevec_lookup_entries(struct pagevec *pvec,
^~~~~~~~~~~~~~~~~~~~~~
mm/swap.c:859:6: warning: '__pagevec_lru_add' defined but not used [-Wunused-function]
void __pagevec_lru_add(struct pagevec *pvec)
^~~~~~~~~~~~~~~~~
mm/swap.c:801:6: warning: 'lru_add_page_tail' defined but not used [-Wunused-function]
void lru_add_page_tail(struct page *page, struct page *page_tail,
^~~~~~~~~~~~~~~~~
mm/swap.c:791:6: warning: '__pagevec_release' defined but not used [-Wunused-function]
void __pagevec_release(struct pagevec *pvec)
^~~~~~~~~~~~~~~~~
mm/swap.c:670:6: warning: 'lru_add_drain_all' defined but not used [-Wunused-function]
void lru_add_drain_all(void)
^~~~~~~~~~~~~~~~~
mm/swap.c:645:6: warning: 'deactivate_page' defined but not used [-Wunused-function]
void deactivate_page(struct page *page)
^~~~~~~~~~~~~~~
mm/swap.c:619:6: warning: 'deactivate_file_page' defined but not used [-Wunused-function]
void deactivate_file_page(struct page *page)
^~~~~~~~~~~~~~~~~~~~
mm/swap.c:469:6: warning: 'lru_cache_add_active_or_unevictable' defined but not used [-Wunused-function]
void lru_cache_add_active_or_unevictable(struct page *page,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
mm/swap.c:411:6: warning: 'lru_cache_add_file' defined but not used [-Wunused-function]
void lru_cache_add_file(struct page *page)
^~~~~~~~~~~~~~~~~~
mm/swap.c:404:6: warning: 'lru_cache_add_anon' defined but not used [-Wunused-function]
void lru_cache_add_anon(struct page *page)
^~~~~~~~~~~~~~~~~~
mm/swap.c:362:6: warning: 'mark_page_accessed' defined but not used [-Wunused-function]
void mark_page_accessed(struct page *page)
^~~~~~~~~~~~~~~~~~
mm/swap.c:235:6: warning: 'rotate_reclaimable_page' defined but not used [-Wunused-function]
void rotate_reclaimable_page(struct page *page)
^~~~~~~~~~~~~~~~~~~~~~~
mm/swap.c:166:5: warning: 'get_kernel_page' defined but not used [-Wunused-function]
int get_kernel_page(unsigned long start, int write, struct page **pages)
^~~~~~~~~~~~~~~
mm/swap.c:113:6: warning: 'put_pages_list' defined but not used [-Wunused-function]
void put_pages_list(struct list_head *pages)
^~~~~~~~~~~~~~
mm/swap.c:97:6: warning: '__put_page' defined but not used [-Wunused-function]
void __put_page(struct page *page)
^~~~~~~~~~
cc1: some warnings being treated as errors
--
static bool invalid_mkclean_vma(struct vm_area_struct *vma, void *arg)
^~~~~~~~~~~~~~~~~~~
mm/rmap.c:1094:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
void page_move_anon_rmap(struct page *page,
^~~~
mm/rmap.c:1119:13: error: invalid storage class for function '__page_set_anon_rmap'
static void __page_set_anon_rmap(struct page *page,
^~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1148:13: error: invalid storage class for function '__page_check_anon_rmap'
static void __page_check_anon_rmap(struct page *page,
^~~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1284:13: error: invalid storage class for function 'page_remove_file_rmap'
static void page_remove_file_rmap(struct page *page)
^~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1313:13: error: invalid storage class for function 'page_remove_anon_compound_rmap'
static void page_remove_anon_compound_rmap(struct page *page)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1398:17: error: field 'flags' has incomplete type
enum ttu_flags flags;
^~~~~
mm/rmap.c:1405:12: error: invalid storage class for function 'try_to_unmap_one'
static int try_to_unmap_one(struct page *page, struct vm_area_struct *vma,
^~~~~~~~~~~~~~~~
mm/rmap.c: In function 'try_to_unmap_one':
mm/rmap.c:1414:7: error: variable 'flags' has initializer but incomplete type
enum ttu_flags flags = rp->flags;
^~~~~~~~~
mm/rmap.c:1414:17: error: storage size of 'flags' isn't known
enum ttu_flags flags = rp->flags;
^~~~~
mm/rmap.c:1414:17: warning: unused variable 'flags' [-Wunused-variable]
mm/rmap.c: In function 'page_is_young':
mm/rmap.c:1568:13: error: invalid storage class for function 'invalid_migration_vma'
static bool invalid_migration_vma(struct vm_area_struct *vma, void *arg)
^~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1573:12: error: invalid storage class for function 'page_mapcount_is_zero'
static int page_mapcount_is_zero(struct page *page)
^~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1592:52: error: parameter 2 ('flags') has incomplete type
int try_to_unmap(struct page *page, enum ttu_flags flags)
^~~~~
mm/rmap.c:1592:5: error: function declaration isn't a prototype [-Werror=strict-prototypes]
int try_to_unmap(struct page *page, enum ttu_flags flags)
^~~~~~~~~~~~
mm/rmap.c:1631:12: error: invalid storage class for function 'page_not_mapped'
static int page_not_mapped(struct page *page)
^~~~~~~~~~~~~~~
mm/rmap.c:1651:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
int try_to_munlock(struct page *page)
^~~
mm/rmap.c:1682:25: error: invalid storage class for function 'rmap_walk_anon_lock'
static struct anon_vma *rmap_walk_anon_lock(struct page *page,
^~~~~~~~~~~~~~~~~~~
mm/rmap.c:1718:12: error: invalid storage class for function 'rmap_walk_anon'
static int rmap_walk_anon(struct page *page, struct rmap_walk_control *rwc,
^~~~~~~~~~~~~~
mm/rmap.c:1771:12: error: invalid storage class for function 'rmap_walk_file'
static int rmap_walk_file(struct page *page, struct rmap_walk_control *rwc,
^~~~~~~~~~~~~~
mm/rmap.c:1841:13: error: invalid storage class for function '__hugepage_set_anon_rmap'
static void __hugepage_set_anon_rmap(struct page *page,
^~~~~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1878:1: error: expected declaration or statement at end of input
}
^
In file included from mm/rmap.c:69:0:
mm/internal.h:485:39: warning: unused variable 'gfpflag_names' [-Wunused-variable]
extern const struct trace_print_flags gfpflag_names[];
^~~~~~~~~~~~~
mm/internal.h:484:39: warning: unused variable 'vmaflag_names' [-Wunused-variable]
extern const struct trace_print_flags vmaflag_names[];
^~~~~~~~~~~~~
mm/internal.h:483:39: warning: unused variable 'pageflag_names' [-Wunused-variable]
extern const struct trace_print_flags pageflag_names[];
^~~~~~~~~~~~~~
mm/internal.h:443:12: warning: unused variable 'hwpoison_filter_enable' [-Wunused-variable]
extern u32 hwpoison_filter_enable;
^~~~~~~~~~~~~~~~~~~~~~
mm/internal.h:442:12: warning: unused variable 'hwpoison_filter_memcg' [-Wunused-variable]
extern u64 hwpoison_filter_memcg;
^~~~~~~~~~~~~~~~~~~~~
mm/internal.h:441:12: warning: unused variable 'hwpoison_filter_flags_value' [-Wunused-variable]
extern u64 hwpoison_filter_flags_value;
^~~~~~~~~~~~~~~~~~~~~~~~~~~
mm/internal.h:440:12: warning: unused variable 'hwpoison_filter_flags_mask' [-Wunused-variable]
extern u64 hwpoison_filter_flags_mask;
^~~~~~~~~~~~~~~~~~~~~~~~~~
mm/internal.h:439:12: warning: unused variable 'hwpoison_filter_dev_minor' [-Wunused-variable]
extern u32 hwpoison_filter_dev_minor;
^~~~~~~~~~~~~~~~~~~~~~~~~
mm/internal.h:438:12: warning: unused variable 'hwpoison_filter_dev_major' [-Wunused-variable]
extern u32 hwpoison_filter_dev_major;
^~~~~~~~~~~~~~~~~~~~~~~~~
mm/internal.h:152:12: warning: unused variable 'user_min_free_kbytes' [-Wunused-variable]
extern int user_min_free_kbytes;
^~~~~~~~~~~~~~~~~~~~
mm/internal.h:71:22: warning: unused variable 'highest_memmap_pfn' [-Wunused-variable]
extern unsigned long highest_memmap_pfn;
^~~~~~~~~~~~~~~~~~
In file included from mm/rmap.c:63:0:
>> include/linux/page_idle.h:49:19: warning: unused variable 'page_ext' [-Wunused-variable]
struct page_ext *page_ext = lookup_page_ext(page);
^~~~~~~~
mm/rmap.c:1878:1: warning: no return statement in function returning non-void [-Wreturn-type]
}
^
At top level:
mm/rmap.c:1872:6: warning: 'hugepage_add_new_anon_rmap' defined but not used [-Wunused-function]
void hugepage_add_new_anon_rmap(struct page *page,
^~~~~~~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1858:6: warning: 'hugepage_add_anon_rmap' defined but not used [-Wunused-function]
void hugepage_add_anon_rmap(struct page *page,
^~~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1825:5: warning: 'rmap_walk_locked' defined but not used [-Wunused-function]
int rmap_walk_locked(struct page *page, struct rmap_walk_control *rwc)
^~~~~~~~~~~~~~~~
mm/rmap.c:1814:5: warning: 'rmap_walk' defined but not used [-Wunused-function]
int rmap_walk(struct page *page, struct rmap_walk_control *rwc)
^~~~~~~~~
mm/rmap.c:1673:6: warning: '__put_anon_vma' defined but not used [-Wunused-function]
void __put_anon_vma(struct anon_vma *anon_vma)
^~~~~~~~~~~~~~
mm/rmap.c:1651:5: warning: 'try_to_munlock' defined but not used [-Wunused-function]
int try_to_munlock(struct page *page)
^~~~~~~~~~~~~~
mm/rmap.c:1592:5: warning: 'try_to_unmap' defined but not used [-Wunused-function]
int try_to_unmap(struct page *page, enum ttu_flags flags)
^~~~~~~~~~~~
mm/rmap.c:1274:6: warning: 'page_add_file_rmap' defined but not used [-Wunused-function]
void page_add_file_rmap(struct page *page)
^~~~~~~~~~~~~~~~~~
mm/rmap.c:1246:6: warning: 'page_add_new_anon_rmap' defined but not used [-Wunused-function]
void page_add_new_anon_rmap(struct page *page,
^~~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1192:6: warning: 'do_page_add_anon_rmap' defined but not used [-Wunused-function]
void do_page_add_anon_rmap(struct page *page,
^~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:1181:6: warning: 'page_add_anon_rmap' defined but not used [-Wunused-function]
void page_add_anon_rmap(struct page *page,
^~~~~~~~~~~~~~~~~~
mm/rmap.c:1094:6: warning: 'page_move_anon_rmap' defined but not used [-Wunused-function]
void page_move_anon_rmap(struct page *page,
^~~~~~~~~~~~~~~~~~~
mm/rmap.c:1058:5: warning: 'page_mkclean' defined but not used [-Wunused-function]
int page_mkclean(struct page *page)
^~~~~~~~~~~~
mm/rmap.c:968:5: warning: 'page_referenced' defined but not used [-Wunused-function]
int page_referenced(struct page *page,
^~~~~~~~~~~~~~~
mm/rmap.c:771:5: warning: 'page_mapped_in_vma' defined but not used [-Wunused-function]
int page_mapped_in_vma(struct page *page, struct vm_area_struct *vma)
^~~~~~~~~~~~~~~~~~
mm/rmap.c:723:8: warning: '__page_check_address' defined but not used [-Wunused-function]
pte_t *__page_check_address(struct page *page, struct mm_struct *mm,
^~~~~~~~~~~~~~~~~~~~
mm/rmap.c:662:15: warning: 'page_address_in_vma' defined but not used [-Wunused-function]
unsigned long page_address_in_vma(struct page *page, struct vm_area_struct *vma)
^~~~~~~~~~~~~~~~~~~
mm/rmap.c:603:6: warning: 'try_to_unmap_flush_dirty' defined but not used [-Wunused-function]
void try_to_unmap_flush_dirty(void)
^~~~~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:566:6: warning: 'page_unlock_anon_vma_read' defined but not used [-Wunused-function]
void page_unlock_anon_vma_read(struct anon_vma *anon_vma)
^~~~~~~~~~~~~~~~~~~~~~~~~
mm/rmap.c:461:18: warning: 'page_get_anon_vma' defined but not used [-Wunused-function]
struct anon_vma *page_get_anon_vma(struct page *page)
^~~~~~~~~~~~~~~~~
mm/rmap.c:429:13: warning: 'anon_vma_init' defined but not used [-Wunused-function]
void __init anon_vma_init(void)
^~~~~~~~~~~~~
mm/rmap.c:373:6: warning: 'unlink_anon_vmas' defined but not used [-Wunused-function]
void unlink_anon_vmas(struct vm_area_struct *vma)
^~~~~~~~~~~~~~~~
mm/rmap.c:312:5: warning: 'anon_vma_fork' defined but not used [-Wunused-function]
int anon_vma_fork(struct vm_area_struct *vma, struct vm_area_struct *pvma)
^~~~~~~~~~~~~
mm/rmap.c:170:5: warning: 'anon_vma_prepare' defined but not used [-Wunused-function]
int anon_vma_prepare(struct vm_area_struct *vma)
^~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
..
vim +/page_ext +49 include/linux/page_idle.h
33 SetPageIdle(page);
34 }
35
36 static inline void clear_page_idle(struct page *page)
37 {
38 ClearPageIdle(page);
39 }
40 #else /* !CONFIG_64BIT */
41 /*
42 * If there is not enough space to store Idle and Young bits in page flags, use
43 * page ext flags instead.
44 */
45 extern struct page_ext_operations page_idle_ops;
46
47 static inline bool page_is_young(struct page *page)
48 {
> 49 struct page_ext *page_ext = lookup_page_ext(page);
50
51 if (unlikely(!page_ext)
52 return false;
53
54 return test_bit(PAGE_EXT_YOUNG, &page_ext->flags);
55 }
56
57 static inline void set_page_young(struct page *page)
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 22676 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [linux-next:master 11896/11991] include/linux/page_idle.h:49:19: warning: unused variable 'page_ext'
2016-05-24 10:48 [linux-next:master 11896/11991] include/linux/page_idle.h:49:19: warning: unused variable 'page_ext' kbuild test robot
@ 2016-05-24 19:44 ` Andrew Morton
2016-05-25 0:14 ` Fengguang Wu
0 siblings, 1 reply; 3+ messages in thread
From: Andrew Morton @ 2016-05-24 19:44 UTC (permalink / raw)
To: kbuild test robot; +Cc: kbuild-all, Yang Shi, Linux Memory Management List
On Tue, 24 May 2016 18:48:23 +0800 kbuild test robot <fengguang.wu@intel.com> wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head: 66c198deda3725c57939c6cdaf2c9f5375cd79ad
> commit: 186ba1a848cef542bdf7c881f863783e9e7a91df [11896/11991] mm-check-the-return-value-of-lookup_page_ext-for-all-call-sites-checkpatch-fixes
> config: i386-randconfig-h1-05241552 (attached as .config)
> compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
> reproduce:
> git checkout 186ba1a848cef542bdf7c881f863783e9e7a91df
> # save the attached .config to linux build tree
> make ARCH=i386
>
> All warnings (new ones prefixed by >>):
>
> ^~~~~~~~~~~~~~~~~~~~~~~~
> fs/proc/task_mmu.c:408:30: warning: unused variable 'proc_pid_maps_operations' [-Wunused-variable]
> const struct file_operations proc_pid_maps_operations = {
> ^~~~~~~~~~~~~~~~~~~~~~~
Confused. proc_pid_maps_operations is referenced from fs/proc/base.o.
> In file included from fs/proc/task_mmu.c:22:0:
> fs/proc/internal.h:299:37: warning: unused variable 'proc_pagemap_operations' [-Wunused-variable]
> extern const struct file_operations proc_pagemap_operations;
> ^~~~~~~~~~~~~~~~~~~~~~~
Even more confused. Your config has CONFIG_PROC_PAGE_MONITOR=n, so
proc_pagemap_operations doesn't get past the cpp stage.
> In file included from fs/proc/task_mmu.c:16:0:
> >> include/linux/page_idle.h:49:19: warning: unused variable 'page_ext' [-Wunused-variable]
> struct page_ext *page_ext = lookup_page_ext(page);
This is the new one and is presumably caused by the great stream of
missing ')'s in the CONFIG_64BIT=n section of page_idle.h.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [linux-next:master 11896/11991] include/linux/page_idle.h:49:19: warning: unused variable 'page_ext'
2016-05-24 19:44 ` Andrew Morton
@ 2016-05-25 0:14 ` Fengguang Wu
0 siblings, 0 replies; 3+ messages in thread
From: Fengguang Wu @ 2016-05-25 0:14 UTC (permalink / raw)
To: Andrew Morton; +Cc: kbuild-all, Yang Shi, Linux Memory Management List
On Tue, May 24, 2016 at 12:44:57PM -0700, Andrew Morton wrote:
> On Tue, 24 May 2016 18:48:23 +0800 kbuild test robot <fengguang.wu@intel.com> wrote:
>
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > head: 66c198deda3725c57939c6cdaf2c9f5375cd79ad
> > commit: 186ba1a848cef542bdf7c881f863783e9e7a91df [11896/11991] mm-check-the-return-value-of-lookup_page_ext-for-all-call-sites-checkpatch-fixes
> > config: i386-randconfig-h1-05241552 (attached as .config)
> > compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
> > reproduce:
> > git checkout 186ba1a848cef542bdf7c881f863783e9e7a91df
> > # save the attached .config to linux build tree
> > make ARCH=i386
> >
> > All warnings (new ones prefixed by >>):
> >
> > ^~~~~~~~~~~~~~~~~~~~~~~~
> > fs/proc/task_mmu.c:408:30: warning: unused variable 'proc_pid_maps_operations' [-Wunused-variable]
> > const struct file_operations proc_pid_maps_operations = {
> > ^~~~~~~~~~~~~~~~~~~~~~~
>
> Confused. proc_pid_maps_operations is referenced from fs/proc/base.o.
>
> > In file included from fs/proc/task_mmu.c:22:0:
> > fs/proc/internal.h:299:37: warning: unused variable 'proc_pagemap_operations' [-Wunused-variable]
> > extern const struct file_operations proc_pagemap_operations;
> > ^~~~~~~~~~~~~~~~~~~~~~~
>
> Even more confused. Your config has CONFIG_PROC_PAGE_MONITOR=n, so
> proc_pagemap_operations doesn't get past the cpp stage.
They are confusing indeed.
> > In file included from fs/proc/task_mmu.c:16:0:
> > >> include/linux/page_idle.h:49:19: warning: unused variable 'page_ext' [-Wunused-variable]
> > struct page_ext *page_ext = lookup_page_ext(page);
>
> This is the new one and is presumably caused by the great stream of
> missing ')'s in the CONFIG_64BIT=n section of page_idle.h.
That triggered so many noises (pasted below), it may be easier to look
closer at the error/warnings after fixing that issue.
- if (unlikely(!page_ext)
+ if (unlikely(!page_ext))
All the error/warnings magically disappears after adding the missing ')'s!
Thanks,
Fengguang
---
make ARCH=i386 M=fs/proc/
grep -a -F fs/proc/ /tmp/build-err-186ba1a848cef542bdf7c881f863783e9e7a91df-wfg --color
warning: (VIDEO_EM28XX_V4L2) selects VIDEO_MT9V011 which has unmet direct dependencies (MEDIA_SUPPORT && I2C && VIDEO_V4L2 && MEDIA_CAMERA_SUPPORT)
warning: (VIDEO_EM28XX_V4L2) selects VIDEO_MT9V011 which has unmet direct dependencies (MEDIA_SUPPORT && I2C && VIDEO_V4L2 && MEDIA_CAMERA_SUPPORT)
In file included from ../fs/proc/task_mmu.c:16:0:
../include/linux/page_idle.h: In function 'page_is_young':
../include/linux/page_idle.h:139:0: error: unterminated argument list invoking macro "if"
--
In file included from ../include/linux/shmem_fs.h:4:0,
from ../fs/proc/task_mmu.c:17:
../include/linux/file.h:12:1: error: expected '(' before 'struct'
struct file;
--
^~~~~~~~~
In file included from ../include/linux/shmem_fs.h:9:0,
from ../fs/proc/task_mmu.c:17:
../include/linux/xattr.h:62:27: error: invalid storage class for function 'xattr_prefix'
static inline const char *xattr_prefix(const struct xattr_handler *handler)
--
static inline void simple_xattrs_free(struct simple_xattrs *xattrs)
^~~~~~~~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:17:0:
../include/linux/shmem_fs.h:22:16: error: field 'vfs_inode' has incomplete type
struct inode vfs_inode;
--
from ../include/linux/mmdebug.h:4,
from ../include/linux/mm.h:8,
from ../fs/proc/task_mmu.c:1:
../include/linux/shmem_fs.h: In function 'SHMEM_I':
../include/linux/kernel.h:831:48: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
--
return container_of(inode, struct shmem_inode_info, vfs_inode);
^~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:17:0:
../include/linux/shmem_fs.h: In function 'page_is_young':
../include/linux/shmem_fs.h:64:28: error: invalid storage class for function 'shmem_read_mapping_page'
--
^~~~~~~~~~~~~~~~~~~~~~~
In file included from ../arch/x86/include/asm/elf.h:94:0,
from ../fs/proc/task_mmu.c:19:
../arch/x86/include/asm/desc.h:11:20: error: invalid storage class for function 'fill_ldt'
static inline void fill_ldt(struct desc_struct *desc, const struct user_desc *info)
--
from ../include/linux/gfp.h:5,
from ../include/linux/mm.h:9,
from ../fs/proc/task_mmu.c:1:
../include/linux/percpu-defs.h:86:33: error: section attribute cannot be specified for local variables
extern __PCPU_DUMMY_ATTRS char __pcpu_scope_##name; \
--
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from ../arch/x86/include/asm/elf.h:94:0,
from ../fs/proc/task_mmu.c:19:
../arch/x86/include/asm/desc.h:48:35: error: invalid storage class for function 'get_cpu_gdt_table'
static inline struct desc_struct *get_cpu_gdt_table(unsigned int cpu)
--
static inline void load_current_idt(void)
^~~~~~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:19:0:
../arch/x86/include/asm/elf.h:345:19: error: invalid storage class for function 'mmap_is_ia32'
static inline int mmap_is_ia32(void)
^~~~~~~~~~~~
In file included from ../fs/proc/internal.h:12:0,
from ../fs/proc/task_mmu.c:22:
../include/linux/proc_fs.h:30:38: error: invalid storage class for function 'proc_create'
static inline struct proc_dir_entry *proc_create(
--
static inline struct proc_dir_entry *proc_net_mkdir(
^~~~~~~~~~~~~~
In file included from ../fs/proc/internal.h:13:0,
from ../fs/proc/task_mmu.c:22:
../include/linux/proc_ns.h:64:19: error: invalid storage class for function 'ns_alloc_inum'
static inline int ns_alloc_inum(struct ns_common *ns)
^~~~~~~~~~~~~
In file included from ../fs/proc/internal.h:16:0,
from ../fs/proc/task_mmu.c:22:
../include/linux/binfmts.h:86:20: error: invalid storage class for function 'register_binfmt'
static inline void register_binfmt(struct linux_binfmt *fmt)
--
static inline void insert_binfmt(struct linux_binfmt *fmt)
^~~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:22:0:
../fs/proc/internal.h:69:15: error: field 'vfs_inode' has incomplete type
struct inode vfs_inode;
^~~~~~~~~
../fs/proc/internal.h:75:34: error: invalid storage class for function 'PROC_I'
static inline struct proc_inode *PROC_I(const struct inode *inode)
^~~~~~
--
from ../include/linux/mmdebug.h:4,
from ../include/linux/mm.h:8,
from ../fs/proc/task_mmu.c:1:
../fs/proc/internal.h: In function 'PROC_I':
../include/linux/kernel.h:831:48: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
const typeof( ((type *)0)->member ) *__mptr = (ptr); \
^
../fs/proc/internal.h:77:9: note: in expansion of macro 'container_of'
return container_of(inode, struct proc_inode, vfs_inode);
^~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:22:0:
../fs/proc/internal.h: In function 'page_is_young':
../fs/proc/internal.h:80:38: error: invalid storage class for function 'PDE'
static inline struct proc_dir_entry *PDE(const struct inode *inode)
^~~
../fs/proc/internal.h:85:21: error: invalid storage class for function '__PDE_DATA'
static inline void *__PDE_DATA(const struct inode *inode)
^~~~~~~~~~
../fs/proc/internal.h:90:27: error: invalid storage class for function 'proc_pid'
static inline struct pid *proc_pid(struct inode *inode)
^~~~~~~~
../fs/proc/internal.h:95:35: error: invalid storage class for function 'get_proc_task'
static inline struct task_struct *get_proc_task(struct inode *inode)
^~~~~~~~~~~~~
../fs/proc/internal.h: In function 'get_proc_task':
../fs/proc/internal.h:97:9: error: return from incompatible pointer type [-Werror=incompatible-pointer-types]
return get_pid_task(proc_pid(inode), PIDTYPE_PID);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h: In function 'page_is_young':
../fs/proc/internal.h:100:19: error: invalid storage class for function 'task_dumpable'
static inline int task_dumpable(struct task_struct *task)
^~~~~~~~~~~~~
../fs/proc/internal.h: In function 'task_dumpable':
../fs/proc/internal.h:105:12: error: passing argument 1 of 'task_lock' from incompatible pointer type [-Werror=incompatible-pointer-types]
task_lock(task);
^~~~
In file included from ../include/linux/vmacache.h:4:0,
from ../fs/proc/task_mmu.c:2:
../include/linux/sched.h:2912:20: note: expected 'struct task_struct *' but argument is of type 'struct task_struct *'
static inline void task_lock(struct task_struct *p)
^~~~~~~~~
In file included from ../fs/proc/task_mmu.c:22:0:
../fs/proc/internal.h:106:11: error: dereferencing pointer to incomplete type 'struct task_struct'
mm = task->mm;
^~
../fs/proc/internal.h:109:14: error: passing argument 1 of 'task_unlock' from incompatible pointer type [-Werror=incompatible-pointer-types]
task_unlock(task);
^~~~
In file included from ../include/linux/vmacache.h:4:0,
from ../fs/proc/task_mmu.c:2:
../include/linux/sched.h:2917:20: note: expected 'struct task_struct *' but argument is of type 'struct task_struct *'
static inline void task_unlock(struct task_struct *p)
^~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:22:0:
../fs/proc/internal.h: In function 'page_is_young':
../fs/proc/internal.h:115:24: error: invalid storage class for function 'name_to_int'
static inline unsigned name_to_int(const struct qstr *qstr)
^~~~~~~~~~~
../fs/proc/internal.h:187:38: error: invalid storage class for function 'pde_get'
static inline struct proc_dir_entry *pde_get(struct proc_dir_entry *pde)
^~~~~~~
../fs/proc/internal.h:194:20: error: invalid storage class for function 'is_empty_pde'
static inline bool is_empty_pde(const struct proc_dir_entry *pde)
^~~~~~~~~~~~
../fs/proc/task_mmu.c:24:6: error: static declaration of 'task_mem' follows non-static declaration
void task_mem(struct seq_file *m, struct mm_struct *mm)
^~~~~~~~
In file included from ../fs/proc/task_mmu.c:22:0:
../fs/proc/internal.h:305:13: note: previous declaration of 'task_mem' was here
extern void task_mem(struct seq_file *, struct mm_struct *);
^~~~~~~~
../fs/proc/task_mmu.c:86:15: error: static declaration of 'task_vsize' follows non-static declaration
unsigned long task_vsize(struct mm_struct *mm)
^~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:22:0:
../fs/proc/internal.h:301:22: note: previous declaration of 'task_vsize' was here
extern unsigned long task_vsize(struct mm_struct *);
^~~~~~~~~~
../fs/proc/task_mmu.c:91:15: error: static declaration of 'task_statm' follows non-static declaration
unsigned long task_statm(struct mm_struct *mm,
^~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:22:0:
../fs/proc/internal.h:302:22: note: previous declaration of 'task_statm' was here
extern unsigned long task_statm(struct mm_struct *,
^~~~~~~~~~
../fs/proc/task_mmu.c:122:13: error: invalid storage class for function 'hold_task_mempolicy'
static void hold_task_mempolicy(struct proc_maps_private *priv)
^~~~~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:125:13: error: invalid storage class for function 'release_task_mempolicy'
static void release_task_mempolicy(struct proc_maps_private *priv)
^~~~~~~~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:130:13: error: invalid storage class for function 'vma_stop'
static void vma_stop(struct proc_maps_private *priv)
^~~~~~~~
../fs/proc/task_mmu.c:140:1: error: invalid storage class for function 'm_next_vma'
m_next_vma(struct proc_maps_private *priv, struct vm_area_struct *vma)
^~~~~~~~~~
../fs/proc/task_mmu.c:147:13: error: invalid storage class for function 'm_cache_vma'
static void m_cache_vma(struct seq_file *m, struct vm_area_struct *vma)
^~~~~~~~~~~
../fs/proc/task_mmu.c:153:14: error: invalid storage class for function 'm_start'
static void *m_start(struct seq_file *m, loff_t *ppos)
^~~~~~~
../fs/proc/task_mmu.c:200:14: error: invalid storage class for function 'm_next'
static void *m_next(struct seq_file *m, void *v, loff_t *pos)
^~~~~~
../fs/proc/task_mmu.c:212:13: error: invalid storage class for function 'm_stop'
static void m_stop(struct seq_file *m, void *v)
^~~~~~
../fs/proc/task_mmu.c: In function 'm_stop':
../fs/proc/task_mmu.c:219:19: error: passing argument 1 of 'put_task_struct' from incompatible pointer type [-Werror=incompatible-pointer-types]
put_task_struct(priv->task);
^~~~
In file included from ../include/linux/vmacache.h:4:0,
from ../fs/proc/task_mmu.c:2:
../include/linux/sched.h:2135:20: note: expected 'struct task_struct *' but argument is of type 'struct task_struct *'
static inline void put_task_struct(struct task_struct *t)
^~~~~~~~~~~~~~~
../fs/proc/task_mmu.c: In function 'page_is_young':
../fs/proc/task_mmu.c:224:12: error: invalid storage class for function 'proc_maps_open'
static int proc_maps_open(struct inode *inode, struct file *file,
^~~~~~~~~~~~~~
../fs/proc/task_mmu.c: In function 'proc_maps_open':
../fs/proc/task_mmu.c:237:23: error: passing argument 1 of 'seq_release_private' from incompatible pointer type [-Werror=incompatible-pointer-types]
seq_release_private(inode, file);
^~~~~
In file included from ../include/linux/cgroup.h:17:0,
from ../include/linux/hugetlb.h:8,
from ../fs/proc/task_mmu.c:3:
../include/linux/seq_file.h:140:5: note: expected 'struct inode *' but argument is of type 'struct inode *'
int seq_release_private(struct inode *, struct file *);
^~~~~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c: In function 'page_is_young':
../fs/proc/task_mmu.c:244:12: error: invalid storage class for function 'proc_map_release'
static int proc_map_release(struct inode *inode, struct file *file)
^~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c: In function 'proc_map_release':
../fs/proc/task_mmu.c:252:29: error: passing argument 1 of 'seq_release_private' from incompatible pointer type [-Werror=incompatible-pointer-types]
return seq_release_private(inode, file);
^~~~~
In file included from ../include/linux/cgroup.h:17:0,
from ../include/linux/hugetlb.h:8,
from ../fs/proc/task_mmu.c:3:
../include/linux/seq_file.h:140:5: note: expected 'struct inode *' but argument is of type 'struct inode *'
int seq_release_private(struct inode *, struct file *);
^~~~~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c: In function 'page_is_young':
../fs/proc/task_mmu.c:255:12: error: invalid storage class for function 'do_maps_open'
static int do_maps_open(struct inode *inode, struct file *file,
^~~~~~~~~~~~
../fs/proc/task_mmu.c:266:12: error: invalid storage class for function 'is_stack'
static int is_stack(struct proc_maps_private *priv,
^~~~~~~~
../fs/proc/task_mmu.c: In function 'is_stack':
../fs/proc/task_mmu.c:279:8: error: assignment from incompatible pointer type [-Werror=incompatible-pointer-types]
task = pid_task(proc_pid(inode), PIDTYPE_PID);
^
../fs/proc/task_mmu.c:281:39: error: passing argument 2 of 'vma_is_stack_for_task' from incompatible pointer type [-Werror=incompatible-pointer-types]
stack = vma_is_stack_for_task(vma, task);
^~~~
In file included from ../fs/proc/task_mmu.c:1:0:
../include/linux/mm.h:1360:5: note: expected 'struct task_struct *' but argument is of type 'struct task_struct *'
int vma_is_stack_for_task(struct vm_area_struct *vma, struct task_struct *t);
^~~~~~~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c: In function 'page_is_young':
../fs/proc/task_mmu.c:288:1: error: invalid storage class for function 'show_map_vma'
show_map_vma(struct seq_file *m, struct vm_area_struct *vma, int is_pid)
^~~~~~~~~~~~
../fs/proc/task_mmu.c: In function 'show_map_vma':
../fs/proc/task_mmu.c:301:25: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
struct inode *inode = file_inode(vma->vm_file);
^~~~~~~~~~
../fs/proc/task_mmu.c:302:14: error: dereferencing pointer to incomplete type 'struct inode'
dev = inode->i_sb->s_dev;
^~
../fs/proc/task_mmu.c: In function 'page_is_young':
../fs/proc/task_mmu.c:367:12: error: invalid storage class for function 'show_map'
static int show_map(struct seq_file *m, void *v, int is_pid)
^~~~~~~~
../fs/proc/task_mmu.c:374:12: error: invalid storage class for function 'show_pid_map'
static int show_pid_map(struct seq_file *m, void *v)
^~~~~~~~~~~~
../fs/proc/task_mmu.c:379:12: error: invalid storage class for function 'show_tid_map'
static int show_tid_map(struct seq_file *m, void *v)
^~~~~~~~~~~~
../fs/proc/task_mmu.c:385:11: error: initializer element is not constant
.start = m_start,
^~~~~~~
../fs/proc/task_mmu.c:385:11: note: (near initialization for 'proc_pid_maps_op.start')
../fs/proc/task_mmu.c:386:10: error: initializer element is not constant
.next = m_next,
^~~~~~
../fs/proc/task_mmu.c:386:10: note: (near initialization for 'proc_pid_maps_op.next')
../fs/proc/task_mmu.c:387:10: error: initializer element is not constant
.stop = m_stop,
^~~~~~
../fs/proc/task_mmu.c:387:10: note: (near initialization for 'proc_pid_maps_op.stop')
../fs/proc/task_mmu.c:388:10: error: initializer element is not constant
.show = show_pid_map
^~~~~~~~~~~~
../fs/proc/task_mmu.c:388:10: note: (near initialization for 'proc_pid_maps_op.show')
../fs/proc/task_mmu.c:392:11: error: initializer element is not constant
.start = m_start,
^~~~~~~
../fs/proc/task_mmu.c:392:11: note: (near initialization for 'proc_tid_maps_op.start')
../fs/proc/task_mmu.c:393:10: error: initializer element is not constant
.next = m_next,
^~~~~~
../fs/proc/task_mmu.c:393:10: note: (near initialization for 'proc_tid_maps_op.next')
../fs/proc/task_mmu.c:394:10: error: initializer element is not constant
.stop = m_stop,
^~~~~~
../fs/proc/task_mmu.c:394:10: note: (near initialization for 'proc_tid_maps_op.stop')
../fs/proc/task_mmu.c:395:10: error: initializer element is not constant
.show = show_tid_map
^~~~~~~~~~~~
../fs/proc/task_mmu.c:395:10: note: (near initialization for 'proc_tid_maps_op.show')
../fs/proc/task_mmu.c:398:12: error: invalid storage class for function 'pid_maps_open'
static int pid_maps_open(struct inode *inode, struct file *file)
^~~~~~~~~~~~~
../fs/proc/task_mmu.c:403:12: error: invalid storage class for function 'tid_maps_open'
static int tid_maps_open(struct inode *inode, struct file *file)
^~~~~~~~~~~~~
../fs/proc/task_mmu.c:408:14: error: variable 'proc_pid_maps_operations' has initializer but incomplete type
const struct file_operations proc_pid_maps_operations = {
^~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:408:30: error: declaration of 'proc_pid_maps_operations' with no linkage follows extern declaration
const struct file_operations proc_pid_maps_operations = {
^~~~~~~~~~~~~~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:22:0:
../fs/proc/internal.h:292:37: note: previous declaration of 'proc_pid_maps_operations' was here
extern const struct file_operations proc_pid_maps_operations;
^~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:409:2: error: unknown field 'open' specified in initializer
.open = pid_maps_open,
^
../fs/proc/task_mmu.c:409:11: warning: excess elements in struct initializer
.open = pid_maps_open,
^~~~~~~~~~~~~
../fs/proc/task_mmu.c:409:11: note: (near initialization for 'proc_pid_maps_operations')
../fs/proc/task_mmu.c:410:2: error: unknown field 'read' specified in initializer
.read = seq_read,
^
../fs/proc/task_mmu.c:410:11: warning: excess elements in struct initializer
.read = seq_read,
^~~~~~~~
../fs/proc/task_mmu.c:410:11: note: (near initialization for 'proc_pid_maps_operations')
../fs/proc/task_mmu.c:411:2: error: unknown field 'llseek' specified in initializer
.llseek = seq_lseek,
^
../fs/proc/task_mmu.c:411:13: warning: excess elements in struct initializer
.llseek = seq_lseek,
^~~~~~~~~
../fs/proc/task_mmu.c:411:13: note: (near initialization for 'proc_pid_maps_operations')
../fs/proc/task_mmu.c:412:2: error: unknown field 'release' specified in initializer
.release = proc_map_release,
^
../fs/proc/task_mmu.c:412:13: warning: excess elements in struct initializer
.release = proc_map_release,
^~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:412:13: note: (near initialization for 'proc_pid_maps_operations')
../fs/proc/task_mmu.c:408:30: error: storage size of 'proc_pid_maps_operations' isn't known
const struct file_operations proc_pid_maps_operations = {
^~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:415:14: error: variable 'proc_tid_maps_operations' has initializer but incomplete type
const struct file_operations proc_tid_maps_operations = {
^~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:415:30: error: declaration of 'proc_tid_maps_operations' with no linkage follows extern declaration
const struct file_operations proc_tid_maps_operations = {
^~~~~~~~~~~~~~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:22:0:
../fs/proc/internal.h:293:37: note: previous declaration of 'proc_tid_maps_operations' was here
extern const struct file_operations proc_tid_maps_operations;
^~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:416:2: error: unknown field 'open' specified in initializer
.open = tid_maps_open,
^
../fs/proc/task_mmu.c:416:11: warning: excess elements in struct initializer
.open = tid_maps_open,
^~~~~~~~~~~~~
../fs/proc/task_mmu.c:416:11: note: (near initialization for 'proc_tid_maps_operations')
../fs/proc/task_mmu.c:417:2: error: unknown field 'read' specified in initializer
.read = seq_read,
^
../fs/proc/task_mmu.c:417:11: warning: excess elements in struct initializer
.read = seq_read,
^~~~~~~~
../fs/proc/task_mmu.c:417:11: note: (near initialization for 'proc_tid_maps_operations')
../fs/proc/task_mmu.c:418:2: error: unknown field 'llseek' specified in initializer
.llseek = seq_lseek,
^
../fs/proc/task_mmu.c:418:13: warning: excess elements in struct initializer
.llseek = seq_lseek,
^~~~~~~~~
../fs/proc/task_mmu.c:418:13: note: (near initialization for 'proc_tid_maps_operations')
../fs/proc/task_mmu.c:419:2: error: unknown field 'release' specified in initializer
.release = proc_map_release,
^
../fs/proc/task_mmu.c:419:13: warning: excess elements in struct initializer
.release = proc_map_release,
^~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:419:13: note: (near initialization for 'proc_tid_maps_operations')
../fs/proc/task_mmu.c:415:30: error: storage size of 'proc_tid_maps_operations' isn't known
const struct file_operations proc_tid_maps_operations = {
^~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:420:1: error: expected declaration or statement at end of input
};
^
../fs/proc/task_mmu.c:415:30: warning: unused variable 'proc_tid_maps_operations' [-Wunused-variable]
const struct file_operations proc_tid_maps_operations = {
^~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/task_mmu.c:408:30: warning: unused variable 'proc_pid_maps_operations' [-Wunused-variable]
const struct file_operations proc_pid_maps_operations = {
^~~~~~~~~~~~~~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:22:0:
../fs/proc/internal.h:299:37: warning: unused variable 'proc_pagemap_operations' [-Wunused-variable]
extern const struct file_operations proc_pagemap_operations;
^~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:298:37: warning: unused variable 'proc_clear_refs_operations' [-Wunused-variable]
extern const struct file_operations proc_clear_refs_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:297:37: warning: unused variable 'proc_tid_smaps_operations' [-Wunused-variable]
extern const struct file_operations proc_tid_smaps_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:296:37: warning: unused variable 'proc_pid_smaps_operations' [-Wunused-variable]
extern const struct file_operations proc_pid_smaps_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:295:37: warning: unused variable 'proc_tid_numa_maps_operations' [-Wunused-variable]
extern const struct file_operations proc_tid_numa_maps_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:294:37: warning: unused variable 'proc_pid_numa_maps_operations' [-Wunused-variable]
extern const struct file_operations proc_pid_numa_maps_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:270:30: warning: unused variable 'proc_root' [-Wunused-variable]
extern struct proc_dir_entry proc_root;
^~~~~~~~~
../fs/proc/internal.h:228:38: warning: unused variable 'proc_net_inode_operations' [-Wunused-variable]
extern const struct inode_operations proc_net_inode_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:227:37: warning: unused variable 'proc_net_operations' [-Wunused-variable]
extern const struct file_operations proc_net_operations;
^~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:222:37: warning: unused variable 'proc_ns_dir_operations' [-Wunused-variable]
extern const struct file_operations proc_ns_dir_operations;
^~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:221:38: warning: unused variable 'proc_ns_dir_inode_operations' [-Wunused-variable]
extern const struct inode_operations proc_ns_dir_inode_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:211:38: warning: unused variable 'proc_pid_link_inode_operations' [-Wunused-variable]
extern const struct inode_operations proc_pid_link_inode_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:209:38: warning: unused variable 'proc_link_inode_operations' [-Wunused-variable]
extern const struct inode_operations proc_link_inode_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:162:39: warning: unused variable 'pid_dentry_operations' [-Wunused-variable]
extern const struct dentry_operations pid_dentry_operations;
^~~~~~~~~~~~~~~~~~~~~
../fs/proc/internal.h:148:37: warning: unused variable 'proc_tid_children_operations' [-Wunused-variable]
extern const struct file_operations proc_tid_children_operations;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from ../fs/proc/internal.h:16:0,
from ../fs/proc/task_mmu.c:22:
../include/linux/binfmts.h:105:12: warning: unused variable 'suid_dumpable' [-Wunused-variable]
extern int suid_dumpable;
^~~~~~~~~~~~~
In file included from ../fs/proc/internal.h:13:0,
from ../fs/proc/task_mmu.c:22:
../include/linux/proc_ns.h:29:40: warning: unused variable 'cgroupns_operations' [-Wunused-variable]
extern const struct proc_ns_operations cgroupns_operations;
--
extern const struct proc_ns_operations netns_operations;
^~~~~~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:19:0:
../arch/x86/include/asm/elf.h:364:28: warning: unused variable 'va_align' [-Wunused-variable]
extern struct va_alignment va_align;
^~~~~~~~
In file included from ../arch/x86/include/asm/elf.h:94:0,
from ../fs/proc/task_mmu.c:19:
../arch/x86/include/asm/desc.h:40:18: warning: unused variable 'debug_idt_table' [-Wunused-variable]
extern gate_desc debug_idt_table[];
--
extern struct desc_ptr debug_idt_descr;
^~~~~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:19:0:
../arch/x86/include/asm/elf.h:82:21: warning: unused variable 'vdso32_enabled' [-Wunused-variable]
extern unsigned int vdso32_enabled;
^~~~~~~~~~~~~~
In file included from ../arch/x86/include/asm/elf.h:76:0,
from ../fs/proc/task_mmu.c:19:
../arch/x86/include/asm/vdso.h:39:32: warning: unused variable 'vdso_image_32' [-Wunused-variable]
extern const struct vdso_image vdso_image_32;
^~~~~~~~~~~~~
In file included from ../fs/proc/task_mmu.c:16:0:
../include/linux/page_idle.h:49:19: warning: unused variable 'page_ext' [-Wunused-variable]
struct page_ext *page_ext = lookup_page_ext(page);
^~~~~~~~
../fs/proc/task_mmu.c:420:1: warning: no return statement in function returning non-void [-Wreturn-type]
};
^
At top level:
../fs/proc/task_mmu.c:91:15: warning: 'task_statm' defined but not used [-Wunused-function]
unsigned long task_statm(struct mm_struct *mm,
^~~~~~~~~~
../fs/proc/task_mmu.c:86:15: warning: 'task_vsize' defined but not used [-Wunused-function]
unsigned long task_vsize(struct mm_struct *mm)
^~~~~~~~~~
../fs/proc/task_mmu.c:24:6: warning: 'task_mem' defined but not used [-Wunused-function]
void task_mem(struct seq_file *m, struct mm_struct *mm)
^~~~~~~~
cc1: some warnings being treated as errors
make[2]: *** [fs/proc/task_mmu.o] Error 1
make[2]: Target '__build' not remade because of errors.
make[1]: *** [fs/proc/] Error 2
make: *** [sub-make] Error 2
make[2]: *** No rule to make target 'fs/proc//task_mmu.o', needed by 'fs/proc//proc.o'.
make[2]: Target '__build' not remade because of errors.
make[1]: *** [_module_fs/proc/] Error 2
make[1]: Target '_all' not remade because of errors.
make: *** [sub-make] Error 2
Thanks,
Fengguang
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-05-25 0:14 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-24 10:48 [linux-next:master 11896/11991] include/linux/page_idle.h:49:19: warning: unused variable 'page_ext' kbuild test robot
2016-05-24 19:44 ` Andrew Morton
2016-05-25 0:14 ` Fengguang Wu
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.