From: swhiteho@redhat.com
To: linux-kernel@vger.kernel.org, cluster-devel@redhat.com
Cc: Steven Whitehouse <swhiteho@redhat.com>
Subject: [PATCH 39/51] [GFS2] Use slab operations for all gfs2_bufdata allocations
Date: Thu, 4 Oct 2007 09:49:32 +0100 [thread overview]
Message-ID: <1191487861683-git-send-email-swhiteho@redhat.com> (raw)
In-Reply-To: <11914878592314-git-send-email-swhiteho@redhat.com>
From: Steven Whitehouse <swhiteho@redhat.com>
The old revoke structure was allocated using kalloc/kfree but
there is a slab cache for gfs2_bufdata, so we should use that
now that the structures have been converted.
This is part two of the patch series to merge the revoke
and gfs2_bufdata structures.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
diff --git a/fs/gfs2/lops.c b/fs/gfs2/lops.c
index cf6fe36..4cbef4c 100644
--- a/fs/gfs2/lops.c
+++ b/fs/gfs2/lops.c
@@ -393,7 +393,7 @@ static void revoke_lo_before_commit(struct gfs2_sbd *sdp)
}
*(__be64 *)(bh->b_data + offset) = cpu_to_be64(bd->bd_blkno);
- kfree(bd);
+ kmem_cache_free(gfs2_bufdata_cachep, bd);
offset += sizeof(u64);
}
diff --git a/fs/gfs2/trans.c b/fs/gfs2/trans.c
index eadf96e..01cc27f 100644
--- a/fs/gfs2/trans.c
+++ b/fs/gfs2/trans.c
@@ -144,7 +144,7 @@ void gfs2_trans_add_bh(struct gfs2_glock *gl, struct buffer_head *bh, int meta)
void gfs2_trans_add_revoke(struct gfs2_sbd *sdp, u64 blkno)
{
- struct gfs2_bufdata *bd = kmalloc(sizeof(struct gfs2_bufdata),
+ struct gfs2_bufdata *bd = kmem_cache_alloc(gfs2_bufdata_cachep,
GFP_NOFS | __GFP_NOFAIL);
lops_init_le(&bd->bd_le, &gfs2_revoke_lops);
bd->bd_blkno = blkno;
@@ -172,7 +172,7 @@ void gfs2_trans_add_unrevoke(struct gfs2_sbd *sdp, u64 blkno)
if (found) {
struct gfs2_trans *tr = current->journal_info;
- kfree(bd);
+ kmem_cache_free(gfs2_bufdata_cachep, bd);
tr->tr_num_revoke_rm++;
}
}
--
1.5.1.2
next prev parent reply other threads:[~2007-10-04 9:32 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-04 8:48 [GFS2/DLM] Pre-pull patch posting swhiteho
2007-10-04 8:48 ` [PATCH 01/51] [GFS2] Fix two races relating to glock callbacks swhiteho
2007-10-04 8:48 ` [PATCH 02/51] [GFS2] Fix calculation of demote state swhiteho
2007-10-04 8:48 ` [PATCH 03/51] [GFS2] Clean up duplicate includes in fs/gfs2/ swhiteho
2007-10-04 8:48 ` [PATCH 04/51] [GFS2] GFS2 not checking pointer on create when running under nfsd swhiteho
2007-10-04 8:48 ` [PATCH 05/51] [GFS2] Fix an oops in glock dumping swhiteho
2007-10-04 8:48 ` [PATCH 06/51] [GFS2] Move some code inside the log lock swhiteho
2007-10-04 8:49 ` [PATCH 07/51] [GFS2] Revert part of earlier log.c changes swhiteho
2007-10-04 8:49 ` [PATCH 08/51] [GFS2] Prevent infinite loop in try_rgrp_unlink() swhiteho
2007-10-04 8:49 ` [PATCH 09/51] [GFS2] use an temp variable to reduce a spin_unlock swhiteho
2007-10-04 8:49 ` [PATCH 10/51] [GFS2] Detach buf data during in-place writeback swhiteho
2007-10-04 8:49 ` [PATCH 11/51] [GFS2] mark struct *_operations const swhiteho
2007-10-04 8:49 ` [PATCH 12/51] [GFS2] use the declaration of gfs2_dops in the header file instead swhiteho
2007-10-04 8:49 ` [PATCH 13/51] [GFS2] Reduce number of gfs2_scand processes to one swhiteho
2007-10-04 8:49 ` [PATCH 14/51] [GFS2] invalid metadata block - REVISED swhiteho
2007-10-04 8:49 ` [PATCH 15/51] [GFS2] Ensure journal file cache is flushed after recovery swhiteho
2007-10-04 8:49 ` [PATCH 16/51] [GFS2] use list_for_each_entry instead swhiteho
2007-10-04 8:49 ` [PATCH 17/51] [GFS2] unneeded typecast swhiteho
2007-10-04 8:49 ` [PATCH 18/51] [GFS2] better code for translating characters swhiteho
2007-10-04 8:49 ` [PATCH 19/51] [GFS2] Force unstuff of hidden quota inode swhiteho
2007-10-04 8:49 ` [PATCH 20/51] [GFS2] fixed a NULL pointer assignment BUG swhiteho
2007-10-04 8:49 ` [PATCH 21/51] [GFS2] Fix quota do_list operation hang swhiteho
2007-10-04 8:49 ` [PATCH 22/51] [GFS2] Clean up invalidatepage/releasepage swhiteho
2007-10-04 8:49 ` [PATCH 23/51] [GFS2] Add a missing gfs2_trans_add_bh() swhiteho
2007-10-04 8:49 ` [PATCH 24/51] [GFS2] Add NULL entry to token table swhiteho
2007-10-04 8:49 ` [PATCH 25/51] [GFS2] Reduce truncate IO traffic swhiteho
2007-10-04 8:49 ` [PATCH 26/51] [DLM] Fix lowcomms socket closing swhiteho
2007-10-04 8:49 ` [PATCH 27/51] [GFS2] Wendy's dump lockname in hex & fix glock dump swhiteho
2007-10-04 8:49 ` [PATCH 28/51] [GFS2] Patch to protect sd_log_num_jdata swhiteho
2007-10-04 8:49 ` [PATCH 29/51] [GFS2] panic after can't parse mount arguments swhiteho
2007-10-04 8:49 ` [PATCH 30/51] [GFS2] delay glock demote for a minimum hold time swhiteho
2007-10-04 8:49 ` [PATCH 31/51] [GFS2] fix inode meta data corruption swhiteho
2007-10-04 8:49 ` [PATCH 32/51] [GFS2] Correct lock ordering in unlink swhiteho
2007-10-04 8:49 ` [PATCH 33/51] [GFS2] Introduce gfs2_remove_from_ail swhiteho
2007-10-04 8:49 ` [PATCH 34/51] [GFS2] Don't mark jdata dirty in gfs2_unstuffer_page() swhiteho
2007-10-04 8:49 ` [PATCH 35/51] [GFS2] Move pin/unpin into lops.c, clean up locking swhiteho
2007-10-04 8:49 ` [PATCH 36/51] [GFS2] Clean up ordered write code swhiteho
2007-10-04 8:49 ` [PATCH 37/51] [GFS2] Fix ordering of dirty/journal for ordered buffer unstuffing swhiteho
2007-10-04 8:49 ` [PATCH 38/51] [GFS2] Replace revoke structure with bufdata structure swhiteho
2007-10-04 8:49 ` swhiteho [this message]
2007-10-04 8:49 ` [PATCH 40/51] [GFS2] Clean up gfs2_trans_add_revoke() swhiteho
2007-10-04 8:49 ` [PATCH 41/51] [GFS2] flocks from same process trip kernel BUG at fs/gfs2/glock.c:1118! swhiteho
2007-10-04 8:49 ` [PATCH 42/51] [GFS2] Move inode deletion out of blocking_cb swhiteho
2007-10-04 8:49 ` [PATCH 43/51] [DLM] Make dlm_sendd cond_resched more swhiteho
2007-10-04 8:49 ` [PATCH 44/51] [GFS2] GFS2: chmod hung - fix race in thread creation swhiteho
2007-10-04 8:49 ` [PATCH 45/51] [GFS2] Clean up journaled data writing swhiteho
2007-10-04 8:49 ` [PATCH 46/51] [GFS2] Data corruption fix swhiteho
2007-10-04 8:49 ` [PATCH 47/51] [GFS2] Alternate gfs2_iget to avoid looking up inodes being freed swhiteho
2007-10-04 8:49 ` [PATCH 48/51] [GFS2] Don't try to remove buffers that don't exist swhiteho
2007-10-04 8:49 ` [PATCH 49/51] [GFS2] Get superblock a different way swhiteho
2007-10-04 8:49 ` [PATCH 50/51] [DLM] don't overwrite castparam if it's NULL swhiteho
2007-10-04 8:49 ` [PATCH 51/51] [DLM] block dlm_recv in recovery transition swhiteho
2007-10-12 7:47 ` [GFS2/DLM] Pull request Steven Whitehouse
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1191487861683-git-send-email-swhiteho@redhat.com \
--to=swhiteho@redhat.com \
--cc=cluster-devel@redhat.com \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox