nouveau: Unreference pushbuf objects on channel destruction.

- unreference pushbuf objects on channel destruction

Based on Krzysztof Smiechowicz's patch.

nouveau: Update some object definitions from renouveau.xml.

Signed-off-by: Francisco Jerez <currojerez@riseup.net>

nouveau: Update nouveau_class.h with new renouveau.xml definitions.

Signed-off-by: Francisco Jerez <currojerez@riseup.net>

libdrm 2.4.17

radeon: fix BO null check, should be in higher level fn

radeon: straighten out the API insanity.

as Michel pointed out we are exposing too much info for these object
for this to be maintainable going forward.

This patch set minimises the exposed parts of the radeon_bo and
radeon_cs objects to the piece necessary for ddx/mesa to operate
at a decent speed.

The major problem is mesa contains a legacy BO/CS managers which we still
need to expose functionality to, and we really cannot change the API
until we can drop the non-KMS codepaths.

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

nouveau: remove delayed kernel bo creation

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

Be less chatty in drmSetMaster/drmDropMaster

modetest: Error out if pageflipping is requested but not available

nouveau: Use drmIoctl so we restart ioctl on EINTR or EAGAIN

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

nouveau: move reloc code down, nothing to see here

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

intel: Clear virtual after failing to mmap_gtt.

Don't store the error return in bo_gem->gtt_virtual or else we will
attempt to use that as a valid pointer in future mappings.

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

Ignore config.h.in

radeon: Use drmIoctl so we restart ioctl on EINTR or EAGAIN

This is needed as change in kernel will lead to ioctl returning
EINTR if they are interrupted.

Signed-off-by: Jerome Glisse <jglisse@redhat.com>

intel: Expect caller to guarantee thread-safety of bo during reloc

This removes the foremost prolific user of mutexes in libdrm_intel.so.
The other uses of the bufmgr_gem->mutex to serial access to individual
bos are currently required by Mesa, and are far less frequent.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
[anholt: This chunk looks good...]
Acked-by: Eric Anholt <eric@anholt.net>

modetest: fix build error due to page_flip_handler name change

Forgot to update this when pushing the pageflip bits.

Merge branch 'modesetting-dirty-libdrm'

Conflicts:
include/drm/drm.h

Add RELEASING to document the release process

Bump event context structure version for page flipping

Merge branch 'pageflip' of git://people.freedesktop.org/~jbarnes/drm

Conflicts:
include/drm/drm.h - RMFB had its signature changed to avoid uint32_t

Bump to 2.4.16 for release

Enable experimental APIs for distcheck

Change the number on the dirty ioctl to match upstream

intel: Free memory before inserting bo into cache.

This has the unfortunate behaviour of releasing our malloc cache, but
the alternative is for X to consume a couple of gigabytes of ram and
die during testing. Fortunately the extra mallocs have little impact on
performance whereas avoiding swap and death, lots.

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

intel: Check and propagate errors from building reloc-tree

Instead of forcing the caller to check after every emit_reloc(), we can
flag the object as being in error, propagating that error upwards through
the relocation tree, and failing the eventual batch buffer execution.

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

intel: Repeat execbuffer after EINTR

EAGAIN cannot be raised by the current code, but the system call maybe
interrupted and so return EINTR.

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

intel: Review use of errno.

Hitting this error lead to a segfault:

  intel_bufmgr_gem.c:919: Error mapping buffer 48607 (pixmap):
                          Cannot allocate memory.

because the errno was reused as the function return value after being
reset by the fprintf(), so caller thought the mapping had succeeded. The
convention established by libdrm is that the return value is the
negative errno and that uses of libdrm cannot trust the value of errno
afterwards, but must use the return code.

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

intel: Make bo_reference() inline for internal use.

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

intel: Remove the extra reference while validating the reloc tree

Buffers on the relocation tree are guarded by the reference to the batch
object and so do not need an extra reference whilst constructing the
list of execution buffer objects.

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

intel: Wrap a few more syscalls with EINTR protection

