Re: [PATCH] mds: fix setting/removing xattrs on root

2013-04-18 Thread Big Chiu
I didn't notice the bug. Guessing it was hidden because CephFS had been accessed by other daemons in my test environment. Thank you for the hint! The signed-off patches are resent, also including your fix. On Wed, Apr 17, 2013 at 4:06 AM, Gregory Farnum wrote: > On Mon, Apr 15, 2013 at 3:23 AM,

Re: [PATCH] mds: fix setting/removing xattrs on root

2013-04-16 Thread Gregory Farnum
On Mon, Apr 15, 2013 at 3:23 AM, Kuan Kai Chiu wrote: > MDS crashes while journaling dirty root inode in handle_client_setxattr > and handle_client_removexattr. We should use journal_dirty_inode to > safely log root inode here. > --- > src/mds/Server.cc |6 ++ > 1 file changed, 2 insertio

[PATCH] mds: fix setting/removing xattrs on root

2013-04-15 Thread Kuan Kai Chiu
MDS crashes while journaling dirty root inode in handle_client_setxattr and handle_client_removexattr. We should use journal_dirty_inode to safely log root inode here. --- src/mds/Server.cc |6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/mds/Server.cc b/src/mds/Ser

[PATCH] mds: fix setting/removing xattrs on root

2013-04-15 Thread Kuan Kai Chiu
MDS crashes while journaling dirty root inode in handle_client_setxattr and handle_client_removexattr. We should use journal_dirty_inode to safely log root inode here. Kuan Kai Chiu (1): mds: fix setting/removing xattrs on root src/mds/Server.cc |6 ++ 1 file changed, 2 insertions(+),

[PATCH] mds: fix setting/removing xattrs on root

2013-04-15 Thread Kuan Kai Chiu
MDS crashes while journaling dirty root inode in handle_client_setxattr and handle_client_removexattr. We should use journal_dirty_inode to safely log root inode here. Kuan Kai Chiu (1): mds: fix setting/removing xattrs on root src/mds/Server.cc |6 ++ 1 file changed, 2 insertions(+),