From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sasha Levin Subject: Re: [PATCH 0/4 v9] MSI-X MMIO support for KVM Date: Tue, 06 Sep 2011 12:00:00 +0300 Message-ID: <1315299600.2615.6.camel@sasha> References: <1298019193-12742-1-git-send-email-sheng@linux.intel.com> <20110222180858.GA5784@amt.cnet> <4E5F6C20.7020701@siemens.com> <4E65D149.1070807@redhat.com> <20110906081247.GE13286@redhat.com> <4E65DB89.6020909@siemens.com> <1315298760.2615.2.camel@sasha> <4E65DEA4.8070203@siemens.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: "Michael S. Tsirkin" , Avi Kivity , Marcelo Tosatti , "kvm@vger.kernel.org" , Alex Williamson To: Jan Kiszka Return-path: Received: from mail-wy0-f174.google.com ([74.125.82.174]:34503 "EHLO mail-wy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753808Ab1IFJAE (ORCPT ); Tue, 6 Sep 2011 05:00:04 -0400 Received: by wyh22 with SMTP id 22so4231516wyh.19 for ; Tue, 06 Sep 2011 02:00:03 -0700 (PDT) In-Reply-To: <4E65DEA4.8070203@siemens.com> Sender: kvm-owner@vger.kernel.org List-ID: On Tue, 2011-09-06 at 10:49 +0200, Jan Kiszka wrote: > On 2011-09-06 10:46, Sasha Levin wrote: > > On Tue, 2011-09-06 at 10:36 +0200, Jan Kiszka wrote: > >> On 2011-09-06 10:12, Michael S. Tsirkin wrote: > >>> On Tue, Sep 06, 2011 at 10:52:41AM +0300, Avi Kivity wrote: > >>>>> > >>>>> BTW, the same is also true for that optional per-vector masking of > >>>>> legacy MSI. Are there devices in the field that actually support this? I > >>>>> haven't found one so far and tend to consider this feature not worth > >>>>> implementing. > >>>> > >>>> Don't know. I don't like implementing features on the basis of bug > >>>> reports, though. On the other hand we can't really test it without > >>>> a real device. > >>> > >>> Linux will use this capability if present. So > >>> we could add support for an emulated device (e.g. e1000), > >>> then test with nested virt once iommu emulation lands :) > >> > >> Yeah, would be kind of cool. Still, I would feel better having it tested > >> against a real silicon as well. Also to prove that there is a real need. > >> > >> So, in case someone stumbles for such a device (bit 8 set in MSI control > >> word), please let us know! > > > > Doesn't any device that supports MSI-X supports per-vector masking? > > > >>From the spec: > > > > "MSI and MSI-X each support per-vector masking. Per-vector masking is an > > optional extension to MSI, and a standard feature with MSI-X" > > Right, but the cap flag still has to be set. > Are you sure? Take a look at the table in section 6.8.2.3 in the spec (pci v3). Unlike the message control for MSI, this table doesn't mention anything about bit 8 or the per-vector masking capability for MSI-X, it just assumes it's there. > Do you have any MSI-X device in reach? Don't think so, but I think it would be much easier to find a MSI-X device rather than a MSI with per-vector capability. -- Sasha.