Re: [Xen-ia64-devel] serial console settings on Tiger4

["KUWAMURA Shin'ya" <kuwa@xxxxxxxxxxxxxx>]

Hi Alex,

>>>>> On Fri, 09 Feb 2007 10:51:55 -0700
>>>>> alex.williamson@xxxxxx(Alex Williamson)  said:
> 
> On Fri, 2007-02-09 at 09:26 -0700, Alex Williamson wrote:
> 
> >    If anyone wants to write some auto-detection of these, it should be
> > pretty easy.  I would start with something like
> > xen/arch/ia64/xen/pcdp.c:pcdp_hp_irq_fixup(), determine you're on a
> > Tiger2 or Tiger4 based on XSDT oem_id and oem_table_id and poke the
> > right values into the ns16550_com1 and ns16550_com2 structures.  Based
> > on google searches for boot logs, you're probably looking for "INTEL"
> > and "SR870xxx", where the last 3 characters of the oem_table_id might
> > distinguish between a Tiger2 and Tiger4.  Thanks,
> 
>    I went ahead and wrote some code for this.  The patch below should be
> able to detect between a SR870BH2 (Tiger2, I think) and a SR870BN4
> (Tiger4, I hope).  Could someone with these systems please fill in the
> appropriate ns16550_com1/com2 value and try it out?  I took a guess, but
> have no idea if they're correct.  If Tiger4 only has one exposed UART, I
> think we should probably set it up as com1.  The ns16550 code should be
> able to read the baud out of the UART, so if all goes well, maybe Tiger
> users won't have to specify all those console parameters that they do
> now.  This patch should be used in addition to the previous patch I sent
> out in this thread.  Thanks,

Thank you for your work. These patch work fine with an additional
patch. I attach it. 

The final setting:
  append="dom0_mem=2G tbuf_size=8 dom0_max_vcpus=2 -- console=tty0 
console=ttyS1 rhgb root=/dev/sda2"

Best Regards,
-- 
  KUWAMURA Shin'ya

--- xen/arch/ia64/linux-xen/setup.c.orig        2007-02-13 14:29:02.000000000 
+0900
+++ xen/arch/ia64/linux-xen/setup.c     2007-02-13 16:55:32.000000000 +0900
@@ -354,7 +354,7 @@ intel_tiger_console_setup(void)
                return -ENODEV;
 
        /* Only looking for Intel systems */
-       if (strncmp(hdr->oem_id, "INTEL", sizeof(hdr->oem_id)))
+       if (strncmp(hdr->oem_id, "INTEL", 5))
                return -ENODEV;
 
        if (!strncmp(hdr->oem_table_id, "SR870BH2",

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