Skip to content

Commit 5337e74

Browse files
committed
OpenXR loader: add API layer discovery support.
1 parent 9e9d023 commit 5337e74

7 files changed

+553
-60
lines changed

src/common/platform_utils.hpp

+11
Original file line numberDiff line numberDiff line change
@@ -324,6 +324,7 @@ static inline std::string PlatformUtilsGetSecureEnv(const char* name) {
324324
}
325325

326326
#elif defined(XR_OS_ANDROID)
327+
#include <sys/system_properties.h>
327328

328329
static inline bool PlatformUtilsGetEnvSet(const char* /* name */) {
329330
// Stub func
@@ -340,6 +341,16 @@ static inline std::string PlatformUtilsGetSecureEnv(const char* /* name */) {
340341
return {};
341342
}
342343

344+
static inline bool PlatformUtilsGetSysProp(const char* name, bool default_value) {
345+
bool result = default_value;
346+
char value[PROP_VALUE_MAX] = {};
347+
if (__system_property_get(name, value) != 0) {
348+
result = (value[0] == 't');
349+
}
350+
351+
return result;
352+
}
353+
343354
// Intended to be only used as a fallback on Android, with a more open, "native" technique used in most cases
344355
static inline bool PlatformGetGlobalRuntimeFileName(uint16_t major_version, std::string& file_name) {
345356
// Prefix for the runtime JSON file name

src/loader/android_utilities.cpp

+322-54
Large diffs are not rendered by default.

src/loader/android_utilities.h

+14-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,20 @@ using wrap::android::content::Context;
2626
*
2727
* @return 0 on success, something else on failure.
2828
*/
29-
int getActiveRuntimeVirtualManifest(wrap::android::content::Context const &context, Json::Value &virtualManifest);
29+
int getActiveRuntimeVirtualManifest(wrap::android::content::Context const &context, Json::Value &virtualManifest,
30+
bool &systemBroker);
31+
32+
/*!
33+
* Find the implicit/explicit API layers on the system, and return a constructed JSON object representing it.
34+
*
35+
* @param type An String to indicate layer type of API layers, implicit or explicit.
36+
* @param context An Android context, preferably an Activity Context.
37+
* @param[out] virtualManifest The Json::Value to fill with the virtual manifest.
38+
*
39+
* @return 0 on success, something else on failure.
40+
*/
41+
int getApiLayerVirtualManifests(std::string layerType, wrap::android::content::Context const &context,
42+
std::vector<Json::Value> &virtualManifests, bool systemBroker);
3043
} // namespace openxr_android
3144

3245
#endif // __ANDROID__

src/loader/loader_init_data.hpp

+3-1
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
#include <json/value.h>
1717
#include <android/asset_manager_jni.h>
1818
#include "android_utilities.h"
19+
#include "manifest_file.hpp"
1920
#endif // XR_USE_PLATFORM_ANDROID
2021

2122
#ifdef XR_KHR_LOADER_INIT_SUPPORT
@@ -84,7 +85,8 @@ class LoaderInitData {
8485
XrResult InitializeLoaderInitData(const XrLoaderInitInfoBaseHeaderKHR* loaderInitInfo);
8586

8687
#ifdef XR_USE_PLATFORM_ANDROID
87-
XrResult GetPlatformRuntimeVirtualManifest(Json::Value& out_manifest);
88+
XrResult GetPlatformRuntimeVirtualManifest(Json::Value& out_manifest, ManifestFileSource& runtime_source);
89+
XrResult GetPlatformApiLayerVirtualManifests(std::string type, std::vector<Json::Value>& out_manifest, bool system_broker);
8890
std::string GetAndroidNativeLibraryDir();
8991
void* Android_Get_Asset_Manager();
9092
#endif // XR_USE_PLATFORM_ANDROID

src/loader/manifest_file.cpp

+168-2
Original file line numberDiff line numberDiff line change
@@ -682,7 +682,8 @@ XrResult RuntimeManifestFile::FindManifestFiles(const std::string &openxr_comman
682682

683683
#if defined(XR_KHR_LOADER_INIT_SUPPORT) && defined(XR_USE_PLATFORM_ANDROID)
684684
Json::Value virtualManifest;
685-
result = GetPlatformRuntimeVirtualManifest(virtualManifest);
685+
ManifestFileSource runtimeSource = ManifestFileSource::FROM_JSON_MANIFEST;
686+
result = GetPlatformRuntimeVirtualManifest(virtualManifest, runtimeSource);
686687
if (XR_SUCCESS == result) {
687688
RuntimeManifestFile::CreateIfValid(virtualManifest, "", manifest_files);
688689
return result;
@@ -779,6 +780,134 @@ void ApiLayerManifestFile::AddManifestFilesAndroid(const std::string &openxr_com
779780
}
780781
#endif // defined(XR_USE_PLATFORM_ANDROID) && defined(XR_KHR_LOADER_INIT_SUPPORT)
781782

783+
void ApiLayerManifestFile::CreateIfValid(ManifestFileType type, const Json::Value &root_node, const std::string &filename,
784+
std::vector<std::unique_ptr<ApiLayerManifestFile>> &manifest_files) {
785+
std::ostringstream error_ss("ApiLayerManifestFile::CreateIfValid ");
786+
JsonVersion file_version = {};
787+
if (!ManifestFile::IsValidJson(root_node, file_version)) {
788+
error_ss << "isValidJson indicates " << filename << " is not a valid manifest file.";
789+
LoaderLogger::LogErrorMessage("", error_ss.str());
790+
return;
791+
}
792+
793+
Json::Value layer_root_node = root_node["api_layer"];
794+
795+
// The API Layer manifest file needs the "api_layer" root as well as other sub-nodes.
796+
// If any of those aren't there, fail.
797+
if (layer_root_node.isNull() || layer_root_node["name"].isNull() || !layer_root_node["name"].isString() ||
798+
layer_root_node["api_version"].isNull() || !layer_root_node["api_version"].isString() ||
799+
layer_root_node["library_path"].isNull() || !layer_root_node["library_path"].isString() ||
800+
layer_root_node["implementation_version"].isNull() || !layer_root_node["implementation_version"].isString()) {
801+
error_ss << filename << " is missing required fields. Verify all proper fields exist.";
802+
LoaderLogger::LogErrorMessage("", error_ss.str());
803+
return;
804+
}
805+
if (MANIFEST_TYPE_IMPLICIT_API_LAYER == type) {
806+
bool enabled = true;
807+
// Implicit layers require the disable environment variable.
808+
if (layer_root_node["disable_environment"].isNull() || !layer_root_node["disable_environment"].isString()) {
809+
error_ss << "Implicit layer " << filename << " is missing \"disable_environment\"";
810+
LoaderLogger::LogErrorMessage("", error_ss.str());
811+
return;
812+
}
813+
// Check if there's an enable environment variable provided
814+
if (!layer_root_node["enable_environment"].isNull() && layer_root_node["enable_environment"].isString()) {
815+
std::string env_var = layer_root_node["enable_environment"].asString();
816+
// If it's not set in the environment, disable the layer
817+
if (!PlatformUtilsGetEnvSet(env_var.c_str())) {
818+
enabled = false;
819+
}
820+
}
821+
// Check for the disable environment variable, which must be provided in the JSON
822+
std::string env_var = layer_root_node["disable_environment"].asString();
823+
// If the env var is set, disable the layer. Disable env var overrides enable above
824+
if (PlatformUtilsGetEnvSet(env_var.c_str())) {
825+
enabled = false;
826+
}
827+
828+
#if defined(XR_OS_ANDROID)
829+
// Check if there's an system property to enable this API layer
830+
if (!layer_root_node["enable_sys_prop"].isNull() && layer_root_node["enable_sys_prop"].isString()) {
831+
std::string enable_sys_prop = layer_root_node["enable_sys_prop"].asString();
832+
// If it's not set to true, disable this layer
833+
if (!PlatformUtilsGetSysProp(enable_sys_prop.c_str(), true)) {
834+
enabled = false;
835+
}
836+
}
837+
838+
std::string disable_sys_prop = layer_root_node["disable_sys_prop"].asString();
839+
if (PlatformUtilsGetSysProp(disable_sys_prop.c_str(), false)) {
840+
enabled = false;
841+
}
842+
#endif
843+
844+
// Not enabled, so pretend like it isn't even there.
845+
if (!enabled) {
846+
error_ss << "Implicit layer " << filename << " is disabled";
847+
LoaderLogger::LogInfoMessage("", error_ss.str());
848+
return;
849+
}
850+
}
851+
std::string layer_name = layer_root_node["name"].asString();
852+
std::string api_version_string = layer_root_node["api_version"].asString();
853+
JsonVersion api_version = {};
854+
const int num_fields = sscanf(api_version_string.c_str(), "%u.%u", &api_version.major, &api_version.minor);
855+
api_version.patch = 0;
856+
857+
if ((num_fields != 2) || (api_version.major == 0 && api_version.minor == 0) ||
858+
api_version.major > XR_VERSION_MAJOR(XR_CURRENT_API_VERSION)) {
859+
error_ss << "layer " << filename << " has invalid API Version. Skipping layer.";
860+
LoaderLogger::LogWarningMessage("", error_ss.str());
861+
return;
862+
}
863+
864+
char *end_ptr;
865+
uint32_t implementation_version = strtol(layer_root_node["implementation_version"].asString().c_str(), &end_ptr, 10);
866+
if (*end_ptr != '\0') {
867+
error_ss << "layer " << filename << " has invalid implementation version.";
868+
LoaderLogger::LogWarningMessage("", error_ss.str());
869+
}
870+
871+
std::string library_path = layer_root_node["library_path"].asString();
872+
873+
// If the library_path variable has no directory symbol, it's just a file name and should be accessible on the
874+
// global library path.
875+
if (library_path.find('\\') != std::string::npos || library_path.find('/') != std::string::npos) {
876+
// If the library_path is an absolute path, just use that if it exists
877+
if (FileSysUtilsIsAbsolutePath(library_path)) {
878+
if (!FileSysUtilsPathExists(library_path)) {
879+
error_ss << filename << " library " << library_path << " does not appear to exist";
880+
LoaderLogger::LogErrorMessage("", error_ss.str());
881+
return;
882+
}
883+
} else {
884+
// Otherwise, treat the library path as a relative path based on the JSON file.
885+
std::string combined_path;
886+
std::string file_parent;
887+
if (!FileSysUtilsGetParentPath(filename, file_parent) ||
888+
!FileSysUtilsCombinePaths(file_parent, library_path, combined_path) || !FileSysUtilsPathExists(combined_path)) {
889+
error_ss << filename << " library " << combined_path << " does not appear to exist";
890+
LoaderLogger::LogErrorMessage("", error_ss.str());
891+
return;
892+
}
893+
library_path = combined_path;
894+
}
895+
}
896+
897+
std::string description;
898+
if (!layer_root_node["description"].isNull() && layer_root_node["description"].isString()) {
899+
description = layer_root_node["description"].asString();
900+
}
901+
902+
// Add this layer manifest file
903+
manifest_files.emplace_back(
904+
new ApiLayerManifestFile(type, filename, layer_name, description, api_version, implementation_version, library_path));
905+
906+
// Add any extensions to it after the fact.
907+
// Handle any renamed functions
908+
manifest_files.back()->ParseCommon(layer_root_node);
909+
}
910+
782911
void ApiLayerManifestFile::CreateIfValid(ManifestFileType type, const std::string &filename, std::istream &json_stream,
783912
LibraryLocator locate_library,
784913
std::vector<std::unique_ptr<ApiLayerManifestFile>> &manifest_files) {
@@ -892,6 +1021,7 @@ void ApiLayerManifestFile::CreateIfValid(ManifestFileType type, const std::strin
8921021
new ApiLayerManifestFile(type, filename, layer_name, description, api_version, implementation_version, library_path));
8931022

8941023
// Add any extensions to it after the fact.
1024+
// Handle any renamed functions
8951025
manifest_files.back()->ParseCommon(layer_root_node);
8961026
}
8971027

@@ -951,6 +1081,24 @@ void ApiLayerManifestFile::PopulateApiLayerProperties(XrApiLayerProperties &prop
9511081
// Find all layer manifest files in the appropriate search paths/registries for the given type.
9521082
XrResult ApiLayerManifestFile::FindManifestFiles(const std::string &openxr_command, ManifestFileType type,
9531083
std::vector<std::unique_ptr<ApiLayerManifestFile>> &manifest_files) {
1084+
bool search_json_layer = true;
1085+
bool search_broker_layer = true;
1086+
1087+
#if defined(XR_KHR_LOADER_INIT_SUPPORT) && defined(XR_USE_PLATFORM_ANDROID)
1088+
Json::Value virtualManifest;
1089+
bool systemBroker = true;
1090+
ManifestFileSource runtimeSource = ManifestFileSource::FROM_JSON_MANIFEST;
1091+
XrResult result = GetPlatformRuntimeVirtualManifest(virtualManifest, runtimeSource);
1092+
if (XR_SUCCESS == result) {
1093+
if (runtimeSource == ManifestFileSource::FROM_INSTALLABLE_BROKER) {
1094+
systemBroker = false;
1095+
search_json_layer = false;
1096+
}
1097+
} else {
1098+
search_broker_layer = false;
1099+
}
1100+
#endif // defined(XR_USE_PLATFORM_ANDROID) && defined(XR_KHR_LOADER_INIT_SUPPORT)
1101+
9541102
std::string relative_path;
9551103
std::string override_env_var;
9561104
std::string registry_location;
@@ -983,7 +1131,9 @@ XrResult ApiLayerManifestFile::FindManifestFiles(const std::string &openxr_comma
9831131

9841132
bool override_active = false;
9851133
std::vector<std::string> filenames;
986-
ReadDataFilesInSearchPaths(override_env_var, relative_path, override_active, filenames);
1134+
if (search_json_layer) {
1135+
ReadDataFilesInSearchPaths(override_env_var, relative_path, override_active, filenames);
1136+
}
9871137

9881138
#ifdef XR_OS_WINDOWS
9891139
// Read the registry if the override wasn't active.
@@ -997,6 +1147,22 @@ XrResult ApiLayerManifestFile::FindManifestFiles(const std::string &openxr_comma
9971147
}
9981148

9991149
#if defined(XR_KHR_LOADER_INIT_SUPPORT) && defined(XR_USE_PLATFORM_ANDROID)
1150+
if (search_broker_layer) {
1151+
std::vector<Json::Value> virtualManifests;
1152+
std::string layerType = (type == ManifestFileType::MANIFEST_TYPE_IMPLICIT_API_LAYER) ? "implicit" : "explicit";
1153+
result = GetPlatformApiLayerVirtualManifests(layerType, virtualManifests, systemBroker);
1154+
if (XR_SUCCESS == result) {
1155+
for (const auto &virtualManifest : virtualManifests) {
1156+
ApiLayerManifestFile::CreateIfValid(type, virtualManifest, "virtual manifest", manifest_files);
1157+
}
1158+
} else {
1159+
LoaderLogger::LogErrorMessage(
1160+
"",
1161+
"ApiLayerManifestFile::FindManifestFiles - failed to get virtual manifest files from system/installable broker.");
1162+
assert(0);
1163+
}
1164+
}
1165+
10001166
ApiLayerManifestFile::AddManifestFilesAndroid(openxr_command, type, manifest_files);
10011167
#endif // defined(XR_USE_PLATFORM_ANDROID) && defined(XR_KHR_LOADER_INIT_SUPPORT)
10021168

src/loader/manifest_file.hpp

+8
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,12 @@ enum ManifestFileType {
2828
MANIFEST_TYPE_EXPLICIT_API_LAYER,
2929
};
3030

31+
enum ManifestFileSource {
32+
FROM_JSON_MANIFEST = 0,
33+
FROM_INSTALLABLE_BROKER,
34+
FROM_SYSTEM_BROKER,
35+
};
36+
3137
struct JsonVersion {
3238
uint32_t major;
3339
uint32_t minor;
@@ -101,6 +107,8 @@ class ApiLayerManifestFile : public ManifestFile {
101107

102108
static void CreateIfValid(ManifestFileType type, const std::string &filename, std::istream &json_stream,
103109
LibraryLocator locate_library, std::vector<std::unique_ptr<ApiLayerManifestFile>> &manifest_files);
110+
static void CreateIfValid(ManifestFileType type, const Json::Value &root_node, const std::string &filename,
111+
std::vector<std::unique_ptr<ApiLayerManifestFile>> &manifest_files);
104112
static void CreateIfValid(ManifestFileType type, const std::string &filename,
105113
std::vector<std::unique_ptr<ApiLayerManifestFile>> &manifest_files);
106114
/// @return false if we could not find the library.

src/loader/runtime_interface.cpp

+27-2
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
#endif // XR_USE_PLATFORM_ANDROID
3636

3737
#if defined(XR_KHR_LOADER_INIT_SUPPORT) && defined(XR_USE_PLATFORM_ANDROID)
38-
XrResult GetPlatformRuntimeVirtualManifest(Json::Value& out_manifest) {
38+
XrResult GetPlatformRuntimeVirtualManifest(Json::Value& out_manifest, ManifestFileSource& runtime_source) {
3939
using wrap::android::content::Context;
4040
auto& initData = LoaderInitData::instance();
4141
if (!initData.initialized()) {
@@ -45,13 +45,38 @@ XrResult GetPlatformRuntimeVirtualManifest(Json::Value& out_manifest) {
4545
if (context.isNull()) {
4646
return XR_ERROR_INITIALIZATION_FAILED;
4747
}
48+
bool systemBroker = false;
4849
Json::Value virtualManifest;
49-
if (0 != openxr_android::getActiveRuntimeVirtualManifest(context, virtualManifest)) {
50+
if (0 != openxr_android::getActiveRuntimeVirtualManifest(context, virtualManifest, systemBroker)) {
51+
runtime_source = ManifestFileSource::FROM_JSON_MANIFEST;
5052
return XR_ERROR_INITIALIZATION_FAILED;
5153
}
54+
if (systemBroker) {
55+
runtime_source = ManifestFileSource::FROM_SYSTEM_BROKER;
56+
} else {
57+
runtime_source = ManifestFileSource::FROM_INSTALLABLE_BROKER;
58+
}
5259
out_manifest = virtualManifest;
5360
return XR_SUCCESS;
5461
}
62+
63+
XrResult GetPlatformApiLayerVirtualManifests(std::string type, std::vector<Json::Value>& out_manifest, bool system_broker) {
64+
using wrap::android::content::Context;
65+
auto& initData = LoaderInitData::instance();
66+
if (!initData.initialized()) {
67+
return XR_ERROR_INITIALIZATION_FAILED;
68+
}
69+
auto context = Context(reinterpret_cast<jobject>(initData.getData().applicationContext));
70+
if (context.isNull()) {
71+
return XR_ERROR_INITIALIZATION_FAILED;
72+
}
73+
std::vector<Json::Value> virtualManifests;
74+
if (0 != openxr_android::getApiLayerVirtualManifests(type, context, virtualManifests, system_broker)) {
75+
return XR_ERROR_INITIALIZATION_FAILED;
76+
}
77+
out_manifest = virtualManifests;
78+
return XR_SUCCESS;
79+
}
5580
#endif // defined(XR_USE_PLATFORM_ANDROID) && defined(XR_KHR_LOADER_INIT_SUPPORT)
5681

5782
XrResult RuntimeInterface::TryLoadingSingleRuntime(const std::string& openxr_command,

0 commit comments

Comments
 (0)