All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] spapr guests warn "System page size 0x2000 is not enabled in page_size_mask (0x11000). Performance may be slow" on SPARC hosts
@ 2017-06-15 13:33 Peter Maydell
  2017-06-19  9:39 ` Alexander Graf
  0 siblings, 1 reply; 5+ messages in thread
From: Peter Maydell @ 2017-06-15 13:33 UTC (permalink / raw)
  To: QEMU Developers; +Cc: David Gibson, Alexander Graf

I've just noticed that on a SPARC host, some of the PPC guests
warn during make check:

  /ppc64/prom-env/pseries:
qemu-system-ppc64: System page size 0x2000 is not enabled in
page_size_mask (0x11000). Performance may be slow

Is this really a performance problem on a TCG guest?
It makes the 'make check' output a bit noisy, but not
unbearably so.

thanks
-- PMM

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] spapr guests warn "System page size 0x2000 is not enabled in page_size_mask (0x11000). Performance may be slow" on SPARC hosts
  2017-06-15 13:33 [Qemu-devel] spapr guests warn "System page size 0x2000 is not enabled in page_size_mask (0x11000). Performance may be slow" on SPARC hosts Peter Maydell
@ 2017-06-19  9:39 ` Alexander Graf
  2017-06-19 10:32   ` Thomas Huth
  0 siblings, 1 reply; 5+ messages in thread
From: Alexander Graf @ 2017-06-19  9:39 UTC (permalink / raw)
  To: Peter Maydell, QEMU Developers; +Cc: David Gibson

On 06/15/2017 03:33 PM, Peter Maydell wrote:
> I've just noticed that on a SPARC host, some of the PPC guests
> warn during make check:
>
>    /ppc64/prom-env/pseries:
> qemu-system-ppc64: System page size 0x2000 is not enabled in
> page_size_mask (0x11000). Performance may be slow
>
> Is this really a performance problem on a TCG guest?
> It makes the 'make check' output a bit noisy, but not
> unbearably so.

 From what I can tell this really only affects DDW which should be page 
size agnostic when only using emulated devices. Maybe Alexey thought of 
using VFIO-pci devices in a TCG guest?


Alex

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] spapr guests warn "System page size 0x2000 is not enabled in page_size_mask (0x11000). Performance may be slow" on SPARC hosts
  2017-06-19  9:39 ` Alexander Graf
@ 2017-06-19 10:32   ` Thomas Huth
  2017-06-20  1:02     ` David Gibson
  0 siblings, 1 reply; 5+ messages in thread
From: Thomas Huth @ 2017-06-19 10:32 UTC (permalink / raw)
  To: Alexander Graf, Peter Maydell, QEMU Developers
  Cc: David Gibson, Alexey Kardashevskiy

On 19.06.2017 11:39, Alexander Graf wrote:
> On 06/15/2017 03:33 PM, Peter Maydell wrote:
>> I've just noticed that on a SPARC host, some of the PPC guests
>> warn during make check:
>>
>>    /ppc64/prom-env/pseries:
>> qemu-system-ppc64: System page size 0x2000 is not enabled in
>> page_size_mask (0x11000). Performance may be slow
>>
>> Is this really a performance problem on a TCG guest?
>> It makes the 'make check' output a bit noisy, but not
>> unbearably so.
> 
> From what I can tell this really only affects DDW which should be page
> size agnostic when only using emulated devices. Maybe Alexey thought of
> using VFIO-pci devices in a TCG guest?

