vbo: remove old debug code, add comments

vbo: rename, document function params

vbo: comments

vbo: replace assert(0) with proper assertions

vbo: rename some vars, add new comments, fix formatting, etc.

vbo: use ctx instead of exec->ctx

radeon: add default switch case to silence unhandled enum warning

Use C-style system headers in C++ code to avoid issues with std:: namespace

intel: Fix insufficient integer width for upload buffer offset

I was being overly miserly and gave the offset of the buffer into the bo
insufficient bits, distracted by the adjacency of the buffer[4096].

Ref: https://bugs.freedesktop.org/show_bug.cgi?id=34541
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

svga: Remove some remaining fake S3TC rendering support.

i965: Remove spurious duplicate ADVANCE_BATCH

... a leftover from a bad merge.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i915: Emit a single relocation per vbo

Reducing the number of relocations has lots of nice knock-on effects,
not least including reducing batch buffer size, auxilliary array sizes
(vmalloced and copied into the kernel), processing of uncached
relocations etc.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i915: Suppress emission of redundant stencil updates

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i915: Separate BLEND from general context state.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i915: Only flag context changes if the actual state is changed

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i915: suppress repeated sampler state emission

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i915: Eliminate redundant CONSTANTS updates

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: Use compiler builtins when available

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: Micro-optimise check_state

Replace the intermediate tests due to the logical or with the bitwise
or.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel: use throttle ioctl for throttling

Rather than waiting on the first batch after the last swapbuffers to be
retired, call into the kernel to wait upon the retirement of any request
less than 20ms old. This has the twofold advantage of (a) not blocking
any other clients from utilizing the device whilst we wait and (b) we
attain higher throughput without overloading the system.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: Remove unused 'next_free_page' member

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel: Skip the flush before read-pixels via blit

As we will flush when reading the return values of the blit, we can forgo
the earlier flush.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel: extend current vertex buffers

If the next vertex arrays are a (discontiguous) continuation of the
current arrays, such that the new vertices are simply offset from the
start of the current vertex buffer definitions we can reuse those
defintions and avoid the overhead of relocations and invalidations.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel: Use specified alignment for writes into the upload buffer

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: Clean up brw_prepare_vertices()

Use a temporary glarray variable to replace the numerous input->glarray.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel: combine short memcpy using a temporary allocated buffer

Using a temporary buffer for large discontiguous uploads into the common
buffer and a single buffered upload is faster than performing the
discontiguous copies through a mapping into the GTT.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: upload normal arrays as interleaved

Upload the non-vbo arrays into a single interleaved buffer object, and
so need to just emit a single vertex buffer relocation.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: interleaved vbo

If the user passed in several arrays interleaved in the same vbo, only
emit a single vertex buffer and relocation.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: emit one vb packet per vbo

Track reuse of the vertex buffer objects and so minimise the number of
vertex buffers used by the hardware (and their relocations).

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: upload transient indices into the same discontiguous buffer

As we now pack the indices into a common upload buffer, we can reuse a
single CMD_INDEX_BUFFER packet and translate each invocation with a
start vertex offset.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: suppress repeat-emission of identical vertex elements

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: Move repeat-instruction-suppression to batchbuffer core

Move the tracking of the last emitted instructions into the core
batchbuffer routines and take advantage of the shadow batch copy to
avoid extra memory allocations and copies.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel: use pwrite for batch

It's faster. Not only is the memcpy more efficiently performed in the
kernel (making up for the system call overhead), but by not using mmap
we remove the greater overhead of tracking the vma of every batch.

And it means we can read back from the batch buffer without incurring
the cost of a uncached read through the GTT.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: drop state_bo references to batch_bo

As we use state relocations and we know that all the state belongs to
the same bo, we can drop the multiple references to the same bo.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: directly write wm state to batch

As we write directly into the batch in system memory, we do not need to
write first to the stack (as was to avoid read back through the GTT)

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: write cc straight to batch

As we write directly into the batch in system memory, we do not need to
write first to the stack (as was to avoid read back through the GTT)

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: switch gen6 to use its own cc state bo

In preparation for a greater change, use the color_calc_state_bo already
provisioned for this purpose.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel: Buffered upload

Rather than performing lots of little writes to update the common bo
upon each update, write those into a static buffer and flush that when
full (or at the end of the batch). Doing so gives a dramatic performance
improvement over and above using mmaped access.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel: Replace the bo for a complete update

Rather than performing a blit to completely overwrite a busy bo, simply
discard it and create a new one with the fresh data.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: Combine vb upload buffer with the general upload buffer

Reuse the new common upload buffer for uploading temporary indices and
rebuilt vertex arrays.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel: Pack dynamic draws together

Dynamic arrays have the tendency to be small and so allocating a bo for
each one is overkill and we can exploit many efficiency gains by packing
them together.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel: Use system memory for DYNAMIC_DRAW source objects

Dynamic draw buffers are used by clients for temporary arrays and for
uploading normal vertex arrays. By keeping the data in memory, we can
avoid reusing active buffer objects and reallocate them as they are
changed. This is important for Sandybridge which can not issue blits
within a batch and so ends up flushing the batch upon every update, that
is each batch only contains a single draw operation (if using dynamic
arrays or regular arrays from system memory).

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: Trim the trailing NOOP from 3DSTATE_INDEX_BUFFER

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i965: Fallback on encountering a NULL render buffer

Following a GPU hang, or other error, the render target is not likely to
have an allocated BO and so we must fallback to avoid using it.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=32534
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

i915g: s/__func__/__FUNCTION__/

i915g: kill remnants of mmapped batchbuffer support

We're using bo_subdata.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

i915g: Add winsys/i915/sw to scons build

i915g: Fix void ptr arith

i915g: Add dummy flush_frontbuffer

Fix --enable-shared-glapi configure option

Fix a typo which meant that --enable-shared-glapi didn't actually cause a shared glapi to be built

Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>

egl_dri2: Return NULL when xcb_get_geometry_reply fails.

This should fix bug #33946.

scons: Add aliases for the llvmpipe unit tests.

Now one can simply do

  scons lp_test_format

gallivm: Use simple scaling plus casting in more unorm->float cases.

glsl: Remove $(PWD) from Makefile in favor of .

Hopefully should fix bug #34468.

r300g: fix a possible race when counting contexts

Atomics aren't sufficient here.

r300g: fix invalid dereference in winsys

radeon_bo_unref may destroy the buffer, so call it after p_atomic_dec, not before.

svga: Fix NULL dereference.

Probably introduced with the surface view move from screen to context.

vbo: add debug code to verify that buffers are unmapped before drawing

mesa: MESA_VERBOSE logging for glRead/Draw/CopyPixels, glBlitFramebuffer

st/mesa: set renderbuffer _BaseFormat in a few places

NOTE: This is a candidate for the 7.9 and 7.10 branches

st/mesa: check buffer orientation in blit_copy_pixels()

Can't invert the region if copying between surfaces with different
orientations.

svga: Ensure pending drawing commands other surface operations are emitted before DMAs.

This behavior was last when moving the transfers to the contexts.

This fixes several piglit failures, which were reading the color renderbuffer
before the draw operations were emitted.

svga: Cannot use negate or abs on source to dsx/dsy instructions.

svga: Ensure SWTNL is created after HWTNL.

Matches the internal driver layering, and prevents null svga->hwtnl
dereferencing from inside the swtnl.

svga: Ensure LRP's restrictions are observed in all uses.

The dst reg must be a temporary, and not be the same as src0 or src2.

svga: Preserve src swizzles in submit_op2/3/4.

Several opcodes require scalar swizzle, and this requirement was
being was not being observed when creating temporaries for other reasons.

r300g: remove tracking whether vertex buffers need to be validated

This was getting hard to maintain and didn't really bring any real benefits.
Instead, validate buffers when the vertex array state is dirty.

st/mesa: fix geometry corruption by always re-binding vertex arrays

This is a temporary workaround. It fixes sauerbrauten with shaders enabled.

I guess we might be changing vertex attribs somewhere and not updating
the appropriate dirty flags, therefore we can't rely on them for now.
Or maybe we need to make this state dependent on some other flags too.

More info:
https://bugs.freedesktop.org/show_bug.cgi?id=34378

svga: Disable surface cache for textures

