<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
2016-07-20 16:21 GMT+03:00 Amos Jeffries
<a class="moz-txt-link-rfc2396E" href="mailto:squid3@treenet.co.nz"><squid3@treenet.co.nz></a>:<br>
> * smpAware() is documented as indicating whether CF is allowed.
Yet the<br>
> the new comment inside the loop of Store::Disks::smpAware()
seems to be<br>
> saying the opposite.<br>
<br>
As this patch says, collapsing for revalidation requests is not<br>
supported for SMP-aware caches, i.e. if we have configured several<br>
caches and even one of them is SMP-aware, the new feature should be<br>
disabled.<br>
<br>
> * collapsing requires Transients to exist yes?<br>
> So why disable Transients initialization when !smpAware() ?<br>
<br>
Transients is not required for collapsing to work for:<br>
* Non-SMP Squid.<br>
* SMP Squid with SMP-unaware-caches(like if we had several
independent<br>
Squid instances). In this configuration no sharing among workes<br>
is supposed. Since Transients is a sharing mechanism, it should
be<br>
disabled in this case.<br>
<br>
<br>
Eduard.<br>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
</body>
</html>