Having been bitten by a missing EINTR check during mmap_gtt(), I thought
it prudent to add some more protection around the ioctls.

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

intel: Clear bo->used_as_reloc_target flag on destroy

This allows us to keep the assert added in the previous commit that we do
not modify the tree_reloc_size after inserting the buffer into a relocation
tree, which was being hit here:

  #0  0xb78c2424 in __kernel_vsyscall ()
  #1  0xb74f6401 in raise () from /lib/libc.so.6
  #2  0xb74f7b42 in abort () from /lib/libc.so.6
  #3  0xb74ef5a8 in __assert_fail () from /lib/libc.so.6
  #4  0xb737e78b in drm_intel_bo_gem_set_in_aperture_size (bufmgr_gem=<value optimized out>, bo_gem=0x6) at intel_bufmgr_gem.c:373
  #5  0xb737f519 in drm_intel_gem_bo_set_tiling (bo=0xa1030a0, tiling_mode=0xbff6c85c, stride=0) at intel_bufmgr_gem.c:1386
  #6  0xb737f67f in drm_intel_gem_bo_unreference_final (bo=0xa1030a0, time=<value optimized out>) at intel_bufmgr_gem.c:768
  #7  0xb737f5e3 in drm_intel_gem_bo_unreference_locked_timed (bo=0xa1e50d0, time=<value optimized out>) at intel_bufmgr_gem.c:805
  #8  drm_intel_gem_bo_unreference_final (bo=0xa1e50d0, time=<value optimized out>) at intel_bufmgr_gem.c:756
  #9  0xb737fcbb in drm_intel_gem_bo_unreference (bo=0xa1e50d0) at intel_bufmgr_gem.c:821
  #10 0xb737b4e6 in drm_intel_bo_unreference (bo=0x0) at intel_bufmgr.c:80
  #11 0xb7325625 in intel_batch_flush (scrn=0x9d91f78, flush=1) at i830_batchbuffer.c:200

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

intel: Apply pessimistic alignment to in-aperture buffer size

For the older chipsets, i.e. pre-i965, which have severe alignment
restrictions for tiled buffers we need to pessimistically assume that we
will waste the size of buffer to meet those alignment constraints.

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

intel: Only store a buffer in the cache if it is retained.

If the kernel immediately frees the backing store for a buffer when
marking it purgeable, then there is not point adding to the cache. Free
it immediately, instead.

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

Change the dirty ioctl a bit and comment it

Bring dirty code from old branch

Correctly set DRM_MAX_MINOR for all platforms.

DRM_MAJOR is platform specific, but not used outside of xf86drm.c
that I can find.

Add missing DRM_MAX_MINOR define

lost in 500f5b524000ed5930301f4303744cb4c0a19b75

Signed-off-by: Alex Deucher <alexdeucher@gmail.com>

More fixups to allow mesa to build

Finish fixing the build on FreeBSD

Add drmGetDeviceNameFromFd function

Determines the /dev filename of the drm fd argument.

Fix build on *BSD

This adds a minimal #ifdef clause to drm.h that we'll push upstream.
Once that goes in we can share drm.h between linux, libdrm, and
the bsd's.

Don't hardcore 'yes', use  in configure.ac output

Oops, I assumed intel was always enable, but it just defaults to on.

Output summary of enabled features at the end of configure.ac

Drop duplicated radeon_*.h files in include/drm

These files are userspace headers and live in radeon/

Put mach64_drm.h back in to avoid breaking mesa build

We may want to drop mach64 from mesa instead, but that's a different
discussion.

Install kernel headers in ${includedir}/drm

nouveau: fix DRM headers

The nouveau_drmif.h is not the Nouveau DRM kernel ABI file, but purely
user space stuff. Remove it, it does not belong in include/drm/.

Copy the right header from Nouveau kernel v2.6.31-rc9-757-gaca551c.

Signed-off-by: Pekka Paalanen <pq@iki.fi>

Bump libdrm version to 2.4.16 for page flipping

modetest: add pageflip test case to modetest

