On Tue, Dec 15, 2020 at 08:42:08AM +0800, Yafang Shao wrote:
> On Tue, Dec 15, 2020 at 5:08 AM Dave Chinner <david@xxxxxxxxxxxxx> wrote:
> > On Sun, Dec 13, 2020 at 05:09:02PM +0800, Yafang Shao wrote:
> > > On Thu, Dec 10, 2020 at 3:52 AM Darrick J. Wong <darrick.wong@xxxxxxxxxx> wrote:
> > > > On Wed, Dec 09, 2020 at 09:11:45PM +0800, Yafang Shao wrote:
> > static inline void
> > xfs_trans_context_clear(struct xfs_trans *tp)
> > {
> > /*
> > * If xfs_trans_context_swap() handed the NOFS context to a
> > * new transaction we do not clear the context here.
> > */
> > if (current->journal_info != tp)
>
> current->journal_info hasn't been used in patch #3, that will make
> patch #3 a little more complex.
> We have to do some workaround in patch #3. I will think about it.
What I wrote is how the function should look at the end of the patch
series. Do not add the current->journal_info parts of it until the
patch that introduces the current->journal_info tracking.
-Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
