From 07a6b9aac43a6a9e3cc414b59951db357613d296 Mon Sep 17 00:00:00 2001 From: "Daniel P. Berrange" Date: Tue, 23 Apr 2013 11:15:48 +0100 Subject: [PATCH] Rename 'DeviceMonitor' to 'NodeDeviceDriver' The driver.h struct for node devices used an inconsistent naming scheme 'DeviceMonitor' instead of the more usual 'NodeDeviceDriver'. Fix this everywhere it has leaked out to. Signed-off-by: Daniel P. Berrange --- docs/hvsupport.pl | 4 +- src/conf/node_device_conf.h | 6 +-- src/datatypes.c | 4 +- src/datatypes.h | 4 +- src/driver.h | 10 ++-- src/esx/esx_device_monitor.c | 8 +-- src/hyperv/hyperv_device_monitor.c | 8 +-- src/libvirt.c | 76 ++++++++++++++-------------- src/libvirt_private.syms | 2 +- src/node_device/node_device_driver.c | 28 +++++----- src/node_device/node_device_driver.h | 4 +- src/node_device/node_device_hal.c | 30 +++++------ src/node_device/node_device_udev.c | 30 +++++------ src/remote/remote_driver.c | 24 ++++----- src/rpc/gendispatch.pl | 4 +- src/test/test_driver.c | 20 ++++---- tests/object-locking.ml | 4 +- 17 files changed, 133 insertions(+), 133 deletions(-) diff --git a/docs/hvsupport.pl b/docs/hvsupport.pl index baebfc49e1..707b512179 100755 --- a/docs/hvsupport.pl +++ b/docs/hvsupport.pl @@ -18,7 +18,7 @@ my %groupheaders = ( "virDriver" => "Hypervisor APIs", "virNetworkDriver" => "Virtual Network APIs", "virInterfaceDriver" => "Host Interface APIs", - "virDeviceMonitor" => "Host Device APIs", + "virNodeDeviceDriver" => "Host Device APIs", "virStorageDriver" => "Storage Pool APIs", "virSecretDriver" => "Secret APIs", "virNWFilterDriver" => "Network Filter APIs", @@ -182,7 +182,7 @@ open FILE, "<$drivertable" my %groups; my $ingrp; while (defined($line = )) { - if ($line =~ /struct _(vir\w*(?:Driver|Monitor))/) { + if ($line =~ /struct _(vir\w*Driver)/) { my $grp = $1; if ($grp ne "virStateDriver" && $grp ne "virStreamDriver") { diff --git a/src/conf/node_device_conf.h b/src/conf/node_device_conf.h index ca5ec726d5..1c5855c355 100644 --- a/src/conf/node_device_conf.h +++ b/src/conf/node_device_conf.h @@ -200,9 +200,9 @@ struct _virNodeDeviceObjList { virNodeDeviceObjPtr *objs; }; -typedef struct _virDeviceMonitorState virDeviceMonitorState; -typedef virDeviceMonitorState *virDeviceMonitorStatePtr; -struct _virDeviceMonitorState { +typedef struct _virNodeDeviceDriverState virNodeDeviceDriverState; +typedef virNodeDeviceDriverState *virNodeDeviceDriverStatePtr; +struct _virNodeDeviceDriverState { virMutex lock; virNodeDeviceObjList devs; /* currently-known devices */ diff --git a/src/datatypes.c b/src/datatypes.c index f8b6f494fa..f6c01cea98 100644 --- a/src/datatypes.c +++ b/src/datatypes.c @@ -149,8 +149,8 @@ virConnectDispose(void *obj) conn->interfaceDriver->connectClose(conn); if (conn->storageDriver) conn->storageDriver->connectClose(conn); - if (conn->deviceMonitor) - conn->deviceMonitor->connectClose(conn); + if (conn->nodeDeviceDriver) + conn->nodeDeviceDriver->connectClose(conn); if (conn->secretDriver) conn->secretDriver->connectClose(conn); if (conn->nwfilterDriver) diff --git a/src/datatypes.h b/src/datatypes.h index e5ecf18368..6026bb0ebb 100644 --- a/src/datatypes.h +++ b/src/datatypes.h @@ -128,7 +128,7 @@ struct _virConnect { virNetworkDriverPtr networkDriver; virInterfaceDriverPtr interfaceDriver; virStorageDriverPtr storageDriver; - virDeviceMonitorPtr deviceMonitor; + virNodeDeviceDriverPtr nodeDeviceDriver; virSecretDriverPtr secretDriver; virNWFilterDriverPtr nwfilterDriver; @@ -140,7 +140,7 @@ struct _virConnect { void * networkPrivateData; void * interfacePrivateData; void * storagePrivateData; - void * devMonPrivateData; + void * nodeDevicePrivateData; void * secretPrivateData; void * nwfilterPrivateData; diff --git a/src/driver.h b/src/driver.h index ee1fae5389..882cbc6a00 100644 --- a/src/driver.h +++ b/src/driver.h @@ -1757,17 +1757,17 @@ typedef int -typedef struct _virDeviceMonitor virDeviceMonitor; -typedef virDeviceMonitor *virDeviceMonitorPtr; +typedef struct _virNodeDeviceDriver virNodeDeviceDriver; +typedef virNodeDeviceDriver *virNodeDeviceDriverPtr; /** - * _virDeviceMonitor: + * _virNodeDeviceDriver: * * Structure associated with monitoring the devices * on a virtualized node. * */ -struct _virDeviceMonitor { +struct _virNodeDeviceDriver { const char * name; /* the name of the driver */ virDrvConnectOpen connectOpen; virDrvConnectClose connectClose; @@ -1980,7 +1980,7 @@ int virRegisterDriver(virDriverPtr); int virRegisterNetworkDriver(virNetworkDriverPtr); int virRegisterInterfaceDriver(virInterfaceDriverPtr); int virRegisterStorageDriver(virStorageDriverPtr); -int virRegisterDeviceMonitor(virDeviceMonitorPtr); +int virRegisterNodeDeviceDriver(virNodeDeviceDriverPtr); int virRegisterSecretDriver(virSecretDriverPtr); int virRegisterNWFilterDriver(virNWFilterDriverPtr); # ifdef WITH_LIBVIRTD diff --git a/src/esx/esx_device_monitor.c b/src/esx/esx_device_monitor.c index 826b171ae8..0812bdf5f0 100644 --- a/src/esx/esx_device_monitor.c +++ b/src/esx/esx_device_monitor.c @@ -50,7 +50,7 @@ esxDeviceOpen(virConnectPtr conn, return VIR_DRV_OPEN_DECLINED; } - conn->devMonPrivateData = conn->privateData; + conn->nodeDevicePrivateData = conn->privateData; return VIR_DRV_OPEN_SUCCESS; } @@ -60,14 +60,14 @@ esxDeviceOpen(virConnectPtr conn, static int esxDeviceClose(virConnectPtr conn) { - conn->devMonPrivateData = NULL; + conn->nodeDevicePrivateData = NULL; return 0; } -static virDeviceMonitor esxDeviceMonitor = { +static virNodeDeviceDriver esxNodeDeviceDriver = { .name = "ESX", .connectOpen = esxDeviceOpen, /* 0.7.6 */ .connectClose = esxDeviceClose, /* 0.7.6 */ @@ -78,5 +78,5 @@ static virDeviceMonitor esxDeviceMonitor = { int esxDeviceRegister(void) { - return virRegisterDeviceMonitor(&esxDeviceMonitor); + return virRegisterNodeDeviceDriver(&esxNodeDeviceDriver); } diff --git a/src/hyperv/hyperv_device_monitor.c b/src/hyperv/hyperv_device_monitor.c index cf3ba257ca..ace9e7184d 100644 --- a/src/hyperv/hyperv_device_monitor.c +++ b/src/hyperv/hyperv_device_monitor.c @@ -47,7 +47,7 @@ hypervDeviceOpen(virConnectPtr conn, return VIR_DRV_OPEN_DECLINED; } - conn->devMonPrivateData = conn->privateData; + conn->nodeDevicePrivateData = conn->privateData; return VIR_DRV_OPEN_SUCCESS; } @@ -57,14 +57,14 @@ hypervDeviceOpen(virConnectPtr conn, static int hypervDeviceClose(virConnectPtr conn) { - conn->devMonPrivateData = NULL; + conn->nodeDevicePrivateData = NULL; return 0; } -static virDeviceMonitor hypervDeviceMonitor = { +static virNodeDeviceDriver hypervNodeDeviceDriver = { "Hyper-V", .connectOpen = hypervDeviceOpen, /* 0.9.5 */ .connectClose = hypervDeviceClose, /* 0.9.5 */ @@ -75,5 +75,5 @@ static virDeviceMonitor hypervDeviceMonitor = { int hypervDeviceRegister(void) { - return virRegisterDeviceMonitor(&hypervDeviceMonitor); + return virRegisterNodeDeviceDriver(&hypervNodeDeviceDriver); } diff --git a/src/libvirt.c b/src/libvirt.c index 7818e7f7a6..b784bee24e 100644 --- a/src/libvirt.c +++ b/src/libvirt.c @@ -112,8 +112,8 @@ static virInterfaceDriverPtr virInterfaceDriverTab[MAX_DRIVERS]; static int virInterfaceDriverTabCount = 0; static virStorageDriverPtr virStorageDriverTab[MAX_DRIVERS]; static int virStorageDriverTabCount = 0; -static virDeviceMonitorPtr virDeviceMonitorTab[MAX_DRIVERS]; -static int virDeviceMonitorTabCount = 0; +static virNodeDeviceDriverPtr virNodeDeviceDriverTab[MAX_DRIVERS]; +static int virNodeDeviceDriverTabCount = 0; static virSecretDriverPtr virSecretDriverTab[MAX_DRIVERS]; static int virSecretDriverTabCount = 0; static virNWFilterDriverPtr virNWFilterDriverTab[MAX_DRIVERS]; @@ -672,7 +672,7 @@ virRegisterStorageDriver(virStorageDriverPtr driver) } /** - * virRegisterDeviceMonitor: + * virRegisterNodeDeviceDriver: * @driver: pointer to a device monitor block * * Register a device monitor @@ -680,11 +680,11 @@ virRegisterStorageDriver(virStorageDriverPtr driver) * Returns the driver priority or -1 in case of error. */ int -virRegisterDeviceMonitor(virDeviceMonitorPtr driver) +virRegisterNodeDeviceDriver(virNodeDeviceDriverPtr driver) { virCheckNonNullArgReturn(driver, -1); - if (virDeviceMonitorTabCount >= MAX_DRIVERS) { + if (virNodeDeviceDriverTabCount >= MAX_DRIVERS) { virLibConnError(VIR_ERR_INTERNAL_ERROR, _("Too many drivers, cannot register %s"), driver->name); @@ -692,10 +692,10 @@ virRegisterDeviceMonitor(virDeviceMonitorPtr driver) } VIR_DEBUG("registering %s as device driver %d", - driver->name, virDeviceMonitorTabCount); + driver->name, virNodeDeviceDriverTabCount); - virDeviceMonitorTab[virDeviceMonitorTabCount] = driver; - return virDeviceMonitorTabCount++; + virNodeDeviceDriverTab[virNodeDeviceDriverTabCount] = driver; + return virNodeDeviceDriverTabCount++; } /** @@ -1282,16 +1282,16 @@ do_open(const char *name, } /* Node driver (optional) */ - for (i = 0; i < virDeviceMonitorTabCount; i++) { - res = virDeviceMonitorTab[i]->connectOpen(ret, auth, flags); + for (i = 0; i < virNodeDeviceDriverTabCount; i++) { + res = virNodeDeviceDriverTab[i]->connectOpen(ret, auth, flags); VIR_DEBUG("node driver %d %s returned %s", - i, virDeviceMonitorTab[i]->name, + i, virNodeDeviceDriverTab[i]->name, res == VIR_DRV_OPEN_SUCCESS ? "SUCCESS" : (res == VIR_DRV_OPEN_DECLINED ? "DECLINED" : (res == VIR_DRV_OPEN_ERROR ? "ERROR" : "unknown status"))); if (res == VIR_DRV_OPEN_SUCCESS) { - ret->deviceMonitor = virDeviceMonitorTab[i]; + ret->nodeDeviceDriver = virNodeDeviceDriverTab[i]; break; } else if (res == VIR_DRV_OPEN_ERROR) { break; @@ -14220,9 +14220,9 @@ virNodeNumOfDevices(virConnectPtr conn, const char *cap, unsigned int flags) return -1; } - if (conn->deviceMonitor && conn->deviceMonitor->nodeNumOfDevices) { + if (conn->nodeDeviceDriver && conn->nodeDeviceDriver->nodeNumOfDevices) { int ret; - ret = conn->deviceMonitor->nodeNumOfDevices(conn, cap, flags); + ret = conn->nodeDeviceDriver->nodeNumOfDevices(conn, cap, flags); if (ret < 0) goto error; return ret; @@ -14291,10 +14291,10 @@ virConnectListAllNodeDevices(virConnectPtr conn, return -1; } - if (conn->deviceMonitor && - conn->deviceMonitor->connectListAllNodeDevices) { + if (conn->nodeDeviceDriver && + conn->nodeDeviceDriver->connectListAllNodeDevices) { int ret; - ret = conn->deviceMonitor->connectListAllNodeDevices(conn, devices, flags); + ret = conn->nodeDeviceDriver->connectListAllNodeDevices(conn, devices, flags); if (ret < 0) goto error; return ret; @@ -14343,9 +14343,9 @@ virNodeListDevices(virConnectPtr conn, virCheckNonNullArgGoto(names, error); virCheckNonNegativeArgGoto(maxnames, error); - if (conn->deviceMonitor && conn->deviceMonitor->nodeListDevices) { + if (conn->nodeDeviceDriver && conn->nodeDeviceDriver->nodeListDevices) { int ret; - ret = conn->deviceMonitor->nodeListDevices(conn, cap, names, maxnames, flags); + ret = conn->nodeDeviceDriver->nodeListDevices(conn, cap, names, maxnames, flags); if (ret < 0) goto error; return ret; @@ -14382,9 +14382,9 @@ virNodeDevicePtr virNodeDeviceLookupByName(virConnectPtr conn, const char *name) virCheckNonNullArgGoto(name, error); - if (conn->deviceMonitor && conn->deviceMonitor->nodeDeviceLookupByName) { + if (conn->nodeDeviceDriver && conn->nodeDeviceDriver->nodeDeviceLookupByName) { virNodeDevicePtr ret; - ret = conn->deviceMonitor->nodeDeviceLookupByName(conn, name); + ret = conn->nodeDeviceDriver->nodeDeviceLookupByName(conn, name); if (!ret) goto error; return ret; @@ -14427,10 +14427,10 @@ virNodeDeviceLookupSCSIHostByWWN(virConnectPtr conn, virCheckNonNullArgGoto(wwnn, error); virCheckNonNullArgGoto(wwpn, error); - if (conn->deviceMonitor && - conn->deviceMonitor->nodeDeviceLookupSCSIHostByWWN) { + if (conn->nodeDeviceDriver && + conn->nodeDeviceDriver->nodeDeviceLookupSCSIHostByWWN) { virNodeDevicePtr ret; - ret = conn->deviceMonitor->nodeDeviceLookupSCSIHostByWWN(conn, wwnn, + ret = conn->nodeDeviceDriver->nodeDeviceLookupSCSIHostByWWN(conn, wwnn, wwpn, flags); if (!ret) goto error; @@ -14466,9 +14466,9 @@ char *virNodeDeviceGetXMLDesc(virNodeDevicePtr dev, unsigned int flags) return NULL; } - if (dev->conn->deviceMonitor && dev->conn->deviceMonitor->nodeDeviceGetXMLDesc) { + if (dev->conn->nodeDeviceDriver && dev->conn->nodeDeviceDriver->nodeDeviceGetXMLDesc) { char *ret; - ret = dev->conn->deviceMonitor->nodeDeviceGetXMLDesc(dev, flags); + ret = dev->conn->nodeDeviceDriver->nodeDeviceGetXMLDesc(dev, flags); if (!ret) goto error; return ret; @@ -14525,8 +14525,8 @@ const char *virNodeDeviceGetParent(virNodeDevicePtr dev) } if (!dev->parent) { - if (dev->conn->deviceMonitor && dev->conn->deviceMonitor->nodeDeviceGetParent) { - dev->parent = dev->conn->deviceMonitor->nodeDeviceGetParent(dev); + if (dev->conn->nodeDeviceDriver && dev->conn->nodeDeviceDriver->nodeDeviceGetParent) { + dev->parent = dev->conn->nodeDeviceDriver->nodeDeviceGetParent(dev); } else { virLibConnError(VIR_ERR_NO_SUPPORT, __FUNCTION__); virDispatchError(dev->conn); @@ -14556,9 +14556,9 @@ int virNodeDeviceNumOfCaps(virNodeDevicePtr dev) return -1; } - if (dev->conn->deviceMonitor && dev->conn->deviceMonitor->nodeDeviceNumOfCaps) { + if (dev->conn->nodeDeviceDriver && dev->conn->nodeDeviceDriver->nodeDeviceNumOfCaps) { int ret; - ret = dev->conn->deviceMonitor->nodeDeviceNumOfCaps(dev); + ret = dev->conn->nodeDeviceDriver->nodeDeviceNumOfCaps(dev); if (ret < 0) goto error; return ret; @@ -14599,9 +14599,9 @@ int virNodeDeviceListCaps(virNodeDevicePtr dev, virCheckNonNullArgGoto(names, error); virCheckNonNegativeArgGoto(maxnames, error); - if (dev->conn->deviceMonitor && dev->conn->deviceMonitor->nodeDeviceListCaps) { + if (dev->conn->nodeDeviceDriver && dev->conn->nodeDeviceDriver->nodeDeviceListCaps) { int ret; - ret = dev->conn->deviceMonitor->nodeDeviceListCaps(dev, names, maxnames); + ret = dev->conn->nodeDeviceDriver->nodeDeviceListCaps(dev, names, maxnames); if (ret < 0) goto error; return ret; @@ -14854,9 +14854,9 @@ virNodeDeviceCreateXML(virConnectPtr conn, virCheckNonNullArgGoto(xmlDesc, error); - if (conn->deviceMonitor && - conn->deviceMonitor->nodeDeviceCreateXML) { - virNodeDevicePtr dev = conn->deviceMonitor->nodeDeviceCreateXML(conn, xmlDesc, flags); + if (conn->nodeDeviceDriver && + conn->nodeDeviceDriver->nodeDeviceCreateXML) { + virNodeDevicePtr dev = conn->nodeDeviceDriver->nodeDeviceCreateXML(conn, xmlDesc, flags); if (dev == NULL) goto error; return dev; @@ -14897,9 +14897,9 @@ virNodeDeviceDestroy(virNodeDevicePtr dev) goto error; } - if (dev->conn->deviceMonitor && - dev->conn->deviceMonitor->nodeDeviceDestroy) { - int retval = dev->conn->deviceMonitor->nodeDeviceDestroy(dev); + if (dev->conn->nodeDeviceDriver && + dev->conn->nodeDeviceDriver->nodeDeviceDestroy) { + int retval = dev->conn->nodeDeviceDriver->nodeDeviceDestroy(dev); if (retval < 0) { goto error; } diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index c4ae681302..b3f8521727 100644 --- a/src/libvirt_private.syms +++ b/src/libvirt_private.syms @@ -713,10 +713,10 @@ virDomainMigratePrepare2; virDomainMigratePrepare3; virDomainMigratePrepareTunnel; virDomainMigratePrepareTunnel3; -virRegisterDeviceMonitor; virRegisterDriver; virRegisterInterfaceDriver; virRegisterNetworkDriver; +virRegisterNodeDeviceDriver; virRegisterNWFilterDriver; virRegisterSecretDriver; virRegisterStorageDriver; diff --git a/src/node_device/node_device_driver.c b/src/node_device/node_device_driver.c index 6be7a25476..c9bd68b65c 100644 --- a/src/node_device/node_device_driver.c +++ b/src/node_device/node_device_driver.c @@ -112,11 +112,11 @@ static int update_driver_name(virNodeDeviceObjPtr dev ATTRIBUTE_UNUSED) #endif -void nodeDeviceLock(virDeviceMonitorStatePtr driver) +void nodeDeviceLock(virNodeDeviceDriverStatePtr driver) { virMutexLock(&driver->lock); } -void nodeDeviceUnlock(virDeviceMonitorStatePtr driver) +void nodeDeviceUnlock(virNodeDeviceDriverStatePtr driver) { virMutexUnlock(&driver->lock); } @@ -126,7 +126,7 @@ nodeNumOfDevices(virConnectPtr conn, const char *cap, unsigned int flags) { - virDeviceMonitorStatePtr driver = conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = conn->nodeDevicePrivateData; int ndevs = 0; unsigned int i; @@ -151,7 +151,7 @@ nodeListDevices(virConnectPtr conn, char **const names, int maxnames, unsigned int flags) { - virDeviceMonitorStatePtr driver = conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = conn->nodeDevicePrivateData; int ndevs = 0; unsigned int i; @@ -187,7 +187,7 @@ nodeListAllNodeDevices(virConnectPtr conn, virNodeDevicePtr **devices, unsigned int flags) { - virDeviceMonitorStatePtr driver = conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = conn->nodeDevicePrivateData; int ret = -1; virCheckFlags(VIR_CONNECT_LIST_NODE_DEVICES_FILTERS_CAP, -1); @@ -201,7 +201,7 @@ nodeListAllNodeDevices(virConnectPtr conn, virNodeDevicePtr nodeDeviceLookupByName(virConnectPtr conn, const char *name) { - virDeviceMonitorStatePtr driver = conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = conn->nodeDevicePrivateData; virNodeDeviceObjPtr obj; virNodeDevicePtr ret = NULL; @@ -230,7 +230,7 @@ nodeDeviceLookupSCSIHostByWWN(virConnectPtr conn, unsigned int flags) { unsigned int i; - virDeviceMonitorStatePtr driver = conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = conn->nodeDevicePrivateData; virNodeDeviceObjListPtr devs = &driver->devs; virNodeDevCapsDefPtr cap = NULL; virNodeDeviceObjPtr obj = NULL; @@ -274,7 +274,7 @@ char * nodeDeviceGetXMLDesc(virNodeDevicePtr dev, unsigned int flags) { - virDeviceMonitorStatePtr driver = dev->conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = dev->conn->nodeDevicePrivateData; virNodeDeviceObjPtr obj; char *ret = NULL; @@ -306,7 +306,7 @@ cleanup: char * nodeDeviceGetParent(virNodeDevicePtr dev) { - virDeviceMonitorStatePtr driver = dev->conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = dev->conn->nodeDevicePrivateData; virNodeDeviceObjPtr obj; char *ret = NULL; @@ -340,7 +340,7 @@ cleanup: int nodeDeviceNumOfCaps(virNodeDevicePtr dev) { - virDeviceMonitorStatePtr driver = dev->conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = dev->conn->nodeDevicePrivateData; virNodeDeviceObjPtr obj; virNodeDevCapsDefPtr caps; int ncaps = 0; @@ -371,7 +371,7 @@ cleanup: int nodeDeviceListCaps(virNodeDevicePtr dev, char **const names, int maxnames) { - virDeviceMonitorStatePtr driver = dev->conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = dev->conn->nodeDevicePrivateData; virNodeDeviceObjPtr obj; virNodeDevCapsDefPtr caps; int ncaps = 0; @@ -442,7 +442,7 @@ get_time(time_t *t) static virNodeDevicePtr find_new_device(virConnectPtr conn, const char *wwnn, const char *wwpn) { - virDeviceMonitorStatePtr driver = conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = conn->nodeDevicePrivateData; virNodeDevicePtr dev = NULL; time_t start = 0, now = 0; @@ -481,7 +481,7 @@ nodeDeviceCreateXML(virConnectPtr conn, const char *xmlDesc, unsigned int flags) { - virDeviceMonitorStatePtr driver = conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = conn->nodeDevicePrivateData; virNodeDeviceDefPtr def = NULL; char *wwnn = NULL, *wwpn = NULL; int parent_host = -1; @@ -537,7 +537,7 @@ int nodeDeviceDestroy(virNodeDevicePtr dev) { int ret = -1; - virDeviceMonitorStatePtr driver = dev->conn->devMonPrivateData; + virNodeDeviceDriverStatePtr driver = dev->conn->nodeDevicePrivateData; virNodeDeviceObjPtr obj = NULL; char *parent_name = NULL, *wwnn = NULL, *wwpn = NULL; int parent_host = -1; diff --git a/src/node_device/node_device_driver.h b/src/node_device/node_device_driver.h index ea819177e1..b6c6f182d3 100644 --- a/src/node_device/node_device_driver.h +++ b/src/node_device/node_device_driver.h @@ -46,8 +46,8 @@ int halNodeRegister(void); int udevNodeRegister(void); # endif -void nodeDeviceLock(virDeviceMonitorStatePtr driver); -void nodeDeviceUnlock(virDeviceMonitorStatePtr driver); +void nodeDeviceLock(virNodeDeviceDriverStatePtr driver); +void nodeDeviceUnlock(virNodeDeviceDriverStatePtr driver); int nodedevRegister(void); diff --git a/src/node_device/node_device_hal.c b/src/node_device/node_device_hal.c index 16c59d9cb4..682b66acb1 100644 --- a/src/node_device/node_device_hal.c +++ b/src/node_device/node_device_hal.c @@ -46,10 +46,10 @@ * Host device enumeration (HAL implementation) */ -static virDeviceMonitorStatePtr driverState; +static virNodeDeviceDriverStatePtr driverState; #define CONN_DRV_STATE(conn) \ - ((virDeviceMonitorStatePtr)((conn)->devMonPrivateData)) + ((virNodeDeviceDriverStatePtr)((conn)->nodeDevicePrivateData)) #define DRV_STATE_HAL_CTX(ds) ((LibHalContext *)((ds)->privateData)) #define CONN_HAL_CTX(conn) DRV_STATE_HAL_CTX(CONN_DRV_STATE(conn)) @@ -590,9 +590,9 @@ static void device_prop_modified(LibHalContext *ctx ATTRIBUTE_UNUSED, } -static int halDeviceMonitorStartup(bool privileged ATTRIBUTE_UNUSED, - virStateInhibitCallback callback ATTRIBUTE_UNUSED, - void *opaque ATTRIBUTE_UNUSED) +static int halNodeDeviceDriverStartup(bool privileged ATTRIBUTE_UNUSED, + virStateInhibitCallback callback ATTRIBUTE_UNUSED, + void *opaque ATTRIBUTE_UNUSED) { LibHalContext *hal_ctx = NULL; char **udi = NULL; @@ -691,7 +691,7 @@ static int halDeviceMonitorStartup(bool privileged ATTRIBUTE_UNUSED, } -static int halDeviceMonitorShutdown(void) +static int halNodeDeviceDriverShutdown(void) { if (driverState) { nodeDeviceLock(driverState); @@ -708,7 +708,7 @@ static int halDeviceMonitorShutdown(void) } -static int halDeviceMonitorReload(void) +static int halNodeDeviceDriverReload(void) { DBusError err; char **udi = NULL; @@ -749,20 +749,20 @@ static virDrvOpenStatus halNodeDrvOpen(virConnectPtr conn, if (driverState == NULL) return VIR_DRV_OPEN_DECLINED; - conn->devMonPrivateData = driverState; + conn->nodeDevicePrivateData = driverState; return VIR_DRV_OPEN_SUCCESS; } static int halNodeDrvClose(virConnectPtr conn ATTRIBUTE_UNUSED) { - conn->devMonPrivateData = NULL; + conn->nodeDevicePrivateData = NULL; return 0; } -static virDeviceMonitor halDeviceMonitor = { - .name = "halDeviceMonitor", +static virNodeDeviceDriver halNodeDeviceDriver = { + .name = "halNodeDeviceDriver", .connectOpen = halNodeDrvOpen, /* 0.5.0 */ .connectClose = halNodeDrvClose, /* 0.5.0 */ .nodeNumOfDevices = nodeNumOfDevices, /* 0.5.0 */ @@ -781,14 +781,14 @@ static virDeviceMonitor halDeviceMonitor = { static virStateDriver halStateDriver = { .name = "HAL", - .stateInitialize = halDeviceMonitorStartup, /* 0.5.0 */ - .stateCleanup = halDeviceMonitorShutdown, /* 0.5.0 */ - .stateReload = halDeviceMonitorReload, /* 0.5.0 */ + .stateInitialize = halNodeDeviceDriverStartup, /* 0.5.0 */ + .stateCleanup = halNodeDeviceDriverShutdown, /* 0.5.0 */ + .stateReload = halNodeDeviceDriverReload, /* 0.5.0 */ }; int halNodeRegister(void) { - if (virRegisterDeviceMonitor(&halDeviceMonitor) < 0) + if (virRegisterNodeDeviceDriver(&halNodeDeviceDriver) < 0) return -1; return virRegisterStateDriver(&halStateDriver); } diff --git a/src/node_device/node_device_udev.c b/src/node_device/node_device_udev.c index 353cac5b2f..5a9c6e3200 100644 --- a/src/node_device/node_device_udev.c +++ b/src/node_device/node_device_udev.c @@ -50,7 +50,7 @@ struct _udevPrivate { int watch; }; -static virDeviceMonitorStatePtr driverState = NULL; +static virNodeDeviceDriverStatePtr driverState = NULL; static int udevStrToLong_ull(char const *s, char **end_ptr, @@ -1437,7 +1437,7 @@ out: } -static int udevDeviceMonitorShutdown(void) +static int udevNodeDeviceDriverShutdown(void) { int ret = 0; @@ -1649,9 +1649,9 @@ out: return ret; } -static int udevDeviceMonitorStartup(bool privileged ATTRIBUTE_UNUSED, - virStateInhibitCallback callback ATTRIBUTE_UNUSED, - void *opaque ATTRIBUTE_UNUSED) +static int udevNodeDeviceDriverStartup(bool privileged ATTRIBUTE_UNUSED, + virStateInhibitCallback callback ATTRIBUTE_UNUSED, + void *opaque ATTRIBUTE_UNUSED) { udevPrivate *priv = NULL; struct udev *udev = NULL; @@ -1758,13 +1758,13 @@ out_unlock: out: if (ret == -1) { - udevDeviceMonitorShutdown(); + udevNodeDeviceDriverShutdown(); } return ret; } -static int udevDeviceMonitorReload(void) +static int udevNodeDeviceDriverReload(void) { return 0; } @@ -1780,19 +1780,19 @@ static virDrvOpenStatus udevNodeDrvOpen(virConnectPtr conn, return VIR_DRV_OPEN_DECLINED; } - conn->devMonPrivateData = driverState; + conn->nodeDevicePrivateData = driverState; return VIR_DRV_OPEN_SUCCESS; } static int udevNodeDrvClose(virConnectPtr conn) { - conn->devMonPrivateData = NULL; + conn->nodeDevicePrivateData = NULL; return 0; } -static virDeviceMonitor udevDeviceMonitor = { - .name = "udevDeviceMonitor", +static virNodeDeviceDriver udevNodeDeviceDriver = { + .name = "udevNodeDeviceDriver", .connectOpen = udevNodeDrvOpen, /* 0.7.3 */ .connectClose = udevNodeDrvClose, /* 0.7.3 */ .nodeNumOfDevices = nodeNumOfDevices, /* 0.7.3 */ @@ -1810,16 +1810,16 @@ static virDeviceMonitor udevDeviceMonitor = { static virStateDriver udevStateDriver = { .name = "udev", - .stateInitialize = udevDeviceMonitorStartup, /* 0.7.3 */ - .stateCleanup = udevDeviceMonitorShutdown, /* 0.7.3 */ - .stateReload = udevDeviceMonitorReload, /* 0.7.3 */ + .stateInitialize = udevNodeDeviceDriverStartup, /* 0.7.3 */ + .stateCleanup = udevNodeDeviceDriverShutdown, /* 0.7.3 */ + .stateReload = udevNodeDeviceDriverReload, /* 0.7.3 */ }; int udevNodeRegister(void) { VIR_DEBUG("Registering udev node device backend"); - if (virRegisterDeviceMonitor(&udevDeviceMonitor) < 0) { + if (virRegisterNodeDeviceDriver(&udevNodeDeviceDriver) < 0) { return -1; } diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c index 4aec1508f5..83dc0fefdb 100644 --- a/src/remote/remote_driver.c +++ b/src/remote/remote_driver.c @@ -3367,16 +3367,16 @@ done: /*----------------------------------------------------------------------*/ static virDrvOpenStatus ATTRIBUTE_NONNULL(1) -remoteDevMonOpen(virConnectPtr conn, virConnectAuthPtr auth, - unsigned int flags) +remoteNodeDeviceOpen(virConnectPtr conn, virConnectAuthPtr auth, + unsigned int flags) { - return remoteGenericOpen(conn, auth, flags, &conn->devMonPrivateData); + return remoteGenericOpen(conn, auth, flags, &conn->nodeDevicePrivateData); } static int -remoteDevMonClose(virConnectPtr conn) +remoteNodeDeviceClose(virConnectPtr conn) { - return remoteGenericClose(conn, &conn->devMonPrivateData); + return remoteGenericClose(conn, &conn->nodeDevicePrivateData); } static int @@ -3385,7 +3385,7 @@ remoteNodeDeviceDettach(virNodeDevicePtr dev) int rv = -1; remote_node_device_dettach_args args; /* This method is unusual in that it uses the HV driver, not the devMon driver - * hence its use of privateData, instead of devMonPrivateData */ + * hence its use of privateData, instead of nodeDevicePrivateData */ struct private_data *priv = dev->conn->privateData; remoteDriverLock(priv); @@ -3410,7 +3410,7 @@ remoteNodeDeviceReAttach(virNodeDevicePtr dev) int rv = -1; remote_node_device_re_attach_args args; /* This method is unusual in that it uses the HV driver, not the devMon driver - * hence its use of privateData, instead of devMonPrivateData */ + * hence its use of privateData, instead of nodeDevicePrivateData */ struct private_data *priv = dev->conn->privateData; remoteDriverLock(priv); @@ -3435,7 +3435,7 @@ remoteNodeDeviceReset(virNodeDevicePtr dev) int rv = -1; remote_node_device_reset_args args; /* This method is unusual in that it uses the HV driver, not the devMon driver - * hence its use of privateData, instead of devMonPrivateData */ + * hence its use of privateData, instead of nodeDevicePrivateData */ struct private_data *priv = dev->conn->privateData; remoteDriverLock(priv); @@ -6414,10 +6414,10 @@ static virSecretDriver secret_driver = { .secretUndefine = remoteSecretUndefine /* 0.7.1 */ }; -static virDeviceMonitor dev_monitor = { +static virNodeDeviceDriver node_device_driver = { .name = "remote", - .connectOpen = remoteDevMonOpen, /* 0.5.0 */ - .connectClose = remoteDevMonClose, /* 0.5.0 */ + .connectOpen = remoteNodeDeviceOpen, /* 0.5.0 */ + .connectClose = remoteNodeDeviceClose, /* 0.5.0 */ .nodeNumOfDevices = remoteNodeNumOfDevices, /* 0.5.0 */ .nodeListDevices = remoteNodeListDevices, /* 0.5.0 */ .connectListAllNodeDevices = remoteConnectListAllNodeDevices, /* 0.10.2 */ @@ -6469,7 +6469,7 @@ remoteRegister(void) if (virRegisterNetworkDriver(&network_driver) == -1) return -1; if (virRegisterInterfaceDriver(&interface_driver) == -1) return -1; if (virRegisterStorageDriver(&storage_driver) == -1) return -1; - if (virRegisterDeviceMonitor(&dev_monitor) == -1) return -1; + if (virRegisterNodeDeviceDriver(&node_device_driver) == -1) return -1; if (virRegisterSecretDriver(&secret_driver) == -1) return -1; if (virRegisterNWFilterDriver(&nwfilter_driver) == -1) return -1; #ifdef WITH_LIBVIRTD diff --git a/src/rpc/gendispatch.pl b/src/rpc/gendispatch.pl index 431380753d..61c6606d0f 100755 --- a/src/rpc/gendispatch.pl +++ b/src/rpc/gendispatch.pl @@ -1072,7 +1072,7 @@ elsif ($mode eq "client") { !($argtype =~ m/^remote_node_device_lookup_by_name_/) and !($argtype =~ m/^remote_node_device_create_xml_/)) { $has_node_device = 1; - $priv_name = "devMonPrivateData"; + $priv_name = "nodeDevicePrivateData"; } foreach my $args_member (@{$call->{args_members}}) { @@ -1309,7 +1309,7 @@ elsif ($mode eq "client") { my $type_name = name_to_TypeName($name); if ($name eq "node_device") { - $priv_name = "devMonPrivateData"; + $priv_name = "nodeDevicePrivateData"; } elsif ($name =~ m/^storage_/) { $priv_name = "storagePrivateData"; } elsif (!($name =~ m/^domain/)) { diff --git a/src/test/test_driver.c b/src/test/test_driver.c index 5272f068a3..ebcb160b87 100644 --- a/src/test/test_driver.c +++ b/src/test/test_driver.c @@ -5245,21 +5245,21 @@ cleanup: /* Node device implementations */ -static virDrvOpenStatus testDevMonOpen(virConnectPtr conn, - virConnectAuthPtr auth ATTRIBUTE_UNUSED, - unsigned int flags) +static virDrvOpenStatus testNodeDeviceOpen(virConnectPtr conn, + virConnectAuthPtr auth ATTRIBUTE_UNUSED, + unsigned int flags) { virCheckFlags(VIR_CONNECT_RO, VIR_DRV_OPEN_ERROR); if (STRNEQ(conn->driver->name, "Test")) return VIR_DRV_OPEN_DECLINED; - conn->devMonPrivateData = conn->privateData; + conn->nodeDevicePrivateData = conn->privateData; return VIR_DRV_OPEN_SUCCESS; } -static int testDevMonClose(virConnectPtr conn) { - conn->devMonPrivateData = NULL; +static int testNodeDeviceClose(virConnectPtr conn) { + conn->nodeDevicePrivateData = NULL; return 0; } @@ -5952,10 +5952,10 @@ static virStorageDriver testStorageDriver = { .storagePoolIsPersistent = testStoragePoolIsPersistent, /* 0.7.3 */ }; -static virDeviceMonitor testDevMonitor = { +static virNodeDeviceDriver testNodeDeviceDriver = { .name = "Test", - .connectOpen = testDevMonOpen, /* 0.6.0 */ - .connectClose = testDevMonClose, /* 0.6.0 */ + .connectOpen = testNodeDeviceOpen, /* 0.6.0 */ + .connectClose = testNodeDeviceClose, /* 0.6.0 */ .nodeNumOfDevices = testNodeNumOfDevices, /* 0.7.2 */ .nodeListDevices = testNodeListDevices, /* 0.7.2 */ @@ -5997,7 +5997,7 @@ testRegister(void) return -1; if (virRegisterStorageDriver(&testStorageDriver) < 0) return -1; - if (virRegisterDeviceMonitor(&testDevMonitor) < 0) + if (virRegisterNodeDeviceDriver(&testNodeDeviceDriver) < 0) return -1; if (virRegisterSecretDriver(&testSecretDriver) < 0) return -1; diff --git a/tests/object-locking.ml b/tests/object-locking.ml index 88485fcd27..009b8f8f45 100644 --- a/tests/object-locking.ml +++ b/tests/object-locking.ml @@ -47,7 +47,7 @@ let driverTables = [ "virDriver"; "virNetworkDriver"; "virStorageDriver"; - "virDeviceMonitor"; + "virNodeDeviceDriver"; (* "virStateDriver"; Disable for now, since shutdown/startup have weird locking rules *) ] @@ -160,7 +160,7 @@ let lockableDrivers = [ "uml_driver"; "virStorageDriverStatePtr"; "network_driver"; - "virDeviceMonitorState"; + "virNodeDeviceState"; "one_driver_t"; ]