libdrm: add libdrm support for page flip ioctl

Merge remote branch 'origin/master' into libdrm

Update README

libdrm_radeon: add radeon_bo_is_referenced_by_cs function

Signed-off-by: Alex Deucher <alexdeucher@gmail.com>

drm/i915: add GETPARAM request for page flipping

Drop stale TODO and unused ChangeLog

Move libdrm/ up one level

Drop shared-core, bsd-core, linux-core and scripts subdirs

Use headers copied from kernel instead of shared-core

Fix typo in i915 pipe_from_crtc_id ioctl struct name

Copy headers from kernel v2.6.32-rc6-130-g5b8f0be

Add makefile rule to copy headers from kernel tree

nouveau: Update some nouveau_class.h definitions from renouveau.xml.

Signed-off-by: Francisco Jerez <currojerez@riseup.net>
Signed-off-by: Pekka Paalanen <pq@iki.fi>

intel: Export madvise

Wrap the madvise ioctl for use in APPLE_object_purgeable.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Acked-by: Jesse Barnes <jbarnes@virtuousgeek.org>

Only install libdrm_intel.pc if we build libdrm_intel

nouveau: turn another assertion into an error

Signed-off-by: Ben Skeggs <skeggsb@beleth.(none)>

nouveau: modify api slightly to allow caller to handle reloc failures

Signed-off-by: Ben Skeggs <skeggsb@beleth.(none)>

nouveau: add reloc refcnt to pending bo list

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

nouveau: drop rendering on the floor rather than asserting if flush fails

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

nouveau: update view of available aperture space after each flush

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

radeon: fix allocation

The old code increments the command stream size by another kbyte, but does
not make sure that the requested packet size fits into the stream. The patch
ensures that the whole next packet fits there and rounds the allocated size to
a power of two.

Signed-off-by: Alex Deucher <alexdeucher@gmail.com>

Add support for vblank events

The kernel will now write data to the DRM fd for various event types if
requested.  Currently, the only supported event is a vblank event: it contains
the vblank count for the event as well as a timestamp from when the event
ocurred.  Since the DRM fd is now pollable, it's easy to integrate into
existing event loops.

intel: Only call clock_gettime once per unreference_final.

Notably when freeing a batchbuffer, we often end up freeing many of the
buffers it points at as well.  Avoiding repeated calls brings us a 9% CPU
win for cairo-gl.

