From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47749) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZyiiI-0005hS-H6 for qemu-devel@nongnu.org; Tue, 17 Nov 2015 11:02:40 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZyiiF-00039r-9W for qemu-devel@nongnu.org; Tue, 17 Nov 2015 11:02:34 -0500 Received: from mx1.redhat.com ([209.132.183.28]:33247) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZyiiF-00039l-45 for qemu-devel@nongnu.org; Tue, 17 Nov 2015 11:02:31 -0500 Date: Tue, 17 Nov 2015 16:02:26 +0000 From: "Dr. David Alan Gilbert" Message-ID: <20151117160225.GL2498@work-vm> References: <20151116164618.GE2417@work-vm> <564B49DF.7010009@mailbox.tu-dresden.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <564B49DF.7010009@mailbox.tu-dresden.de> Subject: Re: [Qemu-devel] [PATCH RFC 1/3] memory: Add dump-pc-mem command for checkpointing List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Bohdan Trach Cc: amit.shah@redhat.com, thomas.knauth@googlemail.com, qemu-devel@nongnu.org, quintela@redhat.com * Bohdan Trach (bohdan.trach@mailbox.tu-dresden.de) wrote: > Hi David, thank you for the feedback! No problem, sorry for the delay, > On 11/16/2015 05:46 PM, Dr. David Alan Gilbert wrote: > > * Bohdan Trach (bv.trach@gmail.com) wrote: > >> From: Bohdan Trach > >> > >> 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. I think the ELF one should actually turn out to be the easiest; there's probably some existing ELF code you can find in the QEMU code base to make it easier to parse, but once you find the start of the block it should then be contiguous. Dave > > -- > > Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK > > > > -- > With best regards, > Bohdan Trach -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK