All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: qemu-devel@nongnu.org, Luiz Capitulino <lcapitulino@redhat.com>
Subject: Re: [Qemu-devel] [PATCH RFC 0/5] making address spaces 64 bit wide
Date: Mon, 11 Nov 2013 17:59:59 +0100	[thread overview]
Message-ID: <52810D0F.50500@redhat.com> (raw)
In-Reply-To: <1384187560-20557-1-git-send-email-mst@redhat.com>

Il 11/11/2013 17:40, Michael S. Tsirkin ha scritto:
> At the moment, exec ignores high bits in each address,
> for efficiency.
> This is incorrect: devices can do full 64 bit DMA, it's
> only the CPU that is limited by target address space.
> Using full 64 bit addresses was clocked at 12% performance
> hit on a microbenchmark.
> To solve, teach pagetables to skip bits at any level
> and not just the lowest level.
> 
> This should solve the performance problem (only one line
> of code changed on the data path).
> I'm still trying to figure out how to measure speed
> properly with TCG, sending this out for early feedback and flames.

I used this:

x86_64-softmmu/qemu-system-x86_64 -kernel
../../kvm-unit-tests/x86/vmexit.flat  -serial stdio -device
isa-debug-exit,iobase=0xf4

with only one test enabled (I tried both inl_from_qemu and
inl_from_pmtimer) and with roughly the same inlining of the "inb %dx,
%al" instruction that you suggested earlier on the mailing list.

Paolo

      parent reply	other threads:[~2013-11-11 17:00 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-11 16:40 [Qemu-devel] [PATCH RFC 0/5] making address spaces 64 bit wide Michael S. Tsirkin
2013-11-11 16:40 ` [Qemu-devel] [PATCH RFC 1/5] split definitions for exec.c and translate-all.c radix trees Michael S. Tsirkin
2013-11-11 16:40 ` [Qemu-devel] [PATCH RFC 2/5] exec: make address spaces 64-bit wide Michael S. Tsirkin
2013-11-11 16:40 ` [Qemu-devel] [PATCH RFC 3/5] exec: relace leaf with skip Michael S. Tsirkin
2013-11-11 16:41 ` [Qemu-devel] [PATCH RFC 4/5] exec: extend skip field to 3 bits Michael S. Tsirkin
2013-11-11 16:41 ` [Qemu-devel] [PATCH RFC 5/5] exec: memory radix tree page level compression Michael S. Tsirkin
2013-11-11 17:58   ` Eric Blake
2013-11-11 16:59 ` Paolo Bonzini [this message]

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=52810D0F.50500@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=lcapitulino@redhat.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.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.