Re: [Xen-devel] Re: [XenPPC] [PATCH] [POWERPC] fix vga.c compilation

2006-08-16 Thread Alex Williamson
On Wed, 2006-08-16 at 11:49 +0100, Keir Fraser wrote:
 Here you go. Arch/powerpc/vga.c isn't great but I assume it's
 temporary
 until vga support is fixed properly.
 
 If you think it looks okay I'll apply it. Also Sign-off or Ack if you
 like.

Hi Keir,

   In general this looks a lot better, but I think ia64 is still going
to have trouble with the chunk below.  It seems that a VGA card
operating in a standard text mode doesn't necessarily respond to all of
these addresses.  On some ia64 platforms that causes a hard fail
response (the bus goes fatal and a reboot follows).  On my system, the
0xB8000 test looks like it will probably work, but we never get there
because either the 0xA or the 0xB test will cause the hardfail.
Do we need to poke the card through I/O port space to get it into the
right mode before probing it in MMIO space?  I don't know enough about
the VGA programming model to be able to do that.  The card works once we
start talking to it correctly, but this probe is a little too brute
force.  Thanks,

Alex

 +
 +p = ioremap(0xA, 0x1000);
 +detected = detect_video(p);
 +iounmap(p);
 +if ( detected )
 +return 1;
 +
 +p = ioremap(0xB, 0x1000);
 +detected = detect_video(p);
 +iounmap(p);
 +if ( detected )
 +return 1;
 +
 +p = ioremap(0xB8000, 0x1000);
 +detected = detect_video(p);
 +iounmap(p);
 +if ( detected )
 +return 1;
 + 
-- 
Alex Williamson HP Open Source  Linux Org.


___
Xen-ppc-devel mailing list
Xen-ppc-devel@lists.xensource.com
http://lists.xensource.com/xen-ppc-devel


Re: [Xen-devel] Re: [XenPPC] [PATCH] [POWERPC] fix vga.c compilation

2006-08-16 Thread Keir Fraser



On 16/8/06 2:59 pm, Alex Williamson [EMAIL PROTECTED] wrote:

In general this looks a lot better, but I think ia64 is still going
 to have trouble with the chunk below.  It seems that a VGA card
 operating in a standard text mode doesn't necessarily respond to all of
 these addresses.  On some ia64 platforms that causes a hard fail
 response (the bus goes fatal and a reboot follows).  On my system, the
 0xB8000 test looks like it will probably work, but we never get there
 because either the 0xA or the 0xB test will cause the hardfail.
 Do we need to poke the card through I/O port space to get it into the
 right mode before probing it in MMIO space?  I don't know enough about
 the VGA programming model to be able to do that.  The card works once we
 start talking to it correctly, but this probe is a little too brute
 force.  Thanks,

Actually I'm not sure this is particularly necessary for headless x86
either. I could move the test to the end of setup_vga(), and only probe
0xb8000? x86 is more 'resilient' to random memory and port accesses. I don't
think it'll matter that we do lots of port accesses even if a vga adapter
isn't present. If we keep the conventional-ram test as well, then that'll be
the same level of checking that Linux does on ia64.

 -- Keir



___
Xen-ppc-devel mailing list
Xen-ppc-devel@lists.xensource.com
http://lists.xensource.com/xen-ppc-devel