icd: Rename icd-bil.h to icd-spv.h

fix cube demo for .spv filenames

Revert "misc: Explicitly run python for generated code"

This reverts commit 10e987d610d43b37556df9b12349f9ffc8cb30c4.

misc: Explicitly run python for generated code
On windows if multiple python versions installed, then command line args
for python scripts fail.  This works around python issue

layers: Added fence tracking for command buffer completion

Cleaned up MemTracker fence tracking for command buffer completions,
fixed several issues with local fences.

layers: Reduce unmeaningful error messages for memtracker

It is a common case for a command buffer not to have a fence associated
with it -- no reason to call these cases out as errors.

layers: MemTracker add check to verify CB complete prior to Reset

If CB fence has not completed on call to Begin or Reset CB, flag validation error.

Various improvements to make sure the CB fence is getting reset as appropriate if the fence completes or is destroyed.

layers: Update DrawState to use generated struct size functions

Update struct size codegen to handle callback void* case and ppMemBarriers void** cases conservatively for now.

glave: Use generated code to get pAppInfo struct size

glave: Update more trace custom code to use generated struct sizing code instead

glave: Add codgen helper for struct size calculation and use for trace packet creation

glave: Codegen cleanup for tracing code to add/finalize ptrs in packet

glave: Simplify codegen for trace packet sizing

glave-generate: Replace asserts with proper error detection and error logging.

* On Linux debug builds, the asserts cause the debugger to abort, but the code wasn't handling the error case safely anyway, so even a release build would likely crash.
* These situations were being hit if trying to load a trace file from an earlier header version, but are likely to be conditions we'll see when new developers start to use the API.
* With these changes, the error is properly reported and error return values propogated so that the UI detects the invalid file and reports an error to the user.
* The new glv_LogError(..) messages get printed to the console, but are not yet passed to the UI. This will be done as part of a larger change sometime in the future.

glave: Fix createRenderpass replay restore of remapped FB

Win/C11: Use _align_malloc() on Windows, align_alloc() elsewhere.

layers: Improve PresentableImage memory handling in mem/objecttracker

Add handling to eliminate meaningless error reporting for the
xglWsiX11CreatePresentableImage memory resources.

Win: Fix warnings for parameter of getLayerOptionEnum()

Win: Fix a compiler warning in "icd/common"

Update xgl_helper to handle all of the dynamic arrays

glave: Remove unnecessary dot dump.

Win/glvdebug: Update the BUILD.md dependencies.

Update the BUILD.md dependencies in order to build Glave debugger/GUI on
Windows.

nulldrv: build it on Linux

It compiles!

nulldrv: fix gcc errors/warnings

The only functional change should be that _aligned_malloc() is replaced by
malloc().  It does not seem to matter.

icd: remove stale function declarations

Remove declarations of

  icdDbgRegisterMsgCallback()
  icdDbgUnregisterMsgCallback()
  icdDbgSetGlobalOption()

icd: remove unused icd-alloc.[ch]

nulldrv: remove icd-alloc.h dependency

icd: add support for per-instance DRM enumeration

Add icd_instance to DRM enumeration functions, and use it for memory
allocation or logging.

icd: add generic icd_instance

It provides memory allocation and logging support.

Win: Fix compilation errors.

layers: Fix Windows build issue

object-tracker: Do not track GPU object

The GPU object is not something that is created by the app
and thus object tracker layer does not have to track it.

layers: allocate enough space for pointer sprintf

memTracker: Use local GetProcAddr rather than auto generated one

Clean up pass through  functions.  Generated GetProcAddr  requires drawState
to implement all entrypoints so don't use this generated GPA.

drawState: Use local GetProcAddr rather than auto generated one

Clean up pass through  functions.  Generated GetProcAddr  requires drawState
to implement all entrypoints so don't use this generated GPA.

layers: if no settings file or call overrides make callback override loging

