diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2014-10-13 21:49:02 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-10-13 21:49:02 -0400 |
| commit | 31003e3a9df675f1ac85d7bcf8e5a5d622576375 (patch) | |
| tree | 6b08171deca1bf3cdcd6968ce0859e505048c73a /arch/um | |
| parent | 1ee07ef6b5db7235b133ee257a3adf507697e6b3 (diff) | |
| parent | 5f786595a291092d20fafe10c5a30378971a8cc3 (diff) | |
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/rw/uml
Pull UML update from Richard Weinberger:
"Besides of fixes this contains also support for CONFIG_STACKTRACE by
Daniel Walter"
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/rw/uml:
um: net: Eliminate NULL test after alloc_bootmem
um: Add support for CONFIG_STACKTRACE
um: ubd: Fix for processes stuck in D state forever
um: delete unnecessary bootmem struct page array
um: remove csum_partial_copy_generic_i386 to clean up exception table
Diffstat (limited to 'arch/um')
| -rw-r--r-- | arch/um/Kconfig.common | 3 | ||||
| -rw-r--r-- | arch/um/drivers/net_kern.c | 4 | ||||
| -rw-r--r-- | arch/um/drivers/ubd_kern.c | 5 | ||||
| -rw-r--r-- | arch/um/include/asm/stacktrace.h | 42 | ||||
| -rw-r--r-- | arch/um/include/shared/mem_user.h | 2 | ||||
| -rw-r--r-- | arch/um/kernel/Makefile | 1 | ||||
| -rw-r--r-- | arch/um/kernel/physmem.c | 32 | ||||
| -rw-r--r-- | arch/um/kernel/stacktrace.c | 80 | ||||
| -rw-r--r-- | arch/um/kernel/sysrq.c | 69 | ||||
| -rw-r--r-- | arch/um/kernel/um_arch.c | 7 |
10 files changed, 153 insertions, 92 deletions
diff --git a/arch/um/Kconfig.common b/arch/um/Kconfig.common index 6915d28cf118..87bc86821bc9 100644 --- a/arch/um/Kconfig.common +++ b/arch/um/Kconfig.common | |||
| @@ -39,7 +39,8 @@ config LOCKDEP_SUPPORT | |||
| 39 | 39 | ||
| 40 | config STACKTRACE_SUPPORT | 40 | config STACKTRACE_SUPPORT |
| 41 | bool | 41 | bool |
| 42 | default n | 42 | default y |
| 43 | select STACKTRACE | ||
| 43 | 44 | ||
| 44 | config GENERIC_CALIBRATE_DELAY | 45 | config GENERIC_CALIBRATE_DELAY |
| 45 | bool | 46 | bool |
diff --git a/arch/um/drivers/net_kern.c b/arch/um/drivers/net_kern.c index 7d26d9c0b2fb..f70dd540655d 100644 --- a/arch/um/drivers/net_kern.c +++ b/arch/um/drivers/net_kern.c | |||
| @@ -659,10 +659,6 @@ static int __init eth_setup(char *str) | |||
| 659 | } | 659 | } |
| 660 | 660 | ||
| 661 | new = alloc_bootmem(sizeof(*new)); | 661 | new = alloc_bootmem(sizeof(*new)); |
| 662 | if (new == NULL) { | ||
| 663 | printk(KERN_ERR "eth_init : alloc_bootmem failed\n"); | ||
| 664 | return 1; | ||
| 665 | } | ||
| 666 | 662 | ||
| 667 | INIT_LIST_HEAD(&new->list); | 663 | INIT_LIST_HEAD(&new->list); |
| 668 | new->index = n; | 664 | new->index = n; |
diff --git a/arch/um/drivers/ubd_kern.c b/arch/um/drivers/ubd_kern.c index 3716e6952554..e8ab93c3e638 100644 --- a/arch/um/drivers/ubd_kern.c +++ b/arch/um/drivers/ubd_kern.c | |||
| @@ -1277,7 +1277,7 @@ static void do_ubd_request(struct request_queue *q) | |||
| 1277 | 1277 | ||
| 1278 | while(1){ | 1278 | while(1){ |
| 1279 | struct ubd *dev = q->queuedata; | 1279 | struct ubd *dev = q->queuedata; |
| 1280 | if(dev->end_sg == 0){ | 1280 | if(dev->request == NULL){ |
| 1281 | struct request *req = blk_fetch_request(q); | 1281 | struct request *req = blk_fetch_request(q); |
| 1282 | if(req == NULL) | 1282 | if(req == NULL) |
| 1283 | return; | 1283 | return; |
| @@ -1299,7 +1299,8 @@ static void do_ubd_request(struct request_queue *q) | |||
| 1299 | return; | 1299 | return; |
| 1300 | } | 1300 | } |
| 1301 | prepare_flush_request(req, io_req); | 1301 | prepare_flush_request(req, io_req); |
| 1302 | submit_request(io_req, dev); | 1302 | if (submit_request(io_req, dev) == false) |
| 1303 | return; | ||
| 1303 | } | 1304 | } |
| 1304 | 1305 | ||
| 1305 | while(dev->start_sg < dev->end_sg){ | 1306 | while(dev->start_sg < dev->end_sg){ |
diff --git a/arch/um/include/asm/stacktrace.h b/arch/um/include/asm/stacktrace.h new file mode 100644 index 000000000000..9a864328c67f --- /dev/null +++ b/arch/um/include/asm/stacktrace.h | |||
| @@ -0,0 +1,42 @@ | |||
| 1 | #ifndef _ASM_UML_STACKTRACE_H | ||
| 2 | #define _ASM_UML_STACKTRACE_H | ||
| 3 | |||
| 4 | #include <linux/uaccess.h> | ||
| 5 | #include <linux/ptrace.h> | ||
| 6 | |||
| 7 | struct stack_frame { | ||
| 8 | struct stack_frame *next_frame; | ||
| 9 | unsigned long return_address; | ||
| 10 | }; | ||
| 11 | |||
| 12 | struct stacktrace_ops { | ||
| 13 | void (*address)(void *data, unsigned long address, int reliable); | ||
| 14 | }; | ||
| 15 | |||
| 16 | #ifdef CONFIG_FRAME_POINTER | ||
| 17 | static inline unsigned long | ||
| 18 | get_frame_pointer(struct task_struct *task, struct pt_regs *segv_regs) | ||
| 19 | { | ||
| 20 | if (!task || task == current) | ||
| 21 | return segv_regs ? PT_REGS_BP(segv_regs) : current_bp(); | ||
| 22 | return KSTK_EBP(task); | ||
| 23 | } | ||
| 24 | #else | ||
| 25 | static inline unsigned long | ||
| 26 | get_frame_pointer(struct task_struct *task, struct pt_regs *segv_regs) | ||
| 27 | { | ||
| 28 | return 0; | ||
| 29 | } | ||
| 30 | #endif | ||
| 31 | |||
| 32 | static inline unsigned long | ||
| 33 | *get_stack_pointer(struct task_struct *task, struct pt_regs *segv_regs) | ||
| 34 | { | ||
| 35 | if (!task || task == current) | ||
| 36 | return segv_regs ? (unsigned long *)PT_REGS_SP(segv_regs) : current_sp(); | ||
| 37 | return (unsigned long *)KSTK_ESP(task); | ||
| 38 | } | ||
| 39 | |||
| 40 | void dump_trace(struct task_struct *tsk, const struct stacktrace_ops *ops, void *data); | ||
| 41 | |||
| 42 | #endif /* _ASM_UML_STACKTRACE_H */ | ||
diff --git a/arch/um/include/shared/mem_user.h b/arch/um/include/shared/mem_user.h index 46384acd547b..cb84414e3e66 100644 --- a/arch/um/include/shared/mem_user.h +++ b/arch/um/include/shared/mem_user.h | |||
| @@ -49,7 +49,7 @@ extern int iomem_size; | |||
| 49 | extern int init_mem_user(void); | 49 | extern int init_mem_user(void); |
| 50 | extern void setup_memory(void *entry); | 50 | extern void setup_memory(void *entry); |
| 51 | extern unsigned long find_iomem(char *driver, unsigned long *len_out); | 51 | extern unsigned long find_iomem(char *driver, unsigned long *len_out); |
| 52 | extern int init_maps(unsigned long physmem, unsigned long iomem, | 52 | extern void mem_total_pages(unsigned long physmem, unsigned long iomem, |
| 53 | unsigned long highmem); | 53 | unsigned long highmem); |
| 54 | extern unsigned long get_vm(unsigned long len); | 54 | extern unsigned long get_vm(unsigned long len); |
| 55 | extern void setup_physmem(unsigned long start, unsigned long usable, | 55 | extern void setup_physmem(unsigned long start, unsigned long usable, |
diff --git a/arch/um/kernel/Makefile b/arch/um/kernel/Makefile index d8b78a03855c..2d840a070c8b 100644 --- a/arch/um/kernel/Makefile +++ b/arch/um/kernel/Makefile | |||
| @@ -19,6 +19,7 @@ obj-$(CONFIG_BLK_DEV_INITRD) += initrd.o | |||
| 19 | obj-$(CONFIG_GPROF) += gprof_syms.o | 19 | obj-$(CONFIG_GPROF) += gprof_syms.o |
| 20 | obj-$(CONFIG_GCOV) += gmon_syms.o | 20 | obj-$(CONFIG_GCOV) += gmon_syms.o |
| 21 | obj-$(CONFIG_EARLY_PRINTK) += early_printk.o | 21 | obj-$(CONFIG_EARLY_PRINTK) += early_printk.o |
| 22 | obj-$(CONFIG_STACKTRACE) += stacktrace.o | ||
| 22 | 23 | ||
| 23 | USER_OBJS := config.o | 24 | USER_OBJS := config.o |
| 24 | 25 | ||
diff --git a/arch/um/kernel/physmem.c b/arch/um/kernel/physmem.c index 30fdd5d0067b..549ecf3f5857 100644 --- a/arch/um/kernel/physmem.c +++ b/arch/um/kernel/physmem.c | |||
| @@ -22,39 +22,19 @@ EXPORT_SYMBOL(high_physmem); | |||
| 22 | 22 | ||
| 23 | extern unsigned long long physmem_size; | 23 | extern unsigned long long physmem_size; |
| 24 | 24 | ||
| 25 | int __init init_maps(unsigned long physmem, unsigned long iomem, | 25 | void __init mem_total_pages(unsigned long physmem, unsigned long iomem, |
| 26 | unsigned long highmem) | 26 | unsigned long highmem) |
| 27 | { | 27 | { |
| 28 | struct page *p, *map; | 28 | unsigned long phys_pages, highmem_pages; |
| 29 | unsigned long phys_len, phys_pages, highmem_len, highmem_pages; | 29 | unsigned long iomem_pages, total_pages; |
| 30 | unsigned long iomem_len, iomem_pages, total_len, total_pages; | ||
| 31 | int i; | ||
| 32 | |||
| 33 | phys_pages = physmem >> PAGE_SHIFT; | ||
| 34 | phys_len = phys_pages * sizeof(struct page); | ||
| 35 | |||
| 36 | iomem_pages = iomem >> PAGE_SHIFT; | ||
| 37 | iomem_len = iomem_pages * sizeof(struct page); | ||
| 38 | 30 | ||
| 31 | phys_pages = physmem >> PAGE_SHIFT; | ||
| 32 | iomem_pages = iomem >> PAGE_SHIFT; | ||
| 39 | highmem_pages = highmem >> PAGE_SHIFT; | 33 | highmem_pages = highmem >> PAGE_SHIFT; |
| 40 | highmem_len = highmem_pages * sizeof(struct page); | ||
| 41 | |||
| 42 | total_pages = phys_pages + iomem_pages + highmem_pages; | ||
| 43 | total_len = phys_len + iomem_len + highmem_len; | ||
| 44 | 34 | ||
| 45 | map = alloc_bootmem_low_pages(total_len); | 35 | total_pages = phys_pages + iomem_pages + highmem_pages; |
| 46 | if (map == NULL) | ||
| 47 | return -ENOMEM; | ||
| 48 | |||
| 49 | for (i = 0; i < total_pages; i++) { | ||
| 50 | p = &map[i]; | ||
| 51 | memset(p, 0, sizeof(struct page)); | ||
| 52 | SetPageReserved(p); | ||
| 53 | INIT_LIST_HEAD(&p->lru); | ||
| 54 | } | ||
| 55 | 36 | ||
| 56 | max_mapnr = total_pages; | 37 | max_mapnr = total_pages; |
| 57 | return 0; | ||
| 58 | } | 38 | } |
| 59 | 39 | ||
| 60 | void map_memory(unsigned long virt, unsigned long phys, unsigned long len, | 40 | void map_memory(unsigned long virt, unsigned long phys, unsigned long len, |
diff --git a/arch/um/kernel/stacktrace.c b/arch/um/kernel/stacktrace.c new file mode 100644 index 000000000000..ebe7bcf62684 --- /dev/null +++ b/arch/um/kernel/stacktrace.c | |||
| @@ -0,0 +1,80 @@ | |||
| 1 | /* | ||
| 2 | * Copyright (C) 2001 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com) | ||
| 3 | * Copyright (C) 2013 Richard Weinberger <richard@nod.at> | ||
| 4 | * Copyright (C) 2014 Google Inc., Author: Daniel Walter <dwalter@google.com> | ||
| 5 | * | ||
| 6 | * This program is free software; you can redistribute it and/or modify | ||
| 7 | * it under the terms of the GNU General Public License version 2 as | ||
| 8 | * published by the Free Software Foundation. | ||
| 9 | */ | ||
| 10 | |||
| 11 | #include <linux/kallsyms.h> | ||
| 12 | #include <linux/kernel.h> | ||
| 13 | #include <linux/sched.h> | ||
| 14 | #include <linux/stacktrace.h> | ||
| 15 | #include <linux/module.h> | ||
| 16 | #include <linux/uaccess.h> | ||
| 17 | #include <asm/stacktrace.h> | ||
| 18 | |||
| 19 | void dump_trace(struct task_struct *tsk, | ||
| 20 | const struct stacktrace_ops *ops, | ||
| 21 | void *data) | ||
| 22 | { | ||
| 23 | int reliable = 0; | ||
| 24 | unsigned long *sp, bp, addr; | ||
| 25 | struct pt_regs *segv_regs = tsk->thread.segv_regs; | ||
| 26 | struct stack_frame *frame; | ||
| 27 | |||
| 28 | bp = get_frame_pointer(tsk, segv_regs); | ||
| 29 | sp = get_stack_pointer(tsk, segv_regs); | ||
| 30 | |||
| 31 | frame = (struct stack_frame *)bp; | ||
| 32 | while (((long) sp & (THREAD_SIZE-1)) != 0) { | ||
| 33 | addr = *sp; | ||
| 34 | if (__kernel_text_address(addr)) { | ||
| 35 | reliable = 0; | ||
| 36 | if ((unsigned long) sp == bp + sizeof(long)) { | ||
| 37 | frame = frame ? frame->next_frame : NULL; | ||
| 38 | bp = (unsigned long)frame; | ||
| 39 | reliable = 1; | ||
| 40 | } | ||
| 41 | ops->address(data, addr, reliable); | ||
| 42 | } | ||
| 43 | sp++; | ||
| 44 | } | ||
| 45 | } | ||
| 46 | |||
| 47 | static void save_addr(void *data, unsigned long address, int reliable) | ||
| 48 | { | ||
| 49 | struct stack_trace *trace = data; | ||
| 50 | |||
| 51 | if (!reliable) | ||
| 52 | return; | ||
| 53 | if (trace->nr_entries >= trace->max_entries) | ||
| 54 | return; | ||
| 55 | |||
| 56 | trace->entries[trace->nr_entries++] = address; | ||
| 57 | } | ||
| 58 | |||
| 59 | static const struct stacktrace_ops dump_ops = { | ||
| 60 | .address = save_addr | ||
| 61 | }; | ||
| 62 | |||
| 63 | static void __save_stack_trace(struct task_struct *tsk, struct stack_trace *trace) | ||
| 64 | { | ||
| 65 | dump_trace(tsk, &dump_ops, trace); | ||
| 66 | if (trace->nr_entries < trace->max_entries) | ||
| 67 | trace->entries[trace->nr_entries++] = ULONG_MAX; | ||
| 68 | } | ||
| 69 | |||
| 70 | void save_stack_trace(struct stack_trace *trace) | ||
| 71 | { | ||
| 72 | __save_stack_trace(current, trace); | ||
| 73 | } | ||
| 74 | EXPORT_SYMBOL_GPL(save_stack_trace); | ||
| 75 | |||
| 76 | void save_stack_trace_tsk(struct task_struct *tsk, struct stack_trace *trace) | ||
| 77 | { | ||
| 78 | __save_stack_trace(tsk, trace); | ||
| 79 | } | ||
| 80 | EXPORT_SYMBOL_GPL(save_stack_trace_tsk); | ||
diff --git a/arch/um/kernel/sysrq.c b/arch/um/kernel/sysrq.c index 799d7e413bf5..894c8d303cda 100644 --- a/arch/um/kernel/sysrq.c +++ b/arch/um/kernel/sysrq.c | |||
| @@ -12,57 +12,20 @@ | |||
| 12 | #include <linux/module.h> | 12 | #include <linux/module.h> |
| 13 | #include <linux/sched.h> | 13 | #include <linux/sched.h> |
| 14 | #include <asm/sysrq.h> | 14 | #include <asm/sysrq.h> |
| 15 | #include <asm/stacktrace.h> | ||
| 15 | #include <os.h> | 16 | #include <os.h> |
| 16 | 17 | ||
| 17 | struct stack_frame { | 18 | static void _print_addr(void *data, unsigned long address, int reliable) |
| 18 | struct stack_frame *next_frame; | ||
| 19 | unsigned long return_address; | ||
| 20 | }; | ||
| 21 | |||
| 22 | static void do_stack_trace(unsigned long *sp, unsigned long bp) | ||
| 23 | { | 19 | { |
| 24 | int reliable; | 20 | pr_info(" [<%08lx>]", address); |
| 25 | unsigned long addr; | 21 | pr_cont(" %s", reliable ? "" : "? "); |
| 26 | struct stack_frame *frame = (struct stack_frame *)bp; | 22 | print_symbol("%s", address); |
| 27 | 23 | pr_cont("\n"); | |
| 28 | printk(KERN_INFO "Call Trace:\n"); | ||
| 29 | while (((long) sp & (THREAD_SIZE-1)) != 0) { | ||
| 30 | addr = *sp; | ||
| 31 | if (__kernel_text_address(addr)) { | ||
| 32 | reliable = 0; | ||
| 33 | if ((unsigned long) sp == bp + sizeof(long)) { | ||
| 34 | frame = frame ? frame->next_frame : NULL; | ||
| 35 | bp = (unsigned long)frame; | ||
| 36 | reliable = 1; | ||
| 37 | } | ||
| 38 | |||
| 39 | printk(KERN_INFO " [<%08lx>]", addr); | ||
| 40 | printk(KERN_CONT " %s", reliable ? "" : "? "); | ||
| 41 | print_symbol(KERN_CONT "%s", addr); | ||
| 42 | printk(KERN_CONT "\n"); | ||
| 43 | } | ||
| 44 | sp++; | ||
| 45 | } | ||
| 46 | printk(KERN_INFO "\n"); | ||
| 47 | } | 24 | } |
| 48 | 25 | ||
| 49 | static unsigned long get_frame_pointer(struct task_struct *task, | 26 | static const struct stacktrace_ops stackops = { |
| 50 | struct pt_regs *segv_regs) | 27 | .address = _print_addr |
| 51 | { | 28 | }; |
| 52 | if (!task || task == current) | ||
| 53 | return segv_regs ? PT_REGS_BP(segv_regs) : current_bp(); | ||
| 54 | else | ||
| 55 | return KSTK_EBP(task); | ||
| 56 | } | ||
| 57 | |||
| 58 | static unsigned long *get_stack_pointer(struct task_struct *task, | ||
| 59 | struct pt_regs *segv_regs) | ||
| 60 | { | ||
| 61 | if (!task || task == current) | ||
| 62 | return segv_regs ? (unsigned long *)PT_REGS_SP(segv_regs) : current_sp(); | ||
| 63 | else | ||
| 64 | return (unsigned long *)KSTK_ESP(task); | ||
| 65 | } | ||
| 66 | 29 | ||
| 67 | void show_stack(struct task_struct *task, unsigned long *stack) | 30 | void show_stack(struct task_struct *task, unsigned long *stack) |
| 68 | { | 31 | { |
| @@ -71,7 +34,7 @@ void show_stack(struct task_struct *task, unsigned long *stack) | |||
| 71 | int i; | 34 | int i; |
| 72 | 35 | ||
| 73 | if (!segv_regs && os_is_signal_stack()) { | 36 | if (!segv_regs && os_is_signal_stack()) { |
| 74 | printk(KERN_ERR "Received SIGSEGV in SIGSEGV handler," | 37 | pr_err("Received SIGSEGV in SIGSEGV handler," |
| 75 | " aborting stack trace!\n"); | 38 | " aborting stack trace!\n"); |
| 76 | return; | 39 | return; |
| 77 | } | 40 | } |
| @@ -83,16 +46,18 @@ void show_stack(struct task_struct *task, unsigned long *stack) | |||
| 83 | if (!stack) | 46 | if (!stack) |
| 84 | sp = get_stack_pointer(task, segv_regs); | 47 | sp = get_stack_pointer(task, segv_regs); |
| 85 | 48 | ||
| 86 | printk(KERN_INFO "Stack:\n"); | 49 | pr_info("Stack:\n"); |
| 87 | stack = sp; | 50 | stack = sp; |
| 88 | for (i = 0; i < 3 * STACKSLOTS_PER_LINE; i++) { | 51 | for (i = 0; i < 3 * STACKSLOTS_PER_LINE; i++) { |
| 89 | if (kstack_end(stack)) | 52 | if (kstack_end(stack)) |
| 90 | break; | 53 | break; |
| 91 | if (i && ((i % STACKSLOTS_PER_LINE) == 0)) | 54 | if (i && ((i % STACKSLOTS_PER_LINE) == 0)) |
| 92 | printk(KERN_CONT "\n"); | 55 | pr_cont("\n"); |
| 93 | printk(KERN_CONT " %08lx", *stack++); | 56 | pr_cont(" %08lx", *stack++); |
| 94 | } | 57 | } |
| 95 | printk(KERN_CONT "\n"); | 58 | pr_cont("\n"); |
| 96 | 59 | ||
| 97 | do_stack_trace(sp, bp); | 60 | pr_info("Call Trace:\n"); |
| 61 | dump_trace(current, &stackops, NULL); | ||
| 62 | pr_info("\n"); | ||
| 98 | } | 63 | } |
diff --git a/arch/um/kernel/um_arch.c b/arch/um/kernel/um_arch.c index 016adf0985d5..9274eae6ae7b 100644 --- a/arch/um/kernel/um_arch.c +++ b/arch/um/kernel/um_arch.c | |||
| @@ -348,12 +348,7 @@ int __init linux_main(int argc, char **argv) | |||
| 348 | start_vm = VMALLOC_START; | 348 | start_vm = VMALLOC_START; |
| 349 | 349 | ||
| 350 | setup_physmem(uml_physmem, uml_reserved, physmem_size, highmem); | 350 | setup_physmem(uml_physmem, uml_reserved, physmem_size, highmem); |
| 351 | if (init_maps(physmem_size, iomem_size, highmem)) { | 351 | mem_total_pages(physmem_size, iomem_size, highmem); |
| 352 | printf("Failed to allocate mem_map for %Lu bytes of physical " | ||
| 353 | "memory and %Lu bytes of highmem\n", physmem_size, | ||
| 354 | highmem); | ||
| 355 | exit(1); | ||
| 356 | } | ||
| 357 | 352 | ||
| 358 | virtmem_size = physmem_size; | 353 | virtmem_size = physmem_size; |
| 359 | stack = (unsigned long) argv; | 354 | stack = (unsigned long) argv; |
