commit 5b4785a6e4dd45dbce359247b74e49935c08ac9c Author: Tomohiro Kusumi <kusumi.tomoh...@gmail.com> Date: Fri Dec 11 00:11:44 2015 +0900
sys/vfs/hammer: Fix wrong parent localization for PFS root The idea of PFS root inode having ip localization value of its parent directory's inode has been #if0'd after ea434b6f in 2008. (Since then parent_obj_localization field of ondisk inode seems to be always 0 because the field is never touched after it's first allocated, unless it's a symlink which uses symlink[24] part of union) Since the field is now deprecated, using this as a localization value of inode's parent is wrong when the inode is PFS root. It should use a proper localization value HAMMER_DEF_LOCALIZATION which is also 0. Note that HAMMER userspace doesn't allow creating a PFS in PFS unless the parent PFS is PFS0 (see hammer_cmd_pseudofs_create()), so it doesn't need to consider the parent inode's localization value other than the default localization value. If that's allowed, there seems to be no easy way to tell the localization value. Also remove parent_obj_localization field of ondisk inode since it's no longer used after this commit. Summary of changes: sys/vfs/hammer/hammer_disk.h | 5 ----- sys/vfs/hammer/hammer_inode.c | 12 ------------ sys/vfs/hammer/hammer_vnops.c | 2 +- 3 files changed, 1 insertion(+), 18 deletions(-) http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/5b4785a6e4dd45dbce359247b74e49935c08ac9c -- DragonFly BSD source repository