From: Sascha Hauer <s.hauer@pengutronix.de>
To: linux-mtd@lists.infradead.org
Cc: David Gstir <david@sigma-star.at>,
Richard Weinberger <richard@nod.at>,
kernel@pengutronix.de, linux-kernel@vger.kernel.org,
Sascha Hauer <s.hauer@pengutronix.de>
Subject: [PATCH 07/25] ubifs: drop write_node
Date: Fri, 7 Sep 2018 14:36:28 +0200 [thread overview]
Message-ID: <20180907123646.12688-8-s.hauer@pengutronix.de> (raw)
In-Reply-To: <20180907123646.12688-1-s.hauer@pengutronix.de>
write_node() is used only once and can easily be replaced with calls
to ubifs_prepare_node()/write_head() which makes the code a bit shorter.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
fs/ubifs/journal.c | 39 +++++----------------------------------
1 file changed, 5 insertions(+), 34 deletions(-)
diff --git a/fs/ubifs/journal.c b/fs/ubifs/journal.c
index 802565a17733..de7ac9e7acce 100644
--- a/fs/ubifs/journal.c
+++ b/fs/ubifs/journal.c
@@ -228,36 +228,6 @@ static int reserve_space(struct ubifs_info *c, int jhead, int len)
return err;
}
-/**
- * write_node - write node to a journal head.
- * @c: UBIFS file-system description object
- * @jhead: journal head
- * @node: node to write
- * @len: node length
- * @lnum: LEB number written is returned here
- * @offs: offset written is returned here
- *
- * This function writes a node to reserved space of journal head @jhead.
- * Returns zero in case of success and a negative error code in case of
- * failure.
- */
-static int write_node(struct ubifs_info *c, int jhead, void *node, int len,
- int *lnum, int *offs)
-{
- struct ubifs_wbuf *wbuf = &c->jheads[jhead].wbuf;
-
- ubifs_assert(c, jhead != GCHD);
-
- *lnum = c->jheads[jhead].wbuf.lnum;
- *offs = c->jheads[jhead].wbuf.offs + c->jheads[jhead].wbuf.used;
-
- dbg_jnl("jhead %s, LEB %d:%d, len %d",
- dbg_jhead(jhead), *lnum, *offs, len);
- ubifs_prepare_node(c, node, len, 0);
-
- return ubifs_wbuf_write_nolock(wbuf, node, len);
-}
-
/**
* write_head - write data to a journal head.
* @c: UBIFS file-system description object
@@ -268,9 +238,9 @@ static int write_node(struct ubifs_info *c, int jhead, void *node, int len,
* @offs: offset written is returned here
* @sync: non-zero if the write-buffer has to by synchronized
*
- * This function is the same as 'write_node()' but it does not assume the
- * buffer it is writing is a node, so it does not prepare it (which means
- * initializing common header and calculating CRC).
+ * This function writes data to the reserved space of journal head @jhead.
+ * Returns zero in case of success and a negative error code in case of
+ * failure.
*/
static int write_head(struct ubifs_info *c, int jhead, void *buf, int len,
int *lnum, int *offs, int sync)
@@ -764,7 +734,8 @@ int ubifs_jnl_write_data(struct ubifs_info *c, const struct inode *inode,
if (err)
goto out_free;
- err = write_node(c, DATAHD, data, dlen, &lnum, &offs);
+ ubifs_prepare_node(c, data, dlen, 0);
+ err = write_head(c, DATAHD, data, dlen, &lnum, &offs, 0);
if (err)
goto out_release;
ubifs_wbuf_add_ino_nolock(&c->jheads[DATAHD].wbuf, key_inum(c, key));
--
2.18.0
next prev parent reply other threads:[~2018-09-07 12:40 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-07 12:36 [PATCH v2 00/25] UBIFS authentication support Sascha Hauer
2018-09-07 12:36 ` [PATCH 01/25] ARM: imx_v6_v7_defconfig: update Sascha Hauer
2018-09-07 12:36 ` [PATCH 02/25] ubifs: refactor create_default_filesystem() Sascha Hauer
2018-09-07 12:36 ` [PATCH 03/25] ubifs: pass ubifs_zbranch to try_read_node() Sascha Hauer
2018-09-07 12:36 ` [PATCH 04/25] ubifs: pass ubifs_zbranch to read_znode() Sascha Hauer
2018-09-07 12:36 ` [PATCH 05/25] ubifs: export pnode_lookup as ubifs_pnode_lookup Sascha Hauer
2018-09-07 12:36 ` [PATCH 06/25] ubifs: implement ubifs_lpt_lookup using ubifs_pnode_lookup Sascha Hauer
2018-09-07 12:36 ` Sascha Hauer [this message]
2018-09-07 12:36 ` [PATCH 08/25] ubifs: Store read superblock node Sascha Hauer
2018-09-07 12:36 ` [PATCH 09/25] ubifs: Format changes for authentication support Sascha Hauer
2018-09-07 12:36 ` [PATCH 10/25] ubifs: add separate functions to init/crc a node Sascha Hauer
2018-09-07 12:36 ` [PATCH 11/25] ubifs: add helper functions for authentication support Sascha Hauer
2018-09-07 12:36 ` [PATCH 12/25] ubifs: Create functions to embed a HMAC in a node Sascha Hauer
2018-09-07 12:36 ` [PATCH 13/25] ubifs: Add hashes to the tree node cache Sascha Hauer
2018-09-07 12:36 ` [PATCH 14/25] ubifs: authentication: Add hashes to index nodes Sascha Hauer
2018-09-07 12:36 ` [PATCH 15/25] ubifs: Add authentication nodes to journal Sascha Hauer
2018-09-07 12:36 ` [PATCH 16/25] ubifs: Add auth nodes to garbage collector journal head Sascha Hauer
2018-09-07 12:36 ` [PATCH 17/25] ubifs: authenticate replayed journal Sascha Hauer
2018-09-07 12:36 ` [PATCH 18/25] ubifs: authentication: authenticate LPT Sascha Hauer
2018-09-07 12:36 ` [PATCH 19/25] ubfis: authentication: authenticate master node Sascha Hauer
2018-09-07 12:36 ` [PATCH 20/25] ubifs: Create hash for default LPT Sascha Hauer
2018-09-07 12:36 ` [PATCH 21/25] ubifs: authentication: Authenticate super block node Sascha Hauer
2018-09-07 12:36 ` [PATCH 22/25] ubifs: Add hashes and HMACs to default filesystem Sascha Hauer
2018-09-07 12:36 ` [PATCH 23/25] ubifs: do not update inode size in-place in authenticated mode Sascha Hauer
2018-09-07 12:36 ` [PATCH 24/25] ubifs: Enable authentication support Sascha Hauer
2018-09-07 12:36 ` [PATCH 25/25] Documentation: ubifs: Add authentication whitepaper Sascha Hauer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180907123646.12688-8-s.hauer@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=david@sigma-star.at \
--cc=kernel@pengutronix.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=richard@nod.at \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).