Just resend.
Thanks & Best Regards
-Xiantao
OTC,Intel Corporation
-----Original Message-----
From: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
[mailto:xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Zhang, Xiantao
Sent: 2006年6月1日 16:29
To: Alex Williamson
Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
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:alex.williamson@xxxxxx]
> Sent: 2006年6月1日 10:11
> To: Zhang, Xiantao
> Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> 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.
xen0_console.patch
Description: xen0_console.patch
_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel
|
Home