Signed-off-by: Jakob Bornecrantz <jakob@vmware.com>

svga: Describe svga_sampler_views for refcnt debugging

Signed-off-by: Jakob Bornecrantz <jakob@vmware.com>

svga: Make sure that refcnt debugger gets the correct backtrace for create

Signed-off-by: Jakob Bornecrantz <jakob@vmware.com>

util: Make refcnt and symbol debuggers work on windows

Signed-off-by: Jakob Bornecrantz <jakob@vmware.com>

Point to bugs.freedesktop.org rather than bugzilla.freedesktop.org

Suggested by a freedesktop.org admin.

Signed-off-by: Cyril Brulebois <kibi@debian.org>

u_vbuf_mgr: initialize flag indicating that buffers have been updated

This fixes r300g errors:
r300: Cannot get a relocation in radeon_drm_cs_write_reloc.

gallium/svga: Fix unnecessary swtnl fallbacks

When we drop the in_swtnl_draw flag, we must force a rerun of
update_need_swtnl to reset the need_swtnl flag to its correct value outside
of a swtnl vbo draw.

Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>

r600g: reorganise rgtc pieces.

when the cs checker fixes go upstream a lot of this can disappear
into a drm version check.

Signed-off-by: Dave Airlie <airlied@redhat.com>

st/mesa: implement blit-based path for glCopyPixels

If doing a simple non-overlapping glCopyPixels with no per-fragment ops
we can use pipe_context::resource_copy_region().

mesa: fix comments for _mesa_clip_readpixels()

st/mesa: indentation fix

r600g: Start a new TEX clause if the texture lookup address was fetched in the current clause

Signed-off-by: Dave Airlie <airlied@redhat.com>

r600g: Add support to dump vertex- and texture-fetch clauses

Signed-off-by: Dave Airlie <airlied@redhat.com>

r600g: add BC4/5 to RGTC conversion

this doesn't do anything much since the rest of mesa doesn't
support RGTC yet.

util: Fix typo in last commit.

st/mesa: fix incorrect glCopyPixels position on fallback path

If we hit the pipe_get/put_tile() path for setting up the glCopyPixels
texture we were passing the wrong x/y position to pipe_get_tile().
The x/y position was already accounted for in the pipe_get_transfer()
call so we were effectively reading from 2*readX, 2*readY.

NOTE: This is a candidate for the 7.9 and 7.10 branches.

draw: update comments, rename vars in pstipple code

svga: Don't use more than one constant per IFC instruction.

mesa: Do copy propagation across if-else-endif.

Addresses excessive TEMP allocation in vertex shaders where all CONSTs are
stored into TEMPS at the start, but copy propagation was failing due to
the presence of IFs.

We could do something about loops, but ifs are easy enough.

util: Cleanup symbol name resolution on Windows.

- Support symbol name resolution on MinGW.
- Use dbghelp.dll (which should allow 64bit support), but untested yet.
- Cleanup.

docs: updated environment variable list

st/mesa: remove unused screen variables

mesa: remove the MESA_NO_DITHER env var

This was sometimes useful back when 16-bit framebuffers were prominent.

softpipe: rename env vars to be consistent

egl_dri2: add swrast

This enables the egl_dri2 driver to load swrast driver
for software rendering. It could be used when hardware
dri2 drivers are not available, such as in VM.

Signed-off-by: Haitao Feng <haitao.feng@intel.com>

r600g: get s3tc working on cards with crappy 64/128 bit types.

Some cards don't appear to work correctly with the UNORM type,
so switch to the integer type, however since gallium has no
integer types yet from what I can see we need to do a hack to
workaround it for the blitter.

Signed-off-by: Dave Airlie <airlied@redhat.com>

r600g: add missing type to color buffer swap.

gallium/util: init key with memset()

To silence missing initializers warning.

svga: flush when transitioning between HW and SW rendering paths

To avoid mixing HW and SW rendering with the same vertex buffer.

rtasm: add dummy return statement to silence MSVC warning

And use assert(0) instead of abort() to be consistent with rest
of Gallium.

svga: use TRUE/FALSE instead of 0/1

Some fields are booleans, others are bitmasks.  Use TRUE/FALSE to
clarify what's what.

svga: fix incorrect comment