[master] 613fa4334 Remove extra call to SES_Reserve_proto_priv

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


commit 613fa4334b50889b82304e6d75f76e2686101b1b
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 618f26b33..4cf946546 100644
--- a/bin/varnishd/http2/cache_http2_session.c
+++ b/bin/varnishd/http2/cache_http2_session.c
@@ -132,7 +132,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