On Tue, Jun 28, 2022 at 01:51:00PM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <djwong@xxxxxxxxxx>
>
> Right now, we don't make it easy to spot the transaction reservation
> used to compute the minimum log size in userspace:
>
> # xfs_db -c logres /dev/sda
> type 0 logres 168184 logcount 5 flags 0x4
> ...
> type 25 logres 760 logcount 0 flags 0x0
> type -1 logres 547200 logcount 8 flags 0x4
>
> Type "-1" doesn't communicate the purpose at all, it just looks like a
> math error. Help out the user a bit by printing more information:
>
> minlogsize logres 547200 logcount 8
>
> Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx>
> ---
> db/logformat.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
>
> diff --git a/db/logformat.c b/db/logformat.c
> index 38b0af11..5edaa549 100644
> --- a/db/logformat.c
> +++ b/db/logformat.c
> @@ -160,8 +160,10 @@ logres_f(
> end_res = (struct xfs_trans_res *)(M_RES(mp) + 1);
> for (i = 0; res < end_res; i++, res++)
> print_logres(i, res);
> +
> libxfs_log_get_max_trans_res(mp, &resv);
> - print_logres(-1, &resv);
> + dbprintf(_("minlogsize logres %u logcount %d\n"),
> + resv.tr_logres, resv.tr_logcount);
Looks good.
Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
--
Dave Chinner
david@xxxxxxxxxxxxx
