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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox