vbo: Avoid unnecessary copy to/from current in vertex format upgrade.

Rebuilding the vertex format from scratch every time we see a new
vertex attribute is rather costly, new attributes can be appended at
the end avoiding a copy to current and then back again, and the full
attr pointer recalculation.

In the not so likely case of an already existing attribute having its
size increased the old behavior is preserved, this could be optimized
more, not sure if it's worth it.

It's a modest improvement in FlightGear (that game punishes the VBO
module pretty hard in general, framerate goes from some 46 FPS to 50
FPS with the nouveau classic driver).

Signed-off-by: Brian Paul <brianp@vmware.com>

scons: Unify state tracker SConscripts

scons: Move dependancy checks to the main gallium scons file

scons: Check for libdrm_[intel|radeon] as well

And run SConscripts if they are present.
Also make dri depend on both drm and x11.

scons: Check for pkg-config before trying to use it

Silences warning about missing packages

scons: Detabify

Drivers scons files for a later time

scons: Remove old pipebuffer SConscript

softpipe: disable vertex texturing with draw/llvm

This is a temporary work around to prevent crashes with glean/glsl1
(for example) which try to do vertex shader texturing.

gallivm: add const qualifiers, fix comment string

gallivm: alloca() was called too often for temporary arrays

Need to increment the array index to point to the last value.
Before, we were calling lp_build_array_alloca() over and over for
no reason.

i965: Silence uninitialized variable warning.

Silences this GCC warning.
brw_wm_fp.c: In function 'brw_wm_pass_fp':
brw_wm_fp.c:966: warning: 'last_inst' may be used uninitialized in this function
brw_wm_fp.c:966: note: 'last_inst' was declared here

i965: Silence uninitialized variable warning.

Silences this GCC warning.
brw_wm_fp.c: In function 'precalc_tex':
brw_wm_fp.c:666: warning: 'tmpcoord.Index' may be used uninitialized in this function

r300/compiler: Move declaration before code.

Fixes this GCC warning with linux-x86 build.
radeon_dataflow.c: In function 'get_readers_normal_read_callback':
radeon_dataflow.c:472: warning: ISO C90 forbids mixed declarations and code

llvmpipe: added some debug assertions, but disabled

r300/compiler: Move declaration before code.

Fixes this GCC warning with linux-x86 build.
radeon_pair_schedule.c: In function 'merge_presub_sources':
radeon_pair_schedule.c:312: warning: ISO C90 forbids mixed declarations and code

meta: Fix incorrect rendering of the bitmap alpha component.

Signed-off-by: Brian Paul <brianp@vmware.com>

meta: Don't leak alpha function/reference value changes.

Signed-off-by: Brian Paul <brianp@vmware.com>

tgsi: remove unused function

st/mesa: Reset the constant buffers before destroying the pipe context.

Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>

gallivm: implement execution mask for scatter stores

gallivm: added lp_elem_type()

gallivm: add pixel offsets in scatter stores

We want to do the scatter store to sequential locations in memory
for the vector of pixels we're processing in SOA format.

gallivm: added debug code to dump temp registers

graw/gdi: Fix window dimensions.

The requested window size is of the client area,
so account for surrounding borders and bars when
creating the window.

scons: Hook-up graw-gdi target.

graw/gdi: Initial commit.

r300g: Do not use buf param before checking for NULL.

Commit 8dfafbf0861fe3d2542332658dd5493851053c78 forgot to update r300g.
There is a buf == NULL check, but buf is used before for var init.

Tested-by: Guillermo S. Romero <gsromero@infernal-iceberg.com>

tgsi/build: Reduce interface clutter.

Make private those functions that are used internally only.

tgsi/exec: Get rid of obsolete condition codes.

tgsi/exec: Cleanup the remaining arithmetic instructions.

As a result remove some nasty macros.

dri/nouveau: Silence uninitialized variable warning.

Fixes this GCC warning.
nouveau_vbo_t.c: In function 'nv10_vbo_render_prims':
nouveau_render_t.c:161: warning: 'max_out' may be used uninitialized in this function
nouveau_render_t.c:161: note: 'max_out' was declared here

gallivm: add some LLVM var labels

gallivm: implement scatter stores into temp register file

Something is not quite right, however.  The piglit tests mentioned in
fd.o bug 31226 still don't pass.

ir_reader: Fix some potential NULL pointer dereferences.

Found by inspection.

ir_reader: Remove useless error check.

It's already been determined that length == 3, so clearly swiz->next is
a valid S-Expression.

ir_reader: Return a specific ir_dereference variant.

There's really no reason to return the base class when we have more
specific information about what type it is.

glsl: Don't print a useless space at the end of an S-Expression list.

We really only want to print spaces -between- elements, not after each
element.  This cleans up error messages from IR reader, making them
(mildly) easier to read.

Refresh autogenerated file builtin_function.cpp.

glsl/builtins: Clean up some ugly autogenerated code in atan.

In particular, calling the abs function is silly, since there's already
an expression opcode for that.  Also, assigning to temporaries then
assigning those to the final location is rather redundant.

glsl/builtins: Rename 'x' to 'y_over_x' in atan(float) implementation.

For consistency with the vec2/vec3/vec4 variants.

r600g: Swap the util_blitter_destroy call order.

Trivial change that avoids a segmentation fault when the blitter state
happens to be bound when the context is destroyed.

The free calls should probably removed altogether in the future -- the
responsibility to destroy the state atoms lies with whoever created it,
and the safest thing for the pipe driver is to not touch any bound state
in its destructor.

mesa: code to unpack RGBA as uints

xorg/vmwgfx: Link libkms when available.

st/xorg: Detect libkms with scons too.

st/xorg: Add missing \n to error message.

xorg/vmwgfx: Add missing source file to SConscript.

intel: Remove leftover dri1 locking fields in the context.

intel: Remove duplicated teximage miptree to object miptree promotion.

intel_finalize_mipmap_tree() does this optimization too, just more
aggressively.

intel: Avoid taking logbase2 of several things that we max.

logbase2(max(width, height, depth)) ==
max(logbase2(width), logbase2(height), logbase2(depth)), but in 60
bytes less code.

i965: Remove dead intel_structs.h file.

intel: Remove the magic unaligned memcpy code.

In testing on Ironlake, the histogram of clocks/pixel results for the
system memcpy and magic unaligned memcpy show no noticeable difference
(and no statistically significant difference with the 5510 samples
taken, though the stddev is large due to what looks like the cache
effects from the different texture sizes used).

intel: Annotate debug printout checks with unlikely().

This provides the optimizer with hints about code hotness, which we're
quite certain about for debug printouts (or, rather, while we
developers often hit the checks for debug printouts, we don't care
about performance while doing so).

egl/gdi: fix typo: xsurf->gsurf

evergreeng: set hardware pixelcenters according to gl_rasterization_rules

evergreeng: respect linewidth state, use integer widths only

Discard fractional bits from linewidth.  This matches the nvidia
closed drivers, my reading of the OpenGL SI and current llvmpipe
behaviour.

It looks a lot nicer & avoids ugliness where lines alternate between n
and n+1 pixels in width along their length.

Also fix up r600g to match.

r600g: don't call debug_get_bool_option for tiling more than once

evergreeng: protect against null constant buffers

Should do better than this and actually unbind the buffer, but haven't
yet gotten it to work.

st/egl: Use native_display_buffer for EGL_MESA_drm_image.

native_display_buffer is just a wrapper to resource_{from,get}_handle
for drm backend.

st/egl: Add native_display_buffer interface.

The interface is a wrapper to pipe_screen::resource_from_handle and
pipe_screen::resource_get_handle.  A winsys handle is
platform-dependent.

st/egl: Add extern "C" wrapper to native.h.

This allows a backend to be written in C++.

r600g: set hardware pixel centers according to gl_rasterization_rules

These were previously being left in the default (D3D) mode.  This mean
that triangles were drawn slightly incorrectly, but also because this
state is relied on by the u_blitter code, all blits were half a pixel
off.

r600g: remove unused flink, domain fields from r600_resource

These were being set but not used anywhere.

r600g: use a buffer in GTT as intermediate on texture up and downloads

Generalize the existing tiled_buffer path in texture transfers for use
in some non-tiled up and downloads.

Use a staging buffer, which the winsys will restrict to GTT memory.

GTT buffers have the major advantage when they are mapped, they are
cachable, which is a very nice property for downloads, usually the CPU
will want to do look at the data it downloaded.

r600g: propogate resource usage flags to winsys, use to choose bo domains

This opens the question of what interface the winsys layer should
really have for talking about these concepts.

For now I'm using the existing gallium resource usage concept, but
there is no reason not use terms closer to what the hardware
understands - eg. the domains themselves.

r600g: propagate usage flags in texture transfers

st/egl: Add support for EGL_MATCH_NATIVE_PIXMAP.

Added for completeness.  It makes sense to have such mechanism, but I am
not aware of any user of that..