[ # ]  backend                         test   min(s) median(s) stddev. count
before:
[  0]       gl            firefox-talos-gfx   58.941   58.966   0.75%    3/3
after:
[  0]       gl            firefox-talos-gfx   54.186   54.195   0.49%    3/3

intel: Improve bo_references performance by skipping the tree walk.

If the target we're asking about hasn't ever been used as a relocation
target, then it obviously hasn't been used as a target by the batch's reloc
tree.  This is the common case for good GL programming where you only map
fresh buffers, and gives us a 5% win in cairo-gl.

[ # ]  backend                         test   min(s) median(s) stddev. count
before:
[  0]       gl            firefox-talos-gfx   64.680   64.756   0.06%    3/3
after:
[  0]       gl            firefox-talos-gfx   60.816   60.970   0.29%    3/3

nouveau: retry if pushbuf ioctl interrupted by signal

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

nouveau: function to return status of bo being referenced in pushbuf

Will be used to implement gallium's is_{texture,buffer}_referenced()
callbacks properly.

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

nouveau: nv30: add render target logbase2 width,height

intel: Add the defines for the kernel overlay support landing in 2.6.33.

intel: Fallback to atomic-ops.h [libatomic-ops-dev]

Use the external implementation for atomic operations across a wide
range of architectures.

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

configure: Typo in error message.

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

configure: Conditionally build libdrm_intel

Only build libdrm_intel automatically if we have support for atomic
operations. To force configure to build drm pass --enable-intel, which
will cause the configure to error if no support is found. Or pass
--disable-intel to explicitly prevent libdrm_intel from being built.

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

Bump to 2.4.15 for release.

intel: Remove the asserts about the ignored alignment parameter.

I slipped it in with the alloc_tiled changes, since we were explicitly
throwing the parameter away.  It caught some bogus released code, which
we've now fixed, so remove the asserts to keep old drivers working.

intel: Add a bo_alloc function for tiled BOs.

This simplifies driver code in handling object allocation, and also gives us
an opportunity to possibly cache tiled buffers if it turns out to be a win.

[anholt: This is chopped out of the execbuf2 patch, as it seems to be useful
separately and cleans up the execbuf2 changes to be more obvious]

intel: Fix up some stale doxygen comments.

intel: Reformat to the kernel coding style.  Welcome to the 8-space future.

This is done with:
Lindent *.[ch]
perl -pi -e 's|drm_intel_bo \* |drm_intel_bo *|g' *.[ch]
perl -pi -e 's|drm_intel_bufmgr \* |drm_intel_bufmgr *|g' *.[ch]
perl -pi -e 's|drm_intel_bo_gem \* |drm_intel_bo_gem *|g' *.[ch]
perl -pi -e 's|drm_intel_bufmgr_gem \* |drm_intel_bufmgr_gem *|g' *.[ch]
perl -pi -e 's|_fake \* |_fake *|g' *.[ch]
hand-editing to whack indented comments into line and other touchups.

intel: Don't allocate more relocation entries than the BO could support.

This saves 32k of relocation entry storage for many 965 state buffers.  No
noticeable impact on performance for cairo-gl firefox.

tests: Disable intel-specific tests with --disable-intel

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

intel: Add a configure option to *disable* building libdrm-intel

In conjunction with the atomic operation patch, it may be more
convenient for some people to disable building libdrm-intel and its
dependencies upon the atomic intrinsics then it is for them to use a
supported compiler.

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

intel: report errno

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

libdrm_radeon: Update RADEON_TILING_* flags to what's in current kernels.

libdrm_radeon: Zero-initialize structures to silence valgrind warnings

Signed-off-by: Nicolai Hähnle <nhaehnle@gmail.com>

intel: Use atomic refcounters

As the target architecture for Intel GPUs is the x86, we can presume to
have reasonable compiler support for Intel atomic intrinsics, i.e. gcc,
and so use those in preference to pulling in a complicated mess of
fragile assembly.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
[anholt: hand-resolved against my previous commit.  This brings cairo-gl
firefox-talos-gfx time from 65 seconds back down to 62 seconds.]
Signed-off-by: Eric Anholt <eric@anholt.net>

intel: Mark cached bo as purgeable

Set the DONTNEED flag on cached buffers so that the kernel is free to
discard those when under memory pressure.

[anholt: This takes firefox-talos-gfx time from ~62 seconds to ~65 seconds
on my GM965, but it seems like a hit worth taking for the improved
functionality from saving memory]

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Eric Anholt <eric@anholt.net>

intel: Don't free the reloc list when putting a freed BO in the cache.

This takes firefox-talos-gfx from 74 seconds to 70 seconds on my GM965.

intel: Add a new function to check if a BO's reloc tree references some BO.

There are a bunch of places in GL where if we can't do this we have to
flush the batchbuffer, and the cost of lookups here is outweighed by flush
savings.

Bump to 2.4.14 for release.

intel: Remove the max_entries stuff that complicated bo reuse.

I thought I was going to do all sorts of crazy experiments with it.  I never
did, and it turned out the free-after-a-few-seconds plan is working out fine.

intel: Remove the old swrast flag for reducing cache flushing.

It hasn't been doing anything effective since
52e5d24fae4af6f2f4a5304a516c8c5ab347a11b, and we pretty much don't bo_map
pinned buffers any more anyway.

drmModeFreeConnector: free encoders and properties

These were leaking.

Signed-off-by: Keith Packard <keithp@keithp.com>

radeon: fix 32/64 bit issue with sign extension

Not sure what intptr_t was up to here.

Reported and tested by: Kevin DeKorte
Signed-off-by: Dave Airlie <airlied@redhat.com>