sna: Fix no relaxed delta path for gen4+

This is for debug purposes only.

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

sna: Auto-retire upload proxies

This was disabled in

commit 9f4f855ba37966fb91d31e9081d03cf72affb154
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Mon May 26 07:06:18 2014 +0100

    sna: Implicit release of upload buffers considered bad

as retiring the buffers during the command setup could free one of the
earlier bo used in the command. But discarding the snooped bo could
still be advantageous. So restore the automatic discard of upload
proxies, but make sure we only do between operations.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Matti Hämäläinen <ccr@tnsp.org>

sna: Only allow the inplace copy to replace a complete overwritten GPU bo

And double check after recreating the new GPU bo that is still mappable.

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

sna: Rearrange scanout_flush checking to avoid potential crash

With a shadow, we may never attach a GPU bo to the frontbuffer, so be
careful when checking the scanout not to dereference that NULL pointer.

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

sna: Report KMS driver version

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

man: Update notes about TearFree

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

sna/dri2: Restore flush on front buffers after swapping bo

Fallout from commit 8369166349c92a20d9a2e7d0256e63f66fe2682b
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Wed Jun 4 08:29:51 2014 +0100

    sna/dri2: Enable immediate buffer exchanges

We set the flush hint too early, as we may need to swap the GPU bo for a
more appropriate buffer for DRI.

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

sna: Hook up a backlight udev monitor for external changes

Changes to the backlights are notified through uevents. Hooking up a
udev monitor to listen out for external changes to the backlight (e.g.
through ACPI function keys, or by the user writing to
/sys/class/backlight directly) is easier than enabling polling on the
backlight sysfs file using X's select() mechanism.

Since we listen to backlight changes, we have to be careful not to
confuse the side-effects of disabling connectors (which may cause either
ourselves or the kernel to turn off the backlight) with the user value.

Many thanks to Alexander Mezin for the suggestion to use udev for
tracking the notifications for external changes to the backlight.

Reported-by: Alexander Mezin <mezin.alexander@gmail.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79699
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

sna: When the output is off, report the cached backlight value

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

sna: Inline calls to save/restore backlight around output DPMS

The intertwined logic was a little confusing to read mixed between the
functions, so inline it.

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

sna: Set initial output DPMS state from kernel

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

sna: Make sure we recompute the vblank interval after reconfiguring the CRTC

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

sna: Skip marking the CRTC as damaged for TearFree

As we have reorder the application of damage on a TearFree frontbuffer
to before we attach the CRTC, we do not need to then schedule the update
afterwards (until it gets drawn by the client).

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

sna: Fix DBG compilation

Update the DBG to reflect the new arguments to sna_damage_reduce_all()

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

sna: Curry parameters to sna_damage_all()

It is far easily to pass the PixmapPtr into the function and have it
pluck out the width and height than do so in all callers.

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

sna: Mark CPU as all damaged when discarding a busy GPU bo

When we discard a GPU bo, we need to make sure that the remaining
content is marked as only accessible via the CPU shadow pointer.

Regression from commit 65301412ecf2d55ab55a2d7faeaa048d4ee8b1d0
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Sat May 17 20:59:38 2014 +0100

    sna: Discard active GPU buffers before uploading into them

Reported-by: Jan Alexander Steffens <jan.steffens@gmail.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79517
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

sna/dri2: Enable immediate buffer exchanges

The primary benefit of this is avoid the extra blit when using a
compositor and instead propagate the compositor flip on the frontbuffer
to the scanout, or equivalently allows a fullscreen game to flip onto
the scanout without intervention by TearFree.

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

configure: XORG_DRIVER_CHECK_EXT does not create CFLAGS/LIBS variables

XORG_DRIVER_CHECK_EXT only adds the package name to the list of
REQUIRED_MODULES (which is later passed to pkg-config). For our optional
dependencies, we must do a later call to pkg-config to verify the
package is available and then remember to include the includes with the
CFLAGS.

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

configure: Remove xf86driproto from REQUIRED_MODULES

We only depend upon the original DRI1 infrastructure when building i810
with DRI support. So unless that is true, do not throw an error if the
legacy protocol is absent.

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

configure: Don't link the driver against libX11

