[master] aa80275 Remove lingering VSL flush calls from when we used FD as log ID.

Martin Blix Grydeland martin at varnish-software.com
Tue Sep 9 16:47:30 CEST 2014


commit aa802750fae04c36fb2ab6d23fec57dee701286c
Author: Martin Blix Grydeland <martin at varnish-software.com>
Date:   Wed Aug 27 17:08:40 2014 +0200

    Remove lingering VSL flush calls from when we used FD as log ID.

diff --git a/bin/varnishd/cache/cache_backend.c b/bin/varnishd/cache/cache_backend.c
index 6622f4b..9854b41 100644
--- a/bin/varnishd/cache/cache_backend.c
+++ b/bin/varnishd/cache/cache_backend.c
@@ -283,10 +283,6 @@ vbe_GetVbe(struct busyobj *bo, struct vdi_simple *vs)
 		VSLb(bo->vsl, SLT_BackendClose, "%d %s toolate",
 		    vc->fd, bp->display_name);
 
-		/* Checkpoint log to flush all info related to this connection
-		   before the OS reuses the FD */
-		VSL_Flush(bo->vsl, 0);
-
 		VTCP_close(&vc->fd);
 		VBE_DropRefConn(bp, NULL);
 		vc->backend = NULL;
diff --git a/bin/varnishd/cache/cache_dir.c b/bin/varnishd/cache/cache_dir.c
index 5d374ab..9687366 100644
--- a/bin/varnishd/cache/cache_dir.c
+++ b/bin/varnishd/cache/cache_dir.c
@@ -56,13 +56,7 @@ VDI_CloseFd(struct vbc **vbp, const struct acct_bereq *acct_bereq)
 
 	VSLb(vc->vsl, SLT_BackendClose, "%d %s", vc->fd, bp->display_name);
 
-	/*
-	 * Checkpoint log to flush all info related to this connection
-	 * before the OS reuses the FD
-	 */
-	VSL_Flush(vc->vsl, 0);
 	vc->vsl = NULL;
-
 	VTCP_close(&vc->fd);
 	VBE_DropRefConn(bp, acct_bereq);
 	vc->backend = NULL;
@@ -88,8 +82,6 @@ VDI_RecycleFd(struct vbc **vbp, const struct acct_bereq *acct_bereq)
 
 	VSLb(vc->vsl, SLT_BackendReuse, "%d %s", vc->fd, bp->display_name);
 
-	/* XXX: revisit this hack */
-	VSL_Flush(vc->vsl, 0);
 	vc->vsl = NULL;
 
 	Lck_Lock(&bp->mtx);



More information about the varnish-commit mailing list