From: "Michael S. Tsirkin" <mst@redhat.com>
To: Isaku Yamahata <yamahata@valinux.co.jp>
Cc: qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [PATCH 0/3] piix_pci: optimize irq data path
Date: Thu, 17 Mar 2011 16:42:29 +0200 [thread overview]
Message-ID: <20110317144229.GB22213@redhat.com> (raw)
In-Reply-To: <cover.1300368943.git.yamahata@valinux.co.jp>
On Thu, Mar 17, 2011 at 10:49:50PM +0900, Isaku Yamahata wrote:
> This patch series optimizes irq data path of piix_pci.
> So far piix3 tracks each pirq level and checks whether a given pic pins is
> asserted by seeing if each pirq is mapped into the pic pin.
> This is independent on irq routing, but data path is on slow path.
>
> Given that irq routing is rarely changed and asserting pic pins is on
> data path, the path that asserts pic pins should be optimized and
> chainging irq routing should be on slow path.
> The new behavior with this patch series is to use bitmap which is addressed
> by pirq and pic pins with a given irq routing.
> When pirq is asserted, the bitmap is set and see if the pic pins is
> asserted by checking the bitmaps.
> When irq routing is changed, rebuild the bitmap and re-assert pic pins.
>
> Isaku Yamahata (3):
> pci: add accessor function to get irq levels
> piix_pci: eliminate PIIX3State::pci_irq_levels
> piix_pci: optimize set irq path
>
> hw/pci.c | 7 +++
> hw/pci.h | 1 +
> hw/piix_pci.c | 126 ++++++++++++++++++++++++++++++++++++++++++++++++---------
> 3 files changed, 115 insertions(+), 19 deletions(-)
Thanks, I'll apply 1-2, 3 can be improved a bit but nothing major so
if you are busy I can apply as is and we can add fixups on top.
Let me know.
--
MST
prev parent reply other threads:[~2011-03-17 14:42 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-17 13:49 [Qemu-devel] [PATCH 0/3] piix_pci: optimize irq data path Isaku Yamahata
2011-03-17 13:49 ` [Qemu-devel] [PATCH 1/3] pci: add accessor function to get irq levels Isaku Yamahata
2011-03-17 13:49 ` [Qemu-devel] [PATCH 2/3] piix_pci: eliminate PIIX3State::pci_irq_levels Isaku Yamahata
2011-03-17 13:49 ` [Qemu-devel] [PATCH 3/3] piix_pci: optimize set irq path Isaku Yamahata
2011-03-17 14:41 ` [Qemu-devel] " Michael S. Tsirkin
2011-03-17 22:56 ` Isaku Yamahata
2011-03-17 14:42 ` Michael S. Tsirkin [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=20110317144229.GB22213@redhat.com \
--to=mst@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=yamahata@valinux.co.jp \
/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.