Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a41b96e4 authored by Subodh Nijsure's avatar Subodh Nijsure Committed by Matt Wagantall
Browse files

UBIFS: add ubifs_err() to print error reason 



This patch adds ubifs_err() output to some error paths to tell the user
what's going on.

Artem: improve the messages, rename too long variable

Change-Id: Ib6ed7a8696bf1a18fc6d11389406621193ac8219
Signed-off-by: default avatarSubodh Nijsure <snijsure@grid-net.com>
Signed-off-by: default avatarMarc Kleine-Budde <mkl@pengutronix.de>
Signed-off-by: default avatarBen Shelton <ben.shelton@ni.com>
Acked-by: default avatarBrad Mouring <brad.mouring@ni.com>
Acked-by: default avatarTerry Wilcox <terry.wilcox@ni.com>
Acked-by: default avatarGratian Crisan <gratian.crisan@ni.com>
Signed-off-by: default avatarArtem Bityutskiy <artem.bityutskiy@linux.intel.com>
Git-commit: fee1756d80c24d5a3171cb9f76d612e512439dd2
Git-repo: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git


Signed-off-by: default avatarNikhilesh Reddy <reddyn@codeaurora.org>
parent a0ac03cd

fs/ubifs/xattr.c

+13 −4
Original line number Diff line number Diff line
@@ -100,24 +100,30 @@ static const struct file_operations empty_fops; 
static int create_xattr(struct ubifs_info *c, struct inode *host,

			const struct qstr *nm, const void *value, int size)

{

	int err;

	int err, names_len;

	struct inode *inode;

	struct ubifs_inode *ui, *host_ui = ubifs_inode(host);

	struct ubifs_budget_req req = { .new_ino = 1, .new_dent = 1,

				.new_ino_d = ALIGN(size, 8), .dirtied_ino = 1,

				.dirtied_ino_d = ALIGN(host_ui->data_len, 8) };



	if (host_ui->xattr_cnt >= MAX_XATTRS_PER_INODE)

	if (host_ui->xattr_cnt >= MAX_XATTRS_PER_INODE) {

		ubifs_err("inode %lu already has too many xattrs (%d), cannot create more",

			  host->i_ino, host_ui->xattr_cnt);

		return -ENOSPC;

	}

	/*

	 * Linux limits the maximum size of the extended attribute names list

	 * to %XATTR_LIST_MAX. This means we should not allow creating more

	 * extended attributes if the name list becomes larger. This limitation

	 * is artificial for UBIFS, though.

	 */

	if (host_ui->xattr_names + host_ui->xattr_cnt +

					nm->len + 1 > XATTR_LIST_MAX)

	names_len = host_ui->xattr_names + host_ui->xattr_cnt + nm->len + 1;

	if (names_len > XATTR_LIST_MAX) {

		ubifs_err("cannot add one more xattr name to inode %lu, total names length would become %d, max. is %d",

			  host->i_ino, names_len, XATTR_LIST_MAX);

		return -ENOSPC;

	}



	err = ubifs_budget_space(c, &req);

	if (err)
@@ -651,5 +657,8 @@ int ubifs_init_security(struct inode *dentry, struct inode *inode, 
					   &init_xattrs, 0);

	mutex_unlock(&inode->i_mutex);



	if (err)

		ubifs_err("cannot initialize security for inode %lu, error %d",

			  inode->i_ino, err);

	return err;

}