MPS issue job003340

TitleMPS_TELEMETRY_CONTROL=65535 is ugly and not future-proof
Statusclosed
Priorityoptional
Assigned userRichard Brooksby
OrganizationRavenbrook
DescriptionIf you want to see all events in the telemetry stream, you have to turn them all on in the telemetry filter. The recommended way to do this seems to be to set

MPS_TELEMETRY_CONTROL=65535

but it is not clear what it means (turn on the first 16 event categories?) and hard to document. Also it is not future-proof: should there ever be more than 16 event categories it would go wrong. If we are recommending that people put this kind of thing in their .gdbinit file, we should have something better than this.
AnalysisPerhaps just support

MPS_TELEMETRY_CONTROL=all

RB thought that there might be a slippery slope here: if we allow this, then we'll have to allow

MPS_TELEMETRY_CONTROL=all-Arena+ArenaAlloc

or whatever, but I think that's unlikely. However, while we're bike-shedding, I think the event categories should be made case-insensitive, so that you can write

MPS_TELEMETRY_CONTROL="arena pool"

instead of this meaning "no events" as at present.
How foundinspection
EvidencePers. comm. from RB.
Observed in1.110.0
Created byGareth Rees
Created on2012-10-23 15:58:51
Last modified byNick Barnes
Last modified on2012-10-26 13:04:27
History2012-10-23 GDR Created.

Fixes

Change Effect Date User Description
180098 closed 2012-10-26 13:03:44 Nick Barnes Make MPS_TELEMETRY_CONTROL case-insensitive, and make it accept "all" as an event class.