Re: [Xen-devel] Re: changeset 22526:7a5ee3800417

[George Dunlap <George.Dunlap@xxxxxxxxxxxxx>]

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
Description: Text Data

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