On Monday 24 July 2017 12:33:55 Vlad Krasnov via nginx-devel wrote: [..]
- ngx_http_v2_hpack_enc_entry_t htable[HPACK_ENC_HTABLE_SZ];
- ngx_http_v2_hpack_name_entry_t heads[HPACK_ENC_DYNAMIC_KEY_TBL_SZ];
- u_char storage[NGX_HTTP_V2_MAX_HPACK_TABLE_SIZE +
+} ngx_http_v2_hpack_enc_t; +#endif
Well, it means that even idle connection will consume 18+ KB more memory than before.
That doesn't look like a generic solution suitable for most of our users. It should be at least configurable.
I can make it configurable fairly easily, but then it will require an extra allocation.
With your current patch ngx_http_v2_connection_s doesn't fit into memory pool anymore and requires an extra allocation anyway.
wbr, Valentin V. Bartenev