| To: | Alex Elder <aelder@xxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 4/5] [PATCH] xfs: simplify buffer to transaction matching |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Thu, 29 Apr 2010 09:38:07 -0400 |
| Cc: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1272548122.3221.77.camel@doink> |
| References: | <20100418001041.865247520@xxxxxxxxxxxxxxxxxxxxxx> <20100418001058.677429475@xxxxxxxxxxxxxxxxxxxxxx> <1272548122.3221.77.camel@doink> |
| User-agent: | Mutt/1.5.19 (2009-01-05) |
On Thu, Apr 29, 2010 at 08:35:22AM -0500, Alex Elder wrote: > Dave suggested using the xfs_trans_*_item() routines (which maybe > could be renamed xfs_trans_item_*() someday). That is the right > thing to do, but not necessary at this point. In fact I prefer > this smaller change anyway, since it clearly just transforms > the original xfs_trans_buf_item_match_all() code directly to > the new result. The switch to use xfs_trans_*_item() can (and > should) happen as a follow-on patch. Yes, I'll prepare an incremental patch in the next days. |
| Previous by Date: | Re: [PATCH 4/5] [PATCH] xfs: simplify buffer to transaction matching, Alex Elder |
|---|---|
| Next by Date: | Re: [PATCH 01/11] xfs: limit xfs_imap_to_bmap to a single mapping, Alex Elder |
| Previous by Thread: | Re: [PATCH 4/5] [PATCH] xfs: simplify buffer to transaction matching, Alex Elder |
| Next by Thread: | [PATCH 5/5] xfs: remove dead XFS_LOUD_RECOVERY code, Christoph Hellwig |
| Indexes: | [Date] [Thread] [Top] [All Lists] |