On Wed, Jun 24, 2015 at 14:04:06 -0400, John Ferlan wrote:
On 06/24/2015 10:11 AM, Peter Krempa wrote:
> Don't lock the driver when registering event callbacks.
> ---
> src/test/test_driver.c | 12 ------------
> 1 file changed, 12 deletions(-)
>
> diff --git a/src/test/test_driver.c b/src/test/test_driver.c
> index 2ab0402..b08c1e5 100644
> --- a/src/test/test_driver.c
> +++ b/src/test/test_driver.c
> @@ -5915,11 +5915,9 @@ testConnectDomainEventRegister(virConnectPtr conn,
> testDriverPtr driver = conn->privateData;
> int ret = 0;
>
> - testDriverLock(driver);
> if (virDomainEventStateRegister(conn, driver->eventState,
> callback, opaque, freecb) < 0)
> ret = -1;
> - testDriverUnlock(driver);
>
> return ret;
> }
> @@ -5932,11 +5930,9 @@ testConnectDomainEventDeregister(virConnectPtr conn,
> testDriverPtr driver = conn->privateData;
> int ret = 0;
>
> - testDriverLock(driver);
> if (virDomainEventStateDeregister(conn, driver->eventState,
> callback) < 0)
> ret = -1;
> - testDriverUnlock(driver);
>
> return ret;
> }
> @@ -5953,12 +5949,10 @@ testConnectDomainEventRegisterAny(virConnectPtr conn,
> testDriverPtr driver = conn->privateData;
> int ret;
int ret = 0;
Well this certainly isn't material for this patch.
Peter