From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Mackerras Subject: Re: [PATCH] KVM: PPC: Book3S PR: Set hflag to indicate that POWER9 supports 1T segments Date: Tue, 18 Dec 2018 12:00:55 +1100 Message-ID: <20181218010055.GC25748@blackberry> References: <20181207034326.3564-1-sjitindarsingh@gmail.com> <20181207034326.3564-2-sjitindarsingh@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: aik@ozlabs.ru, linuxppc-dev@lists.ozlabs.org, kvm@vger.kernel.org, kvm-ppc@vger.kernel.org To: Suraj Jitindar Singh Return-path: Content-Disposition: inline In-Reply-To: <20181207034326.3564-2-sjitindarsingh@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linuxppc-dev-bounces+glppe-linuxppc-embedded-2=m.gmane.org@lists.ozlabs.org Sender: "Linuxppc-dev" List-Id: kvm.vger.kernel.org On Fri, Dec 07, 2018 at 02:43:18PM +1100, Suraj Jitindar Singh wrote: > When booting a kvm-pr guest on a POWER9 machine the following message is > observed: > "qemu-system-ppc64: KVM does not support 1TiB segments which guest expects" > > This is because the guest is expecting to be able to use 1T segments > however we don't indicate support for it. This is because we don't set > the BOOK3S_HFLAG_MULTI_PGSIZE flag in the hflags in kvmppc_set_pvr_pr() > on POWER9. > > POWER9 does indeed have support for 1T segments, so add a case for > POWER9 to the switch statement to ensure it is set. > > Signed-off-by: Suraj Jitindar Singh Thanks, patch applied to my kvm-ppc-next branch. Paul.