Skip to content

Commit

Permalink
Moved UMD headers to types/ subfolder (#15753)
Browse files Browse the repository at this point in the history
### Ticket
All related to #13948

### Problem description
Some api header files have been moved around in UMD repo.
UMD PR: tenstorrent/tt-umd#357

### What's changed
- umd/device/tt_arch_type.h -> umd/device/types/arch.h
- umd/device/tt_cluster_descriptor_types.h ->
umd/device/types/cluster_descriptor_types.h
- umd/device/xy_pair.h -> umd/device/types/xy_pair.h
- get_arch_str -> tt::arch_to_str

### Checklist
- [x] All post-commit tests :
https://github.com/tenstorrent/tt-metal/actions/runs/12224723517
- [x] Blackhole post-commit tests :
https://github.com/tenstorrent/tt-metal/actions/runs/12224723937
- [x] (Single-card) Model perf tests :
https://github.com/tenstorrent/tt-metal/actions/runs/12224724459
- [ ] (Single-card) Device perf regressions :
https://github.com/tenstorrent/tt-metal/actions/runs/12224725026
- [x] (T3K) T3000 unit tests :
https://github.com/tenstorrent/tt-metal/actions/runs/12224725640
- [x] (T3K) T3000 demo tests :
https://github.com/tenstorrent/tt-metal/actions/runs/12224726199
- [x] (TG) TG unit tests :
https://github.com/tenstorrent/tt-metal/actions/runs/12224726548
- [x] (TG) TG demo tests :
https://github.com/tenstorrent/tt-metal/actions/runs/12224726936
- [x] (TGG) TGG unit tests :
https://github.com/tenstorrent/tt-metal/actions/runs/12224727791
- [x] (TGG) TGG demo tests :
https://github.com/tenstorrent/tt-metal/actions/runs/12224728493
  • Loading branch information
broskoTT authored Dec 8, 2024
1 parent 93d8d27 commit 9eada08
Show file tree
Hide file tree
Showing 25 changed files with 30 additions and 30 deletions.
2 changes: 1 addition & 1 deletion tests/tt_metal/tt_metal/common/command_queue_fixture.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
#include "dispatch_fixture.hpp"
#include "hostdevcommon/common_values.hpp"
#include "impl/device/device.hpp"
#include "umd/device/tt_cluster_descriptor_types.h"
#include "umd/device/types/cluster_descriptor_types.h"
#include "tt_metal/host_api.hpp"
#include "tt_metal/detail/tt_metal.hpp"
#include "tt_metal/test_utils/env_vars.hpp"
Expand Down
2 changes: 1 addition & 1 deletion tests/tt_metal/tt_metal/common/multi_device_fixture.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

#include "host_api.hpp"
#include "dispatch_fixture.hpp"
#include "umd/device/tt_cluster_descriptor_types.h"
#include "umd/device/types/cluster_descriptor_types.h"
#include "tt_metal/test_utils/env_vars.hpp"
#include "tt_metal/impl/device/device_pool.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
#include "impl/kernels/data_types.hpp"
#include "impl/kernels/kernel_types.hpp"
#include "impl/program/program.hpp"
#include "umd/device/tt_cluster_descriptor_types.h"
#include "umd/device/types/cluster_descriptor_types.h"
#include "program_with_kernel_created_from_string_fixture.hpp"

TEST_F(ProgramWithKernelCreatedFromStringFixture, TensixDataMovementKernel) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
#include "hostdevcommon/common_values.hpp"
#include "impl/device/device.hpp"
#include "llrt/hal.hpp"
#include "umd/device/tt_cluster_descriptor_types.h"
#include "umd/device/types/cluster_descriptor_types.h"
#include "tt_metal/host_api.hpp"
#include "tt_metal/detail/tt_metal.hpp"
#include "tt_metal/test_utils/env_vars.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
#include <tuple>

#include "gtest/gtest.h"
#include "umd/device/tt_arch_types.h"
#include "umd/device/types/arch.h"
#include "command_queue_fixture.hpp"
#include "tt_metal/common/logger.hpp"
#include "impl/device/device.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
//
// SPDX-License-Identifier: Apache-2.0

#include "umd/device/tt_cluster_descriptor_types.h"
#include "umd/device/types/cluster_descriptor_types.h"
#include "tt_metal/host_api.hpp"
#include "tt_metal/detail/tt_metal.hpp"
#include "tt_metal/impl/dispatch/command_queue.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
#include <random>
#include <tuple>

#include "umd/device/tt_arch_types.h"
#include "umd/device/types/arch.h"
#include "impl/device/device.hpp"
#include "impl/kernels/kernel_types.hpp"
#include "tt_backend_api_types.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@

#include "tt_metal/distributed/mesh_device_view.hpp"
#include "tt_metal/common/logger.hpp"
#include "umd/device/tt_arch_types.h"
#include "umd/device/types/arch.h"
#include "impl/device/device.hpp"
#include "impl/kernels/data_types.hpp"
#include "impl/kernels/kernel_types.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
#include <tuple>
#include <map>

#include "umd/device/tt_arch_types.h"
#include "umd/device/types/arch.h"
#include "impl/device/device.hpp"
#include "impl/kernels/kernel_types.hpp"
#include "tt_backend_api_types.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
#include <limits>
#include <random>

#include "umd/device/tt_arch_types.h"
#include "umd/device/types/arch.h"
#include "tt_backend_api_types.hpp"
#include "tt_metal/common/core_coord.hpp"
#include "tt_metal/common/math.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@

#include "gtest/gtest.h"

#include "umd/device/tt_arch_types.h"
#include "umd/device/types/arch.h"
// #include "tt_backend_api_types.hpp"
#include "tt_metal/common/core_coord.hpp"
#include "tt_metal/common/math.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
#include <limits>
#include <random>

#include "umd/device/tt_arch_types.h"
#include "umd/device/types/arch.h"
#include "gtest/gtest.h"
// #include "tt_backend_api_types.hpp"
#include "tt_metal/common/core_coord.hpp"
Expand Down
2 changes: 1 addition & 1 deletion tt_metal/common/tt_backend_api_types.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
#include <vector>

#include "fmt/base.h"
#include "umd/device/tt_arch_types.h"
#include "umd/device/types/arch.h"

namespace tt {

Expand Down
2 changes: 1 addition & 1 deletion tt_metal/detail/tt_metal.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
#include <vector>
#include <map>

#include "umd/device/tt_cluster_descriptor_types.h"
#include "umd/device/types/cluster_descriptor_types.h"
#include "umd/device/tt_soc_descriptor.h"
#include "tt_metal/hostdevcommon/common_values.hpp"
#include "tt_metal/common/core_coord.hpp"
Expand Down
2 changes: 1 addition & 1 deletion tt_metal/distributed/mesh_device.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
#include <unordered_map>
#include <utility>

#include "umd/device/tt_cluster_descriptor_types.h"
#include "umd/device/types/cluster_descriptor_types.h"
#include "tt_metal/common/logger.hpp"
#include "tt_metal/detail/tt_metal.hpp"
#include "tt_metal/host_api.hpp"
Expand Down
2 changes: 1 addition & 1 deletion tt_metal/impl/allocator/l1_banking_allocator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
#include "tt_metal/impl/buffers/buffer_constants.hpp"
#include "tt_metal/common/assert.hpp"
#include "tt_metal/common/core_coord.hpp"
#include "umd/device/xy_pair.h"
#include "umd/device/types/xy_pair.h"
#include <fmt/base.h>

#include "llrt/hal.hpp"
Expand Down
2 changes: 1 addition & 1 deletion tt_metal/impl/buffers/buffer.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
#include "tt_metal/impl/buffers/buffer_constants.hpp"
#include "tt_metal/impl/sub_device/sub_device_types.hpp"
#include "umd/device/tt_soc_descriptor.h"
#include "umd/device/xy_pair.h"
#include "umd/device/types/xy_pair.h"
#include "tt_metal/tt_stl/concepts.hpp"
#include "tt_metal/common/assert.hpp"
#include "third_party/json/json.hpp"
Expand Down
4 changes: 2 additions & 2 deletions tt_metal/impl/debug/watcher_device_reader.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@
#include "eth_l1_address_map.h" // for address_map
#include "hw/inc/dev_msgs.h"

#include "umd/device/tt_arch_types.h"
#include "umd/device/xy_pair.h"
#include "umd/device/types/arch.h"
#include "umd/device/types/xy_pair.h"
#include <fmt/base.h>
#include "llrt/llrt.hpp"
#include "llrt/tt_cluster.hpp"
Expand Down
2 changes: 1 addition & 1 deletion tt_metal/impl/device/device_pool.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

#pragma once

#include "umd/device/tt_cluster_descriptor_types.h"
#include "umd/device/types/cluster_descriptor_types.h"
#include "tt_metal/host_api.hpp"
#include "impl/debug/dprint_server.hpp"
#include "tt_metal/impl/device/device.hpp"
Expand Down
2 changes: 1 addition & 1 deletion tt_metal/jit_build/data_format.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
#include <cstdint>
#include <vector>
#include "common/tt_backend_api_types.hpp" // for DataFormat
#include "umd/device/tt_arch_types.h" // for ARCH
#include "umd/device/types/arch.h" // for ARCH
#include "tt_metal/hw/inc/circular_buffer_constants.h" // for NUM_CIRCULAR_BUFFERS

enum class UnpackToDestMode : std::uint8_t;
Expand Down
6 changes: 3 additions & 3 deletions tt_metal/llrt/get_platform_architecture.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ namespace tt::tt_metal {
* if (arch == tt::ARCH::Invalid) {
* std::cerr << "Failed to detect architecture!" << std::endl;
* } else {
* std::cout << "Detected architecture: " << tt::get_arch_str(arch) << std::endl;
* std::cout << "Detected architecture: " << tt::arch_to_str(arch) << std::endl;
* }
* @endcode
*
Expand All @@ -68,9 +68,9 @@ inline tt::ARCH get_platform_architecture() {
TT_FATAL(
arch == detected_arch,
"Expected all devices to be {} but device {} is {}",
get_arch_str(arch),
tt::arch_to_str(arch),
device_id,
get_arch_str(detected_arch));
tt::arch_to_str(detected_arch));
}
}
}
Expand Down
6 changes: 3 additions & 3 deletions tt_metal/llrt/tt_cluster.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,13 +26,13 @@
#include "tt_metal/common/metal_soc_descriptor.h"
#include "tt_metal/common/test_common.hpp"
#include "tt_metal/common/tt_backend_api_types.hpp"
#include "umd/device/tt_arch_types.h"
#include "umd/device/types/arch.h"
#include "umd/device/tt_cluster_descriptor.h"
#include "umd/device/tt_cluster_descriptor_types.h"
#include "umd/device/types/cluster_descriptor_types.h"
#include "umd/device/cluster.h"
#include "umd/device/tt_soc_descriptor.h"
#include "umd/device/tt_xy_pair.h"
#include "umd/device/xy_pair.h"
#include "umd/device/types/xy_pair.h"
#include "umd/device/hugepage.h"

