[RESEND][PATCH] selftests/vm: Fix vm tests build and run

2020-02-09 Thread Harish
A recent change overrides the ARCH env variable and hence runs using make fails with the following. $ make -C vm/ make: Entering directory '/home/harish/linux/tools/testing/selftests/vm' make --no-builtin-rules ARCH=ppc64le -C ../../../.. headers_install make[1]: Entering directory

Re: [PATCH V13] mm/debug: Add tests validating architecture page table helpers

2020-02-09 Thread Christophe Leroy
Le 10/02/2020 à 06:35, Anshuman Khandual a écrit : On 02/10/2020 10:22 AM, Andrew Morton wrote: On Thu, 6 Feb 2020 13:49:35 +0530 Anshuman Khandual wrote: On 02/06/2020 04:40 AM, kbuild test robot wrote: Hi Anshuman, Thank you for the patch! Yet something to improve: [auto build

Re: [PATCH V13] mm/debug: Add tests validating architecture page table helpers

2020-02-09 Thread Anshuman Khandual
On 02/10/2020 10:22 AM, Andrew Morton wrote: > On Thu, 6 Feb 2020 13:49:35 +0530 Anshuman Khandual > wrote: > >> >> On 02/06/2020 04:40 AM, kbuild test robot wrote: >>> Hi Anshuman, >>> >>> Thank you for the patch! Yet something to improve: >>> >>> [auto build test ERROR on powerpc/next] >>>

Re: [PATCH V13] mm/debug: Add tests validating architecture page table helpers

2020-02-09 Thread Andrew Morton
On Thu, 6 Feb 2020 13:49:35 +0530 Anshuman Khandual wrote: > > On 02/06/2020 04:40 AM, kbuild test robot wrote: > > Hi Anshuman, > > > > Thank you for the patch! Yet something to improve: > > > > [auto build test ERROR on powerpc/next] > > [also build test ERROR on s390/features linus/master

Re: [PATCH V5 06/14] powerpc/vas: Setup thread IRQ handler per VAS instance

2020-02-09 Thread Haren Myneni
On Fri, 2020-02-07 at 16:57 +1100, Michael Neuling wrote: > > /* > > + * Process CRBs that we receive on the fault window. > > + */ > > +irqreturn_t vas_fault_handler(int irq, void *data) > > +{ > > + struct vas_instance *vinst = data; > > + struct coprocessor_request_block buf, *crb; > > +

Re: [PATCH V5 09/14] powerpc/vas: Update CSB and notify process for fault CRBs

2020-02-09 Thread Haren Myneni
Mikey, Thanks for your review comments. On Fri, 2020-02-07 at 16:46 +1100, Michael Neuling wrote: > On Wed, 2020-01-22 at 00:17 -0800, Haren Myneni wrote: > > For each fault CRB, update fault address in CRB (fault_storage_addr) > > and translation error status in CSB so that user space can touch

Re: [PATCH v6 2/6] sysfs: wrap __compat_only_sysfs_link_entry_to_kobj function to change the symlink name

2020-02-09 Thread Michael Ellerman
Sourabh Jain writes: > The __compat_only_sysfs_link_entry_to_kobj function creates a symlink to a > kobject but doesn't provide an option to change the symlink file name. > > This patch adds a wrapper function compat_only_sysfs_link_entry_to_kobj > that extends the

[PATCH] powerpc/8xx: Fix clearing of bits 20-23 in ITLB miss

2020-02-09 Thread Christophe Leroy
In ITLB miss handled the line supposed to clear bits 20-23 on the L2 ITLB entry is buggy and does indeed nothing, leading to undefined value which could allow execution when it shouldn't. Properly do the clearing with the relevant instruction. Fixes: 74fabcadfd43 ("powerpc/8xx: don't use

Re: [PATCH v2] libnvdimm: Update persistence domain value for of_pmem and papr_scm device

2020-02-09 Thread Dan Williams
On Tue, Feb 4, 2020 at 9:21 PM Aneesh Kumar K.V wrote: > > Currently, kernel shows the below values > "persistence_domain":"cpu_cache" > "persistence_domain":"memory_controller" > "persistence_domain":"unknown" > > "cpu_cache" indicates no extra instructions is needed to

[PATCH v2] powerpc/hugetlb: Fix 8M hugepages on 8xx

2020-02-09 Thread Christophe Leroy
With HW assistance all page tables must be 4k aligned, the 8xx drops the last 12 bits during the walk. Redefine HUGEPD_SHIFT_MASK to mask last 12 bits out. HUGEPD_SHIFT_MASK is used to for alignment of page table cache. Fixes: 22569b881d37 ("powerpc/8xx: Enable 8M hugepage support with HW

[PATCH 1/6] powerpc: kernel: no need to check return value of debugfs_create functions

2020-02-09 Thread Greg Kroah-Hartman
When calling debugfs functions, there is no need to ever check the return value. The function can work or not, but the code logic should never do something different based on this. Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman Cc: Hari Bathini Cc:

[PATCH 6/6] powerpc: powernv: no need to check return value of debugfs_create functions

2020-02-09 Thread Greg Kroah-Hartman
When calling debugfs functions, there is no need to ever check the return value. The function can work or not, but the code logic should never do something different based on this. Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman Cc: Sukadev Bhattiprolu Cc:

[PATCH 5/6] powerpc: cell: axon_msi: no need to check return value of debugfs_create functions

2020-02-09 Thread Greg Kroah-Hartman
When calling debugfs functions, there is no need to ever check the return value. The function can work or not, but the code logic should never do something different based on this. Cc: Arnd Bergmann Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman Cc:

[PATCH 4/6] powerpc: mm: ptdump: no need to check return value of debugfs_create functions

2020-02-09 Thread Greg Kroah-Hartman
When calling debugfs functions, there is no need to ever check the return value. The function can work or not, but the code logic should never do something different based on this. Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman Cc: Christophe Leroy Cc:

[PATCH 3/6] powerpc: mm: book3s64: hash_utils: no need to check return value of debugfs_create functions

2020-02-09 Thread Greg Kroah-Hartman
When calling debugfs functions, there is no need to ever check the return value. The function can work or not, but the code logic should never do something different based on this. Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman Cc: "Aneesh Kumar K.V" Cc: Christophe Leroy

[PATCH 2/6] powerpc: kvm: no need to check return value of debugfs_create functions

2020-02-09 Thread Greg Kroah-Hartman
When calling debugfs functions, there is no need to ever check the return value. The function can work or not, but the code logic should never do something different based on this. Because of this cleanup, we get to remove a few fields in struct kvm_arch that are now unused. Cc: Paul Mackerras

Re: [PATCH] powerpc/hugetlb: Fix 8M hugepages on 8xx

2020-02-09 Thread Christophe Leroy
Le 06/02/2020 à 14:50, Christophe Leroy a écrit : Commit 55c8fc3f4930 ("powerpc/8xx: reintroduce 16K pages with HW assistance") redefined pte_t as a struct of 4 pte_basic_t, because in 16K pages mode there are four identical entries in the page table. But hugepd entries for 8k pages require