Maybe we could at least silence the warning by checking qtest_enabled()?

 Thomas

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] spapr guests warn "System page size 0x2000 is not enabled in page_size_mask (0x11000). Performance may be slow" on SPARC hosts
  2017-06-19 10:32   ` Thomas Huth
@ 2017-06-20  1:02     ` David Gibson
  2017-06-20  2:54       ` Alexey Kardashevskiy
  0 siblings, 1 reply; 5+ messages in thread
From: David Gibson @ 2017-06-20  1:02 UTC (permalink / raw)
  To: Thomas Huth
  Cc: Alexander Graf, Peter Maydell, QEMU Developers,
	Alexey Kardashevskiy

[-- Attachment #1: Type: text/plain, Size: 1387 bytes --]

On Mon, Jun 19, 2017 at 12:32:48PM +0200, Thomas Huth wrote:
> On 19.06.2017 11:39, Alexander Graf wrote:
> > On 06/15/2017 03:33 PM, Peter Maydell wrote:
> >> I've just noticed that on a SPARC host, some of the PPC guests
> >> warn during make check:
> >>
> >>    /ppc64/prom-env/pseries:
> >> qemu-system-ppc64: System page size 0x2000 is not enabled in
> >> page_size_mask (0x11000). Performance may be slow
> >>
> >> Is this really a performance problem on a TCG guest?
> >> It makes the 'make check' output a bit noisy, but not
> >> unbearably so.
> > 
> > From what I can tell this really only affects DDW which should be page
> > size agnostic when only using emulated devices. Maybe Alexey thought of
> > using VFIO-pci devices in a TCG guest?
> 
> Maybe we could at least silence the warning by checking
> qtest_enabled()?


So,

1) Yes, we should at least silence the warning.

2) Certainly the mentioned slowdown shouldn't be significant for TCG
guests.

3) I _think_ the restriction which causes the slowdown doesn't even
exist for TCG, but I haven't 100% convinced myself of this.  I've been
sick lately, so my brain isn't working all that well.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] spapr guests warn "System page size 0x2000 is not enabled in page_size_mask (0x11000). Performance may be slow" on SPARC hosts
  2017-06-20  1:02     ` David Gibson
@ 2017-06-20  2:54       ` Alexey Kardashevskiy
  0 siblings, 0 replies; 5+ messages in thread
From: Alexey Kardashevskiy @ 2017-06-20  2:54 UTC (permalink / raw)
  To: David Gibson, Thomas Huth; +Cc: Alexander Graf, Peter Maydell, QEMU Developers

[-- Attachment #1: Type: text/plain, Size: 1827 bytes --]

On 20/06/17 11:02, David Gibson wrote:
> On Mon, Jun 19, 2017 at 12:32:48PM +0200, Thomas Huth wrote:
>> On 19.06.2017 11:39, Alexander Graf wrote:
>>> On 06/15/2017 03:33 PM, Peter Maydell wrote:
>>>> I've just noticed that on a SPARC host, some of the PPC guests
>>>> warn during make check:
>>>>
>>>>    /ppc64/prom-env/pseries:
>>>> qemu-system-ppc64: System page size 0x2000 is not enabled in
>>>> page_size_mask (0x11000). Performance may be slow
>>>>
>>>> Is this really a performance problem on a TCG guest?
>>>> It makes the 'make check' output a bit noisy, but not
>>>> unbearably so.
>>>
>>> From what I can tell this really only affects DDW which should be page
>>> size agnostic when only using emulated devices. Maybe Alexey thought of
>>> using VFIO-pci devices in a TCG guest?
>>
>> Maybe we could at least silence the warning by checking
>> qtest_enabled()?
> 
> 
> So,
> 
> 1) Yes, we should at least silence the warning.

Agree, wrapping it into kvm_enabled() should do it.


> 
> 2) Certainly the mentioned slowdown shouldn't be significant for TCG
> guests.

It is not significant at all. While in-kernel TCE handling improves
throughput for VFIO devices quite a lot (200mb/s -> 1000mb/s), TCG could
not do any better than 20mb/s anyway.


> 3) I _think_ the restriction which causes the slowdown doesn't even
> exist for TCG, but I haven't 100% convinced myself of this.  I've been
> sick lately, so my brain isn't working all that well.

Mismatch between system page size and IOMMU page size means H_PUT_TCE is
going to be called more often than needed; if VFIO is involved, then more
ioctl(vfio_container_fd, DMAP_MAP) calls. But - we cannot do 8K IOMMU pages
at all, SPARC does not support VFIO (or does it?), and TCG is slow anyway.



-- 
Alexey


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 839 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-06-20  2:54 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-15 13:33 [Qemu-devel] spapr guests warn "System page size 0x2000 is not enabled in page_size_mask (0x11000). Performance may be slow" on SPARC hosts Peter Maydell
2017-06-19  9:39 ` Alexander Graf
2017-06-19 10:32   ` Thomas Huth
2017-06-20  1:02     ` David Gibson
2017-06-20  2:54       ` Alexey Kardashevskiy

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.