Validate during NN conversions by default -- hal am: 6547b2ac9c am: ee3a8f4970

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1520618

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I00283ad0353ea76bfd0927d2fdd98640b5f94208
This commit is contained in:
Michael Butler
2020-12-08 18:38:36 +00:00
committed by Automerger Merge Worker
24 changed files with 994 additions and 609 deletions

View File

@@ -24,20 +24,28 @@
namespace android::nn { namespace android::nn {
GeneralResult<OperandType> convert(const hal::V1_0::OperandType& operandType); GeneralResult<OperandType> unvalidatedConvert(const hal::V1_0::OperandType& operandType);
GeneralResult<OperationType> convert(const hal::V1_0::OperationType& operationType); GeneralResult<OperationType> unvalidatedConvert(const hal::V1_0::OperationType& operationType);
GeneralResult<Operand::LifeTime> convert(const hal::V1_0::OperandLifeTime& lifetime); GeneralResult<Operand::LifeTime> unvalidatedConvert(const hal::V1_0::OperandLifeTime& lifetime);
GeneralResult<DeviceStatus> convert(const hal::V1_0::DeviceStatus& deviceStatus); GeneralResult<DeviceStatus> unvalidatedConvert(const hal::V1_0::DeviceStatus& deviceStatus);
GeneralResult<Capabilities::PerformanceInfo> convert( GeneralResult<Capabilities::PerformanceInfo> unvalidatedConvert(
const hal::V1_0::PerformanceInfo& performanceInfo); const hal::V1_0::PerformanceInfo& performanceInfo);
GeneralResult<Capabilities> unvalidatedConvert(const hal::V1_0::Capabilities& capabilities);
GeneralResult<DataLocation> unvalidatedConvert(const hal::V1_0::DataLocation& location);
GeneralResult<Operand> unvalidatedConvert(const hal::V1_0::Operand& operand);
GeneralResult<Operation> unvalidatedConvert(const hal::V1_0::Operation& operation);
GeneralResult<Model::OperandValues> unvalidatedConvert(
const hardware::hidl_vec<uint8_t>& operandValues);
GeneralResult<Memory> unvalidatedConvert(const hardware::hidl_memory& memory);
GeneralResult<Model> unvalidatedConvert(const hal::V1_0::Model& model);
GeneralResult<Request::Argument> unvalidatedConvert(
const hal::V1_0::RequestArgument& requestArgument);
GeneralResult<Request> unvalidatedConvert(const hal::V1_0::Request& request);
GeneralResult<ErrorStatus> unvalidatedConvert(const hal::V1_0::ErrorStatus& status);
GeneralResult<DeviceStatus> convert(const hal::V1_0::DeviceStatus& deviceStatus);
GeneralResult<Capabilities> convert(const hal::V1_0::Capabilities& capabilities); GeneralResult<Capabilities> convert(const hal::V1_0::Capabilities& capabilities);
GeneralResult<DataLocation> convert(const hal::V1_0::DataLocation& location);
GeneralResult<Operand> convert(const hal::V1_0::Operand& operand);
GeneralResult<Operation> convert(const hal::V1_0::Operation& operation);
GeneralResult<Model::OperandValues> convert(const hardware::hidl_vec<uint8_t>& operandValues);
GeneralResult<Memory> convert(const hardware::hidl_memory& memory);
GeneralResult<Model> convert(const hal::V1_0::Model& model); GeneralResult<Model> convert(const hal::V1_0::Model& model);
GeneralResult<Request::Argument> convert(const hal::V1_0::RequestArgument& requestArgument);
GeneralResult<Request> convert(const hal::V1_0::Request& request); GeneralResult<Request> convert(const hal::V1_0::Request& request);
GeneralResult<ErrorStatus> convert(const hal::V1_0::ErrorStatus& status); GeneralResult<ErrorStatus> convert(const hal::V1_0::ErrorStatus& status);
@@ -45,21 +53,28 @@ GeneralResult<ErrorStatus> convert(const hal::V1_0::ErrorStatus& status);
namespace android::hardware::neuralnetworks::V1_0::utils { namespace android::hardware::neuralnetworks::V1_0::utils {
nn::GeneralResult<OperandType> convert(const nn::OperandType& operandType); nn::GeneralResult<OperandType> unvalidatedConvert(const nn::OperandType& operandType);
nn::GeneralResult<OperationType> convert(const nn::OperationType& operationType); nn::GeneralResult<OperationType> unvalidatedConvert(const nn::OperationType& operationType);
nn::GeneralResult<OperandLifeTime> convert(const nn::Operand::LifeTime& lifetime); nn::GeneralResult<OperandLifeTime> unvalidatedConvert(const nn::Operand::LifeTime& lifetime);
nn::GeneralResult<DeviceStatus> convert(const nn::DeviceStatus& deviceStatus); nn::GeneralResult<DeviceStatus> unvalidatedConvert(const nn::DeviceStatus& deviceStatus);
nn::GeneralResult<PerformanceInfo> convert( nn::GeneralResult<PerformanceInfo> unvalidatedConvert(
const nn::Capabilities::PerformanceInfo& performanceInfo); const nn::Capabilities::PerformanceInfo& performanceInfo);
nn::GeneralResult<Capabilities> unvalidatedConvert(const nn::Capabilities& capabilities);
nn::GeneralResult<DataLocation> unvalidatedConvert(const nn::DataLocation& location);
nn::GeneralResult<Operand> unvalidatedConvert(const nn::Operand& operand);
nn::GeneralResult<Operation> unvalidatedConvert(const nn::Operation& operation);
nn::GeneralResult<hidl_vec<uint8_t>> unvalidatedConvert(
const nn::Model::OperandValues& operandValues);
nn::GeneralResult<hidl_memory> unvalidatedConvert(const nn::Memory& memory);
nn::GeneralResult<Model> unvalidatedConvert(const nn::Model& model);
nn::GeneralResult<RequestArgument> unvalidatedConvert(const nn::Request::Argument& requestArgument);
nn::GeneralResult<hidl_memory> unvalidatedConvert(const nn::Request::MemoryPool& memoryPool);
nn::GeneralResult<Request> unvalidatedConvert(const nn::Request& request);
nn::GeneralResult<ErrorStatus> unvalidatedConvert(const nn::ErrorStatus& status);
nn::GeneralResult<DeviceStatus> convert(const nn::DeviceStatus& deviceStatus);
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities); nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities);
nn::GeneralResult<DataLocation> convert(const nn::DataLocation& location);
nn::GeneralResult<Operand> convert(const nn::Operand& operand);
nn::GeneralResult<Operation> convert(const nn::Operation& operation);
nn::GeneralResult<hidl_vec<uint8_t>> convert(const nn::Model::OperandValues& operandValues);
nn::GeneralResult<hidl_memory> convert(const nn::Memory& memory);
nn::GeneralResult<Model> convert(const nn::Model& model); nn::GeneralResult<Model> convert(const nn::Model& model);
nn::GeneralResult<RequestArgument> convert(const nn::Request::Argument& requestArgument);
nn::GeneralResult<hidl_memory> convert(const nn::Request::MemoryPool& memoryPool);
nn::GeneralResult<Request> convert(const nn::Request& request); nn::GeneralResult<Request> convert(const nn::Request& request);
nn::GeneralResult<ErrorStatus> convert(const nn::ErrorStatus& status); nn::GeneralResult<ErrorStatus> convert(const nn::ErrorStatus& status);

View File

@@ -22,25 +22,16 @@
#include <android-base/logging.h> #include <android-base/logging.h>
#include <android/hardware/neuralnetworks/1.0/types.h> #include <android/hardware/neuralnetworks/1.0/types.h>
#include <nnapi/Result.h> #include <nnapi/Result.h>
#include <nnapi/TypeUtils.h>
#include <nnapi/Types.h> #include <nnapi/Types.h>
#include <nnapi/Validation.h>
namespace android::hardware::neuralnetworks::V1_0::utils { namespace android::hardware::neuralnetworks::V1_0::utils {
constexpr auto kVersion = nn::Version::ANDROID_OC_MR1;
template <typename Type> template <typename Type>
nn::Result<void> validate(const Type& halObject) { nn::Result<void> validate(const Type& halObject) {
const auto maybeCanonical = nn::convert(halObject); const auto maybeCanonical = nn::convert(halObject);
if (!maybeCanonical.has_value()) { if (!maybeCanonical.has_value()) {
return nn::error() << maybeCanonical.error().message; return nn::error() << maybeCanonical.error().message;
} }
const auto version = NN_TRY(nn::validate(maybeCanonical.value()));
if (version > utils::kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required "
<< utils::kVersion;
}
return {}; return {};
} }
@@ -53,21 +44,6 @@ bool valid(const Type& halObject) {
return result.has_value(); return result.has_value();
} }
template <typename Type>
decltype(nn::convert(std::declval<Type>())) validatedConvertToCanonical(const Type& halObject) {
auto canonical = NN_TRY(nn::convert(halObject));
const auto maybeVersion = nn::validate(canonical);
if (!maybeVersion.has_value()) {
return nn::error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > utils::kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required "
<< utils::kVersion;
}
return canonical;
}
} // namespace android::hardware::neuralnetworks::V1_0::utils } // namespace android::hardware::neuralnetworks::V1_0::utils
#endif // ANDROID_HARDWARE_INTERFACES_NEURALNETWORKS_1_0_UTILS_H #endif // ANDROID_HARDWARE_INTERFACES_NEURALNETWORKS_1_0_UTILS_H

View File

@@ -45,8 +45,7 @@ nn::GeneralResult<nn::SharedPreparedModel> convertPreparedModel(
Return<void> PreparedModelCallback::notify(ErrorStatus status, Return<void> PreparedModelCallback::notify(ErrorStatus status,
const sp<IPreparedModel>& preparedModel) { const sp<IPreparedModel>& preparedModel) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status));
} else if (preparedModel == nullptr) { } else if (preparedModel == nullptr) {
notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
@@ -73,8 +72,7 @@ void PreparedModelCallback::notifyInternal(PreparedModelCallback::Data result) {
Return<void> ExecutionCallback::notify(ErrorStatus status) { Return<void> ExecutionCallback::notify(ErrorStatus status) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status));
} else { } else {
notifyInternal({}); notifyInternal({});

View File

@@ -22,7 +22,9 @@
#include <nnapi/OperationTypes.h> #include <nnapi/OperationTypes.h>
#include <nnapi/Result.h> #include <nnapi/Result.h>
#include <nnapi/SharedMemory.h> #include <nnapi/SharedMemory.h>
#include <nnapi/TypeUtils.h>
#include <nnapi/Types.h> #include <nnapi/Types.h>
#include <nnapi/Validation.h>
#include <nnapi/hal/CommonUtils.h> #include <nnapi/hal/CommonUtils.h>
#include <algorithm> #include <algorithm>
@@ -40,6 +42,8 @@ constexpr std::underlying_type_t<Type> underlyingType(Type value) {
return static_cast<std::underlying_type_t<Type>>(value); return static_cast<std::underlying_type_t<Type>>(value);
} }
constexpr auto kVersion = android::nn::Version::ANDROID_OC_MR1;
} // namespace } // namespace
namespace android::nn { namespace android::nn {
@@ -49,37 +53,53 @@ using hardware::hidl_memory;
using hardware::hidl_vec; using hardware::hidl_vec;
template <typename Input> template <typename Input>
using ConvertOutput = std::decay_t<decltype(convert(std::declval<Input>()).value())>; using unvalidatedConvertOutput =
std::decay_t<decltype(unvalidatedConvert(std::declval<Input>()).value())>;
template <typename Type> template <typename Type>
GeneralResult<std::vector<ConvertOutput<Type>>> convert(const hidl_vec<Type>& arguments) { GeneralResult<std::vector<unvalidatedConvertOutput<Type>>> unvalidatedConvert(
std::vector<ConvertOutput<Type>> canonical; const hidl_vec<Type>& arguments) {
std::vector<unvalidatedConvertOutput<Type>> canonical;
canonical.reserve(arguments.size()); canonical.reserve(arguments.size());
for (const auto& argument : arguments) { for (const auto& argument : arguments) {
canonical.push_back(NN_TRY(nn::convert(argument))); canonical.push_back(NN_TRY(nn::unvalidatedConvert(argument)));
}
return canonical;
}
template <typename Type>
decltype(nn::unvalidatedConvert(std::declval<Type>())) validatedConvert(const Type& halObject) {
auto canonical = NN_TRY(nn::unvalidatedConvert(halObject));
const auto maybeVersion = validate(canonical);
if (!maybeVersion.has_value()) {
return error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required " << kVersion;
} }
return canonical; return canonical;
} }
} // anonymous namespace } // anonymous namespace
GeneralResult<OperandType> convert(const hal::V1_0::OperandType& operandType) { GeneralResult<OperandType> unvalidatedConvert(const hal::V1_0::OperandType& operandType) {
return static_cast<OperandType>(operandType); return static_cast<OperandType>(operandType);
} }
GeneralResult<OperationType> convert(const hal::V1_0::OperationType& operationType) { GeneralResult<OperationType> unvalidatedConvert(const hal::V1_0::OperationType& operationType) {
return static_cast<OperationType>(operationType); return static_cast<OperationType>(operationType);
} }
GeneralResult<Operand::LifeTime> convert(const hal::V1_0::OperandLifeTime& lifetime) { GeneralResult<Operand::LifeTime> unvalidatedConvert(const hal::V1_0::OperandLifeTime& lifetime) {
return static_cast<Operand::LifeTime>(lifetime); return static_cast<Operand::LifeTime>(lifetime);
} }
GeneralResult<DeviceStatus> convert(const hal::V1_0::DeviceStatus& deviceStatus) { GeneralResult<DeviceStatus> unvalidatedConvert(const hal::V1_0::DeviceStatus& deviceStatus) {
return static_cast<DeviceStatus>(deviceStatus); return static_cast<DeviceStatus>(deviceStatus);
} }
GeneralResult<Capabilities::PerformanceInfo> convert( GeneralResult<Capabilities::PerformanceInfo> unvalidatedConvert(
const hal::V1_0::PerformanceInfo& performanceInfo) { const hal::V1_0::PerformanceInfo& performanceInfo) {
return Capabilities::PerformanceInfo{ return Capabilities::PerformanceInfo{
.execTime = performanceInfo.execTime, .execTime = performanceInfo.execTime,
@@ -87,9 +107,10 @@ GeneralResult<Capabilities::PerformanceInfo> convert(
}; };
} }
GeneralResult<Capabilities> convert(const hal::V1_0::Capabilities& capabilities) { GeneralResult<Capabilities> unvalidatedConvert(const hal::V1_0::Capabilities& capabilities) {
const auto quantized8Performance = NN_TRY(convert(capabilities.quantized8Performance)); const auto quantized8Performance =
const auto float32Performance = NN_TRY(convert(capabilities.float32Performance)); NN_TRY(unvalidatedConvert(capabilities.quantized8Performance));
const auto float32Performance = NN_TRY(unvalidatedConvert(capabilities.float32Performance));
auto table = hal::utils::makeQuantized8PerformanceConsistentWithP(float32Performance, auto table = hal::utils::makeQuantized8PerformanceConsistentWithP(float32Performance,
quantized8Performance); quantized8Performance);
@@ -101,7 +122,7 @@ GeneralResult<Capabilities> convert(const hal::V1_0::Capabilities& capabilities)
}; };
} }
GeneralResult<DataLocation> convert(const hal::V1_0::DataLocation& location) { GeneralResult<DataLocation> unvalidatedConvert(const hal::V1_0::DataLocation& location) {
return DataLocation{ return DataLocation{
.poolIndex = location.poolIndex, .poolIndex = location.poolIndex,
.offset = location.offset, .offset = location.offset,
@@ -109,35 +130,35 @@ GeneralResult<DataLocation> convert(const hal::V1_0::DataLocation& location) {
}; };
} }
GeneralResult<Operand> convert(const hal::V1_0::Operand& operand) { GeneralResult<Operand> unvalidatedConvert(const hal::V1_0::Operand& operand) {
return Operand{ return Operand{
.type = NN_TRY(convert(operand.type)), .type = NN_TRY(unvalidatedConvert(operand.type)),
.dimensions = operand.dimensions, .dimensions = operand.dimensions,
.scale = operand.scale, .scale = operand.scale,
.zeroPoint = operand.zeroPoint, .zeroPoint = operand.zeroPoint,
.lifetime = NN_TRY(convert(operand.lifetime)), .lifetime = NN_TRY(unvalidatedConvert(operand.lifetime)),
.location = NN_TRY(convert(operand.location)), .location = NN_TRY(unvalidatedConvert(operand.location)),
}; };
} }
GeneralResult<Operation> convert(const hal::V1_0::Operation& operation) { GeneralResult<Operation> unvalidatedConvert(const hal::V1_0::Operation& operation) {
return Operation{ return Operation{
.type = NN_TRY(convert(operation.type)), .type = NN_TRY(unvalidatedConvert(operation.type)),
.inputs = operation.inputs, .inputs = operation.inputs,
.outputs = operation.outputs, .outputs = operation.outputs,
}; };
} }
GeneralResult<Model::OperandValues> convert(const hidl_vec<uint8_t>& operandValues) { GeneralResult<Model::OperandValues> unvalidatedConvert(const hidl_vec<uint8_t>& operandValues) {
return Model::OperandValues(operandValues.data(), operandValues.size()); return Model::OperandValues(operandValues.data(), operandValues.size());
} }
GeneralResult<Memory> convert(const hidl_memory& memory) { GeneralResult<Memory> unvalidatedConvert(const hidl_memory& memory) {
return createSharedMemoryFromHidlMemory(memory); return createSharedMemoryFromHidlMemory(memory);
} }
GeneralResult<Model> convert(const hal::V1_0::Model& model) { GeneralResult<Model> unvalidatedConvert(const hal::V1_0::Model& model) {
auto operations = NN_TRY(convert(model.operations)); auto operations = NN_TRY(unvalidatedConvert(model.operations));
// Verify number of consumers. // Verify number of consumers.
const auto numberOfConsumers = const auto numberOfConsumers =
@@ -152,7 +173,7 @@ GeneralResult<Model> convert(const hal::V1_0::Model& model) {
} }
auto main = Model::Subgraph{ auto main = Model::Subgraph{
.operands = NN_TRY(convert(model.operands)), .operands = NN_TRY(unvalidatedConvert(model.operands)),
.operations = std::move(operations), .operations = std::move(operations),
.inputIndexes = model.inputIndexes, .inputIndexes = model.inputIndexes,
.outputIndexes = model.outputIndexes, .outputIndexes = model.outputIndexes,
@@ -160,35 +181,35 @@ GeneralResult<Model> convert(const hal::V1_0::Model& model) {
return Model{ return Model{
.main = std::move(main), .main = std::move(main),
.operandValues = NN_TRY(convert(model.operandValues)), .operandValues = NN_TRY(unvalidatedConvert(model.operandValues)),
.pools = NN_TRY(convert(model.pools)), .pools = NN_TRY(unvalidatedConvert(model.pools)),
}; };
} }
GeneralResult<Request::Argument> convert(const hal::V1_0::RequestArgument& argument) { GeneralResult<Request::Argument> unvalidatedConvert(const hal::V1_0::RequestArgument& argument) {
const auto lifetime = argument.hasNoValue ? Request::Argument::LifeTime::NO_VALUE const auto lifetime = argument.hasNoValue ? Request::Argument::LifeTime::NO_VALUE
: Request::Argument::LifeTime::POOL; : Request::Argument::LifeTime::POOL;
return Request::Argument{ return Request::Argument{
.lifetime = lifetime, .lifetime = lifetime,
.location = NN_TRY(convert(argument.location)), .location = NN_TRY(unvalidatedConvert(argument.location)),
.dimensions = argument.dimensions, .dimensions = argument.dimensions,
}; };
} }
GeneralResult<Request> convert(const hal::V1_0::Request& request) { GeneralResult<Request> unvalidatedConvert(const hal::V1_0::Request& request) {
auto memories = NN_TRY(convert(request.pools)); auto memories = NN_TRY(unvalidatedConvert(request.pools));
std::vector<Request::MemoryPool> pools; std::vector<Request::MemoryPool> pools;
pools.reserve(memories.size()); pools.reserve(memories.size());
std::move(memories.begin(), memories.end(), std::back_inserter(pools)); std::move(memories.begin(), memories.end(), std::back_inserter(pools));
return Request{ return Request{
.inputs = NN_TRY(convert(request.inputs)), .inputs = NN_TRY(unvalidatedConvert(request.inputs)),
.outputs = NN_TRY(convert(request.outputs)), .outputs = NN_TRY(unvalidatedConvert(request.outputs)),
.pools = std::move(pools), .pools = std::move(pools),
}; };
} }
GeneralResult<ErrorStatus> convert(const hal::V1_0::ErrorStatus& status) { GeneralResult<ErrorStatus> unvalidatedConvert(const hal::V1_0::ErrorStatus& status) {
switch (status) { switch (status) {
case hal::V1_0::ErrorStatus::NONE: case hal::V1_0::ErrorStatus::NONE:
case hal::V1_0::ErrorStatus::DEVICE_UNAVAILABLE: case hal::V1_0::ErrorStatus::DEVICE_UNAVAILABLE:
@@ -201,46 +222,81 @@ GeneralResult<ErrorStatus> convert(const hal::V1_0::ErrorStatus& status) {
<< "Invalid ErrorStatus " << underlyingType(status); << "Invalid ErrorStatus " << underlyingType(status);
} }
GeneralResult<DeviceStatus> convert(const hal::V1_0::DeviceStatus& deviceStatus) {
return validatedConvert(deviceStatus);
}
GeneralResult<Capabilities> convert(const hal::V1_0::Capabilities& capabilities) {
return validatedConvert(capabilities);
}
GeneralResult<Model> convert(const hal::V1_0::Model& model) {
return validatedConvert(model);
}
GeneralResult<Request> convert(const hal::V1_0::Request& request) {
return validatedConvert(request);
}
GeneralResult<ErrorStatus> convert(const hal::V1_0::ErrorStatus& status) {
return validatedConvert(status);
}
} // namespace android::nn } // namespace android::nn
namespace android::hardware::neuralnetworks::V1_0::utils { namespace android::hardware::neuralnetworks::V1_0::utils {
namespace { namespace {
template <typename Input> template <typename Input>
using ConvertOutput = std::decay_t<decltype(convert(std::declval<Input>()).value())>; using unvalidatedConvertOutput =
std::decay_t<decltype(unvalidatedConvert(std::declval<Input>()).value())>;
template <typename Type> template <typename Type>
nn::GeneralResult<hidl_vec<ConvertOutput<Type>>> convert(const std::vector<Type>& arguments) { nn::GeneralResult<hidl_vec<unvalidatedConvertOutput<Type>>> unvalidatedConvert(
hidl_vec<ConvertOutput<Type>> halObject(arguments.size()); const std::vector<Type>& arguments) {
hidl_vec<unvalidatedConvertOutput<Type>> halObject(arguments.size());
for (size_t i = 0; i < arguments.size(); ++i) { for (size_t i = 0; i < arguments.size(); ++i) {
halObject[i] = NN_TRY(utils::convert(arguments[i])); halObject[i] = NN_TRY(utils::unvalidatedConvert(arguments[i]));
} }
return halObject; return halObject;
} }
template <typename Type>
decltype(utils::unvalidatedConvert(std::declval<Type>())) validatedConvert(const Type& canonical) {
const auto maybeVersion = nn::validate(canonical);
if (!maybeVersion.has_value()) {
return nn::error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required " << kVersion;
}
return utils::unvalidatedConvert(canonical);
}
} // anonymous namespace } // anonymous namespace
nn::GeneralResult<OperandType> convert(const nn::OperandType& operandType) { nn::GeneralResult<OperandType> unvalidatedConvert(const nn::OperandType& operandType) {
return static_cast<OperandType>(operandType); return static_cast<OperandType>(operandType);
} }
nn::GeneralResult<OperationType> convert(const nn::OperationType& operationType) { nn::GeneralResult<OperationType> unvalidatedConvert(const nn::OperationType& operationType) {
return static_cast<OperationType>(operationType); return static_cast<OperationType>(operationType);
} }
nn::GeneralResult<OperandLifeTime> convert(const nn::Operand::LifeTime& lifetime) { nn::GeneralResult<OperandLifeTime> unvalidatedConvert(const nn::Operand::LifeTime& lifetime) {
if (lifetime == nn::Operand::LifeTime::POINTER) { if (lifetime == nn::Operand::LifeTime::POINTER) {
return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT) return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT)
<< "Model cannot be converted because it contains pointer-based memory"; << "Model cannot be unvalidatedConverted because it contains pointer-based memory";
} }
return static_cast<OperandLifeTime>(lifetime); return static_cast<OperandLifeTime>(lifetime);
} }
nn::GeneralResult<DeviceStatus> convert(const nn::DeviceStatus& deviceStatus) { nn::GeneralResult<DeviceStatus> unvalidatedConvert(const nn::DeviceStatus& deviceStatus) {
return static_cast<DeviceStatus>(deviceStatus); return static_cast<DeviceStatus>(deviceStatus);
} }
nn::GeneralResult<PerformanceInfo> convert( nn::GeneralResult<PerformanceInfo> unvalidatedConvert(
const nn::Capabilities::PerformanceInfo& performanceInfo) { const nn::Capabilities::PerformanceInfo& performanceInfo) {
return PerformanceInfo{ return PerformanceInfo{
.execTime = performanceInfo.execTime, .execTime = performanceInfo.execTime,
@@ -248,16 +304,16 @@ nn::GeneralResult<PerformanceInfo> convert(
}; };
} }
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities) { nn::GeneralResult<Capabilities> unvalidatedConvert(const nn::Capabilities& capabilities) {
return Capabilities{ return Capabilities{
.float32Performance = NN_TRY(convert( .float32Performance = NN_TRY(unvalidatedConvert(
capabilities.operandPerformance.lookup(nn::OperandType::TENSOR_FLOAT32))), capabilities.operandPerformance.lookup(nn::OperandType::TENSOR_FLOAT32))),
.quantized8Performance = NN_TRY(convert( .quantized8Performance = NN_TRY(unvalidatedConvert(
capabilities.operandPerformance.lookup(nn::OperandType::TENSOR_QUANT8_ASYMM))), capabilities.operandPerformance.lookup(nn::OperandType::TENSOR_QUANT8_ASYMM))),
}; };
} }
nn::GeneralResult<DataLocation> convert(const nn::DataLocation& location) { nn::GeneralResult<DataLocation> unvalidatedConvert(const nn::DataLocation& location) {
return DataLocation{ return DataLocation{
.poolIndex = location.poolIndex, .poolIndex = location.poolIndex,
.offset = location.offset, .offset = location.offset,
@@ -265,42 +321,43 @@ nn::GeneralResult<DataLocation> convert(const nn::DataLocation& location) {
}; };
} }
nn::GeneralResult<Operand> convert(const nn::Operand& operand) { nn::GeneralResult<Operand> unvalidatedConvert(const nn::Operand& operand) {
return Operand{ return Operand{
.type = NN_TRY(convert(operand.type)), .type = NN_TRY(unvalidatedConvert(operand.type)),
.dimensions = operand.dimensions, .dimensions = operand.dimensions,
.numberOfConsumers = 0, .numberOfConsumers = 0,
.scale = operand.scale, .scale = operand.scale,
.zeroPoint = operand.zeroPoint, .zeroPoint = operand.zeroPoint,
.lifetime = NN_TRY(convert(operand.lifetime)), .lifetime = NN_TRY(unvalidatedConvert(operand.lifetime)),
.location = NN_TRY(convert(operand.location)), .location = NN_TRY(unvalidatedConvert(operand.location)),
}; };
} }
nn::GeneralResult<Operation> convert(const nn::Operation& operation) { nn::GeneralResult<Operation> unvalidatedConvert(const nn::Operation& operation) {
return Operation{ return Operation{
.type = NN_TRY(convert(operation.type)), .type = NN_TRY(unvalidatedConvert(operation.type)),
.inputs = operation.inputs, .inputs = operation.inputs,
.outputs = operation.outputs, .outputs = operation.outputs,
}; };
} }
nn::GeneralResult<hidl_vec<uint8_t>> convert(const nn::Model::OperandValues& operandValues) { nn::GeneralResult<hidl_vec<uint8_t>> unvalidatedConvert(
const nn::Model::OperandValues& operandValues) {
return hidl_vec<uint8_t>(operandValues.data(), operandValues.data() + operandValues.size()); return hidl_vec<uint8_t>(operandValues.data(), operandValues.data() + operandValues.size());
} }
nn::GeneralResult<hidl_memory> convert(const nn::Memory& memory) { nn::GeneralResult<hidl_memory> unvalidatedConvert(const nn::Memory& memory) {
return hidl_memory(memory.name, NN_TRY(hal::utils::hidlHandleFromSharedHandle(memory.handle)), return hidl_memory(memory.name, NN_TRY(hal::utils::hidlHandleFromSharedHandle(memory.handle)),
memory.size); memory.size);
} }
nn::GeneralResult<Model> convert(const nn::Model& model) { nn::GeneralResult<Model> unvalidatedConvert(const nn::Model& model) {
if (!hal::utils::hasNoPointerData(model)) { if (!hal::utils::hasNoPointerData(model)) {
return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT) return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT)
<< "Mdoel cannot be converted because it contains pointer-based memory"; << "Mdoel cannot be unvalidatedConverted because it contains pointer-based memory";
} }
auto operands = NN_TRY(convert(model.main.operands)); auto operands = NN_TRY(unvalidatedConvert(model.main.operands));
// Update number of consumers. // Update number of consumers.
const auto numberOfConsumers = const auto numberOfConsumers =
@@ -312,45 +369,46 @@ nn::GeneralResult<Model> convert(const nn::Model& model) {
return Model{ return Model{
.operands = std::move(operands), .operands = std::move(operands),
.operations = NN_TRY(convert(model.main.operations)), .operations = NN_TRY(unvalidatedConvert(model.main.operations)),
.inputIndexes = model.main.inputIndexes, .inputIndexes = model.main.inputIndexes,
.outputIndexes = model.main.outputIndexes, .outputIndexes = model.main.outputIndexes,
.operandValues = NN_TRY(convert(model.operandValues)), .operandValues = NN_TRY(unvalidatedConvert(model.operandValues)),
.pools = NN_TRY(convert(model.pools)), .pools = NN_TRY(unvalidatedConvert(model.pools)),
}; };
} }
nn::GeneralResult<RequestArgument> convert(const nn::Request::Argument& requestArgument) { nn::GeneralResult<RequestArgument> unvalidatedConvert(
const nn::Request::Argument& requestArgument) {
if (requestArgument.lifetime == nn::Request::Argument::LifeTime::POINTER) { if (requestArgument.lifetime == nn::Request::Argument::LifeTime::POINTER) {
return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT) return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT)
<< "Request cannot be converted because it contains pointer-based memory"; << "Request cannot be unvalidatedConverted because it contains pointer-based memory";
} }
const bool hasNoValue = requestArgument.lifetime == nn::Request::Argument::LifeTime::NO_VALUE; const bool hasNoValue = requestArgument.lifetime == nn::Request::Argument::LifeTime::NO_VALUE;
return RequestArgument{ return RequestArgument{
.hasNoValue = hasNoValue, .hasNoValue = hasNoValue,
.location = NN_TRY(convert(requestArgument.location)), .location = NN_TRY(unvalidatedConvert(requestArgument.location)),
.dimensions = requestArgument.dimensions, .dimensions = requestArgument.dimensions,
}; };
} }
nn::GeneralResult<hidl_memory> convert(const nn::Request::MemoryPool& memoryPool) { nn::GeneralResult<hidl_memory> unvalidatedConvert(const nn::Request::MemoryPool& memoryPool) {
return convert(std::get<nn::Memory>(memoryPool)); return unvalidatedConvert(std::get<nn::Memory>(memoryPool));
} }
nn::GeneralResult<Request> convert(const nn::Request& request) { nn::GeneralResult<Request> unvalidatedConvert(const nn::Request& request) {
if (!hal::utils::hasNoPointerData(request)) { if (!hal::utils::hasNoPointerData(request)) {
return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT) return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT)
<< "Request cannot be converted because it contains pointer-based memory"; << "Request cannot be unvalidatedConverted because it contains pointer-based memory";
} }
return Request{ return Request{
.inputs = NN_TRY(convert(request.inputs)), .inputs = NN_TRY(unvalidatedConvert(request.inputs)),
.outputs = NN_TRY(convert(request.outputs)), .outputs = NN_TRY(unvalidatedConvert(request.outputs)),
.pools = NN_TRY(convert(request.pools)), .pools = NN_TRY(unvalidatedConvert(request.pools)),
}; };
} }
nn::GeneralResult<ErrorStatus> convert(const nn::ErrorStatus& status) { nn::GeneralResult<ErrorStatus> unvalidatedConvert(const nn::ErrorStatus& status) {
switch (status) { switch (status) {
case nn::ErrorStatus::NONE: case nn::ErrorStatus::NONE:
case nn::ErrorStatus::DEVICE_UNAVAILABLE: case nn::ErrorStatus::DEVICE_UNAVAILABLE:
@@ -363,4 +421,24 @@ nn::GeneralResult<ErrorStatus> convert(const nn::ErrorStatus& status) {
} }
} }
nn::GeneralResult<DeviceStatus> convert(const nn::DeviceStatus& deviceStatus) {
return validatedConvert(deviceStatus);
}
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities) {
return validatedConvert(capabilities);
}
nn::GeneralResult<Model> convert(const nn::Model& model) {
return validatedConvert(model);
}
nn::GeneralResult<Request> convert(const nn::Request& request) {
return validatedConvert(request);
}
nn::GeneralResult<ErrorStatus> convert(const nn::ErrorStatus& status) {
return validatedConvert(status);
}
} // namespace android::hardware::neuralnetworks::V1_0::utils } // namespace android::hardware::neuralnetworks::V1_0::utils

View File

@@ -48,11 +48,10 @@ nn::GeneralResult<nn::Capabilities> initCapabilities(V1_0::IDevice* device) {
<< "uninitialized"; << "uninitialized";
const auto cb = [&result](ErrorStatus status, const Capabilities& capabilities) { const auto cb = [&result](ErrorStatus status, const Capabilities& capabilities) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "getCapabilities failed with " << toString(status); result = NN_ERROR(canonical) << "getCapabilities failed with " << toString(status);
} else { } else {
result = validatedConvertToCanonical(capabilities); result = nn::convert(capabilities);
} }
}; };
@@ -135,8 +134,7 @@ nn::GeneralResult<std::vector<bool>> Device::getSupportedOperations(const nn::Mo
<< "uninitialized"; << "uninitialized";
auto cb = [&result, &model](ErrorStatus status, const hidl_vec<bool>& supportedOperations) { auto cb = [&result, &model](ErrorStatus status, const hidl_vec<bool>& supportedOperations) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) result = NN_ERROR(canonical)
<< "getSupportedOperations failed with " << toString(status); << "getSupportedOperations failed with " << toString(status);
} else if (supportedOperations.size() != model.main.operations.size()) { } else if (supportedOperations.size() != model.main.operations.size()) {
@@ -172,8 +170,7 @@ nn::GeneralResult<nn::SharedPreparedModel> Device::prepareModel(
const auto ret = kDevice->prepareModel(hidlModel, cb); const auto ret = kDevice->prepareModel(hidlModel, cb);
const auto status = NN_TRY(hal::utils::handleTransportError(ret)); const auto status = NN_TRY(hal::utils::handleTransportError(ret));
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "prepareModel failed with " << toString(status); return NN_ERROR(canonical) << "prepareModel failed with " << toString(status);
} }

