Just resend.

Thanks & Best Regards
-Xiantao

OTC,Intel Corporation

-----Original Message-----
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Zhang, Xiantao
Sent: 2006年6月1日 16:29
To: Alex Williamson
Cc: xen-ia64-devel@lists.xensource.com
Subject: RE: [Xen-ia64-devel] [PATCH]Fix domain0 no VGA console bug.

Hi Alex,
        The updated patch should be happy for all platforms. If EFI doesn't 
provide md for range(0xa0000-0xc0000) to OS. It maybe a hole or occupied by 
legacy vga.  Therefore, I used the efi_mmio function to check it. If these 
pages was not mapped yet and efi_mmio return true, we can map them as MMIO 
safely.
BTW, this patch based on Yamahata's check memory descriptor overlap patch.
Please give comments. :)
Thanks
-Xiantao

> -----Original Message-----
> From: Alex Williamson [mailto:[EMAIL PROTECTED]
> Sent: 2006年6月1日 10:11
> To: Zhang, Xiantao
> Cc: xen-ia64-devel@lists.xensource.com
> Subject: RE: [Xen-ia64-devel] [PATCH]Fix domain0 no VGA console bug.
> 
> 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.

Attachment: xen0_console.patch
Description: xen0_console.patch

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

Reply via email to