cvs commit: src/sys/compat/ndis kern_ndis.c ndis_var.h pe_var.h subr_ndis.c subr_ntoskrnl.c subr_pe.c src/sys/dev/if_ndis if_ndis.c

Don Lewis truckman at FreeBSD.org
Tue Jan 6 11:28:32 PST 2004


On  6 Jan, John Baldwin wrote:
> 
> On 06-Jan-2004 Bill Paul wrote:
>> wpaul       2004/01/05 23:09:26 PST
>> 
>>   FreeBSD src repository
>> 
>>   Modified files:
>>     sys/compat/ndis      kern_ndis.c ndis_var.h pe_var.h 
>>                          subr_ndis.c subr_ntoskrnl.c subr_pe.c 
>>     sys/dev/if_ndis      if_ndis.c 
>>   Log:
>>   - Change the implementation of KeInitializeSpinLock(). There is
>>     no complementary KeFreeSpinLock() function, so creating a new
>>     mutex on each call to KeInitializeSpinLock() leaks resources
>>     when a driver is unloaded. For now, KeInitializeSpinLock()
>>     returns a handle to the ntoskrnl interlock mutex.
> 
> You could perhaps use a pool mutex if it helps lessen contention.  Maybe
> something like:
> 
>         *lock = (kspin_lock)mtx_pool_alloc(&mtxpool_sleep);

If these really need to be spin locks, a spin lock pool would need to be
created with mtx_pool_create().




More information about the cvs-all mailing list