Re: [PATCH 1/2] Added timestamps to storage volumes

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

 



On 07/19/2012 01:13 AM, Hendrik Schwartke wrote:
> The access, birth, modification and change times are added to
> storage volumes and corresponding xml representations.
> ---
>  docs/formatstorage.html.in    |   27 +++++++++++++++++++++++++++
>  docs/schemas/storagevol.rng   |   39 +++++++++++++++++++++++++++++++++++++++
>  src/conf/storage_conf.c       |   21 +++++++++++++++++++++
>  src/conf/storage_conf.h       |   13 +++++++++++++
>  src/storage/storage_backend.c |   21 +++++++++++++++++++++
>  5 files changed, 121 insertions(+)
> 
> diff --git a/docs/formatstorage.html.in b/docs/formatstorage.html.in
> index d0e4319..d8ffbea 100644
> --- a/docs/formatstorage.html.in
> +++ b/docs/formatstorage.html.in
> @@ -141,6 +141,20 @@
>              <mode>0744</mode>
>              <label>virt_image_t</label>
>            </permissions>
> +          <timestamps>
> +            <atime>
> +              <sec>1341933637</sec>
> +              <nsec>27319099</nsec>

Just because C can't represent struct timespec as a single number
doesn't mean that our XML has to suffer; I still think %d.%09lld is a
better format than splitting <sec> and <nsec> into separate elements.

On the other hand, will XPath queries of the XML be easier if it is a
floating point number compared to two different elements?  Does anyone
else have an opinion on which color to paint this bikeshed?

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]