Merge branch 'x86-asm-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip

Pull x86 asm updates from Ingo Molnar:
 "The main changes in this development cycle were:

   - a large number of call stack dumping/printing improvements: higher
     robustness, better cross-context dumping, improved output, etc.
     (Josh Poimboeuf)

   - vDSO getcpu() performance improvement for future Intel CPUs with
     the RDPID instruction (Andy Lutomirski)

   - add two new Intel AVX512 features and the CPUID support
     infrastructure for it: AVX512IFMA and AVX512VBMI. (Gayatri Kammela,
     He Chen)

   - more copy-user unification (Borislav Petkov)

   - entry code assembly macro simplifications (Alexander Kuleshov)

   - vDSO C/R support improvements (Dmitry Safonov)

   - misc fixes and cleanups (Borislav Petkov, Paul Bolle)"

* 'x86-asm-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip: (40 commits)
  scripts/decode_stacktrace.sh: Fix address line detection on x86
  x86/boot/64: Use defines for page size
  x86/dumpstack: Make stack name tags more comprehensible
  selftests/x86: Add test_vdso to test getcpu()
  x86/vdso: Use RDPID in preference to LSL when available
  x86/dumpstack: Handle NULL stack pointer in show_trace_log_lvl()
  x86/cpufeatures: Enable new AVX512 cpu features
  x86/cpuid: Provide get_scattered_cpuid_leaf()
  x86/cpuid: Cleanup cpuid_regs definitions
  x86/copy_user: Unify the code by removing the 64-bit asm _copy_*_user() variants
  x86/unwind: Ensure stack grows down
  x86/vdso: Set vDSO pointer only after success
  x86/prctl/uapi: Remove #ifdef for CHECKPOINT_RESTORE
  x86/unwind: Detect bad stack return address
  x86/dumpstack: Warn on stack recursion
  x86/unwind: Warn on bad frame pointer
  x86/decoder: Use stderr if insn sanity test fails
  x86/decoder: Use stdout if insn decoder test is successful
  mm/page_alloc: Remove kernel address exposure in free_reserved_area()
  x86/dumpstack: Remove raw stack dump
  ...

1 files changed, 28 insertions, 40 deletions

diff --git a/arch/x86/kernel/dumpstack.c b/arch/x86/kernel/dumpstack.c
index 85f854b98a9d..0cfd01d2754c 100644
--- a/arch/x86/kernel/dumpstack.c
+++ b/arch/x86/kernel/dumpstack.c
@@ -22,7 +22,6 @@
 int panic_on_unrecovered_nmi;
 int panic_on_io_nmi;
 unsigned int code_bytes = 64;