View File

@@ -70,8 +70,7 @@ nn::ExecutionResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> Prepare
const auto status = const auto status =
NN_TRY(hal::utils::makeExecutionFailure(hal::utils::handleTransportError(ret))); NN_TRY(hal::utils::makeExecutionFailure(hal::utils::handleTransportError(ret)));
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "execute failed with " << toString(status); return NN_ERROR(canonical) << "execute failed with " << toString(status);
} }

View File

@@ -24,9 +24,14 @@
namespace android::nn { namespace android::nn {
GeneralResult<OperationType> convert(const hal::V1_1::OperationType& operationType); GeneralResult<OperationType> unvalidatedConvert(const hal::V1_1::OperationType& operationType);
GeneralResult<Capabilities> unvalidatedConvert(const hal::V1_1::Capabilities& capabilities);
GeneralResult<Operation> unvalidatedConvert(const hal::V1_1::Operation& operation);
GeneralResult<Model> unvalidatedConvert(const hal::V1_1::Model& model);
GeneralResult<ExecutionPreference> unvalidatedConvert(
const hal::V1_1::ExecutionPreference& executionPreference);
GeneralResult<Capabilities> convert(const hal::V1_1::Capabilities& capabilities); GeneralResult<Capabilities> convert(const hal::V1_1::Capabilities& capabilities);
GeneralResult<Operation> convert(const hal::V1_1::Operation& operation);
GeneralResult<Model> convert(const hal::V1_1::Model& model); GeneralResult<Model> convert(const hal::V1_1::Model& model);
GeneralResult<ExecutionPreference> convert( GeneralResult<ExecutionPreference> convert(
const hal::V1_1::ExecutionPreference& executionPreference); const hal::V1_1::ExecutionPreference& executionPreference);
@@ -35,9 +40,14 @@ GeneralResult<ExecutionPreference> convert(
namespace android::hardware::neuralnetworks::V1_1::utils { namespace android::hardware::neuralnetworks::V1_1::utils {
nn::GeneralResult<OperationType> convert(const nn::OperationType& operationType); nn::GeneralResult<OperationType> unvalidatedConvert(const nn::OperationType& operationType);
nn::GeneralResult<Capabilities> unvalidatedConvert(const nn::Capabilities& capabilities);
nn::GeneralResult<Operation> unvalidatedConvert(const nn::Operation& operation);
nn::GeneralResult<Model> unvalidatedConvert(const nn::Model& model);
nn::GeneralResult<ExecutionPreference> unvalidatedConvert(
const nn::ExecutionPreference& executionPreference);
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities); nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities);
nn::GeneralResult<Operation> convert(const nn::Operation& operation);
nn::GeneralResult<Model> convert(const nn::Model& model); nn::GeneralResult<Model> convert(const nn::Model& model);
nn::GeneralResult<ExecutionPreference> convert(const nn::ExecutionPreference& executionPreference); nn::GeneralResult<ExecutionPreference> convert(const nn::ExecutionPreference& executionPreference);

View File

@@ -22,15 +22,12 @@
#include <android-base/logging.h> #include <android-base/logging.h>
#include <android/hardware/neuralnetworks/1.1/types.h> #include <android/hardware/neuralnetworks/1.1/types.h>
#include <nnapi/Result.h> #include <nnapi/Result.h>
#include <nnapi/TypeUtils.h>
#include <nnapi/Types.h> #include <nnapi/Types.h>
#include <nnapi/Validation.h>
#include <nnapi/hal/1.0/Conversions.h> #include <nnapi/hal/1.0/Conversions.h>
namespace android::hardware::neuralnetworks::V1_1::utils { namespace android::hardware::neuralnetworks::V1_1::utils {
constexpr auto kDefaultExecutionPreference = ExecutionPreference::FAST_SINGLE_ANSWER; constexpr auto kDefaultExecutionPreference = ExecutionPreference::FAST_SINGLE_ANSWER;
constexpr auto kVersion = nn::Version::ANDROID_P;
template <typename Type> template <typename Type>
nn::Result<void> validate(const Type& halObject) { nn::Result<void> validate(const Type& halObject) {
@@ -38,11 +35,6 @@ nn::Result<void> validate(const Type& halObject) {
if (!maybeCanonical.has_value()) { if (!maybeCanonical.has_value()) {
return nn::error() << maybeCanonical.error().message; return nn::error() << maybeCanonical.error().message;
} }
const auto version = NN_TRY(nn::validate(maybeCanonical.value()));
if (version > utils::kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required "
<< utils::kVersion;
}
return {}; return {};
} }
@@ -55,21 +47,6 @@ bool valid(const Type& halObject) {
return result.has_value(); return result.has_value();
} }
template <typename Type>
decltype(nn::convert(std::declval<Type>())) validatedConvertToCanonical(const Type& halObject) {
auto canonical = NN_TRY(nn::convert(halObject));
const auto maybeVersion = nn::validate(canonical);
if (!maybeVersion.has_value()) {
return nn::error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > utils::kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required "
<< utils::kVersion;
}
return canonical;
}
} // namespace android::hardware::neuralnetworks::V1_1::utils } // namespace android::hardware::neuralnetworks::V1_1::utils
#endif // ANDROID_HARDWARE_INTERFACES_NEURALNETWORKS_1_1_UTILS_H #endif // ANDROID_HARDWARE_INTERFACES_NEURALNETWORKS_1_1_UTILS_H

View File

@@ -23,7 +23,9 @@
#include <nnapi/OperationTypes.h> #include <nnapi/OperationTypes.h>
#include <nnapi/Result.h> #include <nnapi/Result.h>
#include <nnapi/SharedMemory.h> #include <nnapi/SharedMemory.h>
#include <nnapi/TypeUtils.h>
#include <nnapi/Types.h> #include <nnapi/Types.h>
#include <nnapi/Validation.h>
#include <nnapi/hal/1.0/Conversions.h> #include <nnapi/hal/1.0/Conversions.h>
#include <nnapi/hal/CommonUtils.h> #include <nnapi/hal/CommonUtils.h>
@@ -33,35 +35,58 @@
#include <type_traits> #include <type_traits>
#include <utility> #include <utility>
namespace {
constexpr auto kVersion = android::nn::Version::ANDROID_P;
} // namespace
namespace android::nn { namespace android::nn {
namespace { namespace {
using hardware::hidl_vec; using hardware::hidl_vec;
template <typename Input> template <typename Input>
using convertOutput = std::decay_t<decltype(convert(std::declval<Input>()).value())>; using unvalidatedConvertOutput =
std::decay_t<decltype(unvalidatedConvert(std::declval<Input>()).value())>;
template <typename Type> template <typename Type>
GeneralResult<std::vector<convertOutput<Type>>> convert(const hidl_vec<Type>& arguments) { GeneralResult<std::vector<unvalidatedConvertOutput<Type>>> unvalidatedConvert(
std::vector<convertOutput<Type>> canonical; const hidl_vec<Type>& arguments) {
std::vector<unvalidatedConvertOutput<Type>> canonical;
canonical.reserve(arguments.size()); canonical.reserve(arguments.size());
for (const auto& argument : arguments) { for (const auto& argument : arguments) {
canonical.push_back(NN_TRY(nn::convert(argument))); canonical.push_back(NN_TRY(nn::unvalidatedConvert(argument)));
}
return canonical;
}
template <typename Type>
decltype(nn::unvalidatedConvert(std::declval<Type>())) validatedConvert(const Type& halObject) {
auto canonical = NN_TRY(nn::unvalidatedConvert(halObject));
const auto maybeVersion = validate(canonical);
if (!maybeVersion.has_value()) {
return error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required " << kVersion;
} }
return canonical; return canonical;
} }
} // anonymous namespace } // anonymous namespace
GeneralResult<OperationType> convert(const hal::V1_1::OperationType& operationType) { GeneralResult<OperationType> unvalidatedConvert(const hal::V1_1::OperationType& operationType) {
return static_cast<OperationType>(operationType); return static_cast<OperationType>(operationType);
} }
GeneralResult<Capabilities> convert(const hal::V1_1::Capabilities& capabilities) { GeneralResult<Capabilities> unvalidatedConvert(const hal::V1_1::Capabilities& capabilities) {
const auto quantized8Performance = NN_TRY(convert(capabilities.quantized8Performance)); const auto quantized8Performance =
const auto float32Performance = NN_TRY(convert(capabilities.float32Performance)); NN_TRY(unvalidatedConvert(capabilities.quantized8Performance));
const auto float32Performance = NN_TRY(unvalidatedConvert(capabilities.float32Performance));
const auto relaxedFloat32toFloat16Performance = const auto relaxedFloat32toFloat16Performance =
NN_TRY(convert(capabilities.relaxedFloat32toFloat16Performance)); NN_TRY(unvalidatedConvert(capabilities.relaxedFloat32toFloat16Performance));
auto table = hal::utils::makeQuantized8PerformanceConsistentWithP(float32Performance, auto table = hal::utils::makeQuantized8PerformanceConsistentWithP(float32Performance,
quantized8Performance); quantized8Performance);
@@ -73,16 +98,16 @@ GeneralResult<Capabilities> convert(const hal::V1_1::Capabilities& capabilities)
}; };
} }
GeneralResult<Operation> convert(const hal::V1_1::Operation& operation) { GeneralResult<Operation> unvalidatedConvert(const hal::V1_1::Operation& operation) {
return Operation{ return Operation{
.type = NN_TRY(convert(operation.type)), .type = NN_TRY(unvalidatedConvert(operation.type)),
.inputs = operation.inputs, .inputs = operation.inputs,
.outputs = operation.outputs, .outputs = operation.outputs,
}; };
} }
GeneralResult<Model> convert(const hal::V1_1::Model& model) { GeneralResult<Model> unvalidatedConvert(const hal::V1_1::Model& model) {
auto operations = NN_TRY(convert(model.operations)); auto operations = NN_TRY(unvalidatedConvert(model.operations));
// Verify number of consumers. // Verify number of consumers.
const auto numberOfConsumers = const auto numberOfConsumers =
@@ -97,7 +122,7 @@ GeneralResult<Model> convert(const hal::V1_1::Model& model) {
} }
auto main = Model::Subgraph{ auto main = Model::Subgraph{
.operands = NN_TRY(convert(model.operands)), .operands = NN_TRY(unvalidatedConvert(model.operands)),
.operations = std::move(operations), .operations = std::move(operations),
.inputIndexes = model.inputIndexes, .inputIndexes = model.inputIndexes,
.outputIndexes = model.outputIndexes, .outputIndexes = model.outputIndexes,
@@ -105,85 +130,114 @@ GeneralResult<Model> convert(const hal::V1_1::Model& model) {
return Model{ return Model{
.main = std::move(main), .main = std::move(main),
.operandValues = NN_TRY(convert(model.operandValues)), .operandValues = NN_TRY(unvalidatedConvert(model.operandValues)),
.pools = NN_TRY(convert(model.pools)), .pools = NN_TRY(unvalidatedConvert(model.pools)),
.relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16, .relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16,
}; };
} }
GeneralResult<ExecutionPreference> convert( GeneralResult<ExecutionPreference> unvalidatedConvert(
const hal::V1_1::ExecutionPreference& executionPreference) { const hal::V1_1::ExecutionPreference& executionPreference) {
return static_cast<ExecutionPreference>(executionPreference); return static_cast<ExecutionPreference>(executionPreference);
} }
GeneralResult<Capabilities> convert(const hal::V1_1::Capabilities& capabilities) {
return validatedConvert(capabilities);
}
GeneralResult<Model> convert(const hal::V1_1::Model& model) {
return validatedConvert(model);
}
GeneralResult<ExecutionPreference> convert(
const hal::V1_1::ExecutionPreference& executionPreference) {
return validatedConvert(executionPreference);
}
} // namespace android::nn } // namespace android::nn
namespace android::hardware::neuralnetworks::V1_1::utils { namespace android::hardware::neuralnetworks::V1_1::utils {
namespace { namespace {
using utils::convert; using utils::unvalidatedConvert;
nn::GeneralResult<V1_0::PerformanceInfo> convert( nn::GeneralResult<V1_0::PerformanceInfo> unvalidatedConvert(
const nn::Capabilities::PerformanceInfo& performanceInfo) { const nn::Capabilities::PerformanceInfo& performanceInfo) {
return V1_0::utils::convert(performanceInfo); return V1_0::utils::unvalidatedConvert(performanceInfo);
} }
nn::GeneralResult<V1_0::Operand> convert(const nn::Operand& operand) { nn::GeneralResult<V1_0::Operand> unvalidatedConvert(const nn::Operand& operand) {
return V1_0::utils::convert(operand); return V1_0::utils::unvalidatedConvert(operand);
} }
nn::GeneralResult<hidl_vec<uint8_t>> convert(const nn::Model::OperandValues& operandValues) { nn::GeneralResult<hidl_vec<uint8_t>> unvalidatedConvert(
return V1_0::utils::convert(operandValues); const nn::Model::OperandValues& operandValues) {
return V1_0::utils::unvalidatedConvert(operandValues);
} }
nn::GeneralResult<hidl_memory> convert(const nn::Memory& memory) { nn::GeneralResult<hidl_memory> unvalidatedConvert(const nn::Memory& memory) {
return V1_0::utils::convert(memory); return V1_0::utils::unvalidatedConvert(memory);
} }
template <typename Input> template <typename Input>
using convertOutput = std::decay_t<decltype(convert(std::declval<Input>()).value())>; using unvalidatedConvertOutput =
std::decay_t<decltype(unvalidatedConvert(std::declval<Input>()).value())>;
template <typename Type> template <typename Type>
nn::GeneralResult<hidl_vec<convertOutput<Type>>> convert(const std::vector<Type>& arguments) { nn::GeneralResult<hidl_vec<unvalidatedConvertOutput<Type>>> unvalidatedConvert(
hidl_vec<convertOutput<Type>> halObject(arguments.size()); const std::vector<Type>& arguments) {
hidl_vec<unvalidatedConvertOutput<Type>> halObject(arguments.size());
for (size_t i = 0; i < arguments.size(); ++i) { for (size_t i = 0; i < arguments.size(); ++i) {
halObject[i] = NN_TRY(convert(arguments[i])); halObject[i] = NN_TRY(unvalidatedConvert(arguments[i]));
} }
return halObject; return halObject;
} }
template <typename Type>
decltype(utils::unvalidatedConvert(std::declval<Type>())) validatedConvert(const Type& canonical) {
const auto maybeVersion = nn::validate(canonical);
if (!maybeVersion.has_value()) {
return nn::error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required " << kVersion;
}
return utils::unvalidatedConvert(canonical);
}
} // anonymous namespace } // anonymous namespace
nn::GeneralResult<OperationType> convert(const nn::OperationType& operationType) { nn::GeneralResult<OperationType> unvalidatedConvert(const nn::OperationType& operationType) {
return static_cast<OperationType>(operationType); return static_cast<OperationType>(operationType);
} }
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities) { nn::GeneralResult<Capabilities> unvalidatedConvert(const nn::Capabilities& capabilities) {
return Capabilities{ return Capabilities{
.float32Performance = NN_TRY(convert( .float32Performance = NN_TRY(unvalidatedConvert(
capabilities.operandPerformance.lookup(nn::OperandType::TENSOR_FLOAT32))), capabilities.operandPerformance.lookup(nn::OperandType::TENSOR_FLOAT32))),
.quantized8Performance = NN_TRY(convert( .quantized8Performance = NN_TRY(unvalidatedConvert(
capabilities.operandPerformance.lookup(nn::OperandType::TENSOR_QUANT8_ASYMM))), capabilities.operandPerformance.lookup(nn::OperandType::TENSOR_QUANT8_ASYMM))),
.relaxedFloat32toFloat16Performance = .relaxedFloat32toFloat16Performance = NN_TRY(
NN_TRY(convert(capabilities.relaxedFloat32toFloat16PerformanceTensor)), unvalidatedConvert(capabilities.relaxedFloat32toFloat16PerformanceTensor)),
}; };
} }
nn::GeneralResult<Operation> convert(const nn::Operation& operation) { nn::GeneralResult<Operation> unvalidatedConvert(const nn::Operation& operation) {
return Operation{ return Operation{
.type = NN_TRY(convert(operation.type)), .type = NN_TRY(unvalidatedConvert(operation.type)),
.inputs = operation.inputs, .inputs = operation.inputs,
.outputs = operation.outputs, .outputs = operation.outputs,
}; };
} }
nn::GeneralResult<Model> convert(const nn::Model& model) { nn::GeneralResult<Model> unvalidatedConvert(const nn::Model& model) {
if (!hal::utils::hasNoPointerData(model)) { if (!hal::utils::hasNoPointerData(model)) {
return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT) return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT)
<< "Mdoel cannot be converted because it contains pointer-based memory"; << "Mdoel cannot be unvalidatedConverted because it contains pointer-based memory";
} }
auto operands = NN_TRY(convert(model.main.operands)); auto operands = NN_TRY(unvalidatedConvert(model.main.operands));
// Update number of consumers. // Update number of consumers.
const auto numberOfConsumers = const auto numberOfConsumers =
@@ -195,17 +249,30 @@ nn::GeneralResult<Model> convert(const nn::Model& model) {
return Model{ return Model{
.operands = std::move(operands), .operands = std::move(operands),
.operations = NN_TRY(convert(model.main.operations)), .operations = NN_TRY(unvalidatedConvert(model.main.operations)),
.inputIndexes = model.main.inputIndexes, .inputIndexes = model.main.inputIndexes,
.outputIndexes = model.main.outputIndexes, .outputIndexes = model.main.outputIndexes,
.operandValues = NN_TRY(convert(model.operandValues)), .operandValues = NN_TRY(unvalidatedConvert(model.operandValues)),
.pools = NN_TRY(convert(model.pools)), .pools = NN_TRY(unvalidatedConvert(model.pools)),
.relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16, .relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16,
}; };
} }
nn::GeneralResult<ExecutionPreference> convert(const nn::ExecutionPreference& executionPreference) { nn::GeneralResult<ExecutionPreference> unvalidatedConvert(
const nn::ExecutionPreference& executionPreference) {
return static_cast<ExecutionPreference>(executionPreference); return static_cast<ExecutionPreference>(executionPreference);
} }
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities) {
return validatedConvert(capabilities);
}
nn::GeneralResult<Model> convert(const nn::Model& model) {
return validatedConvert(model);
}
nn::GeneralResult<ExecutionPreference> convert(const nn::ExecutionPreference& executionPreference) {
return validatedConvert(executionPreference);
}
} // namespace android::hardware::neuralnetworks::V1_1::utils } // namespace android::hardware::neuralnetworks::V1_1::utils

View File

@@ -49,11 +49,10 @@ nn::GeneralResult<nn::Capabilities> initCapabilities(V1_1::IDevice* device) {
<< "uninitialized"; << "uninitialized";
const auto cb = [&result](V1_0::ErrorStatus status, const Capabilities& capabilities) { const auto cb = [&result](V1_0::ErrorStatus status, const Capabilities& capabilities) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "getCapabilities_1_1 failed with " << toString(status); result = NN_ERROR(canonical) << "getCapabilities_1_1 failed with " << toString(status);
} else { } else {
result = validatedConvertToCanonical(capabilities); result = nn::convert(capabilities);
} }
}; };
@@ -137,8 +136,7 @@ nn::GeneralResult<std::vector<bool>> Device::getSupportedOperations(const nn::Mo
auto cb = [&result, &model](V1_0::ErrorStatus status, auto cb = [&result, &model](V1_0::ErrorStatus status,
const hidl_vec<bool>& supportedOperations) { const hidl_vec<bool>& supportedOperations) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) result = NN_ERROR(canonical)
<< "getSupportedOperations_1_1 failed with " << toString(status); << "getSupportedOperations_1_1 failed with " << toString(status);
} else if (supportedOperations.size() != model.main.operations.size()) { } else if (supportedOperations.size() != model.main.operations.size()) {
@@ -175,8 +173,7 @@ nn::GeneralResult<nn::SharedPreparedModel> Device::prepareModel(
const auto ret = kDevice->prepareModel_1_1(hidlModel, hidlPreference, cb); const auto ret = kDevice->prepareModel_1_1(hidlModel, hidlPreference, cb);
const auto status = NN_TRY(hal::utils::handleTransportError(ret)); const auto status = NN_TRY(hal::utils::handleTransportError(ret));
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "prepareModel failed with " << toString(status); return NN_ERROR(canonical) << "prepareModel failed with " << toString(status);
} }

View File

@@ -24,27 +24,34 @@
namespace android::nn { namespace android::nn {
GeneralResult<OperandType> convert(const hal::V1_2::OperandType& operandType); GeneralResult<OperandType> unvalidatedConvert(const hal::V1_2::OperandType& operandType);
GeneralResult<OperationType> convert(const hal::V1_2::OperationType& operationType); GeneralResult<OperationType> unvalidatedConvert(const hal::V1_2::OperationType& operationType);
GeneralResult<DeviceType> unvalidatedConvert(const hal::V1_2::DeviceType& deviceType);
GeneralResult<Capabilities> unvalidatedConvert(const hal::V1_2::Capabilities& capabilities);
GeneralResult<Capabilities::OperandPerformance> unvalidatedConvert(
const hal::V1_2::Capabilities::OperandPerformance& operandPerformance);
GeneralResult<Operation> unvalidatedConvert(const hal::V1_2::Operation& operation);
GeneralResult<Operand::SymmPerChannelQuantParams> unvalidatedConvert(
const hal::V1_2::SymmPerChannelQuantParams& symmPerChannelQuantParams);
GeneralResult<Operand> unvalidatedConvert(const hal::V1_2::Operand& operand);
GeneralResult<Operand::ExtraParams> unvalidatedConvert(
const hal::V1_2::Operand::ExtraParams& extraParams);
GeneralResult<Model> unvalidatedConvert(const hal::V1_2::Model& model);
GeneralResult<Model::ExtensionNameAndPrefix> unvalidatedConvert(
const hal::V1_2::Model::ExtensionNameAndPrefix& extensionNameAndPrefix);
GeneralResult<OutputShape> unvalidatedConvert(const hal::V1_2::OutputShape& outputShape);
GeneralResult<MeasureTiming> unvalidatedConvert(const hal::V1_2::MeasureTiming& measureTiming);
GeneralResult<Timing> unvalidatedConvert(const hal::V1_2::Timing& timing);
GeneralResult<Extension> unvalidatedConvert(const hal::V1_2::Extension& extension);
GeneralResult<Extension::OperandTypeInformation> unvalidatedConvert(
const hal::V1_2::Extension::OperandTypeInformation& operandTypeInformation);
GeneralResult<SharedHandle> unvalidatedConvert(const hardware::hidl_handle& handle);
GeneralResult<DeviceType> convert(const hal::V1_2::DeviceType& deviceType); GeneralResult<DeviceType> convert(const hal::V1_2::DeviceType& deviceType);
GeneralResult<Capabilities> convert(const hal::V1_2::Capabilities& capabilities); GeneralResult<Capabilities> convert(const hal::V1_2::Capabilities& capabilities);
GeneralResult<Capabilities::OperandPerformance> convert(
const hal::V1_2::Capabilities::OperandPerformance& operandPerformance);
GeneralResult<Operation> convert(const hal::V1_2::Operation& operation);
GeneralResult<Operand::SymmPerChannelQuantParams> convert(
const hal::V1_2::SymmPerChannelQuantParams& symmPerChannelQuantParams);
GeneralResult<Operand> convert(const hal::V1_2::Operand& operand);
GeneralResult<Operand::ExtraParams> convert(const hal::V1_2::Operand::ExtraParams& extraParams);
GeneralResult<Model> convert(const hal::V1_2::Model& model); GeneralResult<Model> convert(const hal::V1_2::Model& model);
GeneralResult<Model::ExtensionNameAndPrefix> convert(
const hal::V1_2::Model::ExtensionNameAndPrefix& extensionNameAndPrefix);
GeneralResult<OutputShape> convert(const hal::V1_2::OutputShape& outputShape);
GeneralResult<MeasureTiming> convert(const hal::V1_2::MeasureTiming& measureTiming); GeneralResult<MeasureTiming> convert(const hal::V1_2::MeasureTiming& measureTiming);
GeneralResult<Timing> convert(const hal::V1_2::Timing& timing); GeneralResult<Timing> convert(const hal::V1_2::Timing& timing);
GeneralResult<Extension> convert(const hal::V1_2::Extension& extension);
GeneralResult<Extension::OperandTypeInformation> convert(
const hal::V1_2::Extension::OperandTypeInformation& operandTypeInformation);
GeneralResult<SharedHandle> convert(const hardware::hidl_handle& handle);
GeneralResult<std::vector<Extension>> convert( GeneralResult<std::vector<Extension>> convert(
const hardware::hidl_vec<hal::V1_2::Extension>& extensions); const hardware::hidl_vec<hal::V1_2::Extension>& extensions);
@@ -57,27 +64,34 @@ GeneralResult<std::vector<OutputShape>> convert(
namespace android::hardware::neuralnetworks::V1_2::utils { namespace android::hardware::neuralnetworks::V1_2::utils {
nn::GeneralResult<OperandType> convert(const nn::OperandType& operandType); nn::GeneralResult<OperandType> unvalidatedConvert(const nn::OperandType& operandType);
nn::GeneralResult<OperationType> convert(const nn::OperationType& operationType); nn::GeneralResult<OperationType> unvalidatedConvert(const nn::OperationType& operationType);
nn::GeneralResult<DeviceType> unvalidatedConvert(const nn::DeviceType& deviceType);
nn::GeneralResult<Capabilities> unvalidatedConvert(const nn::Capabilities& capabilities);
nn::GeneralResult<Capabilities::OperandPerformance> unvalidatedConvert(
const nn::Capabilities::OperandPerformance& operandPerformance);
nn::GeneralResult<Operation> unvalidatedConvert(const nn::Operation& operation);
nn::GeneralResult<SymmPerChannelQuantParams> unvalidatedConvert(
const nn::Operand::SymmPerChannelQuantParams& symmPerChannelQuantParams);
nn::GeneralResult<Operand> unvalidatedConvert(const nn::Operand& operand);
nn::GeneralResult<Operand::ExtraParams> unvalidatedConvert(
const nn::Operand::ExtraParams& extraParams);
nn::GeneralResult<Model> unvalidatedConvert(const nn::Model& model);
nn::GeneralResult<Model::ExtensionNameAndPrefix> unvalidatedConvert(
const nn::Model::ExtensionNameAndPrefix& extensionNameAndPrefix);
nn::GeneralResult<OutputShape> unvalidatedConvert(const nn::OutputShape& outputShape);
nn::GeneralResult<MeasureTiming> unvalidatedConvert(const nn::MeasureTiming& measureTiming);
nn::GeneralResult<Timing> unvalidatedConvert(const nn::Timing& timing);
nn::GeneralResult<Extension> unvalidatedConvert(const nn::Extension& extension);
nn::GeneralResult<Extension::OperandTypeInformation> unvalidatedConvert(
const nn::Extension::OperandTypeInformation& operandTypeInformation);
nn::GeneralResult<hidl_handle> unvalidatedConvert(const nn::SharedHandle& handle);
nn::GeneralResult<DeviceType> convert(const nn::DeviceType& deviceType); nn::GeneralResult<DeviceType> convert(const nn::DeviceType& deviceType);
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities); nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities);
nn::GeneralResult<Capabilities::OperandPerformance> convert(
const nn::Capabilities::OperandPerformance& operandPerformance);
nn::GeneralResult<Operation> convert(const nn::Operation& operation);
nn::GeneralResult<SymmPerChannelQuantParams> convert(
const nn::Operand::SymmPerChannelQuantParams& symmPerChannelQuantParams);
nn::GeneralResult<Operand> convert(const nn::Operand& operand);
nn::GeneralResult<Operand::ExtraParams> convert(const nn::Operand::ExtraParams& extraParams);
nn::GeneralResult<Model> convert(const nn::Model& model); nn::GeneralResult<Model> convert(const nn::Model& model);
nn::GeneralResult<Model::ExtensionNameAndPrefix> convert(
const nn::Model::ExtensionNameAndPrefix& extensionNameAndPrefix);
nn::GeneralResult<OutputShape> convert(const nn::OutputShape& outputShape);
nn::GeneralResult<MeasureTiming> convert(const nn::MeasureTiming& measureTiming); nn::GeneralResult<MeasureTiming> convert(const nn::MeasureTiming& measureTiming);
nn::GeneralResult<Timing> convert(const nn::Timing& timing); nn::GeneralResult<Timing> convert(const nn::Timing& timing);
nn::GeneralResult<Extension> convert(const nn::Extension& extension);
nn::GeneralResult<Extension::OperandTypeInformation> convert(
const nn::Extension::OperandTypeInformation& operandTypeInformation);
nn::GeneralResult<hidl_handle> convert(const nn::SharedHandle& handle);
nn::GeneralResult<hidl_vec<Extension>> convert(const std::vector<nn::Extension>& extensions); nn::GeneralResult<hidl_vec<Extension>> convert(const std::vector<nn::Extension>& extensions);
nn::GeneralResult<hidl_vec<hidl_handle>> convert(const std::vector<nn::SharedHandle>& handles); nn::GeneralResult<hidl_vec<hidl_handle>> convert(const std::vector<nn::SharedHandle>& handles);

View File

@@ -22,9 +22,7 @@
#include <android-base/logging.h> #include <android-base/logging.h>
#include <android/hardware/neuralnetworks/1.2/types.h> #include <android/hardware/neuralnetworks/1.2/types.h>
#include <nnapi/Result.h> #include <nnapi/Result.h>
#include <nnapi/TypeUtils.h>
#include <nnapi/Types.h> #include <nnapi/Types.h>
#include <nnapi/Validation.h>
#include <nnapi/hal/1.0/Conversions.h> #include <nnapi/hal/1.0/Conversions.h>
#include <nnapi/hal/1.1/Conversions.h> #include <nnapi/hal/1.1/Conversions.h>
@@ -35,7 +33,6 @@ namespace android::hardware::neuralnetworks::V1_2::utils {
constexpr auto kDefaultMesaureTiming = MeasureTiming::NO; constexpr auto kDefaultMesaureTiming = MeasureTiming::NO;
constexpr auto kNoTiming = Timing{.timeOnDevice = std::numeric_limits<uint64_t>::max(), constexpr auto kNoTiming = Timing{.timeOnDevice = std::numeric_limits<uint64_t>::max(),
.timeInDriver = std::numeric_limits<uint64_t>::max()}; .timeInDriver = std::numeric_limits<uint64_t>::max()};
constexpr auto kVersion = nn::Version::ANDROID_Q;
template <typename Type> template <typename Type>
nn::Result<void> validate(const Type& halObject) { nn::Result<void> validate(const Type& halObject) {
@@ -43,11 +40,6 @@ nn::Result<void> validate(const Type& halObject) {
if (!maybeCanonical.has_value()) { if (!maybeCanonical.has_value()) {
return nn::error() << maybeCanonical.error().message; return nn::error() << maybeCanonical.error().message;
} }
const auto version = NN_TRY(nn::validate(maybeCanonical.value()));
if (version > utils::kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required "
<< utils::kVersion;
}
return {}; return {};
} }
@@ -60,21 +52,6 @@ bool valid(const Type& halObject) {
return result.has_value(); return result.has_value();
} }
template <typename Type>
decltype(nn::convert(std::declval<Type>())) validatedConvertToCanonical(const Type& halObject) {
auto canonical = NN_TRY(nn::convert(halObject));
const auto maybeVersion = nn::validate(canonical);
if (!maybeVersion.has_value()) {
return nn::error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > utils::kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required "
<< utils::kVersion;
}
return canonical;
}
} // namespace android::hardware::neuralnetworks::V1_2::utils } // namespace android::hardware::neuralnetworks::V1_2::utils
#endif // ANDROID_HARDWARE_INTERFACES_NEURALNETWORKS_1_2_UTILS_H #endif // ANDROID_HARDWARE_INTERFACES_NEURALNETWORKS_1_2_UTILS_H

View File

@@ -52,8 +52,7 @@ nn::GeneralResult<nn::SharedPreparedModel> convertPreparedModel(
nn::GeneralResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> nn::GeneralResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>>
convertExecutionGeneralResultsHelper(const hidl_vec<OutputShape>& outputShapes, convertExecutionGeneralResultsHelper(const hidl_vec<OutputShape>& outputShapes,
const Timing& timing) { const Timing& timing) {
return std::make_pair(NN_TRY(validatedConvertToCanonical(outputShapes)), return std::make_pair(NN_TRY(nn::convert(outputShapes)), NN_TRY(nn::convert(timing)));
NN_TRY(validatedConvertToCanonical(timing)));
} }
nn::ExecutionResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> nn::ExecutionResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>>
@@ -67,8 +66,7 @@ convertExecutionGeneralResults(const hidl_vec<OutputShape>& outputShapes, const
Return<void> PreparedModelCallback::notify(V1_0::ErrorStatus status, Return<void> PreparedModelCallback::notify(V1_0::ErrorStatus status,
const sp<V1_0::IPreparedModel>& preparedModel) { const sp<V1_0::IPreparedModel>& preparedModel) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status));
} else if (preparedModel == nullptr) { } else if (preparedModel == nullptr) {
notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
@@ -82,8 +80,7 @@ Return<void> PreparedModelCallback::notify(V1_0::ErrorStatus status,
Return<void> PreparedModelCallback::notify_1_2(V1_0::ErrorStatus status, Return<void> PreparedModelCallback::notify_1_2(V1_0::ErrorStatus status,
const sp<IPreparedModel>& preparedModel) { const sp<IPreparedModel>& preparedModel) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status));
} else if (preparedModel == nullptr) { } else if (preparedModel == nullptr) {
notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
@@ -110,8 +107,7 @@ void PreparedModelCallback::notifyInternal(PreparedModelCallback::Data result) {
Return<void> ExecutionCallback::notify(V1_0::ErrorStatus status) { Return<void> ExecutionCallback::notify(V1_0::ErrorStatus status) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status));
} else { } else {
notifyInternal({}); notifyInternal({});
@@ -123,8 +119,7 @@ Return<void> ExecutionCallback::notify_1_2(V1_0::ErrorStatus status,
const hidl_vec<OutputShape>& outputShapes, const hidl_vec<OutputShape>& outputShapes,
const Timing& timing) { const Timing& timing) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status));
} else { } else {
notifyInternal(convertExecutionGeneralResults(outputShapes, timing)); notifyInternal(convertExecutionGeneralResults(outputShapes, timing));

View File

@@ -24,6 +24,7 @@
#include <nnapi/SharedMemory.h> #include <nnapi/SharedMemory.h>
#include <nnapi/TypeUtils.h> #include <nnapi/TypeUtils.h>
#include <nnapi/Types.h> #include <nnapi/Types.h>
#include <nnapi/Validation.h>
#include <nnapi/hal/1.0/Conversions.h> #include <nnapi/hal/1.0/Conversions.h>
#include <nnapi/hal/CommonUtils.h> #include <nnapi/hal/CommonUtils.h>
#include <nnapi/hal/HandleError.h> #include <nnapi/hal/HandleError.h>
@@ -42,6 +43,8 @@ constexpr std::underlying_type_t<Type> underlyingType(Type value) {
return static_cast<std::underlying_type_t<Type>>(value); return static_cast<std::underlying_type_t<Type>>(value);
} }
constexpr auto kVersion = android::nn::Version::ANDROID_Q;
} // namespace } // namespace
namespace android::nn { namespace android::nn {
@@ -76,42 +79,70 @@ using hardware::hidl_handle;
using hardware::hidl_vec; using hardware::hidl_vec;
template <typename Input> template <typename Input>
using ConvertOutput = std::decay_t<decltype(convert(std::declval<Input>()).value())>; using unvalidatedConvertOutput =
std::decay_t<decltype(unvalidatedConvert(std::declval<Input>()).value())>;
template <typename Type> template <typename Type>
GeneralResult<std::vector<ConvertOutput<Type>>> convertVec(const hidl_vec<Type>& arguments) { GeneralResult<std::vector<unvalidatedConvertOutput<Type>>> unvalidatedConvertVec(
std::vector<ConvertOutput<Type>> canonical; const hidl_vec<Type>& arguments) {
std::vector<unvalidatedConvertOutput<Type>> canonical;
canonical.reserve(arguments.size()); canonical.reserve(arguments.size());
for (const auto& argument : arguments) { for (const auto& argument : arguments) {
canonical.push_back(NN_TRY(nn::convert(argument))); canonical.push_back(NN_TRY(nn::unvalidatedConvert(argument)));
} }
return canonical; return canonical;
} }
template <typename Type> template <typename Type>
GeneralResult<std::vector<ConvertOutput<Type>>> convert(const hidl_vec<Type>& arguments) { GeneralResult<std::vector<unvalidatedConvertOutput<Type>>> unvalidatedConvert(
return convertVec(arguments); const hidl_vec<Type>& arguments) {
return unvalidatedConvertVec(arguments);
}
template <typename Type>
decltype(nn::unvalidatedConvert(std::declval<Type>())) validatedConvert(const Type& halObject) {
auto canonical = NN_TRY(nn::unvalidatedConvert(halObject));
const auto maybeVersion = validate(canonical);
if (!maybeVersion.has_value()) {
return error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required " << kVersion;
}
return canonical;
}
template <typename Type>
GeneralResult<std::vector<unvalidatedConvertOutput<Type>>> validatedConvert(
const hidl_vec<Type>& arguments) {
std::vector<unvalidatedConvertOutput<Type>> canonical;
canonical.reserve(arguments.size());
for (const auto& argument : arguments) {
canonical.push_back(NN_TRY(validatedConvert(argument)));
}
return canonical;
} }
} // anonymous namespace } // anonymous namespace
GeneralResult<OperandType> convert(const hal::V1_2::OperandType& operandType) { GeneralResult<OperandType> unvalidatedConvert(const hal::V1_2::OperandType& operandType) {
return static_cast<OperandType>(operandType); return static_cast<OperandType>(operandType);
} }
GeneralResult<OperationType> convert(const hal::V1_2::OperationType& operationType) { GeneralResult<OperationType> unvalidatedConvert(const hal::V1_2::OperationType& operationType) {
return static_cast<OperationType>(operationType); return static_cast<OperationType>(operationType);
} }
GeneralResult<DeviceType> convert(const hal::V1_2::DeviceType& deviceType) { GeneralResult<DeviceType> unvalidatedConvert(const hal::V1_2::DeviceType& deviceType) {
return static_cast<DeviceType>(deviceType); return static_cast<DeviceType>(deviceType);
} }
GeneralResult<Capabilities> convert(const hal::V1_2::Capabilities& capabilities) { GeneralResult<Capabilities> unvalidatedConvert(const hal::V1_2::Capabilities& capabilities) {
const bool validOperandTypes = std::all_of( const bool validOperandTypes = std::all_of(
capabilities.operandPerformance.begin(), capabilities.operandPerformance.end(), capabilities.operandPerformance.begin(), capabilities.operandPerformance.end(),
[](const hal::V1_2::Capabilities::OperandPerformance& operandPerformance) { [](const hal::V1_2::Capabilities::OperandPerformance& operandPerformance) {
const auto maybeType = convert(operandPerformance.type); const auto maybeType = unvalidatedConvert(operandPerformance.type);
return !maybeType.has_value() ? false : validOperandType(maybeType.value()); return !maybeType.has_value() ? false : validOperandType(maybeType.value());
}); });
if (!validOperandTypes) { if (!validOperandTypes) {
@@ -120,10 +151,10 @@ GeneralResult<Capabilities> convert(const hal::V1_2::Capabilities& capabilities)
} }
const auto relaxedFloat32toFloat16PerformanceScalar = const auto relaxedFloat32toFloat16PerformanceScalar =
NN_TRY(convert(capabilities.relaxedFloat32toFloat16PerformanceScalar)); NN_TRY(unvalidatedConvert(capabilities.relaxedFloat32toFloat16PerformanceScalar));
const auto relaxedFloat32toFloat16PerformanceTensor = const auto relaxedFloat32toFloat16PerformanceTensor =
NN_TRY(convert(capabilities.relaxedFloat32toFloat16PerformanceTensor)); NN_TRY(unvalidatedConvert(capabilities.relaxedFloat32toFloat16PerformanceTensor));
auto operandPerformance = NN_TRY(convert(capabilities.operandPerformance)); auto operandPerformance = NN_TRY(unvalidatedConvert(capabilities.operandPerformance));
auto table = NN_TRY(hal::utils::makeGeneralFailure( auto table = NN_TRY(hal::utils::makeGeneralFailure(
Capabilities::OperandPerformanceTable::create(std::move(operandPerformance)), Capabilities::OperandPerformanceTable::create(std::move(operandPerformance)),
@@ -136,23 +167,23 @@ GeneralResult<Capabilities> convert(const hal::V1_2::Capabilities& capabilities)
}; };
} }
GeneralResult<Capabilities::OperandPerformance> convert( GeneralResult<Capabilities::OperandPerformance> unvalidatedConvert(
const hal::V1_2::Capabilities::OperandPerformance& operandPerformance) { const hal::V1_2::Capabilities::OperandPerformance& operandPerformance) {
return Capabilities::OperandPerformance{ return Capabilities::OperandPerformance{
.type = NN_TRY(convert(operandPerformance.type)), .type = NN_TRY(unvalidatedConvert(operandPerformance.type)),
.info = NN_TRY(convert(operandPerformance.info)), .info = NN_TRY(unvalidatedConvert(operandPerformance.info)),
}; };
} }
GeneralResult<Operation> convert(const hal::V1_2::Operation& operation) { GeneralResult<Operation> unvalidatedConvert(const hal::V1_2::Operation& operation) {
return Operation{ return Operation{
.type = NN_TRY(convert(operation.type)), .type = NN_TRY(unvalidatedConvert(operation.type)),
.inputs = operation.inputs, .inputs = operation.inputs,
.outputs = operation.outputs, .outputs = operation.outputs,
}; };
} }
GeneralResult<Operand::SymmPerChannelQuantParams> convert( GeneralResult<Operand::SymmPerChannelQuantParams> unvalidatedConvert(
const hal::V1_2::SymmPerChannelQuantParams& symmPerChannelQuantParams) { const hal::V1_2::SymmPerChannelQuantParams& symmPerChannelQuantParams) {
return Operand::SymmPerChannelQuantParams{ return Operand::SymmPerChannelQuantParams{
.scales = symmPerChannelQuantParams.scales, .scales = symmPerChannelQuantParams.scales,
@@ -160,25 +191,26 @@ GeneralResult<Operand::SymmPerChannelQuantParams> convert(
}; };
} }
GeneralResult<Operand> convert(const hal::V1_2::Operand& operand) { GeneralResult<Operand> unvalidatedConvert(const hal::V1_2::Operand& operand) {
return Operand{ return Operand{
.type = NN_TRY(convert(operand.type)), .type = NN_TRY(unvalidatedConvert(operand.type)),
.dimensions = operand.dimensions, .dimensions = operand.dimensions,
.scale = operand.scale, .scale = operand.scale,
.zeroPoint = operand.zeroPoint, .zeroPoint = operand.zeroPoint,
.lifetime = NN_TRY(convert(operand.lifetime)), .lifetime = NN_TRY(unvalidatedConvert(operand.lifetime)),
.location = NN_TRY(convert(operand.location)), .location = NN_TRY(unvalidatedConvert(operand.location)),
.extraParams = NN_TRY(convert(operand.extraParams)), .extraParams = NN_TRY(unvalidatedConvert(operand.extraParams)),
}; };
} }
GeneralResult<Operand::ExtraParams> convert(const hal::V1_2::Operand::ExtraParams& extraParams) { GeneralResult<Operand::ExtraParams> unvalidatedConvert(
const hal::V1_2::Operand::ExtraParams& extraParams) {
using Discriminator = hal::V1_2::Operand::ExtraParams::hidl_discriminator; using Discriminator = hal::V1_2::Operand::ExtraParams::hidl_discriminator;
switch (extraParams.getDiscriminator()) { switch (extraParams.getDiscriminator()) {
case Discriminator::none: case Discriminator::none:
return Operand::NoParams{}; return Operand::NoParams{};
case Discriminator::channelQuant: case Discriminator::channelQuant:
return convert(extraParams.channelQuant()); return unvalidatedConvert(extraParams.channelQuant());
case Discriminator::extension: case Discriminator::extension:
return extraParams.extension(); return extraParams.extension();
} }
@@ -187,8 +219,8 @@ GeneralResult<Operand::ExtraParams> convert(const hal::V1_2::Operand::ExtraParam
<< underlyingType(extraParams.getDiscriminator()); << underlyingType(extraParams.getDiscriminator());
} }
GeneralResult<Model> convert(const hal::V1_2::Model& model) { GeneralResult<Model> unvalidatedConvert(const hal::V1_2::Model& model) {
auto operations = NN_TRY(convert(model.operations)); auto operations = NN_TRY(unvalidatedConvert(model.operations));
// Verify number of consumers. // Verify number of consumers.
const auto numberOfConsumers = const auto numberOfConsumers =
@@ -203,7 +235,7 @@ GeneralResult<Model> convert(const hal::V1_2::Model& model) {
} }
auto main = Model::Subgraph{ auto main = Model::Subgraph{
.operands = NN_TRY(convert(model.operands)), .operands = NN_TRY(unvalidatedConvert(model.operands)),
.operations = std::move(operations), .operations = std::move(operations),
.inputIndexes = model.inputIndexes, .inputIndexes = model.inputIndexes,
.outputIndexes = model.outputIndexes, .outputIndexes = model.outputIndexes,
@@ -211,14 +243,14 @@ GeneralResult<Model> convert(const hal::V1_2::Model& model) {
return Model{ return Model{
.main = std::move(main), .main = std::move(main),
.operandValues = NN_TRY(convert(model.operandValues)), .operandValues = NN_TRY(unvalidatedConvert(model.operandValues)),
.pools = NN_TRY(convert(model.pools)), .pools = NN_TRY(unvalidatedConvert(model.pools)),
.relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16, .relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16,
.extensionNameToPrefix = NN_TRY(convert(model.extensionNameToPrefix)), .extensionNameToPrefix = NN_TRY(unvalidatedConvert(model.extensionNameToPrefix)),
}; };
} }
GeneralResult<Model::ExtensionNameAndPrefix> convert( GeneralResult<Model::ExtensionNameAndPrefix> unvalidatedConvert(
const hal::V1_2::Model::ExtensionNameAndPrefix& extensionNameAndPrefix) { const hal::V1_2::Model::ExtensionNameAndPrefix& extensionNameAndPrefix) {
return Model::ExtensionNameAndPrefix{ return Model::ExtensionNameAndPrefix{
.name = extensionNameAndPrefix.name, .name = extensionNameAndPrefix.name,
@@ -226,29 +258,29 @@ GeneralResult<Model::ExtensionNameAndPrefix> convert(
}; };
} }
GeneralResult<OutputShape> convert(const hal::V1_2::OutputShape& outputShape) { GeneralResult<OutputShape> unvalidatedConvert(const hal::V1_2::OutputShape& outputShape) {
return OutputShape{ return OutputShape{
.dimensions = outputShape.dimensions, .dimensions = outputShape.dimensions,
.isSufficient = outputShape.isSufficient, .isSufficient = outputShape.isSufficient,
}; };
} }
GeneralResult<MeasureTiming> convert(const hal::V1_2::MeasureTiming& measureTiming) { GeneralResult<MeasureTiming> unvalidatedConvert(const hal::V1_2::MeasureTiming& measureTiming) {
return static_cast<MeasureTiming>(measureTiming); return static_cast<MeasureTiming>(measureTiming);
} }
GeneralResult<Timing> convert(const hal::V1_2::Timing& timing) { GeneralResult<Timing> unvalidatedConvert(const hal::V1_2::Timing& timing) {
return Timing{.timeOnDevice = timing.timeOnDevice, .timeInDriver = timing.timeInDriver}; return Timing{.timeOnDevice = timing.timeOnDevice, .timeInDriver = timing.timeInDriver};
} }
GeneralResult<Extension> convert(const hal::V1_2::Extension& extension) { GeneralResult<Extension> unvalidatedConvert(const hal::V1_2::Extension& extension) {
return Extension{ return Extension{
.name = extension.name, .name = extension.name,
.operandTypes = NN_TRY(convert(extension.operandTypes)), .operandTypes = NN_TRY(unvalidatedConvert(extension.operandTypes)),
}; };
} }
GeneralResult<Extension::OperandTypeInformation> convert( GeneralResult<Extension::OperandTypeInformation> unvalidatedConvert(
const hal::V1_2::Extension::OperandTypeInformation& operandTypeInformation) { const hal::V1_2::Extension::OperandTypeInformation& operandTypeInformation) {
return Extension::OperandTypeInformation{ return Extension::OperandTypeInformation{
.type = operandTypeInformation.type, .type = operandTypeInformation.type,
@@ -257,21 +289,41 @@ GeneralResult<Extension::OperandTypeInformation> convert(
}; };
} }
GeneralResult<SharedHandle> convert(const hidl_handle& hidlHandle) { GeneralResult<SharedHandle> unvalidatedConvert(const hidl_handle& hidlHandle) {
return hal::utils::sharedHandleFromNativeHandle(hidlHandle.getNativeHandle()); return hal::utils::sharedHandleFromNativeHandle(hidlHandle.getNativeHandle());
} }
GeneralResult<DeviceType> convert(const hal::V1_2::DeviceType& deviceType) {
return validatedConvert(deviceType);
}
GeneralResult<Capabilities> convert(const hal::V1_2::Capabilities& capabilities) {
return validatedConvert(capabilities);
}
GeneralResult<Model> convert(const hal::V1_2::Model& model) {
return validatedConvert(model);
}
GeneralResult<MeasureTiming> convert(const hal::V1_2::MeasureTiming& measureTiming) {
return validatedConvert(measureTiming);
}
GeneralResult<Timing> convert(const hal::V1_2::Timing& timing) {
return validatedConvert(timing);
}
GeneralResult<std::vector<Extension>> convert(const hidl_vec<hal::V1_2::Extension>& extensions) { GeneralResult<std::vector<Extension>> convert(const hidl_vec<hal::V1_2::Extension>& extensions) {
return convertVec(extensions); return validatedConvert(extensions);
} }
GeneralResult<std::vector<SharedHandle>> convert(const hidl_vec<hidl_handle>& handles) { GeneralResult<std::vector<SharedHandle>> convert(const hidl_vec<hidl_handle>& handles) {
return convertVec(handles); return validatedConvert(handles);
} }
GeneralResult<std::vector<OutputShape>> convert( GeneralResult<std::vector<OutputShape>> convert(
const hidl_vec<hal::V1_2::OutputShape>& outputShapes) { const hidl_vec<hal::V1_2::OutputShape>& outputShapes) {
return convertVec(outputShapes); return validatedConvert(outputShapes);
} }
} // namespace android::nn } // namespace android::nn
@@ -279,44 +331,48 @@ GeneralResult<std::vector<OutputShape>> convert(
namespace android::hardware::neuralnetworks::V1_2::utils { namespace android::hardware::neuralnetworks::V1_2::utils {
namespace { namespace {
using utils::convert; using utils::unvalidatedConvert;
nn::GeneralResult<V1_0::OperandLifeTime> convert(const nn::Operand::LifeTime& lifetime) { nn::GeneralResult<V1_0::OperandLifeTime> unvalidatedConvert(const nn::Operand::LifeTime& lifetime) {
return V1_0::utils::convert(lifetime); return V1_0::utils::unvalidatedConvert(lifetime);
} }
nn::GeneralResult<V1_0::PerformanceInfo> convert( nn::GeneralResult<V1_0::PerformanceInfo> unvalidatedConvert(
const nn::Capabilities::PerformanceInfo& performanceInfo) { const nn::Capabilities::PerformanceInfo& performanceInfo) {
return V1_0::utils::convert(performanceInfo); return V1_0::utils::unvalidatedConvert(performanceInfo);
} }
nn::GeneralResult<V1_0::DataLocation> convert(const nn::DataLocation& location) { nn::GeneralResult<V1_0::DataLocation> unvalidatedConvert(const nn::DataLocation& location) {
return V1_0::utils::convert(location); return V1_0::utils::unvalidatedConvert(location);
} }
nn::GeneralResult<hidl_vec<uint8_t>> convert(const nn::Model::OperandValues& operandValues) { nn::GeneralResult<hidl_vec<uint8_t>> unvalidatedConvert(
return V1_0::utils::convert(operandValues); const nn::Model::OperandValues& operandValues) {
return V1_0::utils::unvalidatedConvert(operandValues);
} }
nn::GeneralResult<hidl_memory> convert(const nn::Memory& memory) { nn::GeneralResult<hidl_memory> unvalidatedConvert(const nn::Memory& memory) {
return V1_0::utils::convert(memory); return V1_0::utils::unvalidatedConvert(memory);
} }
template <typename Input> template <typename Input>
using ConvertOutput = std::decay_t<decltype(convert(std::declval<Input>()).value())>; using unvalidatedConvertOutput =
std::decay_t<decltype(unvalidatedConvert(std::declval<Input>()).value())>;
template <typename Type> template <typename Type>
nn::GeneralResult<hidl_vec<ConvertOutput<Type>>> convertVec(const std::vector<Type>& arguments) { nn::GeneralResult<hidl_vec<unvalidatedConvertOutput<Type>>> unvalidatedConvertVec(
hidl_vec<ConvertOutput<Type>> halObject(arguments.size()); const std::vector<Type>& arguments) {
hidl_vec<unvalidatedConvertOutput<Type>> halObject(arguments.size());
for (size_t i = 0; i < arguments.size(); ++i) { for (size_t i = 0; i < arguments.size(); ++i) {
halObject[i] = NN_TRY(convert(arguments[i])); halObject[i] = NN_TRY(unvalidatedConvert(arguments[i]));
} }
return halObject; return halObject;
} }
template <typename Type> template <typename Type>
nn::GeneralResult<hidl_vec<ConvertOutput<Type>>> convert(const std::vector<Type>& arguments) { nn::GeneralResult<hidl_vec<unvalidatedConvertOutput<Type>>> unvalidatedConvert(
return convertVec(arguments); const std::vector<Type>& arguments) {
return unvalidatedConvertVec(arguments);
} }
nn::GeneralResult<Operand::ExtraParams> makeExtraParams(nn::Operand::NoParams /*noParams*/) { nn::GeneralResult<Operand::ExtraParams> makeExtraParams(nn::Operand::NoParams /*noParams*/) {
@@ -326,7 +382,7 @@ nn::GeneralResult<Operand::ExtraParams> makeExtraParams(nn::Operand::NoParams /*
nn::GeneralResult<Operand::ExtraParams> makeExtraParams( nn::GeneralResult<Operand::ExtraParams> makeExtraParams(
const nn::Operand::SymmPerChannelQuantParams& channelQuant) { const nn::Operand::SymmPerChannelQuantParams& channelQuant) {
Operand::ExtraParams ret; Operand::ExtraParams ret;
ret.channelQuant(NN_TRY(convert(channelQuant))); ret.channelQuant(NN_TRY(unvalidatedConvert(channelQuant)));
return ret; return ret;
} }
@@ -337,17 +393,40 @@ nn::GeneralResult<Operand::ExtraParams> makeExtraParams(
return ret; return ret;
} }
template <typename Type>
decltype(utils::unvalidatedConvert(std::declval<Type>())) validatedConvert(const Type& canonical) {
const auto maybeVersion = nn::validate(canonical);
if (!maybeVersion.has_value()) {
return nn::error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required " << kVersion;
}
return utils::unvalidatedConvert(canonical);
}
template <typename Type>
nn::GeneralResult<hidl_vec<unvalidatedConvertOutput<Type>>> validatedConvert(
const std::vector<Type>& arguments) {
hidl_vec<unvalidatedConvertOutput<Type>> halObject(arguments.size());
for (size_t i = 0; i < arguments.size(); ++i) {
halObject[i] = NN_TRY(validatedConvert(arguments[i]));
}
return halObject;
}
} // anonymous namespace } // anonymous namespace
nn::GeneralResult<OperandType> convert(const nn::OperandType& operandType) { nn::GeneralResult<OperandType> unvalidatedConvert(const nn::OperandType& operandType) {
return static_cast<OperandType>(operandType); return static_cast<OperandType>(operandType);
} }
nn::GeneralResult<OperationType> convert(const nn::OperationType& operationType) { nn::GeneralResult<OperationType> unvalidatedConvert(const nn::OperationType& operationType) {
return static_cast<OperationType>(operationType); return static_cast<OperationType>(operationType);
} }
nn::GeneralResult<DeviceType> convert(const nn::DeviceType& deviceType) { nn::GeneralResult<DeviceType> unvalidatedConvert(const nn::DeviceType& deviceType) {
switch (deviceType) { switch (deviceType) {
case nn::DeviceType::UNKNOWN: case nn::DeviceType::UNKNOWN:
return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) << "Invalid DeviceType UNKNOWN"; return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) << "Invalid DeviceType UNKNOWN";
@@ -361,7 +440,7 @@ nn::GeneralResult<DeviceType> convert(const nn::DeviceType& deviceType) {
<< "Invalid DeviceType " << underlyingType(deviceType); << "Invalid DeviceType " << underlyingType(deviceType);
} }
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities) { nn::GeneralResult<Capabilities> unvalidatedConvert(const nn::Capabilities& capabilities) {
std::vector<nn::Capabilities::OperandPerformance> operandPerformance; std::vector<nn::Capabilities::OperandPerformance> operandPerformance;
operandPerformance.reserve(capabilities.operandPerformance.asVector().size()); operandPerformance.reserve(capabilities.operandPerformance.asVector().size());
std::copy_if(capabilities.operandPerformance.asVector().begin(), std::copy_if(capabilities.operandPerformance.asVector().begin(),
@@ -372,31 +451,31 @@ nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities) {
}); });
return Capabilities{ return Capabilities{
.relaxedFloat32toFloat16PerformanceScalar = .relaxedFloat32toFloat16PerformanceScalar = NN_TRY(
NN_TRY(convert(capabilities.relaxedFloat32toFloat16PerformanceScalar)), unvalidatedConvert(capabilities.relaxedFloat32toFloat16PerformanceScalar)),
.relaxedFloat32toFloat16PerformanceTensor = .relaxedFloat32toFloat16PerformanceTensor = NN_TRY(
NN_TRY(convert(capabilities.relaxedFloat32toFloat16PerformanceTensor)), unvalidatedConvert(capabilities.relaxedFloat32toFloat16PerformanceTensor)),
.operandPerformance = NN_TRY(convert(operandPerformance)), .operandPerformance = NN_TRY(unvalidatedConvert(operandPerformance)),
}; };
} }
nn::GeneralResult<Capabilities::OperandPerformance> convert( nn::GeneralResult<Capabilities::OperandPerformance> unvalidatedConvert(
const nn::Capabilities::OperandPerformance& operandPerformance) { const nn::Capabilities::OperandPerformance& operandPerformance) {
return Capabilities::OperandPerformance{ return Capabilities::OperandPerformance{
.type = NN_TRY(convert(operandPerformance.type)), .type = NN_TRY(unvalidatedConvert(operandPerformance.type)),
.info = NN_TRY(convert(operandPerformance.info)), .info = NN_TRY(unvalidatedConvert(operandPerformance.info)),
}; };
} }
nn::GeneralResult<Operation> convert(const nn::Operation& operation) { nn::GeneralResult<Operation> unvalidatedConvert(const nn::Operation& operation) {
return Operation{ return Operation{
.type = NN_TRY(convert(operation.type)), .type = NN_TRY(unvalidatedConvert(operation.type)),
.inputs = operation.inputs, .inputs = operation.inputs,
.outputs = operation.outputs, .outputs = operation.outputs,
}; };
} }
nn::GeneralResult<SymmPerChannelQuantParams> convert( nn::GeneralResult<SymmPerChannelQuantParams> unvalidatedConvert(
const nn::Operand::SymmPerChannelQuantParams& symmPerChannelQuantParams) { const nn::Operand::SymmPerChannelQuantParams& symmPerChannelQuantParams) {
return SymmPerChannelQuantParams{ return SymmPerChannelQuantParams{
.scales = symmPerChannelQuantParams.scales, .scales = symmPerChannelQuantParams.scales,
@@ -404,30 +483,31 @@ nn::GeneralResult<SymmPerChannelQuantParams> convert(
}; };
} }
nn::GeneralResult<Operand> convert(const nn::Operand& operand) { nn::GeneralResult<Operand> unvalidatedConvert(const nn::Operand& operand) {
return Operand{ return Operand{
.type = NN_TRY(convert(operand.type)), .type = NN_TRY(unvalidatedConvert(operand.type)),
.dimensions = operand.dimensions, .dimensions = operand.dimensions,
.numberOfConsumers = 0, .numberOfConsumers = 0,
.scale = operand.scale, .scale = operand.scale,
.zeroPoint = operand.zeroPoint, .zeroPoint = operand.zeroPoint,
.lifetime = NN_TRY(convert(operand.lifetime)), .lifetime = NN_TRY(unvalidatedConvert(operand.lifetime)),
.location = NN_TRY(convert(operand.location)), .location = NN_TRY(unvalidatedConvert(operand.location)),
.extraParams = NN_TRY(convert(operand.extraParams)), .extraParams = NN_TRY(unvalidatedConvert(operand.extraParams)),
}; };
} }
nn::GeneralResult<Operand::ExtraParams> convert(const nn::Operand::ExtraParams& extraParams) { nn::GeneralResult<Operand::ExtraParams> unvalidatedConvert(
const nn::Operand::ExtraParams& extraParams) {
return std::visit([](const auto& x) { return makeExtraParams(x); }, extraParams); return std::visit([](const auto& x) { return makeExtraParams(x); }, extraParams);
} }
nn::GeneralResult<Model> convert(const nn::Model& model) { nn::GeneralResult<Model> unvalidatedConvert(const nn::Model& model) {
if (!hal::utils::hasNoPointerData(model)) { if (!hal::utils::hasNoPointerData(model)) {
return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT) return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT)
<< "Model cannot be converted because it contains pointer-based memory"; << "Model cannot be unvalidatedConverted because it contains pointer-based memory";
} }
auto operands = NN_TRY(convert(model.main.operands)); auto operands = NN_TRY(unvalidatedConvert(model.main.operands));
// Update number of consumers. // Update number of consumers.
const auto numberOfConsumers = const auto numberOfConsumers =
@@ -439,17 +519,17 @@ nn::GeneralResult<Model> convert(const nn::Model& model) {
return Model{ return Model{
.operands = std::move(operands), .operands = std::move(operands),
.operations = NN_TRY(convert(model.main.operations)), .operations = NN_TRY(unvalidatedConvert(model.main.operations)),
.inputIndexes = model.main.inputIndexes, .inputIndexes = model.main.inputIndexes,
.outputIndexes = model.main.outputIndexes, .outputIndexes = model.main.outputIndexes,
.operandValues = NN_TRY(convert(model.operandValues)), .operandValues = NN_TRY(unvalidatedConvert(model.operandValues)),
.pools = NN_TRY(convert(model.pools)), .pools = NN_TRY(unvalidatedConvert(model.pools)),
.relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16, .relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16,
.extensionNameToPrefix = NN_TRY(convert(model.extensionNameToPrefix)), .extensionNameToPrefix = NN_TRY(unvalidatedConvert(model.extensionNameToPrefix)),
}; };
} }
nn::GeneralResult<Model::ExtensionNameAndPrefix> convert( nn::GeneralResult<Model::ExtensionNameAndPrefix> unvalidatedConvert(
const nn::Model::ExtensionNameAndPrefix& extensionNameAndPrefix) { const nn::Model::ExtensionNameAndPrefix& extensionNameAndPrefix) {
return Model::ExtensionNameAndPrefix{ return Model::ExtensionNameAndPrefix{
.name = extensionNameAndPrefix.name, .name = extensionNameAndPrefix.name,
@@ -457,27 +537,27 @@ nn::GeneralResult<Model::ExtensionNameAndPrefix> convert(
}; };
} }
nn::GeneralResult<OutputShape> convert(const nn::OutputShape& outputShape) { nn::GeneralResult<OutputShape> unvalidatedConvert(const nn::OutputShape& outputShape) {
return OutputShape{.dimensions = outputShape.dimensions, return OutputShape{.dimensions = outputShape.dimensions,
.isSufficient = outputShape.isSufficient}; .isSufficient = outputShape.isSufficient};
} }
nn::GeneralResult<MeasureTiming> convert(const nn::MeasureTiming& measureTiming) { nn::GeneralResult<MeasureTiming> unvalidatedConvert(const nn::MeasureTiming& measureTiming) {
return static_cast<MeasureTiming>(measureTiming); return static_cast<MeasureTiming>(measureTiming);
} }
nn::GeneralResult<Timing> convert(const nn::Timing& timing) { nn::GeneralResult<Timing> unvalidatedConvert(const nn::Timing& timing) {
return Timing{.timeOnDevice = timing.timeOnDevice, .timeInDriver = timing.timeInDriver}; return Timing{.timeOnDevice = timing.timeOnDevice, .timeInDriver = timing.timeInDriver};
} }
nn::GeneralResult<Extension> convert(const nn::Extension& extension) { nn::GeneralResult<Extension> unvalidatedConvert(const nn::Extension& extension) {
return Extension{ return Extension{
.name = extension.name, .name = extension.name,
.operandTypes = NN_TRY(convert(extension.operandTypes)), .operandTypes = NN_TRY(unvalidatedConvert(extension.operandTypes)),
}; };
} }
nn::GeneralResult<Extension::OperandTypeInformation> convert( nn::GeneralResult<Extension::OperandTypeInformation> unvalidatedConvert(
const nn::Extension::OperandTypeInformation& operandTypeInformation) { const nn::Extension::OperandTypeInformation& operandTypeInformation) {
return Extension::OperandTypeInformation{ return Extension::OperandTypeInformation{
.type = operandTypeInformation.type, .type = operandTypeInformation.type,
@@ -486,20 +566,40 @@ nn::GeneralResult<Extension::OperandTypeInformation> convert(
}; };
} }
nn::GeneralResult<hidl_handle> convert(const nn::SharedHandle& handle) { nn::GeneralResult<hidl_handle> unvalidatedConvert(const nn::SharedHandle& handle) {
return hal::utils::hidlHandleFromSharedHandle(handle); return hal::utils::hidlHandleFromSharedHandle(handle);
} }
nn::GeneralResult<DeviceType> convert(const nn::DeviceType& deviceType) {
return validatedConvert(deviceType);
}
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities) {
return validatedConvert(capabilities);
}
nn::GeneralResult<Model> convert(const nn::Model& model) {
return validatedConvert(model);
}
nn::GeneralResult<MeasureTiming> convert(const nn::MeasureTiming& measureTiming) {
return validatedConvert(measureTiming);
}
nn::GeneralResult<Timing> convert(const nn::Timing& timing) {
return validatedConvert(timing);
}
nn::GeneralResult<hidl_vec<Extension>> convert(const std::vector<nn::Extension>& extensions) { nn::GeneralResult<hidl_vec<Extension>> convert(const std::vector<nn::Extension>& extensions) {
return convertVec(extensions); return validatedConvert(extensions);
} }
nn::GeneralResult<hidl_vec<hidl_handle>> convert(const std::vector<nn::SharedHandle>& handles) { nn::GeneralResult<hidl_vec<hidl_handle>> convert(const std::vector<nn::SharedHandle>& handles) {
return convertVec(handles); return validatedConvert(handles);
} }
nn::GeneralResult<hidl_vec<OutputShape>> convert(const std::vector<nn::OutputShape>& outputShapes) { nn::GeneralResult<hidl_vec<OutputShape>> convert(const std::vector<nn::OutputShape>& outputShapes) {
return convertVec(outputShapes); return validatedConvert(outputShapes);
} }
} // namespace android::hardware::neuralnetworks::V1_2::utils } // namespace android::hardware::neuralnetworks::V1_2::utils

View File

@@ -51,11 +51,10 @@ nn::GeneralResult<nn::Capabilities> initCapabilities(V1_2::IDevice* device) {
<< "uninitialized"; << "uninitialized";
const auto cb = [&result](V1_0::ErrorStatus status, const Capabilities& capabilities) { const auto cb = [&result](V1_0::ErrorStatus status, const Capabilities& capabilities) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "getCapabilities_1_2 failed with " << toString(status); result = NN_ERROR(canonical) << "getCapabilities_1_2 failed with " << toString(status);
} else { } else {
result = validatedConvertToCanonical(capabilities); result = nn::convert(capabilities);
} }
}; };
@@ -74,8 +73,7 @@ nn::GeneralResult<std::string> initVersionString(V1_2::IDevice* device) {
<< "uninitialized"; << "uninitialized";
const auto cb = [&result](V1_0::ErrorStatus status, const hidl_string& versionString) { const auto cb = [&result](V1_0::ErrorStatus status, const hidl_string& versionString) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "getVersionString failed with " << toString(status); result = NN_ERROR(canonical) << "getVersionString failed with " << toString(status);
} else { } else {
result = versionString; result = versionString;
@@ -95,8 +93,7 @@ nn::GeneralResult<nn::DeviceType> initDeviceType(V1_2::IDevice* device) {
<< "uninitialized"; << "uninitialized";
const auto cb = [&result](V1_0::ErrorStatus status, DeviceType deviceType) { const auto cb = [&result](V1_0::ErrorStatus status, DeviceType deviceType) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "getDeviceType failed with " << toString(status); result = NN_ERROR(canonical) << "getDeviceType failed with " << toString(status);
} else { } else {
result = nn::convert(deviceType); result = nn::convert(deviceType);
@@ -116,8 +113,7 @@ nn::GeneralResult<std::vector<nn::Extension>> initExtensions(V1_2::IDevice* devi
NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) << "uninitialized"; NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) << "uninitialized";
const auto cb = [&result](V1_0::ErrorStatus status, const hidl_vec<Extension>& extensions) { const auto cb = [&result](V1_0::ErrorStatus status, const hidl_vec<Extension>& extensions) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "getExtensions failed with " << toString(status); result = NN_ERROR(canonical) << "getExtensions failed with " << toString(status);
} else { } else {
result = nn::convert(extensions); result = nn::convert(extensions);
@@ -139,8 +135,7 @@ nn::GeneralResult<std::pair<uint32_t, uint32_t>> initNumberOfCacheFilesNeeded(
const auto cb = [&result](V1_0::ErrorStatus status, uint32_t numModelCache, const auto cb = [&result](V1_0::ErrorStatus status, uint32_t numModelCache,
uint32_t numDataCache) { uint32_t numDataCache) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) result = NN_ERROR(canonical)
<< "getNumberOfCacheFilesNeeded failed with " << toString(status); << "getNumberOfCacheFilesNeeded failed with " << toString(status);
} else { } else {
@@ -238,8 +233,7 @@ nn::GeneralResult<std::vector<bool>> Device::getSupportedOperations(const nn::Mo
auto cb = [&result, &model](V1_0::ErrorStatus status, auto cb = [&result, &model](V1_0::ErrorStatus status,
const hidl_vec<bool>& supportedOperations) { const hidl_vec<bool>& supportedOperations) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) result = NN_ERROR(canonical)
<< "getSupportedOperations_1_2 failed with " << toString(status); << "getSupportedOperations_1_2 failed with " << toString(status);
} else if (supportedOperations.size() != model.main.operations.size()) { } else if (supportedOperations.size() != model.main.operations.size()) {
@@ -280,8 +274,7 @@ nn::GeneralResult<nn::SharedPreparedModel> Device::prepareModel(
hidlDataCache, hidlToken, cb); hidlDataCache, hidlToken, cb);
const auto status = NN_TRY(hal::utils::handleTransportError(ret)); const auto status = NN_TRY(hal::utils::handleTransportError(ret));
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "prepareModel_1_2 failed with " << toString(status); return NN_ERROR(canonical) << "prepareModel_1_2 failed with " << toString(status);
} }
@@ -301,8 +294,7 @@ nn::GeneralResult<nn::SharedPreparedModel> Device::prepareModelFromCache(
const auto ret = kDevice->prepareModelFromCache(hidlModelCache, hidlDataCache, hidlToken, cb); const auto ret = kDevice->prepareModelFromCache(hidlModelCache, hidlDataCache, hidlToken, cb);
const auto status = NN_TRY(hal::utils::handleTransportError(ret)); const auto status = NN_TRY(hal::utils::handleTransportError(ret));
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "prepareModelFromCache failed with " << toString(status); return NN_ERROR(canonical) << "prepareModelFromCache failed with " << toString(status);
} }

View File

@@ -42,8 +42,7 @@ namespace {
nn::GeneralResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> nn::GeneralResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>>
convertExecutionResultsHelper(const hidl_vec<OutputShape>& outputShapes, const Timing& timing) { convertExecutionResultsHelper(const hidl_vec<OutputShape>& outputShapes, const Timing& timing) {
return std::make_pair(NN_TRY(validatedConvertToCanonical(outputShapes)), return std::make_pair(NN_TRY(nn::convert(outputShapes)), NN_TRY(nn::convert(timing)));
NN_TRY(validatedConvertToCanonical(timing)));
} }
nn::ExecutionResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> convertExecutionResults( nn::ExecutionResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> convertExecutionResults(
@@ -76,8 +75,7 @@ PreparedModel::executeSynchronously(const V1_0::Request& request, MeasureTiming
const auto cb = [&result](V1_0::ErrorStatus status, const hidl_vec<OutputShape>& outputShapes, const auto cb = [&result](V1_0::ErrorStatus status, const hidl_vec<OutputShape>& outputShapes,
const Timing& timing) { const Timing& timing) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "executeSynchronously failed with " << toString(status); result = NN_ERROR(canonical) << "executeSynchronously failed with " << toString(status);
} else { } else {
result = convertExecutionResults(outputShapes, timing); result = convertExecutionResults(outputShapes, timing);
@@ -99,8 +97,7 @@ PreparedModel::executeAsynchronously(const V1_0::Request& request, MeasureTiming
const auto status = const auto status =
NN_TRY(hal::utils::makeExecutionFailure(hal::utils::handleTransportError(ret))); NN_TRY(hal::utils::makeExecutionFailure(hal::utils::handleTransportError(ret)));
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "execute failed with " << toString(status); return NN_ERROR(canonical) << "execute failed with " << toString(status);
} }

View File

@@ -25,26 +25,41 @@
namespace android::nn { namespace android::nn {
GeneralResult<OperandType> convert(const hal::V1_3::OperandType& operandType); GeneralResult<OperandType> unvalidatedConvert(const hal::V1_3::OperandType& operandType);
GeneralResult<OperationType> convert(const hal::V1_3::OperationType& operationType); GeneralResult<OperationType> unvalidatedConvert(const hal::V1_3::OperationType& operationType);
GeneralResult<Priority> unvalidatedConvert(const hal::V1_3::Priority& priority);
GeneralResult<Capabilities> unvalidatedConvert(const hal::V1_3::Capabilities& capabilities);
GeneralResult<Capabilities::OperandPerformance> unvalidatedConvert(
const hal::V1_3::Capabilities::OperandPerformance& operandPerformance);
GeneralResult<Operation> unvalidatedConvert(const hal::V1_3::Operation& operation);
GeneralResult<Operand::LifeTime> unvalidatedConvert(
const hal::V1_3::OperandLifeTime& operandLifeTime);
GeneralResult<Operand> unvalidatedConvert(const hal::V1_3::Operand& operand);
GeneralResult<Model> unvalidatedConvert(const hal::V1_3::Model& model);
GeneralResult<Model::Subgraph> unvalidatedConvert(const hal::V1_3::Subgraph& subgraph);
GeneralResult<BufferDesc> unvalidatedConvert(const hal::V1_3::BufferDesc& bufferDesc);
GeneralResult<BufferRole> unvalidatedConvert(const hal::V1_3::BufferRole& bufferRole);
GeneralResult<Request> unvalidatedConvert(const hal::V1_3::Request& request);
GeneralResult<Request::MemoryPool> unvalidatedConvert(
const hal::V1_3::Request::MemoryPool& memoryPool);
GeneralResult<OptionalTimePoint> unvalidatedConvert(
const hal::V1_3::OptionalTimePoint& optionalTimePoint);
GeneralResult<OptionalTimeoutDuration> unvalidatedConvert(
const hal::V1_3::OptionalTimeoutDuration& optionalTimeoutDuration);
GeneralResult<ErrorStatus> unvalidatedConvert(const hal::V1_3::ErrorStatus& errorStatus);
GeneralResult<Priority> convert(const hal::V1_3::Priority& priority); GeneralResult<Priority> convert(const hal::V1_3::Priority& priority);
GeneralResult<Capabilities> convert(const hal::V1_3::Capabilities& capabilities); GeneralResult<Capabilities> convert(const hal::V1_3::Capabilities& capabilities);
GeneralResult<Capabilities::OperandPerformance> convert(
const hal::V1_3::Capabilities::OperandPerformance& operandPerformance);
GeneralResult<Operation> convert(const hal::V1_3::Operation& operation);
GeneralResult<Operand::LifeTime> convert(const hal::V1_3::OperandLifeTime& operandLifeTime);
GeneralResult<Operand> convert(const hal::V1_3::Operand& operand);
GeneralResult<Model> convert(const hal::V1_3::Model& model); GeneralResult<Model> convert(const hal::V1_3::Model& model);
GeneralResult<Model::Subgraph> convert(const hal::V1_3::Subgraph& subgraph);
GeneralResult<BufferDesc> convert(const hal::V1_3::BufferDesc& bufferDesc); GeneralResult<BufferDesc> convert(const hal::V1_3::BufferDesc& bufferDesc);
GeneralResult<BufferRole> convert(const hal::V1_3::BufferRole& bufferRole);
GeneralResult<Request> convert(const hal::V1_3::Request& request); GeneralResult<Request> convert(const hal::V1_3::Request& request);
GeneralResult<Request::MemoryPool> convert(const hal::V1_3::Request::MemoryPool& memoryPool);
GeneralResult<OptionalTimePoint> convert(const hal::V1_3::OptionalTimePoint& optionalTimePoint); GeneralResult<OptionalTimePoint> convert(const hal::V1_3::OptionalTimePoint& optionalTimePoint);
GeneralResult<OptionalTimeoutDuration> convert( GeneralResult<OptionalTimeoutDuration> convert(
const hal::V1_3::OptionalTimeoutDuration& optionalTimeoutDuration); const hal::V1_3::OptionalTimeoutDuration& optionalTimeoutDuration);
GeneralResult<ErrorStatus> convert(const hal::V1_3::ErrorStatus& errorStatus); GeneralResult<ErrorStatus> convert(const hal::V1_3::ErrorStatus& errorStatus);
GeneralResult<SharedHandle> convert(const hardware::hidl_handle& handle);
GeneralResult<Memory> convert(const hardware::hidl_memory& memory);
GeneralResult<std::vector<BufferRole>> convert( GeneralResult<std::vector<BufferRole>> convert(
const hardware::hidl_vec<hal::V1_3::BufferRole>& bufferRoles); const hardware::hidl_vec<hal::V1_3::BufferRole>& bufferRoles);
@@ -52,26 +67,40 @@ GeneralResult<std::vector<BufferRole>> convert(
namespace android::hardware::neuralnetworks::V1_3::utils { namespace android::hardware::neuralnetworks::V1_3::utils {
nn::GeneralResult<OperandType> convert(const nn::OperandType& operandType); nn::GeneralResult<OperandType> unvalidatedConvert(const nn::OperandType& operandType);
nn::GeneralResult<OperationType> convert(const nn::OperationType& operationType); nn::GeneralResult<OperationType> unvalidatedConvert(const nn::OperationType& operationType);
nn::GeneralResult<Priority> unvalidatedConvert(const nn::Priority& priority);
nn::GeneralResult<Capabilities> unvalidatedConvert(const nn::Capabilities& capabilities);
nn::GeneralResult<Capabilities::OperandPerformance> unvalidatedConvert(
const nn::Capabilities::OperandPerformance& operandPerformance);
nn::GeneralResult<Operation> unvalidatedConvert(const nn::Operation& operation);
nn::GeneralResult<OperandLifeTime> unvalidatedConvert(const nn::Operand::LifeTime& operandLifeTime);
nn::GeneralResult<Operand> unvalidatedConvert(const nn::Operand& operand);
nn::GeneralResult<Model> unvalidatedConvert(const nn::Model& model);
nn::GeneralResult<Subgraph> unvalidatedConvert(const nn::Model::Subgraph& subgraph);
nn::GeneralResult<BufferDesc> unvalidatedConvert(const nn::BufferDesc& bufferDesc);
nn::GeneralResult<BufferRole> unvalidatedConvert(const nn::BufferRole& bufferRole);
nn::GeneralResult<Request> unvalidatedConvert(const nn::Request& request);
nn::GeneralResult<Request::MemoryPool> unvalidatedConvert(
const nn::Request::MemoryPool& memoryPool);
nn::GeneralResult<OptionalTimePoint> unvalidatedConvert(
const nn::OptionalTimePoint& optionalTimePoint);
nn::GeneralResult<OptionalTimeoutDuration> unvalidatedConvert(
const nn::OptionalTimeoutDuration& optionalTimeoutDuration);
nn::GeneralResult<ErrorStatus> unvalidatedConvert(const nn::ErrorStatus& errorStatus);
nn::GeneralResult<Priority> convert(const nn::Priority& priority); nn::GeneralResult<Priority> convert(const nn::Priority& priority);
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities); nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities);
nn::GeneralResult<Capabilities::OperandPerformance> convert(
const nn::Capabilities::OperandPerformance& operandPerformance);
nn::GeneralResult<Operation> convert(const nn::Operation& operation);
nn::GeneralResult<OperandLifeTime> convert(const nn::Operand::LifeTime& operandLifeTime);
nn::GeneralResult<Operand> convert(const nn::Operand& operand);
nn::GeneralResult<Model> convert(const nn::Model& model); nn::GeneralResult<Model> convert(const nn::Model& model);
nn::GeneralResult<Subgraph> convert(const nn::Model::Subgraph& subgraph);
nn::GeneralResult<BufferDesc> convert(const nn::BufferDesc& bufferDesc); nn::GeneralResult<BufferDesc> convert(const nn::BufferDesc& bufferDesc);
nn::GeneralResult<BufferRole> convert(const nn::BufferRole& bufferRole);
nn::GeneralResult<Request> convert(const nn::Request& request); nn::GeneralResult<Request> convert(const nn::Request& request);
nn::GeneralResult<Request::MemoryPool> convert(const nn::Request::MemoryPool& memoryPool);
nn::GeneralResult<OptionalTimePoint> convert(const nn::OptionalTimePoint& optionalTimePoint); nn::GeneralResult<OptionalTimePoint> convert(const nn::OptionalTimePoint& optionalTimePoint);
nn::GeneralResult<OptionalTimeoutDuration> convert( nn::GeneralResult<OptionalTimeoutDuration> convert(
const nn::OptionalTimeoutDuration& optionalTimeoutDuration); const nn::OptionalTimeoutDuration& optionalTimeoutDuration);
nn::GeneralResult<ErrorStatus> convert(const nn::ErrorStatus& errorStatus); nn::GeneralResult<ErrorStatus> convert(const nn::ErrorStatus& errorStatus);
nn::GeneralResult<hidl_handle> convert(const nn::SharedHandle& handle);
nn::GeneralResult<hidl_memory> convert(const nn::Memory& memory);
nn::GeneralResult<hidl_vec<BufferRole>> convert(const std::vector<nn::BufferRole>& bufferRoles); nn::GeneralResult<hidl_vec<BufferRole>> convert(const std::vector<nn::BufferRole>& bufferRoles);
} // namespace android::hardware::neuralnetworks::V1_3::utils } // namespace android::hardware::neuralnetworks::V1_3::utils

View File

@@ -22,9 +22,7 @@
#include <android-base/logging.h> #include <android-base/logging.h>
#include <android/hardware/neuralnetworks/1.3/types.h> #include <android/hardware/neuralnetworks/1.3/types.h>
#include <nnapi/Result.h> #include <nnapi/Result.h>
#include <nnapi/TypeUtils.h>
#include <nnapi/Types.h> #include <nnapi/Types.h>
#include <nnapi/Validation.h>
#include <nnapi/hal/1.0/Conversions.h> #include <nnapi/hal/1.0/Conversions.h>
#include <nnapi/hal/1.1/Conversions.h> #include <nnapi/hal/1.1/Conversions.h>
#include <nnapi/hal/1.2/Conversions.h> #include <nnapi/hal/1.2/Conversions.h>
@@ -32,7 +30,6 @@
namespace android::hardware::neuralnetworks::V1_3::utils { namespace android::hardware::neuralnetworks::V1_3::utils {
constexpr auto kDefaultPriority = Priority::MEDIUM; constexpr auto kDefaultPriority = Priority::MEDIUM;
constexpr auto kVersion = nn::Version::ANDROID_R;
template <typename Type> template <typename Type>
nn::Result<void> validate(const Type& halObject) { nn::Result<void> validate(const Type& halObject) {
@@ -40,11 +37,6 @@ nn::Result<void> validate(const Type& halObject) {
if (!maybeCanonical.has_value()) { if (!maybeCanonical.has_value()) {
return nn::error() << maybeCanonical.error().message; return nn::error() << maybeCanonical.error().message;
} }
const auto version = NN_TRY(nn::validate(maybeCanonical.value()));
if (version > utils::kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required "
<< utils::kVersion;
}
return {}; return {};
} }
@@ -57,21 +49,6 @@ bool valid(const Type& halObject) {
return result.has_value(); return result.has_value();
} }
template <typename Type>
decltype(nn::convert(std::declval<Type>())) validatedConvertToCanonical(const Type& halObject) {
auto canonical = NN_TRY(nn::convert(halObject));
const auto maybeVersion = nn::validate(canonical);
if (!maybeVersion.has_value()) {
return nn::error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > utils::kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required "
<< utils::kVersion;
}
return canonical;
}
} // namespace android::hardware::neuralnetworks::V1_3::utils } // namespace android::hardware::neuralnetworks::V1_3::utils
#endif // ANDROID_HARDWARE_INTERFACES_NEURALNETWORKS_1_3_UTILS_H #endif // ANDROID_HARDWARE_INTERFACES_NEURALNETWORKS_1_3_UTILS_H

View File

@@ -61,13 +61,12 @@ nn::Request::MemoryDomainToken Buffer::getToken() const {
} }
nn::GeneralResult<void> Buffer::copyTo(const nn::Memory& dst) const { nn::GeneralResult<void> Buffer::copyTo(const nn::Memory& dst) const {
const auto hidlDst = NN_TRY(V1_0::utils::convert(dst)); const auto hidlDst = NN_TRY(convert(dst));
const auto ret = kBuffer->copyTo(hidlDst); const auto ret = kBuffer->copyTo(hidlDst);
const auto status = NN_TRY(hal::utils::handleTransportError(ret)); const auto status = NN_TRY(hal::utils::handleTransportError(ret));
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "IBuffer::copyTo failed with " << toString(status); return NN_ERROR(canonical) << "IBuffer::copyTo failed with " << toString(status);
} }
@@ -76,14 +75,13 @@ nn::GeneralResult<void> Buffer::copyTo(const nn::Memory& dst) const {
nn::GeneralResult<void> Buffer::copyFrom(const nn::Memory& src, nn::GeneralResult<void> Buffer::copyFrom(const nn::Memory& src,
const nn::Dimensions& dimensions) const { const nn::Dimensions& dimensions) const {
const auto hidlSrc = NN_TRY(V1_0::utils::convert(src)); const auto hidlSrc = NN_TRY(convert(src));
const auto hidlDimensions = hidl_vec<uint32_t>(dimensions); const auto hidlDimensions = hidl_vec<uint32_t>(dimensions);
const auto ret = kBuffer->copyFrom(hidlSrc, hidlDimensions); const auto ret = kBuffer->copyFrom(hidlSrc, hidlDimensions);
const auto status = NN_TRY(hal::utils::handleTransportError(ret)); const auto status = NN_TRY(hal::utils::handleTransportError(ret));
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "IBuffer::copyFrom failed with " << toString(status); return NN_ERROR(canonical) << "IBuffer::copyFrom failed with " << toString(status);
} }

View File

@@ -60,8 +60,7 @@ nn::GeneralResult<nn::SharedPreparedModel> convertPreparedModel(
nn::GeneralResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> nn::GeneralResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>>
convertExecutionGeneralResultsHelper(const hidl_vec<V1_2::OutputShape>& outputShapes, convertExecutionGeneralResultsHelper(const hidl_vec<V1_2::OutputShape>& outputShapes,
const V1_2::Timing& timing) { const V1_2::Timing& timing) {
return std::make_pair(NN_TRY(validatedConvertToCanonical(outputShapes)), return std::make_pair(NN_TRY(nn::convert(outputShapes)), NN_TRY(nn::convert(timing)));
NN_TRY(validatedConvertToCanonical(timing)));
} }
nn::ExecutionResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> nn::ExecutionResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>>
@@ -76,8 +75,7 @@ convertExecutionGeneralResults(const hidl_vec<V1_2::OutputShape>& outputShapes,
Return<void> PreparedModelCallback::notify(V1_0::ErrorStatus status, Return<void> PreparedModelCallback::notify(V1_0::ErrorStatus status,
const sp<V1_0::IPreparedModel>& preparedModel) { const sp<V1_0::IPreparedModel>& preparedModel) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status));
} else if (preparedModel == nullptr) { } else if (preparedModel == nullptr) {
notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
@@ -91,8 +89,7 @@ Return<void> PreparedModelCallback::notify(V1_0::ErrorStatus status,
Return<void> PreparedModelCallback::notify_1_2(V1_0::ErrorStatus status, Return<void> PreparedModelCallback::notify_1_2(V1_0::ErrorStatus status,
const sp<V1_2::IPreparedModel>& preparedModel) { const sp<V1_2::IPreparedModel>& preparedModel) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status));
} else if (preparedModel == nullptr) { } else if (preparedModel == nullptr) {
notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
@@ -106,8 +103,7 @@ Return<void> PreparedModelCallback::notify_1_2(V1_0::ErrorStatus status,
Return<void> PreparedModelCallback::notify_1_3(ErrorStatus status, Return<void> PreparedModelCallback::notify_1_3(ErrorStatus status,
const sp<IPreparedModel>& preparedModel) { const sp<IPreparedModel>& preparedModel) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "preparedModel failed with " << toString(status));
} else if (preparedModel == nullptr) { } else if (preparedModel == nullptr) {
notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) notifyInternal(NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
@@ -134,8 +130,7 @@ void PreparedModelCallback::notifyInternal(PreparedModelCallback::Data result) {
Return<void> ExecutionCallback::notify(V1_0::ErrorStatus status) { Return<void> ExecutionCallback::notify(V1_0::ErrorStatus status) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status));
} else { } else {
notifyInternal({}); notifyInternal({});
@@ -147,8 +142,7 @@ Return<void> ExecutionCallback::notify_1_2(V1_0::ErrorStatus status,
const hidl_vec<V1_2::OutputShape>& outputShapes, const hidl_vec<V1_2::OutputShape>& outputShapes,
const V1_2::Timing& timing) { const V1_2::Timing& timing) {
if (status != V1_0::ErrorStatus::NONE) { if (status != V1_0::ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status));
} else { } else {
notifyInternal(convertExecutionGeneralResults(outputShapes, timing)); notifyInternal(convertExecutionGeneralResults(outputShapes, timing));
@@ -160,8 +154,7 @@ Return<void> ExecutionCallback::notify_1_3(ErrorStatus status,
const hidl_vec<V1_2::OutputShape>& outputShapes, const hidl_vec<V1_2::OutputShape>& outputShapes,
const V1_2::Timing& timing) { const V1_2::Timing& timing) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status)); notifyInternal(NN_ERROR(canonical) << "execute failed with " << toString(status));
} else { } else {
notifyInternal(convertExecutionGeneralResults(outputShapes, timing)); notifyInternal(convertExecutionGeneralResults(outputShapes, timing));

View File

@@ -24,6 +24,7 @@
#include <nnapi/SharedMemory.h> #include <nnapi/SharedMemory.h>
#include <nnapi/TypeUtils.h> #include <nnapi/TypeUtils.h>
#include <nnapi/Types.h> #include <nnapi/Types.h>
#include <nnapi/Validation.h>
#include <nnapi/hal/1.0/Conversions.h> #include <nnapi/hal/1.0/Conversions.h>
#include <nnapi/hal/1.2/Conversions.h> #include <nnapi/hal/1.2/Conversions.h>
#include <nnapi/hal/CommonUtils.h> #include <nnapi/hal/CommonUtils.h>
@@ -44,6 +45,8 @@ constexpr std::underlying_type_t<Type> underlyingType(Type value) {
return static_cast<std::underlying_type_t<Type>>(value); return static_cast<std::underlying_type_t<Type>>(value);
} }
constexpr auto kVersion = android::nn::Version::ANDROID_R;
} // namespace } // namespace
namespace android::nn { namespace android::nn {
@@ -77,110 +80,140 @@ constexpr auto validOperandType(nn::OperandType operandType) {
using hardware::hidl_vec; using hardware::hidl_vec;
template <typename Input> template <typename Input>
using ConvertOutput = std::decay_t<decltype(convert(std::declval<Input>()).value())>; using unvalidatedConvertOutput =
std::decay_t<decltype(unvalidatedConvert(std::declval<Input>()).value())>;
template <typename Type> template <typename Type>
GeneralResult<std::vector<ConvertOutput<Type>>> convertVec(const hidl_vec<Type>& arguments) { GeneralResult<std::vector<unvalidatedConvertOutput<Type>>> unvalidatedConvertVec(
std::vector<ConvertOutput<Type>> canonical; const hidl_vec<Type>& arguments) {
std::vector<unvalidatedConvertOutput<Type>> canonical;
canonical.reserve(arguments.size()); canonical.reserve(arguments.size());
for (const auto& argument : arguments) { for (const auto& argument : arguments) {
canonical.push_back(NN_TRY(nn::convert(argument))); canonical.push_back(NN_TRY(nn::unvalidatedConvert(argument)));
} }
return canonical; return canonical;
} }
template <typename Type> template <typename Type>
GeneralResult<std::vector<ConvertOutput<Type>>> convert(const hidl_vec<Type>& arguments) { GeneralResult<std::vector<unvalidatedConvertOutput<Type>>> unvalidatedConvert(
return convertVec(arguments); const hidl_vec<Type>& arguments) {
return unvalidatedConvertVec(arguments);
}
template <typename Type>
decltype(nn::unvalidatedConvert(std::declval<Type>())) validatedConvert(const Type& halObject) {
auto canonical = NN_TRY(nn::unvalidatedConvert(halObject));
const auto maybeVersion = validate(canonical);
if (!maybeVersion.has_value()) {
return error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required " << kVersion;
}
return canonical;
}
template <typename Type>
GeneralResult<std::vector<unvalidatedConvertOutput<Type>>> validatedConvert(
const hidl_vec<Type>& arguments) {
std::vector<unvalidatedConvertOutput<Type>> canonical;
canonical.reserve(arguments.size());
for (const auto& argument : arguments) {
canonical.push_back(NN_TRY(validatedConvert(argument)));
}
return canonical;
} }
} // anonymous namespace } // anonymous namespace
GeneralResult<OperandType> convert(const hal::V1_3::OperandType& operandType) { GeneralResult<OperandType> unvalidatedConvert(const hal::V1_3::OperandType& operandType) {
return static_cast<OperandType>(operandType); return static_cast<OperandType>(operandType);
} }
GeneralResult<OperationType> convert(const hal::V1_3::OperationType& operationType) { GeneralResult<OperationType> unvalidatedConvert(const hal::V1_3::OperationType& operationType) {
return static_cast<OperationType>(operationType); return static_cast<OperationType>(operationType);
} }
GeneralResult<Priority> convert(const hal::V1_3::Priority& priority) { GeneralResult<Priority> unvalidatedConvert(const hal::V1_3::Priority& priority) {
return static_cast<Priority>(priority); return static_cast<Priority>(priority);
} }
GeneralResult<Capabilities> convert(const hal::V1_3::Capabilities& capabilities) { GeneralResult<Capabilities> unvalidatedConvert(const hal::V1_3::Capabilities& capabilities) {
const bool validOperandTypes = std::all_of( const bool validOperandTypes = std::all_of(
capabilities.operandPerformance.begin(), capabilities.operandPerformance.end(), capabilities.operandPerformance.begin(), capabilities.operandPerformance.end(),
[](const hal::V1_3::Capabilities::OperandPerformance& operandPerformance) { [](const hal::V1_3::Capabilities::OperandPerformance& operandPerformance) {
const auto maybeType = convert(operandPerformance.type); const auto maybeType = unvalidatedConvert(operandPerformance.type);
return !maybeType.has_value() ? false : validOperandType(maybeType.value()); return !maybeType.has_value() ? false : validOperandType(maybeType.value());
}); });
if (!validOperandTypes) { if (!validOperandTypes) {
return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
<< "Invalid OperandType when converting OperandPerformance in Capabilities"; << "Invalid OperandType when unvalidatedConverting OperandPerformance in "
"Capabilities";
} }
auto operandPerformance = NN_TRY(convert(capabilities.operandPerformance)); auto operandPerformance = NN_TRY(unvalidatedConvert(capabilities.operandPerformance));
auto table = NN_TRY(hal::utils::makeGeneralFailure( auto table = NN_TRY(hal::utils::makeGeneralFailure(
Capabilities::OperandPerformanceTable::create(std::move(operandPerformance)), Capabilities::OperandPerformanceTable::create(std::move(operandPerformance)),
nn::ErrorStatus::GENERAL_FAILURE)); nn::ErrorStatus::GENERAL_FAILURE));
return Capabilities{ return Capabilities{
.relaxedFloat32toFloat16PerformanceScalar = .relaxedFloat32toFloat16PerformanceScalar = NN_TRY(
NN_TRY(convert(capabilities.relaxedFloat32toFloat16PerformanceScalar)), unvalidatedConvert(capabilities.relaxedFloat32toFloat16PerformanceScalar)),
.relaxedFloat32toFloat16PerformanceTensor = .relaxedFloat32toFloat16PerformanceTensor = NN_TRY(
NN_TRY(convert(capabilities.relaxedFloat32toFloat16PerformanceTensor)), unvalidatedConvert(capabilities.relaxedFloat32toFloat16PerformanceTensor)),
.operandPerformance = std::move(table), .operandPerformance = std::move(table),
.ifPerformance = NN_TRY(convert(capabilities.ifPerformance)), .ifPerformance = NN_TRY(unvalidatedConvert(capabilities.ifPerformance)),
.whilePerformance = NN_TRY(convert(capabilities.whilePerformance)), .whilePerformance = NN_TRY(unvalidatedConvert(capabilities.whilePerformance)),
}; };
} }
GeneralResult<Capabilities::OperandPerformance> convert( GeneralResult<Capabilities::OperandPerformance> unvalidatedConvert(
const hal::V1_3::Capabilities::OperandPerformance& operandPerformance) { const hal::V1_3::Capabilities::OperandPerformance& operandPerformance) {
return Capabilities::OperandPerformance{ return Capabilities::OperandPerformance{
.type = NN_TRY(convert(operandPerformance.type)), .type = NN_TRY(unvalidatedConvert(operandPerformance.type)),
.info = NN_TRY(convert(operandPerformance.info)), .info = NN_TRY(unvalidatedConvert(operandPerformance.info)),
}; };
} }
GeneralResult<Operation> convert(const hal::V1_3::Operation& operation) { GeneralResult<Operation> unvalidatedConvert(const hal::V1_3::Operation& operation) {
return Operation{ return Operation{
.type = NN_TRY(convert(operation.type)), .type = NN_TRY(unvalidatedConvert(operation.type)),
.inputs = operation.inputs, .inputs = operation.inputs,
.outputs = operation.outputs, .outputs = operation.outputs,
}; };
} }
GeneralResult<Operand::LifeTime> convert(const hal::V1_3::OperandLifeTime& operandLifeTime) { GeneralResult<Operand::LifeTime> unvalidatedConvert(
const hal::V1_3::OperandLifeTime& operandLifeTime) {
return static_cast<Operand::LifeTime>(operandLifeTime); return static_cast<Operand::LifeTime>(operandLifeTime);
} }
GeneralResult<Operand> convert(const hal::V1_3::Operand& operand) { GeneralResult<Operand> unvalidatedConvert(const hal::V1_3::Operand& operand) {
return Operand{ return Operand{
.type = NN_TRY(convert(operand.type)), .type = NN_TRY(unvalidatedConvert(operand.type)),
.dimensions = operand.dimensions, .dimensions = operand.dimensions,
.scale = operand.scale, .scale = operand.scale,
.zeroPoint = operand.zeroPoint, .zeroPoint = operand.zeroPoint,
.lifetime = NN_TRY(convert(operand.lifetime)), .lifetime = NN_TRY(unvalidatedConvert(operand.lifetime)),
.location = NN_TRY(convert(operand.location)), .location = NN_TRY(unvalidatedConvert(operand.location)),
.extraParams = NN_TRY(convert(operand.extraParams)), .extraParams = NN_TRY(unvalidatedConvert(operand.extraParams)),
}; };
} }
GeneralResult<Model> convert(const hal::V1_3::Model& model) { GeneralResult<Model> unvalidatedConvert(const hal::V1_3::Model& model) {
return Model{ return Model{
.main = NN_TRY(convert(model.main)), .main = NN_TRY(unvalidatedConvert(model.main)),
.referenced = NN_TRY(convert(model.referenced)), .referenced = NN_TRY(unvalidatedConvert(model.referenced)),
.operandValues = NN_TRY(convert(model.operandValues)), .operandValues = NN_TRY(unvalidatedConvert(model.operandValues)),
.pools = NN_TRY(convert(model.pools)), .pools = NN_TRY(unvalidatedConvert(model.pools)),
.relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16, .relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16,
.extensionNameToPrefix = NN_TRY(convert(model.extensionNameToPrefix)), .extensionNameToPrefix = NN_TRY(unvalidatedConvert(model.extensionNameToPrefix)),
}; };
} }
GeneralResult<Model::Subgraph> convert(const hal::V1_3::Subgraph& subgraph) { GeneralResult<Model::Subgraph> unvalidatedConvert(const hal::V1_3::Subgraph& subgraph) {
auto operations = NN_TRY(convert(subgraph.operations)); auto operations = NN_TRY(unvalidatedConvert(subgraph.operations));
// Verify number of consumers. // Verify number of consumers.
const auto numberOfConsumers = const auto numberOfConsumers =
@@ -196,18 +229,18 @@ GeneralResult<Model::Subgraph> convert(const hal::V1_3::Subgraph& subgraph) {
} }
return Model::Subgraph{ return Model::Subgraph{
.operands = NN_TRY(convert(subgraph.operands)), .operands = NN_TRY(unvalidatedConvert(subgraph.operands)),
.operations = std::move(operations), .operations = std::move(operations),
.inputIndexes = subgraph.inputIndexes, .inputIndexes = subgraph.inputIndexes,
.outputIndexes = subgraph.outputIndexes, .outputIndexes = subgraph.outputIndexes,
}; };
} }
GeneralResult<BufferDesc> convert(const hal::V1_3::BufferDesc& bufferDesc) { GeneralResult<BufferDesc> unvalidatedConvert(const hal::V1_3::BufferDesc& bufferDesc) {
return BufferDesc{.dimensions = bufferDesc.dimensions}; return BufferDesc{.dimensions = bufferDesc.dimensions};
} }
GeneralResult<BufferRole> convert(const hal::V1_3::BufferRole& bufferRole) { GeneralResult<BufferRole> unvalidatedConvert(const hal::V1_3::BufferRole& bufferRole) {
return BufferRole{ return BufferRole{
.modelIndex = bufferRole.modelIndex, .modelIndex = bufferRole.modelIndex,
.ioIndex = bufferRole.ioIndex, .ioIndex = bufferRole.ioIndex,
@@ -215,15 +248,16 @@ GeneralResult<BufferRole> convert(const hal::V1_3::BufferRole& bufferRole) {
}; };
} }
GeneralResult<Request> convert(const hal::V1_3::Request& request) { GeneralResult<Request> unvalidatedConvert(const hal::V1_3::Request& request) {
return Request{ return Request{
.inputs = NN_TRY(convert(request.inputs)), .inputs = NN_TRY(unvalidatedConvert(request.inputs)),
.outputs = NN_TRY(convert(request.outputs)), .outputs = NN_TRY(unvalidatedConvert(request.outputs)),
.pools = NN_TRY(convert(request.pools)), .pools = NN_TRY(unvalidatedConvert(request.pools)),
}; };
} }
GeneralResult<Request::MemoryPool> convert(const hal::V1_3::Request::MemoryPool& memoryPool) { GeneralResult<Request::MemoryPool> unvalidatedConvert(
const hal::V1_3::Request::MemoryPool& memoryPool) {
using Discriminator = hal::V1_3::Request::MemoryPool::hidl_discriminator; using Discriminator = hal::V1_3::Request::MemoryPool::hidl_discriminator;
switch (memoryPool.getDiscriminator()) { switch (memoryPool.getDiscriminator()) {
case Discriminator::hidlMemory: case Discriminator::hidlMemory:
@@ -236,12 +270,14 @@ GeneralResult<Request::MemoryPool> convert(const hal::V1_3::Request::MemoryPool&
<< underlyingType(memoryPool.getDiscriminator()); << underlyingType(memoryPool.getDiscriminator());
} }
GeneralResult<OptionalTimePoint> convert(const hal::V1_3::OptionalTimePoint& optionalTimePoint) { GeneralResult<OptionalTimePoint> unvalidatedConvert(
const hal::V1_3::OptionalTimePoint& optionalTimePoint) {
constexpr auto kTimePointMaxCount = TimePoint::max().time_since_epoch().count(); constexpr auto kTimePointMaxCount = TimePoint::max().time_since_epoch().count();
const auto makeTimePoint = [](uint64_t count) -> GeneralResult<OptionalTimePoint> { const auto makeTimePoint = [](uint64_t count) -> GeneralResult<OptionalTimePoint> {
if (count > kTimePointMaxCount) { if (count > kTimePointMaxCount) {
return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
<< "Unable to convert OptionalTimePoint because the count exceeds the max"; << "Unable to unvalidatedConvert OptionalTimePoint because the count exceeds "
"the max";
} }
const auto nanoseconds = std::chrono::nanoseconds{count}; const auto nanoseconds = std::chrono::nanoseconds{count};
return TimePoint{nanoseconds}; return TimePoint{nanoseconds};
@@ -259,13 +295,14 @@ GeneralResult<OptionalTimePoint> convert(const hal::V1_3::OptionalTimePoint& opt
<< underlyingType(optionalTimePoint.getDiscriminator()); << underlyingType(optionalTimePoint.getDiscriminator());
} }
GeneralResult<OptionalTimeoutDuration> convert( GeneralResult<OptionalTimeoutDuration> unvalidatedConvert(
const hal::V1_3::OptionalTimeoutDuration& optionalTimeoutDuration) { const hal::V1_3::OptionalTimeoutDuration& optionalTimeoutDuration) {
constexpr auto kTimeoutDurationMaxCount = TimeoutDuration::max().count(); constexpr auto kTimeoutDurationMaxCount = TimeoutDuration::max().count();
const auto makeTimeoutDuration = [](uint64_t count) -> GeneralResult<OptionalTimeoutDuration> { const auto makeTimeoutDuration = [](uint64_t count) -> GeneralResult<OptionalTimeoutDuration> {
if (count > kTimeoutDurationMaxCount) { if (count > kTimeoutDurationMaxCount) {
return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
<< "Unable to convert OptionalTimeoutDuration because the count exceeds the max"; << "Unable to unvalidatedConvert OptionalTimeoutDuration because the count "
"exceeds the max";
} }
return TimeoutDuration{count}; return TimeoutDuration{count};
}; };
@@ -282,7 +319,7 @@ GeneralResult<OptionalTimeoutDuration> convert(
<< underlyingType(optionalTimeoutDuration.getDiscriminator()); << underlyingType(optionalTimeoutDuration.getDiscriminator());
} }
GeneralResult<ErrorStatus> convert(const hal::V1_3::ErrorStatus& status) { GeneralResult<ErrorStatus> unvalidatedConvert(const hal::V1_3::ErrorStatus& status) {
switch (status) { switch (status) {
case hal::V1_3::ErrorStatus::NONE: case hal::V1_3::ErrorStatus::NONE:
case hal::V1_3::ErrorStatus::DEVICE_UNAVAILABLE: case hal::V1_3::ErrorStatus::DEVICE_UNAVAILABLE:
@@ -299,9 +336,50 @@ GeneralResult<ErrorStatus> convert(const hal::V1_3::ErrorStatus& status) {
<< "Invalid ErrorStatus " << underlyingType(status); << "Invalid ErrorStatus " << underlyingType(status);
} }
GeneralResult<Priority> convert(const hal::V1_3::Priority& priority) {
return validatedConvert(priority);
}
GeneralResult<Capabilities> convert(const hal::V1_3::Capabilities& capabilities) {
return validatedConvert(capabilities);
}
GeneralResult<Model> convert(const hal::V1_3::Model& model) {
return validatedConvert(model);
}
GeneralResult<BufferDesc> convert(const hal::V1_3::BufferDesc& bufferDesc) {
return validatedConvert(bufferDesc);
}
GeneralResult<Request> convert(const hal::V1_3::Request& request) {
return validatedConvert(request);
}
GeneralResult<OptionalTimePoint> convert(const hal::V1_3::OptionalTimePoint& optionalTimePoint) {
return validatedConvert(optionalTimePoint);
}
GeneralResult<OptionalTimeoutDuration> convert(
const hal::V1_3::OptionalTimeoutDuration& optionalTimeoutDuration) {
return validatedConvert(optionalTimeoutDuration);
}
GeneralResult<ErrorStatus> convert(const hal::V1_3::ErrorStatus& errorStatus) {
return validatedConvert(errorStatus);
}
GeneralResult<SharedHandle> convert(const hardware::hidl_handle& handle) {
return validatedConvert(handle);
}
GeneralResult<Memory> convert(const hardware::hidl_memory& memory) {
return validatedConvert(memory);
}
GeneralResult<std::vector<BufferRole>> convert( GeneralResult<std::vector<BufferRole>> convert(
const hardware::hidl_vec<hal::V1_3::BufferRole>& bufferRoles) { const hardware::hidl_vec<hal::V1_3::BufferRole>& bufferRoles) {
return convertVec(bufferRoles); return validatedConvert(bufferRoles);
} }
} // namespace android::nn } // namespace android::nn
@@ -309,58 +387,67 @@ GeneralResult<std::vector<BufferRole>> convert(
namespace android::hardware::neuralnetworks::V1_3::utils { namespace android::hardware::neuralnetworks::V1_3::utils {
namespace { namespace {
using utils::convert; using utils::unvalidatedConvert;
nn::GeneralResult<V1_0::PerformanceInfo> convert( nn::GeneralResult<V1_0::PerformanceInfo> unvalidatedConvert(
const nn::Capabilities::PerformanceInfo& performanceInfo) { const nn::Capabilities::PerformanceInfo& performanceInfo) {
return V1_0::utils::convert(performanceInfo); return V1_0::utils::unvalidatedConvert(performanceInfo);
} }
nn::GeneralResult<V1_0::DataLocation> convert(const nn::DataLocation& dataLocation) { nn::GeneralResult<V1_0::DataLocation> unvalidatedConvert(const nn::DataLocation& dataLocation) {
return V1_0::utils::convert(dataLocation); return V1_0::utils::unvalidatedConvert(dataLocation);
} }
nn::GeneralResult<hidl_vec<uint8_t>> convert(const nn::Model::OperandValues& operandValues) { nn::GeneralResult<hidl_vec<uint8_t>> unvalidatedConvert(
return V1_0::utils::convert(operandValues); const nn::Model::OperandValues& operandValues) {
return V1_0::utils::unvalidatedConvert(operandValues);
} }
nn::GeneralResult<hidl_memory> convert(const nn::Memory& memory) { nn::GeneralResult<hidl_handle> unvalidatedConvert(const nn::SharedHandle& handle) {
return V1_0::utils::convert(memory); return V1_2::utils::unvalidatedConvert(handle);
} }
nn::GeneralResult<V1_0::RequestArgument> convert(const nn::Request::Argument& argument) { nn::GeneralResult<hidl_memory> unvalidatedConvert(const nn::Memory& memory) {
return V1_0::utils::convert(argument); return V1_0::utils::unvalidatedConvert(memory);
} }
nn::GeneralResult<V1_2::Operand::ExtraParams> convert(const nn::Operand::ExtraParams& extraParams) { nn::GeneralResult<V1_0::RequestArgument> unvalidatedConvert(const nn::Request::Argument& argument) {
return V1_2::utils::convert(extraParams); return V1_0::utils::unvalidatedConvert(argument);
} }
nn::GeneralResult<V1_2::Model::ExtensionNameAndPrefix> convert( nn::GeneralResult<V1_2::Operand::ExtraParams> unvalidatedConvert(
const nn::Operand::ExtraParams& extraParams) {
return V1_2::utils::unvalidatedConvert(extraParams);
}
nn::GeneralResult<V1_2::Model::ExtensionNameAndPrefix> unvalidatedConvert(
const nn::Model::ExtensionNameAndPrefix& extensionNameAndPrefix) { const nn::Model::ExtensionNameAndPrefix& extensionNameAndPrefix) {
return V1_2::utils::convert(extensionNameAndPrefix); return V1_2::utils::unvalidatedConvert(extensionNameAndPrefix);
} }
template <typename Input> template <typename Input>
using ConvertOutput = std::decay_t<decltype(convert(std::declval<Input>()).value())>; using unvalidatedConvertOutput =
std::decay_t<decltype(unvalidatedConvert(std::declval<Input>()).value())>;
template <typename Type> template <typename Type>
nn::GeneralResult<hidl_vec<ConvertOutput<Type>>> convertVec(const std::vector<Type>& arguments) { nn::GeneralResult<hidl_vec<unvalidatedConvertOutput<Type>>> unvalidatedConvertVec(
hidl_vec<ConvertOutput<Type>> halObject(arguments.size()); const std::vector<Type>& arguments) {
hidl_vec<unvalidatedConvertOutput<Type>> halObject(arguments.size());
for (size_t i = 0; i < arguments.size(); ++i) { for (size_t i = 0; i < arguments.size(); ++i) {
halObject[i] = NN_TRY(convert(arguments[i])); halObject[i] = NN_TRY(unvalidatedConvert(arguments[i]));
} }
return halObject; return halObject;
} }
template <typename Type> template <typename Type>
nn::GeneralResult<hidl_vec<ConvertOutput<Type>>> convert(const std::vector<Type>& arguments) { nn::GeneralResult<hidl_vec<unvalidatedConvertOutput<Type>>> unvalidatedConvert(
return convertVec(arguments); const std::vector<Type>& arguments) {
return unvalidatedConvertVec(arguments);
} }
nn::GeneralResult<Request::MemoryPool> makeMemoryPool(const nn::Memory& memory) { nn::GeneralResult<Request::MemoryPool> makeMemoryPool(const nn::Memory& memory) {
Request::MemoryPool ret; Request::MemoryPool ret;
ret.hidlMemory(NN_TRY(convert(memory))); ret.hidlMemory(NN_TRY(unvalidatedConvert(memory)));
return ret; return ret;
} }
@@ -374,21 +461,46 @@ nn::GeneralResult<Request::MemoryPool> makeMemoryPool(const nn::SharedBuffer& /*
return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) << "Unable to make memory pool from IBuffer"; return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) << "Unable to make memory pool from IBuffer";
} }
using utils::unvalidatedConvert;
template <typename Type>
decltype(unvalidatedConvert(std::declval<Type>())) validatedConvert(const Type& canonical) {
const auto maybeVersion = nn::validate(canonical);
if (!maybeVersion.has_value()) {
return nn::error() << maybeVersion.error();
}
const auto version = maybeVersion.value();
if (version > kVersion) {
return NN_ERROR() << "Insufficient version: " << version << " vs required " << kVersion;
}
return unvalidatedConvert(canonical);
}
template <typename Type>
nn::GeneralResult<hidl_vec<unvalidatedConvertOutput<Type>>> validatedConvert(
const std::vector<Type>& arguments) {
hidl_vec<unvalidatedConvertOutput<Type>> halObject(arguments.size());
for (size_t i = 0; i < arguments.size(); ++i) {
halObject[i] = NN_TRY(validatedConvert(arguments[i]));
}
return halObject;
}
} // anonymous namespace } // anonymous namespace
nn::GeneralResult<OperandType> convert(const nn::OperandType& operandType) { nn::GeneralResult<OperandType> unvalidatedConvert(const nn::OperandType& operandType) {
return static_cast<OperandType>(operandType); return static_cast<OperandType>(operandType);
} }
nn::GeneralResult<OperationType> convert(const nn::OperationType& operationType) { nn::GeneralResult<OperationType> unvalidatedConvert(const nn::OperationType& operationType) {
return static_cast<OperationType>(operationType); return static_cast<OperationType>(operationType);
} }
nn::GeneralResult<Priority> convert(const nn::Priority& priority) { nn::GeneralResult<Priority> unvalidatedConvert(const nn::Priority& priority) {
return static_cast<Priority>(priority); return static_cast<Priority>(priority);
} }
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities) { nn::GeneralResult<Capabilities> unvalidatedConvert(const nn::Capabilities& capabilities) {
std::vector<nn::Capabilities::OperandPerformance> operandPerformance; std::vector<nn::Capabilities::OperandPerformance> operandPerformance;
operandPerformance.reserve(capabilities.operandPerformance.asVector().size()); operandPerformance.reserve(capabilities.operandPerformance.asVector().size());
std::copy_if(capabilities.operandPerformance.asVector().begin(), std::copy_if(capabilities.operandPerformance.asVector().begin(),
@@ -399,71 +511,72 @@ nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities) {
}); });
return Capabilities{ return Capabilities{
.relaxedFloat32toFloat16PerformanceScalar = .relaxedFloat32toFloat16PerformanceScalar = NN_TRY(
NN_TRY(convert(capabilities.relaxedFloat32toFloat16PerformanceScalar)), unvalidatedConvert(capabilities.relaxedFloat32toFloat16PerformanceScalar)),
.relaxedFloat32toFloat16PerformanceTensor = .relaxedFloat32toFloat16PerformanceTensor = NN_TRY(
NN_TRY(convert(capabilities.relaxedFloat32toFloat16PerformanceTensor)), unvalidatedConvert(capabilities.relaxedFloat32toFloat16PerformanceTensor)),
.operandPerformance = NN_TRY(convert(operandPerformance)), .operandPerformance = NN_TRY(unvalidatedConvert(operandPerformance)),
.ifPerformance = NN_TRY(convert(capabilities.ifPerformance)), .ifPerformance = NN_TRY(unvalidatedConvert(capabilities.ifPerformance)),
.whilePerformance = NN_TRY(convert(capabilities.whilePerformance)), .whilePerformance = NN_TRY(unvalidatedConvert(capabilities.whilePerformance)),
}; };
} }
nn::GeneralResult<Capabilities::OperandPerformance> convert( nn::GeneralResult<Capabilities::OperandPerformance> unvalidatedConvert(
const nn::Capabilities::OperandPerformance& operandPerformance) { const nn::Capabilities::OperandPerformance& operandPerformance) {
return Capabilities::OperandPerformance{ return Capabilities::OperandPerformance{
.type = NN_TRY(convert(operandPerformance.type)), .type = NN_TRY(unvalidatedConvert(operandPerformance.type)),
.info = NN_TRY(convert(operandPerformance.info)), .info = NN_TRY(unvalidatedConvert(operandPerformance.info)),
}; };
} }
nn::GeneralResult<Operation> convert(const nn::Operation& operation) { nn::GeneralResult<Operation> unvalidatedConvert(const nn::Operation& operation) {
return Operation{ return Operation{
.type = NN_TRY(convert(operation.type)), .type = NN_TRY(unvalidatedConvert(operation.type)),
.inputs = operation.inputs, .inputs = operation.inputs,
.outputs = operation.outputs, .outputs = operation.outputs,
}; };
} }
nn::GeneralResult<OperandLifeTime> convert(const nn::Operand::LifeTime& operandLifeTime) { nn::GeneralResult<OperandLifeTime> unvalidatedConvert(
const nn::Operand::LifeTime& operandLifeTime) {
if (operandLifeTime == nn::Operand::LifeTime::POINTER) { if (operandLifeTime == nn::Operand::LifeTime::POINTER) {
return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT) return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT)
<< "Model cannot be converted because it contains pointer-based memory"; << "Model cannot be unvalidatedConverted because it contains pointer-based memory";
} }
return static_cast<OperandLifeTime>(operandLifeTime); return static_cast<OperandLifeTime>(operandLifeTime);
} }
nn::GeneralResult<Operand> convert(const nn::Operand& operand) { nn::GeneralResult<Operand> unvalidatedConvert(const nn::Operand& operand) {
return Operand{ return Operand{
.type = NN_TRY(convert(operand.type)), .type = NN_TRY(unvalidatedConvert(operand.type)),
.dimensions = operand.dimensions, .dimensions = operand.dimensions,
.numberOfConsumers = 0, .numberOfConsumers = 0,
.scale = operand.scale, .scale = operand.scale,
.zeroPoint = operand.zeroPoint, .zeroPoint = operand.zeroPoint,
.lifetime = NN_TRY(convert(operand.lifetime)), .lifetime = NN_TRY(unvalidatedConvert(operand.lifetime)),
.location = NN_TRY(convert(operand.location)), .location = NN_TRY(unvalidatedConvert(operand.location)),
.extraParams = NN_TRY(convert(operand.extraParams)), .extraParams = NN_TRY(unvalidatedConvert(operand.extraParams)),
}; };
} }
nn::GeneralResult<Model> convert(const nn::Model& model) { nn::GeneralResult<Model> unvalidatedConvert(const nn::Model& model) {
if (!hal::utils::hasNoPointerData(model)) { if (!hal::utils::hasNoPointerData(model)) {
return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT) return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT)
<< "Model cannot be converted because it contains pointer-based memory"; << "Model cannot be unvalidatedConverted because it contains pointer-based memory";
} }
return Model{ return Model{
.main = NN_TRY(convert(model.main)), .main = NN_TRY(unvalidatedConvert(model.main)),
.referenced = NN_TRY(convert(model.referenced)), .referenced = NN_TRY(unvalidatedConvert(model.referenced)),
.operandValues = NN_TRY(convert(model.operandValues)), .operandValues = NN_TRY(unvalidatedConvert(model.operandValues)),
.pools = NN_TRY(convert(model.pools)), .pools = NN_TRY(unvalidatedConvert(model.pools)),
.relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16, .relaxComputationFloat32toFloat16 = model.relaxComputationFloat32toFloat16,
.extensionNameToPrefix = NN_TRY(convert(model.extensionNameToPrefix)), .extensionNameToPrefix = NN_TRY(unvalidatedConvert(model.extensionNameToPrefix)),
}; };
} }
nn::GeneralResult<Subgraph> convert(const nn::Model::Subgraph& subgraph) { nn::GeneralResult<Subgraph> unvalidatedConvert(const nn::Model::Subgraph& subgraph) {
auto operands = NN_TRY(convert(subgraph.operands)); auto operands = NN_TRY(unvalidatedConvert(subgraph.operands));
// Update number of consumers. // Update number of consumers.
const auto numberOfConsumers = const auto numberOfConsumers =
@@ -475,17 +588,17 @@ nn::GeneralResult<Subgraph> convert(const nn::Model::Subgraph& subgraph) {
return Subgraph{ return Subgraph{
.operands = std::move(operands), .operands = std::move(operands),
.operations = NN_TRY(convert(subgraph.operations)), .operations = NN_TRY(unvalidatedConvert(subgraph.operations)),
.inputIndexes = subgraph.inputIndexes, .inputIndexes = subgraph.inputIndexes,
.outputIndexes = subgraph.outputIndexes, .outputIndexes = subgraph.outputIndexes,
}; };
} }
nn::GeneralResult<BufferDesc> convert(const nn::BufferDesc& bufferDesc) { nn::GeneralResult<BufferDesc> unvalidatedConvert(const nn::BufferDesc& bufferDesc) {
return BufferDesc{.dimensions = bufferDesc.dimensions}; return BufferDesc{.dimensions = bufferDesc.dimensions};
} }
nn::GeneralResult<BufferRole> convert(const nn::BufferRole& bufferRole) { nn::GeneralResult<BufferRole> unvalidatedConvert(const nn::BufferRole& bufferRole) {
return BufferRole{ return BufferRole{
.modelIndex = bufferRole.modelIndex, .modelIndex = bufferRole.modelIndex,
.ioIndex = bufferRole.ioIndex, .ioIndex = bufferRole.ioIndex,
@@ -493,30 +606,33 @@ nn::GeneralResult<BufferRole> convert(const nn::BufferRole& bufferRole) {
}; };
} }
nn::GeneralResult<Request> convert(const nn::Request& request) { nn::GeneralResult<Request> unvalidatedConvert(const nn::Request& request) {
if (!hal::utils::hasNoPointerData(request)) { if (!hal::utils::hasNoPointerData(request)) {
return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT) return NN_ERROR(nn::ErrorStatus::INVALID_ARGUMENT)
<< "Request cannot be converted because it contains pointer-based memory"; << "Request cannot be unvalidatedConverted because it contains pointer-based memory";
} }
return Request{ return Request{
.inputs = NN_TRY(convert(request.inputs)), .inputs = NN_TRY(unvalidatedConvert(request.inputs)),
.outputs = NN_TRY(convert(request.outputs)), .outputs = NN_TRY(unvalidatedConvert(request.outputs)),
.pools = NN_TRY(convert(request.pools)), .pools = NN_TRY(unvalidatedConvert(request.pools)),
}; };
} }
nn::GeneralResult<Request::MemoryPool> convert(const nn::Request::MemoryPool& memoryPool) { nn::GeneralResult<Request::MemoryPool> unvalidatedConvert(
const nn::Request::MemoryPool& memoryPool) {
return std::visit([](const auto& o) { return makeMemoryPool(o); }, memoryPool); return std::visit([](const auto& o) { return makeMemoryPool(o); }, memoryPool);
} }
nn::GeneralResult<OptionalTimePoint> convert(const nn::OptionalTimePoint& optionalTimePoint) { nn::GeneralResult<OptionalTimePoint> unvalidatedConvert(
const nn::OptionalTimePoint& optionalTimePoint) {
OptionalTimePoint ret; OptionalTimePoint ret;
if (optionalTimePoint.has_value()) { if (optionalTimePoint.has_value()) {
const auto count = optionalTimePoint.value().time_since_epoch().count(); const auto count = optionalTimePoint.value().time_since_epoch().count();
if (count < 0) { if (count < 0) {
return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
<< "Unable to convert OptionalTimePoint because time since epoch count is " << "Unable to unvalidatedConvert OptionalTimePoint because time since epoch "
"count is "
"negative"; "negative";
} }
ret.nanosecondsSinceEpoch(count); ret.nanosecondsSinceEpoch(count);
@@ -524,21 +640,22 @@ nn::GeneralResult<OptionalTimePoint> convert(const nn::OptionalTimePoint& option
return ret; return ret;
} }
nn::GeneralResult<OptionalTimeoutDuration> convert( nn::GeneralResult<OptionalTimeoutDuration> unvalidatedConvert(
const nn::OptionalTimeoutDuration& optionalTimeoutDuration) { const nn::OptionalTimeoutDuration& optionalTimeoutDuration) {
OptionalTimeoutDuration ret; OptionalTimeoutDuration ret;
if (optionalTimeoutDuration.has_value()) { if (optionalTimeoutDuration.has_value()) {
const auto count = optionalTimeoutDuration.value().count(); const auto count = optionalTimeoutDuration.value().count();
if (count < 0) { if (count < 0) {
return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) return NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE)
<< "Unable to convert OptionalTimeoutDuration because count is negative"; << "Unable to unvalidatedConvert OptionalTimeoutDuration because count is "
"negative";
} }
ret.nanoseconds(count); ret.nanoseconds(count);
} }
return ret; return ret;
} }
nn::GeneralResult<ErrorStatus> convert(const nn::ErrorStatus& errorStatus) { nn::GeneralResult<ErrorStatus> unvalidatedConvert(const nn::ErrorStatus& errorStatus) {
switch (errorStatus) { switch (errorStatus) {
case nn::ErrorStatus::NONE: case nn::ErrorStatus::NONE:
case nn::ErrorStatus::DEVICE_UNAVAILABLE: case nn::ErrorStatus::DEVICE_UNAVAILABLE:
@@ -555,8 +672,49 @@ nn::GeneralResult<ErrorStatus> convert(const nn::ErrorStatus& errorStatus) {
} }
} }
nn::GeneralResult<Priority> convert(const nn::Priority& priority) {
return validatedConvert(priority);
}
nn::GeneralResult<Capabilities> convert(const nn::Capabilities& capabilities) {
return validatedConvert(capabilities);
}
nn::GeneralResult<Model> convert(const nn::Model& model) {
return validatedConvert(model);
}
nn::GeneralResult<BufferDesc> convert(const nn::BufferDesc& bufferDesc) {
return validatedConvert(bufferDesc);
}
nn::GeneralResult<Request> convert(const nn::Request& request) {
return validatedConvert(request);
}
nn::GeneralResult<OptionalTimePoint> convert(const nn::OptionalTimePoint& optionalTimePoint) {
return validatedConvert(optionalTimePoint);
}
nn::GeneralResult<OptionalTimeoutDuration> convert(
const nn::OptionalTimeoutDuration& optionalTimeoutDuration) {
return validatedConvert(optionalTimeoutDuration);
}
nn::GeneralResult<ErrorStatus> convert(const nn::ErrorStatus& errorStatus) {
return validatedConvert(errorStatus);
}
nn::GeneralResult<hidl_handle> convert(const nn::SharedHandle& handle) {
return validatedConvert(handle);
}
nn::GeneralResult<hidl_memory> convert(const nn::Memory& memory) {
return validatedConvert(memory);
}
nn::GeneralResult<hidl_vec<BufferRole>> convert(const std::vector<nn::BufferRole>& bufferRoles) { nn::GeneralResult<hidl_vec<BufferRole>> convert(const std::vector<nn::BufferRole>& bufferRoles) {
return convertVec(bufferRoles); return validatedConvert(bufferRoles);
} }
} // namespace android::hardware::neuralnetworks::V1_3::utils } // namespace android::hardware::neuralnetworks::V1_3::utils

View File

@@ -78,11 +78,10 @@ nn::GeneralResult<nn::Capabilities> initCapabilities(V1_3::IDevice* device) {
<< "uninitialized"; << "uninitialized";
const auto cb = [&result](ErrorStatus status, const Capabilities& capabilities) { const auto cb = [&result](ErrorStatus status, const Capabilities& capabilities) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "getCapabilities_1_3 failed with " << toString(status); result = NN_ERROR(canonical) << "getCapabilities_1_3 failed with " << toString(status);
} else { } else {
result = validatedConvertToCanonical(capabilities); result = nn::convert(capabilities);
} }
}; };
@@ -178,8 +177,7 @@ nn::GeneralResult<std::vector<bool>> Device::getSupportedOperations(const nn::Mo
<< "uninitialized"; << "uninitialized";
auto cb = [&result, &model](ErrorStatus status, const hidl_vec<bool>& supportedOperations) { auto cb = [&result, &model](ErrorStatus status, const hidl_vec<bool>& supportedOperations) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) result = NN_ERROR(canonical)
<< "IDevice::getSupportedOperations_1_3 failed with " << toString(status); << "IDevice::getSupportedOperations_1_3 failed with " << toString(status);
} else if (supportedOperations.size() != model.main.operations.size()) { } else if (supportedOperations.size() != model.main.operations.size()) {
@@ -223,8 +221,7 @@ nn::GeneralResult<nn::SharedPreparedModel> Device::prepareModel(
hidlModelCache, hidlDataCache, hidlToken, cb); hidlModelCache, hidlDataCache, hidlToken, cb);
const auto status = NN_TRY(hal::utils::handleTransportError(ret)); const auto status = NN_TRY(hal::utils::handleTransportError(ret));
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "prepareModel_1_3 failed with " << toString(status); return NN_ERROR(canonical) << "prepareModel_1_3 failed with " << toString(status);
} }
@@ -246,8 +243,7 @@ nn::GeneralResult<nn::SharedPreparedModel> Device::prepareModelFromCache(
hidlToken, cb); hidlToken, cb);
const auto status = NN_TRY(hal::utils::handleTransportError(ret)); const auto status = NN_TRY(hal::utils::handleTransportError(ret));
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "prepareModelFromCache_1_3 failed with " << toString(status); return NN_ERROR(canonical) << "prepareModelFromCache_1_3 failed with " << toString(status);
} }
@@ -267,8 +263,7 @@ nn::GeneralResult<nn::SharedBuffer> Device::allocate(
<< "uninitialized"; << "uninitialized";
auto cb = [&result](ErrorStatus status, const sp<IBuffer>& buffer, uint32_t token) { auto cb = [&result](ErrorStatus status, const sp<IBuffer>& buffer, uint32_t token) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "IDevice::allocate failed with " << toString(status); result = NN_ERROR(canonical) << "IDevice::allocate failed with " << toString(status);
} else if (buffer == nullptr) { } else if (buffer == nullptr) {
result = NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) << "Returned buffer is nullptr"; result = NN_ERROR(nn::ErrorStatus::GENERAL_FAILURE) << "Returned buffer is nullptr";

View File

@@ -27,6 +27,7 @@
#include <android/hardware/neuralnetworks/1.3/types.h> #include <android/hardware/neuralnetworks/1.3/types.h>
#include <nnapi/IPreparedModel.h> #include <nnapi/IPreparedModel.h>
#include <nnapi/Result.h> #include <nnapi/Result.h>
#include <nnapi/TypeUtils.h>
#include <nnapi/Types.h> #include <nnapi/Types.h>
#include <nnapi/hal/1.2/Conversions.h> #include <nnapi/hal/1.2/Conversions.h>
#include <nnapi/hal/CommonUtils.h> #include <nnapi/hal/CommonUtils.h>
@@ -44,8 +45,7 @@ namespace {
nn::GeneralResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> nn::GeneralResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>>
convertExecutionResultsHelper(const hidl_vec<V1_2::OutputShape>& outputShapes, convertExecutionResultsHelper(const hidl_vec<V1_2::OutputShape>& outputShapes,
const V1_2::Timing& timing) { const V1_2::Timing& timing) {
return std::make_pair(NN_TRY(validatedConvertToCanonical(outputShapes)), return std::make_pair(NN_TRY(nn::convert(outputShapes)), NN_TRY(nn::convert(timing)));
NN_TRY(validatedConvertToCanonical(timing)));
} }
nn::ExecutionResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> convertExecutionResults( nn::ExecutionResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> convertExecutionResults(
@@ -55,8 +55,7 @@ nn::ExecutionResult<std::pair<std::vector<nn::OutputShape>, nn::Timing>> convert
nn::GeneralResult<std::pair<nn::Timing, nn::Timing>> convertFencedExecutionCallbackResults( nn::GeneralResult<std::pair<nn::Timing, nn::Timing>> convertFencedExecutionCallbackResults(
const V1_2::Timing& timingLaunched, const V1_2::Timing& timingFenced) { const V1_2::Timing& timingLaunched, const V1_2::Timing& timingFenced) {
return std::make_pair(NN_TRY(validatedConvertToCanonical(timingLaunched)), return std::make_pair(NN_TRY(nn::convert(timingLaunched)), NN_TRY(nn::convert(timingFenced)));
NN_TRY(validatedConvertToCanonical(timingFenced)));
} }
nn::GeneralResult<std::pair<nn::SyncFence, nn::ExecuteFencedInfoCallback>> nn::GeneralResult<std::pair<nn::SyncFence, nn::ExecuteFencedInfoCallback>>
@@ -64,9 +63,9 @@ convertExecuteFencedResults(const hidl_handle& syncFence,
const sp<IFencedExecutionCallback>& callback) { const sp<IFencedExecutionCallback>& callback) {
auto resultSyncFence = nn::SyncFence::createAsSignaled(); auto resultSyncFence = nn::SyncFence::createAsSignaled();
if (syncFence.getNativeHandle() != nullptr) { if (syncFence.getNativeHandle() != nullptr) {
auto nativeHandle = NN_TRY(validatedConvertToCanonical(syncFence)); auto sharedHandle = NN_TRY(nn::convert(syncFence));
resultSyncFence = NN_TRY(hal::utils::makeGeneralFailure( resultSyncFence = NN_TRY(hal::utils::makeGeneralFailure(
nn::SyncFence::create(std::move(nativeHandle)), nn::ErrorStatus::GENERAL_FAILURE)); nn::SyncFence::create(std::move(sharedHandle)), nn::ErrorStatus::GENERAL_FAILURE));
} }
if (callback == nullptr) { if (callback == nullptr) {
@@ -81,8 +80,8 @@ convertExecuteFencedResults(const hidl_handle& syncFence,
auto cb = [&result](ErrorStatus status, const V1_2::Timing& timingLaunched, auto cb = [&result](ErrorStatus status, const V1_2::Timing& timingLaunched,
const V1_2::Timing& timingFenced) { const V1_2::Timing& timingFenced) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = validatedConvertToCanonical(status).value_or( const auto canonical =
nn::ErrorStatus::GENERAL_FAILURE); nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "getExecutionInfo failed with " << toString(status); result = NN_ERROR(canonical) << "getExecutionInfo failed with " << toString(status);
} else { } else {
result = convertFencedExecutionCallbackResults(timingLaunched, timingFenced); result = convertFencedExecutionCallbackResults(timingLaunched, timingFenced);
@@ -125,8 +124,7 @@ PreparedModel::executeSynchronously(const Request& request, V1_2::MeasureTiming
const auto cb = [&result](ErrorStatus status, const hidl_vec<V1_2::OutputShape>& outputShapes, const auto cb = [&result](ErrorStatus status, const hidl_vec<V1_2::OutputShape>& outputShapes,
const V1_2::Timing& timing) { const V1_2::Timing& timing) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "executeSynchronously failed with " << toString(status); result = NN_ERROR(canonical) << "executeSynchronously failed with " << toString(status);
} else { } else {
result = convertExecutionResults(outputShapes, timing); result = convertExecutionResults(outputShapes, timing);
@@ -152,8 +150,7 @@ PreparedModel::executeAsynchronously(const Request& request, V1_2::MeasureTiming
const auto status = const auto status =
NN_TRY(hal::utils::makeExecutionFailure(hal::utils::handleTransportError(ret))); NN_TRY(hal::utils::makeExecutionFailure(hal::utils::handleTransportError(ret)));
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
return NN_ERROR(canonical) << "executeAsynchronously failed with " << toString(status); return NN_ERROR(canonical) << "executeAsynchronously failed with " << toString(status);
} }
@@ -223,8 +220,7 @@ PreparedModel::executeFenced(const nn::Request& request, const std::vector<nn::S
auto cb = [&result](ErrorStatus status, const hidl_handle& syncFence, auto cb = [&result](ErrorStatus status, const hidl_handle& syncFence,
const sp<IFencedExecutionCallback>& callback) { const sp<IFencedExecutionCallback>& callback) {
if (status != ErrorStatus::NONE) { if (status != ErrorStatus::NONE) {
const auto canonical = const auto canonical = nn::convert(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
validatedConvertToCanonical(status).value_or(nn::ErrorStatus::GENERAL_FAILURE);
result = NN_ERROR(canonical) << "executeFenced failed with " << toString(status); result = NN_ERROR(canonical) << "executeFenced failed with " << toString(status);
} else { } else {
result = convertExecuteFencedResults(syncFence, callback); result = convertExecuteFencedResults(syncFence, callback);

View File

@@ -0,0 +1,50 @@
# NNAPI Conversions
`convert` fails if either the source type or the destination type is invalid, and it yields a valid
object if the conversion succeeds. For example, let's say that an enumeration in the current
version has fewer possible values than the "same" canonical enumeration, such as `OperationType`.
The new value of `HARD_SWISH` (introduced in Android R / NN HAL 1.3) does not map to any valid
existing value in `OperationType`, but an older value of `ADD` (introduced in Android OC-MR1 / NN
HAL 1.0) is valid. This can be seen in the following model conversions:
```cpp
// Unsuccessful conversion
const nn::Model canonicalModel = createModelWhichHasV1_3Operations();
const nn::Result<V1_0::Model> maybeVersionedModel = V1_0::utils::convert(canonicalModel);
EXPECT_FALSE(maybeVersionedModel.has_value());
```
```cpp
// Successful conversion
const nn::Model canonicalModel = createModelWhichHasOnlyV1_0Operations();
const nn::Result<V1_0::Model> maybeVersionedModel = V1_0::utils::convert(canonicalModel);
ASSERT_TRUE(maybeVersionedModel.has_value());
const V1_0::Model& versionedModel = maybeVersionedModel.value();
EXPECT_TRUE(V1_0::utils::valid(versionedModel));
```
`V1_X::utils::convert` does not guarantee that all information is preserved. For example, In the
case of `nn::ErrorStatus`, the new value of `MISSED_DEADLINE_TRANSIENT` can be represented by the
existing value of `V1_0::GENERAL_FAILURE`:
```cpp
// Lossy Canonical -> HAL -> Canonical conversion
const nn::ErrorStatus canonicalBefore = nn::ErrorStatus::MISSED_DEADLINE_TRANSIENT;
const V1_0::ErrorStatus versioned = V1_0::utils::convert(canonicalBefore).value();
const nn::ErrorStatus canonicalAfter = nn::convert(versioned).value();
EXPECT_NE(canonicalBefore, canonicalAfter);
```
However, `nn::convert` is guaranteed to preserve all information:
```cpp
// Lossless HAL -> Canonical -> HAL conversion
const V1_0::ErrorStatus versionedBefore = V1_0::ErrorStatus::GENERAL_FAILURE;
const nn::ErrorStatus canonical = nn::convert(versionedBefore).value();
const V1_0::ErrorStatus versionedAfter = V1_0::utils::convert(canonical).value();
EXPECT_EQ(versionedBefore, versionedAfter);
```
The `convert` functions operate only on types that used in a HIDL method call directly. The
`unvalidatedConvert` functions operate on types that are either used in a HIDL method call directly
(i.e., not as a nested class) or used in a subsequent version of the NN HAL. Prefer using `convert`
over `unvalidatedConvert`.