From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 23 Jul 2012 09:46:23 +1000 From: Paul Mackerras To: "Aneesh Kumar K.V" Subject: Re: [PATCH -V3 04/11] arch/powerpc: Rename va to vpn Message-ID: <20120722234623.GD17790@bloggs.ozlabs.ibm.com> References: <1341839621-28332-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> <1341839621-28332-5-git-send-email-aneesh.kumar@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1341839621-28332-5-git-send-email-aneesh.kumar@linux.vnet.ibm.com> Cc: linuxppc-dev@lists.ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, Jul 09, 2012 at 06:43:34PM +0530, Aneesh Kumar K.V wrote: > From: "Aneesh Kumar K.V" > > Rename the variable to better reflect the values. No functional change > in this patch. > > Signed-off-by: Aneesh Kumar K.V I think you missed a couple: kernel_map_linear_page and kernel_unmap_linear_page are now broken since you change va to vpn in the declaration but not in the uses, and you change host_va to host_vpn in struct hpte_cache but don't fix up the use in kvmppc_mmu_invalidate_pte. How many configs did you compile-test this with? Paul.