From: Mark Fasheh <mfasheh@suse.com>
To: ocfs2-devel@oss.oracle.com
Subject: [Ocfs2-devel] [PATCH 0/5] Ocfs2 allocation reservations
Date: Wed, 17 Mar 2010 13:17:26 -0700 [thread overview]
Message-ID: <20100317201726.GD11402@wotan.suse.de> (raw)
In-Reply-To: <1268809154-1020-1-git-send-email-mfasheh@suse.com>
On Tue, Mar 16, 2010 at 11:59:09PM -0700, Mark Fasheh wrote:
> Changes from the last patch set:
> - added a check for overlapping reservations in ocfs2_resv_insert()
> - cleaned up the comments in ocfs2_cannibalize_resv()
> - the check for reservation past bitmap end in ocfs2_check_resmap() is more
> strict now.
> - removed the unused m_search_start member of ocfs2_reservation_map
> - optimized __ocfs2_resv_find_window() to ignore regions that are too small
> for the current alloc
> - major cleanup of ocfs2_resmap_claimed_bits()
> - added a set of BUG_ON's to in ocfs2_resmap_claimed_bits() to check that
> the passed allocation range is within the window.
> - fixed ocfs2_local_alloc_find_clear_bits() to return actual bits allocated
> - add a check for a null data_ac in ocfs2_write_begin_nolock()
>
> I also added a fifth patch, "ocfs2: remove ocfs2_local_alloc_in_range()".
> I could spin this as it's own patch to go upstream earlier if we want.
Attached is a very small follow-up patch to turn off some of the expensive
debug checks I placed in the code.
--Mark
From: Mark Fasheh <mfasheh@suse.com>
[PATCH] ocfs2: turn off expensive reservations debugging by default
They can still be turned back on by enabling the "expensive checks" config
option for Ocfs2.
Signed-off-by: Mark Fasheh <mfasheh@suse.com>
---
fs/ocfs2/reservations.c | 3 ---
1 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/fs/ocfs2/reservations.c b/fs/ocfs2/reservations.c
index ecffb1c..97f8b25 100644
--- a/fs/ocfs2/reservations.c
+++ b/fs/ocfs2/reservations.c
@@ -41,9 +41,6 @@
#define OCFS2_CHECK_RESERVATIONS
#endif
-#define OCFS2_CHECK_RESERVATIONS
-
-
DEFINE_SPINLOCK(resv_lock);
#define OCFS2_MIN_RESV_WINDOW_BITS 8
--
1.6.4.2
prev parent reply other threads:[~2010-03-17 20:17 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-17 6:59 [Ocfs2-devel] [PATCH 0/5] Ocfs2 allocation reservations Mark Fasheh
2010-03-17 6:59 ` [Ocfs2-devel] [PATCH 1/5] ocfs2: " Mark Fasheh
2010-03-19 22:40 ` Joel Becker
2010-03-19 23:56 ` Mark Fasheh
2010-03-17 6:59 ` [Ocfs2-devel] [PATCH 2/5] ocfs2: use allocation reservations during file write Mark Fasheh
2010-03-17 6:59 ` [Ocfs2-devel] [PATCH 3/5] ocfs2: use allocation reservations for directory data Mark Fasheh
2010-03-19 22:43 ` Joel Becker
2010-03-20 0:14 ` Mark Fasheh
2010-03-20 1:25 ` Joel Becker
2010-03-20 3:47 ` Mark Fasheh
2010-03-20 6:18 ` Joel Becker
2010-03-21 23:26 ` Mark Fasheh
2010-03-22 20:41 ` Joel Becker
2010-03-17 6:59 ` [Ocfs2-devel] [PATCH 4/5] ocfs2: allocate btree internal block groups from the global bitmap Mark Fasheh
2010-03-17 6:59 ` [Ocfs2-devel] [PATCH 5/5] ocfs2: remove ocfs2_local_alloc_in_range() Mark Fasheh
2010-03-17 20:17 ` Mark Fasheh [this message]
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=20100317201726.GD11402@wotan.suse.de \
--to=mfasheh@suse.com \
--cc=ocfs2-devel@oss.oracle.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.