From: "Darrick J. Wong" <darrick.wong@oracle.com>
To: Christoph Hellwig <hch@lst.de>
Cc: linux-xfs@vger.kernel.org
Subject: Re: [PATCH 14/34] xfs: make the xfs_dir3_icfree_hdr available to xfs_dir2_node_addname_int
Date: Mon, 4 Nov 2019 12:25:23 -0800 [thread overview]
Message-ID: <20191104202523.GS4153244@magnolia> (raw)
In-Reply-To: <20191101220719.29100-15-hch@lst.de>
On Fri, Nov 01, 2019 at 03:06:59PM -0700, Christoph Hellwig wrote:
> Return the xfs_dir3_icfree_hdr used by the helpers called from
> xfs_dir2_node_addname_int to the main function to prepare for the
> next round of changes.
How does this help? Is this purely to reduce stack usage? Or will we
use this later to skip some xfs_dir2_free_hdr_from_disk calls?
--D
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
> fs/xfs/libxfs/xfs_dir2_node.c | 42 +++++++++++++++++------------------
> 1 file changed, 20 insertions(+), 22 deletions(-)
>
> diff --git a/fs/xfs/libxfs/xfs_dir2_node.c b/fs/xfs/libxfs/xfs_dir2_node.c
> index 26032eba1e32..d400243c9556 100644
> --- a/fs/xfs/libxfs/xfs_dir2_node.c
> +++ b/fs/xfs/libxfs/xfs_dir2_node.c
> @@ -1660,14 +1660,13 @@ xfs_dir2_node_add_datablk(
> xfs_dir2_db_t *dbno,
> struct xfs_buf **dbpp,
> struct xfs_buf **fbpp,
> + struct xfs_dir3_icfree_hdr *hdr,
> int *findex)
> {
> struct xfs_inode *dp = args->dp;
> struct xfs_trans *tp = args->trans;
> struct xfs_mount *mp = dp->i_mount;
> - struct xfs_dir3_icfree_hdr freehdr;
> struct xfs_dir2_data_free *bf;
> - struct xfs_dir2_data_hdr *hdr;
> struct xfs_dir2_free *free = NULL;
> xfs_dir2_db_t fbno;
> struct xfs_buf *fbp;
> @@ -1730,25 +1729,25 @@ xfs_dir2_node_add_datablk(
> return error;
> free = fbp->b_addr;
> bests = dp->d_ops->free_bests_p(free);
> - xfs_dir2_free_hdr_from_disk(mp, &freehdr, free);
> + xfs_dir2_free_hdr_from_disk(mp, hdr, free);
>
> /* Remember the first slot as our empty slot. */
> - freehdr.firstdb = (fbno - xfs_dir2_byte_to_db(args->geo,
> + hdr->firstdb = (fbno - xfs_dir2_byte_to_db(args->geo,
> XFS_DIR2_FREE_OFFSET)) *
> dp->d_ops->free_max_bests(args->geo);
> } else {
> free = fbp->b_addr;
> bests = dp->d_ops->free_bests_p(free);
> - xfs_dir2_free_hdr_from_disk(mp, &freehdr, free);
> + xfs_dir2_free_hdr_from_disk(mp, hdr, free);
> }
>
> /* Set the freespace block index from the data block number. */
> *findex = dp->d_ops->db_to_fdindex(args->geo, *dbno);
>
> /* Extend the freespace table if the new data block is off the end. */
> - if (*findex >= freehdr.nvalid) {
> + if (*findex >= hdr->nvalid) {
> ASSERT(*findex < dp->d_ops->free_max_bests(args->geo));
> - freehdr.nvalid = *findex + 1;
> + hdr->nvalid = *findex + 1;
> bests[*findex] = cpu_to_be16(NULLDATAOFF);
> }
>
> @@ -1757,14 +1756,13 @@ xfs_dir2_node_add_datablk(
> * true) then update the header.
> */
> if (bests[*findex] == cpu_to_be16(NULLDATAOFF)) {
> - freehdr.nused++;
> - xfs_dir2_free_hdr_to_disk(mp, fbp->b_addr, &freehdr);
> + hdr->nused++;
> + xfs_dir2_free_hdr_to_disk(mp, fbp->b_addr, hdr);
> xfs_dir2_free_log_header(args, fbp);
> }
>
> /* Update the freespace value for the new block in the table. */
> - hdr = dbp->b_addr;
> - bf = dp->d_ops->data_bestfree_p(hdr);
> + bf = dp->d_ops->data_bestfree_p(dbp->b_addr);
> bests[*findex] = bf[0].length;
>
> *dbpp = dbp;
> @@ -1778,10 +1776,10 @@ xfs_dir2_node_find_freeblk(
> struct xfs_da_state_blk *fblk,
> xfs_dir2_db_t *dbnop,
> struct xfs_buf **fbpp,
> + struct xfs_dir3_icfree_hdr *hdr,
> int *findexp,
> int length)
> {
> - struct xfs_dir3_icfree_hdr freehdr;
> struct xfs_dir2_free *free = NULL;
> struct xfs_inode *dp = args->dp;
> struct xfs_trans *tp = args->trans;
> @@ -1808,13 +1806,12 @@ xfs_dir2_node_find_freeblk(
> if (findex >= 0) {
> /* caller already found the freespace for us. */
> bests = dp->d_ops->free_bests_p(free);
> - xfs_dir2_free_hdr_from_disk(dp->i_mount, &freehdr,
> - free);
> + xfs_dir2_free_hdr_from_disk(dp->i_mount, hdr, free);
>
> - ASSERT(findex < freehdr.nvalid);
> + ASSERT(findex < hdr->nvalid);
> ASSERT(be16_to_cpu(bests[findex]) != NULLDATAOFF);
> ASSERT(be16_to_cpu(bests[findex]) >= length);
> - dbno = freehdr.firstdb + findex;
> + dbno = hdr->firstdb + findex;
> goto found_block;
> }
>
> @@ -1858,13 +1855,13 @@ xfs_dir2_node_find_freeblk(
>
> free = fbp->b_addr;
> bests = dp->d_ops->free_bests_p(free);
> - xfs_dir2_free_hdr_from_disk(dp->i_mount, &freehdr, free);
> + xfs_dir2_free_hdr_from_disk(dp->i_mount, hdr, free);
>
> /* Scan the free entry array for a large enough free space. */
> - for (findex = freehdr.nvalid - 1; findex >= 0; findex--) {
> + for (findex = hdr->nvalid - 1; findex >= 0; findex--) {
> if (be16_to_cpu(bests[findex]) != NULLDATAOFF &&
> be16_to_cpu(bests[findex]) >= length) {
> - dbno = freehdr.firstdb + findex;
> + dbno = hdr->firstdb + findex;
> goto found_block;
> }
> }
> @@ -1898,6 +1895,7 @@ xfs_dir2_node_addname_int(
> struct xfs_dir2_free *free = NULL; /* freespace block structure */
> struct xfs_trans *tp = args->trans;
> struct xfs_inode *dp = args->dp;
> + struct xfs_dir3_icfree_hdr freehdr;
> struct xfs_buf *dbp; /* data block buffer */
> struct xfs_buf *fbp; /* freespace buffer */
> xfs_dir2_data_aoff_t aoff;
> @@ -1912,8 +1910,8 @@ xfs_dir2_node_addname_int(
> __be16 *bests;
>
> length = dp->d_ops->data_entsize(args->namelen);
> - error = xfs_dir2_node_find_freeblk(args, fblk, &dbno, &fbp, &findex,
> - length);
> + error = xfs_dir2_node_find_freeblk(args, fblk, &dbno, &fbp, &freehdr,
> + &findex, length);
> if (error)
> return error;
>
> @@ -1935,7 +1933,7 @@ xfs_dir2_node_addname_int(
> /* we're going to have to log the free block index later */
> logfree = 1;
> error = xfs_dir2_node_add_datablk(args, fblk, &dbno, &dbp, &fbp,
> - &findex);
> + &freehdr, &findex);
> } else {
> /* Read the data block in. */
> error = xfs_dir3_data_read(tp, dp,
> --
> 2.20.1
>
next prev parent reply other threads:[~2019-11-04 20:27 UTC|newest]
Thread overview: 81+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-01 22:06 remove m_dirops Christoph Hellwig
2019-11-01 22:06 ` [PATCH 01/34] xfs: move incore structures out of xfs_da_format.h Christoph Hellwig
2019-11-04 18:21 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 02/34] xfs: refactor btree node scrubbing Christoph Hellwig
2019-11-04 18:36 ` Darrick J. Wong
2019-11-05 1:35 ` Christoph Hellwig
2019-11-01 22:06 ` [PATCH 03/34] xfs: devirtualize ->node_hdr_from_disk Christoph Hellwig
2019-11-04 18:37 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 04/34] xfs: devirtualize ->node_hdr_to_disk Christoph Hellwig
2019-11-04 18:39 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 05/34] xfs: add a btree entries pointer to struct xfs_da3_icnode_hdr Christoph Hellwig
2019-11-04 19:52 ` Darrick J. Wong
2019-11-05 1:38 ` Christoph Hellwig
2019-11-05 1:58 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 06/34] xfs: move the node header size to struct xfs_da_geometry Christoph Hellwig
2019-11-04 19:58 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 07/34] xfs: devirtualize ->leaf_hdr_from_disk Christoph Hellwig
2019-11-04 19:59 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 08/34] xfs: devirtualize ->leaf_hdr_to_disk Christoph Hellwig
2019-11-04 20:00 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 09/34] xfs: add a entries pointer to struct xfs_dir3_icleaf_hdr Christoph Hellwig
2019-11-04 20:04 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 10/34] xfs: move the dir2 leaf header size to struct xfs_da_geometry Christoph Hellwig
2019-11-04 20:05 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 11/34] xfs: move the max dir2 leaf entries count " Christoph Hellwig
2019-11-04 20:07 ` Darrick J. Wong
2019-11-05 1:42 ` Christoph Hellwig
2019-11-05 1:58 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 12/34] xfs: devirtualize ->free_hdr_from_disk Christoph Hellwig
2019-11-04 20:08 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 13/34] xfs: devirtualize ->free_hdr_to_disk Christoph Hellwig
2019-11-04 20:08 ` Darrick J. Wong
2019-11-01 22:06 ` [PATCH 14/34] xfs: make the xfs_dir3_icfree_hdr available to xfs_dir2_node_addname_int Christoph Hellwig
2019-11-04 20:25 ` Darrick J. Wong [this message]
2019-11-05 1:52 ` Christoph Hellwig
2019-11-05 1:59 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 15/34] xfs: add a bests pointer to struct xfs_dir3_icfree_hdr Christoph Hellwig
2019-11-04 20:21 ` Darrick J. Wong
2019-11-05 1:44 ` Christoph Hellwig
2019-11-05 2:05 ` Darrick J. Wong
2019-11-05 16:44 ` Christoph Hellwig
2019-11-01 22:07 ` [PATCH 16/34] xfs: move the dir2 free header size to struct xfs_da_geometry Christoph Hellwig
2019-11-04 20:22 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 17/34] xfs: move the max dir2 free bests count " Christoph Hellwig
2019-11-04 20:23 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 18/34] xfs: devirtualize ->db_to_fdb and ->db_to_fdindex Christoph Hellwig
2019-11-04 20:26 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 19/34] xfs: devirtualize ->sf_get_parent_ino and ->sf_put_parent_ino Christoph Hellwig
2019-11-04 20:26 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 20/34] xfs: devirtualize ->sf_entsize and ->sf_nextentry Christoph Hellwig
2019-11-04 20:28 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 21/34] xfs: devirtualize ->sf_get_ino and ->sf_put_ino Christoph Hellwig
2019-11-04 20:33 ` Darrick J. Wong
2019-11-07 1:05 ` Darrick J. Wong
2019-11-07 8:29 ` Christoph Hellwig
2019-11-01 22:07 ` [PATCH 22/34] xfs: devirtualize ->sf_get_ftype and ->sf_put_ftype Christoph Hellwig
2019-11-04 20:34 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 23/34] xfs: remove the ->data_dot_entry_p and ->data_dotdot_entry_p methods Christoph Hellwig
2019-11-04 20:37 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 24/34] xfs: remove the unused ->data_first_entry_p method Christoph Hellwig
2019-11-04 20:38 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 25/34] xfs: remove the ->data_entry_entry_p method Christoph Hellwig
2019-11-04 20:41 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 26/34] xfs: devirtualize ->data_entsize Christoph Hellwig
2019-11-04 20:45 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 27/34] xfs: devirtualize ->data_entry_tag_p Christoph Hellwig
2019-11-04 20:45 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 28/34] xfs: move the dir2 data block fixed offsets to struct xfs_da_geometry Christoph Hellwig
2019-11-04 20:46 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 29/34] xfs: cleanup xfs_dir2_data_entsize Christoph Hellwig
2019-11-04 20:48 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 30/34] xfs: devirtualize ->data_bestfree_p Christoph Hellwig
2019-11-04 20:49 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 31/34] xfs: devirtualize ->data_get_ftype and ->data_put_ftype Christoph Hellwig
2019-11-04 20:50 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 32/34] xfs: remove the now unused dir ops infrastructure Christoph Hellwig
2019-11-04 20:50 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 33/34] xfs: merge xfs_dir2_data_freescan and xfs_dir2_data_freescan_int Christoph Hellwig
2019-11-04 20:51 ` Darrick J. Wong
2019-11-01 22:07 ` [PATCH 34/34] xfs: always pass a valid hdr to xfs_dir3_leaf_check_int Christoph Hellwig
2019-11-04 20:53 ` Darrick J. Wong
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=20191104202523.GS4153244@magnolia \
--to=darrick.wong@oracle.com \
--cc=hch@lst.de \
--cc=linux-xfs@vger.kernel.org \
/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).