[squid-dev] [PATCH] Purge cache entries in SMP-aware caches

Amos Jeffries squid3 at treenet.co.nz
Sat Aug 5 05:57:28 UTC 2017


On 05/08/17 07:52, Eduard Bagdasaryan wrote:
> To avoid this 'going around releaseRequest() API' I would like
> to suggest renaming 'RELEASE_REQUEST' flag to something
> more suitable, e.g., 'ENTRY_REUSABLE' and adding a helper
> API avoiding direct flag manipulation.

I don't see how renaming would help much. Any API control over use of a 
flag can be added regardless of its name.

Also, the 'RELEASE' flag value being true indicates REUSABLE false 
value. So the rename implies a careful and complete inversion of all 
logic conditions relying on or otherwise touching the flag. Which risks 
many new bug additions.

If done at all I think the rename should be left out-of-scope for this 
PR / patching.

Amos


More information about the squid-dev mailing list