[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


  • To: George Dunlap <george.dunlap@xxxxxxxxxx>
  • From: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
  • Date: Wed, 9 Mar 2011 13:59:11 +0000
  • Cc: xen devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, åéé <zpfalpc23@xxxxxxxxx>
  • Delivery-date: Wed, 09 Mar 2011 06:03:05 -0800
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; b=IZD2p2tkkzPjuJqC+gSy2mIT7PxSh2OYJ3/l1Bku8lWnLoJnt83sKg1KSc+qDN5sPs mYkf/4mw0PMQRroOvHY0kQKAiyvNC8YZwkR2rJ/j5hxdJtgJb5CP9FaAwDPEWZDuBSZm /P5+29XXDsutEbcns00Im7JsqRzEHw4Ct0jHY=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

Peng fei,

I just noticed this patch has a bug -- it doesn't set new_entry.mfn if
the new mfn is the same as the one already in ept_entry!  Standby for
a new one...

 -George

On Mon, Mar 7, 2011 at 4:41 PM, George Dunlap
<George.Dunlap@xxxxxxxxxxxxx> wrote:
> 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
>>
>

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.