RE: [PATCH] Kvm: Qemu: save nvram

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Ok, I will save it in current directory and don't read nvram from file, if don't specify -nvram.

Best Regards
--yang
-----Original Message-----
From: Zhang, Xiantao 
Sent: 2008年12月3日 14:02
To: Daniel P. Berrange; Avi Kivity
Cc: Zhang, Yang; kvm-ia64@xxxxxxxxxxxxxxx; kvm@xxxxxxxxxxxxxxx
Subject: RE: [PATCH] Kvm: Qemu: save nvram

Daniel P. Berrange wrote:
> 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

If acceptable for upstream, it should be the best choice.  

>>> 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

So we don't need a default path ? 
Yang, could you address Daniel and Avi's comments in next version ? 

Xiantao

?韬{.n?????%??檩??w?{.n??ぞo???ay????j?f"??????_璁(????"??m???G??⒏??璀?x??


[Index of Archives]     [Linux KVM Devel]     [Linux Virtualization]     [Big List of Linux Books]     [Linux SCSI]     [Yosemite Forum]

  Powered by Linux