diff mbox series

[mtd-utils,v2,RESEND,035/102] ubifs-utils: Adapt journal.c in libubifs

Message ID 20241111083711.1916125-36-chengzhihao1@huawei.com
State Accepted
Delegated to: David Oberhollenzer
Headers show
Series Add fsck.ubifs support | expand

Commit Message

Zhihao Cheng Nov. 11, 2024, 8:37 a.m. UTC
Adapt journal.c in libubifs, compared with linux kernel implementations:
 1. Remove all ubifs_jnl_XXX functions. Only keep the basic space
    reservation code, fsck will add new functions for journaling
    operations without using linux in-memory inode/dentry.

Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com>
---
 ubifs-utils/libubifs/journal.c | 1497 +---------------------------------------
 1 file changed, 4 insertions(+), 1493 deletions(-)
diff mbox series

Patch

diff --git a/ubifs-utils/libubifs/journal.c b/ubifs-utils/libubifs/journal.c
index 4590d616..37dc3f0e 100644
--- a/ubifs-utils/libubifs/journal.c
+++ b/ubifs-utils/libubifs/journal.c
@@ -46,7 +46,11 @@ 
  * all the nodes.
  */
 
+#include "bitops.h"
 #include "ubifs.h"
+#include "defs.h"
+#include "debug.h"
+#include "misc.h"
 
 /**
  * zero_ino_node_unused - zero out unused fields of an on-flash inode node.
@@ -68,16 +72,6 @@  static inline void zero_dent_node_unused(struct ubifs_dent_node *dent)
 	dent->padding1 = 0;
 }
 
-/**
- * zero_trun_node_unused - zero out unused fields of an on-flash truncation
- *                         node.
- * @trun: the truncation node to zero out
- */
-static inline void zero_trun_node_unused(struct ubifs_trun_node *trun)
-{
-	memset(trun->padding, 0, 12);
-}
-
 static void ubifs_add_auth_dirt(struct ubifs_info *c, int lnum)
 {
 	if (ubifs_authenticated(c))
@@ -436,70 +430,6 @@  static int get_dent_type(int mode)
 	return 0;
 }
 
-/**
- * pack_inode - pack an inode node.
- * @c: UBIFS file-system description object
- * @ino: buffer in which to pack inode node
- * @inode: inode to pack
- * @last: indicates the last node of the group
- */
-static void pack_inode(struct ubifs_info *c, struct ubifs_ino_node *ino,
-		       const struct inode *inode, int last)
-{
-	int data_len = 0, last_reference = !inode->i_nlink;
-	struct ubifs_inode *ui = ubifs_inode(inode);
-
-	ino->ch.node_type = UBIFS_INO_NODE;
-	ino_key_init_flash(c, &ino->key, inode->i_ino);
-	ino->creat_sqnum = cpu_to_le64(ui->creat_sqnum);
-	ino->atime_sec  = cpu_to_le64(inode_get_atime_sec(inode));
-	ino->atime_nsec = cpu_to_le32(inode_get_atime_nsec(inode));
-	ino->ctime_sec  = cpu_to_le64(inode_get_ctime_sec(inode));
-	ino->ctime_nsec = cpu_to_le32(inode_get_ctime_nsec(inode));
-	ino->mtime_sec  = cpu_to_le64(inode_get_mtime_sec(inode));
-	ino->mtime_nsec = cpu_to_le32(inode_get_mtime_nsec(inode));
-	ino->uid   = cpu_to_le32(i_uid_read(inode));
-	ino->gid   = cpu_to_le32(i_gid_read(inode));
-	ino->mode  = cpu_to_le32(inode->i_mode);
-	ino->flags = cpu_to_le32(ui->flags);
-	ino->size  = cpu_to_le64(ui->ui_size);
-	ino->nlink = cpu_to_le32(inode->i_nlink);
-	ino->compr_type  = cpu_to_le16(ui->compr_type);
-	ino->data_len    = cpu_to_le32(ui->data_len);
-	ino->xattr_cnt   = cpu_to_le32(ui->xattr_cnt);
-	ino->xattr_size  = cpu_to_le32(ui->xattr_size);
-	ino->xattr_names = cpu_to_le32(ui->xattr_names);
-	zero_ino_node_unused(ino);
-
-	/*
-	 * Drop the attached data if this is a deletion inode, the data is not
-	 * needed anymore.
-	 */
-	if (!last_reference) {
-		memcpy(ino->data, ui->data, ui->data_len);
-		data_len = ui->data_len;
-	}
-
-	ubifs_prep_grp_node(c, ino, UBIFS_INO_NODE_SZ + data_len, last);
-}
-
-/**
- * mark_inode_clean - mark UBIFS inode as clean.
- * @c: UBIFS file-system description object
- * @ui: UBIFS inode to mark as clean
- *
- * This helper function marks UBIFS inode @ui as clean by cleaning the
- * @ui->dirty flag and releasing its budget. Note, VFS may still treat the
- * inode as dirty and try to write it back, but 'ubifs_write_inode()' would
- * just do nothing.
- */
-static void mark_inode_clean(struct ubifs_info *c, struct ubifs_inode *ui)
-{
-	if (ui->dirty)
-		ubifs_release_dirty_inode_budget(c, ui);
-	ui->dirty = 0;
-}
-
 static void set_dent_cookie(struct ubifs_info *c, struct ubifs_dent_node *dent)
 {
 	if (c->double_hash)
@@ -507,1422 +437,3 @@  static void set_dent_cookie(struct ubifs_info *c, struct ubifs_dent_node *dent)
 	else
 		dent->cookie = 0;
 }
