From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com ([217.140.101.70]:45535 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753607AbbGANHP (ORCPT ); Wed, 1 Jul 2015 09:07:15 -0400 Message-ID: <5593E600.2090107@arm.com> Date: Wed, 01 Jul 2015 14:07:12 +0100 From: Marc Zyngier MIME-Version: 1.0 To: Christoffer Dall , "linux-arm-kernel@lists.infradead.org" CC: Steve Capper , "agraf@suse.de" , Catalin Marinas , "kvmarm@lists.cs.columbia.edu" , "dirk@dmllr.de" , "stable@vger.kernel.org" Subject: Re: [PATCH] arm64: Don't report clear pmds and puds as huge References: <1435752511-7079-1-git-send-email-christoffer.dall@linaro.org> In-Reply-To: <1435752511-7079-1-git-send-email-christoffer.dall@linaro.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: stable-owner@vger.kernel.org List-ID: On 01/07/15 13:08, Christoffer Dall wrote: > The current pmd_huge() and pud_huge() functions simply check if the table > bit is not set and reports the entries as huge in that case. This is > counter-intuitive as a clear pmd/pud cannot also be a huge pmd/pud, and > it is inconsistent with at least arm and x86. > > To prevent others from making the same mistake as me in looking at code > that calls these functions and to fix an issue with KVM on arm64 that > causes memory corruption due to incorrect page reference counting > resulting from this mistake, let's change the behavior. > > Cc: stable@vger.kernel.org > Signed-off-by: Christoffer Dall FWIW: Acked-by: Marc Zyngier M. -- Jazz is not dead. It just smells funny...