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

intel: enable HiZ on Gen7+

The support is restricted to Gen7+.  To support HiZ on Gen6, we need to
perform manual combined/separated stencil resolves and that may hurt the
performance instead.

Also, we do not support fast depth clears.  xglSetFastClearDepth() will be
removed in a later xgl.h revision.

intel: add cmd_meta_ds_op()

It can perform fast depth clear, depth resolve, and hiz resolve.  There is a
TODO about fast depth clear.

With our current xgl.h revision, I have to assume a layout in
xglCmdClearDepthStencil().  It will get fixed when we are at revision 59.
Alternatively, I could also assume the worst case and always do a depth
resolve after clearing, but that may hurt the performance.

In other words, it is invalid to use XGL_IMAGE_LAYOUT_GENERAL for shadow
textures.

intel: make sense of gen6_meta_DEPTH_STENCIL_STATE()

When operating on DEPTH, enable only depth write.  When operating on STENCIL,
enable only stencil test and stencil write for both faces.  The old code
should supposedly work, but it is hard to reason about it.

intel: avoid unnecessary stencil and hiz relocs

When the depth/stencil view does not need a separate stencil buffer or a hiz
buffer, do not set up relocs for them.

intel: add intel_format_has_depth()

Replace intel_format_is_depth() by intel_format_has_depth().

intel: remove intel_format_is_stencil()

It is not needed since the update to XGL_FORMAT by
2b7e88a1fdefc49d78e80765745e87e00034c2a0.

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

intel: ICD_EXPORT all API functions

intel: format API functions as declared in xgl.h

intel: rename cmd_prepare.c to cmd_barrier.c

intel: minor improvements to fb.c

Less surprises when the applications do things we do not support.

Revert "intel: Be sure to set width & height for depth-only"

This reverts commit da71cc37e5ab9720cc073ac5a40866ded83cb760.  The
width and height are always set to default values since
71fcc2dae2f302fd2e52836b4098b233762d56f8.

intel: mostly cosmetic changes to fb.[ch]

intel: do not cast away constness in intel_cmd_begin()

intel: move xglCmd{Begin,End}RenderPass to cmd_pipeline.c

All xglCmd*() functions should be defined in cmd*.c.

intel: rename intel_rp_from_obj() to intel_render_pass_from_obj()

For consistency.

intel: rename intel_framebuffer to intel_fb

For consistency.

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

compiler: add support to create undefined initializer value for structs

This was a TODO that got tripped by dota_75.frag.

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.

intel: silence a compiler warning

Silence

  warning: ‘buf_format’ may be used uninitialized in this function

intel: rework command buffer waiting

When a bo is submitted, we need the submission sequence number which we can
wait for.  However, the kernel does not return the seqno.  We have to keep the
bo around and wait for the bo.  This fixes

  xglQueueSubmit(..., cmd, ...);
  xglDeviceWaitIdle(...);
  xglDestroyObject(..., cmd);
  xglDeviceWaitIdle(...);

where the second wait waited for a destroyed cmd buffer.

intel: rework xglQueueSubmit()

cmdBufferCount is usually large.  Add queue_submit_cmd_debug() and
queue_submit_cmd() depending on whether INTEL_DEBUG is set to reduce
branching.  Error handling is also improved such that xglQueueSubmit() appears
more atomic to callers.

icd: add likely()/unlikely()

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

tests:  Fix cut-and-paste error in QuadWithIndexedVertexFetch test.

Looks like the wrong memory barrier was set.

compiler: fix "fireflies" over water in dota2 with BIL

A shader with an initialized global array was causing an llvm
code sequence the glassy backend translator had not yet seen. In
particular, a store of the array constant through a local pointer
into alloca space made it into Bottom IR. Loads and stores are not
normally seen in Bottom IR. Fixed emitIRStore to deal properly with
these stores.

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

loader: Enable env extensions as well as app.

tests: Fix XGLTriangleTest to not bind VBO since it doesn't use one

compiler: Disable faceforward decomposition, backend handles it

Revert "compiler: Allow texture unit bindings to also set sampler index"

This reverts commit c33ac9267a70115c29f97eb20966de4c9aed6938.

Revert "compiler: If XGL is being used, assign buffer bindings directly from the shader"

This reverts commit 2bec741ece71b9a5deffd31227e2c7a434dfa2cd.

compiler: Support layout(set=,binding=)

This change allows shaders to specify set and binding.
The values come from BottomIR as packed 32-bit values:
16 high bits for set, 16 low bits for binding.
We allow them to remain together through the backend, then
split apart during resource map creation.

To facilitate this change, we also switched to using the
resource mapping tracked by the compiler, rather than
forcing bindings to match the descriptor set layout, which
was a short term solution.  This results in greatly reduced
binding table size.

compiler: Bump Mesa SamplerUnits to 32 bits

This will facilitate packing set and binding through the stack.

cmake: support clang as the compiler

One can use clang by running cmake like

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

intel: fix a compile error with clang

sizeof(ir_texture::lod_info) is considered invalid by clang because lod_info
is not a static member.

intel: fix the param type of img_type_to_view_type()

It should be XGL_IMAGE_TYPE, not XGL_IMAGE_VIEW_TYPE.

intel: fix stencil refs for stencil clears

This was overlooked in the prior fix.

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

glave: Fix layer settings typo.

glave: Add ObjectTracker layer.

Also remove MemTracker layer for now since it is not currently built.

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.

intel: fix stencil refs

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

intel: fix mulitple descriptor sets/regions

Without adding set->pool_begin to the offsets provided by rmap, we always read
from the beginning of the descriptor pool.  Replace the error-prone
intel_desc_pool_read_*() by intel_desc_set_read_*() to get automatic
offsetting.

intel: fix -Wmaybe-uninitialized warnings

intel: fix -Wunused-(but-set-)variable warnings

icd: add U_ASSERT_ONLY

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

intel: add INTEL_DEBUG=hang

When enabled, make command buffer submission synchronous and decode the
command buffer that hangs the GPU.

intel: add intel_winsys_read_reset_stats()

It can be used to read how many active and pending batch buffers are discarded
so far due to GPU resets.

intel: do not submit invalid command buffers

We do not know what are in an invalid command buffer and we should not submit
it to GPU.

intel: add cmd_fail() to signal command buffer errors

The main purpose is have a place to set breakpoing on on command buffer
errors.

intel: always emit "post" WAs

For workaround PIPE_CONTROLs that are needed "after" commands, they should
never be skipped.

intel: set up mappable and total aperture sizes

The mappable size is usually 256MB.  This is the amount of memories that can
be simultaneously mapped.  The total size is usually 2GB.  This is the amount
of memories that can be referenced in a command buffer.  The real limit
imposed by intel_winsys_can_submit_bo() is lower: 3/4 the total size minus fb
and etc.

These are set up but not used.  Not sure if we want to expose a separate heap
for mappable memories.

intel: use the same PCB alloc setup for normal and meta paths

Eventually, we will set up PCB alloc once per command buffer.  Let both paths
agree with each other for now.

intel: double PCB alloc on GT3

The available space is doubled on GT3.

intel: set up dummy drawing rect when unused in meta

We do not want it to have misleading values when analysizing the batch buffer
dumps.

intel: do not emit 3DSTATE_SBE on Gen6

The commands in intel_pipeline are copied into the batch buffer directly.  We
should not have 3DSTATE_SBE in it on Gen6.

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

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

intel: increase max_batch_buffer_size to 150K dwords

Increasing the max batch buffer size allows more relocations per command buffer.

intel: suppress PIPE_CONTROL with 0 flags