[squid-dev] [PATCH] %<lp missing for persistent connections
Alex Rousskov
rousskov at measurement-factory.com
Tue Oct 27 03:51:30 UTC 2015
On 10/21/2015 03:14 AM, Amos Jeffries wrote:
> On 21/10/2015 4:42 p.m., Alex Rousskov wrote:
>> Hello,
>>
>> Connection stats, including %<lp, were missing for persistent
>> connections.
>>
>> The code reusing a pconn was missing a hier.note() call, resulting in 0
>> values logged for %<lp (local port number of the last server or peer
>> connection) and probably other missing stats.
>>
>> Also refactored poorly copied statistics collection code to remove
>> duplication and always update to-server connection stats when the actual
>> connection becomes available.
>>
>> Positive side effect: Upon setsockopt(2) failures, the tos and nfmark
>> fields of a pinned connection were set to the desired (but not actually
>> applied) values, while persistent connection fields were left intact
>> (and, hence, stale). Both fields are now reset to zero on failures, for
>> both types of connections.
>>
>
> +1.
Committed to trunk (r14367).
Thank you,
Alex.
More information about the squid-dev
mailing list