Re: cvs commit: src/sys/kern vfs_syscalls.c

[Joerg Sonnenberger <joerg@xxxxxxxxxxxxxxxxx>]

On Wed, Aug 17, 2005 at 04:46:09PM +0200, Simon 'corecode' Schubert wrote:
> >As soon as the "new" operand already exists it is not a pure namespace
> >operation anymore. What happens if two NFS clients have the directory
> >cached, the first issues "mv a b", the second "mv b a"? Are both 
> >clients
> >issuing remove operations with the new code?
> 
> Do the clients issue remove operations?  I was thinking they just send 
> "rename a to b" the server, which in turn does the job.

No, it doesn't. The check Matt added is in kern_rename, which decides
whether to call the rename VOP or the remove VOP.

> 
> NFS is not a good example anyways because it operates with 
> maybe-outdated data.

Well, one more reason why this checks have to be carefully placed.

Joerg