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.

Merge pull request #2 from FslNopper/master

Merge pull request #2 from KhronosGroup/master

Merge pull request #2 from KhronosGroup/master

Update to head fork.

Win: Allow the demos "tri" and "xglinfo" to build on Windows.

These had the WSI hacked out of them, since there isn't yet a standard API
in this space, and what was originally done is only for X11.

Restore xgl.h comments and original whitespace

This keeps the real 47.1 changes in tact.

Get xglPlatform.h working with old MSVC

This was added to revision SVN 29370 for Khronos bug 13274.

Merge branch 'FslNopper-master'

Fix unitialized value in xglCreateInstance().

This causes a problem when no ICD is found.  The function's "res" variable is
returned despite never being intialized.  For now, I chose to return
XGL_ERROR_INCOMPATIBLE_DRIVER.

Changed note, that everything can be done in the console.

Improved note layout.

Added note on how to reinstall working driver.

Merge pull request #1 from KhronosGroup/master

Update from head fork.

misc:Fixes to get better behavior in release builds

Add -fno-strict-aliasing -fno-builtin-memcmp - Both of these are
used by the Mesa driver, and without the first one, we hang? in
link_shaders on release builds

Move cube's LoadTexture out of an assert so it isn't skipped in
release builds

misc: Add warning about recent Ubuntu 14.10 issue

glave: fix replay of AllocMemory packets with info chained structures

BIL: Update BIL version

glave:Fix off by one error in image generation

glave: Fix debugger to export g_pReplaySettings for xglreplayer library

Add build instructions for Windows.

BIL: Update revision number for BIL

tools:Incorperate review feedback on glave screenshots

Move variables into display struct
Make width and height vectors
Change names from snapshot to screenshot
Fix bug when no screenshot frames specified

glave:add code to snapshot frames

layers: Update README.md with current layer status

Win: Eliminate VS2013 miscellaneous additional compiler warnings.

Win: Eliminate VS2013 compiler warnings in auto-gen'd layer code.

Win: Eliminate VS2013 compiler warnings for "loader.c" ...

As part of this, eliminated some similar warnings within/across usage of
"loader_platform.h".  One of these is #define'ing _CRT_SECURE_NO_WARNINGS, so
that VisualStudio won't issue warnings with our use of _snprint() (which they
don't consider secure enough; but their desired function doesn't match the
signature of snprintf()).

Win: Move some macros to "loader_platform.h"

misc: Update readme for alpha header post

glave: More fixes to UpdateDescriptors UPDATE_BUFFERS and UPDATE_IMAGES

glave: Don't dereference null pUserData in APIDump* layers

glave: Fix bugs with UpdateDescriptors pUpdateChain on replay

glave: Fixes in UpdateDescriptar and ClearDescriptorSets

Also removed dead replay code for StateTransitions and AttachDescriptorSet

glave: Add interpret function for xglCreateDescriptorSetLayout

glave: Create interpret function for pUpdateChain of xglUpdateDescriptors()

glave: fix CreateDescriptorRegion trace/replay

glave: Fix trace/replay of CreateDynamicViewportState

glave: Various fixes for UPDATE_SAMPLERS  and PIPELINE_CB_STATE structures

glave: Fix EnumerateLayers replay to handle if layer libs differ from tracing

Was not handling the case where the layer libraries enumerated at replay time
were greater than those enumerated at trace time.

xgl:  Final edits for rev47+ alignment

Took care of the remaining substantive differences.  Still many
formatting/commenting differences. Version set to 47.1 to indicate
some post-47 additions.

layers:  Add validation for mapped mem objects ref'd in cmd buffer

If any mem object is still mapped while referenced in a command
buffer, throw a validation error message.

glave: Glave updated for resource binding model

In glave trace and replay, account for the fact that CreateDescriptorSetLayout and UpdateDescriptors both take chains of structs.

Can compile "loader" and "layers" on Windows and Linux ...

These directories build and are partially turned-on on Windows, using the "tri"
demo (follow-on commit) and a "NULL driver" that was created out of the
sample/Intel driver.  The GetProcAddress() is not yet finding symbols in the
NULL driver.

For now:

- "C:\Windows\System32" is the default XGL driver directory.  The getenv()
  isn't yet working.  I suggest creating your own #define in order to point to
  where a driver is.

- In order to recognize a Windows driver, we must look at both its prefix and
  suffix (i.e. it is named "XGL_*.dll", e.g. "XGL_i965.dll).

- We autogenerate Windows ".def" files for the layers.  Additional info is:

  - This is necessary in order for a DLL to export symbols that can be queried
    using GetProcAddress().  We can't use the normal Windows approach of
    declaring these functions using "__declspec(dllexport)", because these
    functions are declared in "xgl.h".

  - This involves adding and running the new "xgl-win-def-file-generate.py"
    file.

  - NOTE: Layers don't have the xglInitAndEnumerateGpus() entrypoint, just the
    xglGetProcAddr() entrypoint (and now the xglEnumerateLayers() entrypoint).
    Generating them is pretty simple.

NOTE: In order to build on a 64-bit Windows 7/8 system, I did the following:

- Install VisualStudio 2013 Professional

- Install CMake from: http://www.cmake.org/cmake/resources/software.html

  - I let it add itself to the system PATH environment variable.

- Install Python 3 from: https://www.python.org/downloads

  - I let it add itself to the system PATH environment variable.

- Obtain the Git repository, checkout the "ian-150127-WinBuild" branch.

- Using a Cygwin shell: I did the following:

  - "cd" to the top-level directory (i.e. the one that contains the ".git"
    directory).

  - "mkdir _out64"

  - "cd _out64"

  - "cmake -G "Visual Studio 12 Win64" .."

- At this point, I used WindowsExplorer to open the "XGL.sln" file.  I can
  build.  CMake causes the build shortcut to be "Ctrl-Shift-B" instead of the
  normal "F7".  I had to right-click the "ALL_BUILD" project, go to
  Properties->Debugging and change the debug Command and Working Directory to
  point to "tri.exe" and where the executable are.  At this point, I can debug
  (using the normal "F5" shortcut).