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().

Minor fix/enhancement to BUILD.md.

1) Point CMake to www.cmake.org insead of www.python.org.

2) Give a couple of helpful pointers for ICD developers.

Win: Attempt to keep from using Linux functions.

This is a hopefully-temporary solution to prevent some of the problems of
people breaking the Windows build while developing on Linux (or vice-versa).

This uses macros of the names of Linux/Windows-specific functions in order to
catch people who use those functions directly, instead of using the
platform-compatibility-layer functions.

In order to avoid problems with the layers #include'ing "loader_platform.h"
before they #include system files (which can mess them up), I #include
"loader_platform.h" twice.  The 2nd time, it #define's the hopefully-temporary
macros.

Note: For some reason, we can't #define LoadLibrary().  It generates warnings
on Windows.

Linux: Add back the "C99" comment within "loader_platform.h"

icd: add loader magic word to verify ICD is compatible

We wanted a more explicit way to determine if the driver
ICD being loaded is providing compatible objects. To do
that we check for a magic dword value at the beginning
of the object. Non-compliant ICDs will assert in the loader
or the loader's dispatch functions if an object does
not have the correct value.
Dispatch checks are debug only.

Win/icd: Move STATIC_INLINE macro, and use for magic funcs.

The STATIC_INLINE macro is now in "xglPlatform.h".  It is also now being used
by the *_loader_magic_value() functions.  This allows it to be compiled on
Windows.

glave: Unregister debug callback on DestroyInstance()

icd: Fix removing callback from global list

Callback wasn't getting actually removed from the list.

glave: Handle null pointers in stringify of entrypoint parameters

Also add "*" for those pointers that are dereferenced.

Update BUILD.md

Update to build steps from Mike Sartain

icd: add U_ASSERT_ONLY

It can be applied to variables that are only used in assert().

Win: "layers_config.cpp" must #include "loader_platform.h"

Win: Build both "layer_utils.lib" and "layer_utils.dll".

Add draw state diagram on pause events.

    Add a Qt image viewer.
    Use Qt image viewer to view draw state diagram.
    Dump DOT files in appropriate entry points.

Conflicts:
layers/draw_state.c

glave: Add xgl settings that control the MemTracker and DrawState layers

Conflicts:
glave-generate.py
tools/glave/src/glv_extensions/glvreplay_xgl/glvreplay_xgl_settings.cpp
tools/glave/src/glv_extensions/glvreplay_xgl/glvreplay_xgl_settings.h

layers: Add ability to override layer settings from file with a call

Add setLayerOption function call.

layers: Make layer utilities a shared library for one instance per all layers

The global singleton contain map of layer settings had multiple instances. Fix
this by making it a shared library layer_utils.

Conflicts:
layers/CMakeLists.txt

Win/Linux: Eliminate gcc compiler warning for "xglinfo.c"

layers: Fixes for Windows build

tests demos: Fix pDataSize to be either unitinitalized or initialized.

Better tests tracing in uninit case and was a bug in the case where init was
added.

glave: Fix entrypoints with pDataSize and pData to handle non-input param

Properly handle tracing case of pDataSize == don't care as input when
pData == NULL.
Also fix bug on replay about not warning in cases where pData == NULL and
*pDataSize is a mismatch.

glave: remap sampler object in xglCreateDescriptorSetLayout()

glave: Actually add the memory barrier list of structures to trace file

Was just adding a pointer to these structures.

glave: Remap buf/image obj for MEMORY_BARRIER for cmdWaitEvents,cmdPipeBarrier

glave: Fix interpretation of allocMemory and beginCommandBuffer trace packets

glave: Qualify adding to trace packet descriptor set layout structs

loader: Update README.md to instancing  API and C++ ICDs

layers: Initial Descriptor Set output in DrawState Dot file

Also includes various code clean-up in DrawState layer to remove non-functional code.

layers: Restore DrawState to general level of functionality

DrawState updated to track new resource binding model. Currently flagging some errors and printing out Region/Set/Layout/Update info for descriptors. Still lots of work to do. Next step is getting descriptors in dot file working again.

Win: Eliminate VS2013 compiler warnings in "xglinfo.c"

Win: Build both STATIC (.lib) and SHARED (.dll) loader libraries.

glave: Fix for not correctly parsing down tree of structs in AllocMemory

glave: Add custom code for embedded ptrs in CmdWaitEvents and CmdPipelineBarrier calls

layers:  Validate mem ref counts in queueSubmit call

    Grab and save max from GetGpuInfo, use this to validate count in
    QueueSubmit call, warn if GetGpuInfo wasn't called beforehand.

layers:  Handle null fence objects in validation

QueueSubmit can be called with a NULL fence object, updated validation
to correctly handle this case.