Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/miscfs



On Thu, Jun 17, 2004 at 08:56:06AM +0900, YAMAMOTO Takashi wrote:
> > Module Name:        src
> > Committed By:       wrstuden
> > Date:               Wed Jun 16 17:59:53 UTC 2004
> > 
> > Modified Files:
> >     src/sys/miscfs/genfs: layer_vnops.c
> >     src/sys/miscfs/umapfs: umap_vnops.c
> > 
> > Log Message:
> > Make sure we actually locked the parent vnode before we clear
> > PDIRUNLOCK. The whole reason we have the flag is to note (rare)
> > cases where we are supposed to have the parent directory locked
> > but don't. Permits error handling code to know what to do with
> > the parrent vnode (vrele() vs vput()).
> 
> how can vn_lock(LK_EXCLUSIVE | LK_RETRY) fail in this case?

Somehow the lock is dying. Yes, for it to happen we have to have
wandered into the weeds, but this code methodology is to try and help us 
get somewhere a bit safer. i.e. not add more error states on top if we 
don't have to.

Take care,

Bill

Attachment: pgpGNsvp_3bvJ.pgp
Description: PGP signature



Home | Main Index | Thread Index | Old Index