| To: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: TAKE 981498 - kill bhv_vnode_t |
| From: | Lachlan McIlroy <lachlan@xxxxxxx> |
| Date: | Tue, 29 Jul 2008 11:32:36 +1000 |
| Cc: | sgi.bugs.xfs@xxxxxxxxxxxx, xfs@xxxxxxxxxxx |
| In-reply-to: | <20080728121534.GA12715@xxxxxxxxxxxxx> |
| References: | <20080728062625.BB57158C4C3F@xxxxxxxxxxxxxxxxxxxxxxx> <20080728121534.GA12715@xxxxxxxxxxxxx> |
| Reply-to: | lachlan@xxxxxxx |
| Sender: | xfs-bounce@xxxxxxxxxxx |
| User-agent: | Thunderbird 2.0.0.16 (X11/20080707) |
Christoph Hellwig wrote: On Mon, Jul 28, 2008 at 04:26:25PM +1000, Lachlan McIlroy wrote:kill bhv_vnode_t All remaining bhv_vnode_t instance are in code that's more or less Linux specific. (Well, for xfs_acl.c that could be argued, but that code is on the removal list, too). So just do an s/bhv_vnode_t/struct inode/ over the whole tree. We can clean up variable naming and some useless helpers later.Thanks, but I wonder how this could even apply without the intialize_vnode cleanup.. It applied fine. Didn't build though - I had to change a couple more bhv_vnode_t's. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | TAKE 981498 - sanitize xfs_initialize_vnode, Niv Sardi-Altivanik |
|---|---|
| Next by Date: | [REVIEW] Cleanup more dir v1 macros and stuff, Barry Naujok |
| Previous by Thread: | Re: TAKE 981498 - kill bhv_vnode_t, Christoph Hellwig |
| Next by Thread: | xfs_lowbit32 implementation, Barry Naujok |
| Indexes: | [Date] [Thread] [Top] [All Lists] |