Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1366750
| From | Andrew Morton <akpm@linux-foundation.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v4 3/8] mm: Add support for PUD-sized transparent hugepages |
| Date | 2016-03-30 00:20 +0200 |
| Message-ID | <ria9t-4ad-25@gated-at.bofh.it> (permalink) |
| References | <qWYZk-35l-11@gated-at.bofh.it> <qWZ90-38Q-3@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Sun, 31 Jan 2016 23:09:30 +1100 Matthew Wilcox <matthew.r.wilcox@intel.com> wrote:
> From: Matthew Wilcox <willy@linux.intel.com>
>
> The current transparent hugepage code only supports PMDs. This patch
> adds support for transparent use of PUDs with DAX. It does not include
> support for anonymous pages.
>
> Most of this patch simply parallels the work that was done for huge PMDs.
> The only major difference is how the new ->pud_entry method in mm_walk
> works. The ->pmd_entry method replaces the ->pte_entry method, whereas
> the ->pud_entry method works along with either ->pmd_entry or ->pte_entry.
> The pagewalk code takes care of locking the PUD before calling ->pud_walk,
> so handlers do not need to worry whether the PUD is stable.
Why is this patchset always so hard to compile :(
> ...
>
> --- a/include/linux/pfn_t.h
> +++ b/include/linux/pfn_t.h
> @@ -82,6 +82,13 @@ static inline pmd_t pfn_t_pmd(pfn_t pfn, pgprot_t pgprot)
> {
> return pfn_pmd(pfn_t_to_pfn(pfn), pgprot);
> }
> +
> +#ifdef CONFIG_HAVE_ARCH_TRANSPARENT_HUGEPAGE_PUD
> +static inline pud_t pfn_t_pud(pfn_t pfn, pgprot_t pgprot)
> +{
> + return pfn_pud(pfn_t_to_pfn(pfn), pgprot);
> +}
> +#endif
> #endif
>
> #ifdef __HAVE_ARCH_PTE_DEVMAP
> @@ -98,5 +105,6 @@ static inline bool pfn_t_devmap(pfn_t pfn)
> }
> pte_t pte_mkdevmap(pte_t pte);
> pmd_t pmd_mkdevmap(pmd_t pmd);
> +pud_t pud_mkdevmap(pud_t pud);
arm allnoconfig:
In file included from kernel/memremap.c:17:
include/linux/pfn_t.h:107: error: 'pud_mkdevmap' declared as function returning an array
because it expands to
pgd_t pud_mkdevmap(pgd_t pud);
and
typedef unsigned long pgd_t[2];
Also the patch provides no implementation of pud_mkdevmap() so it's
obviously going to break bisection.
Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread
Re: [PATCH v4 3/8] mm: Add support for PUD-sized transparent hugepages Andrew Morton <akpm@linux-foundation.org> - 2016-03-30 00:20 +0200 RE: [PATCH v4 3/8] mm: Add support for PUD-sized transparent hugepages "Wilcox, Matthew R" <matthew.r.wilcox@intel.com> - 2016-03-30 16:10 +0200
csiph-web