cvs commit: src/sys/kern kern_subr.c

Don Lewis truckman at FreeBSD.org
Tue Oct 12 15:17:08 PDT 2004


On 12 Oct, Alfred Perlstein wrote:
> I like this, my only concern is that there may be places that
> call this with locks held but with the kernel/user buffer wired
> so that it can't fault.
> 
> Are you sure this isn't the case?  (specifically for some sysctls)

There are a number of sysctl handlers where I added code to wire the
buffer so that there was no danger of sleeping while a mutex is held.
Allocating a kernel buffer and doing an extra copy would be ugly.

> * John Baldwin <jhb at FreeBSD.org> [041012 11:27] wrote:
>> jhb         2004-10-12 18:27:14 UTC
>> 
>>   FreeBSD src repository
>> 
>>   Modified files:
>>     sys/kern             kern_subr.c 
>>   Log:
>>   Add a WITNESS_WARN() to uiomove() to whine if locks are held when this
>>   function is called.
>>   
>>   MFC after:      1 month
>>   
>>   Revision  Changes    Path
>>   1.89      +2 -0      src/sys/kern/kern_subr.c
> 



More information about the cvs-src mailing list