From 587ac01ce836286aab54bfcb7a693a0170c7ebd3 Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Thu, 22 Apr 2010 13:54:05 +0930 Subject: [PATCH] tdb: remove num_locks This was redundant before this patch series: it mirrored num_lockrecs exactly. It still does. Also, skip useless branch when locks == 1: unconditional assignment is cheaper anyway. Signed-off-by: Rusty Russell (Imported from commit 1ab8776247f89b143b6e58f4b038ab4bcea20d3a) --- lib/tdb/common/lock.c | 12 ++---------- lib/tdb/common/tdb_private.h | 1 - 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/lib/tdb/common/lock.c b/lib/tdb/common/lock.c index 19b92d9a..73d34417 100644 --- a/lib/tdb/common/lock.c +++ b/lib/tdb/common/lock.c @@ -277,12 +277,10 @@ int tdb_nest_lock(struct tdb_context *tdb, uint32_t offset, int ltype, return -1; } - tdb->num_locks++; - tdb->lockrecs[tdb->num_lockrecs].off = offset; tdb->lockrecs[tdb->num_lockrecs].count = 1; tdb->lockrecs[tdb->num_lockrecs].ltype = ltype; - tdb->num_lockrecs += 1; + tdb->num_lockrecs++; return 0; } @@ -368,17 +366,12 @@ int tdb_nest_unlock(struct tdb_context *tdb, uint32_t offset, int ltype, } else { ret = tdb->methods->brunlock(tdb, ltype, offset, 1); } - tdb->num_locks--; /* * Shrink the array by overwriting the element just unlocked with the * last array element. */ - - if (tdb->num_lockrecs > 1) { - *lck = tdb->lockrecs[tdb->num_lockrecs-1]; - } - tdb->num_lockrecs -= 1; + *lck = tdb->lockrecs[--tdb->num_lockrecs]; /* * We don't bother with realloc when the array shrinks, but if we have @@ -712,7 +705,6 @@ void tdb_release_extra_locks(struct tdb_context *tdb) tdb_brunlock(tdb, lck->ltype, lck->off, 1); } } - tdb->num_locks = extra; tdb->num_lockrecs = extra; if (tdb->num_lockrecs == 0) { SAFE_FREE(tdb->lockrecs); diff --git a/lib/tdb/common/tdb_private.h b/lib/tdb/common/tdb_private.h index 2e3c30f0..0ac37dfa 100644 --- a/lib/tdb/common/tdb_private.h +++ b/lib/tdb/common/tdb_private.h @@ -204,7 +204,6 @@ struct tdb_context { struct tdb_logging_context log; unsigned int (*hash_fn)(TDB_DATA *key); int open_flags; /* flags used in the open - needed by reopen */ - unsigned int num_locks; /* number of chain locks held */ const struct tdb_methods *methods; struct tdb_transaction *transaction; int page_size; -- 2.34.1