Blender Git Loki

Git Commits -> Revision 8537cdc

November 5, 2019, 14:00 (GMT)
Core XR Support [part 3]: Ghost-XR API based on OpenXR

## Design Overview
* For code using this API, the most important object is a GHOST_XrContext handle. Through it, all API functions and internal state can be accessed/modified.
* Main responsibilities of the Ghost XR-context are to manage lifetimes of the OpenXR runtime connection (represented by XrInstance), the session and to delegate operations/data to the session.
* The OpenXR related graphics code, which is OS dependent, is managed through a `GHOST_IXrGraphicsBinding` interface, that can be implemented for the different graphics libraries supported (currently OpenGL and DirectX).
* Much of this code here has to follow the OpenXR specification and is based on the OpenXR [[https://github.com/KhronosGroup/OpenXR-SDK-Source/tree/master/src/tests/hello_xr | `hello_xr`]] implentation.
* In future we may want to take some code out of the context, e.g. extension and API layer management.
* There are runtime debugging and benchmarking options (exposed through --debug-xr and --debug-xr-time, but not as part of this patch).
* Error handling is described in a section below.

## Why have this in Ghost?

Early on, I decided to do the OpenXR level access through GHOST. Main reasons:
* OpenXR requires access to low level, OS dependent graphics lib data (e.g. see [[https://www.khronos.org/registry/OpenXR/specs/0.90/man/html/openxr.html#XrGraphicsBindingOpenGLXlibKHR| XrGraphicsBindingOpenGLXlibKHR]])
* Some C++ features appeared handy (`std::vector`, RAII + exception handling, cleaner code through object methods, etc.)
* General low level nature of the OpenXR API

After all I think much of the functionality is too high level to live in GHOST however. I would like to address this by having a separate `VAMR` (virtual + augmented + mixed reality) module, placed in `intern/`. The main issue is getting this to work well with Ghost data, especially how to get the mentioned low level data out of Ghost.
This is something I'd like to look into again before too long, but for now I think having this in Ghost is reasonable.

## Error Handling Strategy

The error handling strategy I chose uses C++ exceptions, a controversial feature. Let me explain why I think this is reasonable here.

The strategy requirements were:
* If an error occurs, cleanly exit the VR session (or destroy the entire context), causing no resource leaks or side effects to the rest of Blender.
* Show a *useful* error message to the user.
* Don't impair readability of code too much with error handling.

Here's why I chose an exception based strategy:
* Most alternatives require early exiting functions. This early exiting has to be 'bubbled up' the call stack to the point that performs error handling. For safe code, early exit checks have to be performed everywhere and code gets really impaired by error checking. Tried this first and wasn't happy at all. Even if error handling is wrapped into macros.
* All `GHOST_Xr` resources are managed via RAII. So stack unwinding will cause them to be released cleanly whenever an exception is thrown.
* `GHOST_Xr` has a clear boundary (the Ghost C-API) with only a handful of public functions. That is the only place we need to have try-catch blocks at. (Generally, try-catch blocks at kinda random places are a bad code smell IMHO. Module boundaries are a valid place to put them.)
* Exceptions allow us to pass multiple bits of error information through mulitple layers of the call stack. This information can also be made specific with a useful error message. As of now, they conain a user error message, the OpenXR error code (if any), as well as the exact source code location the error was caught at.

So the strategy I went with works as follows:
* If a VR related error occurs within `GHOST_Xr`, throw an exception (`GHOST_XrException` currently).
* OpenXR calls are wrapped into a macro throwing an exception if the return value indicates an error.
* Useful debugging information and user messages are stored in the exceptions.
* All resources must be managed through RAII, so throwing an exception will release 'dangling' ones cleanly.
* In the GHOST C-API wrappers, the exceptions are caught and contained error information is forwarded to a custom error handling callback.
* The error handling callback is set in `wm_xr.c`, prior to creating the XR-Context, and implements clean destruction of the context.

Differential Revision: https://developer.blender.org/D6192

Commit Details:

Full Hash: 8537cdc71d8145572a06eabc7ec39030b3c99118
Parent Commit: 6ae2979
Lines Changed: +2330, -15

12 Added Paths:

/intern/ghost/GHOST_IXrContext.h (+42, -0) (View)
/intern/ghost/intern/GHOST_IXRGraphicsBinding.h (+71, -0) (View)
/intern/ghost/intern/GHOST_XR.cpp (+64, -0) (View)
/intern/ghost/intern/GHOST_XrContext.cpp (+550, -0) (View)
/intern/ghost/intern/GHOST_XrContext.h (+125, -0) (View)
/intern/ghost/intern/GHOST_XREvent.cpp (+62, -0) (View)
/intern/ghost/intern/GHOST_XrException.h (+47, -0) (View)
/intern/ghost/intern/GHOST_XRGraphicsBinding.cpp (+316, -0) (View)
/intern/ghost/intern/GHOST_XRSession.cpp (+565, -0) (View)
/intern/ghost/intern/GHOST_XrSession.h (+78, -0) (View)
/intern/ghost/intern/GHOST_XR_intern.h (+132, -0) (View)
/intern/ghost/intern/GHOST_XR_openxr_includes.h (+52, -0) (View)

8 Modified Paths:

/intern/ghost/CMakeLists.txt (+29, -0) (Diff)
/intern/ghost/GHOST_C-api.h (+30, -15) (Diff)
/intern/ghost/GHOST_Types.h (+93, -0) (Diff)
/intern/ghost/intern/GHOST_C-api.cpp (+64, -0) (Diff)
/intern/ghost/intern/GHOST_ContextD3D.h (+3, -0) (Diff)
/intern/ghost/intern/GHOST_ContextGLX.cpp (+1, -0) (Diff)
/intern/ghost/intern/GHOST_ContextGLX.h (+3, -0) (Diff)
/intern/ghost/intern/GHOST_ContextWGL.h (+3, -0) (Diff)
Tehnyt: Miika HämäläinenViimeksi päivitetty: 07.11.2014 14:18MiikaH:n Sivut a.k.a. MiikaHweb | 2003-2021