tenstorrent / tt-umd

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

Add env var for AICLK get_clock #18

Closed pjanevskiTT closed 3 months ago

pjanevskiTT commented 3 months ago

Add env var to set value which is going to be retreived from get_clock function for Blackhole.

Lot of Blackhole chips used for testing at the moment are underclocked to 800MHz and ARC messages don't work at the moment so this is a problem when we want to actually read AICLK value. It is needed for testing Blackhole on Buda at the moment for Perf infrastructure

pjanevskiTT commented 3 months ago

@joelsmithTT we need this for Buda, specifically Marko Bezulj needs it for Perf infra on Buda. He is waiting for permissions on UMD, he can comment here when this change is ok for him, and you can approve if you think this is ok. We are not aware of other use cases at the moment when this function is called but since it uses ARC probably we don't want to use it for Blackhole anywhere at the moment

mbezuljTT commented 3 months ago

Add env var to set value which is going to be retreived from get_clock function for Blackhole.

Lot of Blackhole chips used for testing at the moment are underclocked to 800MHz and ARC messages don't work at the moment so this is a problem when we want to actually read AICLK value. It is needed for testing Blackhole on Buda at the moment for Perf infrastructure

Thanks for making this change @pjanevskiTT !