Skip to content

Commit 9785bf2

Browse files
author
Anurag Dixit
committed
(core//): Fixed inverted TRTORCH_CHECK conditions
Signed-off-by: Anurag Dixit <[email protected]>
1 parent 9c70a39 commit 9785bf2

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

core/compiler.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ torch::jit::script::Module CompileGraph(const torch::jit::script::Module& mod,
174174
}
175175

176176
void set_device(const int gpu_id) {
177-
TRTORCH_CHECK((cudaSetDevice(gpu_id) != cudaSuccess), "Unable to set CUDA device: " << gpu_id);
177+
TRTORCH_CHECK((cudaSetDevice(gpu_id) == cudaSuccess), "Unable to set CUDA device: " << gpu_id);
178178
}
179179

180180
} // namespace core

core/execution/TRTEngine.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,10 @@ TRTEngine::TRTEngine(std::string mod_name, std::string serialized_engine, std::s
4242
util::logging::get_logger().get_reportable_severity(),
4343
util::logging::get_logger().get_is_colored_output_on()) {
4444

45-
CudaDevice cuda_device;
4645
// Deserialize device meta data if device_info is non-empty
4746
if (!serialized_device_info.empty())
4847
{
49-
cuda_device = deserialize_device(serialized_device_info);
48+
auto cuda_device = deserialize_device(serialized_device_info);
5049
// Set CUDA device as configured in serialized meta data
5150
set_cuda_device(cuda_device);
5251
}
@@ -119,6 +118,7 @@ static auto TRTORCH_UNUSED TRTEngineTSRegistrtion = torch::class_<TRTEngine>("te
119118
auto trt_engine = std::string((const char*)serialized_trt_engine->data(), serialized_trt_engine->size());
120119

121120
CudaDevice cuda_device;
121+
get_cuda_device(cuda_device);
122122
std::vector<std::string> serialize_info;
123123
serialize_info.push_back(serialize_device(cuda_device));
124124
serialize_info.push_back(trt_engine);
@@ -155,13 +155,13 @@ void CudaDevice::set_minor(int minor) {
155155
}
156156

157157
void set_cuda_device(CudaDevice& cuda_device) {
158-
TRTORCH_CHECK((cudaSetDevice(cuda_device.id) != cudaSuccess), "Unable to set device: " << cuda_device.id);
158+
TRTORCH_CHECK((cudaSetDevice(cuda_device.id) == cudaSuccess), "Unable to set device: " << cuda_device.id);
159159
}
160160

161161
void get_cuda_device(CudaDevice& cuda_device) {
162-
TRTORCH_CHECK((cudaGetDevice(&cuda_device.id) != cudaSuccess), "Unable to get current device: " << cuda_device.id);
162+
TRTORCH_CHECK((cudaGetDevice(&cuda_device.id) == cudaSuccess), "Unable to get current device: " << cuda_device.id);
163163
cudaDeviceProp device_prop;
164-
TRTORCH_CHECK((cudaGetDeviceProperties(&device_prop, cuda_device.id) != cudaSuccess), "Unable to get CUDA properties from device:" << cuda_device.id);
164+
TRTORCH_CHECK((cudaGetDeviceProperties(&device_prop, cuda_device.id) == cudaSuccess), "Unable to get CUDA properties from device:" << cuda_device.id);
165165
cuda_device.set_major(device_prop.major);
166166
cuda_device.set_minor(device_prop.minor);
167167
}

0 commit comments

Comments
 (0)