From: John Harrison <John.C.Harrison@Intel.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: Intel-GFX@Lists.FreeDesktop.Org
Subject: Re: [RFC 06/21] drm/i915: Replace last_[rwf]_seqno with last_[rwf]_req
Date: Mon, 20 Oct 2014 16:58:55 +0100 [thread overview]
Message-ID: <5445313F.3040108@Intel.com> (raw)
In-Reply-To: <20141019124006.GU26941@phenom.ffwll.local>
On 19/10/2014 13:40, Daniel Vetter wrote:
> On Mon, Oct 06, 2014 at 03:15:10PM +0100, John.C.Harrison@Intel.com wrote:
>> From: John Harrison <John.C.Harrison@Intel.com>
>>
>> For: VIZ-4377
>> Signed-off-by: John.C.Harrison@Intel.com
>> ---
>> drivers/gpu/drm/i915/i915_debugfs.c | 6 +--
>> drivers/gpu/drm/i915/i915_drv.h | 6 +--
>> drivers/gpu/drm/i915/i915_gem.c | 66 ++++++++++++++++------------
>> drivers/gpu/drm/i915/i915_gem_execbuffer.c | 6 +--
>> drivers/gpu/drm/i915/i915_gem_gtt.h | 4 +-
>> drivers/gpu/drm/i915/i915_gem_tiling.c | 2 +-
>> drivers/gpu/drm/i915/i915_gpu_error.c | 4 +-
>> drivers/gpu/drm/i915/intel_display.c | 10 ++---
>> drivers/gpu/drm/i915/intel_ringbuffer.h | 2 +-
>> 9 files changed, 58 insertions(+), 48 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
>> index 063b448..726a8f0 100644
>> --- a/drivers/gpu/drm/i915/i915_debugfs.c
>> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
>> @@ -133,9 +133,9 @@ describe_obj(struct seq_file *m, struct drm_i915_gem_object *obj)
>> obj->base.size / 1024,
>> obj->base.read_domains,
>> obj->base.write_domain,
>> - obj->last_read_seqno,
>> - obj->last_write_seqno,
>> - obj->last_fenced_seqno,
>> + i915_gem_request_get_seqno(obj->last_read_req),
>> + i915_gem_request_get_seqno(obj->last_write_req),
>> + i915_gem_request_get_seqno(obj->last_fenced_req),
>> i915_cache_level_str(to_i915(obj->base.dev), obj->cache_level),
>> obj->dirty ? " dirty" : "",
>> obj->madv == I915_MADV_DONTNEED ? " purgeable" : "");
>> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
>> index 62c9f66..1401266 100644
>> --- a/drivers/gpu/drm/i915/i915_drv.h
>> +++ b/drivers/gpu/drm/i915/i915_drv.h
>> @@ -1862,10 +1862,10 @@ struct drm_i915_gem_object {
>> struct intel_engine_cs *ring;
>>
>> /** Breadcrumb of last rendering to the buffer. */
>> - uint32_t last_read_seqno;
>> - uint32_t last_write_seqno;
>> + struct drm_i915_gem_request *last_read_req;
>> + struct drm_i915_gem_request *last_write_req;
>> /** Breadcrumb of last fenced GPU access to the buffer. */
>> - uint32_t last_fenced_seqno;
>> + struct drm_i915_gem_request *last_fenced_req;
>>
>> /** Current tiling stride for the object, if it's tiled. */
>> uint32_t stride;
>> diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
>> index 2555cd8..2c33a83 100644
>> --- a/drivers/gpu/drm/i915/i915_gem.c
>> +++ b/drivers/gpu/drm/i915/i915_gem.c
>> @@ -1289,11 +1289,11 @@ i915_gem_object_wait_rendering__tail(struct drm_i915_gem_object *obj)
>> /* Manually manage the write flush as we may have not yet
>> * retired the buffer.
>> *
>> - * Note that the last_write_seqno is always the earlier of
>> - * the two (read/write) seqno, so if we haved successfully waited,
>> + * Note that the last_write_req is always the earlier of
>> + * the two (read/write) requests, so if we haved successfully waited,
>> * we know we have passed the last write.
>> */
>> - obj->last_write_seqno = 0;
>> + obj->last_write_req = NULL;
>>
>> return 0;
>> }
>> @@ -1306,14 +1306,18 @@ static __must_check int
>> i915_gem_object_wait_rendering(struct drm_i915_gem_object *obj,
>> bool readonly)
>> {
>> + struct drm_i915_gem_request *req;
>> struct intel_engine_cs *ring = obj->ring;
>> u32 seqno;
>> int ret;
>>
>> - seqno = readonly ? obj->last_write_seqno : obj->last_read_seqno;
>> - if (seqno == 0)
>> + req = readonly ? obj->last_write_req : obj->last_read_req;
>> + if (!req)
>> return 0;
>>
>> + seqno = i915_gem_request_get_seqno(req);
>> + BUG_ON(seqno == 0);
> Again, you like BUG_ON a bit too much for my taste. If you want these
> checks imo a WARN_ON in i915_gem_request_get_seqno (iff req != NULL ofc)
> in the previous patch would be much better.
> -Daniel
Again a) this patch set was posted as a work in progress, I would like
some quick feedback before attempting to finish it, polish it, flush out
the corner cases, etc. And for internal development debug, BUG_ONs are a
lot more use than WARN_ON. And b) this will disappear in a later patch
when the seqno is removed completely and 'i915_wait_seqno(seqno)'
becomes 'i915_wait_request(request)'.
>> +
>> ret = i915_wait_seqno(ring, seqno);
>> if (ret)
>> return ret;
>> @@ -1329,6 +1333,7 @@ i915_gem_object_wait_rendering__nonblocking(struct drm_i915_gem_object *obj,
>> struct drm_i915_file_private *file_priv,
>> bool readonly)
>> {
>> + struct drm_i915_gem_request *req;
>> struct drm_device *dev = obj->base.dev;
>> struct drm_i915_private *dev_priv = dev->dev_private;
>> struct intel_engine_cs *ring = obj->ring;
>> @@ -1339,10 +1344,13 @@ i915_gem_object_wait_rendering__nonblocking(struct drm_i915_gem_object *obj,
>> BUG_ON(!mutex_is_locked(&dev->struct_mutex));
>> BUG_ON(!dev_priv->mm.interruptible);
>>
>> - seqno = readonly ? obj->last_write_seqno : obj->last_read_seqno;
>> - if (seqno == 0)
>> + req = readonly ? obj->last_write_req : obj->last_read_req;
>> + if (!req)
>> return 0;
>>
>> + seqno = i915_gem_request_get_seqno(req);
>> + BUG_ON(seqno == 0);
>> +
>> ret = i915_gem_check_wedge(&dev_priv->gpu_error, true);
>> if (ret)
>> return ret;
>> @@ -2180,12 +2188,12 @@ static void
>> i915_gem_object_move_to_active(struct drm_i915_gem_object *obj,
>> struct intel_engine_cs *ring)
>> {
>> - u32 seqno = intel_ring_get_seqno(ring);
>> + struct drm_i915_gem_request *req = intel_ring_get_request(ring);
>>
>> BUG_ON(ring == NULL);
>> - if (obj->ring != ring && obj->last_write_seqno) {
>> - /* Keep the seqno relative to the current ring */
>> - obj->last_write_seqno = seqno;
>> + if (obj->ring != ring && obj->last_write_req) {
>> + /* Keep the request relative to the current ring */
>> + obj->last_write_req = req;
>> }
>> obj->ring = ring;
>>
>> @@ -2197,7 +2205,7 @@ i915_gem_object_move_to_active(struct drm_i915_gem_object *obj,
>>
>> list_move_tail(&obj->ring_list, &ring->active_list);
>>
>> - obj->last_read_seqno = seqno;
>> + obj->last_read_req = req;
>> }
>>
>> void i915_vma_move_to_active(struct i915_vma *vma,
>> @@ -2228,11 +2236,11 @@ i915_gem_object_move_to_inactive(struct drm_i915_gem_object *obj)
>> list_del_init(&obj->ring_list);
>> obj->ring = NULL;
>>
>> - obj->last_read_seqno = 0;
>> - obj->last_write_seqno = 0;
>> + obj->last_read_req = NULL;
>> + obj->last_write_req = NULL;
>> obj->base.write_domain = 0;
>>
>> - obj->last_fenced_seqno = 0;
>> + obj->last_fenced_req = NULL;
>>
>> obj->active = 0;
>> drm_gem_object_unreference(&obj->base);
>> @@ -2249,7 +2257,7 @@ i915_gem_object_retire(struct drm_i915_gem_object *obj)
>> return;
>>
>> if (i915_seqno_passed(ring->get_seqno(ring, true),
>> - obj->last_read_seqno))
>> + i915_gem_request_get_seqno(obj->last_read_req)))
>> i915_gem_object_move_to_inactive(obj);
>> }
>>
>> @@ -2669,7 +2677,7 @@ i915_gem_retire_requests_ring(struct intel_engine_cs *ring)
>> struct drm_i915_gem_object,
>> ring_list);
>>
>> - if (!i915_seqno_passed(seqno, obj->last_read_seqno))
>> + if (!i915_seqno_passed(seqno, i915_gem_request_get_seqno(obj->last_read_req)))
>> break;
>>
>> i915_gem_object_move_to_inactive(obj);
>> @@ -2779,7 +2787,7 @@ i915_gem_object_flush_active(struct drm_i915_gem_object *obj)
>> int ret;
>>
>> if (obj->active) {
>> - ret = i915_gem_check_olr(obj->ring, obj->last_read_seqno);
>> + ret = i915_gem_check_olr(obj->ring, i915_gem_request_get_seqno(obj->last_read_req));
>> if (ret)
>> return ret;
>>
>> @@ -2838,13 +2846,14 @@ i915_gem_wait_ioctl(struct drm_device *dev, void *data, struct drm_file *file)
>> goto out;
>>
>> if (obj->active) {
>> - seqno = obj->last_read_seqno;
>> + if (!obj->last_read_req)
>> + goto out;
>> +
>> + seqno = i915_gem_request_get_seqno(obj->last_read_req);
>> + BUG_ON(seqno == 0);
>> ring = obj->ring;
>> }
>>
>> - if (seqno == 0)
>> - goto out;
>> -
>> /* Do this after OLR check to make sure we make forward progress polling
>> * on this IOCTL with a timeout <=0 (like busy ioctl)
>> */
>> @@ -2894,7 +2903,7 @@ i915_gem_object_sync(struct drm_i915_gem_object *obj,
>>
>> idx = intel_ring_sync_index(from, to);
>>
>> - seqno = obj->last_read_seqno;
>> + seqno = i915_gem_request_get_seqno(obj->last_read_req);
>> /* Optimization: Avoid semaphore sync when we are sure we already
>> * waited for an object with higher seqno */
>> if (seqno <= from->semaphore.sync_seqno[idx]) /* <--- broken?! needs to use i915_seqno_passed()??? */
>> @@ -2907,11 +2916,12 @@ i915_gem_object_sync(struct drm_i915_gem_object *obj,
>> trace_i915_gem_ring_sync_to(from, to, seqno);
>> ret = to->semaphore.sync_to(to, from, seqno);
>> if (!ret)
>> - /* We use last_read_seqno because sync_to()
>> + /* We use last_read_req because sync_to()
>> * might have just caused seqno wrap under
>> * the radar.
>> */
>> - from->semaphore.sync_seqno[idx] = obj->last_read_seqno;
>> + from->semaphore.sync_seqno[idx] =
>> + i915_gem_request_get_seqno(obj->last_read_req);
>>
>> return ret;
>> }
>> @@ -3224,12 +3234,12 @@ static void i915_gem_object_update_fence(struct drm_i915_gem_object *obj,
>> static int
>> i915_gem_object_wait_fence(struct drm_i915_gem_object *obj)
>> {
>> - if (obj->last_fenced_seqno) {
>> - int ret = i915_wait_seqno(obj->ring, obj->last_fenced_seqno);
>> + if (obj->last_fenced_req) {
>> + int ret = i915_wait_seqno(obj->ring, i915_gem_request_get_seqno(obj->last_fenced_req));
>> if (ret)
>> return ret;
>>
>> - obj->last_fenced_seqno = 0;
>> + obj->last_fenced_req = NULL;
>> }
>>
>> return 0;
>> diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
>> index 1a0611b..4250211 100644
>> --- a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
>> +++ b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
>> @@ -946,7 +946,7 @@ void
>> i915_gem_execbuffer_move_to_active(struct list_head *vmas,
>> struct intel_engine_cs *ring)
>> {
>> - u32 seqno = intel_ring_get_seqno(ring);
>> + struct drm_i915_gem_request *req = intel_ring_get_request(ring);
>> struct i915_vma *vma;
>>
>> list_for_each_entry(vma, vmas, exec_list) {
>> @@ -963,7 +963,7 @@ i915_gem_execbuffer_move_to_active(struct list_head *vmas,
>> i915_vma_move_to_active(vma, ring);
>> if (obj->base.write_domain) {
>> obj->dirty = 1;
>> - obj->last_write_seqno = seqno;
>> + obj->last_write_req = req;
>>
>> intel_fb_obj_invalidate(obj, ring);
>>
>> @@ -971,7 +971,7 @@ i915_gem_execbuffer_move_to_active(struct list_head *vmas,
>> obj->base.write_domain &= ~I915_GEM_GPU_DOMAINS;
>> }
>> if (entry->flags & EXEC_OBJECT_NEEDS_FENCE) {
>> - obj->last_fenced_seqno = seqno;
>> + obj->last_fenced_req = req;
>> if (entry->flags & __EXEC_OBJECT_HAS_FENCE) {
>> struct drm_i915_private *dev_priv = to_i915(ring->dev);
>> list_move_tail(&dev_priv->fence_regs[obj->fence_reg].lru_list,
>> diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.h b/drivers/gpu/drm/i915/i915_gem_gtt.h
>> index d5c14af..8a220c0 100644
>> --- a/drivers/gpu/drm/i915/i915_gem_gtt.h
>> +++ b/drivers/gpu/drm/i915/i915_gem_gtt.h
>> @@ -178,7 +178,7 @@ struct i915_address_space {
>> * List of objects currently involved in rendering.
>> *
>> * Includes buffers having the contents of their GPU caches
>> - * flushed, not necessarily primitives. last_rendering_seqno
>> + * flushed, not necessarily primitives. last_rendering_req
>> * represents when the rendering involved will be completed.
>> *
>> * A reference is held on the buffer while on this list.
>> @@ -189,7 +189,7 @@ struct i915_address_space {
>> * LRU list of objects which are not in the ringbuffer and
>> * are ready to unbind, but are still in the GTT.
>> *
>> - * last_rendering_seqno is 0 while an object is in this list.
>> + * last_rendering_req is NULL while an object is in this list.
>> *
>> * A reference is not held on the buffer while on this list,
>> * as merely being GTT-bound shouldn't prevent its being
>> diff --git a/drivers/gpu/drm/i915/i915_gem_tiling.c b/drivers/gpu/drm/i915/i915_gem_tiling.c
>> index 2cefb59..fa5fc8c 100644
>> --- a/drivers/gpu/drm/i915/i915_gem_tiling.c
>> +++ b/drivers/gpu/drm/i915/i915_gem_tiling.c
>> @@ -383,7 +383,7 @@ i915_gem_set_tiling(struct drm_device *dev, void *data,
>>
>> if (ret == 0) {
>> obj->fence_dirty =
>> - obj->last_fenced_seqno ||
>> + obj->last_fenced_req ||
>> obj->fence_reg != I915_FENCE_REG_NONE;
>>
>> obj->tiling_mode = args->tiling_mode;
>> diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c b/drivers/gpu/drm/i915/i915_gpu_error.c
>> index 2c87a79..1b58390 100644
>> --- a/drivers/gpu/drm/i915/i915_gpu_error.c
>> +++ b/drivers/gpu/drm/i915/i915_gpu_error.c
>> @@ -666,8 +666,8 @@ static void capture_bo(struct drm_i915_error_buffer *err,
>>
>> err->size = obj->base.size;
>> err->name = obj->base.name;
>> - err->rseqno = obj->last_read_seqno;
>> - err->wseqno = obj->last_write_seqno;
>> + err->rseqno = i915_gem_request_get_seqno(obj->last_read_req);
>> + err->wseqno = i915_gem_request_get_seqno(obj->last_write_req);
>> err->gtt_offset = vma->node.start;
>> err->read_domains = obj->base.read_domains;
>> err->write_domain = obj->base.write_domain;
>> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
>> index b78f00a..5aae3d1e 100644
>> --- a/drivers/gpu/drm/i915/intel_display.c
>> +++ b/drivers/gpu/drm/i915/intel_display.c
>> @@ -9766,16 +9766,16 @@ static int intel_postpone_flip(struct drm_i915_gem_object *obj)
>>
>> lockdep_assert_held(&obj->base.dev->struct_mutex);
>>
>> - if (!obj->last_write_seqno)
>> + if (!obj->last_write_req)
>> return 0;
>>
>> ring = obj->ring;
>>
>> if (i915_seqno_passed(ring->get_seqno(ring, true),
>> - obj->last_write_seqno))
>> + i915_gem_request_get_seqno(obj->last_write_req)))
>> return 0;
>>
>> - ret = i915_gem_check_olr(ring, obj->last_write_seqno);
>> + ret = i915_gem_check_olr(ring, i915_gem_request_get_seqno(obj->last_write_req));
>> if (ret)
>> return ret;
>>
>> @@ -9838,7 +9838,7 @@ static int intel_queue_mmio_flip(struct drm_device *dev,
>> }
>>
>> spin_lock_irqsave(&dev_priv->mmio_flip_lock, irq_flags);
>> - intel_crtc->mmio_flip.seqno = obj->last_write_seqno;
>> + intel_crtc->mmio_flip.seqno = i915_gem_request_get_seqno(obj->last_write_req);
>> intel_crtc->mmio_flip.ring_id = obj->ring->id;
>> spin_unlock_irqrestore(&dev_priv->mmio_flip_lock, irq_flags);
>>
>> @@ -10045,7 +10045,7 @@ static int intel_crtc_page_flip(struct drm_crtc *crtc,
>> if (ret)
>> goto cleanup_unpin;
>>
>> - work->flip_queued_seqno = obj->last_write_seqno;
>> + work->flip_queued_seqno = i915_gem_request_get_seqno(obj->last_write_req);
>> work->flip_queued_ring = obj->ring;
>> } else {
>> ret = dev_priv->display.queue_flip(dev, crtc, fb, obj, ring,
>> diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.h b/drivers/gpu/drm/i915/intel_ringbuffer.h
>> index cc1b62f..d98b964 100644
>> --- a/drivers/gpu/drm/i915/intel_ringbuffer.h
>> +++ b/drivers/gpu/drm/i915/intel_ringbuffer.h
>> @@ -249,7 +249,7 @@ struct intel_engine_cs {
>> * ringbuffer.
>> *
>> * Includes buffers having the contents of their GPU caches
>> - * flushed, not necessarily primitives. last_rendering_seqno
>> + * flushed, not necessarily primitives. last_rendering_req
>> * represents when the rendering involved will be completed.
>> *
>> * A reference is held on the buffer while on this list.
>> --
>> 1.7.9.5
>>
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2014-10-20 16:02 UTC|newest]
Thread overview: 72+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-06 14:15 [RFC 00/21] Replace seqno values with request structures John.C.Harrison
2014-10-06 14:15 ` [RFC 01/21] Bug: missing i915_seqno_passed() call? John.C.Harrison
2014-10-06 14:15 ` [RFC 02/21] drm/i915: Remove redundant parameter to i915_gem_object_wait_rendering__tail() John.C.Harrison
2014-10-06 14:15 ` [RFC 03/21] drm/i915: Ensure OLS & PLR are always in sync John.C.Harrison
2014-10-06 14:15 ` [RFC 04/21] drm/i915: Add reference count to request structure John.C.Harrison
2014-10-06 14:15 ` [RFC 05/21] drm/i915: Add helper functions to aid seqno -> request transition John.C.Harrison
2014-10-06 14:15 ` [RFC 06/21] drm/i915: Replace last_[rwf]_seqno with last_[rwf]_req John.C.Harrison
2014-10-06 14:15 ` [RFC 07/21] drm/i915: Ensure requests stick around during waits John.C.Harrison
2014-10-06 14:15 ` [RFC 08/21] drm/i915: Remove 'outstanding_lazy_seqno' John.C.Harrison
2014-10-06 14:15 ` [RFC 09/21] drm/i915: Make 'i915_gem_check_olr' actually check by request not seqno John.C.Harrison
2014-10-06 14:15 ` [RFC 10/21] drm/i915: Convert 'last_flip_req' to be a request not a seqno John.C.Harrison
2014-10-06 14:15 ` [RFC 11/21] drm/i915: Convert i915_wait_seqno to i915_wait_request John.C.Harrison
2014-10-06 14:15 ` [RFC 12/21] drm/i915: Convert 'i915_add_request' to take a request not a seqno John.C.Harrison
2014-10-06 14:15 ` [RFC 13/21] drm/i915: Convert mmio_flip::seqno to struct request John.C.Harrison
2014-10-06 14:15 ` [RFC 14/21] drm/i915: Convert 'flip_queued_seqno' into 'flip_queued_request' John.C.Harrison
2014-10-06 14:15 ` [RFC 15/21] drm/i915: Convert most 'i915_seqno_passed' calls into 'i915_gem_request_completed' John.C.Harrison
2014-10-06 14:15 ` [RFC 16/21] drm/i915: Convert __wait_seqno() to __wait_request() John.C.Harrison
2014-10-06 14:15 ` [RFC 17/21] drm/i915: Convert trace functions from seqno to request John.C.Harrison
2014-10-06 14:15 ` [RFC 18/21] drm/i915: Convert 'trace_irq' to use requests rather than seqnos John.C.Harrison
2014-10-06 14:15 ` [RFC 19/21] drm/i915: Convert semaphores to handle requests not seqnos John.C.Harrison
2014-10-06 14:15 ` [RFC 20/21] drm/i915: Convert 'ring_idle()' to use " John.C.Harrison
2014-10-06 14:15 ` [RFC 21/21] drm/i915: Remove 'obj->ring' John.C.Harrison
2014-10-19 14:12 ` Daniel Vetter
2014-10-28 15:09 ` John Harrison
2014-11-03 10:38 ` Daniel Vetter
2014-10-19 14:09 ` [RFC 20/21] drm/i915: Convert 'ring_idle()' to use requests not seqnos Daniel Vetter
2014-10-28 14:03 ` John Harrison
2014-11-03 10:44 ` Daniel Vetter
2014-10-19 14:08 ` [RFC 19/21] drm/i915: Convert semaphores to handle " Daniel Vetter
2014-10-10 11:39 ` [RFC 16/25] drm/i915: Convert most 'i915_seqno_passed' calls into 'i915_gem_request_completed' John.C.Harrison
2014-10-19 14:04 ` Daniel Vetter
2014-10-28 14:02 ` John Harrison
2014-10-19 13:11 ` [RFC 14/21] drm/i915: Convert 'flip_queued_seqno' into 'flip_queued_request' Daniel Vetter
2014-10-19 13:07 ` [RFC 13/21] drm/i915: Convert mmio_flip::seqno to struct request Daniel Vetter
2014-10-19 12:57 ` [RFC 10/21] drm/i915: Convert 'last_flip_req' to be a request not a seqno Daniel Vetter
2014-10-19 12:55 ` [RFC 09/21] drm/i915: Make 'i915_gem_check_olr' actually check by request not seqno Daniel Vetter
2014-10-28 14:01 ` John Harrison
2014-11-03 10:51 ` Daniel Vetter
2014-10-10 11:38 ` [RFC 08/25] drm/i915: Remove 'outstanding_lazy_seqno' John.C.Harrison
2014-10-19 13:05 ` Daniel Vetter
2014-10-19 12:48 ` [RFC 08/21] " Daniel Vetter
2014-10-19 12:50 ` Daniel Vetter
2014-10-19 12:40 ` [RFC 06/21] drm/i915: Replace last_[rwf]_seqno with last_[rwf]_req Daniel Vetter
2014-10-20 15:58 ` John Harrison [this message]
2014-10-19 12:35 ` [RFC 05/21] drm/i915: Add helper functions to aid seqno -> request transition Daniel Vetter
2014-10-20 14:49 ` John Harrison
2014-10-19 12:32 ` [RFC 03/21] drm/i915: Ensure OLS & PLR are always in sync Daniel Vetter
2014-10-20 14:39 ` John Harrison
2014-10-19 12:25 ` [RFC 02/21] drm/i915: Remove redundant parameter to i915_gem_object_wait_rendering__tail() Daniel Vetter
2014-10-19 13:03 ` Daniel Vetter
2014-10-06 14:45 ` [RFC 01/21] Bug: missing i915_seqno_passed() call? Daniel Vetter
2014-10-06 14:59 ` John Harrison
2014-10-06 15:17 ` [RFC 00/21] Replace seqno values with request structures Chris Wilson
2014-10-19 17:15 ` Daniel Vetter
2014-10-20 7:19 ` Chris Wilson
2014-10-20 15:49 ` Daniel Vetter
2014-10-07 16:47 ` [RFC 22/21] drm/i915: Cache request completion status John.C.Harrison
2014-10-10 11:40 ` [RFC 23/25] " John.C.Harrison
2014-10-19 14:14 ` [RFC 22/21] " Daniel Vetter
2014-10-28 15:36 ` John Harrison
2014-11-03 10:57 ` Daniel Vetter
2014-10-10 11:38 ` [RFC 15/25] drm/i915: Connect requests to rings at creation not submission John.C.Harrison
2014-10-10 11:41 ` [RFC 24/25] drm/i915: Zero fill the request structure John.C.Harrison
2014-10-19 14:15 ` Daniel Vetter
2014-10-28 15:55 ` John Harrison
2014-11-03 11:02 ` Daniel Vetter
2014-10-10 11:41 ` [RFC 25/25] drm/i915: Defer seqno allocation until actual hardware submission time John.C.Harrison
2014-10-19 14:17 ` Daniel Vetter
2014-10-10 12:03 ` [RFC 00/21] Replace seqno values with request structures John Harrison
2014-10-19 14:21 ` Daniel Vetter
2014-10-20 10:19 ` John Harrison
2014-10-20 15:41 ` Daniel Vetter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5445313F.3040108@Intel.com \
--to=john.c.harrison@intel.com \
--cc=Intel-GFX@Lists.FreeDesktop.Org \
--cc=daniel@ffwll.ch \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.