78dc0c04745ad4485b994f67833f4a155749f01d added REQUIRED_MODULES to the
driver link line for... some reason.  That pulled in the libs from the
XF86DRI check, which near as I can tell has always been wrong, all of
the other extension checks just look for the protocol module and
xextproto doesn't define dri1 protocol in any case.

Signed-off-by: Adam Jackson <ajax@redhat.com>

sna: Replace the bo for tiled uploads if not suitable and being replaced

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

sna: Allow replacements to cancel operations between both bo under a Pixmap

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

sna: Discard unwanted damage when promoting to a full CPU migration

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

sna: Fix early return in download using the GPU

In commit 961139f5878572ebea268a0bbf47caf05af9093f
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Fri May 30 09:45:15 2014 +0100

    sna: Use manual detiling for downloads

the code for deciding when to use the GPU was refactored into a new
function that also performed the transfer, but failed to notice the
early return.

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

sna: Use CPU mmappings for linear uploads

The other half of the commit that accidentally got included with

commit bb49222a514b1d6041f3d9530a22f5701377118b
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Mon Jun 2 08:25:52 2014 +0100

    sna: Add DBG hints for using inplace CPU mmappings

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

Only enable the new Cursor API for the next pre-release of Xorg

Make sure we can compile against the pre-release tarballs at the expense
of an innocuous compile warning if compiled against xserver.git

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

uxa: Add Present extension support

Signed-off-by: Keith Packard <keithp@keithp.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Conflicts:
configure.ac
src/uxa/intel.h
src/uxa/intel_driver.c

uxa: Restructure DRM event handling.

This refactors the drm interrupt handling logic quite a bit, both to
allow for either DRI2 or Present handlers, but also to eliminate
passing pointers through the kernel. Passing pointers left the kernel
holding the only reference to some internal X server data structures.

After a server reset, the X server would end up using stale pointers
stored in those structures. Using simple integers makes it possible to
empty the queue of pending interrupt data and then ignore the stale
kernel data.

Signed-off-by: Keith Packard <keithp@keithp.com>
Reviewed-by: Eric Anholt <eric@anholt.net>

uxa: Support 64-bit MSC values. Handle kernel vageries about MSC reporting

The kernel sometimes reports bogus MSC values, especially when
suspending and resuming the machine. Deal with this by tracking an
offset to ensure that the MSC seen by applications increases
monotonically, and at a reasonable pace.

Also, provide a full 64 bits of MSC value by noticing wrapping and
tracking the high 32-bits of MSC separately.

Signed-off-by: Keith Packard <keithp@keithp.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Conflicts:
src/uxa/intel_dri.c

uxa: Add DRI3 and miSyncShm support

Based on a patch by Keith Packard.

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

sna: Add support for Present

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

sna: Add support for DRI3

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

sna: Relax tiling height restrictions

Only force the even-tile-row alignment if we have an old GPU with an old
kernel that doesn't perform conservative alignment for us (as required).

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

intel-virtual-output: Add DRI3 xfer path

Just as proof-of-principle.

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

Add rudimentary tests for Present

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

Add rudimentary tests for DRI3

This is a simple little test to create a pixmap from a local bo, copy it
to a normal pixmap, then read it back by importing it into anther local
bo. It tests the fundamental mechanisms of opening a DRI3 render device,
importing into pixmaps, exporting into /buffers and a read-barrier.

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

intel: Clear structs for valgrind

When probing the module, clear structs passed into unknown ioctls to
keep valgrind quiet.

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

intel: Add common routines and configure probing for DRI3

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

sna/dri2: Use real async flips

Presuming that we have both kernel support and Xorg support, of course.

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

sna: Enable kernel support for asynchronous flips

If a flip fails, attempt to restore the previous working configuration
(using a modeset) then disable further flipping.

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

sna/dri2: Hide MSC reported as going backwards

OML_sync_control mandates that MSC must be monotonic, so if the kernel
reports that they go backwards, lie.

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

sna/dri2: Always force the async blits to be immediate

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

sna/dri2: Decouple spent flip

If the flip is already completed before we finish the previous flip, the
drawable has been replaced already and we can unchain the flip queue.

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

sna/dri2: Mark queued flip MSC as the one after next

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

sna/gen8: Clear all URB push constant allocations

A little paranoia to clear the unused portion of the URB and dedicate it
to VS entries.

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

sna/gen8: Set GT level

We use the GT level to decide how favourable it is to use the GPU in
various circumstances, now set it.

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

sna: Add DBG hints for using inplace CPU mmappings

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

sna: Silence a DBG compiler warning

kgem.c: In function '_kgem_submit':
kgem.c:3243:12: warning: ignoring return value of 'write', declared with attribute warn_unused_result [-Wunused-result]
       write(fd, kgem->batch, batch_end*sizeof(uint32_t));

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

sna: Reuse the first scanout bo

The path of last resort is meant to reuse the first scanout bo if they
all busy (since it will be the oldest). It chased a dangling pointer
instead.

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

sna: Do not allow imported buffers to be cached

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

sna: Mark all caches for expiration

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

test: Only compute the masked pixel value if depth!=32

Minor saving for when we use a8r8g8b8.

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

sna: Use manual detiling for downloads

If we can CPU mmap the GPU bo, prefer to do so for migration to the CPU
as this saves an extra serialisation step.

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

sna: Unexport kgem_get_unique_id()

This should always be set during bo creation

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

sna/dri2: Reorder declarations to compile on old Xorg

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

Silence CLang (almost)

Fix up all the warnings about implicit enum conversions. The
container_of() macro remains defunct.

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

sna/dri2: Fix the continuation mode for old style triple buffering

Otherwise we end up sending too many SwapCompletes back to the client
and confusing ourselves.

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

sna: Fix the depth/format on the temporary shadow Pictures

When scaling the output, we re-render onto the scanout using a call to
Composite, for which we create temporary Pictures. In most cases, the
mismatching depth/format goes unnoticed, but along one particular
fallback path, we use those to create a new Picture but the mismatching
format results in a segfault.

Reported-by: Matti Hämäläinen <ccr@tnsp.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79320
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

sna: Handle bad picture format/depth mismatches

Scaling the output feed in an invalid picture->format/depth combination
which causes the fallback downsampling function to explode. Whilst this
is a bug in the higher layer, we can handle the error anyway.

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

uxa: Silence the compiler over a couple of unused variables

intel_driver.c: In function 'I830LeaveVT':
intel_driver.c:1085:24: warning: unused variable 'intel' [-Wunused-variable]
  intel_screen_private *intel = intel_get_screen_private(scrn);
                        ^
intel_driver.c: In function 'I830EnterVT':
intel_driver.c:1102:24: warning: unused variable 'intel' [-Wunused-variable]
  intel_screen_private *intel = intel_get_screen_private(scrn);

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

uxa: Silence compiler for warnings in Cursor API changes

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

sna: Silence compiler for warnings in Cursor API changes

The API changed again in the RC, so update to keep the compiler quiet.
At the same time, protect against the ARGB_CURSOR struct changes.

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

sna/dri2: Move scanout processing from frame event to global

The scanout is a global property, track it as so. The primary benefit to
this is it strengthens our assertions that we never hand out an active
scanout for use as a back buffer.

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

sna/dri2: Defer reallocation of backbuffer until request

If we defer the reallocation of the backbuffer until the client requests
the set of current buffers (with GetBuffers), then we can often avoid
allocating the fresh backbuffer as the flip often retires before the
client is ready.

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

sna/dri2: Make the swap-limit transitions more obvious

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

sna: Implicit release of upload buffers considered bad

Currently upload buffers are automatically decoupled when the buffer is
retired. As retiring can happen during command setup after we have
selected which bo to render with, this can free the bo we plan to use.
Which is bad.

Instead of making the release of upload buffers automatic, we manually
check whether the buffer is idle before use as a source to consider
scrapping it and replacing it with a real GPU bo. This is likely to keep
upload buffers alive for longer (limiting reuse between Pixmaps but
making reuse of the buffer within a Pixmap more likely) which is both
good and bad. (Good - may improve the content cache, bad - may increase
the amount of memory used by upload buffers for arbitrary long periods.)

Reported-by: Matti Hämäläinen <ccr@tnsp.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79238
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

sna: Assert that we do not replace active IO buffers

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

sna/dri2: Handle allocation failure adding frame events gracefully

Now that we do not preallocate the storage for the chain in the Window
devPrivates, we need to check for an allocation failure.

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

sna/dri2: Tidy add/remove chain to use the common DRI2 private

The code is more readable directly updating the private rather than
hiding the pointer chasing through the opaque lookups.

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

