RE: [Xen-ia64-devel] [PATCH]Fix domain0 no VGA console bug.

["Zhang, Xiantao" <xiantao.zhang@xxxxxxxxx>]

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.

Attachment: xen0_console.patch
Description: xen0_console.patch

--- Begin Message ---

• To: "Alex Williamson" <alex.williamson@xxxxxx>
• From: "Isaku Yamahata" <yamahata@xxxxxxxxxxxxx>
• Date: Thu, 1 Jun 2006 10:46:42 +0800
• Cc: "Zhang, Xiantao" <xiantao.zhang@xxxxxxxxx>, <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
• Thread-index: AcaFJaLzWP68AcK0SuyGmFOQwFx7ZQ==
• Thread-topic: [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: 10258_e3e02e227b3e_assign_page_to_dom0_precisely.patch

--- End Message ---

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