summaryrefslogtreecommitdiffstats
path: root/src/compiler/glsl/tests
diff options
context:
space:
mode:
authorTimothy Arceri <[email protected]>2016-09-28 08:55:02 +1000
committerTimothy Arceri <[email protected]>2016-09-28 09:16:31 +1000
commit87ab26b2ab35a29d446ae66f1795d40c184c0739 (patch)
tree18db7f28e1cb85d6338c91f5df6087a587d70fd8 /src/compiler/glsl/tests
parent44bcf1ffcced04fd7f2b537a922ffa85dd1418e2 (diff)
glsl: Add initial functions to implement an on-disk cache
This code provides for an on-disk cache of objects. Objects are stored and retrieved via names that are arbitrary 20-byte sequences, (intended to be SHA-1 hashes of something identifying for the content). The directory used for the cache can be specified by means of environment variables in the following priority order: $MESA_GLSL_CACHE_DIR $XDG_CACHE_HOME/mesa <user-home-directory>/.cache/mesa By default the cache will be limited to a maximum size of 1GB. The environment variable: $MESA_GLSL_CACHE_MAX_SIZE can be set (at the time of GL context creation) to choose some other size. This variable is a number that can optionally be followed by 'K', 'M', or 'G' to select a size in kilobytes, megabytes, or gigabytes. By default, an unadorned value will be interpreted as gigabytes. The cache will be entirely disabled at runtime if the variable MESA_GLSL_CACHE_DISABLE is set at the time of GL context creation. Many thanks to Kristian Høgsberg <[email protected]> for the initial implementation of code that led to this patch. In particular, the idea of using an mmapped file, (indexed by a portion of the SHA-1), for the efficent implementation of cache_has_key was entirely his idea. Kristian also provided some very helpful advice in discussions regarding various race conditions to be avoided in this code. Signed-off-by: Timothy Arceri <[email protected]> Reviewed-by: Eric Anholt <[email protected]>
Diffstat (limited to 'src/compiler/glsl/tests')
-rw-r--r--src/compiler/glsl/tests/.gitignore1
-rw-r--r--src/compiler/glsl/tests/cache_test.c416
2 files changed, 417 insertions, 0 deletions
diff --git a/src/compiler/glsl/tests/.gitignore b/src/compiler/glsl/tests/.gitignore
index 13dcdc4ab73..1661ae78bdd 100644
--- a/src/compiler/glsl/tests/.gitignore
+++ b/src/compiler/glsl/tests/.gitignore
@@ -1,4 +1,5 @@
blob-test
+cache-test
ralloc-test
uniform-initializer-test
sampler-types-test
diff --git a/src/compiler/glsl/tests/cache_test.c b/src/compiler/glsl/tests/cache_test.c
new file mode 100644
index 00000000000..1b0403caca6
--- /dev/null
+++ b/src/compiler/glsl/tests/cache_test.c
@@ -0,0 +1,416 @@
+/*
+ * Copyright © 2015 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ */
+
+/* A collection of unit tests for cache.c */
+
+#include <stdio.h>
+#include <stdlib.h>
+#include <stdbool.h>
+#include <string.h>
+#include <ftw.h>
+#include <errno.h>
+#include <stdarg.h>
+
+#include "util/mesa-sha1.h"
+#include "cache.h"
+
+bool error = false;
+
+void
+_mesa_warning(void *ctx, const char *fmt, ...);
+
+void
+_mesa_warning(void *ctx, const char *fmt, ...)
+{
+ va_list vargs;
+ (void) ctx;
+
+ va_start(vargs, fmt);
+
+ /* This output is not thread-safe, but that's good enough for the
+ * standalone compiler.
+ */
+ fprintf(stderr, "Mesa warning: ");
+ vfprintf(stderr, fmt, vargs);
+ fprintf(stderr, "\n");
+
+ va_end(vargs);
+}
+
+static void
+expect_equal(uint64_t actual, uint64_t expected, const char *test)
+{
+ if (actual != expected) {
+ fprintf(stderr, "Error: Test '%s' failed: Expected=%ld, Actual=%ld\n",
+ test, expected, actual);
+ error = true;
+ }
+}
+
+static void
+expect_null(void *ptr, const char *test)
+{
+ if (ptr != NULL) {
+ fprintf(stderr, "Error: Test '%s' failed: Result=%p, but expected NULL.\n",
+ test, ptr);
+ error = true;
+ }
+}
+
+static void
+expect_non_null(void *ptr, const char *test)
+{
+ if (ptr == NULL) {
+ fprintf(stderr, "Error: Test '%s' failed: Result=NULL, but expected something else.\n",
+ test);
+ error = true;
+ }
+}
+
+static void
+expect_equal_str(const char *actual, const char *expected, const char *test)
+{
+ if (strcmp(actual, expected)) {
+ fprintf(stderr, "Error: Test '%s' failed:\n\t"
+ "Expected=\"%s\", Actual=\"%s\"\n",
+ test, expected, actual);
+ error = true;
+ }
+}
+
+/* Callback for nftw used in rmrf_local below.
+ */
+static int
+remove_entry(const char *path,
+ const struct stat *sb,
+ int typeflag,
+ struct FTW *ftwbuf)
+{
+ int err = remove(path);
+
+ if (err)
+ fprintf(stderr, "Error removing %s: %s\n", path, strerror(errno));
+
+ return err;
+}
+
+/* Recursively remove a directory.
+ *
+ * This is equivalent to "rm -rf <dir>" with one bit of protection
+ * that the directory name must begin with "." to ensure we don't
+ * wander around deleting more than intended.
+ *
+ * Returns 0 on success, -1 on any error.
+ */
+static int
+rmrf_local(const char *path)
+{
+ if (path == NULL || *path == '\0' || *path != '.')
+ return -1;
+
+ return nftw(path, remove_entry, 64, FTW_DEPTH | FTW_PHYS | FTW_MOUNT);
+}
+
+#define CACHE_TEST_TMP "./cache-test-tmp"
+
+static void
+test_cache_create(void)
+{
+ struct program_cache *cache;
+ int err;
+
+ /* Before doing anything else, ensure that with
+ * MESA_GLSL_CACHE_DISABLE set, that cache_create returns NULL.
+ */
+ setenv("MESA_GLSL_CACHE_DISABLE", "1", 1);
+ cache = cache_create();
+ expect_null(cache, "cache_create with MESA_GLSL_CACHE_DISABLE set");
+
+ unsetenv("MESA_GLSL_CACHE_DISABLE");
+
+ /* For the first real cache_create() clear these environment
+ * variables to test creation of cache in home directory.
+ */
+ unsetenv("MESA_GLSL_CACHE_DIR");
+ unsetenv("XDG_CACHE_HOME");
+
+ cache = cache_create();
+ expect_non_null(cache, "cache_create with no environment variables");
+
+ cache_destroy(cache);
+
+ /* Test with XDG_CACHE_HOME set */
+ setenv("XDG_CACHE_HOME", CACHE_TEST_TMP "/xdg-cache-home", 1);
+ cache = cache_create();
+ expect_null(cache, "cache_create with XDG_CACHE_HOME set with"
+ "a non-existing parent directory");
+
+ mkdir(CACHE_TEST_TMP, 0755);
+ cache = cache_create();
+ expect_non_null(cache, "cache_create with XDG_CACHE_HOME set");
+
+ cache_destroy(cache);
+
+ /* Test with MESA_GLSL_CACHE_DIR set */
+ err = rmrf_local(CACHE_TEST_TMP);
+ expect_equal(err, 0, "Removing " CACHE_TEST_TMP);
+
+ setenv("MESA_GLSL_CACHE_DIR", CACHE_TEST_TMP "/mesa-glsl-cache-dir", 1);
+ cache = cache_create();
+ expect_null(cache, "cache_create with MESA_GLSL_CACHE_DIR set with"
+ "a non-existing parent directory");
+
+ mkdir(CACHE_TEST_TMP, 0755);
+ cache = cache_create();
+ expect_non_null(cache, "cache_create with MESA_GLSL_CACHE_DIR set");
+
+ cache_destroy(cache);
+}
+
+static bool
+does_cache_contain(struct program_cache *cache, cache_key key)
+{
+ void *result;
+
+ result = cache_get(cache, key, NULL);
+
+ if (result) {
+ free(result);
+ return true;
+ }
+
+ return false;
+}
+
+static void
+test_put_and_get(void)
+{
+ struct program_cache *cache;
+ /* If the text of this blob is changed, then blob_key_byte_zero
+ * also needs to be updated.
+ */
+ char blob[] = "This is a blob of thirty-seven bytes";
+ uint8_t blob_key[20];
+ uint8_t blob_key_byte_zero = 0xca;
+ char string[] = "While this string has thirty-four";
+ uint8_t string_key[20];
+ char *result;
+ size_t size;
+ uint8_t *one_KB, *one_MB;
+ uint8_t one_KB_key[20], one_MB_key[20];
+ int count;
+
+ cache = cache_create();
+
+ _mesa_sha1_compute(blob, sizeof(blob), blob_key);
+
+ /* Ensure that cache_get returns nothing before anything is added. */
+ result = cache_get(cache, blob_key, &size);
+ expect_null(result, "cache_get with non-existent item (pointer)");
+ expect_equal(size, 0, "cache_get with non-existent item (size)");
+
+ /* Simple test of put and get. */
+ cache_put(cache, blob_key, blob, sizeof(blob));
+
+ result = cache_get(cache, blob_key, &size);
+ expect_equal_str(blob, result, "cache_get of existing item (pointer)");
+ expect_equal(size, sizeof(blob), "cache_get of existing item (size)");
+
+ free(result);
+
+ /* Test put and get of a second item. */
+ _mesa_sha1_compute(string, sizeof(string), string_key);
+ cache_put(cache, string_key, string, sizeof(string));
+
+ result = cache_get(cache, string_key, &size);
+ expect_equal_str(result, string, "2nd cache_get of existing item (pointer)");
+ expect_equal(size, sizeof(string), "2nd cache_get of existing item (size)");
+
+ free(result);
+
+ /* Set the cache size to 1KB and add a 1KB item to force an eviction. */
+ cache_destroy(cache);
+
+ setenv("MESA_GLSL_CACHE_MAX_SIZE", "1K", 1);
+ cache = cache_create();
+
+ one_KB = calloc(1, 1024);
+
+ /* Obviously the SHA-1 hash of 1024 zero bytes isn't particularly
+ * interesting. But we do have want to take some special care with
+ * the hash we use here. The issue is that in this artificial case,
+ * (with only three files in the cache), the probability is good
+ * that each of the three files will end up in their own
+ * directory. Then, if the directory containing the .tmp file for
+ * the new item being added for cache_put() is the chosen victim
+ * directory for eviction, then no suitable file will be found and
+ * nothing will be evicted.
+ *
+ * That's actually expected given how the eviction code is
+ * implemented, (which expects to only evict once things are more
+ * interestingly full than that).
+ *
+ * For this test, we force this signature to land in the same
+ * directory as the original blob first written to the cache.
+ */
+ _mesa_sha1_compute(one_KB, 1024, one_KB_key);
+ one_KB_key[0] = blob_key_byte_zero;
+
+ cache_put(cache, one_KB_key, one_KB, 1024);
+
+ free(one_KB);
+
+ result = cache_get(cache, one_KB_key, &size);
+ expect_non_null(result, "3rd cache_get of existing item (pointer)");
+ expect_equal(size, 1024, "3rd cache_get of existing item (size)");
+
+ free(result);
+
+ /* Ensure eviction happened by checking that only one of the two
+ * previously-added items can still be fetched.
+ */
+ count = 0;
+ if (does_cache_contain(cache, blob_key))
+ count++;
+
+ if (does_cache_contain(cache, string_key))
+ count++;
+
+ expect_equal(count, 1, "cache_put eviction with MAX_SIZE=1K");
+
+ /* Now increase the size to 1M, add back both items, and ensure all
+ * three that have been added are available via cache_get.
+ */
+ cache_destroy(cache);
+
+ setenv("MESA_GLSL_CACHE_MAX_SIZE", "1M", 1);
+ cache = cache_create();
+
+ cache_put(cache, blob_key, blob, sizeof(blob));
+ cache_put(cache, string_key, string, sizeof(string));
+
+ count = 0;
+ if (does_cache_contain(cache, blob_key))
+ count++;
+
+ if (does_cache_contain(cache, string_key))
+ count++;
+
+ if (does_cache_contain(cache, one_KB_key))
+ count++;
+
+ expect_equal(count, 3, "no eviction before overflow with MAX_SIZE=1M");
+
+ /* Finally, check eviction again after adding an object of size 1M. */
+ one_MB = calloc(1024, 1024);
+
+ _mesa_sha1_compute(one_MB, 1024 * 1024, one_MB_key);
+ one_MB_key[0] = blob_key_byte_zero;;
+
+ cache_put(cache, one_MB_key, one_MB, 1024 * 1024);
+
+ free(one_MB);
+
+ count = 0;
+ if (does_cache_contain(cache, blob_key))
+ count++;
+
+ if (does_cache_contain(cache, string_key))
+ count++;
+
+ if (does_cache_contain(cache, one_KB_key))
+ count++;
+
+ expect_equal(count, 2, "eviction after overflow with MAX_SIZE=1M");
+
+ cache_destroy(cache);
+}
+
+static void
+test_put_key_and_get_key(void)
+{
+ struct program_cache *cache;
+ bool result;
+
+ uint8_t key_a[20] = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9,
+ 10, 11, 12, 13, 14, 15, 16, 17, 18, 19};
+ uint8_t key_b[20] = { 20, 21, 22, 23, 24, 25, 26, 27, 28, 29,
+ 30, 33, 32, 33, 34, 35, 36, 37, 38, 39};
+ uint8_t key_a_collide[20] =
+ { 0, 1, 42, 43, 44, 45, 46, 47, 48, 49,
+ 50, 55, 52, 53, 54, 55, 56, 57, 58, 59};
+
+ cache = cache_create();
+
+ /* First test that cache_has_key returns false before cache_put_key */
+ result = cache_has_key(cache, key_a);
+ expect_equal(result, 0, "cache_has_key before key added");
+
+ /* Then a couple of tests of cache_put_key followed by cache_has_key */
+ cache_put_key(cache, key_a);
+ result = cache_has_key(cache, key_a);
+ expect_equal(result, 1, "cache_has_key after key added");
+
+ cache_put_key(cache, key_b);
+ result = cache_has_key(cache, key_b);
+ expect_equal(result, 1, "2nd cache_has_key after key added");
+
+ /* Test that a key with the same two bytes as an existing key
+ * forces an eviction.
+ */
+ cache_put_key(cache, key_a_collide);
+ result = cache_has_key(cache, key_a_collide);
+ expect_equal(result, 1, "put_key of a colliding key lands in the cache");
+
+ result = cache_has_key(cache, key_a);
+ expect_equal(result, 0, "put_key of a colliding key evicts from the cache");
+
+ /* And finally test that we can re-add the original key to re-evict
+ * the colliding key.
+ */
+ cache_put_key(cache, key_a);
+ result = cache_has_key(cache, key_a);
+ expect_equal(result, 1, "put_key of original key lands again");
+
+ result = cache_has_key(cache, key_a_collide);
+ expect_equal(result, 0, "put_key of orginal key evicts the colliding key");
+
+ cache_destroy(cache);
+}
+
+int
+main(void)
+{
+ int err;
+
+ test_cache_create();
+
+ test_put_and_get();
+
+ test_put_key_and_get_key();
+
+ err = rmrf_local(CACHE_TEST_TMP);
+ expect_equal(err, 0, "Removing " CACHE_TEST_TMP " again");
+
+ return error ? 1 : 0;
+}