> +              * If pmd isn't transhuge but the page is THP and
> +              * is owned by only this process, split it and
> +              * deactivate all pages.
> +              */
> +             if (PageTransCompound(page)) {
> +                     if (page_mapcount(page) != 1)
> +                             goto out;
> +                     get_page(page);
> +                     if (!trylock_page(page)) {
> +                             put_page(page);
> +                             goto out;
> +                     }
> +                     pte_unmap_unlock(orig_pte, ptl);
> +                     if (split_huge_page(page)) {
> +                             unlock_page(page);
> +                             put_page(page);
> +                             pte_offset_map_lock(mm, pmd, addr, &ptl);
> +                             goto out;
> +                     }
> +                     pte = pte_offset_map_lock(mm, pmd, addr, &ptl);
> +                     pte--;
> +                     addr -= PAGE_SIZE;
> +                     continue;
> +             }

looks like this leaks page count if split_huge_page() is succesfull
(returns zero).

--Mika

--
To unsubscribe from this list: send the line "unsubscribe linux-api" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to