Hi, On 08/02/16 11:18, Fam Zheng wrote: > The QEMU UUID api, including the data structure (qemu_uuid_t), is fully > compatible with libuuid. Use it, and remove the unused code. > > Signed-off-by: Fam Zheng <[email protected]> > --- > block/vdi.c | 49 ++++++++++--------------------------------------- > 1 file changed, 10 insertions(+), 39 deletions(-) > > diff --git a/block/vdi.c b/block/vdi.c > index 8a1cf97..ed63184 100644 > --- a/block/vdi.c > +++ b/block/vdi.c > @@ -58,14 +58,7 @@ > #include "migration/migration.h" > #include "qemu/coroutine.h" > #include "qemu/cutils.h" > - > -#if defined(CONFIG_UUID) > -#include <uuid/uuid.h> > -#else > -/* TODO: move uuid emulation to some central place in QEMU. */ > -#include "sysemu/sysemu.h" /* UUID_FMT */ > -typedef unsigned char uuid_t[16]; > -#endif > +#include "qemu/uuid.h" > > /* Code configuration options. */ > > @@ -140,28 +133,6 @@ typedef unsigned char uuid_t[16]; > #define VDI_DISK_SIZE_MAX ((uint64_t)VDI_BLOCKS_IN_IMAGE_MAX * \ > (uint64_t)DEFAULT_CLUSTER_SIZE) > > -#if !defined(CONFIG_UUID) > -static inline void uuid_generate(uuid_t out) > -{ > - memset(out, 0, sizeof(uuid_t)); > -} > - > -static inline int uuid_is_null(const uuid_t uu) > -{ > - uuid_t null_uuid = { 0 }; > - return memcmp(uu, null_uuid, sizeof(uuid_t)) == 0; > -} > - > -# if defined(CONFIG_VDI_DEBUG) > -static inline void uuid_unparse(const uuid_t uu, char *out) > -{ > - snprintf(out, 37, UUID_FMT, > - uu[0], uu[1], uu[2], uu[3], uu[4], uu[5], uu[6], uu[7], > - uu[8], uu[9], uu[10], uu[11], uu[12], uu[13], uu[14], uu[15]); > -} > -# endif > -#endif > - > typedef struct { > char text[0x40]; > uint32_t signature; > @@ -182,10 +153,10 @@ typedef struct { > uint32_t block_extra; /* unused here */ > uint32_t blocks_in_image; > uint32_t blocks_allocated; > - uuid_t uuid_image; > - uuid_t uuid_last_snap; > - uuid_t uuid_link; > - uuid_t uuid_parent; > + qemu_uuid_t uuid_image; > + qemu_uuid_t uuid_last_snap; > + qemu_uuid_t uuid_link; > + qemu_uuid_t uuid_parent;
As far as I remember, _t should be avoided because that postfix is
reserved by POSIX. Should we use qemu_uuid, or can we ignore POSIX
because the type name uses the qemu_ prefix?
> uint64_t unused2[7];
> } QEMU_PACKED VdiHeader;
>
> @@ -209,7 +180,7 @@ typedef struct {
> /* Change UUID from little endian (IPRT = VirtualBox format) to big endian
> * format (network byte order, standard, see RFC 4122) and vice versa.
> */
> -static void uuid_convert(uuid_t uuid)
> +static void uuid_convert(qemu_uuid_t uuid)
> {
> bswap32s((uint32_t *)&uuid[0]);
> bswap16s((uint16_t *)&uuid[4]);
> @@ -469,11 +440,11 @@ static int vdi_open(BlockDriverState *bs, QDict
> *options, int flags,
> (uint64_t)header.blocks_in_image * header.block_size);
> ret = -ENOTSUP;
> goto fail;
> - } else if (!uuid_is_null(header.uuid_link)) {
> + } else if (!qemu_uuid_is_null(header.uuid_link)) {
> error_setg(errp, "unsupported VDI image (non-NULL link UUID)");
> ret = -ENOTSUP;
> goto fail;
> - } else if (!uuid_is_null(header.uuid_parent)) {
> + } else if (!qemu_uuid_is_null(header.uuid_parent)) {
> error_setg(errp, "unsupported VDI image (non-NULL parent UUID)");
> ret = -ENOTSUP;
> goto fail;
> @@ -821,8 +792,8 @@ static int vdi_create(const char *filename, QemuOpts
> *opts, Error **errp)
> if (image_type == VDI_TYPE_STATIC) {
> header.blocks_allocated = blocks;
> }
> - uuid_generate(header.uuid_image);
> - uuid_generate(header.uuid_last_snap);
> + qemu_uuid_generate(header.uuid_image);
> + qemu_uuid_generate(header.uuid_last_snap);
> /* There is no need to set header.uuid_link or header.uuid_parent here.
> */
> #if defined(CONFIG_VDI_DEBUG)
> vdi_header_print(&header);
>
Either with qemu_uuid_t or with qemu_uuid:
Reviewed-by: Stefan Weil <[email protected]>
signature.asc
Description: OpenPGP digital signature
