diff options
author | Timothy Arceri <[email protected]> | 2016-09-28 08:55:02 +1000 |
---|---|---|
committer | Timothy Arceri <[email protected]> | 2016-09-28 09:16:31 +1000 |
commit | 87ab26b2ab35a29d446ae66f1795d40c184c0739 (patch) | |
tree | 18db7f28e1cb85d6338c91f5df6087a587d70fd8 /src/compiler/glsl/tests | |
parent | 44bcf1ffcced04fd7f2b537a922ffa85dd1418e2 (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/.gitignore | 1 | ||||
-rw-r--r-- | src/compiler/glsl/tests/cache_test.c | 416 |
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; +} |