From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:39163)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <bohdan.trach@mailbox.tu-dresden.de>)
	id 1ZyiKj-0002Ou-FB
	for qemu-devel@nongnu.org; Tue, 17 Nov 2015 10:38:14 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <bohdan.trach@mailbox.tu-dresden.de>)
	id 1ZyiKf-0004CP-EQ
	for qemu-devel@nongnu.org; Tue, 17 Nov 2015 10:38:13 -0500
Received: from mailout1.zih.tu-dresden.de ([141.30.67.72]:57367)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <bohdan.trach@mailbox.tu-dresden.de>)
	id 1ZyiKf-0004Bo-7R
	for qemu-devel@nongnu.org; Tue, 17 Nov 2015 10:38:09 -0500
References: <cover.1429272036.git.bv.trach@gmail.com>
	<d2e6189248c6e4c2d04f849d57409952ff79da94.1429272036.git.bv.trach@gmail.com>
	<20151116164618.GE2417@work-vm>
From: Bohdan Trach <bohdan.trach@mailbox.tu-dresden.de>
Message-ID: <564B49DF.7010009@mailbox.tu-dresden.de>
Date: Tue, 17 Nov 2015 16:38:07 +0100
MIME-Version: 1.0
In-Reply-To: <20151116164618.GE2417@work-vm>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH RFC 1/3] memory: Add dump-pc-mem command
 for checkpointing
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
Cc: amit.shah@redhat.com, thomas.knauth@googlemail.com, qemu-devel@nongnu.org, quintela@redhat.com

Hi David, thank you for the feedback!

On 11/16/2015 05:46 PM, Dr. David Alan Gilbert wrote:
> * Bohdan Trach (bv.trach@gmail.com) wrote:
>> From: Bohdan Trach <bohdan.trach@mailbox.tu-dresden.de>
>>
>> dump-pc-mem command is added for checkpointing guest memory to
>> file. Only system RAM region is saved. This checkpoint is later used to
>> recover unchanged pages.
> 
> Why not just use the 'dump_guest_memory' commands; they dump it in interesting
> existing formats; they have headers in the files as well rather than just
> a raw blob of data.
> If you wanted to restrict to only certain RAM blocks, then I'd suggest adding
> a feature to that existing command.
> You might also find that you want other RAMBlocks as well, for example where
> RAM is added using hot plu, those are separate RAM blocks.

We will try to rework these patches to use existing formats. Current
format was used because it is extremely simple to work with. The
restriction of saving only 'pc.ram' RAMBlock is just the consequence
of this design choice.

> --
> Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
> 

-- 
With best regards,
Bohdan Trach