He, Qing
2008-May-12 08:31 UTC
[Xen-devel] [PATCH] bypass passthrough MMIO mfns in vram dirty logic
This patch fixes the fatal page fault when vram dirty logic handles MMIO of passthrough devices. Signed-off-by: Qing He <qing.he@intel.com> --- diff -r 810d8c3ac992 xen/arch/x86/mm/shadow/multi.c --- a/xen/arch/x86/mm/shadow/multi.c Thu May 08 16:58:33 2008 +0100 +++ b/xen/arch/x86/mm/shadow/multi.c Tue May 13 00:19:08 2008 +0800 @@ -1268,6 +1268,9 @@ static inline void shadow_vram_get_l1e(s if ( !d->dirty_vram ) return; mfn = shadow_l1e_get_mfn(new_sl1e); + + if ( !mfn_valid(mfn) ) return; /* m2p for mmio_direct may not exist */ + gfn = mfn_to_gfn(d, mfn); if ( (gfn >= d->dirty_vram->begin_pfn) && (gfn < d->dirty_vram->end_pfn) ) { @@ -1293,6 +1296,9 @@ static inline void shadow_vram_put_l1e(s if ( !d->dirty_vram ) return; mfn = shadow_l1e_get_mfn(old_sl1e); + + if ( !mfn_valid(mfn) ) return; + gfn = mfn_to_gfn(d, mfn); if ( (gfn >= d->dirty_vram->begin_pfn) && (gfn < d->dirty_vram->end_pfn) ) { _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Samuel Thibault
2008-May-12 08:42 UTC
[Xen-devel] Re: [PATCH] bypass passthrough MMIO mfns in vram dirty logic
He, Qing, le Mon 12 May 2008 16:31:22 +0800, a écrit :> This patch fixes the fatal page fault when vram dirty logic handles MMIO > of passthrough devices. > > Signed-off-by: Qing He <qing.he@intel.com>Signed-off-by: Samuel Thibault <samuel.thibault@eu.citrix.com>> diff -r 810d8c3ac992 xen/arch/x86/mm/shadow/multi.c > --- a/xen/arch/x86/mm/shadow/multi.c Thu May 08 16:58:33 2008 +0100 > +++ b/xen/arch/x86/mm/shadow/multi.c Tue May 13 00:19:08 2008 +0800 > @@ -1268,6 +1268,9 @@ static inline void shadow_vram_get_l1e(s > if ( !d->dirty_vram ) return; > > mfn = shadow_l1e_get_mfn(new_sl1e); > + > + if ( !mfn_valid(mfn) ) return; /* m2p for mmio_direct may not exist > */ > + > gfn = mfn_to_gfn(d, mfn); > > if ( (gfn >= d->dirty_vram->begin_pfn) && (gfn < > d->dirty_vram->end_pfn) ) { > @@ -1293,6 +1296,9 @@ static inline void shadow_vram_put_l1e(s > if ( !d->dirty_vram ) return; > > mfn = shadow_l1e_get_mfn(old_sl1e); > + > + if ( !mfn_valid(mfn) ) return; > + > gfn = mfn_to_gfn(d, mfn); > > if ( (gfn >= d->dirty_vram->begin_pfn) && (gfn < > d->dirty_vram->end_pfn) ) {-- Samuel tohi.cybercable.fr (212.198.0.3) si une personne se reconnait derriere cette adresse que ce soit un pirate ou une victime qu''il se manifeste, cette personne pourrait bien etre un petit malin -+- Fred in NPC : Mamaaaaan, y a le routeur qui veut me hacker -+- _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel