Re: [Qemu-devel] [PULL 011/103] pc-dimm: do not allow to set already used memdev

2014-06-17 Thread Michael S. Tsirkin
On Tue, Jun 17, 2014 at 07:45:10PM +0100, Peter Maydell wrote: > On 17 June 2014 19:39, Eric Blake wrote: > > On 06/17/2014 11:36 AM, Michael S. Tsirkin wrote: > >> From: Igor Mammedov > >> > >> using the same memdev backend more than once will cause > >> assersion at MemoryRegion mapping time be

Re: [Qemu-devel] [PULL 011/103] pc-dimm: do not allow to set already used memdev

2014-06-17 Thread Eric Blake
On 06/17/2014 12:45 PM, Peter Maydell wrote: > On 17 June 2014 19:39, Eric Blake wrote: >> On 06/17/2014 11:36 AM, Michael S. Tsirkin wrote: >>> From: Igor Mammedov >>> >>> using the same memdev backend more than once will cause >>> assersion at MemoryRegion mapping time becase it's already >> >>

Re: [Qemu-devel] [PULL 011/103] pc-dimm: do not allow to set already used memdev

2014-06-17 Thread Peter Maydell
On 17 June 2014 19:39, Eric Blake wrote: > On 06/17/2014 11:36 AM, Michael S. Tsirkin wrote: >> From: Igor Mammedov >> >> using the same memdev backend more than once will cause >> assersion at MemoryRegion mapping time becase it's already > > s/assersion/assertion/; s/becase/because/ > (but prob

Re: [Qemu-devel] [PULL 011/103] pc-dimm: do not allow to set already used memdev

2014-06-17 Thread Michael S. Tsirkin
On Tue, Jun 17, 2014 at 12:39:32PM -0600, Eric Blake wrote: > On 06/17/2014 11:36 AM, Michael S. Tsirkin wrote: > > From: Igor Mammedov > > > > using the same memdev backend more than once will cause > > assersion at MemoryRegion mapping time becase it's already > > s/assersion/assertion/; s/bec

Re: [Qemu-devel] [PULL 011/103] pc-dimm: do not allow to set already used memdev

2014-06-17 Thread Eric Blake
On 06/17/2014 11:36 AM, Michael S. Tsirkin wrote: > From: Igor Mammedov > > using the same memdev backend more than once will cause > assersion at MemoryRegion mapping time becase it's already s/assersion/assertion/; s/becase/because/ (but probably too late now that this is a pull request) > ma