* [Intel-gfx] [PATCH 1/3] drm/i915: Remove trace_i915_gem_object_fault
@ 2020-05-03 20:09 Chris Wilson
2020-05-03 20:09 ` [Intel-gfx] [PATCH 2/3] drm/i915: Remove eviction tracepoints Chris Wilson
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Chris Wilson @ 2020-05-03 20:09 UTC (permalink / raw)
To: intel-gfx; +Cc: Chris Wilson
The tracepoint is only covering one of the possible fault handlers;
clearly no longer of interest.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 2 --
drivers/gpu/drm/i915/i915_trace.h | 25 ------------------------
2 files changed, 27 deletions(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_mman.c b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
index 70f5f82da288..97a58b592c37 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
@@ -296,8 +296,6 @@ static vm_fault_t vm_fault_gtt(struct vm_fault *vmf)
/* We don't use vmf->pgoff since that has the fake offset */
page_offset = (vmf->address - area->vm_start) >> PAGE_SHIFT;
- trace_i915_gem_object_fault(obj, page_offset, true, write);
-
ret = i915_gem_object_pin_pages(obj);
if (ret)
goto err;
diff --git a/drivers/gpu/drm/i915/i915_trace.h b/drivers/gpu/drm/i915/i915_trace.h
index bc854ad60954..af4d1c74b54c 100644
--- a/drivers/gpu/drm/i915/i915_trace.h
+++ b/drivers/gpu/drm/i915/i915_trace.h
@@ -601,31 +601,6 @@ TRACE_EVENT(i915_gem_object_pread,
__entry->obj, __entry->offset, __entry->len)
);
-TRACE_EVENT(i915_gem_object_fault,
- TP_PROTO(struct drm_i915_gem_object *obj, u64 index, bool gtt, bool write),
- TP_ARGS(obj, index, gtt, write),
-
- TP_STRUCT__entry(
- __field(struct drm_i915_gem_object *, obj)
- __field(u64, index)
- __field(bool, gtt)
- __field(bool, write)
- ),
-
- TP_fast_assign(
- __entry->obj = obj;
- __entry->index = index;
- __entry->gtt = gtt;
- __entry->write = write;
- ),
-
- TP_printk("obj=%p, %s index=%llu %s",
- __entry->obj,
- __entry->gtt ? "GTT" : "CPU",
- __entry->index,
- __entry->write ? ", writable" : "")
-);
-
DECLARE_EVENT_CLASS(i915_gem_object,
TP_PROTO(struct drm_i915_gem_object *obj),
TP_ARGS(obj),
--
2.20.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Intel-gfx] [PATCH 2/3] drm/i915: Remove eviction tracepoints
2020-05-03 20:09 [Intel-gfx] [PATCH 1/3] drm/i915: Remove trace_i915_gem_object_fault Chris Wilson
@ 2020-05-03 20:09 ` Chris Wilson
2020-05-03 20:09 ` [Intel-gfx] [PATCH 3/3] drm/i915: Drop pread/pwrite tracepoints Chris Wilson
2020-05-03 20:47 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Remove trace_i915_gem_object_fault Patchwork
2 siblings, 0 replies; 4+ messages in thread
From: Chris Wilson @ 2020-05-03 20:09 UTC (permalink / raw)
To: intel-gfx; +Cc: Chris Wilson
The eviction tracepoints are an odd mismash in the middle of the
bind/unbind sequence. I added them primarily to try and tell why a spate
of unbinds occurred, but in reality they were just noise and not
information or instrumental in refining code.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
drivers/gpu/drm/i915/i915_gem_evict.c | 4 --
drivers/gpu/drm/i915/i915_trace.h | 70 ---------------------------
2 files changed, 74 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_evict.c b/drivers/gpu/drm/i915/i915_gem_evict.c
index 0ba7b1e881c0..b9433b013ad4 100644
--- a/drivers/gpu/drm/i915/i915_gem_evict.c
+++ b/drivers/gpu/drm/i915/i915_gem_evict.c
@@ -100,7 +100,6 @@ i915_gem_evict_something(struct i915_address_space *vm,
int ret;
lockdep_assert_held(&vm->mutex);
- trace_i915_gem_evict(vm, min_size, alignment, flags);
/*
* The goal is to evict objects and amalgamate space in rough LRU order.
@@ -265,8 +264,6 @@ int i915_gem_evict_for_node(struct i915_address_space *vm,
GEM_BUG_ON(!IS_ALIGNED(start, I915_GTT_PAGE_SIZE));
GEM_BUG_ON(!IS_ALIGNED(end, I915_GTT_PAGE_SIZE));
- trace_i915_gem_evict_node(vm, target, flags);
-
/*
* Retire before we search the active list. Although we have
* reasonable accuracy in our retirement lists, we may have
@@ -362,7 +359,6 @@ int i915_gem_evict_vm(struct i915_address_space *vm)
int ret = 0;
lockdep_assert_held(&vm->mutex);
- trace_i915_gem_evict_vm(vm);
/* Switch back to the default context in order to unpin
* the existing context objects. However, such objects only
diff --git a/drivers/gpu/drm/i915/i915_trace.h b/drivers/gpu/drm/i915/i915_trace.h
index af4d1c74b54c..ef8767b880d6 100644
--- a/drivers/gpu/drm/i915/i915_trace.h
+++ b/drivers/gpu/drm/i915/i915_trace.h
@@ -626,76 +626,6 @@ DEFINE_EVENT(i915_gem_object, i915_gem_object_destroy,
TP_ARGS(obj)
);
-TRACE_EVENT(i915_gem_evict,
- TP_PROTO(struct i915_address_space *vm, u64 size, u64 align, unsigned int flags),
- TP_ARGS(vm, size, align, flags),
-
- TP_STRUCT__entry(
- __field(u32, dev)
- __field(struct i915_address_space *, vm)
- __field(u64, size)
- __field(u64, align)
- __field(unsigned int, flags)
- ),
-
- TP_fast_assign(
- __entry->dev = vm->i915->drm.primary->index;
- __entry->vm = vm;
- __entry->size = size;
- __entry->align = align;
- __entry->flags = flags;
- ),
-
- TP_printk("dev=%d, vm=%p, size=0x%llx, align=0x%llx %s",
- __entry->dev, __entry->vm, __entry->size, __entry->align,
- __entry->flags & PIN_MAPPABLE ? ", mappable" : "")
-);
-
-TRACE_EVENT(i915_gem_evict_node,
- TP_PROTO(struct i915_address_space *vm, struct drm_mm_node *node, unsigned int flags),
- TP_ARGS(vm, node, flags),
-
- TP_STRUCT__entry(
- __field(u32, dev)
- __field(struct i915_address_space *, vm)
- __field(u64, start)
- __field(u64, size)
- __field(unsigned long, color)
- __field(unsigned int, flags)
- ),
-
- TP_fast_assign(
- __entry->dev = vm->i915->drm.primary->index;
- __entry->vm = vm;
- __entry->start = node->start;
- __entry->size = node->size;
- __entry->color = node->color;
- __entry->flags = flags;
- ),
-
- TP_printk("dev=%d, vm=%p, start=0x%llx size=0x%llx, color=0x%lx, flags=%x",
- __entry->dev, __entry->vm,
- __entry->start, __entry->size,
- __entry->color, __entry->flags)
-);
-
-TRACE_EVENT(i915_gem_evict_vm,
- TP_PROTO(struct i915_address_space *vm),
- TP_ARGS(vm),
-
- TP_STRUCT__entry(
- __field(u32, dev)
- __field(struct i915_address_space *, vm)
- ),
-
- TP_fast_assign(
- __entry->dev = vm->i915->drm.primary->index;
- __entry->vm = vm;
- ),
-
- TP_printk("dev=%d, vm=%p", __entry->dev, __entry->vm)
-);
-
TRACE_EVENT(i915_request_queue,
TP_PROTO(struct i915_request *rq, u32 flags),
TP_ARGS(rq, flags),
--
2.20.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Intel-gfx] [PATCH 3/3] drm/i915: Drop pread/pwrite tracepoints
2020-05-03 20:09 [Intel-gfx] [PATCH 1/3] drm/i915: Remove trace_i915_gem_object_fault Chris Wilson
2020-05-03 20:09 ` [Intel-gfx] [PATCH 2/3] drm/i915: Remove eviction tracepoints Chris Wilson
@ 2020-05-03 20:09 ` Chris Wilson
2020-05-03 20:47 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Remove trace_i915_gem_object_fault Patchwork
2 siblings, 0 replies; 4+ messages in thread
From: Chris Wilson @ 2020-05-03 20:09 UTC (permalink / raw)
To: intel-gfx; +Cc: Chris Wilson
These pair provide no insight into the driver, are extremely rare in
userspace and only capture the same information as can be gathered from
looking at the syscalls [ioctl].
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
drivers/gpu/drm/i915/i915_gem.c | 4 ----
drivers/gpu/drm/i915/i915_trace.h | 40 -------------------------------
2 files changed, 44 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index 0cbcb9f54e7d..a8e1969df562 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
+++ b/drivers/gpu/drm/i915/i915_gem.c
@@ -517,8 +517,6 @@ i915_gem_pread_ioctl(struct drm_device *dev, void *data,
goto out;
}
- trace_i915_gem_object_pread(obj, args->offset, args->size);
-
ret = i915_gem_object_wait(obj,
I915_WAIT_INTERRUPTIBLE,
MAX_SCHEDULE_TIMEOUT);
@@ -813,8 +811,6 @@ i915_gem_pwrite_ioctl(struct drm_device *dev, void *data,
goto err;
}
- trace_i915_gem_object_pwrite(obj, args->offset, args->size);
-
ret = -ENODEV;
if (obj->ops->pwrite)
ret = obj->ops->pwrite(obj, args);
diff --git a/drivers/gpu/drm/i915/i915_trace.h b/drivers/gpu/drm/i915/i915_trace.h
index ef8767b880d6..dcc26131c69c 100644
--- a/drivers/gpu/drm/i915/i915_trace.h
+++ b/drivers/gpu/drm/i915/i915_trace.h
@@ -561,46 +561,6 @@ TRACE_EVENT(i915_vma_unbind,
__entry->obj, __entry->offset, __entry->size, __entry->vm)
);
-TRACE_EVENT(i915_gem_object_pwrite,
- TP_PROTO(struct drm_i915_gem_object *obj, u64 offset, u64 len),
- TP_ARGS(obj, offset, len),
-
- TP_STRUCT__entry(
- __field(struct drm_i915_gem_object *, obj)
- __field(u64, offset)
- __field(u64, len)
- ),
-
- TP_fast_assign(
- __entry->obj = obj;
- __entry->offset = offset;
- __entry->len = len;
- ),
-
- TP_printk("obj=%p, offset=0x%llx, len=0x%llx",
- __entry->obj, __entry->offset, __entry->len)
-);
-
-TRACE_EVENT(i915_gem_object_pread,
- TP_PROTO(struct drm_i915_gem_object *obj, u64 offset, u64 len),
- TP_ARGS(obj, offset, len),
-
- TP_STRUCT__entry(
- __field(struct drm_i915_gem_object *, obj)
- __field(u64, offset)
- __field(u64, len)
- ),
-
- TP_fast_assign(
- __entry->obj = obj;
- __entry->offset = offset;
- __entry->len = len;
- ),
-
- TP_printk("obj=%p, offset=0x%llx, len=0x%llx",
- __entry->obj, __entry->offset, __entry->len)
-);
-
DECLARE_EVENT_CLASS(i915_gem_object,
TP_PROTO(struct drm_i915_gem_object *obj),
TP_ARGS(obj),
--
2.20.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Remove trace_i915_gem_object_fault
2020-05-03 20:09 [Intel-gfx] [PATCH 1/3] drm/i915: Remove trace_i915_gem_object_fault Chris Wilson
2020-05-03 20:09 ` [Intel-gfx] [PATCH 2/3] drm/i915: Remove eviction tracepoints Chris Wilson
2020-05-03 20:09 ` [Intel-gfx] [PATCH 3/3] drm/i915: Drop pread/pwrite tracepoints Chris Wilson
@ 2020-05-03 20:47 ` Patchwork
2 siblings, 0 replies; 4+ messages in thread
From: Patchwork @ 2020-05-03 20:47 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/3] drm/i915: Remove trace_i915_gem_object_fault
URL : https://patchwork.freedesktop.org/series/76881/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_8415 -> Patchwork_17559
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_17559 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_17559, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17559/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_17559:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live@active:
- fi-apl-guc: [PASS][1] -> [INCOMPLETE][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8415/fi-apl-guc/igt@i915_selftest@live@active.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17559/fi-apl-guc/igt@i915_selftest@live@active.html
Known issues
------------
Here are the changes found in Patchwork_17559 that come from known issues:
### IGT changes ###
#### Possible fixes ####
* igt@i915_selftest@live@gt_lrc:
- fi-bsw-n3050: [INCOMPLETE][3] ([i915#1436]) -> [PASS][4]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8415/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17559/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
Participating hosts (48 -> 42)
------------------------------
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_8415 -> Patchwork_17559
CI-20190529: 20190529
CI_DRM_8415: a20d5d8cc2cec9962cf4241fd8ac4b0c4bb4d9d8 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5628: 652a3fd8966345fa5498904ce80a2027a6782783 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_17559: 7753c3035f5b4b44d6e9dd14c4d3c166b7f71b6f @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
7753c3035f5b drm/i915: Drop pread/pwrite tracepoints
824df510bae8 drm/i915: Remove eviction tracepoints
bb6d8f44a282 drm/i915: Remove trace_i915_gem_object_fault
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17559/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-05-03 20:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-03 20:09 [Intel-gfx] [PATCH 1/3] drm/i915: Remove trace_i915_gem_object_fault Chris Wilson
2020-05-03 20:09 ` [Intel-gfx] [PATCH 2/3] drm/i915: Remove eviction tracepoints Chris Wilson
2020-05-03 20:09 ` [Intel-gfx] [PATCH 3/3] drm/i915: Drop pread/pwrite tracepoints Chris Wilson
2020-05-03 20:47 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Remove trace_i915_gem_object_fault Patchwork
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).