Re: 2.4 iget5_locked port attempt to 2.4

Jan Harkes
2003-02-21T19:59:04Z

On Fri, Feb 21, 2003 at 10:03:41PM +0300, Oleg Drokin wrote:
> Ok, here is my simple attempt. I just took a patch from early 2.5
> days by jaharkes@cs.cmu.edu ;)

Nice to see that it is being considered for a backport to 2.4, that
would allow me to get rid of the lock around the call to iget4.

Why didn't you take the final version that was sent to Linus? It was
against 2.5.14, but should be pretty close for 2.4.x. You can find
it at http://delft.aura.cs.cmu.edu/icreate/, both broken up in small
steps, and as one big patch.

As far as I know, the only change/improvement that went into 2.5 at a
later time was the ilookup code.

> Coda changes are not tested, but look correct.

Those Coda changes are not correct as we really need to use iget4 (or
in the new code, iget5_locked). That patch looks like it won't even
compile, coda_inocmp is simply removed while it is still used by the two
calls to iget4 that you didn't replace with iget5_locked. Let alone
adding the inode initializer. I also don't know why you are adding an
unused local variable to coda_replace_fid.

Jan