Default debugAction in layers is to log based on variable init. Make the
overrides work as follows:
1) app calls xglRegisterMsgCallback()
2) settings file
3) Glave (or app) calls to setlayerOption()

layers: Correctly set last bound PSO in DrawState layer

Win: Cannot use the "dot" functionality yet on Windows.

layers: Made mem_tracker thread-safe

Brought up to par with DrawState and ObjectTracker.

Turn on ability to trace on Windows with NVIDIA driver.

I can now trace the "tri" and "cube" demos.  Two changes were needed:

1) Must link the programs against XGL.dll instead of XGL.lib (the "XGL" library
   instead of "XGLstatic").

2) The glv_MessageStream_SetupHostSocket() function needed to "create" the
   lock/critical section that's used in the Glave interconnect code.

Add NOMINMAX and VC_EXTRALEAN to some windows files

This avoids these headers leaking conflicting Windows symbols to
non-Windows code.

Win/glave: Remove new compiler warnings on Windows.

Compiling the glvtrace_xgl64.dll library on Windows (see prior commit) resulted
in new warnings, some of which seemed serious (e.g. using "sizeof(void)", which
is 1 on Linux, with GCC, but is 0 on Windows).

Win/glave: Compile/link the glvtrace_xgl64.dll library on Windows.

Linux: Must use "__linux__" instead of "PLATFORM_LINUX".

There are some places (e.g. glave) where PLATFORM_LINUX is #define'd, but
normally we need to rely on "__linux__", which is #define'd by the compiler
(i.e. we can rely on it being there).

demos/cube: use a single layout

Conflicts:
demos/cube.c

Get the trace player to work with Windows

This change modifies many of the generators to use "#if
defined(PLATFORM_LINUX) || defined(XCB_NVIDIA)" to choose between using
XCB or WIN32 native.

xcb: Deal with case of getenv() returning NULL.

At least on Windows, getenv() returns NULL when the environment variable is not
found.  This causes a problem inside of "xcb_nvidia.cpp" when a C++ std::string
is assigned the return value of getenv().

Modify xcb_nvidia/nulldrv to also support nulldrv.

This allows LunarG and others without the NVIDIA XGL driver to use the nulldrv
on Windows.

Update tri and cube with stage

These changes were cherry-picked from nvidia-dev-20150218 and
KhronosGroup/GDC-stable.

Revert "misc: Add xglRecordMemoryRange entrypoint and use it in Glave"

This reverts commit fea3e5fc1ef2f3c9e4f06ab5a8d682f89b76377e.

layers: Update last global DS in DrawState when UpdateDescriptors is called

layers: DrawState Cmd Buffer tracking improvements

Add DbgMarker cmds to cmd buffer instruction linked list
Display last 10 touched CBs in dot file
Highlight last touched CB in dot file with green color

misc: Add xglRecordMemoryRange entrypoint and use it in Glave

glave: Set layer extension func pointers to NULL when the device is destroyed.

* This prevents invalid memory access if the pointers are referenced after the device is destroyed and before a new one is created.

layers: Issue endl at end of output

demos: Clean up renderpass and framebuffer

layers: In DrawState track last global pipe state and dump that to dot diagram

Merge ReplayThread change with Detach Replay Window.

glvdebug: Move QReplayWorker into a separate thread.

* In order to achieve this correctly, the QController could not inherit from the QReplayWorker; instead the controller now has a worker object. The QReplayWidget puts the worker on a separate thread.
* Any interaction between the QReplayWorker and the rest of the UI must be done through Qt's signals and slots, which knows how to pass data between threads.
* Performance of replaying within the UI is now on-par with replaying using glvreplay64 from the command line, based on the cube demo. Increasing the validation level will slow down the replay preformance (as expected), and printing of validation messages will also slow down replay performance (as expected).

layers: Fix null pointer reference in MemTracker

Copy-paste error caused segfault in Cube demo, among others.

