From 2d1643e96ed8bfa7a4ae23f3a2b086f66413c680 Mon Sep 17 00:00:00 2001 From: Cliff Woolley Date: Sat, 23 Nov 2002 21:18:16 +0000 Subject: [PATCH] update a comment git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@97616 13f79535-47bb-0310-9956-ffa450edef68 --- modules/cache/mod_file_cache.c | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/modules/cache/mod_file_cache.c b/modules/cache/mod_file_cache.c index 646d77bf0b..37209e2744 100644 --- a/modules/cache/mod_file_cache.c +++ b/modules/cache/mod_file_cache.c @@ -249,14 +249,12 @@ static void cache_the_file(cmd_parms *cmd, const char *filename, int mmap) return; } apr_file_close(fd); - /* We want to cache an apr_mmap_t that's marked as "non-owner" - * to pass to each request so that mmap_setaside()'s call to - * apr_mmap_dup() will never try to move the apr_mmap_t to a - * different pool. This apr_mmap_t is already going to live - * longer than any request, but mmap_setaside() has no way to - * know that because it's allocated out of cmd->pool, - * which is disjoint from r->pool. - */ + /* We want to cache a duplicate apr_mmap_t to pass to each + * request so that nothing in the request will ever think that + * it's allowed to delete the mmap, since the "refcount" will + * never reach zero. */ + /* XXX: the transfer_ownership flag on this call + * will go away soon.. it's ignored right now. */ apr_mmap_dup(&new_file->mm, mm, cmd->pool, 0); new_file->is_mmapped = TRUE; } -- 2.50.1