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 +
> >> + HPACK_ENC_MAX_ENTRY];
> >> +} 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

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel