Re: Re: Re: Re: freeBSD nullfs together with nfs and silly rename

2010-06-13 Thread Rick Macklem



On Sun, 13 Jun 2010, Kostik Belousov wrote:



My note was not an objection, only a note. Also, when committing, please
add a comment explaining what is going on.


Righto, and my response was just my opinion. I'm assuming Mikolaj is
looking at committing this?

rick

___
freebsd-stable@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-stable
To unsubscribe, send any mail to freebsd-stable-unsubscr...@freebsd.org


Re: Re: Re: freeBSD nullfs together with nfs and silly rename

2010-06-12 Thread Rick Macklem



On Sat, 12 Jun 2010, Kostik Belousov wrote:



Yes, I hoped that Mikolaj ends up with something similar :). Please note
that this is racy, since we cannot know why usecount is greater then 1.
This might cause the silly rename to kick in some time where it should
not, but the race is rare.


I'd say that having silly rename happen once in a while for unlink when
it doesn't have to happen is better than having the file deleted on the
server while it is still open on the client.

rick

___
freebsd-stable@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-stable
To unsubscribe, send any mail to freebsd-stable-unsubscr...@freebsd.org


Re: Re: Re: freeBSD nullfs together with nfs and silly rename

2010-06-12 Thread Kostik Belousov
On Sat, Jun 12, 2010 at 07:06:11PM -0400, Rick Macklem wrote:
 
 
 On Sat, 12 Jun 2010, Kostik Belousov wrote:
 
 
 Yes, I hoped that Mikolaj ends up with something similar :). Please note
 that this is racy, since we cannot know why usecount is greater then 1.
 This might cause the silly rename to kick in some time where it should
 not, but the race is rare.
 
 I'd say that having silly rename happen once in a while for unlink when
 it doesn't have to happen is better than having the file deleted on the
 server while it is still open on the client.

My note was not an objection, only a note. Also, when committing, please
add a comment explaining what is going on.


pgplm57gi6oDf.pgp
Description: PGP signature