From: Jan Kiszka <jan.kiszka@siemens.com>
To: Avi Kivity <avi@redhat.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
"Michael S. Tsirkin" <mst@redhat.com>,
Marcelo Tosatti <mtosatti@redhat.com>,
qemu-devel <qemu-devel@nongnu.org>,
Blue Swirl <blauwirbel@gmail.com>
Subject: Re: [PATCH v3 00/16] uq/master: Introduce basic irqchip support
Date: Tue, 06 Dec 2011 15:21:25 +0100 [thread overview]
Message-ID: <4EDE24E5.2000603@siemens.com> (raw)
In-Reply-To: <4EDE1EE3.5090008@redhat.com>
On 2011-12-06 14:55, Avi Kivity wrote:
> On 12/06/2011 02:58 PM, Jan Kiszka wrote:
>> In this revision, I'm now trying the approach of backend/frontend
>> split-ups for the affected IRQ chips. That means we keep a single qdev
>> device description but fork off specific logic early during device init.
>> The backends support this by providing hooks that user space and KVM
>> models can implement differently.
>>
>> The result is slightly larger and comes with the not really beautiful
>> ioapic.kvm_gsi_base property but should otherwise meet expectations.
>>
>> Comments?
>
> Looks good to me, much nicer than the previous approaches. I'll wait a
> bit for more reviews though.
>
>> PS: Series is still against old uq/master, therefore containing patches
>> that took/will take different routes.
>
> I just pushed a rebased uq/master. In the future, either ping me or
> just base on upstream (which uq/master supposedly tracks).
Requires minor rebasing. Will wait for comments before reposting.
Thanks,
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
WARNING: multiple messages have this Message-ID (diff)
From: Jan Kiszka <jan.kiszka@siemens.com>
To: Avi Kivity <avi@redhat.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
"Michael S. Tsirkin" <mst@redhat.com>,
Marcelo Tosatti <mtosatti@redhat.com>,
qemu-devel <qemu-devel@nongnu.org>,
Blue Swirl <blauwirbel@gmail.com>
Subject: Re: [Qemu-devel] [PATCH v3 00/16] uq/master: Introduce basic irqchip support
Date: Tue, 06 Dec 2011 15:21:25 +0100 [thread overview]
Message-ID: <4EDE24E5.2000603@siemens.com> (raw)
In-Reply-To: <4EDE1EE3.5090008@redhat.com>
On 2011-12-06 14:55, Avi Kivity wrote:
> On 12/06/2011 02:58 PM, Jan Kiszka wrote:
>> In this revision, I'm now trying the approach of backend/frontend
>> split-ups for the affected IRQ chips. That means we keep a single qdev
>> device description but fork off specific logic early during device init.
>> The backends support this by providing hooks that user space and KVM
>> models can implement differently.
>>
>> The result is slightly larger and comes with the not really beautiful
>> ioapic.kvm_gsi_base property but should otherwise meet expectations.
>>
>> Comments?
>
> Looks good to me, much nicer than the previous approaches. I'll wait a
> bit for more reviews though.
>
>> PS: Series is still against old uq/master, therefore containing patches
>> that took/will take different routes.
>
> I just pushed a rebased uq/master. In the future, either ping me or
> just base on upstream (which uq/master supposedly tracks).
Requires minor rebasing. Will wait for comments before reposting.
Thanks,
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
next prev parent reply other threads:[~2011-12-06 14:21 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-06 12:58 [PATCH v3 00/16] uq/master: Introduce basic irqchip support Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 01/16] msi: Generalize msix_supported to msi_supported Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 02/16] kvm: Move kvmclock into hw/kvm folder Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 03/16] apic: Stop timer on reset Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 04/16] apic: Introduce backend/frontend infrastructure for KVM reuse Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 05/16] apic: Open-code timer save/restore Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 06/16] i8259: Introduce backend/frontend infrastructure for KVM reuse Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 07/16] ioapic: Convert to memory API Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 08/16] ioapic: Reject non-dword accesses to IOWIN register Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 09/16] ioapic: Introduce backend/frontend infrastructure for KVM reuse Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 10/16] memory: Introduce memory_region_init_reservation Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 11/16] kvm: Introduce core services for in-kernel irqchip support Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 12/16] kvm: x86: Establish IRQ0 override control Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 13/16] kvm: x86: Add user space part for in-kernel APIC Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 14/16] kvm: x86: Add user space part for in-kernel i8259 Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 15/16] kvm: x86: Add user space part for in-kernel IOAPIC Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 12:58 ` [PATCH v3 16/16] kvm: Arm in-kernel irqchip support Jan Kiszka
2011-12-06 12:58 ` [Qemu-devel] " Jan Kiszka
2011-12-06 13:55 ` [PATCH v3 00/16] uq/master: Introduce basic " Avi Kivity
2011-12-06 13:55 ` [Qemu-devel] " Avi Kivity
2011-12-06 14:21 ` Jan Kiszka [this message]
2011-12-06 14:21 ` Jan Kiszka
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=4EDE24E5.2000603@siemens.com \
--to=jan.kiszka@siemens.com \
--cc=aliguori@us.ibm.com \
--cc=avi@redhat.com \
--cc=blauwirbel@gmail.com \
--cc=kvm@vger.kernel.org \
--cc=mst@redhat.com \
--cc=mtosatti@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.