From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Daniel P. Berrange" Subject: Re: [PATCH] Kvm: Qemu: save nvram Date: Tue, 2 Dec 2008 13:10:15 +0000 Message-ID: <20081202131015.GQ25599@redhat.com> References: <10C63FAD690C13458F0B32BCED571F14021760D4@pdsmsx502.ccr.corp.intel.com> <20081202103145.GC25599@redhat.com> <493531A0.2040501@redhat.com> Reply-To: "Daniel P. Berrange" Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "Zhang, Yang" , "kvm-ia64@vger.kernel.org" , "kvm@vger.kernel.org" , "avi@readhat.com" , "Zhang, Xiantao" To: Avi Kivity Return-path: Received: from mx1.redhat.com ([66.187.233.31]:36376 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753902AbYLBNKX (ORCPT ); Tue, 2 Dec 2008 08:10:23 -0500 Content-Disposition: inline In-Reply-To: <493531A0.2040501@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Tue, Dec 02, 2008 at 03:01:20PM +0200, Avi Kivity wrote: > Daniel P. Berrange wrote: > >On Tue, Dec 02, 2008 at 10:25:49AM +0800, Zhang, Yang wrote: > > > >>This patch to save the nvram. It save the nvram by specify the arg of > >>-name.And the saved file named by the arg. If do not specify the arg, > >>it will not save the nvram > >> > > > >I think we might be better off having an explicit command line arg for > >nvram > >path rather than hardcoding the directory, because there may well be times > >where you want to have nvram saved, but don't want to specify -name, and > >vica-verca. > > > > -nvram foo.data > > > >could prepend a default directory of $localstatedir/lib/qemu/nvram, where > >$localstatedir is set from 'configure' script, or > > > > -nvram /some/path/foo.data > > > >would use the explicit path given. > > > > I prefer current directory if relative path is given. Since we > encourage running qemu as an unprivileged user, and we don't want a > world-writable directory, each user will have to provide a private > storage location. Fine by me - avoids needing to embed any path in QEMU code at all then Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|