From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:50623) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RG9TG-0000QP-Pf for qemu-devel@nongnu.org; Tue, 18 Oct 2011 09:12:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RG9TF-0006Di-HJ for qemu-devel@nongnu.org; Tue, 18 Oct 2011 09:12:42 -0400 Received: from fmmailgate03.web.de ([217.72.192.234]:39303) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RG9TF-0006DW-45 for qemu-devel@nongnu.org; Tue, 18 Oct 2011 09:12:41 -0400 Received: from moweb001.kundenserver.de (moweb001.kundenserver.de [172.19.20.114]) by fmmailgate03.web.de (Postfix) with ESMTP id 1B85E1A439D84 for ; Mon, 17 Oct 2011 21:21:49 +0200 (CEST) Message-ID: <4E9C803E.3000702@web.de> Date: Mon, 17 Oct 2011 21:21:34 +0200 From: Jan Kiszka MIME-Version: 1.0 References: <20111017112217.GC4537@redhat.com> <4E9C1123.4070204@siemens.com> <20111017142818.GG6406@redhat.com> In-Reply-To: <20111017142818.GG6406@redhat.com> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enig62D0A8D7189DA7AC1079A759" Subject: Re: [Qemu-devel] [RFC][PATCH 42/45] msix: Introduce msix_init_simple List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: Alex Williamson , Marcelo Tosatti , Avi Kivity , "kvm@vger.kernel.org" , "qemu-devel@nongnu.org" This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enig62D0A8D7189DA7AC1079A759 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 2011-10-17 16:28, Michael S. Tsirkin wrote: > On Mon, Oct 17, 2011 at 01:27:31PM +0200, Jan Kiszka wrote: >> On 2011-10-17 13:22, Michael S. Tsirkin wrote: >>> On Mon, Oct 17, 2011 at 11:28:16AM +0200, Jan Kiszka wrote: >>>> Devices models are usually not interested in specifying MSI-X >>>> configuration details beyond the number of vectors to provide and th= e >>>> BAR number to use. Layout of an exclusively used BAR and its >>>> registration can also be handled centrally. >>>> >>>> This is the purpose of msix_init_simple. It provides handy services = to >>>> the existing users. Future users like device assignment may require = more >>>> detailed setup specification. For them we will (re-)introduce msix_i= nit >>>> with the full list of configuration option (in contrast to the curre= nt >>>> code). >>>> >>>> Signed-off-by: Jan Kiszka >>> >>> Well, this seems a bit of a code churn then, doesn't it? >>> We are also discussing using memory BAR for virtio-pci for other >>> stuff besides MSI-X, so the last user of the _simple variant >>> will be ivshmem then? >> >> We will surely see more MSI-X users over the time. Not sure if they al= l >> mix their MSIX-X BARs with other stuff. But e.g. the e1000 variant I >> have here does not. So there should be users in the future. >> >> Jan >=20 > Question is, how hard is to pass in the BAR and the offset? That is trivial. But have a look at the final simple implementation. It also manages the container memory region for table and PBA and registers/unregisters that container as BAR. So there is measurable added-value. Jan --------------enig62D0A8D7189DA7AC1079A759 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAk6cgD4ACgkQitSsb3rl5xQY4ACgoSnociX2AORIiXB3arF3iHMA tqIAoLidihNmup5bm42TD000xXU5HR+s =/IXN -----END PGP SIGNATURE----- --------------enig62D0A8D7189DA7AC1079A759--