Re: [PATCH v5 09/11] hugetlb: Introduce generic version of huge_ptep_set_wrprotect

2018-08-03 Thread Alexandre Ghiti
Hi Michael, Thanks, I will then remove those two specific implementations and we'll use the generic ones. I send a v6 asap. Thanks again, Alex On 08/03/2018 10:51 AM, Michael Ellerman wrote: Hi Alex, Sorry missed your previous mail. Alex Ghiti writes: Ok, I tried every defconfig avail

Re: [PATCH v5 09/11] hugetlb: Introduce generic version of huge_ptep_set_wrprotect

2018-08-03 Thread Michael Ellerman
Hi Alex, Sorry missed your previous mail. Alex Ghiti writes: > Ok, I tried every defconfig available: > > - for the nohash/32, I found that I could use mpc885_ads_defconfig and I > activated HUGETLBFS. > I removed the definition of huge_ptep_set_wrprotect from > nohash/32/pgtable.h, add an #er

Re: [PATCH v5 09/11] hugetlb: Introduce generic version of huge_ptep_set_wrprotect

2018-08-02 Thread Alex Ghiti
Ok, I tried every defconfig available: - for the nohash/32, I found that I could use mpc885_ads_defconfig and I activated HUGETLBFS. I removed the definition of huge_ptep_set_wrprotect from nohash/32/pgtable.h, add an #error in include/asm-generic/hugetlb.h right before the generic definition o

Re: [PATCH v5 09/11] hugetlb: Introduce generic version of huge_ptep_set_wrprotect

2018-07-31 Thread Alexandre Ghiti
On 07/31/2018 12:06 PM, Michael Ellerman wrote: Alexandre Ghiti writes: arm, ia64, mips, sh, x86 architectures use the same version of huge_ptep_set_wrprotect, so move this generic implementation into asm-generic/hugetlb.h. Note: powerpc uses twice for book3s/32 and nohash/32 the same versio

Re: [PATCH v5 09/11] hugetlb: Introduce generic version of huge_ptep_set_wrprotect

2018-07-31 Thread Michael Ellerman
Alexandre Ghiti writes: > arm, ia64, mips, sh, x86 architectures use the same version > of huge_ptep_set_wrprotect, so move this generic implementation into > asm-generic/hugetlb.h. > Note: powerpc uses twice for book3s/32 and nohash/32 the same version as > the above architectures, but the modif