[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] Re: changeset 22526:7a5ee3800417
Peng fei, Can you test the attached patch (to xen-unstable)? I don't have EPT-enabled hardware handy... -George On Mon, Mar 7, 2011 at 4:20 PM, George Dunlap <george.dunlap@xxxxxxxxxx> wrote: > Hmm, yeah, that's obviously not right! ÂLet me take a look... > > Â-George > > On Mon, 2011-03-07 at 09:55 +0000, åéé wrote: >> Hi, >> Â Â Recently, I did a research on the p2m and EPT,and did apply your >> patch(22526) to the source code of mine. But there is one place >> confused me: >> >> >> >> ept_entry = table + index; >> 1.91 >> Â Â 1.92 - Â Â Â Âept_entry->emt = epte_get_entry_emt(d, gfn, mfn, &ipat, >> direct_mmio); >> Â Â 1.93 - Â Â Â Âept_entry->ipat = ipat; >> Â Â 1.94 - Â Â Â Âept_entry->sp = i ? 1 : 0; >> Â Â 1.95 - Â Â Â Âept_entry->avail1 = p2mt; >> Â Â 1.96 - Â Â Â Âept_entry->avail2 = 0; >> Â Â 1.97 + Â Â Â Ânew_entry.emt = epte_get_entry_emt(d, gfn, mfn, &ipat, >> direct_mmio); >> Â Â 1.98 + Â Â Â Ânew_entry.ipat = ipat; >> Â Â 1.99 + Â Â Â Ânew_entry.sp = i ? 1 : 0; >> Â Â1.100 + Â Â Â Ânew_entry.avail1 = p2mt; >> Â Â1.101 + Â Â Â Ânew_entry.avail2 = 0; >> Â Â1.102 >> Â Â1.103 - Â Â Â Âif ( ept_entry->mfn == mfn_x(mfn) ) >> Â Â1.104 + Â Â Â Âif ( new_entry.mfn == mfn_x(mfn) ) >> Â Â1.105 Â Â Â Â Â Â Â need_modify_vtd_table = 0; >> Â Â1.106 Â Â Â Â Âelse /* the caller should take care of the previous page */ >> Â Â1.107 - Â Â Â Â Â Âept_entry->mfn = mfn_x(mfn); >> Â Â1.108 + Â Â Â Â Â Ânew_entry.mfn = mfn_x(mfn); >> >> I think, The new_entry.mfn is used without assigned any value. Maybe there >> was something else I did not think of. >> Thank you for your kindness! >> >> http://xenbits.xen.org/xen-unstable.hg/rev/7a5ee3800417 > > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@xxxxxxxxxxxxxxxxxxx > http://lists.xensource.com/xen-devel > Attachment:
ept-use-existing-entry.diff _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |