Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 29 Jul 1999 23:52:32 -0400 (EDT)
From:      Bill Paul <wpaul@skynet.ctr.columbia.edu>
To:        dillon@apollo.backplane.com (Matthew Dillon)
Cc:        peter@netplex.com.au, crossd@cs.rpi.edu, current@freebsd.org
Subject:   Re: readdirplus client side fix (was Re: IRIX 6.5.4 NFS v3 TCP client + FreeBSD server = bewm)
Message-ID:  <199907300352.XAA17853@skynet.ctr.columbia.edu>
In-Reply-To: <199907300331.UAA81350@apollo.backplane.com> from "Matthew Dillon" at Jul 29, 99 08:31:38 pm

next in thread | previous in thread | raw e-mail | index | archive | help
Of all the gin joints in all the towns in all the world, Matthew Dillon 
had to walk into mine and say:
 
>     Look up a bit in the code.  If bigenough is not true, cnp does not 
>     get initialized.  This could lead to the bogus length -- or rather,
>     it would be the cnp that is bogus, not the 'len'.
> 
>     The question is how to fix it.  I think we can safely avoid doing the
>     cache_enter so try changing the 'if (doit)' to 'if (doit && bigenough)'.
>     I've included the patch below.
> 
>     I am not 100% sure about this.

Hm. Well, it cures the panic that I was experiencing quite nicely.
I'm going to commit this latest patch for now since it fixes both
the vnode locking problem and a crash condition, which are pretty
serious problems. If you come up with something different, I'll be happy 
to try it out.

Not a bad day's work. :)

-Bill

-- 
=============================================================================
-Bill Paul            (212) 854-6020 | System Manager, Master of Unix-Fu
Work:         wpaul@ctr.columbia.edu | Center for Telecommunications Research
Home:  wpaul@skynet.ctr.columbia.edu | Columbia University, New York City
=============================================================================
 "It is not I who am crazy; it is I who am mad!" - Ren Hoek, "Space Madness"
=============================================================================


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?199907300352.XAA17853>