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

2015-07-22 Thread Jan Beulich
>>> On 22.07.15 at 12:04, wrote: > Tiejun Chen writes ("[v11][PATCH 11/16] 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 h

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

2015-07-22 Thread Chen, Tiejun
I intend to produce a git branch RSN. On the staging? Tomorrow I can pull to check/test this. Thanks Tiejun ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

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

2015-07-22 Thread Ian Jackson
Chen, Tiejun writes ("Re: [v11][PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM"): > We just need to sync two patches: Thanks. > Note I just compiled with these changes since right now I can't access > any machine to test. I intend to produce a git branch RSN. Ian.

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

2015-07-22 Thread Chen, Tiejun
OOPS! Please refer to this version: (One miss changing flag to flags in patch #11 although we can compile successfully.) #1. To patch #8 diff --git a/tools/libxc/include/xenctrl.h b/tools/libxc/include/xenctrl.h index 2991333..9c5ef8b 100644 --- a/tools/libxc/include/xenctrl.h +++ b/tools/libx

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

2015-07-22 Thread Chen, Tiejun
On 2015/7/22 22:04, Ian Jackson wrote: Chen, Tiejun writes ("Re: [v11][PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM"): Sounds you start to merge them into your tree? But now Jan is trying to update patch #1 as you see. I think something needs to be synced on tool sides. Althou

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

2015-07-22 Thread Ian Jackson
Chen, Tiejun writes ("Re: [v11][PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM"): > Sounds you start to merge them into your tree? > > But now Jan is trying to update patch #1 as you see. I think something > needs to be synced on tool sides. Although that is not finished, at > le

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

2015-07-22 Thread Chen, Tiejun
On 2015/7/22 21:24, Ian Jackson wrote: Tiejun Chen writes ("[v11][PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM"): Acked-by: Wei Liu I have dropped Wei's ack on this from my git branch, as it was clearly inappropriate to retain it. (v11 was acked by me, so there is no need f

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

2015-07-22 Thread Ian Jackson
Tiejun Chen writes ("[v11][PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM"): > Acked-by: Wei Liu I have dropped Wei's ack on this from my git branch, as it was clearly inappropriate to retain it. (v11 was acked by me, so there is no need for a re-review by Wei unless he feels it

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

2015-07-22 Thread George Dunlap
On Wed, Jul 22, 2015 at 9:52 AM, Chen, Tiejun wrote: > And I also can't understand why we should drop Reviewed-by/Acked-by from > other guys. And, all new comments I addressed don't conflict with our > previous revision so why? You must remove Reviewed-by / Acked-by *for a given patch* *when you

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

2015-07-22 Thread Jan Beulich
>>> On 22.07.15 at 10:52, wrote: > On 2015/7/22 16:28, Jan Beulich wrote: > On 22.07.15 at 03:30, wrote: >>> CC: Ian Jackson >>> CC: Stefano Stabellini >>> CC: Ian Campbell >>> CC: Wei Liu >>> Acked-by: Wei Liu >>> Signed-off-by: Tiejun Chen >>> Reviewed-by: Kevin Tian >>> --- >>> v11:

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

2015-07-22 Thread Ian Jackson
Tiejun Chen writes ("[v11][PATCH 11/16] 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_pgen

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

2015-07-22 Thread Chen, Tiejun
On 2015/7/22 16:28, Jan Beulich wrote: On 22.07.15 at 03:30, wrote: CC: Ian Jackson CC: Stefano Stabellini CC: Ian Campbell CC: Wei Liu Acked-by: Wei Liu Signed-off-by: Tiejun Chen Reviewed-by: Kevin Tian --- v11: * Use GCNEW_ARRAY to replace libxl__malloc() * #define pfn_to_paddrk is

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

2015-07-22 Thread Jan Beulich
>>> On 22.07.15 at 03:30, wrote: > CC: Ian Jackson > CC: Stefano Stabellini > CC: Ian Campbell > CC: Wei Liu > Acked-by: Wei Liu > Signed-off-by: Tiejun Chen > Reviewed-by: Kevin Tian > --- > v11: > > * Use GCNEW_ARRAY to replace libxl__malloc() > > * #define pfn_to_paddrk is missing safe

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

2015-07-21 Thread 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 address space beyond 4