Re: [Xen-devel] [PATCH v3 10/17] SUPPORT.md: Add Debugging, analysis, crash post-portem

[George Dunlap <george.dunlap@xxxxxxxxxx>]

On 11/23/2017 11:15 AM, Jan Beulich wrote:
>>>> On 22.11.17 at 20:20, <george.dunlap@xxxxxxxxxx> wrote:
>> +## Debugging, analysis, and crash post-mortem
>> +
>> +### Host serial console
>> +
>> +    Status, NS16550: Supported
>> +    Status, EHCI: Supported
> 
> Inconsistent indentation.

And I was so sure I'd checked all those. :-/

> 
>> +    Status, Cadence UART (ARM): Supported
>> +    Status, PL011 UART (ARM): Supported
>> +    Status, Exynos 4210 UART (ARM): Supported
>> +    Status, OMAP UART (ARM): Supported
>> +    Status, SCI(F) UART: Supported
>> +
>> +XXX Should NS16550 and EHCI be limited to x86?  Unlike the ARM
>> +entries, they don't depend on x86 being configured
> 
> ns16550 ought to be usable everywhere. EHCI is x86-only
> anyway (presumably first of all because it takes PCI as a prereq)

But that's just an accident at the moment; I thought there were plans at
some point for ARM servers to have PCI, weren't there?

I'll probably just leave this as it is, unless someone thinks differently.

> With this XXX dropped (and with or without adding (x86) to
> EHCI)
> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>

Thanks,
 -George

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel