Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-19 Thread Chen, Tiejun
On 2015/5/19 4:00, Wei Liu wrote: On Thu, May 14, 2015 at 04:27:45PM +0800, Chen, Tiejun wrote: On 2015/5/11 19:32, Wei Liu wrote: On Mon, May 11, 2015 at 04:09:53PM +0800, Chen, Tiejun wrote: On 2015/5/8 22:43, Wei Liu wrote: Sorry for the late review. This series fell through the crack.

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-19 Thread Wei Liu
On Tue, May 19, 2015 at 01:32:27AM +, Tian, Kevin wrote: From: Wei Liu [mailto:wei.l...@citrix.com] Sent: Tuesday, May 19, 2015 4:00 AM [...] Yes, but this policy is hardcoded in code (as in, you bail when detecting conflict in highmem region). I don't think we have the

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-18 Thread Wei Liu
On Thu, May 14, 2015 at 04:27:45PM +0800, Chen, Tiejun wrote: On 2015/5/11 19:32, Wei Liu wrote: On Mon, May 11, 2015 at 04:09:53PM +0800, Chen, Tiejun wrote: On 2015/5/8 22:43, Wei Liu wrote: Sorry for the late review. This series fell through the crack. Thanks for your review. On

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-18 Thread Tian, Kevin
From: Wei Liu [mailto:wei.l...@citrix.com] Sent: Tuesday, May 19, 2015 4:00 AM On Thu, May 14, 2015 at 04:27:45PM +0800, Chen, Tiejun wrote: On 2015/5/11 19:32, Wei Liu wrote: On Mon, May 11, 2015 at 04:09:53PM +0800, Chen, Tiejun wrote: On 2015/5/8 22:43, Wei Liu wrote: Sorry for the

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-14 Thread Chen, Tiejun
On 2015/5/11 19:32, Wei Liu wrote: On Mon, May 11, 2015 at 04:09:53PM +0800, Chen, Tiejun wrote: On 2015/5/8 22:43, Wei Liu wrote: Sorry for the late review. This series fell through the crack. Thanks for your review. On Fri, Apr 10, 2015 at 05:21:55PM +0800, Tiejun Chen wrote: While

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-11 Thread Chen, Tiejun
On 2015/5/8 23:13, Wei Liu wrote: On Fri, May 08, 2015 at 09:24:56AM +0800, Chen, Tiejun wrote: Campbell, Jackson, Wei and Stefano, Any consideration? I can follow up Jan's idea but I need you guys make sure I'm going to do this properly. Look at my earlier reply. Thanks for your reply.

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-11 Thread Chen, Tiejun
On 2015/5/8 22:43, Wei Liu wrote: Sorry for the late review. This series fell through the crack. Thanks for your review. On Fri, Apr 10, 2015 at 05:21:55PM +0800, Tiejun Chen wrote: While building a VM, HVM domain builder provides struct hvm_info_table{} to help hvmloader. Currently it

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-11 Thread Wei Liu
On Mon, May 11, 2015 at 04:09:53PM +0800, Chen, Tiejun wrote: On 2015/5/8 22:43, Wei Liu wrote: Sorry for the late review. This series fell through the crack. Thanks for your review. On Fri, Apr 10, 2015 at 05:21:55PM +0800, Tiejun Chen wrote: While building a VM, HVM domain builder

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-08 Thread Wei Liu
Sorry for the late review. This series fell through the crack. On Fri, Apr 10, 2015 at 05:21:55PM +0800, Tiejun Chen wrote: 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,

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-08 Thread Wei Liu
On Fri, May 08, 2015 at 09:24:56AM +0800, Chen, Tiejun wrote: Campbell, Jackson, Wei and Stefano, Any consideration? I can follow up Jan's idea but I need you guys make sure I'm going to do this properly. Look at my earlier reply. 1. This function seems to have bug. 2. Caller should

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-07 Thread Chen, Tiejun
Campbell, Jackson, Wei and Stefano, Any consideration? I can follow up Jan's idea but I need you guys make sure I'm going to do this properly. Thanks Tiejun On 2015/5/7 10:22, Chen, Tiejun wrote: On 2015/5/6 23:34, Jan Beulich wrote: On 06.05.15 at 17:00, tiejun.c...@intel.com wrote: On

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-07 Thread Chen, Tiejun
On 2015/5/7 14:04, Jan Beulich wrote: Chen, Tiejun tiejun.c...@intel.com 05/07/15 4:22 AM On 2015/5/6 23:34, Jan Beulich wrote: On 06.05.15 at 17:00, tiejun.c...@intel.com wrote: On 2015/4/20 19:13, Jan Beulich wrote: On 10.04.15 at 11:21, tiejun.c...@intel.com wrote: +

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-06 Thread Chen, Tiejun
On 2015/5/6 23:34, Jan Beulich wrote: On 06.05.15 at 17:00, tiejun.c...@intel.com wrote: On 2015/4/20 19:13, Jan Beulich wrote: On 10.04.15 at 11:21, tiejun.c...@intel.com wrote: --- a/tools/libxc/xc_domain.c +++ b/tools/libxc/xc_domain.c @@ -1665,6 +1665,46 @@ int xc_assign_device(

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-06 Thread Jan Beulich
On 06.05.15 at 17:00, tiejun.c...@intel.com wrote: On 2015/4/20 19:13, Jan Beulich wrote: On 10.04.15 at 11:21, tiejun.c...@intel.com wrote: --- a/tools/libxc/xc_domain.c +++ b/tools/libxc/xc_domain.c @@ -1665,6 +1665,46 @@ int xc_assign_device( return do_domctl(xch, domctl); }

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-05-06 Thread Chen, Tiejun
On 2015/4/20 19:13, Jan Beulich wrote: On 10.04.15 at 11:21, tiejun.c...@intel.com wrote: --- a/tools/libxc/xc_domain.c +++ b/tools/libxc/xc_domain.c @@ -1665,6 +1665,46 @@ int xc_assign_device( return do_domctl(xch, domctl); } +struct xen_reserved_device_memory

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-04-23 Thread Chen, Tiejun
On 2015/4/15 21:10, Ian Jackson wrote: Tiejun Chen writes ([RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM): 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

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-04-20 Thread Jan Beulich
On 10.04.15 at 11:21, tiejun.c...@intel.com wrote: --- a/tools/libxc/xc_domain.c +++ b/tools/libxc/xc_domain.c @@ -1665,6 +1665,46 @@ int xc_assign_device( return do_domctl(xch, domctl); } +struct xen_reserved_device_memory +*xc_device_get_rdm(xc_interface *xch, +

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-04-15 Thread Ian Jackson
Tiejun Chen writes ([RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM): 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.

Re: [Xen-devel] [RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM

2015-04-15 Thread Tian, Kevin
From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] Sent: Wednesday, April 15, 2015 9:10 PM Tiejun Chen writes ([RFC][PATCH 04/13] tools/libxl: detect and avoid conflicts with RDM): While building a VM, HVM domain builder provides struct hvm_info_table{} to help hvmloader. Currently