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

Re: [Xen-devel] [PATCH] amd iommu: Do not adjust paging mode for dom0 devices


  • To: Wei Wang2 <wei.wang2@xxxxxxx>
  • From: Keir Fraser <keir@xxxxxxx>
  • Date: Mon, 07 Feb 2011 15:00:04 +0000
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 07 Feb 2011 07:01:37 -0800
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:user-agent:date:subject:from:to:cc:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; b=NtQ5gG81ygO8gR6TPcpa24QfLMq4qzrWj79Ip2nLaow36zkaAevImqmxVxM1eK8SWt pyyVHC3J/z8RRVk5Rdp5w3pRdCZJFC2WwEZ+JnMCEXx5xDaDxwya8OqxzaYjwBFcLzhX pY7NQJxJoC5C+uZCyAn30Tvs9QLfa5PPEyqBM=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcvG17NhHiqbZS+msEK59N4aXUKmIA==
  • Thread-topic: [Xen-devel] [PATCH] amd iommu: Do not adjust paging mode for dom0 devices

On 07/02/2011 13:30, "Wei Wang2" <wei.wang2@xxxxxxx> wrote:

> On Monday 07 February 2011 11:47:32 Keir Fraser wrote:
>> On 07/02/2011 10:33, "Wei Wang2" <wei.wang2@xxxxxxx> wrote:
> 
>> Personally I would suggest starting with small 2-level tables and
>> dynamically increase their height as bigger mappings are added to them.
>> Else stick with 4-level tables, or size tables according to global variable
>> max_page. I think basing anything on d->max_pages is not a good idea.
>> 
>>  -- Keir
> How does the attached patch look like? It uses global variable max_page for pv
> and dom0 and calculate maxpfn for hvm guest. This should cover gfn holes on
> hvm guests.

The p2m code already tracks the largest gfn for HVM guests. Try using
p2m_get_hostp2m(d)->max_mapped_pfn for HVM guests. Note that this could
increase after you sample it, however. Hence why you really need to have a
statically deep-enough table, or the ability to grow the table depth
dynamically.

Your change for PV guests would definitely be correct, however.

 -- Keir

> Thanks,
> Wei
> Signed-off-by: Wei Wang <wei.wang2@xxxxxxx>
> 
>>> I was assuming max_pdx is the index number... Or are
>>> you referring memory hot plug? If so, we might also need 4 level for
>>> dom0.
> 
> 



_______________________________________________
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®.