Eric Anholt [Fri, 13 Jun 2008 17:04:14 +0000 (10:04 -0700)]
[gem] Catch -EINTR from blocking ioctls and restart them.
Thanks to Thomas Hellstrom for catching the issue, no thanks to the kernel
developer who authoritatively told me that they would get restarted on their
own.
Eric Anholt [Fri, 13 Jun 2008 16:53:36 +0000 (09:53 -0700)]
[gem] Remove the interrupt handler for retiring requests.
This was insufficient once we started masking interrupts to only when someone
was waiting for them (and would thus retire requests themselves). It was
replaced by the retire_timer.
Eric Anholt [Fri, 13 Jun 2008 16:49:05 +0000 (09:49 -0700)]
[gem] Don't require the lock in execbuf now that it's not needed for the ring.
Eric Anholt [Thu, 12 Jun 2008 20:22:40 +0000 (13:22 -0700)]
Bump libdrm version to 2.4.0.
We need a version to depend on from the other components that require GEM and
the bufmgr code. Some interfaces will be removed before the 2.4.0 release.
Eric Anholt [Thu, 12 Jun 2008 17:56:37 +0000 (10:56 -0700)]
Don't clear USER_INTERRUPT if we're not handling it.
This fixes a regression from commit
d434b64f6a760d85295e32298a9a1f3624ee1b69
which could cause us to fail to wake up for user interrupts if we lost a race.
Eric Anholt [Wed, 11 Jun 2008 23:19:23 +0000 (16:19 -0700)]
[gem] Remove the drm_client_lock_take in set_domain.
We no longer need to use it to protect against shared ringbuffer access.
Eric Anholt [Wed, 11 Jun 2008 22:58:33 +0000 (15:58 -0700)]
Move the renaming of mm.c symbols to #defines in the header.
This reduces the diff from Mesa and reduces the illegibility of what I did.
Eric Anholt [Wed, 11 Jun 2008 22:51:17 +0000 (15:51 -0700)]
[gem] Another round of cleanups from checkpatch.pl
Eric Anholt [Wed, 11 Jun 2008 21:42:40 +0000 (14:42 -0700)]
[gem] Move potentially device-specific ioctls to the intel driver.
This is the create (may want location flags), pread/pwrite/mmap
(performance tuning hints), and set_domain (will 32 bits be enough for
everyone?) ioctls. Left in the generic set are just flink/open/close.
The 2D driver must be updated for this change, and API but not ABI is broken
for 3D. The driver version is bumped to mark this.
Eric Anholt [Wed, 11 Jun 2008 19:20:56 +0000 (12:20 -0700)]
Remove override of drm module list in preparation for merge.
Eric Anholt [Wed, 11 Jun 2008 18:28:20 +0000 (11:28 -0700)]
[gem] Remove carefully-sprinkled i915_kernel_lost_context().
They are not unnecessary since the kernel's the only thing touching the ring.
Eric Anholt [Wed, 11 Jun 2008 06:10:33 +0000 (23:10 -0700)]
[intel] Fix BUG_ON trigger in irq masking if you did on/off with irqs disabled.
Eric Anholt [Tue, 10 Jun 2008 22:30:23 +0000 (15:30 -0700)]
[gem] Manage the ringbuffer from the kernel in the GEM case.
This requires that the X Server use the execbuf interface for buffer
submission, as it no longer has direct access to the ring. This is
therefore a flag day for the gem interface.
This also adds enter/leavevt ioctls for use by the X Server. These would
get stubbed out in a modesetting implementation, but are required while
in an environment where the device's state is only managed by the DRM while
X has the VT.
Eric Anholt [Sat, 7 Jun 2008 00:13:16 +0000 (17:13 -0700)]
[gem] Don't forget to munmap in the non-bo-reuse object-freeing case.
Keith Packard [Fri, 6 Jun 2008 20:26:03 +0000 (13:26 -0700)]
[intel] remove settable use_mi_batchbuffer_start
The driver can know what hardware requires MI_BATCH_BUFFER vs
MI_BATCH_BUFFER_START; there's no reason to let user mode configure this.
Keith Packard [Fri, 6 Jun 2008 19:59:52 +0000 (12:59 -0700)]
[intel-gem] Use timers to retire requests periodically.
Without the user IRQ running constantly, there's no wakeup when the ring
empties to go retire requests and free buffers. Use a 1 second timer to make
that happen more often.
Keith Packard [Fri, 6 Jun 2008 19:58:41 +0000 (12:58 -0700)]
[intel] free the hardware status page at driver_unload
This goes with the other hardware status page patch.
Keith Packard [Fri, 6 Jun 2008 19:57:01 +0000 (12:57 -0700)]
[intel-gem] Add explicit throttle ioctl
Instead of throttling and execbuffer time, have the application ask to
throttle explicitly. This allows the throttle to happen less often, and
without holding the DRM lock.
Keith Packard [Thu, 5 Jun 2008 23:05:35 +0000 (16:05 -0700)]
[libdrm/intel] Eliminate extra dri_gem_bo_bucket_entry structure
Place the buffer reuse links right into the dri_bo_gem object.
Keith Packard [Thu, 5 Jun 2008 22:58:55 +0000 (15:58 -0700)]
[libdrm/intel] Remove unused intel_validate_entry structure
Keith Packard [Thu, 5 Jun 2008 22:58:09 +0000 (15:58 -0700)]
[libdrm/intel] Reuse entire dri_bo_gem structure
The code was discarding the dri_bo_gem structure and saving only the kernel
handle. This lost the mmap address, causing pain when the next buffer user
wanted to map the buffer.
Keith Packard [Thu, 5 Jun 2008 21:09:57 +0000 (14:09 -0700)]
[intel] Allocate hardware status page at driver load time
I couldn't get the re-allocated HWS to work on my 965GM, so I just gave up
and made it persist across the lifetime of the driver instead.
Keith Packard [Thu, 5 Jun 2008 20:49:21 +0000 (13:49 -0700)]
Ignore X server provided mmio address
Keith Packard [Thu, 5 Jun 2008 20:47:41 +0000 (13:47 -0700)]
[intel-gem] Dump error status on wait_request failure
Eric Anholt [Thu, 5 Jun 2008 15:44:46 +0000 (08:44 -0700)]
Add a function to bufmgr_fake to evict all buffers in the GTT.
This will be used by the X Server for VT switch.
Keith Packard [Wed, 4 Jun 2008 04:49:51 +0000 (21:49 -0700)]
Drop struct_mutex while waiting in drm_client_lock_take
struct_mutex cannot be held while blocking on DRM lock.
Eric Anholt [Tue, 3 Jun 2008 21:34:54 +0000 (14:34 -0700)]
Fix libdrm to actually include the new code instead of just building it.
Eric Anholt [Tue, 3 Jun 2008 17:20:49 +0000 (10:20 -0700)]
Fix and hook up bufmgr code to the build.
Eric Anholt [Tue, 3 Jun 2008 16:27:37 +0000 (09:27 -0700)]
Import bufmgr code to libdrm. Not yet hooked up to the build.
Keith Packard [Mon, 2 Jun 2008 19:37:10 +0000 (12:37 -0700)]
[intel-gem] reloc_and_validate_object → object_bind_and_relocate
Just renaming this function and related parameters to match terminology used
elsewhere in the driver.
Keith Packard [Mon, 2 Jun 2008 17:59:15 +0000 (10:59 -0700)]
[intel-gem] Propagate set_domain errors.
set_domain can block waiting for rendering to complete. If that process is
interrupted by a signal, it can return -EINTR. Catch this error in all
callers and correctly deal with the result.
Eric Anholt [Fri, 30 May 2008 21:42:08 +0000 (14:42 -0700)]
Merge commit 'origin/master' into drm-gem
Conflicts:
linux-core/Makefile.kernel
shared-core/i915_drv.h
shared-core/nouveau_state.c
Eric Anholt [Fri, 30 May 2008 20:47:34 +0000 (13:47 -0700)]
[intel-gem] Only update obj->write_domain if we're actually changing it.
The problem was revealed where on 965, the display list vertex buffer would see:
create -> (CPU, CPU)
set_domain (CPU, CPU) -> (CPU, CPU)
set_comain (CPU, 0) -> (CPU, 0) (no clflush occurred)
execbuf (GPU, 0) -> (CPU+GPU, 0) (still no clflush)
instead of:
create -> (CPU, CPU)
set_domain (CPU, CPU) -> (CPU, CPU)
set_comain (CPU, 0) -> (CPU, CPU)
execbuf (GPU, 0) -> (CPU+GPU, 0) (clflushed)
Eric Anholt [Fri, 30 May 2008 17:04:22 +0000 (10:04 -0700)]
[intel-gem] Add an option to check GTT versus CPU coherency at execbuf time.
Dave Airlie [Fri, 30 May 2008 10:27:31 +0000 (20:27 +1000)]
r500: attempt to make AGP work by programming agp base in the MC correctly
Eric Anholt [Thu, 29 May 2008 19:53:13 +0000 (12:53 -0700)]
[intel-gem] Write the presumed_offset back out after updating it.
Otherwise, 965 constant state buffers get re-relocated every exec. Ouch.
Keith Packard [Tue, 27 May 2008 23:49:49 +0000 (16:49 -0700)]
[intel-gem] Clean up active/inactive/flushing list debugging.
Dave Airlie [Wed, 28 May 2008 01:12:57 +0000 (11:12 +1000)]
radeon: split microcode out into a separate header file.
Eric Anholt [Wed, 28 May 2008 00:50:39 +0000 (17:50 -0700)]
[intel-gem] Replace idlelock usage with real lock acquisition.
Dave Airlie [Wed, 28 May 2008 00:28:13 +0000 (10:28 +1000)]
i915: fix BSD bh, DRI2 not uses anywhere else
Dave Airlie [Wed, 28 May 2008 00:02:20 +0000 (10:02 +1000)]
radeon: bump release date/version for r500 3D support
Alex Deucher [Tue, 27 May 2008 22:33:33 +0000 (18:33 -0400)]
RADEON: add get_param for number of GB pipes
Owain Ainsworth [Tue, 27 May 2008 22:12:35 +0000 (15:12 -0700)]
[BSD] Move unlock in drm_vm.c from accidental platform #ifdeffing.
Also remove an unreachable unlock.
Owain Ainsworth [Tue, 27 May 2008 22:11:25 +0000 (15:11 -0700)]
[BSD] Fix lock leak in drm_update_draw malloc failure path.
Owain Ainsworth [Tue, 27 May 2008 22:07:04 +0000 (15:07 -0700)]
[BSD] Fix lock leaks in error paths in drm_bufs.c.
Owain Ainsworth [Tue, 27 May 2008 21:59:38 +0000 (14:59 -0700)]
[BSD] Remove superfluous recursive locking in drm_add_magic.
Jie Luo [Tue, 27 May 2008 21:55:01 +0000 (14:55 -0700)]
[i915] Fix typo in (unused) START_ADDR definition.
Robert Noland [Fri, 23 May 2008 18:36:05 +0000 (14:36 -0400)]
[FreeBSD] Add vblank-rework support and get drivers building.
The i915 driver now works again.
Eric Anholt [Tue, 27 May 2008 21:12:51 +0000 (14:12 -0700)]
[FreeBSD] Convert from drm_device_t to struct drm_device for consistency.
Keith Packard [Tue, 27 May 2008 00:41:46 +0000 (17:41 -0700)]
[intel-gem] Must hold DRM lock while setting object domain
Object domain transfer can involve adding flush ops to the request queue,
and so the DRM lock must be held to avoid having the X server smash pointers
badly.
Keith Packard [Mon, 26 May 2008 10:25:16 +0000 (03:25 -0700)]
[i915] leave interrupts masked off when not in use.
The interrupt enable register cannot be used to temporarily disable
interrupts, instead use the interrupt mask register.
Note that this change means that a pile of buffers will be left stuck on the
chip as the final interrupts will not be recognized to come and drain things.
Keith Packard [Mon, 26 May 2008 03:45:20 +0000 (20:45 -0700)]
[intel-gem] Add DRM_I915_GEM_BUSY ioctl to check for idle buffers.
This new ioctl returns whether re-using the buffer would force a wait.
Keith Packard [Mon, 26 May 2008 03:44:19 +0000 (20:44 -0700)]
[intel-gem] Compute npages instead of nbytes in flush_pwrite
i915_gem_flush_pwrite optimizes short writes to the buffer by clflushing
only the modified pages, but it was miscomputing the number of pages.
Keith Packard [Mon, 26 May 2008 03:41:42 +0000 (20:41 -0700)]
[intel-gem] replace call to jiffies_to-msec with simple inline
Keith Packard [Fri, 23 May 2008 06:08:38 +0000 (23:08 -0700)]
[intel-gem] Encourage multiple caches to hold read data
When reading from multiple domains, allow each cache to continue
to hold data until writes occur somewhere. This is done by
first leaving the read_domains alone at bind time (presumably the CPU read
cache contains valid data still) and then in set_domain, if no write_domain
is specified, the new read domains are simply merged into the existing read
domains.
A huge comment was added above set_domain to explain how things are
expected to work.
Keith Packard [Thu, 22 May 2008 18:34:56 +0000 (11:34 -0700)]
[gem] Use CPU domain for new or pageable objects
Newly allocated objects need to be in the CPU domain as they've just been
cleared by the CPU. Also, unmapping objects from the GTT needs to put them
into the CPU domain, both to flush rendering as well as to ensure that any
paging action gets flushed before we remap to the GTT.
Keith Packard [Thu, 22 May 2008 17:59:59 +0000 (10:59 -0700)]
[intel-gem] Force ring retire by emiting flush before user-interrupt.
Commands in the ring are parsed and started when the head pointer passes by
them, but they are not necessarily finished until a MI_FLUSH happens. This
patch inserts a flush after the execbuffer (the only place a flush wasn't
already happening).
Keith Packard [Thu, 22 May 2008 17:48:32 +0000 (10:48 -0700)]
[intel] Add debug code to verify the cached ring tail pointer.
Recording the tail pointer in a local variable improves performance, but if
someone messes up and fails to reload at the right time, the driver will
write commands to the wrong part of the ring and scramble execution badly.
This change (available by setting I915_RING_VALIDATE to 1) checks to make
sure the cached tail pointer matches the hardware tail pointer at each ring
buffer addition, calling BUG_ON when that's not true.
Keith Packard [Wed, 21 May 2008 07:32:02 +0000 (00:32 -0700)]
[intel-gem] invalidate ring locals for pin/unpin/set_domain/free functions
Ring locals must be reloaded from hardware in case the X server ran.
Eric Anholt [Wed, 14 May 2008 22:13:14 +0000 (15:13 -0700)]
[gem] Release GEM buffers from work task scheduled from IRQ.
There are now 3 lists. Active is buffers currently in the ringbuffer.
Flushing is not in the ringbuffer, but needs a flush before unbinding.
Inactive is as before. This prevents object_free → unbind →
wait_rendering → object_reference and a kernel oops about weird refcounting.
This also avoids an synchronous extra flush and wait when freeing a buffer
which had a write_domain set (such as a temporary rendered to and then from
using the 2d engine). It will sit around on the flushing list until the
appropriate flush gets emitted, or we need the GTT space for another
operation.
Dave Airlie [Thu, 22 May 2008 23:39:54 +0000 (09:39 +1000)]
r500: add two more register ranges for mesa driver to setup
Dave Airlie [Thu, 15 May 2008 01:13:03 +0000 (11:13 +1000)]
drm: fix nouveau warning
Eric Anholt [Tue, 20 May 2008 23:27:05 +0000 (16:27 -0700)]
[gem] Replace ring throttling hack with actual time measurement.
Eric Anholt [Wed, 21 May 2008 22:15:58 +0000 (15:15 -0700)]
[gem] Fix bad test for list_for_each completion.
Since it's a circular list, the entry won't be NULL at termination.
Eric Anholt [Wed, 21 May 2008 22:03:47 +0000 (15:03 -0700)]
[gem] Hold a reference on the object in i915_gem_wait_space.
Otherwise, in the middle of the function called using it the last ref
might disappear.
Keith Packard [Wed, 21 May 2008 07:32:02 +0000 (00:32 -0700)]
[intel-gem] invalidate ring locals for pin/unpin/set_domain/free functions
Ring locals must be reloaded from hardware in case the X server ran.
Dave Airlie [Wed, 21 May 2008 10:14:45 +0000 (20:14 +1000)]
rs690/r500: vblank support.
The new display controller has the vblank interrupts in a different place.
Add support for vbl interrupts for these chips
Eric Anholt [Tue, 20 May 2008 21:03:27 +0000 (14:03 -0700)]
[gem] Use a separate sequence number field from classic/ttm
This lets us get some qualities we desire, such as using the full 32-bit
range (except zero), avoiding DRM_WAIT_ON, and a 1:1 mapping of active
sequence numbers to request structs, which will be used soon for throttling
and interrupt-driven list cleanup.
Eric Anholt [Tue, 20 May 2008 17:52:24 +0000 (10:52 -0700)]
[gem] Rename sequence numbers from "cookie" to "seqno"
Eric Anholt [Tue, 20 May 2008 17:48:36 +0000 (10:48 -0700)]
[gem] Clean up active/inactive list handling using helper functions.
Additionally, a boolean active field is added to indicate which list an
object is on, rather than smashing last_rendering_cookie to 0 to show
inactive. This will help with flush-reduction later on, and makes the code
clearer.
Dave Airlie [Sat, 17 May 2008 00:22:12 +0000 (10:22 +1000)]
r500: add more register ranges for Mesa driver
Eric Anholt [Thu, 15 May 2008 18:21:11 +0000 (11:21 -0700)]
[gem] Hold dev->struct_mutex to protect structure data.
Eric Anholt [Thu, 15 May 2008 16:37:49 +0000 (09:37 -0700)]
[gem] Rename the GTT LRU lists to active (executing) and inactive (idle).
Eric Anholt [Thu, 15 May 2008 16:32:34 +0000 (09:32 -0700)]
[gem] typo fix in comment.
Dave Airlie [Wed, 14 May 2008 12:48:12 +0000 (22:48 +1000)]
ati_pcigart: oops wrong way around not that it actually mattered
Dave Airlie [Wed, 14 May 2008 12:44:22 +0000 (22:44 +1000)]
ati_pcigart: stop working in the evenings you mess up too often
Dave Airlie [Wed, 14 May 2008 12:43:28 +0000 (22:43 +1000)]
Revert "ati_pcigart: fixup properly this version might even work"
This reverts commit
bc0836e12a9790f1cc83f8bc29bc05043c4bc840.
tree has some kref hacks in it - oops
Dave Airlie [Wed, 14 May 2008 12:42:21 +0000 (22:42 +1000)]
ati_pcigart: fixup properly this version might even work
Dave Airlie [Wed, 14 May 2008 12:35:32 +0000 (22:35 +1000)]
ati_pcigart: fill out 40-bit gart table support properly
Thanks to Alex for supplying this info.
Alex Deucher [Wed, 14 May 2008 01:02:17 +0000 (21:02 -0400)]
RS4xx: separate out RS400 and RS480 IGP chips
RS400 (intel based IGP) and RS480 (AMD based IGP) have
different MC and GART setups. Currently we only support
RS480.
Eric Anholt [Mon, 12 May 2008 22:42:20 +0000 (15:42 -0700)]
[GEM] Update testcases for new API.
Eric Anholt [Mon, 12 May 2008 19:55:36 +0000 (12:55 -0700)]
[GEM] Typo (and thinking) fixes in drm-gem.txt and doxygen.
Keith Packard [Mon, 12 May 2008 20:04:18 +0000 (13:04 -0700)]
[intel] Minor kludge -- wait for the ring to be nearly empty before queuing
No need to fill the ring that much; wait for it to become nearly empty
before adding the execbuffer request. A better fix will involve scheduling
ring insertion in the irq handler.
Keith Packard [Mon, 12 May 2008 20:03:16 +0000 (13:03 -0700)]
[intel] When polling for ring space, sleep for a lot longer (10ms)
If the ring is full, the engine will surely be running for more than 10ms.
Keith Packard [Mon, 12 May 2008 20:01:57 +0000 (13:01 -0700)]
[gem] Set write domain to CPU when doing pwrite.
Leave the flush call in place, which can fix domains up if necessary.
Keith Packard [Mon, 12 May 2008 20:00:55 +0000 (13:00 -0700)]
[gem] Clarify use of explicit domain control. Remove Gen3 from I-cache usage.
Alex Deucher [Mon, 12 May 2008 18:49:43 +0000 (14:49 -0400)]
RADEON: fix copy/pasto in last commit
Alex Deucher [Mon, 12 May 2008 13:44:20 +0000 (09:44 -0400)]
R3/4/5: init pipe setup in drm
Similar (broken) code in mesa needs to be removed
Alex Deucher [Mon, 12 May 2008 13:35:06 +0000 (09:35 -0400)]
RADEON: cleanup radeon_do_engine_reset()
Alex Deucher [Mon, 12 May 2008 13:30:47 +0000 (09:30 -0400)]
R300+: fixup pixcache flush
Alex Deucher [Mon, 12 May 2008 13:24:13 +0000 (09:24 -0400)]
RS4xx: fix MCIND index mask
Alex Deucher [Mon, 12 May 2008 13:21:45 +0000 (09:21 -0400)]
RADEON: write AGP_BASE_2 on chips that support it
Alex Deucher [Mon, 12 May 2008 13:18:28 +0000 (09:18 -0400)]
R300+: fixup PURGE/FLUSH macros
Alex Deucher [Mon, 12 May 2008 13:13:44 +0000 (09:13 -0400)]
Radeon IGP: merge RS4xx/RS6xx gart setup
Alex Deucher [Mon, 12 May 2008 13:00:40 +0000 (09:00 -0400)]
Radeon IGP: wrap MCIND access
first step in merging rs4xx/rs6xx gart setup
Alex Deucher [Mon, 12 May 2008 12:56:11 +0000 (08:56 -0400)]
Radeon IGP: clean up registers and magic numbers
Keith Packard [Sun, 11 May 2008 07:10:16 +0000 (00:10 -0700)]
[GEM] Make pread/pwrite manage memory domains. No luck with movnti though.
pread and pwrite must update the memory domains to ensure consistency with
the GPU. At some point, it should be possible to avoid clflush through this
path, but that isn't working for me.
Keith Packard [Sun, 11 May 2008 05:04:39 +0000 (22:04 -0700)]
[intel-GEM] exec list can contain pinned, lru cannot.
The exec list contains all objects, in order of use. The lru list contains
only unpinned objects ready to be evicted. This required two changes -- the
first was to not migrate pinned objects from exec to lru, the second was to
search for the first unpinned object in the exec list when doing eviction.
Keith Packard [Sun, 11 May 2008 04:05:25 +0000 (21:05 -0700)]
Merge commit 'anholt/drm-gem' into drm-gem
Keith Packard [Sun, 11 May 2008 04:04:42 +0000 (21:04 -0700)]
[GEM] Add drm-gem.txt
Add some API and implementation documentation for GEM.