From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:47377) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UapXo-0005m9-D6 for qemu-devel@nongnu.org; Fri, 10 May 2013 11:47:42 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UapXm-0006gW-7V for qemu-devel@nongnu.org; Fri, 10 May 2013 11:47:40 -0400 Received: from mx1.redhat.com ([209.132.183.28]:55799) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UapXl-0006gG-Vr for qemu-devel@nongnu.org; Fri, 10 May 2013 11:47:38 -0400 Message-ID: <1368200856.3334.25.camel@ul30vt.home> From: Alex Williamson Date: Fri, 10 May 2013 09:47:36 -0600 In-Reply-To: <518CEAA4.6090005@siemens.com> References: <20130509163157.22536.68566.stgit@bling.home> <518CEAA4.6090005@siemens.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/2] pci-assign: MSI affinity support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jan Kiszka Cc: "qemu-devel@nongnu.org" , "mst@redhat.com" On Fri, 2013-05-10 at 14:40 +0200, Jan Kiszka wrote: > On 2013-05-09 18:35, Alex Williamson wrote: > > I posted these about 6 months ago and Jan felt we should implement > > MSI notifiers like we have for MSI-X. That still hasn't happened. > > Device assignments are the only currently known users - and you provide > this feature, so... > > POWER does this nice configuration of MSI messages via a side channel. > Not that it already fires the MSI-X notifiers properly, but a generic > notifier based approach is the right way to abstract away the different > modification channels (instead of encoding them at the consumer side > like in your patches). > > Moreover, having different designs for MSI and MSI-X is just ugly. OTOH, if device assignment is the only one that needs this and it's trivial to implement there, why would we bloat the common code with notifiers and caching vector messages. I'm not that thrilled with the MSI-X notifiers to go copy them into MSI anyway, they completely miss important things like an overall device MSI-X enable or disable. For this much code: hw/i386/kvm/pci-assign.c | 78 +++++++++++++++++++++++++++++++--------------- 1 file changed, 53 insertions(+), 25 deletions(-) we get a feature that's immediately useful to a guest. Seems like a win to me. Thanks, Alex