static int xattr_unlink(struct inode *dir, struct dentry *dentry)
 {
        int error;
+
        BUG_ON(!mutex_is_locked(&dir->i_mutex));
 
        mutex_lock_nested(&dentry->d_inode->i_mutex, I_MUTEX_CHILD);
 static int xattr_rmdir(struct inode *dir, struct dentry *dentry)
 {
        int error;
+
        BUG_ON(!mutex_is_locked(&dir->i_mutex));
 
        mutex_lock_nested(&dentry->d_inode->i_mutex, I_MUTEX_CHILD);
 {
        struct dentry *privroot = REISERFS_SB(sb)->priv_root;
        struct dentry *xaroot;
+
        if (!privroot->d_inode)
                return ERR_PTR(-ENODATA);
 
                xaroot = ERR_PTR(-ENODATA);
        else if (!xaroot->d_inode) {
                int err = -ENODATA;
+
                if (xattr_may_create(flags))
                        err = xattr_mkdir(privroot->d_inode, xaroot, 0700);
                if (err) {
        xadir = lookup_one_len(namebuf, xaroot, strlen(namebuf));
        if (!IS_ERR(xadir) && !xadir->d_inode) {
                int err = -ENODATA;
+
                if (xattr_may_create(flags))
                        err = xattr_mkdir(xaroot->d_inode, xadir, 0700);
                if (err) {
 {
        struct reiserfs_dentry_buf *dbuf = buf;
        struct dentry *dentry;
+
        WARN_ON_ONCE(!mutex_is_locked(&dbuf->xadir->d_inode->i_mutex));
 
        if (dbuf->count == ARRAY_SIZE(dbuf->dentries))
 cleanup_dentry_buf(struct reiserfs_dentry_buf *buf)
 {
        int i;
+
        for (i = 0; i < buf->count; i++)
                if (buf->dentries[i])
                        dput(buf->dentries[i]);
                int blocks = JOURNAL_PER_BALANCE_CNT * 2 + 2 +
                             4 * REISERFS_QUOTA_TRANS_BLOCKS(inode->i_sb);
                struct reiserfs_transaction_handle th;
+
                reiserfs_write_lock(inode->i_sb);
                err = journal_begin(&th, inode->i_sb, blocks);
                reiserfs_write_unlock(inode->i_sb);
                if (!err) {
                        int jerror;
+
                        mutex_lock_nested(&dir->d_parent->d_inode->i_mutex,
                                          I_MUTEX_XATTR);
                        err = action(dir, data);
 int reiserfs_delete_xattrs(struct inode *inode)
 {
        int err = reiserfs_for_each_xattr(inode, delete_one_xattr, NULL);
+
        if (err)
                reiserfs_warning(inode->i_sb, "jdm-20004",
                                 "Couldn't delete all xattrs (%d)\n", err);
 int reiserfs_chown_xattrs(struct inode *inode, struct iattr *attrs)
 {
        int err = reiserfs_for_each_xattr(inode, chown_one_xattr, attrs);
+
        if (err)
                reiserfs_warning(inode->i_sb, "jdm-20007",
                                 "Couldn't chown all xattrs (%d)\n", err);
 static void update_ctime(struct inode *inode)
 {
        struct timespec now = current_fs_time(inode->i_sb);
+
        if (inode_unhashed(inode) || !inode->i_nlink ||
            timespec_equal(&inode->i_ctime, &now))
                return;
                size_t chunk;
                size_t skip = 0;
                size_t page_offset = (file_pos & (PAGE_CACHE_SIZE - 1));
+
                if (buffer_size - buffer_pos > PAGE_CACHE_SIZE)
                        chunk = PAGE_CACHE_SIZE;
                else
 
                if (file_pos == 0) {
                        struct reiserfs_xattr_header *rxh;
+
                        skip = file_pos = sizeof(struct reiserfs_xattr_header);
                        if (chunk + skip > PAGE_CACHE_SIZE)
                                chunk = PAGE_CACHE_SIZE - skip;
                size_t chunk;
                char *data;
                size_t skip = 0;
+
                if (isize - file_pos > PAGE_CACHE_SIZE)
                        chunk = PAGE_CACHE_SIZE;
                else
 int reiserfs_removexattr(struct dentry *dentry, const char *name)
 {
        const struct xattr_handler *handler;
+
        handler = find_xattr_handler_prefix(dentry->d_sb->s_xattr, name);
 
        if (!handler || get_inode_sd_version(dentry->d_inode) == STAT_DATA_V1)
 {
        struct listxattr_buf *b = (struct listxattr_buf *)buf;
        size_t size;
+
        if (name[0] != '.' ||
            (namelen != 1 && (name[1] != '.' || namelen != 2))) {
                const struct xattr_handler *handler;
+
                handler = find_xattr_handler_prefix(b->dentry->d_sb->s_xattr,
                                                    name);
                if (!handler)   /* Unsupported xattr name */
 {
        int err;
        struct inode *inode = dentry->d_parent->d_inode;
+
        WARN_ON_ONCE(!mutex_is_locked(&inode->i_mutex));
 
        err = xattr_mkdir(inode, dentry, 0700);
                mutex_lock(&privroot->d_inode->i_mutex);
                if (!REISERFS_SB(s)->xattr_root) {
                        struct dentry *dentry;
+
                        dentry = lookup_one_len(XAROOT_NAME, privroot,
                                                strlen(XAROOT_NAME));
                        if (!IS_ERR(dentry))