From c40ed871cc7f90c5ca2acca95a95c73bd3151bfd Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A1n=20Tomko?= Date: Fri, 3 Jun 2016 18:54:43 +0200 Subject: [PATCH] Remove PROPERTY_* constants They are no longer used. --- src/node_device/node_device_udev.c | 41 ++++++++++-------------------- src/node_device/node_device_udev.h | 3 --- 2 files changed, 13 insertions(+), 31 deletions(-) diff --git a/src/node_device/node_device_udev.c b/src/node_device/node_device_udev.c index d9d9887830..8b6cebd092 100644 --- a/src/node_device/node_device_udev.c +++ b/src/node_device/node_device_udev.c @@ -206,10 +206,10 @@ static int udevGetUint64SysfsAttr(struct udev_device *udev_device, if (str && virStrToLong_ull(str, NULL, 0, value) < 0) { VIR_ERROR(_("Failed to convert '%s' to unsigned long long"), str); - return PROPERTY_ERROR; + return -1; } - return str == NULL ? PROPERTY_MISSING : PROPERTY_FOUND; + return 0; } @@ -687,18 +687,12 @@ static int udevProcessDisk(struct udev_device *device, virNodeDevCapDataPtr data = &def->caps->data; int ret = 0; - if (udevGetUint64SysfsAttr(device, - "size", - &data->storage.num_blocks) == PROPERTY_ERROR) { + if (udevGetUint64SysfsAttr(device, "size", &data->storage.num_blocks) < 0) goto out; - } - if (udevGetUint64SysfsAttr(device, - "queue/logical_block_size", - &data->storage.logical_block_size) - == PROPERTY_ERROR) { + if (udevGetUint64SysfsAttr(device, "queue/logical_block_size", + &data->storage.logical_block_size) < 0) goto out; - } data->storage.size = data->storage.num_blocks * data->storage.logical_block_size; @@ -729,17 +723,13 @@ static int udevProcessRemoveableMedia(struct udev_device *device, &data->storage.media_label) < 0) goto out; - if (udevGetUint64SysfsAttr(device, - "size", - &data->storage.num_blocks) == PROPERTY_ERROR) { + if (udevGetUint64SysfsAttr(device, "size", + &data->storage.num_blocks) < 0) goto out; - } - if (udevGetUint64SysfsAttr(device, - "queue/logical_block_size", - &data->storage.logical_block_size) == PROPERTY_ERROR) { + if (udevGetUint64SysfsAttr(device, "queue/logical_block_size", + &data->storage.logical_block_size) < 0) goto out; - } /* XXX This calculation is wrong for the qemu virtual cdrom * which reports the size in 512 byte blocks, but the logical @@ -801,18 +791,13 @@ static int udevProcessSD(struct udev_device *device, virNodeDevCapDataPtr data = &def->caps->data; int ret = 0; - if (udevGetUint64SysfsAttr(device, - "size", - &data->storage.num_blocks) == PROPERTY_ERROR) { + if (udevGetUint64SysfsAttr(device, "size", + &data->storage.num_blocks) < 0) goto out; - } - if (udevGetUint64SysfsAttr(device, - "queue/logical_block_size", - &data->storage.logical_block_size) - == PROPERTY_ERROR) { + if (udevGetUint64SysfsAttr(device, "queue/logical_block_size", + &data->storage.logical_block_size) < 0) goto out; - } data->storage.size = data->storage.num_blocks * data->storage.logical_block_size; diff --git a/src/node_device/node_device_udev.h b/src/node_device/node_device_udev.h index 47eeb1f3f2..9a07ab77e4 100644 --- a/src/node_device/node_device_udev.h +++ b/src/node_device/node_device_udev.h @@ -29,6 +29,3 @@ typedef struct _udevPrivate udevPrivate; #define DRV_STATE_UDEV_MONITOR(ds) (((udevPrivate *)((ds)->privateData))->udev_monitor) #define DMI_DEVPATH "/sys/devices/virtual/dmi/id" #define DMI_DEVPATH_FALLBACK "/sys/class/dmi/id" -#define PROPERTY_FOUND 0 -#define PROPERTY_MISSING 1 -#define PROPERTY_ERROR -1 -- 2.39.5