the bitrig pedro_tmpfs branch has been created by pedro.

it is 0 commits behind master, and 7 commits ahead.

commit c302f60a317d486e1deea7af97454406e3a19f3f
diff: https://github.com/bitrig/bitrig/commit/c302f60
author: Pedro Martelletto <[email protected]>
date: Fri Jan 30 10:25:18 2015 +0100

When loading a tmpfs snap, always cleanup upon failure.

M       sys/tmpfs/tmpfs_snapshot.c

commit 015b1274da712a655052d7a3cef71b34595a0cc3
diff: https://github.com/bitrig/bitrig/commit/015b127
author: Pedro Martelletto <[email protected]>
date: Fri Jan 30 10:18:44 2015 +0100

Ensure that tmpfs_snap_node_setsize() only happens once for a file.

M       sys/tmpfs/tmpfs_snapshot.c

commit 676635dd800b11d2081575371d2c9b82d7d4a3b1
diff: https://github.com/bitrig/bitrig/commit/676635d
author: Pedro Martelletto <[email protected]>
date: Fri Jan 30 10:10:30 2015 +0100

Don't call tmpfs_free_node() from tmpfs_snap_load_{hdr,file}().

As long as tmpfs_snap_load_cleanup() is called in the error path, the
node will be disposed of accordingly. Pointed out by natano@.

M       sys/tmpfs/tmpfs_snapshot.c

commit 542fc7c3b53704c5c6584a90ba92ad43aa7a7231
diff: https://github.com/bitrig/bitrig/commit/542fc7c
author: Pedro Martelletto <[email protected]>
date: Wed Jan 28 11:39:07 2015 +0100

Correctly set the attributes of the root node when loading tmpfs
snapshots.

The permissions were left completely zero'd out, as noticed by Markus
Friedl <[email protected]>. Set them according to the root node entry in
the tmpfs snapshot image.

Bump copyright and add a comment while here.

M       sys/tmpfs/tmpfs_snapshot.c
M       sys/tmpfs/tmpfs_subr.c

commit c195b6f1205f9d0e978265ab7573444eef351596
diff: https://github.com/bitrig/bitrig/commit/c195b6f
author: pedro martelletto <[email protected]>
date: Sat Jan 24 17:02:16 2015 +0100

Simplify error handling.

- Let tmpfs_snap_load_hdr() do tmpfs_free_node().
- Get rid of unnecessary goto's.
- Fix a leak in tmpfs_snap_attach_node().

M       sys/tmpfs/tmpfs_snapshot.c

commit c9a9a3f8ba8f0a337bc2f8b254fd42d3e4ad8d62
diff: https://github.com/bitrig/bitrig/commit/c9a9a3f
author: pedro martelletto <[email protected]>
date: Sat Jan 24 16:56:20 2015 +0100

Handle error conditions gracefully in tmpfs_alloc_node().

Replace a few occurences of KASSERT() with checks followed by
appropriate error handling.

M       sys/tmpfs/tmpfs_mem.c
M       sys/tmpfs/tmpfs_snapshot.c
M       sys/tmpfs/tmpfs_subr.c

commit d29fcea11e7fe639e668565bd7e7bb1393a02151
diff: https://github.com/bitrig/bitrig/commit/d29fcea
author: pedro martelletto <[email protected]>
date: Sat Jan 24 16:40:50 2015 +0100

Initialise td_node in tmpfs_alloc_dirent().

If the allocation of a new node in tmpfs fails before td_node is set,
tmpfs_free_dirent() will be called to dispose of the corresponding
directory entry, and tmpfs_free_dirent() expects td_node to be NULL.
Since these entries are retrieved from a pool without PR_ZERO, td_node
needs to be explicitly set.

M       sys/tmpfs/tmpfs_subr.c

Reply via email to