tenstorrent / tt-umd

User-Mode Driver for Tenstorrent hardware
Apache License 2.0
9 stars 5 forks source link

SoCDescriptor #100

Open broskoTT opened 1 month ago

broskoTT commented 1 month ago

Related to https://docs.google.com/drawings/d/1-m1azdsBqMA0A6ATYRMfkhyeuOJuGCEI62N5a96LXj0/edit?usp=sharing This should not be something which is given to UMD, rather provided by UMD. Remove passing socDescriptor to constructor of tt_SiliconDriver. Add default SoCDescriptors (from tt-metal repo) to UMD repo. Construct one of these based on arch type reported by TTDevice/PCIDevice class.

See if some methods from metal_SocDescriptor make sense to be moved here.

broskoTT commented 4 weeks ago

Per chat on the main channel, make sure to leave the variant with passing a custom soc descriptor in case it is needed for some custom testing

broskoTT commented 4 weeks ago

@pjanevskiTT per chat from today we've had. If you're enthusiastic, you can try to embed default soc_descriptor files while building, using "file" and "configure_file" commands from cmake to embed the contents of soc_desciptors directly into c++ code. Then the actual binary won't need files in the right place alongside it.