Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

CVS commit: src

Module Name:    src
Committed By:   manu
Date:           Sat Jul 21 05:49:42 UTC 2012

Modified Files:
        src/lib/libperfuse: debug.c ops.c perfuse.c perfuse_if.h perfuse_priv.h
        src/usr.sbin/perfused: msg.c perfused.c

Log Message:
- Fix same vnodes associated with multiple cookies
The scheme used to retreive known nodes on lookup was flawed, as it only
used parent and name. This produced a different cookie for the same file
if it was renamed, when looking up ../ or when dealing with multiple files
associated with the same name through link(2).

We therefore abandon the use of node name and introduce hashed lists of
inodes. This causes a huge rewrite of reclaim code, which do not attempt
to keep parents allocated until all their children are reclaimed

- Fix race conditions in reclaim
There are a few situations where we issue multiple FUSE operations for
a PUFFS operation. On reclaim, we therefore have to wait for all FUSE
operation to complete, not just the current exchanges. We do this by
introducing node reference count with node_ref() and node_rele().

- Detect data loss caused by FAF
VOP_PUTPAGES causes FAF writes where the kernel does not check the
operation result. At least issue a warning on error.

- Enjoy FAF shortcut on setattr
No need to wait for the result if the kernel does not want it. There is
however an exception for setattr that touch the size, we need to wait
for completion because we have other operations queued for after the

- Fix fchmod() on write-open file
fchmod() on a node open with write privilege will send setattr with both mode 
and size set. This confuses some FUSE filesystem. Therefore we send two FUSE 
operations, one for mode, and one for size.

- Remove node TTL handling for netbsd-5 for simplicity sake. The code
still builds on netbsd-5 but does not have the node TTL feature anymore.
It works fine with kernel support on netbsd-6.

To generate a diff of this commit:
cvs rdiff -u -r1.11 -r1.12 src/lib/libperfuse/debug.c
cvs rdiff -u -r1.58 -r1.59 src/lib/libperfuse/ops.c
cvs rdiff -u -r1.28 -r1.29 src/lib/libperfuse/perfuse.c
cvs rdiff -u -r1.19 -r1.20 src/lib/libperfuse/perfuse_if.h
cvs rdiff -u -r1.30 -r1.31 src/lib/libperfuse/perfuse_priv.h
cvs rdiff -u -r1.18 -r1.19 src/lib/libperfuse/subr.c
cvs rdiff -u -r1.20 -r1.21 src/usr.sbin/perfused/msg.c
cvs rdiff -u -r1.23 -r1.24 src/usr.sbin/perfused/perfused.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Home | Main Index | Thread Index | Old Index