Hi,

Would it be possible to log free session workspace also?

If I set diag_bitmap to 0x42, I get:

    0 Debug        - "WS(0x3a77bbe070 = (sess, 0x3a77bbe790 18 0 8192)"
    0 Debug        - "WS(0x3a0dbe8070 = (sess, 0x3a0dbe8790 20 4096
8192)"
    0 Debug        - "WS(0x3a0dbe8070 = (sess, 0x3a0dbe8790 685 0 8192)"
    0 Debug        - "WS(0x3a0dbe8070 = (sess, 0x3a0dbe8790 685 0 8192)"

What are those three fields at the end? The first is space needed, the
third is space allocated?

Bye,
Anders.

On Fri, Mar 07, 2008 at 12:43:38PM +0100, [EMAIL PROTECTED] wrote:
> Author: phk
> Date: 2008-03-07 12:43:38 +0100 (Fri, 07 Mar 2008)
> New Revision: 2561
> 
> Modified:
>    trunk/varnish-cache/bin/varnishd/cache.h
>    trunk/varnish-cache/bin/varnishd/cache_hash.c
>    trunk/varnish-cache/bin/varnishd/cache_ws.c
> Log:
> Log free object workspace instead of used to get the size of the object
> structure into the picture.
> 
> 
> Modified: trunk/varnish-cache/bin/varnishd/cache.h
> ===================================================================
> --- trunk/varnish-cache/bin/varnishd/cache.h  2008-03-07 11:36:52 UTC (rev 
> 2560)
> +++ trunk/varnish-cache/bin/varnishd/cache.h  2008-03-07 11:43:38 UTC (rev 
> 2561)
> @@ -581,7 +581,7 @@
>  char *WS_Alloc(struct ws *ws, unsigned bytes);
>  char *WS_Dup(struct ws *ws, const char *);
>  char *WS_Snapshot(struct ws *ws);
> -unsigned WS_Used(struct ws *ws);
> +unsigned WS_Free(struct ws *ws);
>  
>  /* rfc2616.c */
>  int RFC2616_cache_policy(const struct sess *sp, const struct http *hp);
> 
> Modified: trunk/varnish-cache/bin/varnishd/cache_hash.c
> ===================================================================
> --- trunk/varnish-cache/bin/varnishd/cache_hash.c     2008-03-07 11:36:52 UTC 
> (rev 2560)
> +++ trunk/varnish-cache/bin/varnishd/cache_hash.c     2008-03-07 11:43:38 UTC 
> (rev 2561)
> @@ -300,7 +300,7 @@
>               VSL_stats->n_objoverflow++;
>       if (params->diag_bitmap & 0x40)
>               WSP(sp, SLT_Debug, 
> -                 "Object workspace used %u", WS_Used(o->ws_o));
> +                 "Object %u workspace free %u", o->xid, WS_Free(o->ws_o));
>       
>       oh = o->objhead;
>       if (oh != NULL) {
> @@ -366,8 +366,8 @@
>               return;
>  
>       if (params->diag_bitmap & 0x40)
> -             VSL(SLT_Debug, 0, 
> -                 "Object workspace max used %u", WS_Used(o->ws_o));
> +             VSL(SLT_Debug, 0, "Object %u workspace min free %u",
> +                 o->xid, WS_Free(o->ws_o));
>  
>       if (o->vary != NULL)
>               free(o->vary);
> 
> Modified: trunk/varnish-cache/bin/varnishd/cache_ws.c
> ===================================================================
> --- trunk/varnish-cache/bin/varnishd/cache_ws.c       2008-03-07 11:36:52 UTC 
> (rev 2560)
> +++ trunk/varnish-cache/bin/varnishd/cache_ws.c       2008-03-07 11:43:38 UTC 
> (rev 2561)
> @@ -136,11 +136,11 @@
>  }
>  
>  unsigned
> -WS_Used(struct ws *ws)
> +WS_Free(struct ws *ws)
>  {
>  
>       WS_Assert(ws);
> -     return(ws->f - ws->s);
> +     return(ws->e - ws->f);
>  }
>  
>  char *
> 
> _______________________________________________
> varnish-commit mailing list
> [EMAIL PROTECTED]
> http://projects.linpro.no/mailman/listinfo/varnish-commit


-- 
Anders.
_______________________________________________
varnish-dev mailing list
varnish-dev@projects.linpro.no
http://projects.linpro.no/mailman/listinfo/varnish-dev

Reply via email to