From 420beac4fcc9efd6f7d838ef7cc5693c58c98015 Mon Sep 17 00:00:00 2001 From: Bill Pemberton Date: Wed, 29 Jul 2009 17:00:10 -0400 Subject: [PATCH] Staging: hv: remove WaitEventClose() All WaitEventClose() close did was call kfree(), so get rid of it and replace it with a call to kfree() Signed-off-by: Bill Pemberton Cc: Hank Janssen Signed-off-by: Greg Kroah-Hartman --- drivers/staging/hv/Channel.c | 6 +++--- drivers/staging/hv/ChannelMgmt.c | 2 +- drivers/staging/hv/Connection.c | 4 ++-- drivers/staging/hv/NetVsc.c | 4 ++-- drivers/staging/hv/RndisFilter.c | 2 +- drivers/staging/hv/StorVsc.c | 4 ++-- drivers/staging/hv/include/osd.h | 1 - drivers/staging/hv/osd.c | 5 ----- 8 files changed, 11 insertions(+), 17 deletions(-) diff --git a/drivers/staging/hv/Channel.c b/drivers/staging/hv/Channel.c index 34c86628422..e6337418767 100644 --- a/drivers/staging/hv/Channel.c +++ b/drivers/staging/hv/Channel.c @@ -309,7 +309,7 @@ Cleanup: REMOVE_ENTRY_LIST(&openInfo->MsgListEntry); spin_unlock_irqrestore(&gVmbusConnection.channelmsg_lock, flags); - WaitEventClose(openInfo->WaitEvent); + kfree(openInfo->WaitEvent); kfree(openInfo); DPRINT_EXIT(VMBUS); @@ -596,7 +596,7 @@ Cleanup: REMOVE_ENTRY_LIST(&msgInfo->MsgListEntry); spin_unlock_irqrestore(&gVmbusConnection.channelmsg_lock, flags); - WaitEventClose(msgInfo->WaitEvent); + kfree(msgInfo->WaitEvent); kfree(msgInfo); DPRINT_EXIT(VMBUS); @@ -658,7 +658,7 @@ VmbusChannelTeardownGpadl( REMOVE_ENTRY_LIST(&info->MsgListEntry); spin_unlock_irqrestore(&gVmbusConnection.channelmsg_lock, flags); - WaitEventClose(info->WaitEvent); + kfree(info->WaitEvent); kfree(info); DPRINT_EXIT(VMBUS); diff --git a/drivers/staging/hv/ChannelMgmt.c b/drivers/staging/hv/ChannelMgmt.c index 2c4f4c8e5e7..7ef1f2ba19d 100644 --- a/drivers/staging/hv/ChannelMgmt.c +++ b/drivers/staging/hv/ChannelMgmt.c @@ -762,7 +762,7 @@ VmbusChannelRequestOffers( Cleanup: if (msgInfo) { - WaitEventClose(msgInfo->WaitEvent); + kfree(msgInfo->WaitEvent); kfree(msgInfo); } diff --git a/drivers/staging/hv/Connection.c b/drivers/staging/hv/Connection.c index 33e5628b88c..8d76bd45db8 100644 --- a/drivers/staging/hv/Connection.c +++ b/drivers/staging/hv/Connection.c @@ -155,7 +155,7 @@ VmbusConnect(void) } - WaitEventClose(msgInfo->WaitEvent); + kfree(msgInfo->WaitEvent); kfree(msgInfo); DPRINT_EXIT(VMBUS); @@ -183,7 +183,7 @@ Cleanup: if (msgInfo) { if (msgInfo->WaitEvent) - WaitEventClose(msgInfo->WaitEvent); + kfree(msgInfo->WaitEvent); kfree(msgInfo); } diff --git a/drivers/staging/hv/NetVsc.c b/drivers/staging/hv/NetVsc.c index 9445e36423b..dea54098e51 100644 --- a/drivers/staging/hv/NetVsc.c +++ b/drivers/staging/hv/NetVsc.c @@ -880,7 +880,7 @@ Cleanup: if (netDevice) { - WaitEventClose(netDevice->ChannelInitEvent); + kfree(netDevice->ChannelInitEvent); while (!IsListEmpty(&netDevice->ReceivePacketList)) { @@ -963,7 +963,7 @@ NetVscOnDeviceRemove( kfree(netvscPacket); } - WaitEventClose(netDevice->ChannelInitEvent); + kfree(netDevice->ChannelInitEvent); FreeNetDevice(netDevice); DPRINT_EXIT(NETVSC); diff --git a/drivers/staging/hv/RndisFilter.c b/drivers/staging/hv/RndisFilter.c index c6b9dfd2f2e..5b992dfdb0f 100644 --- a/drivers/staging/hv/RndisFilter.c +++ b/drivers/staging/hv/RndisFilter.c @@ -273,7 +273,7 @@ static inline void PutRndisRequest(RNDIS_DEVICE *Device, RNDIS_REQUEST *Request) REMOVE_ENTRY_LIST(&Request->ListEntry); spin_unlock_irqrestore(&Device->request_lock, flags); - WaitEventClose(Request->WaitEvent); + kfree(Request->WaitEvent); kfree(Request); } diff --git a/drivers/staging/hv/StorVsc.c b/drivers/staging/hv/StorVsc.c index 1c82e7923ff..ef8031fd385 100644 --- a/drivers/staging/hv/StorVsc.c +++ b/drivers/staging/hv/StorVsc.c @@ -516,7 +516,7 @@ static int StorVscChannelInit(struct hv_device *Device) Cleanup: if (request->WaitEvent) { - WaitEventClose(request->WaitEvent); + kfree(request->WaitEvent); request->WaitEvent = NULL; } @@ -678,7 +678,7 @@ StorVscOnHostReset( /* FIXME: Add a timeout */ WaitEventWait(request->WaitEvent); - WaitEventClose(request->WaitEvent); + kfree(request->WaitEvent); DPRINT_INFO(STORVSC, "host adapter reset completed"); /* diff --git a/drivers/staging/hv/include/osd.h b/drivers/staging/hv/include/osd.h index 4147aba5d31..e1f37879f52 100644 --- a/drivers/staging/hv/include/osd.h +++ b/drivers/staging/hv/include/osd.h @@ -134,7 +134,6 @@ extern int TimerStop(struct osd_timer *t); extern void TimerStart(struct osd_timer *t, u32 expirationInUs); extern struct osd_waitevent *WaitEventCreate(void); -extern void WaitEventClose(struct osd_waitevent *waitEvent); extern void WaitEventSet(struct osd_waitevent *waitEvent); extern int WaitEventWait(struct osd_waitevent *waitEvent); diff --git a/drivers/staging/hv/osd.c b/drivers/staging/hv/osd.c index 46cdf88cf9d..8ca56a4fce2 100644 --- a/drivers/staging/hv/osd.c +++ b/drivers/staging/hv/osd.c @@ -216,11 +216,6 @@ struct osd_waitevent *WaitEventCreate(void) return wait; } -void WaitEventClose(struct osd_waitevent *waitEvent) -{ - kfree(waitEvent); -} - void WaitEventSet(struct osd_waitevent *waitEvent) { waitEvent->condition = 1; -- 2.43.2