From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: "Cédric Le Goater" <clg@kaod.org>,
"David Gibson" <david@gibson.dropbear.id.au>
Cc: linuxppc-dev@lists.ozlabs.org,
Michael Ellerman <mpe@ellerman.id.au>,
Paul Mackerras <paulus@samba.org>
Subject: Re: [PATCH 02/10] powerpc/xive: guest exploitation of the XIVE interrupt controller
Date: Thu, 10 Aug 2017 21:33:51 +1000 [thread overview]
Message-ID: <1502364831.2563.63.camel@kernel.crashing.org> (raw)
In-Reply-To: <2306956d-c447-24af-a20c-5f2eec011164@kaod.org>
On Thu, 2017-08-10 at 08:45 +0200, Cédric Le Goater wrote:
> > The problem with doorbells on POWER9 guests is that they may have
> > to trap and be emulated by the hypervisor, since the guest threads
> > on P9 don't have to match the HW threads of the core.
>
> Well, the pseries cause_ipi() handler does :
>
> if (doorbell_try_core_ipi(cpu))
> return;
>
> to limit the doorbells to the same core. So we should be fine ?
No. It's theorically possible to create a guest that think it has 4
threads on P9 but those threads run on different cores of the host.
The doorbells are useful if KVM uses a "P8 style" whole-core dispatch
model or with PowerVM. We should probably invent some kind of DT
property to tell the guest I suppoes.
> If not
> I suppose we should check CPU_FTR_ARCH_300 and use IPIs only for XIVE.
>
> > Thus it's quite possible that using XIVE for IPIs is actually faster
> > than doorbells in that case.
>
> How can we measure that ? ebizzy may be.
Or a simple socket ping pong with processes pinned to different
threads.
However the current KVM for P9 doesn't do threads yet afaik.
Cheers,
Ben.
next prev parent reply other threads:[~2017-08-10 11:34 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-08 8:56 [PATCH 00/10] guest exploitation of the XIVE interrupt controller Cédric Le Goater
2017-08-08 8:56 ` [PATCH 01/10] powerpc/xive: fix OV5_XIVE_EXPLOIT bits Cédric Le Goater
2017-08-08 8:56 ` [PATCH 02/10] powerpc/xive: guest exploitation of the XIVE interrupt controller Cédric Le Goater
2017-08-09 3:53 ` David Gibson
2017-08-09 8:48 ` Cédric Le Goater
2017-08-10 4:28 ` David Gibson
2017-08-10 4:46 ` Benjamin Herrenschmidt
2017-08-10 5:54 ` David Gibson
2017-08-10 7:04 ` Cédric Le Goater
2017-08-10 6:45 ` Cédric Le Goater
2017-08-10 11:33 ` Benjamin Herrenschmidt [this message]
2017-08-10 7:19 ` Cédric Le Goater
2017-08-10 11:36 ` Benjamin Herrenschmidt
2017-08-11 3:55 ` David Gibson
2017-08-08 8:56 ` [PATCH 03/10] powerpc/xive: rename xive_poke_esb in xive_esb_read Cédric Le Goater
2017-08-09 3:55 ` David Gibson
2017-08-09 7:12 ` Cédric Le Goater
2017-08-09 7:31 ` David Gibson
2017-08-08 8:56 ` [PATCH 04/10] powerpc/xive: introduce xive_esb_write Cédric Le Goater
2017-08-08 8:56 ` [PATCH 05/10] powerpc/xive: add the HW IRQ number under xive_irq_data Cédric Le Goater
2017-08-08 8:56 ` [PATCH 06/10] powerpc/xive: introduce H_INT_ESB hcall Cédric Le Goater
2017-08-08 8:56 ` [PATCH 07/10] powerpc/xive: add XIVE exploitation mode to CAS Cédric Le Goater
2017-08-10 10:20 ` Cédric Le Goater
2017-08-08 8:56 ` [PATCH 08/10] powerpc/xive: take into account '/ibm, plat-res-int-priorities' Cédric Le Goater
2017-08-09 4:02 ` [PATCH 08/10] powerpc/xive: take into account '/ibm,plat-res-int-priorities' David Gibson
2017-08-09 7:14 ` Cédric Le Goater
2017-08-10 0:54 ` David Gibson
2017-08-08 8:56 ` [PATCH 09/10] powerpc/xive: improve debugging macros Cédric Le Goater
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1502364831.2563.63.camel@kernel.crashing.org \
--to=benh@kernel.crashing.org \
--cc=clg@kaod.org \
--cc=david@gibson.dropbear.id.au \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mpe@ellerman.id.au \
--cc=paulus@samba.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.