All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marcelo Tosatti <mtosatti@redhat.com>
To: Sheng Yang <sheng@linux.intel.com>
Cc: "Michael S. Tsirkin" <mst@redhat.com>,
	Avi Kivity <avi@redhat.com>,
	kvm@vger.kernel.org
Subject: Re: [PATCH 5/5] KVM: assigned dev: MSI-X mask support
Date: Fri, 5 Nov 2010 22:24:51 -0200	[thread overview]
Message-ID: <20101106002451.GA410@amt.cnet> (raw)
In-Reply-To: <201011051853.51093.sheng@linux.intel.com>

On Fri, Nov 05, 2010 at 06:53:50PM +0800, Sheng Yang wrote:
> On Friday 05 November 2010 16:43:33 Michael S. Tsirkin wrote:
> > On Fri, Nov 05, 2010 at 10:44:19AM +0800, Sheng Yang wrote:
> > > > > +};
> > > > > +
> > > > > +This ioctl would enable in-kernel MSI-X emulation, which would
> > > > > handle MSI-X +mask bit in the kernel.
> > > > 
> > > > What happens on repeated calls when it's already enabled?
> > > > How does one disable after it's enabled?
> > > 
> > > Suppose this should only be called once. But again would update the MMIO
> > > base.
> > 
> > So maybe add this all to documentation.
> > 
> > > It
> > > would be disabled along with device deassignment.
> > 
> > So what are you going to do when guest enables and later disables MSIX?
> > Disable assignment completely?
> 
> This device goes with PCI resources allocation, rather than IRQ assignment.

What about hot plug, for example? I can't see how this can function
without unregistering regions.


  parent reply	other threads:[~2010-11-06  0:28 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-04  6:15 [PATCH 0/5 v3] MSI-X mask supporting for assigned device(kernel) Sheng Yang
2010-11-04  6:15 ` [PATCH 1/5] PCI: MSI: Move MSI-X entry definition to pci_regs.h Sheng Yang
2010-11-05  0:43   ` Hidetoshi Seto
2010-11-04  6:15 ` [PATCH 2/5] PCI: Add mask bit definition for MSI-X table Sheng Yang
2010-11-04  9:50   ` Michael S. Tsirkin
2010-11-05  2:48     ` Sheng Yang
2010-11-05  0:48   ` Hidetoshi Seto
2010-11-05  2:49     ` Sheng Yang
2010-11-04  6:15 ` [PATCH 3/5] KVM: Move struct kvm_io_device to kvm_host.h Sheng Yang
2010-11-04  6:15 ` [PATCH 4/5] KVM: Add kvm_get_irq_routing_entry() func Sheng Yang
2010-11-04  6:15 ` [PATCH 5/5] KVM: assigned dev: MSI-X mask support Sheng Yang
2010-11-04 10:43   ` Michael S. Tsirkin
2010-11-05  2:44     ` Sheng Yang
2010-11-05  4:20       ` Sheng Yang
2010-11-05  7:16         ` Sheng Yang
2010-11-05  8:51           ` Michael S. Tsirkin
2010-11-05 10:54             ` Sheng Yang
2010-11-05  8:43       ` Michael S. Tsirkin
2010-11-05 10:53         ` Sheng Yang
2010-11-05 12:01           ` Sheng Yang
2010-11-05 13:29             ` Michael S. Tsirkin
2010-11-05 13:27           ` Michael S. Tsirkin
2010-11-08  3:18             ` Sheng Yang
2010-11-08  6:42               ` Michael S. Tsirkin
2010-11-06  0:24           ` Marcelo Tosatti [this message]
2010-11-08  5:41             ` Sheng Yang
2010-11-09 14:08               ` Marcelo Tosatti
2010-11-06  0:27 ` [PATCH 0/5 v3] MSI-X mask supporting for assigned device(kernel) Marcelo Tosatti
2010-11-08  0:51   ` Sheng Yang

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=20101106002451.GA410@amt.cnet \
    --to=mtosatti@redhat.com \
    --cc=avi@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=mst@redhat.com \
    --cc=sheng@linux.intel.com \
    /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.