Re: [Qemu-devel] [PATCH] hw/i386: check if nvdimm is enabled before plugging

2017-01-23 Thread Stefan Hajnoczi
On Fri, Jan 13, 2017 at 07:56:51PM +0800, Haozhong Zhang wrote: > The missing of 'nvdimm' in the machine type option '-M' means NVDIMM > is disabled. QEMU should refuse to plug any NVDIMM device in this case > and report the misconfiguration. > > Reported-by: Stefan Hajnoczi > Signed-off-by: Haoz

Re: [Qemu-devel] [PATCH] hw/i386: check if nvdimm is enabled before plugging

2017-01-20 Thread Eduardo Habkost
On Fri, Jan 20, 2017 at 08:55:58AM +0800, Haozhong Zhang wrote: > On 01/16/17 11:00 +, Stefan Hajnoczi wrote: > > On Mon, Jan 16, 2017 at 01:55:34PM +0800, Xiao Guangrong wrote: > > > On 01/14/2017 02:02 AM, Eduardo Habkost wrote: > > > > On Fri, Jan 13, 2017 at 01:17:27PM +, Stefan Hajnocz

Re: [Qemu-devel] [PATCH] hw/i386: check if nvdimm is enabled before plugging

2017-01-19 Thread Haozhong Zhang
On 01/16/17 11:00 +, Stefan Hajnoczi wrote: On Mon, Jan 16, 2017 at 01:55:34PM +0800, Xiao Guangrong wrote: On 01/14/2017 02:02 AM, Eduardo Habkost wrote: > On Fri, Jan 13, 2017 at 01:17:27PM +, Stefan Hajnoczi wrote: > > On Fri, Jan 13, 2017 at 07:56:51PM +0800, Haozhong Zhang wrote: >

Re: [Qemu-devel] [PATCH] hw/i386: check if nvdimm is enabled before plugging

2017-01-16 Thread Stefan Hajnoczi
On Mon, Jan 16, 2017 at 01:55:34PM +0800, Xiao Guangrong wrote: > > > On 01/14/2017 02:02 AM, Eduardo Habkost wrote: > > On Fri, Jan 13, 2017 at 01:17:27PM +, Stefan Hajnoczi wrote: > > > On Fri, Jan 13, 2017 at 07:56:51PM +0800, Haozhong Zhang wrote: > > > > The missing of 'nvdimm' in the ma

Re: [Qemu-devel] [PATCH] hw/i386: check if nvdimm is enabled before plugging

2017-01-15 Thread Xiao Guangrong
On 01/14/2017 02:02 AM, Eduardo Habkost wrote: On Fri, Jan 13, 2017 at 01:17:27PM +, Stefan Hajnoczi wrote: On Fri, Jan 13, 2017 at 07:56:51PM +0800, Haozhong Zhang wrote: The missing of 'nvdimm' in the machine type option '-M' means NVDIMM is disabled. QEMU should refuse to plug any NVDI

Re: [Qemu-devel] [PATCH] hw/i386: check if nvdimm is enabled before plugging

2017-01-13 Thread Eduardo Habkost
On Fri, Jan 13, 2017 at 01:17:27PM +, Stefan Hajnoczi wrote: > On Fri, Jan 13, 2017 at 07:56:51PM +0800, Haozhong Zhang wrote: > > The missing of 'nvdimm' in the machine type option '-M' means NVDIMM > > is disabled. QEMU should refuse to plug any NVDIMM device in this case > > and report the m

Re: [Qemu-devel] [PATCH] hw/i386: check if nvdimm is enabled before plugging

2017-01-13 Thread Haozhong Zhang
On 01/13/17 13:17 +, Stefan Hajnoczi wrote: On Fri, Jan 13, 2017 at 07:56:51PM +0800, Haozhong Zhang wrote: The missing of 'nvdimm' in the machine type option '-M' means NVDIMM is disabled. QEMU should refuse to plug any NVDIMM device in this case and report the misconfiguration. Reported-b

Re: [Qemu-devel] [PATCH] hw/i386: check if nvdimm is enabled before plugging

2017-01-13 Thread Stefan Hajnoczi
On Fri, Jan 13, 2017 at 07:56:51PM +0800, Haozhong Zhang wrote: > The missing of 'nvdimm' in the machine type option '-M' means NVDIMM > is disabled. QEMU should refuse to plug any NVDIMM device in this case > and report the misconfiguration. > > Reported-by: Stefan Hajnoczi > Signed-off-by: Haoz

Re: [Qemu-devel] [PATCH] hw/i386: check if nvdimm is enabled before plugging

2017-01-13 Thread Xiao Guangrong
On 01/13/2017 07:56 PM, Haozhong Zhang wrote: The missing of 'nvdimm' in the machine type option '-M' means NVDIMM is disabled. QEMU should refuse to plug any NVDIMM device in this case and report the misconfiguration. Thanks for your fix. Reviewed-by: Xiao Guangrong

[Qemu-devel] [PATCH] hw/i386: check if nvdimm is enabled before plugging

2017-01-13 Thread Haozhong Zhang
The missing of 'nvdimm' in the machine type option '-M' means NVDIMM is disabled. QEMU should refuse to plug any NVDIMM device in this case and report the misconfiguration. Reported-by: Stefan Hajnoczi Signed-off-by: Haozhong Zhang Message-Id: 20170112110928.GF4621@stefanha-x1.localdomain Messag