From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oo1-f52.google.com (mail-oo1-f52.google.com [209.85.161.52]) (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 11B4647CA65 for ; Thu, 10 Sep 2026 11:46:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.161.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789040798; cv=none; b=j8fiFPkJfbXG3rs9jTGRsnDZbZM7SAuUP1VSCS/qsTGtB4S9JsLp59V5DM2AqYoqxnm5ilxuNRW5RnzX38OSWaJD7hICsj9/xnxwFmz0wa6Q6xPYrH9RBd27U+Ebt3KT9hLZWTJpKDFphw/awaaX4chG9vMe9YhTEzvU2T9UrTY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789040798; c=relaxed/simple; bh=+9G8GC05NdLHXF8cStdhp+Ruyt/HZZEYr3s5pM8Whfc=; h=From:To:Cc:Subject:In-Reply-To:Date:Message-ID:References; b=WejcsUnWOqaJj0OYGfdOpqWdK2P8wKlV8+3SsyXu9/Qf9ZMtL5WnFP39+QuhI0AH78DnW0RvVcLW72oTKT4aMlcwl+dRol7+moPlKT84QlDnIIaGG0LuRTyzw4Aafx7PPBGiCiglL99vDm+KjzcF7gafASXwgGGhtRH8WrKFmZE= 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=b2/VTanL; arc=none smtp.client-ip=209.85.161.52 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="b2/VTanL" Received: by mail-oo1-f52.google.com with SMTP id 006d021491bc7-6b1b3d7f10eso4891438eaf.3 for ; Thu, 10 Sep 2026 04:46:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1789040785; x=1789645585; 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=4oV1R7Sem2YZktphiH9OH9k3wSSCyVuVuzldh1QWGuY=; b=b2/VTanL4b72R4AcNHSpynNtDCaa4WE0PKorNmjOfAm7uCvAXMQAtf3fYC7MEWScop DSgX8sQlt7HWIwoOkABi2eCWTAWBQk+p4rznr9uonM0Aomgy/BqTiScfx+qz8ZjTf0fb W6VYob07EjgkFJh6tp5bbV1j7kmFbulSvWCHlaaEtsd1oJfZWdnlhKVQRd2MTRpDmaMy eMr+v9+Qv2on9A8lzWtV3AA0B1Ru5tUveO7XCGvmqn8iksMEJoePH0MjwSd8+1NFDYg8 SEgF1Kz2zyEewqv3gFPCoFBd/rCP0e1T6cSpJHG6tTfFxBDlhR/luN8smZ9BKXHKoOhd va8Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1789040785; x=1789645585; 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=4oV1R7Sem2YZktphiH9OH9k3wSSCyVuVuzldh1QWGuY=; b=KP3tzSTh1ZnAsyQFKAGy1JzQXGR8MqAM4ispIjye0lJ8uwZCKuByPec2UeoRI94Ow3 eq4WLvFSjxPoD3rwhE3DkFLbnMgMcqTdUVZpfbN7XYscjIv2kVzp7JQDcIpJswQtEpWr gme+WYqdsjtYDyJE5pjtFeZplu9es64aQn4OzkOzOvAyegCOxr3mxmGsgPnxv0Z4iaru GhtSwj9DHoDSlrCQwzi8sEW+x9XY26L8hOWTefanZ82W41rEGU9QLsHiTKAsMnD9trRH PPkZDdHaIW5DNCLy1kgF+5oJXOSuDNWGs6d//VqcGDPd8On14JzERpscjm/wu5gYA7z2 KyTw== X-Forwarded-Encrypted: i=1; AKwUvBwf6urQlGAXGE/K5yLKWQ/vQ5EMZPBDPuFSidiwVhirg0Vt1KZpu2b7EBcdhtFUUmxkHyE6PA+PFU9piAw=@vger.kernel.org X-Gm-Message-State: AFuF++nlsZbYhcO38WGmFnzT3dvbOwm6ocmij81bHFNotE2Vx6ad+pft p6LKi4auH7PIaohSssRjlherqMYGBql+9dVA5tIxbq655Uw5nRYXjDaT X-Gm-Gg: AYBFou13tPPg+uq7S/FFbPJApsHfhyKNUCiMZ5q63tVu902Ob+mgV+y5jZFZ7gvVO5m qbboMrzWgLF9Cv04KB8kEZuS6mhgKF1HNxtIeT8z0cIIR0Tv8T07k9h96JQQCRxu8uCSTVLWQ76 Euq2/xjysrt3BK8Ge5ixKwb9Xwr4RqtnZ+c228yiXGrlv8c72SqGoYiErB8YWo2BhpFFmqaNNOf 66nlOKAqjoqb3yP48CAbkVCUU63d7lySdJe5TGqyyqLBwpYcJNdHr46bjobkX0GI1XJSDRzELQ9 K7am4/wcgOd2fyz44EtSEYL1WbMTWsk+v/L8auJSxkXghRKhF3R9OT8MLCgu3rijuljzHX01TQa O5+QK2nBsvmvowngXsKcFkmQTffVAonyzvIRb2exYWvNKBqaYmsbcjJZaXPTOyrghK9AKne/TGP x289OqH3SFQRU2vZCHwWdnWvfzB3zPmilFa29Zr6Fto6pteN2IAI/KsWmPaM3M/VnILomvLhX99 E4wOtuQihPM1zOsCkCcgiBSkcIDTwe7jgAZJPDINTqLQL9lOluDtyM= X-Received: by 2002:a05:6820:180d:b0:6b7:8415:d78f with SMTP id 006d021491bc7-6b78415d8bbmr19129181eaf.58.1789040785360; Thu, 10 Sep 2026 04:46:25 -0700 (PDT) Received: from pve-server ([49.205.216.49]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-143240d86c3sm46857652c88.0.2026.09.10.04.46.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 10 Sep 2026 04:46:24 -0700 (PDT) From: Ritesh Harjani (IBM) To: Sean Christopherson , Marc Zyngier , Oliver Upton , Sean Christopherson , Paolo Bonzini , Tianrui Zhao , Bibo Mao , Huacai Chen , Anup Patel , Paul Walmsley , Palmer Dabbelt , Albert Ou , Christian Borntraeger , Janosch Frank , Claudio Imbrenda Cc: Fuad Tabba , Joey Gouly , Steffen Eiden , Suzuki K Poulose , Zenghui Yu , Atish Patra , Alexandre Ghiti , David Hildenbrand , linux-arm-kernel@lists.infradead.org, kvmarm@lists.linux.dev, kvm@vger.kernel.org, loongarch@lists.linux.dev, kvm-riscv@lists.infradead.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, Itaru Kitayama , Nicholas Piggin Subject: Re: [PATCH v2 17/20] KVM: selftests: Take the memory region type, not memslot, in page allocators In-Reply-To: <20260902164123.2546762-18-seanjc@google.com> Date: Thu, 10 Sep 2026 17:10:56 +0530 Message-ID: <33vh1ghj.ritesh.list@gmail.com> References: <20260902164123.2546762-1-seanjc@google.com> <20260902164123.2546762-18-seanjc@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Sean Christopherson writes: > Take the memory region type instead of the memslot in all page allocator > APIs, except for the innermost quad-underscores function. This will allow > automatically selecting the minimum GPA based on the allocation type, which > can't be done using the memslot due to all regions sharing a single memslot > by default. PowerPC support will also leverage the type to ensure that > page table allocations are naturally aligned. > yup, PowerPC can leverage this. > No functional change intended. > Sure, even though the callers are changed to pass type instead of slot, the inner page allocator is moved to pass vm->memslots[type] to ____vm_phy_pages_alloc(). So no functional change. LGTM. Reviewed-by: Ritesh Harjani (IBM)