From: Panu Matilainen <pmatilai@redhat.com>
Date: Wed, 29 Aug 2012 14:31:32 +0000 (+0300)
Subject: Define separate hash for rpmdb header check cache
X-Git-Tag: rpm-4.11.0-alpha~216
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f933015fc04ade1189e38f37ebafe56d30e7213f;p=platform%2Fupstream%2Frpm.git

Define separate hash for rpmdb header check cache

- This is really private to rpmdb.c (other than being exposed in
  the struct definition in backend/dbi.h, sigh), let it live its
  own life there.
- No functional changes here, just cleaning up a bit for next steps.
  OTOH we could now cache the actual result, not just success... but
  leaving that to another time.
---

diff --git a/lib/backend/dbi.h b/lib/backend/dbi.h
index 848f288..5405ebe 100644
--- a/lib/backend/dbi.h
+++ b/lib/backend/dbi.h
@@ -37,7 +37,7 @@ struct rpmdb_s {
     int		db_mode;	/*!< open mode */
     int		db_perms;	/*!< open permissions */
     int		db_ver;		/*!< Berkeley DB version */
-    intHash	db_checked;	/*!< headerCheck()'ed package instances */
+    struct dbChk_s * db_checked;/*!< headerCheck()'ed package instances */
     rpmdb	db_next;
     int		db_opens;
     int		db_ndbi;	/*!< No. of tag indices. */
diff --git a/lib/rpmdb.c b/lib/rpmdb.c
index 79abde7..7b83c8c 100644
--- a/lib/rpmdb.c
+++ b/lib/rpmdb.c
@@ -36,6 +36,17 @@
 #include "lib/header_internal.h"	/* XXX for headerSetInstance() */
 #include "debug.h"
 
+#undef HASHTYPE
+#undef HTKEYTYPE
+#undef HTDATATYPE
+#define HASHTYPE dbChk
+#define HTKEYTYPE unsigned int
+#include "lib/rpmhash.H"
+#include "lib/rpmhash.C"
+#undef HASHTYPE
+#undef HTKEYTYPE
+#undef HTDATATYPE
+
 static rpmDbiTag const dbiTags[] = {
     RPMDBI_PACKAGES,
     RPMDBI_NAME,
@@ -186,7 +197,7 @@ static dbiIndex rpmdbOpenIndex(rpmdb db, rpmDbiTagVal rpmtag, int flags)
 	if (dbiType(dbi) == DBI_PRIMARY) {
 	    /* Allocate based on max header instance number + some reserve */
 	    if (!verifyonly && (db->db_checked == NULL)) {
-		db->db_checked = intHashCreate(1024 + pkgInstance(dbi, 0) / 4,
+		db->db_checked = dbChkCreate(1024 + pkgInstance(dbi, 0) / 4,
 						uintId, uintCmp, NULL);
 	    }
 	    /* If primary got created, we can safely run without fsync */
@@ -736,7 +747,7 @@ int rpmdbClose(rpmdb db)
     db->db_root = _free(db->db_root);
     db->db_home = _free(db->db_home);
     db->db_fullpath = _free(db->db_fullpath);
-    db->db_checked = intHashFree(db->db_checked);
+    db->db_checked = dbChkFree(db->db_checked);
     db->_dbi = _free(db->_dbi);
 
     prev = &rpmdbRock;
@@ -1703,7 +1714,7 @@ static rpmRC miVerifyHeader(rpmdbMatchIterator mi, const void *uh, size_t uhlen)
 
     /* Don't bother re-checking a previously read header. */
     if (mi->mi_db->db_checked) {
-	if (intHashHasEntry(mi->mi_db->db_checked, mi->mi_offset))
+	if (dbChkHasEntry(mi->mi_db->db_checked, mi->mi_offset))
 	    rpmrc = RPMRC_OK;
     }
 
@@ -1721,7 +1732,7 @@ static rpmRC miVerifyHeader(rpmdbMatchIterator mi, const void *uh, size_t uhlen)
 
 	/* Mark header checked. */
 	if (mi->mi_db && mi->mi_db->db_checked && rpmrc != RPMRC_FAIL) {
-	    intHashAddEntry(mi->mi_db->db_checked, mi->mi_offset);
+	    dbChkAddEntry(mi->mi_db->db_checked, mi->mi_offset);
 	}
     }
     return rpmrc;
@@ -2684,7 +2695,7 @@ int rpmdbAdd(rpmdb db, Header h)
 	headerSetInstance(h, hdrNum);
 	/* Purge our verification cache on added public keys */
 	if (db->db_checked && headerIsEntry(h, RPMTAG_PUBKEYS)) {
-	    intHashEmpty(db->db_checked);
+	    dbChkEmpty(db->db_checked);
 	}
     }