<div dir="ltr">When there are multiple VCLs loaded would they all be exposed via VSM?<br><br>I think kicking off a child to do the work only when necessary would be better and you get the auth check bonus.<br></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Jul 22, 2014 at 10:08 AM, Poul-Henning Kamp <span dir="ltr"><<a href="mailto:phk@phk.freebsd.dk" target="_blank">phk@phk.freebsd.dk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In message <CAEh05Va0upueb8CDQEdNh=NXhpf_=<a href="mailto:rZ1iTfr5EKzrR9tFyi9Hw@mail.gmail.com">rZ1iTfr5EKzrR9tFyi9Hw@mail.gmail.com</a>><br>
, Dag Haavi Finstad writes:<br>
<br>
>As discussed on the Stockholm VDD, here's a proposal for #1457.<br>
>[...]<br>
>The VCLs are output in the same order as they appear in the VCL<br>
>location/profiling table in the compiled VCL (the vcl->ref stuff used<br>
>in the VCL_trace records), so if someone wanted to do a VCL code<br>
>tracer thing, that should be possible.<br>
<br>
I'm not at all happy with the fact that we still dlopen() VCL's for<br>
vcl.show().  That exposes the master process in ways I don't like,<br>
even more so now we have VMODS which pull in strange libraries.<br>
<br>
One way to fix it is to accept that we cannot show the source<br>
unless the child process is running.  I can live with that.<br>
<br>
If that is not acceptable, we could fork a child of master to<br>
dlopen() the compiled VCL (we already do that for testing) and<br>
pull the source out of that.<br>
<br>
But all in all I prefer to make the child responsible, so that<br>
we don't dlopen() compiled VCL's unnecessarily.<br>
<br>
If we go that route, we can do as now, and only exposed it via<br>
VCL or we can expose the counters and VCL source via VSM to<br>
avoid the VCL overhead.<br>
<br>
The latter is probably best for interactive visuals, but does<br>
expose the VCL source without CLI auth check.  I'm not sure<br>
if that is OK.<br>
<br>
Input ?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20<br>
phk@FreeBSD.ORG         | TCP/IP since RFC 956<br>
FreeBSD committer       | BSD since 4.3-tahoe<br>
Never attribute to malice what can adequately be explained by incompetence.<br>
<br>
_______________________________________________<br>
varnish-dev mailing list<br>
<a href="mailto:varnish-dev@varnish-cache.org">varnish-dev@varnish-cache.org</a><br>
<a href="https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev" target="_blank">https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev</a><br>
</font></span></blockquote></div><br></div>