jimzah/openafs
1
0
Fork 0
mirror of https://git.openafs.org/openafs.git synced 2025-01-19 15:30:14 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Windows: Wait for all worker threads to exit

Browse Source 
The signalling mechanism for waking and shutting down worker threads
relies upon a per-queue event.  Therefore it is not guaranteed that
the worker thread that AFSShutdown*Thread() is attempting to wait
for is in fact the thread that will be woken and exit.  Modify the
code to loop waking threads until the one that is being waited for
does in fact exit.

Subsequent calls to AFSShutdown*Thread() will bypass the wait if
the thread has already exited.

Change-Id: I4555df062ac5a6161b5c55f4598d1bd34e144a2b
Reviewed-on: http://gerrit.openafs.org/8783
Reviewed-by: Rod Widdowson <rdw@steadingsoftware.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: Jeffrey Altman <jaltman@your-file-system.com>
This commit is contained in:
Jeffrey Altman 2012-12-19 09:31:06 -05:00
parent 8a4094e9ff
commit 1726aec2b2
1 changed files with 50 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

90
src/WINNT/afsrdr/kernel/lib/AFSWorker.cpp
View File

@ -320,7 +320,7 @@ AFSRemoveWorkerPool()
//
// Loop through the IO workers shutting them down in two stages.
// First, clear AFS_WORKER_PROCESS_REQUESTS so that workers
// stop processing requests. Second, call AFSShutdownWorkerThread()
// stop processing requests. Second, call AFSShutdownIOWorkerThread()
// to wake the workers and wait for them to exit.
//
@ -532,21 +532,23 @@ AFSShutdownVolumeWorker( IN AFSVolumeCB *VolumeCB)
NTSTATUS ntStatus = STATUS_SUCCESS;
AFSWorkQueueContext *pWorker = &VolumeCB->VolumeWorkerContext;
if( pWorker->WorkerThreadObject != NULL &&
BooleanFlagOn( pWorker->State, AFS_WORKER_INITIALIZED))
//
// Clear the 'keep processing' flag
//
ClearFlag( pWorker->State, AFS_WORKER_PROCESS_REQUESTS);
if( pWorker->WorkerThreadObject != NULL)
{
while ( BooleanFlagOn( pWorker->State, AFS_WORKER_INITIALIZED) )
{
//
// Clear the 'keep processing' flag
//
ClearFlag( pWorker->State, AFS_WORKER_PROCESS_REQUESTS);
ntStatus = KeWaitForSingleObject( pWorker->WorkerThreadObject,
Executive,
KernelMode,
FALSE,
NULL);
ntStatus = KeWaitForSingleObject( pWorker->WorkerThreadObject,
Executive,
KernelMode,
FALSE,
NULL);
}
ObDereferenceObject( pWorker->WorkerThreadObject);
@ -575,23 +577,26 @@ AFSShutdownWorkerThread( IN AFSWorkQueueContext *PoolContext)
NTSTATUS ntStatus = STATUS_SUCCESS;
AFSDeviceExt *pDeviceExt = (AFSDeviceExt *)AFSLibraryDeviceObject->DeviceExtension;
if( PoolContext->WorkerThreadObject != NULL &&
BooleanFlagOn( PoolContext->State, AFS_WORKER_INITIALIZED))
if( PoolContext->WorkerThreadObject != NULL)
{
//
// Wake up the thread if it is a sleep
//
while ( BooleanFlagOn( PoolContext->State, AFS_WORKER_INITIALIZED) )
{
KeSetEvent( &pDeviceExt->Specific.Library.WorkerQueueHasItems,
0,
FALSE);
//
// Wake up the thread if it is a sleep
//
ntStatus = KeWaitForSingleObject( PoolContext->WorkerThreadObject,
Executive,
KernelMode,
FALSE,
NULL);
KeSetEvent( &pDeviceExt->Specific.Library.WorkerQueueHasItems,
0,
FALSE);
ntStatus = KeWaitForSingleObject( PoolContext->WorkerThreadObject,
Executive,
KernelMode,
FALSE,
NULL);
}
ObDereferenceObject( PoolContext->WorkerThreadObject);
@ -620,23 +625,26 @@ AFSShutdownIOWorkerThread( IN AFSWorkQueueContext *PoolContext)
NTSTATUS ntStatus = STATUS_SUCCESS;
AFSDeviceExt *pDeviceExt = (AFSDeviceExt *)AFSLibraryDeviceObject->DeviceExtension;
if( PoolContext->WorkerThreadObject != NULL &&
BooleanFlagOn( PoolContext->State, AFS_WORKER_INITIALIZED))
if( PoolContext->WorkerThreadObject != NULL)
{
//
// Wake up the thread if it is a sleep
//
while ( BooleanFlagOn( PoolContext->State, AFS_WORKER_INITIALIZED) )
{
KeSetEvent( &pDeviceExt->Specific.Library.IOWorkerQueueHasItems,
0,
FALSE);
//
// Wake up the thread if it is a sleep
//
ntStatus = KeWaitForSingleObject( PoolContext->WorkerThreadObject,
Executive,
KernelMode,
FALSE,
NULL);
KeSetEvent( &pDeviceExt->Specific.Library.IOWorkerQueueHasItems,
0,
FALSE);
ntStatus = KeWaitForSingleObject( PoolContext->WorkerThreadObject,
Executive,
KernelMode,
FALSE,
NULL);
}
ObDereferenceObject( PoolContext->WorkerThreadObject);
@ -1647,6 +1655,8 @@ AFSPrimaryVolumeWorkerThread( IN PVOID Context)
KeCancelTimer( &Timer);
ClearFlag( pPoolContext->State, AFS_WORKER_INITIALIZED);
AFSDbgLogMsg( AFS_SUBSYSTEM_CLEANUP_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSPrimaryVolumeWorkerThread Exiting\n");