st/egl: Add support for swap interval and swap behavior.

The value of EGL_MAX_SWAP_INTERVAL and whether
EGL_SWAP_BEHAVIOR_PRESERVED_BIT is set will depend on the native
backend used.

st/egl: Remove flush_frontbuffer and swap_buffers.

They are deprecated by native_surface::present and there is no user of
them.

d3d1x: Use native_surface::present.

Replace native_surface::flush_frontbuffer and
native_surface::swap_buffers calls by native_surface::present calls.

st/egl: Use native_surface::present callback.

Replace native_surface::flush_frontbuffer and
native_surface::swap_buffers calls by native_surface::present calls.

st/egl: Add native_surface::present callback.

The callback presents the given attachment to the native engine.  It
allows the swap behavior and interval to be controlled.  It will replace
native_surface::flush_frontbuffer and native_surface::swap_buffers
shortly.

egl: Set up the pthread key even TLS is used.

We have to rely on the pthread key destructor to free the current thread
info when a thread exits.

st/vega: Remove unnecessary headers.

mesa: silence new warnings in texobj.c

Silences warning such as:
main/texobj.c:442:40: warning: ISO C99 requires rest arguments to be used
main/texobj.c:498:58: warning: ISO C99 requires rest arguments to be used

savage: Remove unnecessary header.

intel: For batch, use GTT mapping instead of writing to a malloc and copying.

No measurable performance difference on cairo-perf-trace, but
simplifies the code and should have cache benefit in general.

mesa: Don't compute an unused texture completeness debug string.

This showed up at about 1% on cairo-gl firefox-talos-gfx, where
glClear() is called while a texture is incomplete.

st/mesa: Reset the index buffer before destroying the pipe context.

Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>

r600g: Destroy the winsys in r600_destroy_screen().

Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>

r600g: Fixed two memory leaks in winsys.

Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>

r600g: Delete custom_dsa_flush on shutdown.

Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>

r600g: We don't support PIPE_CAP_PRIMITIVE_RESTART.

Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>

r600g: Made radeon_bo::map_count signed.

That way assert(map_count >= 0) can actually fail when we screwed up.

Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>

r600g: Fixed unmap condition in radeon_bo_pb_destroy().

Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>

r600g: Made radeon_bo_pb_map_internal() actually call radeon_bo_map().

This ensures that we increase bo->map_count when radeon_bo_map_internal()
returns successfully, which in turn makes sure we don't decrement
bo->map_count below zero later.

Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>

r600g: Removed unused 'ptr' argument from radeon_bo().

Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>

graw: Tidy graw xlib scons file a bit

llvmpipe: add a cast

llvmpipe: assign context's frag shader pointer before using it

The call to draw_bind_fragment_shader() was using the old fragment
shader.  This bug would have really only effected the draw module's
use of the fragment shader in the wide point stage.

mesa: Fix C++ includes in sampler.cpp

Some C++ header files were included in an extern "C" block. When building with
Clang, this caused the build to fail due to namespace errors. (GCC did not
report any errors.)

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>

st/mesa: unbind constant buffer when not in use

Important as more constant buffers per shader start to get used.

Fix up r600 (tested) and nv50 (untested) to cope with this.  Drivers
previously didn't see unbinds of constant buffers often or ever, so
this isn't always dealt with cleanly.

For r600 just return and keep the reference.  Will try to do better in
a followup change.

llvmpipe: guard against NULL task->query pointer

This doesn't seem like it should be possible, but some test suites
manage to hit this case.  Avoid crashing release builds under those
circumstances.

llvmpipe: avoid generating tri_16 for tris which extend past tile bounds

Don't trim triangle bounding box to scissor/draw-region until after
the logic for emitting tri_16.  Don't generate tri_16 commands for
triangles with untrimmed bounding boxes outside the current tile.

This is important as the tri-16 itself can extend past tile bounds and
we don't want to add code to it to check against tile bounds (slow) or
restrict it to locations within a tile (pessimistic).

mesa: fix aux/accum comment and error message mixups

mesa: remove always-false conditional in check_compatible()

The two gl_config pointers can never be equal.

dri/util: add a bunch of comments

mesa: move the gl_config struct declaration

It was in the middle of the lighting-related structures before.
Also add some info about field sizes in this structure.

mesa: use GLubyte for edge flag arrays

Fixes http://bugs.freedesktop.org/show_bug.cgi?id=31310

scons: Propagate installation targets.

Fixes libgl-xlib target.

scons: i915 can't build on MSVC either.

I thought I had singled it out before, but apparently not.