Re: [Xen-devel] Re: [PATCH] re-enable HVM PCI passthrough functionality

[Christoph Egger <Christoph.Egger@xxxxxxx>]

On Thursday 02 September 2010 04:26:32 Zhang, Yang Z wrote:
> With allen's patch, both vt-d and sr-iov can work now. Thanks allen.

We were not able to reproduce SR-IOV failures with and without the fix
on AMD machines. I am wondering if there are differences in memory
mappings between AMD and Intel PCI passthrough.

Nonetheless, good to know this has been fixed.

Thanks,
Christoph

> best regards
> yang
>
> > -----Original Message-----
> > From: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
> > [mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Tim Deegan
> > Sent: Wednesday, September 01, 2010 4:57 PM
> > To: Kay, Allen M
> > Cc: Christoph Egger; xen-devel@xxxxxxxxxxxxxxxxxxx; Keir Fraser
> > Subject: [Xen-devel] Re: [PATCH] re-enable HVM PCI passthrough
> > functionality
> >
> > At 17:13 +0100 on 31 Aug (1283274786), Kay, Allen M wrote:
> > > I have updated the patch to indicate valid_mfn() cannot be used here
> > > as MMIO pages are always above max_mem.  Also added sanity checking
> > > for type == p2m_mmio_direct before clearing page table entry.  This
> > > should fix both VT-d and SR-IOV issues reported recently as there are
> > > no SR-IOV specific code in Xen.  As far as Xen is concern, SR-IOV
> > > virtual function is just another PCI device.
> >
> > Applied; thanks for the fix.
> >
> > Tim.
> >
> > > Signed-off-by: Allen Kay allen.m.kay@xxxxxxxxx
> > >
> > > -----Original Message-----
> > > From: Christoph Egger [mailto:Christoph.Egger@xxxxxxx]
> > > Sent: Tuesday, August 31, 2010 2:35 AM
> > > To: Tim Deegan
> > > Cc: Kay, Allen M; xen-devel@xxxxxxxxxxxxxxxxxxx; Keir Fraser
> > > Subject: Re: [PATCH] re-enable HVM PCI passthrough functionality
> > >
> > > On Tuesday 31 August 2010 11:17:27 Tim Deegan wrote:
> > > > At 02:58 +0100 on 31 Aug (1283223494), Kay, Allen M wrote:
> > > > > Attached patch reverts back to INVALID_MFN.  PCI passthrough
> > > > > functionality works with this patch.
> > > >
> > > > Apologies for missing this - I did ask for this enormous patch to
> > > > contain only what it said on the tin (and I did read it to check!)
> > > > but this one got through.
> > >
> > > Also apologies from my side. I did what Tim requested for, but that
> > > one slipped through.
> > >
> > > > > Signed-off-by: Allen Kay allen.m.kay@xxxxxxxxx
> > > >
> > > > This is fine, but please add a comment explaining the rather subtle
> > > > use of INVALID_MFN here so the next tidy-minded person doesn't "fix"
> > > > it again. :)
> > > >
> > > > In fact would it be better to check the returned type ==
> > > > p2m_mmio_direct instead?  That seems like it would fit better with
> > > > the intention.   Or do other callers use this to clean non-mmio
> > > > entries too?
> > >
> > > I'm interested in if this patch fixes the SR-IOV issue. We are still
> > > about investigation...
> > >
> > > Christoph


-- 
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Einsteinring 24, 85609 Dornach b. Muenchen
Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel