From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:58104)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mst@redhat.com>) id 1SFLkP-0005hi-9L
	for qemu-devel@nongnu.org; Wed, 04 Apr 2012 04:39:27 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <mst@redhat.com>) id 1SFLkI-0007OK-VU
	for qemu-devel@nongnu.org; Wed, 04 Apr 2012 04:39:20 -0400
Received: from mx1.redhat.com ([209.132.183.28]:50731)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mst@redhat.com>) id 1SFLkI-0007Na-N5
	for qemu-devel@nongnu.org; Wed, 04 Apr 2012 04:39:14 -0400
Date: Wed, 4 Apr 2012 11:39:28 +0300
From: "Michael S. Tsirkin" <mst@redhat.com>
Message-ID: <20120404083927.GC3003@redhat.com>
References: <cover.1333437795.git.jan.kiszka@web.de>
	<20120403130604.GA11053@redhat.com> <4F7B3308.5060700@siemens.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4F7B3308.5060700@siemens.com>
Subject: Re: [Qemu-devel] [RFC][PATCH v2 0/4] uq/master: Basic MSI support
 for in-kernel irqchip mode
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Jan Kiszka <jan.kiszka@siemens.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>, Avi Kivity <avi@redhat.com>, kvm@vger.kernel.org, qemu-devel <qemu-devel@nongnu.org>

On Tue, Apr 03, 2012 at 07:27:36PM +0200, Jan Kiszka wrote:
> On 2012-04-03 15:06, Michael S. Tsirkin wrote:
> > On Tue, Apr 03, 2012 at 09:23:12AM +0200, Jan Kiszka wrote:
> >> This is v2 of the RFC, fixing a memory leak in
> >> kvm_flush_dynamic_msi_routes and adding support for the proposed
> >> KVM_SIGNAL_MSI IOCTL.
> >>
> >> This series depends on "kvm: set gsi_bits and max_gsi correctly"
> >> (http://thread.gmane.org/gmane.comp.emulators.kvm.devel/88906).
> > 
> > Looks good to me.
> > How hard would it be to add irqfd support?
> 
> Shouldn't be, but the changes will be a bit bigger.
> 
> I'm thinking about a revamped interface between the MSI core and
> affected devices for a while. Will try to put down in some lines of code
> what I have in mind - once the dynamic MSI injection topic has settled.
> 
> Jan

Yes it's not an objection - just a question.

> -- 
> Siemens AG, Corporate Technology, CT T DE IT 1
> Corporate Competence Center Embedded Linux