| To: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 2/3] XFS: Make use of the init-once slab optimisation. |
| From: | Dave Chinner <david@xxxxxxxxxxxxx> |
| Date: | Tue, 16 Sep 2008 14:21:05 +1000 |
| Cc: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20080915211117.GA29743@xxxxxxxxxxxxx> |
| Mail-followup-to: | Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx |
| References: | <1221314627-29065-1-git-send-email-david@xxxxxxxxxxxxx> <1221314627-29065-3-git-send-email-david@xxxxxxxxxxxxx> <20080915211117.GA29743@xxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.18 (2008-05-17) |
On Mon, Sep 15, 2008 at 05:11:17PM -0400, Christoph Hellwig wrote: > On Sun, Sep 14, 2008 at 12:03:46AM +1000, Dave Chinner wrote: > > To avoid having to initialise some fields of the XFS inode > > on every allocation, we can use the slab init-once feature > > to initialise them. All we have to guarantee is that when > > we free the inode, all it's entries are in the initial state. > > Add asserts where possible to ensure debug kernels check this > > initial state before freeing and after allocation. > > This looks like it's already in: > http://oss.sgi.com/cgi-bin/gitweb.cgi?p=xaiki/xfs-linux-2.6-xfs-cvs/.git;a=commitdiff;h=12efb888674600ce73a64f8c6f4a20ea5e1ce4f1 <sigh> Which means that some of the patches won't apply. SGI folks, can you please get the master tree updated so I can rediff all the patches and repost them again? Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] Fix use-after-free with log and quotas, Dave Chinner |
|---|---|
| Next by Date: | Re: [PATCH 2/6] XFS: mark inodes for reclaim via a tag in the inode radix tree, Dave Chinner |
| Previous by Thread: | Re: [PATCH 2/3] XFS: Make use of the init-once slab optimisation., Christoph Hellwig |
| Next by Thread: | [PATCH 1/3] XFS: factor xfs_iget_core() into hit and miss cases, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |