Remove the XXXAZ(smp_chk_sign) after smp_append_sign(), as it causes
heavy unnecessary CPU usage after each sign update. No assertion
reports have come from this test, so there is no reason to expect the
signs not working.
---
 bin/varnishd/storage/storage_persistent_subr.c |    1 -
 1 file changed, 1 deletion(-)

diff --git a/bin/varnishd/storage/storage_persistent_subr.c 
b/bin/varnishd/storage/storage_persistent_subr.c
index f73ae06..a18f2cc 100644
--- a/bin/varnishd/storage/storage_persistent_subr.c
+++ b/bin/varnishd/storage/storage_persistent_subr.c
@@ -128,7 +128,6 @@ smp_append_sign(struct smp_signctx *ctx, const void *ptr, 
uint32_t len)
        SHA256_Update(&cx, &ctx->ss->length, sizeof(ctx->ss->length));
        SHA256_Final(sign, &cx);
        memcpy(SIGN_END(ctx), sign, sizeof sign);
-XXXAZ(smp_chk_sign(ctx));
 }
 
 /*--------------------------------------------------------------------
-- 
1.7.9.5


_______________________________________________
varnish-dev mailing list
[email protected]
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev

Reply via email to