[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: CVS commit: src/sys/dev/dkwedge
Andrew Doran wrote:
> On Sun, Jun 01, 2008 at 01:26:10PM +0100, Andrew Doran wrote:
>> On Sun, Jun 01, 2008 at 11:38:27AM +0000, Chris Gilbert wrote:
>>> Module Name: src
>>> Committed By: chris
>>> Date: Sun Jun 1 11:38:27 UTC 2008
>>> Modified Files:
>>> src/sys/dev/dkwedge: dk.c
>>> Log Message:
>>> Call buf_destroy when finished with an on-stack struct buf.
>>> Spotted by LOCKDEBUG, because the condvars were already initialised.
>> If the LWP is swapped out before the I/O completes, bad things will happen
>> there. I guess it should use getiobuf() instead - I'll take a look.
> Can someone try this easily?
It compiles and a LOCKDEBUG kernel boots with that change.
Note I'm not actually using dkwedges, but there's a check at wd_attach
if the disk should be dkwedged, which is what LOCKDEBUG originally caught.
Main Index |
Thread Index |