Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-14 Thread Haozhong Zhang
On 02/13/18 15:39 +, Roger Pau Monné wrote: > On Tue, Feb 13, 2018 at 06:40:20AM -0700, Jan Beulich wrote: > > >>> On 13.02.18 at 12:13, wrote: > > > On Tue, Feb 13, 2018 at 04:05:45AM -0700, Jan Beulich wrote: > > >> >>> On 13.02.18 at 11:29, wrote: > > >> > On Tue, Feb 13, 2018 at 03:06:24A

Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-14 Thread Haozhong Zhang
On 02/12/18 10:05 +, Roger Pau Monné wrote: > On Mon, Feb 12, 2018 at 09:25:42AM +0800, Haozhong Zhang wrote: > > On 02/09/18 12:33 +, Roger Pau Monné wrote: > > > Thanks for the series, I'm however wondering whether it's appropriate > > > to post a v4 as RFC. Ie: at v4 the reviewer expects

Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-13 Thread Roger Pau Monné
On Tue, Feb 13, 2018 at 06:40:20AM -0700, Jan Beulich wrote: > >>> On 13.02.18 at 12:13, wrote: > > On Tue, Feb 13, 2018 at 04:05:45AM -0700, Jan Beulich wrote: > >> >>> On 13.02.18 at 11:29, wrote: > >> > On Tue, Feb 13, 2018 at 03:06:24AM -0700, Jan Beulich wrote: > >> >> >>> On 12.02.18 at 11:

Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-13 Thread Jan Beulich
>>> On 13.02.18 at 12:13, wrote: > On Tue, Feb 13, 2018 at 04:05:45AM -0700, Jan Beulich wrote: >> >>> On 13.02.18 at 11:29, wrote: >> > On Tue, Feb 13, 2018 at 03:06:24AM -0700, Jan Beulich wrote: >> >> >>> On 12.02.18 at 11:05, wrote: >> >> > If you map the NVDIMM as MMIO to Dom0 you don't nee

Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-13 Thread Roger Pau Monné
On Tue, Feb 13, 2018 at 04:05:45AM -0700, Jan Beulich wrote: > >>> On 13.02.18 at 11:29, wrote: > > On Tue, Feb 13, 2018 at 03:06:24AM -0700, Jan Beulich wrote: > >> >>> On 12.02.18 at 11:05, wrote: > >> > If you map the NVDIMM as MMIO to Dom0 you don't need the M2P entries > >> > IIRC, and if it

Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-13 Thread Jan Beulich
>>> On 13.02.18 at 11:29, wrote: > On Tue, Feb 13, 2018 at 03:06:24AM -0700, Jan Beulich wrote: >> >>> On 12.02.18 at 11:05, wrote: >> > If you map the NVDIMM as MMIO to Dom0 you don't need the M2P entries >> > IIRC, and if it's mapped using 1GB pages it shouldn't use that much >> > memory for th

Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-13 Thread Roger Pau Monné
On Tue, Feb 13, 2018 at 03:06:24AM -0700, Jan Beulich wrote: > >>> On 12.02.18 at 11:05, wrote: > > If you map the NVDIMM as MMIO to Dom0 you don't need the M2P entries > > IIRC, and if it's mapped using 1GB pages it shouldn't use that much > > memory for the page tables (ie: you could just use no

Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-13 Thread Jan Beulich
>>> On 12.02.18 at 11:05, wrote: > If you map the NVDIMM as MMIO to Dom0 you don't need the M2P entries > IIRC, and if it's mapped using 1GB pages it shouldn't use that much > memory for the page tables (ie: you could just use normal RAM for the > page tables that map the NVDIMM IMO). Of course th

Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-12 Thread Roger Pau Monné
On Mon, Feb 12, 2018 at 09:25:42AM +0800, Haozhong Zhang wrote: > On 02/09/18 12:33 +, Roger Pau Monné wrote: > > Thanks for the series, I'm however wondering whether it's appropriate > > to post a v4 as RFC. Ie: at v4 the reviewer expects the submitter to > > have a clear picture of what needs

Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-11 Thread Haozhong Zhang
On 02/09/18 12:33 +, Roger Pau Monné wrote: > Thanks for the series, I'm however wondering whether it's appropriate > to post a v4 as RFC. Ie: at v4 the reviewer expects the submitter to > have a clear picture of what needs to be implemented. > > On Thu, Dec 07, 2017 at 06:09:49PM +0800, Haozh

Re: [Xen-devel] [RFC XEN PATCH v4 00/41] Add vNVDIMM support to HVM domains

2018-02-09 Thread Roger Pau Monné
Thanks for the series, I'm however wondering whether it's appropriate to post a v4 as RFC. Ie: at v4 the reviewer expects the submitter to have a clear picture of what needs to be implemented. On Thu, Dec 07, 2017 at 06:09:49PM +0800, Haozhong Zhang wrote: > All patches can also be found at > Xe