Re: [Qemu-devel] [PATCH 00/16] implement vNVDIMM

2015-07-29 Thread Xiao Guangrong
On 07/03/2015 02:11 AM, Paolo Bonzini wrote: On 02/07/2015 20:01, Xiao Guangrong wrote: Thanks for your review, Stefan and Paolo! On 07/02/2015 05:52 PM, Paolo Bonzini wrote: On 02/07/2015 11:20, Stefan Hajnoczi wrote: Currently, the NVDIMM driver has been merged into upstream Linux Ke

Re: [Qemu-devel] [PATCH 00/16] implement vNVDIMM

2015-07-02 Thread Paolo Bonzini
On 02/07/2015 20:01, Xiao Guangrong wrote: > > Thanks for your review, Stefan and Paolo! > > On 07/02/2015 05:52 PM, Paolo Bonzini wrote: >> >> >> On 02/07/2015 11:20, Stefan Hajnoczi wrote: Currently, the NVDIMM driver has been merged into upstream Linux Kernel and this patchset

Re: [Qemu-devel] [PATCH 00/16] implement vNVDIMM

2015-07-02 Thread Xiao Guangrong
Thanks for your review, Stefan and Paolo! On 07/02/2015 05:52 PM, Paolo Bonzini wrote: On 02/07/2015 11:20, Stefan Hajnoczi wrote: Currently, the NVDIMM driver has been merged into upstream Linux Kernel and this patchset tries to enable it in virtualization field From a device model persp

Re: [Qemu-devel] [PATCH 00/16] implement vNVDIMM

2015-07-02 Thread Paolo Bonzini
On 02/07/2015 11:20, Stefan Hajnoczi wrote: > > Currently, the NVDIMM driver has been merged into upstream Linux Kernel and > > this patchset tries to enable it in virtualization field > > From a device model perspective, have you checked whether it makes sense > to integrate nvdimms into the pc

Re: [Qemu-devel] [PATCH 00/16] implement vNVDIMM

2015-07-02 Thread Stefan Hajnoczi
On Wed, Jul 01, 2015 at 10:50:16PM +0800, Xiao Guangrong wrote: > == Background == > NVDIMM (A Non-Volatile Dual In-line Memory Module) is going to be supported > on Intel's platform. They are discovered via ACPI and configured by _DSM > method of NVDIMM device in ACPI. There has some suppo

Re: [Qemu-devel] [PATCH 00/16] implement vNVDIMM

2015-07-02 Thread Michael S. Tsirkin
On Thu, Jul 02, 2015 at 09:31:23AM +0100, Stefan Hajnoczi wrote: > On Thu, Jul 02, 2015 at 02:34:05PM +0800, Xiao Guangrong wrote: > > On 07/02/2015 02:17 PM, Michael S. Tsirkin wrote: > > >On Wed, Jul 01, 2015 at 10:50:16PM +0800, Xiao Guangrong wrote: > > >> hw/acpi/aml-build.c | 32 +-

Re: [Qemu-devel] [PATCH 00/16] implement vNVDIMM

2015-07-02 Thread Stefan Hajnoczi
On Thu, Jul 02, 2015 at 02:34:05PM +0800, Xiao Guangrong wrote: > On 07/02/2015 02:17 PM, Michael S. Tsirkin wrote: > >On Wed, Jul 01, 2015 at 10:50:16PM +0800, Xiao Guangrong wrote: > >> hw/acpi/aml-build.c | 32 +- > >> hw/i386/acpi-build.c|9 +- > >> hw/i386/acpi-dsdt.dsl

Re: [Qemu-devel] [PATCH 00/16] implement vNVDIMM

2015-07-01 Thread Xiao Guangrong
On 07/02/2015 02:17 PM, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 10:50:16PM +0800, Xiao Guangrong wrote: hw/acpi/aml-build.c | 32 +- hw/i386/acpi-build.c|9 +- hw/i386/acpi-dsdt.dsl |2 +- hw/i386/pc.c| 11 +- hw/mem/Makefile.ob

Re: [Qemu-devel] [PATCH 00/16] implement vNVDIMM

2015-07-01 Thread Michael S. Tsirkin
On Wed, Jul 01, 2015 at 10:50:16PM +0800, Xiao Guangrong wrote: > hw/acpi/aml-build.c | 32 +- > hw/i386/acpi-build.c|9 +- > hw/i386/acpi-dsdt.dsl |2 +- > hw/i386/pc.c| 11 +- > hw/mem/Makefile.objs|1 + > hw/mem/pc-nvdimm.c

[Qemu-devel] [PATCH 00/16] implement vNVDIMM

2015-07-01 Thread Xiao Guangrong
== Background == NVDIMM (A Non-Volatile Dual In-line Memory Module) is going to be supported on Intel's platform. They are discovered via ACPI and configured by _DSM method of NVDIMM device in ACPI. There has some supporting documents which can be found at: ACPI 6: http://www.uefi.org/sites