lkml.org 
[lkml]   [2001]   [Feb]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH] Use slab in pipe code
Hi Linus,

this patch makes the pipe code use the slab allocator instead of
kmalloc/kfree. The changes are pretty small, so I think it's ok
for the stable series.

Christoph

--
Of course it doesn't work. We've performed a software upgrade.


--- linux-2.4.2-pre3/fs/pipe.c Sat Feb 10 20:03:33 2001
+++ linux/fs/pipe.c Sat Feb 10 20:36:57 2001
@@ -21,8 +21,13 @@
*
* Reads with count = 0 should always return 0.
* -- Julian Bradfield 1999-06-07.
+ *
+ * Use slab allocator instead of kmalloc/kfree.
+ * -- Christoph Hellwig 2001-02-07
*/

+static kmem_cache_t * pipe_cachep;
+
/* Drop the inode semaphore and wait for a pipe event, atomically */
void pipe_wait(struct inode * inode)
{
@@ -448,7 +453,7 @@
if (!page)
return NULL;

- inode->i_pipe = kmalloc(sizeof(struct pipe_inode_info), GFP_KERNEL);
+ inode->i_pipe = kmem_cache_alloc(pipe_cachep, SLAB_KERNEL);
if (!inode->i_pipe)
goto fail_page;

@@ -578,7 +583,7 @@
put_unused_fd(i);
close_f12_inode:
free_page((unsigned long) PIPE_BASE(*inode));
- kfree(inode->i_pipe);
+ kmem_cache_free(pipe_cachep, inode->i_pipe);
inode->i_pipe = NULL;
iput(inode);
close_f12:
@@ -635,15 +640,28 @@

static int __init init_pipe_fs(void)
{
- int err = register_filesystem(&pipe_fs_type);
- if (!err) {
- pipe_mnt = kern_mount(&pipe_fs_type);
- err = PTR_ERR(pipe_mnt);
- if (IS_ERR(pipe_mnt))
- unregister_filesystem(&pipe_fs_type);
- else
- err = 0;
- }
+ int err = -ENOMEM;
+
+ pipe_cachep = kmem_cache_create("pipe_cache",
+ sizeof(struct pipe_inode_info), 0,
+ SLAB_HWCACHE_ALIGN, NULL, NULL);
+ if (pipe_cachep == NULL)
+ goto err_out;
+
+ err = register_filesystem(&pipe_fs_type);
+ if (err)
+ goto err_out;
+
+ pipe_mnt = kern_mount(&pipe_fs_type);
+
+ err = PTR_ERR(pipe_mnt);
+ if (!IS_ERR(pipe_mnt))
+ return 0;
+
+ unregister_filesystem(&pipe_fs_type);
+
+err_out:
+ kmem_cache_destroy(pipe_cachep);
return err;
}

@@ -651,6 +669,7 @@
{
unregister_filesystem(&pipe_fs_type);
kern_umount(pipe_mnt);
+ kmem_cache_destroy(pipe_cachep);
}

module_init(init_pipe_fs)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-03-22 13:13    [W:0.024 / U:0.480 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site