Re: [PATCH hurd 0/6] Add file record locking support

2015-01-20 Thread Svante Signell
On Tue, 2015-01-20 at 01:37 +0100, Samuel Thibault wrote: > Hello, > > Thanks for reviving this, it'll definitely be useful! > > Svante Signell, le Thu 08 Jan 2015 12:28:05 +0100, a écrit : > > 1) l_pid of struct flock is not set. According to the manpage (and > > posix) the PID of the process bl

Re: [PATCH hurd 0/6] Add file record locking support

2015-01-20 Thread Samuel Thibault
Svante Signell, le Tue 20 Jan 2015 11:19:21 +0100, a écrit : > On Tue, 2015-01-20 at 01:37 +0100, Samuel Thibault wrote: > > Hello, > > > > Thanks for reviving this, it'll definitely be useful! > > > > Svante Signell, le Thu 08 Jan 2015 12:28:05 +0100, a écrit : > > > 1) l_pid of struct flock is

Re: [PATCH glibc] Add file record locking support

2015-01-20 Thread Svante Signell
On Tue, 2015-01-20 at 01:06 +0100, Samuel Thibault wrote: > Hello, > > Svante Signell, le Thu 08 Jan 2015 19:54:12 +0100, a écrit : > > You made me confused, so all changes were not made. I think the malloc > > version was better for symmetry reasons. Changed anyway. > > Which symmetry? You mean

Re: [PATCH hurd 0/6] Add file record locking support

2015-01-20 Thread Samuel Thibault
Svante Signell, le Tue 20 Jan 2015 11:19:21 +0100, a écrit : > > I don't think tdbtorture forks with locks held: AIUI it just starts > > several processes and let them play at will with locks, without creating > > or terminating them. > > Looking at the code of tdbtorture.c it forks a lot of proce