Re: [Xen-devel] [PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM

2015-07-23 Thread Ian Jackson
Chen, Tiejun writes ("Re: [PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM"): > Thanks for your effort. > > A tiny change may be needed but I don't block this. Thanks. I have applied this change to my tree. There is no functional change and I don't feel the need to collect a new

Re: [Xen-devel] [PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM

2015-07-23 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM"): > Ian, you forgot your S-o-B. So I did, thanks. > Acked-by: Wei Liu Thanks. Also, I should mention that have retained the ack from Kevin Tian (CC'd), since my changes were almost entirely mechanical. Ian.

Re: [Xen-devel] [PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM

2015-07-23 Thread Chen, Tiejun
These cosmetic changes can be fixed by a follow-up patch. If Jackson would like not to fix this directly in his tree, I can post this a small patch but we'd better squash this into the predecessor just as one commit. Thanks Tiejun ___ Xen-devel m

Re: [Xen-devel] [PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM

2015-07-23 Thread Wei Liu
On Thu, Jul 23, 2015 at 08:52:24AM +0800, Chen, Tiejun wrote: > Ian, > > Thanks for your effort. > > A tiny change may be needed but I don't block this. > > >+libxl__xc_device_get_rdm(libxl__gc *gc, > >+ uint32_t flag, > > Since now we are sitting on xc_reserved_device_m

Re: [Xen-devel] [PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM

2015-07-22 Thread Chen, Tiejun
Ian, Thanks for your effort. A tiny change may be needed but I don't block this. +libxl__xc_device_get_rdm(libxl__gc *gc, + uint32_t flag, Since now we are sitting on xc_reserved_device_memory_map(, flags, xxx), s/flag/flags may be better. +

Re: [Xen-devel] [PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM

2015-07-22 Thread Wei Liu
On Wed, Jul 22, 2015 at 04:44:14PM +0100, Ian Jackson wrote: > From: Tiejun Chen > > While building a VM, HVM domain builder provides struct hvm_info_table{} > to help hvmloader. Currently it includes two fields to construct guest > e820 table by hvmloader, low_mem_pgend and high_mem_pgend. So we

[Xen-devel] [PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM

2015-07-22 Thread Ian Jackson
From: Tiejun Chen While building a VM, HVM domain builder provides struct hvm_info_table{} to help hvmloader. Currently it includes two fields to construct guest e820 table by hvmloader, low_mem_pgend and high_mem_pgend. So we should check them to fix any conflict with RDM. RMRR can reside in ad