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

[PATCH] check memory descriptor over lap in dom_fw_init() (was Re: [Xen-ia64-devel] [PATCH]Fix domain0 no VGA console bug.)



This patch tries to check MD's overlap and tries not to assign
page to non conventional area.
Maybe you may want to apply this patch before VGA-related modifications.

On Wed, May 31, 2006 at 08:10:42PM -0600, Alex Williamson wrote:
> On Thu, 2006-06-01 at 09:45 +0800, Zhang, Xiantao wrote:
> > Hi Alex,
> >     We are using tiger4 platform. I didn't find md about space:
> > 0xa0000-0xc0000 in efi memmap, maybe it was assumed
> > EFI_MEMORY_MAPPED_IO in native OS. But dom_fw_init shouldn't neglect
> > it to set IO space according to MDs efi provides. Seems your platform
> > has VGA console. So this patch can enable VGA console on all platforms
> > explicitly, maybe as you said this step is better to do in
> > dom_fw_init :)
> 
> Hi Xiantao,
> 
>    One of my test systems has VGA, the other does not.  We cannot assume
> VGA in the system.  If the MDT on the tiger4 doesn't describe that
> range, then we probably need to at least revert to the test the Linux
> kernel uses and test whether that range has a WB memory attribute before
> assuming it's VGA.  Also, try not to double map the range for platforms
> that do describe this as type EFI_MEMORY_MAPPED_IO.  Thanks,
> 
>       Alex
> 
> -- 
> Alex Williamson                             HP Open Source & Linux Org.
> 
> 
> _______________________________________________
> Xen-ia64-devel mailing list
> Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-ia64-devel
> 

-- 
yamahata

Attachment: 10258:e3e02e227b3e_assign_page_to_dom0_precisely.patch
Description: Text document

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

 


Rackspace

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