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

From: George Dunlap <george.dunlap@citrix.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: StefanoStabellini <sstabellini@kernel.org>,
	Wei Liu <wei.liu2@citrix.com>,
	Konrad Wilk <konrad.wilk@oracle.com>,
	Andrew Cooper <andrew.cooper3@citrix.com>,
	Tim Deegan <tim@xen.org>, Ian Jackson <ian.jackson@citrix.com>,
	xen-devel@lists.xenproject.org
Subject: Re: [PATCH v3 10/17] SUPPORT.md: Add Debugging, analysis, crash post-portem
Date: Thu, 23 Nov 2017 17:08:46 +0000	[thread overview]
Message-ID: <c064cbfd-c17c-493e-66ff-6874620ffee1@citrix.com> (raw)
In-Reply-To: <5A16BBDA0200007800191538@prv-mh.provo.novell.com>

On 11/23/2017 11:15 AM, Jan Beulich wrote:
>>>> On 22.11.17 at 20:20, <george.dunlap@citrix.com> 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@suse.com>

Thanks,
 -George

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