Re: [PATCH uprobe, thp 3/4] uprobe: support huge page by only splitting the pmd

From: Song Liu
Date: Thu May 30 2019 - 13:29:27 EST




> On May 30, 2019, at 4:08 AM, William Kucharski <william.kucharski@xxxxxxxxxx> wrote:
>
>
> Is there any reason to worry about supporting PUD-sized uprobe pages if
> CONFIG_HAVE_ARCH_TRANSPARENT_HUGEPAGE_PUD is defined? I would prefer
> not to bake in the assumption that "huge" means PMD-sized and more than
> it already is.
>
> For example, if CONFIG_HAVE_ARCH_TRANSPARENT_HUGEPAGE_PUD is configured,
> mm_address_trans_huge() should either make the call to pud_trans_huge()
> if appropriate, or a VM_BUG_ON_PAGE should be added in case the routine
> is ever called with one.
>
> Otherwise it looks pretty reasonable to me.
>
> -- Bill
>

I will try that in v2.

Thanks,
Song