[6.0] 1e3e6e2c5 Remove extra call to SES_Reserve_proto_priv

Martin Blix Grydeland martin at varnish-software.com
Tue Feb 4 10:02:08 UTC 2020


commit 1e3e6e2c523f5f96729c12bd9597c919bff686cf
Author: Martin Blix Grydeland <martin at varnish-software.com>
Date:   Thu Dec 12 13:16:40 2019 +0100

    Remove extra call to SES_Reserve_proto_priv
    
    In h2_init_sess, an extra call was always made to SES_Reseve_proto_priv(),
    even though it was already reserved. This wasted a pointer worth of
    session workspace. This patch removes the extra call.

diff --git a/bin/varnishd/http2/cache_http2_session.c b/bin/varnishd/http2/cache_http2_session.c
index ab71cc2aa..5818707f4 100644
--- a/bin/varnishd/http2/cache_http2_session.c
+++ b/bin/varnishd/http2/cache_http2_session.c
@@ -131,7 +131,6 @@ h2_init_sess(const struct worker *wrk, struct sess *sp,
 		AZ(VHT_Init(h2->dectbl,
 			h2->local_settings.header_table_size));
 
-		XXXAN(SES_Reserve_proto_priv(sp, &up));
 		*up = (uintptr_t)h2;
 	}
 	AN(up);


More information about the varnish-commit mailing list