cvs commit: src/sys/kern kern_descrip.c

[Brian F. Feldman]

des at des.no (Dag-Erling =?iso-8859-1?q?Sm=F8rgrav?=) wrote:
> Dag-Erling Smorgrav <des at FreeBSD.org> writes:
> >   WITNESS won't let us hold two filedesc locks at the same time, so juggle
> >   fdp and newfdp around a bit.
> 
> The question though is, is there a real reason why we can't hold two
> filedesc locks at the same time, or is WITNESS being unnecessarily
> pedantic?

I don't see a reason "newfdp" needs to be locked, since it is not referenced 
by anything yet.  If "fdp" alone is locked, that is sufficient to ensure 
they will both be consistent.

-- 
Brian Fundakowski Feldman                           \'[ FreeBSD ]''''''''''\
  <> green at FreeBSD.org                               \  The Power to Serve! \
 Opinions expressed are my own.                       \,,,,,,,,,,,,,,,,,,,,,,\