Re: [PATCH v4 11/16] powerpc/64s: machine check interrupt update NMI accounting

2020-05-11 Thread Michael Ellerman
Nicholas Piggin writes: > Excerpts from kbuild test robot's message of May 9, 2020 1:13 pm: >> Hi Nicholas, >> >> I love your patch! Yet something to improve: > > ... > >> 1419 #if defined(CONFIG_4xx) || defined(CONFIG_BOOKE) >> 1420 pr_cont("DEAR: "REG" ESR:

Re: [PATCH v4 11/16] powerpc/64s: machine check interrupt update NMI accounting

2020-05-09 Thread Nicholas Piggin
Excerpts from kbuild test robot's message of May 9, 2020 1:13 pm: > Hi Nicholas, > > I love your patch! Yet something to improve: ... > 1419#if defined(CONFIG_4xx) || defined(CONFIG_BOOKE) > 1420pr_cont("DEAR: "REG" ESR: "REG" ", regs->dar, > regs->dsisr); >

Re: [PATCH v4 11/16] powerpc/64s: machine check interrupt update NMI accounting

2020-05-08 Thread kbuild test robot
Hi Nicholas, I love your patch! Yet something to improve: [auto build test ERROR on powerpc/next] [also build test ERROR on tip/perf/core v5.7-rc4 next-20200508] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use

[PATCH v4 11/16] powerpc/64s: machine check interrupt update NMI accounting

2020-05-07 Thread Nicholas Piggin
machine_check_early is taken as an NMI, so nmi_enter is used there. machine_check_exception is no longer taken as an NMI (it's invoked via irq_work in the case a machine check hits in kernel mode), so remove the nmi_enter from that case. In NMI context, hash faults don't try to refill the hash