glave: Add command buffer diagram.

layers: Remove spurious assert.

This was unnecessarily crashing the valve tests.

layers: Fix MemTracker memory leak

Call to string helper function was not freeing returned memory.

layers: Correctly represent Descriptor-related arrays in DrawState dot file

glave: cleanup mem_info list as it shrinks during tracing

layer: Fixed options settings for generated layers, fixed typos

Handling of non-zero default values now also fixed.

layers: Fix Memtracker fence-related memory leak

MemTracker created internal fence objects to locally track command buffer
completion, but these objects were never destroyed.

Win: Fix compilation error on Windows (xgl_helper.py).

Visual Studio doesn't like the following code construct:

    if (<some test>)
        return <something>;
    <type> variable [= <intial-value>];

We have to use curly-braces, as in:

    if (<some test>) {
        return <something>;
    }
    <type> variable [= <intial-value>];

Win: layers (draw_state.c): Fix compilation error on Windows.

Visual Studio doesn't like:

            static const uint32_t NUM_COLORS = 7;
            char* edgeColors[NUM_COLORS];

Instead, we need to do:

             char* edgeColors[NUM_COLORS];

layers: Track WsiCreatePresentableImage image object

The memory returned by WsiGetPresentableImage is unique
and cannot be freed or unbound by the app. Thus only object tracker
only needs to track the image object.

layers: Do not track PHYSICAL_GPUS

The application does not create PHYSICAL_GPU objects
so do not track them.

layers: Do not track Queue objects

Queue objects are not created by an application and
thus never destroyed. Thus do not need to track.

layers: flush logged output so it's visible immediately

demos: Clean up objects after cmd buf built

demos: Update to single layout

demos: Be sure to initialize pNext

demos: Bug fix: Unbind image object

demos: Be sure to request correct memory type

demos: Add use_staging option

Specifying --use_staging as a command line argument to tri
will force tri to upload the texture via a linear mapped staging buffer.

cube: Use staging buffer for texture upload

demos: Integrate feedback from NVIDIA

demos: Refactor to simplify and optimize

Will now check if texturing is supported on linear, CPU accessible
textures. If so, just uses that. If not, will use a staging buffer
to copy to optimized layout texture image.

layers: Check for NULL pointer with dynamic_display()

glave: Fix locking of mem_info entries in UnmapMemory

The retruned entry pointer could change after lock released in the case the mem info list was realloc.

layers: Fix DrawState buffer over-run when clearing descriptors

layers: Drawstate color descriptor edges and correctly clear slot mappings

layers: Fix layer config file options

Needed to end up with enums here, added this option.

layers: Validation check for CPU_VISIBLE bit set on MapMemory calls

layers: Fix uninitialized variable in DrawState.

update BUILD.md for imported libdrm

Two new dependencies introduced by libdrm: libpciaccess-dev and
libpthread-stubs0-dev.

glave: For color array params, correctly size trace buffer based on array type and size

cmake: remove unused FindDRM.cmake

intel: add and link with libintelkmd.a

Add cmake rules to build libintelkmd.a, which implements our KMD interface
using our own stripped-down libdrm.  This removes libdrm dependency.

demos/cube: use a single layout

demos: Updated tri to use staging buffer for texture

cube: Fix leaking resources

Demo was leaking Framebuffers and RenderPasses every time a new command
buffer was created (every frame). Now app creates only two command
buffers (one for each render target) and re-uses those command buffers
to draw the cube.

layers: Add simple Cmd Buffer dot file dump to DrawState

layers: For ParamChecker, verify struct ptrs aren't NULL before validating

layers: Remove destroyed/freed objects from both object lists

Some objects, upon being freed or destroyed, were removed from the
object-specific linked list but not from the master object list.

layers:  Remove the automatic dumping of DOT files.
This is now controlled solely by the app calling drawStateDumpDOTFile()

Conflicts:
layers/draw_state.c