From: "Michael S. Tsirkin" <mst@redhat.com>
To: Glauber Costa <glommer@gmail.com>
Cc: Glauber Costa <glommer@redhat.com>,
avi@redhat.com, aliguori@us.ibm.com, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Re: [PATCH 0/9] in-kernel irqchip, new spin
Date: Thu, 3 Dec 2009 13:57:29 +0200 [thread overview]
Message-ID: <20091203115729.GC7352@redhat.com> (raw)
In-Reply-To: <5d6222a80912030342m499eb913lff5fc7a21b142718@mail.gmail.com>
On Thu, Dec 03, 2009 at 09:42:02AM -0200, Glauber Costa wrote:
> >
> > Does msix work with this patchset when in-kernel irqchip
> > is enabled?
>
> Haven't tested. But since I see that msix need some special code in qemu-kvm,
> it probably won't. But I assume we can just add a patch ontop of this to add
> that code and make it work, right?
Sure. However - are you making in-kernel irqchip the default?
If so, I think the best way to do it would be:
1. add in-kernel irqchip, off by default
2. fix msix with in-kernel
3. make in-kernel on by default
Otherwise you hurt bisectability.
> --
> Glauber Costa.
> "Free as in Freedom"
> http://glommer.net
>
> "The less confident you are, the more serious you have to act."
next prev parent reply other threads:[~2009-12-03 12:00 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-02 13:46 [Qemu-devel] [PATCH 0/9] in-kernel irqchip, new spin Glauber Costa
2009-12-02 13:46 ` [Qemu-devel] [PATCH 1/9] introduce VMSTATE_U64 Glauber Costa
2009-12-02 13:46 ` [Qemu-devel] [PATCH 2/9] Provide ioapic-kvm Glauber Costa
2009-12-02 13:46 ` [Qemu-devel] [PATCH 3/9] provide apic_set_irq_delivered Glauber Costa
2009-12-02 13:46 ` [Qemu-devel] [PATCH 4/9] provide i8259-kvm Glauber Costa
2009-12-02 13:46 ` [Qemu-devel] [PATCH 5/9] Don't call apic functions directly from kvm code Glauber Costa
2009-12-02 13:46 ` [Qemu-devel] [PATCH 6/9] export kvm_put_mp_state Glauber Costa
2009-12-02 13:46 ` [Qemu-devel] [PATCH 7/9] provide apic-kvm Glauber Costa
2009-12-02 13:46 ` [Qemu-devel] [PATCH 8/9] Initialize in-kernel irqchip Glauber Costa
2009-12-02 13:46 ` [Qemu-devel] [PATCH 9/9] Do GSI routing Glauber Costa
2009-12-03 12:12 ` [Qemu-devel] Re: [PATCH 5/9] Don't call apic functions directly from kvm code Avi Kivity
2009-12-03 10:27 ` [Qemu-devel] Re: [PATCH 0/9] in-kernel irqchip, new spin Michael S. Tsirkin
2009-12-03 11:42 ` Glauber Costa
2009-12-03 11:57 ` Michael S. Tsirkin [this message]
2009-12-03 12:04 ` Glauber Costa
2009-12-03 12:39 ` Michael S. Tsirkin
2009-12-03 12:45 ` Avi Kivity
2009-12-03 12:57 ` Glauber Costa
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=20091203115729.GC7352@redhat.com \
--to=mst@redhat.com \
--cc=aliguori@us.ibm.com \
--cc=avi@redhat.com \
--cc=glommer@gmail.com \
--cc=glommer@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.