-int kstack_depth_to_print = 3 * STACKSLOTS_PER_LINE;
 static int die_counter;
 
 bool in_task_stack(unsigned long *stack, struct task_struct *task,
@@ -46,14 +45,7 @@ static void printk_stack_address(unsigned long address, int reliable,
                                  char *log_lvl)
 {
         touch_nmi_watchdog();
-        printk("%s [<%p>] %s%pB\n",
-                log_lvl, (void *)address, reliable ? "" : "? ",
-                (void *)address);
-}
-
-void printk_address(unsigned long address)
-{
-        pr_cont(" [<%p>] %pS\n", (void *)address, (void *)address);
+        printk("%s %s%pB\n", log_lvl, reliable ? "" : "? ", (void *)address);
 }
 
 void show_trace_log_lvl(struct task_struct *task, struct pt_regs *regs,
@@ -67,6 +59,7 @@ void show_trace_log_lvl(struct task_struct *task, struct pt_regs *regs,
         printk("%sCall Trace:\n", log_lvl);
 
         unwind_start(&state, task, regs, stack);
+        stack = stack ? : get_stack_pointer(task, regs);
 
         /*
          * Iterate through the stacks, starting with the current stack pointer.
@@ -82,8 +75,8 @@ void show_trace_log_lvl(struct task_struct *task, struct pt_regs *regs,
          * - softirq stack
          * - hardirq stack
          */
-        for (; stack; stack = stack_info.next_sp) {
-                const char *str_begin, *str_end;
+        for (regs = NULL; stack; stack = stack_info.next_sp) {
+                const char *stack_name;
 
                 /*
                  * If we overflowed the task stack into a guard page, jump back
@@ -95,9 +88,9 @@ void show_trace_log_lvl(struct task_struct *task, struct pt_regs *regs,
                 if (get_stack_info(stack, task, &stack_info, &visit_mask))
                         break;
 
-                stack_type_str(stack_info.type, &str_begin, &str_end);
-                if (str_begin)
-                        printk("%s <%s> ", log_lvl, str_begin);
+                stack_name = stack_type_name(stack_info.type);
+                if (stack_name)
+                        printk("%s <%s>\n", log_lvl, stack_name);
 
                 /*
                  * Scan the stack, printing any text addresses we find.  At the
@@ -119,6 +112,15 @@ void show_trace_log_lvl(struct task_struct *task, struct pt_regs *regs,
                         if (!__kernel_text_address(addr))
                                 continue;
 
+                        /*
+                         * Don't print regs->ip again if it was already printed
+                         * by __show_regs() below.
+                         */
+                        if (regs && stack == &regs->ip) {
+                                unwind_next_frame(&state);
+                                continue;
+                        }
+
                         if (stack == ret_addr_p)
                                 reliable = 1;
 
@@ -146,10 +148,15 @@ void show_trace_log_lvl(struct task_struct *task, struct pt_regs *regs,
                          * of the addresses will just be printed as unreliable.
                          */
                         unwind_next_frame(&state);
+
+                        /* if the frame has entry regs, print them */
+                        regs = unwind_get_entry_regs(&state);
+                        if (regs)
+                                __show_regs(regs, 0);
                 }
 
-                if (str_end)
-                        printk("%s <%s> ", log_lvl, str_end);
+                if (stack_name)
+                        printk("%s </%s>\n", log_lvl, stack_name);
         }
 }
 
@@ -164,12 +171,12 @@ void show_stack(struct task_struct *task, unsigned long *sp)
         if (!sp && task == current)
                 sp = get_stack_pointer(current, NULL);
 
-        show_stack_log_lvl(task, NULL, sp, "");
+        show_trace_log_lvl(task, NULL, sp, KERN_DEFAULT);
 }
 
 void show_stack_regs(struct pt_regs *regs)
 {
-        show_stack_log_lvl(current, regs, NULL, "");
+        show_trace_log_lvl(current, regs, NULL, KERN_DEFAULT);
 }
 
 static arch_spinlock_t die_lock = __ARCH_SPIN_LOCK_UNLOCKED;
@@ -261,14 +268,11 @@ int __die(const char *str, struct pt_regs *regs, long err)
                 sp = kernel_stack_pointer(regs);
                 savesegment(ss, ss);
         }
-        printk(KERN_EMERG "EIP: [<%08lx>] ", regs->ip);
-        print_symbol("%s", regs->ip);
-        printk(" SS:ESP %04x:%08lx\n", ss, sp);
+        printk(KERN_EMERG "EIP: %pS SS:ESP: %04x:%08lx\n",
+               (void *)regs->ip, ss, sp);
 #else
         /* Executive summary in case the oops scrolled away */
-        printk(KERN_ALERT "RIP ");
-        printk_address(regs->ip);
-        printk(" RSP <%016lx>\n", regs->sp);
+        printk(KERN_ALERT "RIP: %pS RSP: %016lx\n", (void *)regs->ip, regs->sp);
 #endif
         return 0;
 }
@@ -291,22 +295,6 @@ void die(const char *str, struct pt_regs *regs, long err)
         oops_end(flags, regs, sig);
 }
 
-static int __init kstack_setup(char *s)
-{
-        ssize_t ret;
-        unsigned long val;
-
-        if (!s)
-                return -EINVAL;
-
-        ret = kstrtoul(s, 0, &val);
-        if (ret)
-                return ret;
-        kstack_depth_to_print = val;
-        return 0;
-}
-early_param("kstack", kstack_setup);
-
 static int __init code_bytes_setup(char *s)
 {
         ssize_t ret;