lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20111013131540.GA3328@redhat.com>
Date:	Thu, 13 Oct 2011 15:15:40 +0200
From:	Andrea Arcangeli <aarcange@...hat.com>
To:	Hillf Danton <dhillf@...il.com>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	Johannes Weiner <jweiner@...hat.com>, linux-mm@...ck.org,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] mm/huge_memory: Clean up typo when updating mmu cache

On Thu, Oct 13, 2011 at 08:52:22PM +0800, Hillf Danton wrote:
> Hi Andrea
> 
> There are three cases of update_mmu_cache() in the file, and the case in
> function collapse_huge_page() has a typo, namely the last parameter used,
> which is corrected based on the other two cases.
> 
> Due to the define of update_mmu_cache by X86, the only arch that implements
> THP currently, the change here has no really crystal point, but one or two
> minutes of efforts could be saved for those archs that are likely to support
> THP in future.

Yes, like the previous one, they make no runtime difference today, but
they may save some debug time to who's porting THP to some other arch
in the future, so it's good to clean this up, thanks.

> Signed-off-by: Hillf Danton <dhillf@...il.com>
> ---
> 
> --- a/mm/huge_memory.c	Sat Aug 13 11:45:14 2011
> +++ b/mm/huge_memory.c	Thu Oct 13 20:07:29 2011
> @@ -1906,7 +1906,7 @@ static void collapse_huge_page(struct mm
>  	BUG_ON(!pmd_none(*pmd));
>  	page_add_new_anon_rmap(new_page, vma, address);
>  	set_pmd_at(mm, address, pmd, _pmd);
> -	update_mmu_cache(vma, address, entry);
> +	update_mmu_cache(vma, address, _pmd);
>  	prepare_pmd_huge_pte(pgtable, mm);
>  	mm->nr_ptes--;
>  	spin_unlock(&mm->page_table_lock);

Reviewed-by: Andrea Arcangeli <aarcange@...hat.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