[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:Hi, 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 type.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 future. Lifting the struct to common will imply to add compatibility code, right? Cheers, -- Julien Grall _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |