[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 04/25 v6] xen/arm: vpl011: Add support for vuart in libxl

Hi Wei,

On 07/28/2017 02:49 PM, Wei Liu wrote:
On Tue, Jul 25, 2017 at 11:08:24PM +0530, Bhupinder Thakur wrote:

On 18 July 2017 at 17:00, Wei Liu <wei.liu2@xxxxxxxxxx> wrote:
CC x86 maintainers

On Tue, Jul 18, 2017 at 12:19:19PM +0100, Julien Grall wrote:

      ("arch_arm", Struct(None, [("gic_version", libxl_gic_version),
+                               ("vuart", libxl_vuart_type),

... here it is ARM specific. I am not convinced that we should tie vuart to
ARM only. I cannot see why x86 would not be able to use it in the future.
Any opinions?

I don't know. I asked Bhupinder to put it here because it looked arm
specific to me. I will let x86 maintainers to decide whether they want
such thing.

What is the decision on this?

Unfortunately this email probably slipped through the crack for Andrew
and Jan.

I've prodded Andrew on IRC so he might chime in.

I believe that since most of the vuart code added in libxl is arch
agnostic, it should be fine to keep the libxl_vuart_type as a generic

What about the actual emulation code? Is that arch-agnostic? If not, I
personally don't see a chance of having vuart emulation for x86 in the
near future and I'm inclined to keep the code as-is.

There is always the option to lift the struct to common code in the

Lifting the struct to common will imply to add compatibility code, right?


Julien Grall

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.