On Mon, Nov 27, 2017 at 03:24:31PM -0500, Brian Foster wrote: > The transaction dump code displays the content and reservation > consumption of a particular transaction in the event of an overrun. > It currently displays the reservation associated with the > transaction ticket, but not the original reservation attached to the > transaction. > > The latter value reflects the original transaction reservation > calculation before additional reservation overhead is assigned, such > as for the CIL context header and potential split region headers. > > Update xlog_print_trans() to also print the original transaction > reservation in the event of overrun. This provides a reference point > to identify how much reservation overhead was added to a particular > ticket by xfs_log_calc_unit_res(). > > Signed-off-by: Brian Foster <bfoster@xxxxxxxxxx> > --- > fs/xfs/xfs_log.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/fs/xfs/xfs_log.c b/fs/xfs/xfs_log.c > index 38d4227895ae..3a37fa1ed25e 100644 > --- a/fs/xfs/xfs_log.c > +++ b/fs/xfs/xfs_log.c > @@ -2117,7 +2117,9 @@ xlog_print_trans( > > /* dump core transaction and ticket info */ > xfs_warn(mp, "transaction summary:"); > - xfs_warn(mp, " flags = 0x%x", tp->t_flags); > + xfs_warn(mp, " log res = %d", tp->t_log_res); > + xfs_warn(mp, " log count = %d", tp->t_log_count); > + xfs_warn(mp, " flags = 0x%x", tp->t_flags); > > xlog_print_tic_res(mp, tp->t_ticket); Looks fine. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
