Re: dir_lookup, locking and EINTR

2008-08-20 Thread Thomas Bushnell BSG
On Thu, 2008-08-21 at 00:39 +0100, Samuel Thibault wrote: > That's safer indeed. To make things simpler, the patch below just > systematically resets NEWPI and NEWPO to NULL as soon as they don't need > to be dereferenced, does it look ok? It does, but "NULL" is entirely unnecessary. I much pref

Re: dir_lookup, locking and EINTR

2008-08-20 Thread Samuel Thibault
Thomas Bushnell BSG, le Tue 19 Aug 2008 23:09:56 -0700, a écrit : > Your patch (which I see has been applied) seems to be clearly the Wrong > Thing in the case where NP == DNP. Aow right. > The right thing to do is to throw NEWPI away at the very end of the > function. That's safer indeed. To m

Re: dir_lookup, locking and EINTR

2008-08-19 Thread Thomas Bushnell BSG
On Fri, 2008-06-06 at 00:12 +0100, Samuel Thibault wrotekfs/dir-lookup.c:478 > > dir-lookup.c:478 is as follows: > > 469 if (! error) > 470 { > 471 if (flags & O_EXLOCK) > 472 error = fshelp_acquire_lock (&np->userlock, &newpi->po->lock_status, > 473

dir_lookup, locking and EINTR

2008-06-05 Thread Samuel Thibault
Hello, We are still facing some ext2fs hangs sometimes, and I got an interesting backtrace today: Thread 94 (thread 348.94): #0 0x010d753c in mach_msg_trap () from /lib/libc.so.0.3 #1 0x010d7cc9 in mach_msg () from /lib/libc.so.0.3 #2 0x010a3630 in cproc_block () at /build/mbanck/hurd-2007