xfs
[Top] [All Lists]

Re: [PATCH] xfs: put xfs_trans_t on diet

To: Nathan Scott <nathans@xxxxxxx>, xfs@xxxxxxxxxxx
Subject: Re: [PATCH] xfs: put xfs_trans_t on diet
From: Eric Sandeen <sandeen@xxxxxxx>
Date: Thu, 22 Jun 2006 08:44:23 -0500
In-reply-to: <20060622192952.A1142412@wobbly.melbourne.sgi.com>
References: <20060622074051.GA7697@martell.zuzino.mipt.ru> <20060622192952.A1142412@wobbly.melbourne.sgi.com>
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Thunderbird 1.5.0.4 (Macintosh/20060516)
Nathan Scott wrote:

On Thu, Jun 22, 2006 at 11:40:51AM +0400, Alexey Dobriyan wrote:
* remove ->t_forw, ->t_back -- unused
* ->t_ag_freeblks_delta, ->t_ag_flist_delta, ->t_ag_btree_delta
are debugging aid -- wrap them in everyone's favourite way.

Thanks Alexey, looks good - applied.

--
Nathan

Then you'll probably also want this for the cvs tree:

Index: xfs-GUT-clean/xfsidbg.c
===================================================================
--- xfs-GUT-clean.orig/xfsidbg.c
+++ xfs-GUT-clean/xfsidbg.c
@@ -7412,9 +7412,11 @@ xfsidbg_xtp(xfs_trans_t *tp)
                tp->t_fdblocks_delta, tp->t_res_fdblocks_delta);
        kdb_printf("rt delta %ld res rt delta %ld\n",
                tp->t_frextents_delta, tp->t_res_frextents_delta);
+#ifdef DEBUG
        kdb_printf("ag freeblks delta %ld ag flist delta %ld ag btree delta 
%ld\n",
                tp->t_ag_freeblks_delta, tp->t_ag_flist_delta,
                tp->t_ag_btree_delta);
+#endif
        kdb_printf("dblocks delta %ld agcount delta %ld imaxpct delta %ld\n",
                tp->t_dblocks_delta, tp->t_agcount_delta, tp->t_imaxpct_delta);
        kdb_printf("rextsize delta %ld rbmblocks delta %ld\n",


<Prev in Thread] Current Thread [Next in Thread>