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

glave: Add Dump() to debugger so DrawState dumping can occur on pauses

Conflicts:
glave-generate.py

glvdebug: Use svg instead of png.

Conflicts:
layers/draw_state.c

Conflicts:
layers/draw_state.c

layers: Add DrawState dot file output of descriptor details

layers: Fix buffer overrun in memtracker

Long function name+64-bit pointer caused buffer overrun in rendersystemtest

misc: Use XGL_API_VERSION instead of hard coded value

tri: Change from having a hard-coded API version to using the latest.

Piers actually found and fixed this, but I wanted to get it on "master" without
waiting for the rest of his changes.

layers: In DrawState correctly clear update chain for Struct Node

layers: Lock ObjectTracker linked-list access

layers: Add per-thread tracking of latest cmd buffer

layers: Don't run ObjectTracker checks on NULL mem objects

glave: Don't log error if replay is succesful but trace was not_ready

glave: Fix search of mem_info list  on UnmapMemory during tracing

glave: Improve logging for anomalous conditions around Map and Unmap memory

glave: Lock around access to the global mem_info list

glave: Fix bug in tracing CreateRenderPass with NULL attachments

layers: Multi-threading fixes for DrawState

layers: Disable DrawState dot file autodump and an incorrect validation check

layers: Fix DrawState bug to correctly shadow image/update structs

loader: Fix merge of layer names from env and CreateDevice

loader: initialize result for xglCreateInstance().

nulldrv: Return proper structure size for buffer object

This defect was cloned from the Intel driver; and so it gets the same fix:-).

layers: DrawState bug fixes

Correctly track UpdateChain per Set and free the shadowNode
Fix pTypes array to correctly track update types per descriptor
Fix upperBound code to correctly loop over all elements of layout
Various NULL ptr checks

layers: Fix memtracker reporting related to freeing presentable image memory

Output an error if presentable image memory is explicitly freed, and avoid
warnings if it is not.

loader: do not set RTLD_DEEPBIND

Instead, build layers with -Bsymbolic.  This should fix (or hide) a reported
crash with overriden malloc()/free().

loader: Correct layer path lookup name

layers: Add debug code to DrawState for alloc and free calls

layers: Fix DrawState bug in UPDATE image/buffer shadowing

layers: Add memory management to DrawState

Added code in DrawState to free memory allocations
Updated DrawState Layout to save full CreateInfo tree

layers: Correctly track pSetBindPoints per stage in DrawState layer

Win: Can now use "static inline" on Windows (i.e. no code mods).

loader: Windows now uses registry + env's (diff names)

The loader on Windows now looks in the Windows Registry + in environment
variables for the following (Note the new names--"LIB" was dropped):

- XGL_DRIVERS_PATH
- XGL_LAYERS_PATH
- XGL_LAYER_NAMES

Linux still retains the "LIB" at the start of the environment variable names.

If both are used, they are concatenated into a semi-colon-delimited list.

A generic loader_get_registry_and_env() func is used to perform this (only for
Windows, since Linux doesn't have a registry).

Win: Build Glave replay on Windows (REQUIRES NEW CMAKE COMMAND).

Starting with this commit, building XGL on Windows requires explicitly stating
whether a "Release" or "Debug" build is desired when doing the initial cmake
command.  This is documented in the "Build.md" file.

Linux trace of cube plays to completion on windows!

Changes to the null driver:
    advertise support for both WIN WSI and X11 WSI
    added support for xglCreateBufferView
    and added a logging capability (turned off by default)

Still have some issues:
    - x11 support emulation/replacement
    - Windows null driver creates a bad device queue
    - Need to copy libraries to dir containing the .exe

Glave ext libs are now put in same dir as glvreplay64.exe...

This change only affects Windows -- on Linux the libs and
the glave replay progs were already in the same dir.

Also added a line to BUILD.md regarding the need to have XGL.dll
in PATH.

Redo the way Glave settings are obtained.

nulldrv: Creation of the nulldrv, for use on Windows.

Since the sample Intel driver hasn't been ported to Windows, this nulldrv is
being used to bootstrap Windows support.  It can also provide a hint to new
IHV's for how to set the magic number.

layers:  Updated mem_tracker for new header changes.

Brought mem-tracker up to header rev 47. Worked over tests and
demos to remove errors.

icd: add likely()/unlikely()

They will be expanded to __builtin_expect(...) on gcc.

Win: MUST USE "STATIC_INLINE" instead of "static inline"

loader: Enable env extensions as well as app.

cmake: support clang as the compiler

One can use clang by running cmake like

 $ cmake -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ ..

layers: Fix compilation error on ObjectTracker and Generic layer

intel: Add colorAttachmentCount

bug #13268
header revision: r29554
Add colorAttachmentCount to CREATE_RENDER_PASS_INFO to aid tracing tools.

intel: Remove unnecessary dualBlendMode enable

bug #12925
header version: r29511
Can determine dualBlendModeEnable from blend information provided.
Integrate review feedback.

intel: Remove scissor enable and scissor count

bug #12925
header version: r29511
Remove separate scissor enable and scissor count. Scissor always
enabled and must always provide scissor rect for every viewport.

Add detail on cloning repo

layers: Have DrawState and MemTracker use the common code for reporting

layers: Add settings controls for the generated layers. Includes ParamChecker, ObjectTracker and Generic. Adds the controls for ReportingLevel, ReportingAction and log filename for these layers. Place the common callback code in a header these layers can include.

layers: Add simple txt print of CmdBuffer contents

layers: Track DrawState on a per-cmd-buffer basis

loader: Change sample code to not include a pointer.

It could imply that the ICD needs to allocate space for it.  The pointer is
really to be owned by the loader.

loader: Fix and enhance README.md.

Several adjustments were made, including:

1) Add some MS Windows-related information.  For example, the loader is
   "XGL.dll" on Windows, and "libSGL.so" on Linux.  Also, the separator
   character is given, for each OS, for the PATH environment variables.

2) Add some addition information about dispatching.

3) Add example code for setting the magic value in the ICD.

4) Fix the numbering that looked fine in the "mardown" website, but not on
   GitHub.

5) Try to improve grammar, punctuation, etc.

Eliminate "-Wsign-compare" warnings on Linux ...

With these being newly enabled for the loader, layers, and demos (and it turns
out, for the icd/common directory), it is appropriate to fix them.

Enable additional Linux warnings, for loader/layers/demos.

The intention is for Linux developers to catch more problems (real or
annoyance) before porting to Windows.

The new warnings are:

-Wpointer-arith
-Wsign-compare

These are not used for the Intel sample driver (mostly because the Intel
compiler from Mesa3D generates lots of these warnings, which code we don't want
to modify), nor for the tests.

Not sure how this affects Glave on Linux.

glave: Replay for UpdateDescriptor handle multiple replays

The local chain copy had embedded pointers into the original chain. So
multiple replays/remaps weren't working.  Simplify code to just save a queue
of the actaul objects that need to be remapped  and restore them  after making
the call to xglUpdateDescriptors().