~squid/squid/storeentry-refactor : changes

~squid/squid/storeentry-refactor » Changes from revision 13209

From Revision 13209 to 13190

Rev	Summary	Authors	Date
13209	Added missing includes Added missing includes	Francesco Chemolli	10 years ago
13208	Added missing include Added missing include	Francesco Chemolli	10 years ago
13207	Split StoreEntry declaration from Store Split StoreEntry declaration from Store	Francesco Chemolli	10 years ago
13206	BaseMultiQueue is virtual but had no virtual destructor BaseMultiQueue is virtual but had no virtual destructor	Francesco Chemolli	10 years ago
13205	Got rid of Store.cci Got rid of Store.cci	Francesco Chemolli	10 years ago
13204	Fix SBuf::rfind Fix SBuf::rfind	Amos Jeffries	10 years ago
13203	Fix SBuf and SBufList unit tests linkage Fix SBuf and SBufList unit tests linkage	Francesco Chemolli	10 years ago
13202	tests/testSBufList on CentOS needs MemObject::endOffset(). tests/testSBufList on CentOS needs MemObject::endOffset().	Alex Rousskov	10 years ago
13201	Initial Large Rock and Collapsed Forwarding support. Initial Large Rock and Collapsed Forwarding support. Large Rock: Support disk (and shared memory) caching of responses exceeding one db slot (or one shared memory page) in size. A single db slot/page size is still limited to 32KB (smaller values can be configured for disk caches using the newly added cache_dir slot-size option). Removal of old rock cache dir (followed by squid-z) is required -- the on-disk db structure has changed. Collapsed Forwarding: Optionally merge concurrent cachable requests for the same URI earlier: After the request headers have been parsed (as before), but now _before_ the response headers have been received. Merging of requests received by different SMP workers is supported. Controlled by the new collapsed_forwarding directive in squid.conf. Disabled by default because all but one of the merged requests have to be delayed (until the response headers are received) for the merging to work, which may be worse than forwarding all concurrent requests immediately. The overall feature idea and request eligibility conditions are based on Collapsed Forwarding in Squid2. Summary of other important changes (merged branch log contains the details): * Tightened StoreEntry locking. Split StoreEntry::lock() into "just lock" and "update entry reference time" interfaces, addressing an old XXX. Improved entry lock/unlock debugging. Needs more work. * Adjusted StoreIOState::write() API to allow callers detect write errors. * Simplified MemObject::write() API to remove an essentially unused callback. * Mark client streams that sent everything as STREAM_COMPLETE. The old code used STREAM_UNPLANNED_COMPLETE if the completed stream was associated with a non-persistent connection, which did not make sense to me and, IIRC, led to store entry aborts even though the entries were not damaged in any way. * mem_hdr::hasContigousContentRange() now returns true for empty ranges. * Support "appending" ReadWriteLock state that can be shared by readers and the writer. The writer promises not to update key metadata (except growing object size and next pointers) and readers promise to be careful when reading growing slices. * Fixed StoreEntry::mayStartSwapOut() logic to handle terminated swapouts. * Improved STORE_MEM_CLIENT detection and documented known (and mostly old) StoreEntry::storeClientType() problems. * Removed StoreEntry::hidden_mem_obj hack. * Polished StoreEntry debugging to report more info, less noise. Use e: prefix. * Added a script to extract store entry(ies) debugging from cache.log.	Alex Rousskov	10 years ago
13200	Added MemBlob::appended() method. Handy during out-of-band b... Added MemBlob::appended() method. Handy during out-of-band buffer manipulation.	Alex Rousskov	10 years ago
13199	Prep for 3.4.2 Prep for 3.4.2	Amos Jeffries	10 years ago
13198	Bug 3498: FTP PUT assertion Server.cc:246: "r->body_pipe != ... Bug 3498: FTP PUT assertion Server.cc:246: "r->body_pipe != NULL"	Alex Rousskov	10 years ago
13197	Bug 3985: 60s limit introduced by balance_on_multiple_ip bre... Bug 3985: 60s limit introduced by balance_on_multiple_ip breaks bad IP recovery Alternative fix if necessary would be to calculate a dynamic timeout value based on number and timeouts for connections and forwarding. However, as noted in the bug report the address markings are reset every TTL when the stored RR are replaced. This meets the criteria for periodic retries of marked-BAD IPs. Also if all IPs are marked BAD they all get un-marked and retried. So for now just remove the TTL hack.	Jakob Bohm	10 years ago
13196	Fix \-unescaping in quoted strings from helpers Fix \-unescaping in quoted strings from helpers strwordtok() was unescaping '\' values in unquoted tokens. This causes problems with NTLM helper response user\DOMAIN values which are sent as a un-quoted un-escaped value. Restrict un-escaping of '\' to only occur when inside "quoted" words.	Amos Jeffries	10 years ago
13195	WCCPv2: fix assertion 'Cannot convert non-IPv4 to IPv4' on F... WCCPv2: fix assertion 'Cannot convert non-IPv4 to IPv4' on FreeBSD FreeBSD does not set the from-address on recv() if there was an error reading the packet. This can result in Ip::Address assertion if the error is not checked for before the IP address is mapped into Squid internal format.	Amos Jeffries	10 years ago
13194	Fix missing cast in rev.13162 Fix missing cast in rev.13162	Amos Jeffries	10 years ago
13193	Added missing header in client_side_reply.cc for clang Added missing header in client_side_reply.cc for clang	Francesco Chemolli	10 years ago
13192	Fix missing include in rev.13190 Fix missing include in rev.13190	Amos Jeffries	10 years ago
13191	Rename SBuf::find_first_of and find_first_not_of to camelCas... Rename SBuf::find_first_of and find_first_not_of to camelCase.	Francesco Chemolli	10 years ago
13190	Added send_hit and store_miss squid.conf directives Added send_hit and store_miss squid.conf directives to control caching of responses using response info. The existing "cache" directive is checked before Squid has access to the response and, hence, could not use response-based ACLs such as http_status. Response-based ACLs may be essential when fine-tuning caching. Squid Bug 3937 (StoreID can lead to 302 infinite loop) is a good use case. Updated old "cache" directive documentation to provide more information, to help folks distinguish the three related directives, and to polish for clarity. TODO: Support lookup_hit and possibly deprecate/remove "cache".	Alex Rousskov	10 years ago

Older »