From: Christoph Hellwig <hch@infradead.org> To: "Darrick J. Wong" <darrick.wong@oracle.com> Cc: linux-xfs@vger.kernel.org, hch@infradead.org Subject: Re: [PATCH 9/9] xfs: make xfs_btree_get_buf functions return an error code Date: Thu, 16 Jan 2020 08:33:55 -0800 Message-ID: <20200116163355.GI3802@infradead.org> (raw) In-Reply-To: <157910787821.2028217.9307411154179566922.stgit@magnolia> > - if (XFS_IS_CORRUPT(args->mp, !bp)) { > - error = -EFSCORRUPTED; > + error = xfs_btree_get_bufs(args->mp, args->tp, args->agno, > + fbno, &bp); > + if (error) Should we keep the XFS_IS_CORRUPT checks in some form? Not sure they matter all that much, though. > ASSERT(fsbno != NULLFSBLOCK); > d = XFS_FSB_TO_DADDR(mp, fsbno); > - error = xfs_trans_get_buf(tp, mp->m_ddev_targp, d, mp->m_bsize, 0, &bp); > - if (error) > - return NULL; > - return bp; > + return xfs_trans_get_buf(tp, mp->m_ddev_targp, d, mp->m_bsize, 0, bpp); Maybe kill the local variable while you're at it? > ASSERT(agno != NULLAGNUMBER); > ASSERT(agbno != NULLAGBLOCK); > d = XFS_AGB_TO_DADDR(mp, agno, agbno); > - error = xfs_trans_get_buf(tp, mp->m_ddev_targp, d, mp->m_bsize, 0, &bp); > - if (error) > - return NULL; > - return bp; > + return xfs_trans_get_buf(tp, mp->m_ddev_targp, d, mp->m_bsize, 0, bpp); Same here.
next prev parent reply index Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-01-15 17:03 [PATCH 0/9] xfs: make buffer functions return error codes Darrick J. Wong 2020-01-15 17:03 ` [PATCH 1/9] xfs: make xfs_buf_alloc return an error code Darrick J. Wong 2020-01-16 16:13 ` Christoph Hellwig 2020-01-15 17:03 ` [PATCH 2/9] xfs: make xfs_buf_read " Darrick J. Wong 2020-01-16 16:15 ` Christoph Hellwig 2020-01-16 22:30 ` Darrick J. Wong 2020-01-15 17:03 ` [PATCH 3/9] xfs: make xfs_buf_get " Darrick J. Wong 2020-01-16 16:16 ` Christoph Hellwig 2020-01-16 22:30 ` Darrick J. Wong 2020-01-15 17:04 ` [PATCH 4/9] xfs: make xfs_buf_get_uncached " Darrick J. Wong 2020-01-16 16:17 ` Christoph Hellwig 2020-01-15 17:04 ` [PATCH 5/9] xfs: make xfs_buf_read_map " Darrick J. Wong 2020-01-16 16:21 ` Christoph Hellwig 2020-01-16 23:26 ` Darrick J. Wong 2020-01-15 17:04 ` [PATCH 6/9] xfs: make xfs_buf_get_map " Darrick J. Wong 2020-01-16 16:28 ` Christoph Hellwig 2020-01-16 22:33 ` Darrick J. Wong 2020-01-15 17:04 ` [PATCH 7/9] xfs: make xfs_trans_get_buf_map " Darrick J. Wong 2020-01-16 16:29 ` Christoph Hellwig 2020-01-15 17:04 ` [PATCH 8/9] xfs: make xfs_trans_get_buf " Darrick J. Wong 2020-01-16 16:31 ` Christoph Hellwig 2020-01-15 17:04 ` [PATCH 9/9] xfs: make xfs_btree_get_buf functions " Darrick J. Wong 2020-01-16 16:33 ` Christoph Hellwig [this message] 2020-01-16 16:43 ` Christoph Hellwig 2020-01-16 22:40 ` 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=20200116163355.GI3802@infradead.org \ --to=hch@infradead.org \ --cc=darrick.wong@oracle.com \ --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
Linux-XFS Archive on lore.kernel.org Archives are clonable: git clone --mirror https://lore.kernel.org/linux-xfs/0 linux-xfs/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 linux-xfs linux-xfs/ https://lore.kernel.org/linux-xfs \ linux-xfs@vger.kernel.org public-inbox-index linux-xfs Example config snippet for mirrors Newsgroup available over NNTP: nntp://nntp.lore.kernel.org/org.kernel.vger.linux-xfs AGPL code for this site: git clone https://public-inbox.org/public-inbox.git