sna/dri2: Handle offscreen Window and Pixmap with CRTC tracking

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

test/dri2: Restore original configuration after testing

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

sna/dri2: Combine the DRI2 Window privates together into a single struct

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

sna/dri2: Free the private CRTC tracker on each window

Remember to clean up our privates when the window is destroyed.

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

sna/dri2: Adjust current msc / target seq between CRTCs

The target_msc supplied by the client is required to be monotonic. Since
the client does not know which CRTC the window is on, they do not
control from which pipe we derive the MSC, and so we need to adjust the
hardware values such that the client only ever sees a monotonic value.

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

test/dri2: Check validity of MSC across CRTC

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

sna/dri2: Queue the flip immediately for the next frame

If we compute that we want to flip on the next frame, queue it
immediately rather than take a round trip through the kernel and risk
missing the vblank.

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

sna/dri2: Tidy computation of target sequence

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

sna/dri2: Use the timestamps stored on the CRTC

Now that we store the information for the swap completion on the CRTC,
we don't need to pass it around inside the frame event.

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

sna/dri2: Tweak flip continuation

Minor tidying to the returned target_msc and a useful assertion.

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

intel-virtual-output: Probe after claiming virtual output

Rerun a detection cycle after claiming the virtual output so that it is
hidden again.

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

sna/dri2: DBG fixes

Update the DBG statements to match current code.

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

sna/dri2: Move fixed array allocations to per-crtc

Replace the fixed size MAX_PIPES array by moving the vblank handling to
per-crtc.

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

sna/dri2: Maintain sequence calculation in 64bits

Avoid casting down to 32bits at the begining of the calculation to only
then compare against the full 64bit values later.

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

sna/dri2: Fix WaitMSC to maintain the vblank counters

Fix WaitMSC to also use 64 bit counters and maintain the last vblank
notifications.

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

sna/dri2: Apply a margin of error to the sequence wrap detection

If we have a queue of events, we may set the msc from a recent vblank
query only to then process an older vblank event and declare the counter
wrapped.

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

sna: Fix attachment of the default monitor to the first output

Broke during code motion in

commit add84cd8a8dc6d285912d0ea3a3a3e7faa9e0942
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Fri May 2 13:21:02 2014 +0100

    sna: Perform dynamic connector discovery

Reported-and-tested-by: Jay Little <jaylittle@jaylittle.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79015
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

sna/dri2: Decouple queued events

Becareful when reaping a chain on a destroyed window as some events may
be queued ahead.

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

sna/dri2: Limit pending swaps to 1 when queueing a future blit

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

sna/dri2: Tidy conditional use of XORG_CAN_TRIPLE_BUFFER

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

sna/dri2: Tidy recording of the last swap sequence number and time

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

uxa: fix getmsc to not fail hard

If some outputs go away we race with this call and apps
get X errors and fall over. Do what SNA does and don't
bother trying.

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

sna/dri2: Tidy computation of 64bit ust

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

sna/dri2: Client cannot be NULL, so drop the checks

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

sna: Only mark the scanout as being busy for writes (not solitary reads)

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

sna/dri2: Verify that the reference pipe is still active for flipping

We rely on the reference pipe to drive the event wake up. If we issue a
deferred flip, there is a chance that the user could rearrange the
screen on another crtc whilst otherwise preserving the screen geometry.

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

uxa: fix pageflips on 3 head scenarios

While fixing up UXA for MST I eventually fell over this bug.

Signed-off-by: Dave Airlie <airlied@redhat.com>
[ickle: replace constant with macro to track pipe-dependent code]

sna: Fix assertions for combining shadow transforms and TearFree

When the RandR shadow buffer is enabled we set the sna_crtc->transform
flag. We set the crtc->transform_in_use only when coordinate transforms
are required - i.e. we may enable the shadow buffer if a CRTC is out of
bounds, but otherwise has no transformation. That confused the asserts.

Reported-by: Zdenek Kabelac <zkabelac@redhat.com>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

sna: Switch iterators over to real CRTCs only

Many loops only want to walk the real CRTCs, so update the loop bounds
to be explicit and assert that they are the real ones.

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

sna: Discard active GPU buffers before uploading into them

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

sna: Do not mark an upload as a potential replace if the dst is pinned

This saves us from trying to optimise for an operation that cannot
happen as we cannot replace the destination buffer.

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