On Sat, Mar 13, 2021 at 05:32:19AM -0500, Miaohe Lin wrote:
> +static inline void try_to_split_huge_pmd_address(struct vm_area_struct *vma, 
> unsigned long address)
> +{
> +     if (address & ~HPAGE_PMD_MASK &&
> +         range_in_vma(vma, address & HPAGE_PMD_MASK,
> +                      (address & HPAGE_PMD_MASK) + HPAGE_PMD_SIZE))
> +             split_huge_pmd_address(vma, address, false, NULL);
> +}

This isn't "try to split".  This is "split if needed".  I think the
comments also need to be moved up here.

I'm not even sure this is really an improvement.

Reply via email to