From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:54728) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SnF6o-0002YV-Qp for qemu-devel@nongnu.org; Fri, 06 Jul 2012 16:26:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SnF6m-0006pt-Sn for qemu-devel@nongnu.org; Fri, 06 Jul 2012 16:26:34 -0400 Received: from mx1.redhat.com ([209.132.183.28]:46653) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SnF6m-0006p4-KR for qemu-devel@nongnu.org; Fri, 06 Jul 2012 16:26:32 -0400 Received: from int-mx01.intmail.prod.int.phx2.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id q66KQUmZ026788 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Fri, 6 Jul 2012 16:26:30 -0400 Date: Fri, 6 Jul 2012 16:26:29 -0400 From: Jason Baron Message-ID: <20120706202628.GC23433@redhat.com> References: <201205042123.q44LNp9x029061@int-mx01.intmail.prod.int.phx2.redhat.com> <20120521175335.GD3353@redhat.com> <20120706000418.GB7150@amt.cnet> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120706000418.GB7150@amt.cnet> Subject: Re: [Qemu-devel] [PATCH] memory: add -dont-dump-guest option to reduce core dump size List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Marcelo Tosatti Cc: armbru@redhat.com, avi@redhat.com, qemu-devel@nongnu.org On Thu, Jul 05, 2012 at 09:04:18PM -0300, Marcelo Tosatti wrote: > On Mon, May 21, 2012 at 01:53:36PM -0400, Jason Baron wrote: > > On Fri, May 04, 2012 at 05:23:51PM -0400, Jason Baron wrote: > > > Add a command line parameter to not dump guest memory in the core dump, the > > > command line is: -dont-dump-guest. This brought the core dump down from > > > 383MB to 13 MB on a 1GB guest. > > > > > > Signed-off-by: Jason Baron > > > --- > > > exec.c | 13 +++++++++++++ > > > osdep.h | 7 +++++++ > > > qemu-options.hx | 5 +++++ > > > sysemu.h | 1 + > > > vl.c | 4 ++++ > > > 5 files changed, 30 insertions(+), 0 deletions(-) > > > > > > > Any thoughts on this? Is the guest memory often helpful in debugging > > when the qemu process segfaults? > > For most development related usage, no (with large guests, its > troublesome). > > Please add documentation for the command (patch looks fine otherwise). > > > This feature also seems useful if somebody is running a sensitive > > workload, such that the non-sensitive qemu state can be dumped > > independently of the guest state. In that case, perhaps there should > > also be an option that allows the guest to be dumped on a segfault, now > > separately? > > Isnt what this option does? (well, disabling it). > Right, disabling it gives us both qemu and guest bits. So I was just suggesting a mode that dumps guest bits but not qemu bits. (Probably that is setup in guest, though). So I'm not sure if that makes sense. But as Markus suggested, if we make the interface more general, we are more open to something like that later (at least in terms of the interface). Thanks, -Jason