On 12/27/2008 10:39 PM, Paul Querna wrote: > Ruediger Pluem wrote: >> >> On 12/27/2008 07:27 PM, André Malo wrote: >>> * Ruediger Pluem wrote: >>> >>>> On 12/27/2008 04:07 PM, André Malo wrote:
>>> Isn't that private config? >> >> At least it is one of another module. And yes I agree if there is no >> official >> API to get the setting of d->enable_mmap (currently too lazy to check >> this) it >> is the best to stay with the subrequest approach. > > This is a general flaw with hoe things like EnableMMAP and > EnableSendfile work -- we really should have an API that passes in a > filepath to test for each attribute, because right now when you insert > the buckets into the brigade, you need to enable sendfile and mmap on > creation, and the core filters won't disable them even if they are > disabled. I agree, but I think that the sendfile case is already handled by the core filter and only the MMAP case needs our attention here. The MMAP case needs attention at the point of time we insert the file bucket as any subsequent read on this bucket already starts the MMAP conversion in contrast to sendfile where this decision only matters until we really go on the wire (in the core output filter). Regards Rüdiger