-
-/**
- * ubifs_jnl_update - update inode.
- * @c: UBIFS file-system description object
- * @dir: parent inode or host inode in case of extended attributes
- * @nm: directory entry name
- * @inode: inode to update
- * @deletion: indicates a directory entry deletion i.e unlink or rmdir
- * @xent: non-zero if the directory entry is an extended attribute entry
- * @in_orphan: indicates whether the @inode is in orphan list
- *
- * This function updates an inode by writing a directory entry (or extended
- * attribute entry), the inode itself, and the parent directory inode (or the
- * host inode) to the journal.
- *
- * The function writes the host inode @dir last, which is important in case of
- * extended attributes. Indeed, then we guarantee that if the host inode gets
- * synchronized (with 'fsync()'), and the write-buffer it sits in gets flushed,
- * the extended attribute inode gets flushed too. And this is exactly what the
- * user expects - synchronizing the host inode synchronizes its extended
- * attributes. Similarly, this guarantees that if @dir is synchronized, its
- * directory entry corresponding to @nm gets synchronized too.
- *
- * If the inode (@inode) or the parent directory (@dir) are synchronous, this
- * function synchronizes the write-buffer.
- *
- * This function marks the @dir and @inode inodes as clean and returns zero on
- * success. In case of failure, a negative error code is returned.
- */
-int ubifs_jnl_update(struct ubifs_info *c, const struct inode *dir,
-		     const struct fscrypt_name *nm, const struct inode *inode,
-		     int deletion, int xent, int in_orphan)
-{
-	int err, dlen, ilen, len, lnum, ino_offs, dent_offs, orphan_added = 0;
-	int aligned_dlen, aligned_ilen, sync = IS_DIRSYNC(dir);
-	int last_reference = !!(deletion && inode->i_nlink == 0);
-	struct ubifs_inode *ui = ubifs_inode(inode);
-	struct ubifs_inode *host_ui = ubifs_inode(dir);
-	struct ubifs_dent_node *dent;
-	struct ubifs_ino_node *ino;
-	union ubifs_key dent_key, ino_key;
-	u8 hash_dent[UBIFS_HASH_ARR_SZ];
-	u8 hash_ino[UBIFS_HASH_ARR_SZ];
-	u8 hash_ino_host[UBIFS_HASH_ARR_SZ];
-
-	ubifs_assert(c, mutex_is_locked(&host_ui->ui_mutex));
-
-	dlen = UBIFS_DENT_NODE_SZ + fname_len(nm) + 1;
-	ilen = UBIFS_INO_NODE_SZ;
-
-	/*
-	 * If the last reference to the inode is being deleted, then there is
-	 * no need to attach and write inode data, it is being deleted anyway.
-	 * And if the inode is being deleted, no need to synchronize
-	 * write-buffer even if the inode is synchronous.
-	 */
-	if (!last_reference) {
-		ilen += ui->data_len;
-		sync |= IS_SYNC(inode);
-	}
-
-	aligned_dlen = ALIGN(dlen, 8);
-	aligned_ilen = ALIGN(ilen, 8);
-
-	len = aligned_dlen + aligned_ilen + UBIFS_INO_NODE_SZ;
-	/* Make sure to also account for extended attributes */
-	if (ubifs_authenticated(c))
-		len += ALIGN(host_ui->data_len, 8) + ubifs_auth_node_sz(c);
-	else
-		len += host_ui->data_len;
-
-	dent = kzalloc(len, GFP_NOFS);
-	if (!dent)
-		return -ENOMEM;
-
-	/* Make reservation before allocating sequence numbers */
-	err = make_reservation(c, BASEHD, len);
-	if (err)
-		goto out_free;
-
-	if (!xent) {
-		dent->ch.node_type = UBIFS_DENT_NODE;
-		if (fname_name(nm) == NULL)
-			dent_key_init_hash(c, &dent_key, dir->i_ino, nm->hash);
-		else
-			dent_key_init(c, &dent_key, dir->i_ino, nm);
-	} else {
-		dent->ch.node_type = UBIFS_XENT_NODE;
-		xent_key_init(c, &dent_key, dir->i_ino, nm);
-	}
-
-	key_write(c, &dent_key, dent->key);
-	dent->inum = deletion ? 0 : cpu_to_le64(inode->i_ino);
-	dent->type = get_dent_type(inode->i_mode);
-	dent->nlen = cpu_to_le16(fname_len(nm));
-	memcpy(dent->name, fname_name(nm), fname_len(nm));
-	dent->name[fname_len(nm)] = '\0';
-	set_dent_cookie(c, dent);
-
-	zero_dent_node_unused(dent);
-	ubifs_prep_grp_node(c, dent, dlen, 0);
-	err = ubifs_node_calc_hash(c, dent, hash_dent);
-	if (err)
-		goto out_release;
-
-	ino = (void *)dent + aligned_dlen;
-	pack_inode(c, ino, inode, 0);
-	err = ubifs_node_calc_hash(c, ino, hash_ino);
-	if (err)
-		goto out_release;
-
-	ino = (void *)ino + aligned_ilen;
-	pack_inode(c, ino, dir, 1);
-	err = ubifs_node_calc_hash(c, ino, hash_ino_host);
-	if (err)
-		goto out_release;
-
-	if (last_reference && !in_orphan) {
-		err = ubifs_add_orphan(c, inode->i_ino);
-		if (err) {
-			release_head(c, BASEHD);
-			goto out_finish;
-		}
-		ui->del_cmtno = c->cmt_no;
-		orphan_added = 1;
-	}
-
-	err = write_head(c, BASEHD, dent, len, &lnum, &dent_offs, sync);
-	if (err)
-		goto out_release;
-	if (!sync) {
-		struct ubifs_wbuf *wbuf = &c->jheads[BASEHD].wbuf;
-
-		ubifs_wbuf_add_ino_nolock(wbuf, inode->i_ino);
-		ubifs_wbuf_add_ino_nolock(wbuf, dir->i_ino);
-	}
-	release_head(c, BASEHD);
-	kfree(dent);
-	ubifs_add_auth_dirt(c, lnum);
-
-	if (deletion) {
-		if (fname_name(nm) == NULL)
-			err = ubifs_tnc_remove_dh(c, &dent_key, nm->minor_hash);
-		else
-			err = ubifs_tnc_remove_nm(c, &dent_key, nm);
-		if (err)
-			goto out_ro;
-		err = ubifs_add_dirt(c, lnum, dlen);
-	} else
-		err = ubifs_tnc_add_nm(c, &dent_key, lnum, dent_offs, dlen,
-				       hash_dent, nm);
-	if (err)
-		goto out_ro;
-
-	/*
-	 * Note, we do not remove the inode from TNC even if the last reference
-	 * to it has just been deleted, because the inode may still be opened.
-	 * Instead, the inode has been added to orphan lists and the orphan
-	 * subsystem will take further care about it.
-	 */
-	ino_key_init(c, &ino_key, inode->i_ino);
-	ino_offs = dent_offs + aligned_dlen;
-	err = ubifs_tnc_add(c, &ino_key, lnum, ino_offs, ilen, hash_ino);
-	if (err)
-		goto out_ro;
-
-	ino_key_init(c, &ino_key, dir->i_ino);
-	ino_offs += aligned_ilen;
-	err = ubifs_tnc_add(c, &ino_key, lnum, ino_offs,
-			    UBIFS_INO_NODE_SZ + host_ui->data_len, hash_ino_host);
-	if (err)
-		goto out_ro;
-
-	if (in_orphan && inode->i_nlink)
-		ubifs_delete_orphan(c, inode->i_ino);
-
-	finish_reservation(c);
-	spin_lock(&ui->ui_lock);
-	ui->synced_i_size = ui->ui_size;
-	spin_unlock(&ui->ui_lock);
-	if (xent) {
-		spin_lock(&host_ui->ui_lock);
-		host_ui->synced_i_size = host_ui->ui_size;
-		spin_unlock(&host_ui->ui_lock);
-	}
-	mark_inode_clean(c, ui);
-	mark_inode_clean(c, host_ui);
-	return 0;
-
-out_finish:
-	finish_reservation(c);
-out_free:
-	kfree(dent);
-	return err;
-
-out_release:
-	release_head(c, BASEHD);
-	kfree(dent);
-out_ro:
-	ubifs_ro_mode(c, err);
-	if (orphan_added)
-		ubifs_delete_orphan(c, inode->i_ino);
-	finish_reservation(c);
-	return err;
-}
-
-/**
- * ubifs_jnl_write_data - write a data node to the journal.
- * @c: UBIFS file-system description object
- * @inode: inode the data node belongs to
- * @key: node key
- * @buf: buffer to write
- * @len: data length (must not exceed %UBIFS_BLOCK_SIZE)
- *
- * This function writes a data node to the journal. Returns %0 if the data node
- * was successfully written, and a negative error code in case of failure.
- */
-int ubifs_jnl_write_data(struct ubifs_info *c, const struct inode *inode,
-			 const union ubifs_key *key, const void *buf, int len)
-{
-	struct ubifs_data_node *data;
-	int err, lnum, offs, compr_type, out_len, compr_len, auth_len;
-	int dlen = COMPRESSED_DATA_NODE_BUF_SZ, allocated = 1;
-	int write_len;
-	struct ubifs_inode *ui = ubifs_inode(inode);
-	bool encrypted = IS_ENCRYPTED(inode);
-	u8 hash[UBIFS_HASH_ARR_SZ];
-
-	dbg_jnlk(key, "ino %lu, blk %u, len %d, key ",
-		(unsigned long)key_inum(c, key), key_block(c, key), len);
-	ubifs_assert(c, len <= UBIFS_BLOCK_SIZE);
-
-	if (encrypted)
-		dlen += UBIFS_CIPHER_BLOCK_SIZE;
-
-	auth_len = ubifs_auth_node_sz(c);
-
-	data = kmalloc(dlen + auth_len, GFP_NOFS | __GFP_NOWARN);
-	if (!data) {
-		/*
-		 * Fall-back to the write reserve buffer. Note, we might be
-		 * currently on the memory reclaim path, when the kernel is
-		 * trying to free some memory by writing out dirty pages. The
-		 * write reserve buffer helps us to guarantee that we are
-		 * always able to write the data.
-		 */
-		allocated = 0;
-		mutex_lock(&c->write_reserve_mutex);
-		data = c->write_reserve_buf;
-	}
-
-	data->ch.node_type = UBIFS_DATA_NODE;
-	key_write(c, key, &data->key);
-	data->size = cpu_to_le32(len);
-
-	if (!(ui->flags & UBIFS_COMPR_FL))
-		/* Compression is disabled for this inode */
-		compr_type = UBIFS_COMPR_NONE;
-	else
-		compr_type = ui->compr_type;
-
-	out_len = compr_len = dlen - UBIFS_DATA_NODE_SZ;
-	ubifs_compress(c, buf, len, &data->data, &compr_len, &compr_type);
-	ubifs_assert(c, compr_len <= UBIFS_BLOCK_SIZE);
-
-	if (encrypted) {
-		err = ubifs_encrypt(inode, data, compr_len, &out_len, key_block(c, key));
-		if (err)
-			goto out_free;
-
-	} else {
-		data->compr_size = 0;
-		out_len = compr_len;
-	}
-
-	dlen = UBIFS_DATA_NODE_SZ + out_len;
-	if (ubifs_authenticated(c))
-		write_len = ALIGN(dlen, 8) + auth_len;
-	else
-		write_len = dlen;
-
-	data->compr_type = cpu_to_le16(compr_type);
-
-	/* Make reservation before allocating sequence numbers */
-	err = make_reservation(c, DATAHD, write_len);
-	if (err)
-		goto out_free;
-
-	ubifs_prepare_node(c, data, dlen, 0);
-	err = write_head(c, DATAHD, data, write_len, &lnum, &offs, 0);
-	if (err)
-		goto out_release;
-
-	err = ubifs_node_calc_hash(c, data, hash);
-	if (err)
-		goto out_release;
-
-	ubifs_wbuf_add_ino_nolock(&c->jheads[DATAHD].wbuf, key_inum(c, key));
-	release_head(c, DATAHD);
-
-	ubifs_add_auth_dirt(c, lnum);
-
-	err = ubifs_tnc_add(c, key, lnum, offs, dlen, hash);
-	if (err)
-		goto out_ro;
-
-	finish_reservation(c);
-	if (!allocated)
-		mutex_unlock(&c->write_reserve_mutex);
-	else
-		kfree(data);
-	return 0;
-
-out_release:
-	release_head(c, DATAHD);
-out_ro:
-	ubifs_ro_mode(c, err);
-	finish_reservation(c);
-out_free:
-	if (!allocated)
-		mutex_unlock(&c->write_reserve_mutex);
-	else
-		kfree(data);
-	return err;
-}
-
-/**
- * ubifs_jnl_write_inode - flush inode to the journal.
- * @c: UBIFS file-system description object
- * @inode: inode to flush
- *
- * This function writes inode @inode to the journal. If the inode is
- * synchronous, it also synchronizes the write-buffer. Returns zero in case of
- * success and a negative error code in case of failure.
- */
-int ubifs_jnl_write_inode(struct ubifs_info *c, const struct inode *inode)
-{
-	int err, lnum, offs;
-	struct ubifs_ino_node *ino, *ino_start;
-	struct ubifs_inode *ui = ubifs_inode(inode);
-	int sync = 0, write_len = 0, ilen = UBIFS_INO_NODE_SZ;
-	int last_reference = !inode->i_nlink;
-	int kill_xattrs = ui->xattr_cnt && last_reference;
-	u8 hash[UBIFS_HASH_ARR_SZ];
-
-	dbg_jnl("ino %lu, nlink %u", inode->i_ino, inode->i_nlink);
-
-	/*
-	 * If the inode is being deleted, do not write the attached data. No
-	 * need to synchronize the write-buffer either.
-	 */
-	if (!last_reference) {
-		ilen += ui->data_len;
-		sync = IS_SYNC(inode);
-	} else if (kill_xattrs) {
-		write_len += UBIFS_INO_NODE_SZ * ui->xattr_cnt;
-	}
-
-	if (ubifs_authenticated(c))
-		write_len += ALIGN(ilen, 8) + ubifs_auth_node_sz(c);
-	else
-		write_len += ilen;
-
-	ino_start = ino = kmalloc(write_len, GFP_NOFS);
-	if (!ino)
-		return -ENOMEM;
-
-	/* Make reservation before allocating sequence numbers */
-	err = make_reservation(c, BASEHD, write_len);
-	if (err)
-		goto out_free;
-
-	if (kill_xattrs) {
-		union ubifs_key key;
-		struct fscrypt_name nm = {0};
-		struct inode *xino;
-		struct ubifs_dent_node *xent, *pxent = NULL;
-
-		if (ui->xattr_cnt > ubifs_xattr_max_cnt(c)) {
-			err = -EPERM;
-			ubifs_err(c, "Cannot delete inode, it has too much xattrs!");
-			goto out_release;
-		}
-
-		lowest_xent_key(c, &key, inode->i_ino);
-		while (1) {
-			xent = ubifs_tnc_next_ent(c, &key, &nm);
-			if (IS_ERR(xent)) {
-				err = PTR_ERR(xent);
-				if (err == -ENOENT)
-					break;
-
-				kfree(pxent);
-				goto out_release;
-			}
-
-			fname_name(&nm) = xent->name;
-			fname_len(&nm) = le16_to_cpu(xent->nlen);
-
-			xino = ubifs_iget(c->vfs_sb, le64_to_cpu(xent->inum));
-			if (IS_ERR(xino)) {
-				err = PTR_ERR(xino);
-				ubifs_err(c, "dead directory entry '%s', error %d",
-					  xent->name, err);
-				ubifs_ro_mode(c, err);
-				kfree(pxent);
-				kfree(xent);
-				goto out_release;
-			}
-			ubifs_assert(c, ubifs_inode(xino)->xattr);
-
-			clear_nlink(xino);
-			pack_inode(c, ino, xino, 0);
-			ino = (void *)ino + UBIFS_INO_NODE_SZ;
-			iput(xino);
-
-			kfree(pxent);
-			pxent = xent;
-			key_read(c, &xent->key, &key);
-		}
-		kfree(pxent);
-	}
-
-	pack_inode(c, ino, inode, 1);
-	err = ubifs_node_calc_hash(c, ino, hash);
-	if (err)
-		goto out_release;
-
-	err = write_head(c, BASEHD, ino_start, write_len, &lnum, &offs, sync);
-	if (err)
-		goto out_release;
-	if (!sync)
-		ubifs_wbuf_add_ino_nolock(&c->jheads[BASEHD].wbuf,
-					  inode->i_ino);
-	release_head(c, BASEHD);
-
-	if (last_reference) {
-		err = ubifs_tnc_remove_ino(c, inode->i_ino);
-		if (err)
-			goto out_ro;
-		ubifs_delete_orphan(c, inode->i_ino);
-		err = ubifs_add_dirt(c, lnum, write_len);
-	} else {
-		union ubifs_key key;
-
-		ubifs_add_auth_dirt(c, lnum);
-
-		ino_key_init(c, &key, inode->i_ino);
-		err = ubifs_tnc_add(c, &key, lnum, offs, ilen, hash);
-	}
-	if (err)
-		goto out_ro;
-
-	finish_reservation(c);
-	spin_lock(&ui->ui_lock);
-	ui->synced_i_size = ui->ui_size;
-	spin_unlock(&ui->ui_lock);
-	kfree(ino_start);
-	return 0;
-
-out_release:
-	release_head(c, BASEHD);
-out_ro:
-	ubifs_ro_mode(c, err);
-	finish_reservation(c);
-out_free:
-	kfree(ino_start);
-	return err;
-}
-
-/**
- * ubifs_jnl_delete_inode - delete an inode.
- * @c: UBIFS file-system description object
- * @inode: inode to delete
- *
- * This function deletes inode @inode which includes removing it from orphans,
- * deleting it from TNC and, in some cases, writing a deletion inode to the
- * journal.
- *
- * When regular file inodes are unlinked or a directory inode is removed, the
- * 'ubifs_jnl_update()' function writes a corresponding deletion inode and
- * direntry to the media, and adds the inode to orphans. After this, when the
- * last reference to this inode has been dropped, this function is called. In
- * general, it has to write one more deletion inode to the media, because if
- * a commit happened between 'ubifs_jnl_update()' and
- * 'ubifs_jnl_delete_inode()', the deletion inode is not in the journal
- * anymore, and in fact it might not be on the flash anymore, because it might
- * have been garbage-collected already. And for optimization reasons UBIFS does
- * not read the orphan area if it has been unmounted cleanly, so it would have
- * no indication in the journal that there is a deleted inode which has to be
- * removed from TNC.
- *
- * However, if there was no commit between 'ubifs_jnl_update()' and
- * 'ubifs_jnl_delete_inode()', then there is no need to write the deletion
- * inode to the media for the second time. And this is quite a typical case.
- *
- * This function returns zero in case of success and a negative error code in
- * case of failure.
- */
-int ubifs_jnl_delete_inode(struct ubifs_info *c, const struct inode *inode)
-{
-	int err;
-	struct ubifs_inode *ui = ubifs_inode(inode);
-
-	ubifs_assert(c, inode->i_nlink == 0);
-
-	if (ui->xattr_cnt || ui->del_cmtno != c->cmt_no)
-		/* A commit happened for sure or inode hosts xattrs */
-		return ubifs_jnl_write_inode(c, inode);
-
-	down_read(&c->commit_sem);
-	/*
-	 * Check commit number again, because the first test has been done
-	 * without @c->commit_sem, so a commit might have happened.
-	 */
-	if (ui->del_cmtno != c->cmt_no) {
-		up_read(&c->commit_sem);
-		return ubifs_jnl_write_inode(c, inode);
-	}
-
-	err = ubifs_tnc_remove_ino(c, inode->i_ino);
-	if (err)
-		ubifs_ro_mode(c, err);
-	else
-		ubifs_delete_orphan(c, inode->i_ino);
-	up_read(&c->commit_sem);
-	return err;
-}
-
-/**
- * ubifs_jnl_xrename - cross rename two directory entries.
- * @c: UBIFS file-system description object
- * @fst_dir: parent inode of 1st directory entry to exchange
- * @fst_inode: 1st inode to exchange
- * @fst_nm: name of 1st inode to exchange
- * @snd_dir: parent inode of 2nd directory entry to exchange
- * @snd_inode: 2nd inode to exchange
- * @snd_nm: name of 2nd inode to exchange
- * @sync: non-zero if the write-buffer has to be synchronized
- *
- * This function implements the cross rename operation which may involve
- * writing 2 inodes and 2 directory entries. It marks the written inodes as clean
- * and returns zero on success. In case of failure, a negative error code is
- * returned.
- */
-int ubifs_jnl_xrename(struct ubifs_info *c, const struct inode *fst_dir,
-		      const struct inode *fst_inode,
-		      const struct fscrypt_name *fst_nm,
-		      const struct inode *snd_dir,
-		      const struct inode *snd_inode,
-		      const struct fscrypt_name *snd_nm, int sync)
-{
-	union ubifs_key key;
-	struct ubifs_dent_node *dent1, *dent2;
-	int err, dlen1, dlen2, lnum, offs, len, plen = UBIFS_INO_NODE_SZ;
-	int aligned_dlen1, aligned_dlen2;
-	int twoparents = (fst_dir != snd_dir);
-	void *p;
-	u8 hash_dent1[UBIFS_HASH_ARR_SZ];
-	u8 hash_dent2[UBIFS_HASH_ARR_SZ];
-	u8 hash_p1[UBIFS_HASH_ARR_SZ];
-	u8 hash_p2[UBIFS_HASH_ARR_SZ];
-
-	ubifs_assert(c, ubifs_inode(fst_dir)->data_len == 0);
-	ubifs_assert(c, ubifs_inode(snd_dir)->data_len == 0);
-	ubifs_assert(c, mutex_is_locked(&ubifs_inode(fst_dir)->ui_mutex));
-	ubifs_assert(c, mutex_is_locked(&ubifs_inode(snd_dir)->ui_mutex));
-
-	dlen1 = UBIFS_DENT_NODE_SZ + fname_len(snd_nm) + 1;
-	dlen2 = UBIFS_DENT_NODE_SZ + fname_len(fst_nm) + 1;
-	aligned_dlen1 = ALIGN(dlen1, 8);
-	aligned_dlen2 = ALIGN(dlen2, 8);
-
-	len = aligned_dlen1 + aligned_dlen2 + ALIGN(plen, 8);
-	if (twoparents)
-		len += plen;
-
-	len += ubifs_auth_node_sz(c);
-
-	dent1 = kzalloc(len, GFP_NOFS);
-	if (!dent1)
-		return -ENOMEM;
-
-	/* Make reservation before allocating sequence numbers */
-	err = make_reservation(c, BASEHD, len);
-	if (err)
-		goto out_free;
-
-	/* Make new dent for 1st entry */
-	dent1->ch.node_type = UBIFS_DENT_NODE;
-	dent_key_init_flash(c, &dent1->key, snd_dir->i_ino, snd_nm);
-	dent1->inum = cpu_to_le64(fst_inode->i_ino);
-	dent1->type = get_dent_type(fst_inode->i_mode);
-	dent1->nlen = cpu_to_le16(fname_len(snd_nm));
-	memcpy(dent1->name, fname_name(snd_nm), fname_len(snd_nm));
-	dent1->name[fname_len(snd_nm)] = '\0';
-	set_dent_cookie(c, dent1);
-	zero_dent_node_unused(dent1);
-	ubifs_prep_grp_node(c, dent1, dlen1, 0);
-	err = ubifs_node_calc_hash(c, dent1, hash_dent1);
-	if (err)
-		goto out_release;
-
-	/* Make new dent for 2nd entry */
-	dent2 = (void *)dent1 + aligned_dlen1;
-	dent2->ch.node_type = UBIFS_DENT_NODE;
-	dent_key_init_flash(c, &dent2->key, fst_dir->i_ino, fst_nm);
-	dent2->inum = cpu_to_le64(snd_inode->i_ino);
-	dent2->type = get_dent_type(snd_inode->i_mode);
-	dent2->nlen = cpu_to_le16(fname_len(fst_nm));
-	memcpy(dent2->name, fname_name(fst_nm), fname_len(fst_nm));
-	dent2->name[fname_len(fst_nm)] = '\0';
-	set_dent_cookie(c, dent2);
-	zero_dent_node_unused(dent2);
-	ubifs_prep_grp_node(c, dent2, dlen2, 0);
-	err = ubifs_node_calc_hash(c, dent2, hash_dent2);
-	if (err)
-		goto out_release;
-
-	p = (void *)dent2 + aligned_dlen2;
-	if (!twoparents) {
-		pack_inode(c, p, fst_dir, 1);
-		err = ubifs_node_calc_hash(c, p, hash_p1);
-		if (err)
-			goto out_release;
-	} else {
-		pack_inode(c, p, fst_dir, 0);
-		err = ubifs_node_calc_hash(c, p, hash_p1);
-		if (err)
-			goto out_release;
-		p += ALIGN(plen, 8);
-		pack_inode(c, p, snd_dir, 1);
-		err = ubifs_node_calc_hash(c, p, hash_p2);
-		if (err)
-			goto out_release;
-	}
-
-	err = write_head(c, BASEHD, dent1, len, &lnum, &offs, sync);
-	if (err)
-		goto out_release;
-	if (!sync) {
-		struct ubifs_wbuf *wbuf = &c->jheads[BASEHD].wbuf;
-
-		ubifs_wbuf_add_ino_nolock(wbuf, fst_dir->i_ino);
-		ubifs_wbuf_add_ino_nolock(wbuf, snd_dir->i_ino);
-	}
-	release_head(c, BASEHD);
-
-	ubifs_add_auth_dirt(c, lnum);
-
-	dent_key_init(c, &key, snd_dir->i_ino, snd_nm);
-	err = ubifs_tnc_add_nm(c, &key, lnum, offs, dlen1, hash_dent1, snd_nm);
-	if (err)
-		goto out_ro;
-
-	offs += aligned_dlen1;
-	dent_key_init(c, &key, fst_dir->i_ino, fst_nm);
-	err = ubifs_tnc_add_nm(c, &key, lnum, offs, dlen2, hash_dent2, fst_nm);
-	if (err)
-		goto out_ro;
-
-	offs += aligned_dlen2;
-
-	ino_key_init(c, &key, fst_dir->i_ino);
-	err = ubifs_tnc_add(c, &key, lnum, offs, plen, hash_p1);
-	if (err)
-		goto out_ro;
-
-	if (twoparents) {
-		offs += ALIGN(plen, 8);
-		ino_key_init(c, &key, snd_dir->i_ino);
-		err = ubifs_tnc_add(c, &key, lnum, offs, plen, hash_p2);
-		if (err)
-			goto out_ro;
-	}
-
-	finish_reservation(c);
-
-	mark_inode_clean(c, ubifs_inode(fst_dir));
-	if (twoparents)
-		mark_inode_clean(c, ubifs_inode(snd_dir));
-	kfree(dent1);
-	return 0;
-
-out_release:
-	release_head(c, BASEHD);
-out_ro:
-	ubifs_ro_mode(c, err);
-	finish_reservation(c);
-out_free:
-	kfree(dent1);
-	return err;
-}
-
-/**
- * ubifs_jnl_rename - rename a directory entry.
- * @c: UBIFS file-system description object
- * @old_dir: parent inode of directory entry to rename
- * @old_inode: directory entry's inode to rename
- * @old_nm: name of the old directory entry to rename
- * @new_dir: parent inode of directory entry to rename
- * @new_inode: new directory entry's inode (or directory entry's inode to
- *		replace)
- * @new_nm: new name of the new directory entry
- * @whiteout: whiteout inode
- * @sync: non-zero if the write-buffer has to be synchronized
- * @delete_orphan: indicates an orphan entry deletion for @whiteout
- *
- * This function implements the re-name operation which may involve writing up
- * to 4 inodes(new inode, whiteout inode, old and new parent directory inodes)
- * and 2 directory entries. It marks the written inodes as clean and returns
- * zero on success. In case of failure, a negative error code is returned.
- */
-int ubifs_jnl_rename(struct ubifs_info *c, const struct inode *old_dir,
-		     const struct inode *old_inode,
-		     const struct fscrypt_name *old_nm,
-		     const struct inode *new_dir,
-		     const struct inode *new_inode,
-		     const struct fscrypt_name *new_nm,
-		     const struct inode *whiteout, int sync, int delete_orphan)
-{
-	void *p;
-	union ubifs_key key;
-	struct ubifs_dent_node *dent, *dent2;
-	int err, dlen1, dlen2, ilen, wlen, lnum, offs, len, orphan_added = 0;
-	int aligned_dlen1, aligned_dlen2, plen = UBIFS_INO_NODE_SZ;
-	int last_reference = !!(new_inode && new_inode->i_nlink == 0);
-	int move = (old_dir != new_dir);
-	struct ubifs_inode *new_ui, *whiteout_ui;
-	u8 hash_old_dir[UBIFS_HASH_ARR_SZ];
-	u8 hash_new_dir[UBIFS_HASH_ARR_SZ];
-	u8 hash_new_inode[UBIFS_HASH_ARR_SZ];
-	u8 hash_whiteout_inode[UBIFS_HASH_ARR_SZ];
-	u8 hash_dent1[UBIFS_HASH_ARR_SZ];
-	u8 hash_dent2[UBIFS_HASH_ARR_SZ];
-
-	ubifs_assert(c, ubifs_inode(old_dir)->data_len == 0);
-	ubifs_assert(c, ubifs_inode(new_dir)->data_len == 0);
-	ubifs_assert(c, mutex_is_locked(&ubifs_inode(old_dir)->ui_mutex));
-	ubifs_assert(c, mutex_is_locked(&ubifs_inode(new_dir)->ui_mutex));
-
-	dlen1 = UBIFS_DENT_NODE_SZ + fname_len(new_nm) + 1;
-	dlen2 = UBIFS_DENT_NODE_SZ + fname_len(old_nm) + 1;
-	if (new_inode) {
-		new_ui = ubifs_inode(new_inode);
-		ubifs_assert(c, mutex_is_locked(&new_ui->ui_mutex));
-		ilen = UBIFS_INO_NODE_SZ;
-		if (!last_reference)
-			ilen += new_ui->data_len;
-	} else
-		ilen = 0;
-
-	if (whiteout) {
-		whiteout_ui = ubifs_inode(whiteout);
-		ubifs_assert(c, mutex_is_locked(&whiteout_ui->ui_mutex));
-		ubifs_assert(c, whiteout->i_nlink == 1);
-		ubifs_assert(c, !whiteout_ui->dirty);
-		wlen = UBIFS_INO_NODE_SZ;
-		wlen += whiteout_ui->data_len;
-	} else
-		wlen = 0;
-
-	aligned_dlen1 = ALIGN(dlen1, 8);
-	aligned_dlen2 = ALIGN(dlen2, 8);
-	len = aligned_dlen1 + aligned_dlen2 + ALIGN(ilen, 8) +
-	      ALIGN(wlen, 8) + ALIGN(plen, 8);
-	if (move)
-		len += plen;
-
-	len += ubifs_auth_node_sz(c);
-
-	dent = kzalloc(len, GFP_NOFS);
-	if (!dent)
-		return -ENOMEM;
-
-	/* Make reservation before allocating sequence numbers */
-	err = make_reservation(c, BASEHD, len);
-	if (err)
-		goto out_free;
-
-	/* Make new dent */
-	dent->ch.node_type = UBIFS_DENT_NODE;
-	dent_key_init_flash(c, &dent->key, new_dir->i_ino, new_nm);
-	dent->inum = cpu_to_le64(old_inode->i_ino);
-	dent->type = get_dent_type(old_inode->i_mode);
-	dent->nlen = cpu_to_le16(fname_len(new_nm));
-	memcpy(dent->name, fname_name(new_nm), fname_len(new_nm));
-	dent->name[fname_len(new_nm)] = '\0';
-	set_dent_cookie(c, dent);
-	zero_dent_node_unused(dent);
-	ubifs_prep_grp_node(c, dent, dlen1, 0);
-	err = ubifs_node_calc_hash(c, dent, hash_dent1);
-	if (err)
-		goto out_release;
-
-	dent2 = (void *)dent + aligned_dlen1;
-	dent2->ch.node_type = UBIFS_DENT_NODE;
-	dent_key_init_flash(c, &dent2->key, old_dir->i_ino, old_nm);
-
-	if (whiteout) {
-		dent2->inum = cpu_to_le64(whiteout->i_ino);
-		dent2->type = get_dent_type(whiteout->i_mode);
-	} else {
-		/* Make deletion dent */
-		dent2->inum = 0;
-		dent2->type = DT_UNKNOWN;
-	}
-	dent2->nlen = cpu_to_le16(fname_len(old_nm));
-	memcpy(dent2->name, fname_name(old_nm), fname_len(old_nm));
-	dent2->name[fname_len(old_nm)] = '\0';
-	set_dent_cookie(c, dent2);
-	zero_dent_node_unused(dent2);
-	ubifs_prep_grp_node(c, dent2, dlen2, 0);
-	err = ubifs_node_calc_hash(c, dent2, hash_dent2);
-	if (err)
-		goto out_release;
-
-	p = (void *)dent2 + aligned_dlen2;
-	if (new_inode) {
-		pack_inode(c, p, new_inode, 0);
-		err = ubifs_node_calc_hash(c, p, hash_new_inode);
-		if (err)
-			goto out_release;
-
-		p += ALIGN(ilen, 8);
-	}
-
-	if (whiteout) {
-		pack_inode(c, p, whiteout, 0);
-		err = ubifs_node_calc_hash(c, p, hash_whiteout_inode);
-		if (err)
-			goto out_release;
-
-		p += ALIGN(wlen, 8);
-	}
-
-	if (!move) {
-		pack_inode(c, p, old_dir, 1);
-		err = ubifs_node_calc_hash(c, p, hash_old_dir);
-		if (err)
-			goto out_release;
-	} else {
-		pack_inode(c, p, old_dir, 0);
-		err = ubifs_node_calc_hash(c, p, hash_old_dir);
-		if (err)
-			goto out_release;
-
-		p += ALIGN(plen, 8);
-		pack_inode(c, p, new_dir, 1);
-		err = ubifs_node_calc_hash(c, p, hash_new_dir);
-		if (err)
-			goto out_release;
-	}
-
-	if (last_reference) {
-		err = ubifs_add_orphan(c, new_inode->i_ino);
-		if (err) {
-			release_head(c, BASEHD);
-			goto out_finish;
-		}
-		new_ui->del_cmtno = c->cmt_no;
-		orphan_added = 1;
-	}
-
-	err = write_head(c, BASEHD, dent, len, &lnum, &offs, sync);
-	if (err)
-		goto out_release;
-	if (!sync) {
-		struct ubifs_wbuf *wbuf = &c->jheads[BASEHD].wbuf;
-
-		ubifs_wbuf_add_ino_nolock(wbuf, new_dir->i_ino);
-		ubifs_wbuf_add_ino_nolock(wbuf, old_dir->i_ino);
-		if (new_inode)
-			ubifs_wbuf_add_ino_nolock(&c->jheads[BASEHD].wbuf,
-						  new_inode->i_ino);
-		if (whiteout)
-			ubifs_wbuf_add_ino_nolock(&c->jheads[BASEHD].wbuf,
-						  whiteout->i_ino);
-	}
-	release_head(c, BASEHD);
-
-	ubifs_add_auth_dirt(c, lnum);
-
-	dent_key_init(c, &key, new_dir->i_ino, new_nm);
-	err = ubifs_tnc_add_nm(c, &key, lnum, offs, dlen1, hash_dent1, new_nm);
-	if (err)
-		goto out_ro;
-
-	offs += aligned_dlen1;
-	if (whiteout) {
-		dent_key_init(c, &key, old_dir->i_ino, old_nm);
-		err = ubifs_tnc_add_nm(c, &key, lnum, offs, dlen2, hash_dent2, old_nm);
-		if (err)
-			goto out_ro;
-	} else {
-		err = ubifs_add_dirt(c, lnum, dlen2);
-		if (err)
-			goto out_ro;
-
-		dent_key_init(c, &key, old_dir->i_ino, old_nm);
-		err = ubifs_tnc_remove_nm(c, &key, old_nm);
-		if (err)
-			goto out_ro;
-	}
-
-	offs += aligned_dlen2;
-	if (new_inode) {
-		ino_key_init(c, &key, new_inode->i_ino);
-		err = ubifs_tnc_add(c, &key, lnum, offs, ilen, hash_new_inode);
-		if (err)
-			goto out_ro;
-		offs += ALIGN(ilen, 8);
-	}
-
-	if (whiteout) {
-		ino_key_init(c, &key, whiteout->i_ino);
-		err = ubifs_tnc_add(c, &key, lnum, offs, wlen,
-				    hash_whiteout_inode);
-		if (err)
-			goto out_ro;
-		offs += ALIGN(wlen, 8);
-	}
-
-	ino_key_init(c, &key, old_dir->i_ino);
-	err = ubifs_tnc_add(c, &key, lnum, offs, plen, hash_old_dir);
-	if (err)
-		goto out_ro;
-
-	if (move) {
-		offs += ALIGN(plen, 8);
-		ino_key_init(c, &key, new_dir->i_ino);
-		err = ubifs_tnc_add(c, &key, lnum, offs, plen, hash_new_dir);
-		if (err)
-			goto out_ro;
-	}
-
-	if (delete_orphan)
-		ubifs_delete_orphan(c, whiteout->i_ino);
-
-	finish_reservation(c);
-	if (new_inode) {
-		mark_inode_clean(c, new_ui);
-		spin_lock(&new_ui->ui_lock);
-		new_ui->synced_i_size = new_ui->ui_size;
-		spin_unlock(&new_ui->ui_lock);
-	}
-	/*
-	 * No need to mark whiteout inode clean.
-	 * Whiteout doesn't have non-zero size, no need to update
-	 * synced_i_size for whiteout_ui.
-	 */
-	mark_inode_clean(c, ubifs_inode(old_dir));
-	if (move)
-		mark_inode_clean(c, ubifs_inode(new_dir));
-	kfree(dent);
-	return 0;
-
-out_release:
-	release_head(c, BASEHD);
-out_ro:
-	ubifs_ro_mode(c, err);
-	if (orphan_added)
-		ubifs_delete_orphan(c, new_inode->i_ino);
-out_finish:
-	finish_reservation(c);
-out_free:
-	kfree(dent);
-	return err;
-}
-
-/**
- * truncate_data_node - re-compress/encrypt a truncated data node.
- * @c: UBIFS file-system description object
- * @inode: inode which refers to the data node
- * @block: data block number
- * @dn: data node to re-compress
- * @new_len: new length
- * @dn_size: size of the data node @dn in memory
- *
- * This function is used when an inode is truncated and the last data node of
- * the inode has to be re-compressed/encrypted and re-written.
- */
-static int truncate_data_node(const struct ubifs_info *c, const struct inode *inode,
-			      unsigned int block, struct ubifs_data_node *dn,
-			      int *new_len, int dn_size)
-{
-	void *buf;
-	int err, dlen, compr_type, out_len, data_size;
-
-	out_len = le32_to_cpu(dn->size);
-	buf = kmalloc_array(out_len, WORST_COMPR_FACTOR, GFP_NOFS);
-	if (!buf)
-		return -ENOMEM;
-
-	dlen = le32_to_cpu(dn->ch.len) - UBIFS_DATA_NODE_SZ;
-	data_size = dn_size - UBIFS_DATA_NODE_SZ;
-	compr_type = le16_to_cpu(dn->compr_type);
-
-	if (IS_ENCRYPTED(inode)) {
-		err = ubifs_decrypt(inode, dn, &dlen, block);
-		if (err)
-			goto out;
-	}
-
-	if (compr_type == UBIFS_COMPR_NONE) {
-		out_len = *new_len;
-	} else {
-		err = ubifs_decompress(c, &dn->data, dlen, buf, &out_len, compr_type);
-		if (err)
-			goto out;
-
-		ubifs_compress(c, buf, *new_len, &dn->data, &out_len, &compr_type);
-	}
-
-	if (IS_ENCRYPTED(inode)) {
-		err = ubifs_encrypt(inode, dn, out_len, &data_size, block);
-		if (err)
-			goto out;
-
-		out_len = data_size;
-	} else {
-		dn->compr_size = 0;
-	}
-
-	ubifs_assert(c, out_len <= UBIFS_BLOCK_SIZE);
-	dn->compr_type = cpu_to_le16(compr_type);
-	dn->size = cpu_to_le32(*new_len);
-	*new_len = UBIFS_DATA_NODE_SZ + out_len;
-	err = 0;
-out:
-	kfree(buf);
-	return err;
-}
-
-/**
- * ubifs_jnl_truncate - update the journal for a truncation.
- * @c: UBIFS file-system description object
- * @inode: inode to truncate
- * @old_size: old size
- * @new_size: new size
- *
- * When the size of a file decreases due to truncation, a truncation node is
- * written, the journal tree is updated, and the last data block is re-written
- * if it has been affected. The inode is also updated in order to synchronize
- * the new inode size.
- *
- * This function marks the inode as clean and returns zero on success. In case
- * of failure, a negative error code is returned.
- */
-int ubifs_jnl_truncate(struct ubifs_info *c, const struct inode *inode,
-		       loff_t old_size, loff_t new_size)
-{
-	union ubifs_key key, to_key;
-	struct ubifs_ino_node *ino;
-	struct ubifs_trun_node *trun;
-	struct ubifs_data_node *dn;
-	int err, dlen, len, lnum, offs, bit, sz, sync = IS_SYNC(inode);
-	int dn_size;
-	struct ubifs_inode *ui = ubifs_inode(inode);
-	ino_t inum = inode->i_ino;
-	unsigned int blk;
-	u8 hash_ino[UBIFS_HASH_ARR_SZ];
-	u8 hash_dn[UBIFS_HASH_ARR_SZ];
-
-	dbg_jnl("ino %lu, size %lld -> %lld",
-		(unsigned long)inum, old_size, new_size);
-	ubifs_assert(c, !ui->data_len);
-	ubifs_assert(c, S_ISREG(inode->i_mode));
-	ubifs_assert(c, mutex_is_locked(&ui->ui_mutex));
-
-	dn_size = COMPRESSED_DATA_NODE_BUF_SZ;
-
-	if (IS_ENCRYPTED(inode))
-		dn_size += UBIFS_CIPHER_BLOCK_SIZE;
-
-	sz =  UBIFS_TRUN_NODE_SZ + UBIFS_INO_NODE_SZ +
-		dn_size + ubifs_auth_node_sz(c);
-
-	ino = kmalloc(sz, GFP_NOFS);
-	if (!ino)
-		return -ENOMEM;
-
-	trun = (void *)ino + UBIFS_INO_NODE_SZ;
-	trun->ch.node_type = UBIFS_TRUN_NODE;
-	trun->inum = cpu_to_le32(inum);
-	trun->old_size = cpu_to_le64(old_size);
-	trun->new_size = cpu_to_le64(new_size);
-	zero_trun_node_unused(trun);
-
-	dlen = new_size & (UBIFS_BLOCK_SIZE - 1);
-	if (dlen) {
-		/* Get last data block so it can be truncated */
-		dn = (void *)trun + UBIFS_TRUN_NODE_SZ;
-		blk = new_size >> UBIFS_BLOCK_SHIFT;
-		data_key_init(c, &key, inum, blk);
-		dbg_jnlk(&key, "last block key ");
-		err = ubifs_tnc_lookup(c, &key, dn);
-		if (err == -ENOENT)
-			dlen = 0; /* Not found (so it is a hole) */
-		else if (err)
-			goto out_free;
-		else {
-			int dn_len = le32_to_cpu(dn->size);
-
-			if (dn_len <= 0 || dn_len > UBIFS_BLOCK_SIZE) {
-				ubifs_err(c, "bad data node (block %u, inode %lu)",
-					  blk, inode->i_ino);
-				ubifs_dump_node(c, dn, dn_size);
-				err = -EUCLEAN;
-				goto out_free;
-			}
-
-			if (dn_len <= dlen)
-				dlen = 0; /* Nothing to do */
-			else {
-				err = truncate_data_node(c, inode, blk, dn,
-						&dlen, dn_size);
-				if (err)
-					goto out_free;
-			}
-		}
-	}
-
-	/* Must make reservation before allocating sequence numbers */
-	len = UBIFS_TRUN_NODE_SZ + UBIFS_INO_NODE_SZ;
-
-	if (ubifs_authenticated(c))
-		len += ALIGN(dlen, 8) + ubifs_auth_node_sz(c);
-	else
-		len += dlen;
-
-	err = make_reservation(c, BASEHD, len);
-	if (err)
-		goto out_free;
-
-	pack_inode(c, ino, inode, 0);
-	err = ubifs_node_calc_hash(c, ino, hash_ino);
-	if (err)
-		goto out_release;
-
-	ubifs_prep_grp_node(c, trun, UBIFS_TRUN_NODE_SZ, dlen ? 0 : 1);
-	if (dlen) {
-		ubifs_prep_grp_node(c, dn, dlen, 1);
-		err = ubifs_node_calc_hash(c, dn, hash_dn);
-		if (err)
-			goto out_release;
-	}
-
-	err = write_head(c, BASEHD, ino, len, &lnum, &offs, sync);
-	if (err)
-		goto out_release;
-	if (!sync)
-		ubifs_wbuf_add_ino_nolock(&c->jheads[BASEHD].wbuf, inum);
-	release_head(c, BASEHD);
-
-	ubifs_add_auth_dirt(c, lnum);
-
-	if (dlen) {
-		sz = offs + UBIFS_INO_NODE_SZ + UBIFS_TRUN_NODE_SZ;
-		err = ubifs_tnc_add(c, &key, lnum, sz, dlen, hash_dn);
-		if (err)
-			goto out_ro;
-	}
-
-	ino_key_init(c, &key, inum);
-	err = ubifs_tnc_add(c, &key, lnum, offs, UBIFS_INO_NODE_SZ, hash_ino);
-	if (err)
-		goto out_ro;
-
-	err = ubifs_add_dirt(c, lnum, UBIFS_TRUN_NODE_SZ);
-	if (err)
-		goto out_ro;
-
-	bit = new_size & (UBIFS_BLOCK_SIZE - 1);
-	blk = (new_size >> UBIFS_BLOCK_SHIFT) + (bit ? 1 : 0);
-	data_key_init(c, &key, inum, blk);
-
-	bit = old_size & (UBIFS_BLOCK_SIZE - 1);
-	blk = (old_size >> UBIFS_BLOCK_SHIFT) - (bit ? 0 : 1);
-	data_key_init(c, &to_key, inum, blk);
-
-	err = ubifs_tnc_remove_range(c, &key, &to_key);
-	if (err)
-		goto out_ro;
-
-	finish_reservation(c);
-	spin_lock(&ui->ui_lock);
-	ui->synced_i_size = ui->ui_size;
-	spin_unlock(&ui->ui_lock);
-	mark_inode_clean(c, ui);
-	kfree(ino);
-	return 0;
-
-out_release:
-	release_head(c, BASEHD);
-out_ro:
-	ubifs_ro_mode(c, err);
-	finish_reservation(c);
-out_free:
-	kfree(ino);
-	return err;
-}
-
-
-/**
- * ubifs_jnl_delete_xattr - delete an extended attribute.
- * @c: UBIFS file-system description object
- * @host: host inode
- * @inode: extended attribute inode
- * @nm: extended attribute entry name
- *
- * This function delete an extended attribute which is very similar to
- * un-linking regular files - it writes a deletion xentry, a deletion inode and
- * updates the target inode. Returns zero in case of success and a negative
- * error code in case of failure.
- */
-int ubifs_jnl_delete_xattr(struct ubifs_info *c, const struct inode *host,
-			   const struct inode *inode,
-			   const struct fscrypt_name *nm)
-{
-	int err, xlen, hlen, len, lnum, xent_offs, aligned_xlen, write_len;
-	struct ubifs_dent_node *xent;
-	struct ubifs_ino_node *ino;
-	union ubifs_key xent_key, key1, key2;
-	int sync = IS_DIRSYNC(host);
-	struct ubifs_inode *host_ui = ubifs_inode(host);
-	u8 hash[UBIFS_HASH_ARR_SZ];
-
-	ubifs_assert(c, inode->i_nlink == 0);
-	ubifs_assert(c, mutex_is_locked(&host_ui->ui_mutex));
-
-	/*
-	 * Since we are deleting the inode, we do not bother to attach any data
-	 * to it and assume its length is %UBIFS_INO_NODE_SZ.
-	 */
-	xlen = UBIFS_DENT_NODE_SZ + fname_len(nm) + 1;
-	aligned_xlen = ALIGN(xlen, 8);
-	hlen = host_ui->data_len + UBIFS_INO_NODE_SZ;
-	len = aligned_xlen + UBIFS_INO_NODE_SZ + ALIGN(hlen, 8);
-
-	write_len = len + ubifs_auth_node_sz(c);
-
-	xent = kzalloc(write_len, GFP_NOFS);
-	if (!xent)
-		return -ENOMEM;
-
-	/* Make reservation before allocating sequence numbers */
-	err = make_reservation(c, BASEHD, write_len);
-	if (err) {
-		kfree(xent);
-		return err;
-	}
-
-	xent->ch.node_type = UBIFS_XENT_NODE;
-	xent_key_init(c, &xent_key, host->i_ino, nm);
-	key_write(c, &xent_key, xent->key);
-	xent->inum = 0;
-	xent->type = get_dent_type(inode->i_mode);
-	xent->nlen = cpu_to_le16(fname_len(nm));
-	memcpy(xent->name, fname_name(nm), fname_len(nm));
-	xent->name[fname_len(nm)] = '\0';
-	zero_dent_node_unused(xent);
-	ubifs_prep_grp_node(c, xent, xlen, 0);
-
-	ino = (void *)xent + aligned_xlen;
-	pack_inode(c, ino, inode, 0);
-	ino = (void *)ino + UBIFS_INO_NODE_SZ;
-	pack_inode(c, ino, host, 1);
-	err = ubifs_node_calc_hash(c, ino, hash);
-	if (err)
-		goto out_release;
-
-	err = write_head(c, BASEHD, xent, write_len, &lnum, &xent_offs, sync);
-	if (!sync && !err)
-		ubifs_wbuf_add_ino_nolock(&c->jheads[BASEHD].wbuf, host->i_ino);
-	release_head(c, BASEHD);
-
-	ubifs_add_auth_dirt(c, lnum);
-	kfree(xent);
-	if (err)
-		goto out_ro;
-
-	/* Remove the extended attribute entry from TNC */
-	err = ubifs_tnc_remove_nm(c, &xent_key, nm);
-	if (err)
-		goto out_ro;
-	err = ubifs_add_dirt(c, lnum, xlen);
-	if (err)
-		goto out_ro;
-
-	/*
-	 * Remove all nodes belonging to the extended attribute inode from TNC.
-	 * Well, there actually must be only one node - the inode itself.
-	 */
-	lowest_ino_key(c, &key1, inode->i_ino);
-	highest_ino_key(c, &key2, inode->i_ino);
-	err = ubifs_tnc_remove_range(c, &key1, &key2);
-	if (err)
-		goto out_ro;
-	err = ubifs_add_dirt(c, lnum, UBIFS_INO_NODE_SZ);
-	if (err)
-		goto out_ro;
-
-	/* And update TNC with the new host inode position */
-	ino_key_init(c, &key1, host->i_ino);
-	err = ubifs_tnc_add(c, &key1, lnum, xent_offs + len - hlen, hlen, hash);
-	if (err)
-		goto out_ro;
-
-	finish_reservation(c);
-	spin_lock(&host_ui->ui_lock);
-	host_ui->synced_i_size = host_ui->ui_size;
-	spin_unlock(&host_ui->ui_lock);
-	mark_inode_clean(c, host_ui);
-	return 0;
-
-out_release:
-	kfree(xent);
-	release_head(c, BASEHD);
-out_ro:
-	ubifs_ro_mode(c, err);
-	finish_reservation(c);
-	return err;
-}
-
-/**
- * ubifs_jnl_change_xattr - change an extended attribute.
- * @c: UBIFS file-system description object
- * @inode: extended attribute inode
- * @host: host inode
- *
- * This function writes the updated version of an extended attribute inode and
- * the host inode to the journal (to the base head). The host inode is written
- * after the extended attribute inode in order to guarantee that the extended
- * attribute will be flushed when the inode is synchronized by 'fsync()' and
- * consequently, the write-buffer is synchronized. This function returns zero
- * in case of success and a negative error code in case of failure.
- */
-int ubifs_jnl_change_xattr(struct ubifs_info *c, const struct inode *inode,
-			   const struct inode *host)
-{
-	int err, len1, len2, aligned_len, aligned_len1, lnum, offs;
-	struct ubifs_inode *host_ui = ubifs_inode(host);
-	struct ubifs_ino_node *ino;
-	union ubifs_key key;
-	int sync = IS_DIRSYNC(host);
-	u8 hash_host[UBIFS_HASH_ARR_SZ];
-	u8 hash[UBIFS_HASH_ARR_SZ];
-
-	dbg_jnl("ino %lu, ino %lu", host->i_ino, inode->i_ino);
-	ubifs_assert(c, inode->i_nlink > 0);
-	ubifs_assert(c, mutex_is_locked(&host_ui->ui_mutex));
-
-	len1 = UBIFS_INO_NODE_SZ + host_ui->data_len;
-	len2 = UBIFS_INO_NODE_SZ + ubifs_inode(inode)->data_len;
-	aligned_len1 = ALIGN(len1, 8);
-	aligned_len = aligned_len1 + ALIGN(len2, 8);
-
-	aligned_len += ubifs_auth_node_sz(c);
-
-	ino = kzalloc(aligned_len, GFP_NOFS);
-	if (!ino)
-		return -ENOMEM;
-
-	/* Make reservation before allocating sequence numbers */
-	err = make_reservation(c, BASEHD, aligned_len);
-	if (err)
-		goto out_free;
-
-	pack_inode(c, ino, host, 0);
-	err = ubifs_node_calc_hash(c, ino, hash_host);
-	if (err)
-		goto out_release;
-	pack_inode(c, (void *)ino + aligned_len1, inode, 1);
-	err = ubifs_node_calc_hash(c, (void *)ino + aligned_len1, hash);
-	if (err)
-		goto out_release;
-
-	err = write_head(c, BASEHD, ino, aligned_len, &lnum, &offs, 0);
-	if (!sync && !err) {
-		struct ubifs_wbuf *wbuf = &c->jheads[BASEHD].wbuf;
-
-		ubifs_wbuf_add_ino_nolock(wbuf, host->i_ino);
-		ubifs_wbuf_add_ino_nolock(wbuf, inode->i_ino);
-	}
-	release_head(c, BASEHD);
-	if (err)
-		goto out_ro;
-
-	ubifs_add_auth_dirt(c, lnum);
-
-	ino_key_init(c, &key, host->i_ino);
-	err = ubifs_tnc_add(c, &key, lnum, offs, len1, hash_host);
-	if (err)
-		goto out_ro;
-
-	ino_key_init(c, &key, inode->i_ino);
-	err = ubifs_tnc_add(c, &key, lnum, offs + aligned_len1, len2, hash);
-	if (err)
-		goto out_ro;
-
-	finish_reservation(c);
-	spin_lock(&host_ui->ui_lock);
-	host_ui->synced_i_size = host_ui->ui_size;
-	spin_unlock(&host_ui->ui_lock);
-	mark_inode_clean(c, host_ui);
-	kfree(ino);
-	return 0;
-
-out_release:
-	release_head(c, BASEHD);
-out_ro:
-	ubifs_ro_mode(c, err);
-	finish_reservation(c);
-out_free:
-	kfree(ino);
-	return err;
-}
-