Re: [Xen-devel] [PATCH] qemu-xen-trad: free all the pirqs for msi/msix when driver unloads

[Zhenzhong Duan <zhenzhong.duan@xxxxxxxxxx>]

On 2014-8-4 22:01, Konrad Rzeszutek Wilk wrote:
> On Tue, Jul 08, 2014 at 09:12:39AM +0800, Zhenzhong Duan wrote:
> > On 2014-7-8 4:40, Konrad Rzeszutek Wilk wrote:
> > > On Fri, Jul 04, 2014 at 09:29:38AM +0800, Zhenzhong Duan wrote:
> > > > On 2014-7-4 1:54, Konrad Rzeszutek Wilk wrote:
> > > > > On Thu, Jul 03, 2014 at 11:12:18AM +0800, Zhenzhong Duan wrote:
> > > > > > There is a patch dependency missed.
> > > > > Is that one upstream as well? Or does it not need to be?
> > > > > If it is upstream, what is the title/commit id of that one?
> > > > >
> > > > > Thank you!
> > > > It's upstream, see detail below
> > > [konrad@build-external qemu-xen-dir]$ pwd
> > > /home/konrad/xtt-x86_64/xen/tools/qemu-xen-dir
> > > [konrad@build-external qemu-xen-dir]$ git show 
> > > adf74189dd58014744a4b8c9d64407d629da5e2f
> > > fatal: bad object adf74189dd58014744a4b8c9d64407d629da5e2f
> > > [konrad@build-external qemu-xen-dir]$ git log --grep="clear all MSI"
> > > [konrad@build-external qemu-xen-dir]$
> > >
> > > ?
> > Sorry, I mean qemu-xen-traditional upstream git
> Ok, is that patch upstream? Should it be upstream? If so, had it been
> posted in the past?
It's already in qemu-xen upstream, see below

void xen_pt_msi_disable(XenPCIPassthroughState *s)
{
.......
    /* clear msi info */
    msi->flags &= ~PCI_MSI_FLAGS_ENABLE;
    msi->initialized = false;
    msi->mapped = false;
    msi->pirq = XEN_PT_UNASSIGNED_PIRQ;
}

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel