From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f49.google.com (mail-pj1-f49.google.com [209.85.216.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A43723E9284 for ; Thu, 9 Jul 2026 08:51:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783587086; cv=none; b=lTrBlfpMtrMFaeeuehxU9j2s/Dn1E3GFuc6KmJcaAg2ut9TowzUsHzMSwP6OB/0ONsWhVedvqA8nYfWFkRqT88zSi6U3Y90/ISbZpbR1yt0kYbWCXxs2YZ6lTNEYnJKVbNd9jsymB/swqm/xT4tBqKXJiwY9yeWoqbtqqGnFC/Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783587086; c=relaxed/simple; bh=VjsHFBKy+x9J5i3Up358292Qp0m7f8AymQAqtW4mK18=; h=From:To:Cc:Subject:In-Reply-To:Date:Message-ID:References; b=Rz6zOo0cWFYNt+ezclIqSTWABYx/CJlimVGuByRfm/FSv3w5xjU3F0nTkUCMev7MbeIuMUayFr+fV0bA9NuItzBsgfj4k2OmZd/E1nXq85AitIelGHbVDEbWH/82jjwvNQxZvpdSV+iIpUiz0Zmp7EqvhL92b4radVPHqowi8Rc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=STG09mU7; arc=none smtp.client-ip=209.85.216.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="STG09mU7" Received: by mail-pj1-f49.google.com with SMTP id 98e67ed59e1d1-381507c9380so1168972a91.0 for ; Thu, 09 Jul 2026 01:51:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783587085; x=1784191885; darn=vger.kernel.org; h=references:message-id:date:in-reply-to:subject:cc:to:from:from:to :cc:subject:date:message-id:reply-to:content-type; bh=CNp1e35ro3cQgZeEMYQvoxd6PA+bCrB79PJw0+bppNk=; b=STG09mU72e/56VKo9of+DgK2SDlIYMWIVQCtKXSl+v81GtkrMRmgAUp8o6WGOT7xa5 PhOVlcl5S8YN3Y7NsIeGmMwC8S0N3tt3xd/aCKnL5C0hZHaPqAg9ZVdMoExc2x0fub1U sDaGKZXu9CKw/dZlGREXtq3D9V5k4IR7bqjHD2ixfinkRh0CDXr3YLXHG5QIjQprzE/h aRAiJXKzCVDKUvDNG1rD02r04LhH5p3x+DiZUPKLrcssPm91zl1+dlB3hmPY6dK+4qCT FQSp/OdD8aDucaxLXxVLMgGPY/djM852KMlvJqgquNyTH50TPSCW37IWgIUKJ+HPk/0p sdDQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783587085; x=1784191885; h=references:message-id:date:in-reply-to:subject:cc:to:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=CNp1e35ro3cQgZeEMYQvoxd6PA+bCrB79PJw0+bppNk=; b=NKtIDB014lJ3QSWy3sh6oViG7AerskoyMSILDA8cnYjt5663KbB8SCkx+9RlTXgrjs ZfNneds70UkUiUqJOcg5KVcLGEBMRPdA2qj4S6Vcgd2JKwSOQ/Av1vdMCPYv0SpcWwSL rPL2+1QVLqLBF+kCiSg6QLlKjSpbeBAaTGbbBAvr4Cv54nWuth8vjiF9d66MRnreILSw zMPgZ8b5c3rKZ8Vf01tASRfgmqifGr0MhPJ4Q/ctQ7eeLhetG05+fpn/w96I+j5NcZce NRzKgLHvKfJsxbZPAvXxeEFOl9R/l10wMu0uav+7kMpwuLQV7QYQF2oiJofP+7RNkiQM dmlw== X-Forwarded-Encrypted: i=1; AHgh+RoPmc8ojV8buEqDm6VZlAp34tTFkNxqzdQwqX8q0V426ssyhARVQYg9fUECjC5LfoF+OuKLXOVEuZZednE=@vger.kernel.org X-Gm-Message-State: AOJu0Yz7x+HNH9lxSNbJN3QUGtFelqFMgAKkb0g5z39mpEuSLmN+I9tm C4g2CTThW3eaPtCnB+cirNSw83I/uvbd2ZDFi7r4hgDsVlFoa74ZMkib X-Gm-Gg: AfdE7cmBVWLxZlN8dqH1Tm93OpXmR+55kQ/+3U3xkcd06dn9Ksak0RupB0F4qVvWw7g qSFJy/ysaXTJtZBKKn4txUkENgyRkyZla/WRFb8sQiYqhtCONi7jK0mw0312f35NEE98JvAciqv KS50kujqK3Yt5NLMBhJdu/I3FL9Gl2gS8gH341KK7Fj2f/DMCwNE+qHeIy19BBmIadJPFKq2pr/ 9fdWEfaLaKhcNM0eWM0YSN0nRsi36l/6E/VWOD7LIFRECznkcFAwXnyjWYczrcAga73fjyl0Dlp D33C0CBmOZj2NflK6nB4y6pWcE+L/eWJ35apcmtEGbCB1QjjTUAFXRRtJF6oGn39G32MMYia/cY Yu8CRRTkI1W+91oz4+TSRy6512Pu8W3lXwyER6MZ8MkWQXEC9Au24RMi4DWvcbSCJM0BC71+1Wu l4oxFnPXMie4g= X-Received: by 2002:a05:6a20:3d0b:b0:3bf:7ab7:98a4 with SMTP id adf61e73a8af0-3c0bc95bad7mr8044555637.27.1783587084985; Thu, 09 Jul 2026 01:51:24 -0700 (PDT) Received: from pve-server ([49.205.216.49]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-311a6115e61sm5516616eec.22.2026.07.09.01.51.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Jul 2026 01:51:24 -0700 (PDT) From: Ritesh Harjani (IBM) To: David Woodhouse , Paolo Bonzini Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, Madhavan Srinivasan , Nicholas Piggin , Michael Ellerman , "Christophe Leroy (CS GROUP)" , linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH] KVM: Use vma_lookup() in kvm_host_page_size() In-Reply-To: <71775f561c4b7fb23b7af0725c6ce232a4b27b20.camel@infradead.org> Date: Thu, 09 Jul 2026 14:16:36 +0530 Message-ID: <7bn4wnwz.ritesh.list@gmail.com> References: <71775f561c4b7fb23b7af0725c6ce232a4b27b20.camel@infradead.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: David Woodhouse writes: > From: David Woodhouse > > kvm_host_page_size() uses find_vma() to determine the host page size > backing a guest page. find_vma() returns the next VMA when the address > falls in an unmapped hole, which means an unmapped address inherits the > page size of a completely unrelated VMA at a higher address. This is > never a meaningful result. > > Replace find_vma() with vma_lookup() so that a hole address returns > NULL, causing the function to return PAGE_SIZE (the safe default) > rather than an unrelated VMA's page size. > That's exactly right. > This is currently only cosmetic, as the only caller is in PPC book3s > support (kvmppc_xive_native_set_queue_config()) and subsequently returns > failure if !gfn_to_page() anyway. > Yup. The change make sense to me. Feel free to add: Reviewed-by: Ritesh Harjani (IBM) > Signed-off-by: David Woodhouse > --- > virt/kvm/kvm_main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > index 89489996fbc1..d33761123459 100644 > --- a/virt/kvm/kvm_main.c > +++ b/virt/kvm/kvm_main.c > @@ -2693,7 +2693,7 @@ unsigned long kvm_host_page_size(struct kvm_vcpu *vcpu, gfn_t gfn) > return PAGE_SIZE; > > mmap_read_lock(current->mm); > - vma = find_vma(current->mm, addr); > + vma = vma_lookup(current->mm, addr); > if (!vma) > goto out; > > -- > 2.43.0