Re: biodone panics

From: Alex Lyashkov <umka_at_sevcity.net>
Date: Thu, 29 Sep 2005 10:01:18 +0300
> 
> the call to wakeup may set in motion some events that cause the bio to
> be freed. By the time the mtx_unlock completes, "bp" could point to an
> invalid bio, and the "if (bp->bio_done != NULL)" is bogus. True?
> Shouldn't it be
> 
> > biodone(struct bio *bp)
> > {
> >         void (*done)(struct bio *);
> >
> >         mtx_lock(&bdonelock);
> >         bp->bio_flags |= BIO_DONE;
> >         done = bp->bio_done
> >         if (done == NULL)
> >                 wakeup(bp);
> >         mtx_unlock(&bdonelock);
> >         if (done != NULL)
> >                 bp->bio_done(bp);
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
posible 
if( done != NULL )
	done(bp);
?
> > }
> 
> Anyone agree?
Received on Thu Sep 29 2005 - 05:01:22 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:38:44 UTC