On Tue, Jun 9, 2020 at 4:15 PM Bob Peterson <[email protected]> wrote: > ----- Original Message ----- > > I'm not sure quite what the aim is here... are you sure that it is ok to > > move something to the AIL list if there has been a withdrawal? If the > > log flush has not completed correctly then we should not be moving > > anything to the AIL lists I think, > > > > Steve. > > Yes, I'm sure it's okay in this case. We only add it temporarily to the > ail1 list so that function ail_drain() finds and removes it like the rest. > > I only coded it this way because Andreas didn't like my somewhat longer > previous implementation, which follows.
Note that this previous version fails to free the gfs2_bufdata objects attached to the transaction if the transaction isn't on one of the ail lists. Andreas
