I see what the problem is now. Patch on the way. Bill
> > At 08:47 AM 10/18/2002, Bill Stoddard wrote: > >Hey Bill, > >Just ran into a problem with this patch... mod_mem_cache has an option to > >cache open file descriptors. With this patch, we will never used > sendfile to > >send a cached fd. > > This 'should' work... where is the code broken? > > We cache the open file descriptor from a first request, no? So the flag > should be set on that initial request, correct? When we set > aside the 'fd', > we really must set aside the flags (so we know what sort of fd we set > aside.) The apr_file_dup should be 'keeping' that flag from the original > fd when creating the long-lived fd. > > Can you point out where in the code that flag disappears? > > >wrowe 2002/10/14 10:42:45 > > > > Modified: include http_core.h > > server core.c > > Log: > > Introduce an EnableSendfile directive (defaulted to ON) to > allow users > > to disable sendfile mechanics for NFS volume mounts and other kernel > > objects that don't support sendfile. And EnableSendfile off > can be used > > to help narrow bugs down to the sendfile mechanics or eliminate the > > possiblity that sendfile is a factor on any given platform. >
