Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 12 Jul 2001 02:10:03 -0500
From:      Alfred Perlstein <bright@sneakerz.org>
To:        Seigo Tanimura <tanimura@r.dl.itc.u-tokyo.ac.jp>
Cc:        jake@FreeBSD.org, jhb@FreeBSD.org, current@FreeBSD.org
Subject:   Re: Lock of struct filedesc, file, pgrp, session and sigio
Message-ID:  <20010712021002.H2449@sneakerz.org>
In-Reply-To: <20010712014442.E2449@sneakerz.org>; from bright@sneakerz.org on Thu, Jul 12, 2001 at 01:44:44AM -0500
References:  <200106181004.f5IA4VD63112@rina.r.dl.itc.u-tokyo.ac.jp> <200107020812.f628CfK44241@rina.r.dl.itc.u-tokyo.ac.jp> <20010707164249.C88962@sneakerz.org> <20010709032044.B1894@sneakerz.org> <200107100845.f6A8jqt99404@rina.r.dl.itc.u-tokyo.ac.jp> <20010710035347.Q1894@sneakerz.org> <200107100903.f6A93et02367@rina.r.dl.itc.u-tokyo.ac.jp> <200107111137.f6BBbQt22812@rina.r.dl.itc.u-tokyo.ac.jp> <20010712014205.D2449@sneakerz.org> <20010712014442.E2449@sneakerz.org>

next in thread | previous in thread | raw e-mail | index | archive | help
* Alfred Perlstein <bright@sneakerz.org> [010712 01:45] wrote:
> * Alfred Perlstein <bright@sneakerz.org> [010712 01:42] wrote:
> > * Seigo Tanimura <tanimura@r.dl.itc.u-tokyo.ac.jp> [010711 19:08] wrote:
> > > 
> > > The patch and the results of build test are now on the web page.
> > > 
> > > The discussion of ktrace(2) problem does not cover the solution of
> > > BSD/OS, so it needs updating.
> > 
> > I'm just reviewing the file stuff, I have a number of updates mainly
> > in the areas of needing FFIND_HOLD/fdrop rather than just a simple
> > FFIND.
> > 
> > getvnode(p->p_fd, args->fd, &fp);
> > 
> > I'm quite sure should implicitly do a FHOLD operation on the returned
> > struct file, i don't see this, at least not in linux_getdents()
> > (compat/linux/linux_file.c).
> 
> Or at least an fhold needs to be done before relasing the lock
> on the struct file.

Furthermore, all the fo_* functions should not be calling FHOLD
themselves, the fhold needs to have been done by whomever calls
them to make sure that the 'fp' doesn't disapear out from under
the fo_ function.

-- 
-Alfred Perlstein [alfred@freebsd.org]
Ok, who wrote this damn function called '??'?
And why do my programs keep crashing in it?

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010712021002.H2449>