// TODO: ARCH_NAME specific, must remove
Expand Down
2 changes: 1 addition & 1 deletion tt_metal/third_party/umd
Submodule umd updated 33 files
+4 −0 device/CMakeLists.txt
+3 −2 device/api/umd/device/architecture_implementation.h
+4 −0 device/api/umd/device/blackhole_implementation.h
+9 −202 device/api/umd/device/cluster.h
+1 −1 device/api/umd/device/coordinate_manager.h
+4 −0 device/api/umd/device/grayskull_implementation.h
+1 −1 device/api/umd/device/hugepage.h
+17 −71 device/api/umd/device/pci_device.hpp
+0 −20 device/api/umd/device/tt_arch_types.h
+2 −3 device/api/umd/device/tt_cluster_descriptor.h
+17 −0 device/api/umd/device/tt_device/blackhole_tt_device.h
+16 −0 device/api/umd/device/tt_device/grayskull_tt_device.h
+108 −0 device/api/umd/device/tt_device/tt_device.h
+16 −0 device/api/umd/device/tt_device/wormhole_tt_device.h
+1 −38 device/api/umd/device/tt_soc_descriptor.h
+2 −1 device/api/umd/device/tt_xy_pair.h
+62 −0 device/api/umd/device/types/arch.h
+9 −8 device/api/umd/device/types/cluster_descriptor_types.h
+208 −0 device/api/umd/device/types/cluster_types.h
+0 −0 device/api/umd/device/types/xy_pair.h
+4 −0 device/api/umd/device/wormhole_implementation.h
+135 −134 device/cluster.cpp
+7 −338 device/pcie/pci_device.cpp
+1 −14 device/tt_cluster_descriptor.cpp
+0 −34 device/tt_device.cpp
+32 −0 device/tt_device/blackhole_tt_device.cpp
+12 −0 device/tt_device/grayskull_tt_device.cpp
+313 −0 device/tt_device/tt_device.cpp
+12 −0 device/tt_device/wormhole_tt_device.cpp
+1 −17 device/tt_soc_descriptor.cpp
+1 −1 device/xy_pair.cpp
+1 −2 tests/api/test_chip.cpp
+2 −19 tests/api/test_mockup_device.cpp
2 changes: 1 addition & 1 deletion ttnn/cpp/ttnn/operations/ccl/erisc_datamover_builder.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
#include <vector>

#include "eth_l1_address_map.h"
#include "umd/device/tt_cluster_descriptor_types.h"
#include "umd/device/types/cluster_descriptor_types.h"
#include "ttnn/cpp/ttnn/operations/ccl/kernels/edm_fabric/fabric_edm_types.hpp"
#include "ttnn/cpp/ttnn/operations/ccl/shared_with_host/hetergeneous_data_structs.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
#include <variant>
#include <tuple>
#include <optional>
#include "umd/device/tt_arch_types.h"
#include "umd/device/types/arch.h"
#include "tt_metal/common/base_types.hpp"

namespace ttnn {
Expand Down

0 comments on commit 9eada08

Please sign in to comment.