qemu-block
[Top][All Lists]
Advanced

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

[Qemu-block] [PATCH v2 0/8] UUID clean ups for 2.8


From: Fam Zheng
Subject: [Qemu-block] [PATCH v2 0/8] UUID clean ups for 2.8
Date: Mon, 8 Aug 2016 14:09:20 +0800

v2: Fix the endianness wrongness. [Jeff]
    Add tests/test-uuid.c. [Denial]
    Add r-b of Stefan Weil to the VDI patch.

The facts how we use libuuid now are not particularly pleasant.

- VHDX driver depends on uuid, but is unconditionally checked in iotests 109.
  If it is not built, the test would fail, leaving no hint about that. In fact
  this even makes the feature implementation a bit ugly because it compromises
  the write protection on block 0 with image probing, silently.

- A few other libuuid users, in block/ and crypto/, use it as an optional
  dependency, and compromises certain functionalities if it is not available.

- On the other hand, there are already a few uuid function fallbacks in those
  callers.

So instead of making libuuid a hard requirement, let's simply move the
fallbacks to util and implement the random version of uuid generation, and be
done with it.

Fam

Fam Zheng (8):
  util: Add UUID API
  tests: Add uuid tests
  vhdx: Use QEMU UUID API
  vdi: Use QEMU UUID API
  vpc: Use QEMU UUID API
  crypto: Switch to QEMU UUID API
  tests: No longer dependent on CONFIG_UUID
  configure: Remove detection code for UUID

 arch_init.c               |  19 --------
 block/Makefile.objs       |   2 +-
 block/iscsi.c             |   2 +-
 block/vdi.c               |  49 +++++----------------
 block/vhdx-endian.c       |   3 --
 block/vhdx.c              |   7 ++-
 block/vpc.c               |   8 +---
 configure                 |  70 +++--------------------------
 crypto/block-luks.c       |  26 +++--------
 hw/smbios/smbios.c        |   1 +
 include/qemu/uuid.h       |  40 +++++++++++++++++
 include/sysemu/sysemu.h   |   4 --
 qmp.c                     |   1 +
 stubs/uuid.c              |   2 +-
 tests/Makefile.include    |   2 +
 tests/test-crypto-block.c |   2 +-
 tests/test-uuid.c         | 110 ++++++++++++++++++++++++++++++++++++++++++++++
 util/Makefile.objs        |   1 +
 util/uuid.c               |  68 ++++++++++++++++++++++++++++
 vl.c                      |   1 +
 20 files changed, 257 insertions(+), 161 deletions(-)
 create mode 100644 include/qemu/uuid.h
 create mode 100644 tests/test-uuid.c
 create mode 100644 util/uuid.c

-- 
2.7.4




reply via email to

[Prev in Thread] Current Thread [Next in Thread]