Skip to content

telemetry table: remove dependency on table and use tags / offsets directly #61

@cfriedt

Description

@cfriedt

From tenstorrent/tt-system-firmware#248, it would be really great to not have 1kB of SRAM in FW consumed by what amounts to an identity-mapping hash table.

tenstorrent/tt-system-firmware#248 (comment)

The kind of not-so-nice side of this is that we will need to have a graceful deprecation process:

  1. first firmware deprecates the table, for 2 release cycles.
  2. this must somehow cause warnings when compiling userspace applications (not entirely sure how to do that since they are not directly linked), which forces userspace applications to update the method of mapping telemetry tags to array indices (should be y = x - 1 or something like that). Maybe some option is created with a default for the next release, but causes a warning?
  3. firmware removes the table, which will cause userspace applications to break
  4. userspace applications switch to using tags / offsets directly

There will eventually need to be a minimum version for userspace tools / firmware to be compatible.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions