On Thu, Oct 04, 2012 at 09:53:49AM -0600, Eric Blake wrote:
> On 09/24/2012 09:44 AM, Shawn Furrow wrote:
>
> [sorry for my delay in replying]
>
> > *Created at Virginia Tech's Systems Software Research Group
> >
> > This patch adds the XML schema and implementation for IVSHMEM device driver
> > support. Currently it defaults to using interrupts. A sample IVSHMEM entry
> > in the VM's XML file is:
>
> Thanks for starting to tackle this.
>
> >
> > <ivshmem id='nahanni' size='16834' path='/tmp/'/>
>
> Elsewhere, we have represented memory sized in KiB (1024 byte units);
> what scale are you using here? A default unit of byte might be nicer,
> on the other hand, does shared memory have to be page aligned, at which
> point it will always be a multiple of 4k (and thus listing in KiB still
> makes sense)? At any rate, I think that this argues you need to support
> units on output to show the preferred scale, as well as parse it on
> input to allow users to specify units='M' size='1' to reserve 2**20
> bytes with ease.
I think it'd be worthwhile to push some of the attributes
down to a subelement, eg
<ivshmem id='nahanni'>
<source size="12345" units="M" path='/tmp'/>
</ivshmem>
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 :|
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list