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

Re: [libvirt] [PATCH v2] Add new attribute writeout to <filesystem> element



On Thu, Dec 22, 2011 at 05:25:49PM +0100, Paolo Bonzini wrote:
> On 12/22/2011 03:20 PM, Eric Blake wrote:
> >>>  Usage:
> >>>  <filesystem type='mount' accessmode='passthrough' writeout='immediate'>
> >>>     <source dir='/export/to/guest'/>
> >>>     <target dir='mount_tag'/>
> >>>  </filesystem>
> >I still don't like the proposed XML.  We need to get consensus on where
> >to put it, with my proposal being:
> >
> ><filesystem type='mount'>
> >   <driver type='path' write='immediate'/>
> >   <source dir='/export/to/guest'/>
> >   <target dir='mount_tag'/>
> ></filesystem>
> 
> Right, other types of drivers might include a userspace NFS client,
> or some kind of "artificial" filesystem.
> 
> In addition, instead of write='immediate' I suggest
> cache='writeback'/cache='writethrough'.

Yeah I agree - we should follow the syntax we already use for caching
with the <disk> element instead of inventing new syntax


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|


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