== Series Details ==
Series: fbcon notifier begone! (rev2)
URL : https://patchwork.freedesktop.org/series/60843/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6114 -> Patchwork_13061
Summary
---
**SUCCESS**
No
Quoting Marcin Owsiany (2019-05-22 08:54:50)
> Hello,
>
> Every couple of weeks or so my laptop experiences some graphics-related
> crash. Sometimes it's more frequent - happened twice yesterday.
>
> I'm wondering whether this could be a sign of a hardware failure? Or is this
> some software
On 22/05/2019 09:30, Chris Wilson wrote:
commit a88b6e4cbafd6f23b3450c087acdbe23d90e7606
Author: Chris Wilson
Date: Tue May 21 22:11:34 2019 +0100
drm/i915: Allow specification of parallel execbuf
Signed-off-by: Chris Wilson
Acked-by: Tvrtko Ursulin
Regards,
Tvrtko
---
== Series Details ==
Series: drm/i915: Fix the interpretation of MAX_PRE-EMPHASIS_REACHED bit
inorder to pass Link Layer compliance test number 400.3.1.15 (rev2)
URL : https://patchwork.freedesktop.org/series/60880/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6113_full ->
Quoting Michal Hocko (2019-05-22 07:34:42)
> On Wed 22-05-19 06:06:31, Tetsuo Handa wrote:
> [...]
> > Since OOM notifier will be called after shrinkers are attempted,
> > can i915 move from OOM notifier to shrinker?
>
> That would be indeed preferable. OOM notifier is an API from hell.
We
commit a88b6e4cbafd6f23b3450c087acdbe23d90e7606
Author: Chris Wilson
Date: Tue May 21 22:11:34 2019 +0100
drm/i915: Allow specification of parallel execbuf
Signed-off-by: Chris Wilson
---
include/drm-uapi/i915_drm.h | 167 +++-
1 file changed, 164
On Tue, May 21, 2019 at 11:06 PM Tetsuo Handa
wrote:
>
> On 2019/05/21 23:44, Daniel Vetter wrote:
> OOM notifiers should not depend on any locks or sleepable conditionals.
> If some lock directly or indirectly depended on __GFP_DIRECT_RECLAIM,
> it will deadlock. Thus, despite
== Series Details ==
Series: series starting with [CI,01/13] drm/i915: Split GEM object type
definition to its own header
URL : https://patchwork.freedesktop.org/series/60940/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/i915: Split GEM object
Hello,
Every couple of weeks or so my laptop experiences some graphics-related
crash. Sometimes it's more frequent - happened twice yesterday.
I'm wondering whether this could be a sign of a hardware failure? Or is this
some software lock-up? Please do let me know whether I can do anything to
Hi Tvrtko,
On Mon, May 20, 2019 at 03:47:34PM +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> Test the new engine discovery query.
>
> Signed-off-by: Tvrtko Ursulin
Reviewed-by: Andi Shyti
Thanks,
Andi
___
Intel-gfx mailing list
commit a88b6e4cbafd6f23b3450c087acdbe23d90e7606
Author: Chris Wilson
Date: Tue May 21 22:11:34 2019 +0100
drm/i915: Allow specification of parallel execbuf
Signed-off-by: Chris Wilson
---
include/drm-uapi/i915_drm.h | 164 +++-
1 file changed, 162
== Series Details ==
Series: series starting with [CI,01/13] drm/i915: Split GEM object type
definition to its own header
URL : https://patchwork.freedesktop.org/series/60940/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
21a4f2a14a6c drm/i915: Split GEM object type
== Series Details ==
Series: kernel.h: Add non_block_start/end()
URL : https://patchwork.freedesktop.org/series/60896/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6114_full -> Patchwork_13057_full
Summary
---
Exercise basic creation and swapping between new address spaces.
v2: Check isolation that the same vm_id on different fd are indeed
different VM.
v3: Cross-over check with CREATE_EXT_SETPARAM
Signed-off-by: Chris Wilson
Cc: Tvrtko Ursulin
Reviewed-by: Tvrtko Ursulin
---
lib/Makefile.sources
Quoting Lionel Landwerlin (2019-05-22 10:19:46)
> On 21/05/2019 18:48, Chris Wilson wrote:
> > Quoting Lionel Landwerlin (2019-05-21 18:19:50)
> >> On 21/05/2019 18:07, Chris Wilson wrote:
> >>> Quoting Lionel Landwerlin (2019-05-21 15:08:54)
> + if (eb->oa_config &&
> +
Instead of relying a static obj array inside igt_spin_t,
access it with proper indexing.
v2: IGT_SPIN_BATCH
Cc: Chris Wilson
Signed-off-by: Mika Kuoppala
---
lib/igt_dummyload.c| 13 ++---
lib/igt_dummyload.h| 1 +
tests/i915/gem_exec_schedule.c | 25
== Series Details ==
Series: drm/i915/dp: Support for DP YCbCr4:2:0 outputs (rev4)
URL : https://patchwork.freedesktop.org/series/60404/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6114_full -> Patchwork_13058_full
Instead of relying a static obj array inside igt_spin_t,
access it with proper indexing.
v2: IGT_SPIN_BATCH
v3: indent
Cc: Chris Wilson
Signed-off-by: Mika Kuoppala
Reviewed-by: Chris Wilson
---
lib/igt_dummyload.c| 13 ++---
lib/igt_dummyload.h| 1 +
Hi Daniel,
On 5/21/19 4:23 PM, Daniel Vetter wrote:
> This is unused code since
>
> commit 6104c37094e729f3d4ce65797002112735d49cd1
> Author: Daniel Vetter
> Date: Tue Aug 1 17:32:07 2017 +0200
>
> fbcon: Make fbcon a built-time depency for fbdev
>
> when fbcon was made a compile-time
== Series Details ==
Series: linux-next: manual merge of the drm-misc tree with Linus' tree (rev2)
URL : https://patchwork.freedesktop.org/series/60886/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6114_full -> Patchwork_13056_full
On Wed 22-05-19 08:13:57, Chris Wilson wrote:
> Quoting Michal Hocko (2019-05-22 07:34:42)
> > On Wed 22-05-19 06:06:31, Tetsuo Handa wrote:
> > [...]
> > > Since OOM notifier will be called after shrinkers are attempted,
> > > can i915 move from OOM notifier to shrinker?
> >
> > That would be
== Series Details ==
Series: series starting with [CI,01/10] drm/i915: Restore control over ppgtt
for context creation ABI
URL : https://patchwork.freedesktop.org/series/60909/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6114_full -> Patchwork_13059_full
== Series Details ==
Series: series starting with [v4,1/2] drm/i915/selftests: Verify context
workarounds (rev3)
URL : https://patchwork.freedesktop.org/series/60857/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6114_full -> Patchwork_13062_full
== Series Details ==
Series: series starting with [CI,01/13] drm/i915: Split GEM object type
definition to its own header
URL : https://patchwork.freedesktop.org/series/60940/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6117 -> Patchwork_13067
Instead of relying a static obj array inside igt_spin_t,
access it with proper indexing.
Cc: Chris Wilson
Signed-off-by: Mika Kuoppala
---
lib/igt_dummyload.c| 32 +++-
lib/igt_dummyload.h| 1 +
tests/i915/gem_exec_schedule.c | 25
Fix the duplicate typedef for intel_wakeref_t leading to Clang build
issues. While at it, actually make the intel_runtime_pm.h header
self-contained, which was claimed in the commit being fixed.
Reported-by: Nathan Chancellor
Cc: Nathan Chancellor
Cc: Chris Wilson
References:
On Wed, May 22, 2019 at 12:04 PM Bartlomiej Zolnierkiewicz
wrote:
>
>
> Hi Daniel,
>
> On 5/21/19 4:23 PM, Daniel Vetter wrote:
> > This is unused code since
> >
> > commit 6104c37094e729f3d4ce65797002112735d49cd1
> > Author: Daniel Vetter
> > Date: Tue Aug 1 17:32:07 2017 +0200
> >
> >
Add a new mode for some more stress, submit the all-engines tests
simultaneously, a stream per engine.
Signed-off-by: Chris Wilson
---
tests/i915/gem_exec_whisper.c | 25 +
1 file changed, 21 insertions(+), 4 deletions(-)
diff --git a/tests/i915/gem_exec_whisper.c
The submit-fence + load_balancing apis allow for us to execute a named
pair of engines in parallel; that this by submitting a request to one
engine, we can then use the generated submit-fence to submit a second
request to another engine and have it execute at the same time.
Furthermore, by
To exercise the new I915_CONTEXT_PARAM_ENGINES and interactions with
gem_execbuf().
Signed-off-by: Chris Wilson
Cc: Tvrtko Ursulin
Cc: Andi Shyti
Reviewed-by: Andi Shyti
---
tests/Makefile.sources | 1 +
tests/i915/gem_ctx_engines.c | 517 +++
v2: Test each shared context is its own timeline and allows request
reordering between shared contexts.
v3: Use the modern interface to use physical engines for testing
single-timeline along each real engine.
Signed-off-by: Chris Wilson
Cc: Joonas Lahtinen
Cc: Tvrtko Ursulin
Cc: Mika Kuoppala
Exercise cloning contexts, an extension of merely creating one.
Signed-off-by: Chris Wilson
Reviewed-by: Tvrtko Ursulin
---
tests/Makefile.sources | 1 +
tests/i915/gem_ctx_clone.c | 462 +
tests/meson.build | 1 +
3 files changed, 464
Queues are a form of contexts that share vm and enfore a single timeline
across all engines. Test switching between them, just like ordinary
contexts.
Signed-off-by: Chris Wilson
Reviewed-by: Tvrtko Ursulin
---
tests/i915/gem_ctx_switch.c | 75 +++--
1 file
Signed-off-by: Chris Wilson
---
tests/i915/gem_exec_latency.c | 81 +++
1 file changed, 81 insertions(+)
diff --git a/tests/i915/gem_exec_latency.c b/tests/i915/gem_exec_latency.c
index e88fbbc6a..fd4ceb4d6 100644
--- a/tests/i915/gem_exec_latency.c
+++
Check that the extended create interface accepts setparam.
Signed-off-by: Chris Wilson
---
tests/i915/gem_ctx_create.c | 225 ++--
1 file changed, 213 insertions(+), 12 deletions(-)
diff --git a/tests/i915/gem_ctx_create.c b/tests/i915/gem_ctx_create.c
index
Measure the baseline latency between contexts in order to directly
compare that with the additional cost of preemption.
Signed-off-by: Chris Wilson
---
tests/i915/gem_exec_latency.c | 230 ++
1 file changed, 230 insertions(+)
diff --git
On 22/05/2019 12:37, Chris Wilson wrote:
Exercise the in-kernel load balancer checking that we can distribute
batches across the set of ctx->engines to avoid load.
v2: Many assorted improvements from Tvrtko.
Signed-off-by: Chris Wilson
---
tests/Makefile.am |1 +
== Series Details ==
Series: series starting with [1/4] mm: Check if mmu notifier callbacks are
allowed to fail (rev2)
URL : https://patchwork.freedesktop.org/series/60874/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6114_full -> Patchwork_13063_full
From: Tvrtko Ursulin
Engine discovery query allows userspace to enumerate engines, probe their
configuration features, all without needing to maintain the internal PCI
ID based database.
A new query for the generic i915 query ioctl is added named
DRM_I915_QUERY_ENGINE_INFO, together with
On 21/05/2019 18:51, Patchwork wrote:
== Series Details ==
Series: series starting with [v4,1/2] drm/i915/selftests: Verify context
workarounds (rev3)
URL : https://patchwork.freedesktop.org/series/60857/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6114 ->
On Tue, 21 May 2019, Nathan Chancellor wrote:
> Hi Jani,
>
> On Mon, Apr 29, 2019 at 03:29:36PM +0300, Jani Nikula wrote:
>> It used to be handy that we only had a couple of headers, but over time
>> intel_drv.h has become unwieldy. Extract declarations to a separate
>> header file corresponding
Exercise the in-kernel load balancer checking that we can distribute
batches across the set of ctx->engines to avoid load.
v2: Many assorted improvements from Tvrtko.
Signed-off-by: Chris Wilson
---
tests/Makefile.am |1 +
tests/Makefile.sources |1 +
Exercise basic creation and swapping between new address spaces.
v2: Check isolation that the same vm_id on different fd are indeed
different VM.
v3: Cross-over check with CREATE_EXT_SETPARAM
Signed-off-by: Chris Wilson
Cc: Tvrtko Ursulin
Reviewed-by: Tvrtko Ursulin
---
lib/Makefile.sources
Exercise reusing the GTT of one ctx in another.
v2: Test setting back to the same VM
v3: Check the VM still exists after the parent ctx are dead.
Signed-off-by: Chris Wilson
Cc: Tvrtko Ursulin
Reviewed-by: Tvrtko Ursulin
---
tests/i915/gem_ctx_param.c | 116
On 22/05/2019 11:59, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2019-05-16 09:57:08)
On 15/05/2019 20:35, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2019-05-14 13:57:26)
On 08/05/2019 11:09, Chris Wilson wrote:
Add a new mode for some more stress, submit the all-engines tests
On 21/05/2019 18:48, Chris Wilson wrote:
Quoting Lionel Landwerlin (2019-05-21 18:19:50)
On 21/05/2019 18:07, Chris Wilson wrote:
Quoting Lionel Landwerlin (2019-05-21 15:08:54)
+ if (eb->oa_config &&
+ eb->oa_config != eb->i915->perf.oa.exclusive_stream->oa_config) {
But the
On 22/05/2019 10:25, Chris Wilson wrote:
Quoting Lionel Landwerlin (2019-05-22 10:19:46)
On 21/05/2019 18:48, Chris Wilson wrote:
Quoting Lionel Landwerlin (2019-05-21 18:19:50)
On 21/05/2019 18:07, Chris Wilson wrote:
Quoting Lionel Landwerlin (2019-05-21 15:08:54)
+ if
Quoting Mika Kuoppala (2019-05-22 10:34:02)
> Instead of relying a static obj array inside igt_spin_t,
> access it with proper indexing.
>
> v2: IGT_SPIN_BATCH
>
> Cc: Chris Wilson
> Signed-off-by: Mika Kuoppala
> ---
> lib/igt_dummyload.c| 13 ++---
> lib/igt_dummyload.h
Quoting Tvrtko Ursulin (2019-05-16 09:38:15)
>
> On 15/05/2019 20:05, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2019-05-14 11:15:12)
> >>
> >> On 08/05/2019 11:09, Chris Wilson wrote:
> >>> Check that the extended create interface accepts setparam.
> >>>
> >>> Signed-off-by: Chris Wilson
>
On 22/05/2019 11:24, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2019-05-16 09:38:15)
On 15/05/2019 20:05, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2019-05-14 11:15:12)
On 08/05/2019 11:09, Chris Wilson wrote:
Check that the extended create interface accepts setparam.
Signed-off-by:
Quoting Tvrtko Ursulin (2019-05-16 09:57:08)
>
> On 15/05/2019 20:35, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2019-05-14 13:57:26)
> >>
> >> On 08/05/2019 11:09, Chris Wilson wrote:
> >>> Add a new mode for some more stress, submit the all-engines tests
> >>> simultaneously, a stream per
On 22/05/2019 12:37, Chris Wilson wrote:
The submit-fence + load_balancing apis allow for us to execute a named
pair of engines in parallel; that this by submitting a request to one
engine, we can then use the generated submit-fence to submit a second
request to another engine and have it
Quoting Jani Nikula (2019-05-22 11:35:05)
> Fix the duplicate typedef for intel_wakeref_t leading to Clang build
> issues. While at it, actually make the intel_runtime_pm.h header
> self-contained, which was claimed in the commit being fixed.
>
> Reported-by: Nathan Chancellor
> Cc: Nathan
Quoting Patchwork (2019-05-22 09:56:06)
> Possible regressions
>
> * igt@runner@aborted:
> - fi-ilk-650: NOTRUN -> [FAIL][1]
>[1]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13067/fi-ilk-650/igt@run...@aborted.html
Oh, a new test arrived that needs
Quoting Mika Kuoppala (2019-05-22 10:13:46)
> Instead of relying a static obj array inside igt_spin_t,
> access it with proper indexing.
>
> Cc: Chris Wilson
> Signed-off-by: Mika Kuoppala
> ---
> lib/igt_dummyload.c| 32 +++-
> lib/igt_dummyload.h
On 5/21/2019 7:08 PM, Michal Wajdeczko wrote:
On Tue, 21 May 2019 13:04:12 +0200, Ye, Tony wrote:
There are two users of I915_PARAM_HUC_STATUS, the intel-media driver
and the legacy intel-vaapi-driver.
Both drivers check the return value like this:
has_huc = !! ret_value;
So the ABI
The submit-fence + load_balancing apis allow for us to execute a named
pair of engines in parallel; that this by submitting a request to one
engine, we can then use the generated submit-fence to submit a second
request to another engine and have it execute at the same time.
Furthermore, by
From: Tvrtko Ursulin
Test the new engine discovery query.
Signed-off-by: Tvrtko Ursulin
Reviewed-by: Andi Shyti
---
tests/i915/i915_query.c | 247
1 file changed, 247 insertions(+)
diff --git a/tests/i915/i915_query.c b/tests/i915/i915_query.c
index
Chris Wilson writes:
> Use i915_gem_object_lock() to guard the LUT and active reference to
> allow us to break free of struct_mutex for handling GEM_CLOSE.
>
> Testcase: igt/gem_close_race
> Testcase: igt/gem_exec_parallel
> Signed-off-by: Chris Wilson
> ---
>
Mesa change : https://gitlab.freedesktop.org/mesa/mesa/merge_requests/932
On 22/05/2019 14:05, Lionel Landwerlin wrote:
Hi all,
This small (but maybe not to everybody's taste) series enables us to
support performance queries on Vulkan. We've gone through the process
to define this as a Vulkan
== Series Details ==
Series: drm/i915: Engine discovery query (rev11)
URL : https://patchwork.freedesktop.org/series/39958/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
f26f9f629288 drm/i915: Engine discovery query
-:63: WARNING:TYPO_SPELLING: 'assigment' may be misspelled -
Hi all,
This small (but maybe not to everybody's taste) series enables us to
support performance queries on Vulkan. We've gone through the process
to define this as a Vulkan INTEL extension (it should appear on [1]
soonish).
This v2 includes fixes after Chris' first round of review.
Cheers,
Here we introduce a mechanism by which the execbuf part of the i915
driver will be able to request that a batch buffer containing the
programming for a particular OA config be created.
We'll execute these OA configuration buffers right before executing a
set of userspace commands so that a
Listing configurations at the moment is supported only through sysfs.
This might cause issues for applications wanting to list
configurations from a container where sysfs isn't available.
This change adds a way to query the number of configurations and their
content through the i915 query uAPI.
On 22/05/2019 13:52, Chris Wilson wrote:
The submit-fence + load_balancing apis allow for us to execute a named
pair of engines in parallel; that this by submitting a request to one
engine, we can then use the generated submit-fence to submit a second
request to another engine and have it
Reporting this version will help application figure out what level of
the support the running kernel provides.
Signed-off-by: Lionel Landwerlin
---
drivers/gpu/drm/i915/i915_drv.c | 3 +++
include/uapi/drm/i915_drm.h | 21 +
2 files changed, 24 insertions(+)
diff --git
We would like to make use of perf in Vulkan. The Vulkan API is much
lower level than OpenGL, with applications directly exposed to the
concept of command buffers (pretty much equivalent to our batch
buffers). In Vulkan, queries are always limited in scope to a command
buffer. In OpenGL, the lack
We want the ability to dispatch a set of command buffer to the
hardware, each with a different OA configuration. To achieve this, we
reuse a couple of fields from the execbuf2 struct (I CAN HAZ
execbuf3?) to notify what OA configuration should be used for a batch
buffer. This requires the process
intel_mark_busy(), intel_mark_idle(), and skl_cdclk_get_vco() no longer
exist.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/intel_drv.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
index 270f5b..720a27 100644
Some function declarations in intel_drv.h were missed when
intel_atomic_plane.h was created.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/intel_atomic_plane.h | 16
drivers/gpu/drm/i915/intel_drv.h | 12
2 files changed, 16 insertions(+), 12
Move more missed declarations from i915_drv.h to intel_pm.h.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/i915_drv.h | 10 --
drivers/gpu/drm/i915/i915_irq.c | 1 +
drivers/gpu/drm/i915/intel_pm.h | 9 +
3 files changed, 10 insertions(+), 10 deletions(-)
diff --git
Move i915_reg_read_ioctl() declaration to intel_uncore.h, corresponding
to the definition in intel_uncore.c, and rename to
intel_uncore_reg_read_ioctl(). No functional changes.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/i915_drv.c | 2 +-
drivers/gpu/drm/i915/i915_drv.h | 3 ---
Quoting Mika Kuoppala (2019-05-22 15:32:34)
> Chris Wilson writes:
> > -static void free_engines_rcu(struct work_struct *wrk)
> > +static void free_engines_rcu(struct rcu_head *rcu)
> > {
> > - struct i915_gem_engines *e =
> > - container_of(wrk, struct i915_gem_engines,
Quoting Tvrtko Ursulin (2019-05-22 16:04:13)
> From: Tvrtko Ursulin
>
> Use from_user_pointer when casting __u64 to a pointer.
>
> Signed-off-by: Tvrtko Ursulin
> Cc: Andi Shyti
> ---
> lib/i915/gem_engine_topology.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git
From: Tvrtko Ursulin
This enables static test enumeration to see the engine.
Signed-off-by: Tvrtko Ursulin
---
lib/igt_gt.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/igt_gt.c b/lib/igt_gt.c
index 5e5b9bef03dc..6b7c037e6d10 100644
--- a/lib/igt_gt.c
+++ b/lib/igt_gt.c
@@ -589,6
== Series Details ==
Series: drm/i915: Engine discovery query (rev11)
URL : https://patchwork.freedesktop.org/series/39958/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6120 -> Patchwork_13068
Summary
---
On 22/05/2019 14:27, Patchwork wrote:
== Series Details ==
Series: drm/i915: Engine discovery query (rev11)
URL : https://patchwork.freedesktop.org/series/39958/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6120 -> Patchwork_13068
From: Andi Shyti
The gem_engine_topology library is a set of functions that
interface with the query and getparam/setparam ioctls.
The library's access point is the 'intel_init_engine_list()'
function that, everytime is called, generates the list of active
engines and returns them in a 'struct
From: Andi Shyti
Replace the legacy for_each_engine* defines with the ones
implemented in the gem_engine_topology library.
Use whenever possible gem_engine_can_store_dword() that checks
class instead of flags.
Now the __for_each_engine_class_instance and
for_each_engine_class_instance are
From: Tvrtko Ursulin
Just for CI since the first patch from the original series won't apply any more.
Andi Shyti (13):
lib/i915: add gem_engine_topology library and for_each loop definition
lib: igt_gt: add execution buffer flags to class helper
lib: igt_gt: make
From: Andi Shyti
Replace the legacy for_each_engine* defines with the ones
implemented in the gem_engine_topology library.
Signed-off-by: Andi Shyti
Cc: Chris Wilson
Cc: Daniel Vetter
Reviewed-by: Tvrtko Ursulin
---
tests/i915/gem_cs_tlb.c | 8
1 file changed, 4 insertions(+), 4
From: Andi Shyti
Engines referred by class and instance are getting more popular,
gem_engine_can_store_dword() should handle the situation.
Suggested-by: Tvrtko Ursulin
Signed-off-by: Andi Shyti
Reviewed-by: Tvrtko Ursulin
---
lib/igt_gt.c | 12 +---
lib/igt_gt.h | 1 +
2 files
From: Andi Shyti
With the new getparam/setparam api, engines are mapped to
context. Use for_each_context_engine() to loop through existing
engines.
Suggested-by: Tvrtko Ursulin
Signed-off-by: Andi Shyti
Reviewed-by: Tvrtko Ursulin
---
lib/igt_dummyload.c | 29 -
From: Andi Shyti
Replace the legacy for_each_engine* defines with the ones
implemented in the gem_engine_topology library.
Signed-off-by: Andi Shyti
Cc: Chris Wilson
Cc: Tvrtko Ursulin
Cc: Daniel Vetter
Reviewed-by: Tvrtko Ursulin
---
tests/i915/gem_ctx_exec.c | 16 +++-
1
On 22/05/2019 14:14, Tvrtko Ursulin wrote:
From: Tvrtko Ursulin
This enables static test enumeration to see the engine.
Signed-off-by: Tvrtko Ursulin
---
lib/igt_gt.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/igt_gt.c b/lib/igt_gt.c
index 5e5b9bef03dc..6b7c037e6d10 100644
Quoting Tvrtko Ursulin (2019-05-22 13:26:43)
>
> On 22/05/2019 12:37, Chris Wilson wrote:
> > The submit-fence + load_balancing apis allow for us to execute a named
> > pair of engines in parallel; that this by submitting a request to one
> > engine, we can then use the generated submit-fence to
== Series Details ==
Series: series starting with [v5,1/2] drm/i915: Make sandybridge_pcode_read()
deal with the second data register
URL : https://patchwork.freedesktop.org/series/60921/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6114_full -> Patchwork_13064_full
From: Tvrtko Ursulin
commit c5d3e39caa456b1e061644b739131f2b54c84c08
Author: Tvrtko Ursulin
Date: Wed May 22 10:00:54 2019 +0100
drm/i915: Engine discovery query
Signed-off-by: Tvrtko Ursulin
---
include/drm-uapi/i915_drm.h | 42 +
1 file
From: Andi Shyti
Replace the legacy for_each_engine* defines with the ones
implemented in the gem_engine_topology library.
Signed-off-by: Andi Shyti
Cc: Chris Wilson
Reviewed-by: Tvrtko Ursulin
---
tests/i915/gem_exec_basic.c | 28 ++--
1 file changed, 14
From: Andi Shyti
Replace the legacy for_each_engine* defines with the ones
implemented in the gem_engine_topology library.
Signed-off-by: Andi Shyti
Cc: Chris Wilson
Reviewed-by: Tvrtko Ursulin
---
tests/i915/gem_exec_store.c | 36 ++--
1 file changed, 18
From: Andi Shyti
Replace the legacy for_each_engine* defines with the ones
implemented in the gem_engine_topology library.
Signed-off-by: Andi Shyti
Cc: Mika Kuoppala
Cc: Chris Wilson
Reviewed-by: Tvrtko Ursulin
---
tests/i915/i915_hangman.c | 15 ---
1 file changed, 4
From: Andi Shyti
we have a "class/instance to eb flags" helper but not the
opposite, add it.
Suggested-by: Tvrtko Ursulin
Signed-off-by: Andi Shyti
Reviewed-by: Tvrtko Ursulin
---
lib/igt_gt.c | 18 ++
lib/igt_gt.h | 2 ++
2 files changed, 20 insertions(+)
diff --git
From: Andi Shyti
Replace the legacy for_each_engine* defines with the ones
implemented in the gem_engine_topology library.
Signed-off-by: Andi Shyti
Cc: Chris Wilson
Reviewed-by: Tvrtko Ursulin
---
tests/i915/gem_wait.c | 24
1 file changed, 12 insertions(+), 12
From: Tvrtko Ursulin
commit c5d3e39caa456b1e061644b739131f2b54c84c08
Author: Tvrtko Ursulin
Date: Wed May 22 10:00:54 2019 +0100
drm/i915: Engine discovery query
Signed-off-by: Tvrtko Ursulin
---
include/drm-uapi/i915_drm.h | 42 +
1 file
From: Andi Shyti
Replace the legacy for_each_engine* defines with the ones
implemented in the gem_engine_topology library.
Signed-off-by: Andi Shyti
Cc: Chris Wilson
Reviewed-by: Tvrtko Ursulin
---
tests/i915/gem_busy.c | 128 --
1 file changed, 48
From: Andi Shyti
Replace the legacy for_each_engine* defines with the ones
implemented in the gem_engine_topology library.
Signed-off-by: Andi Shyti
Cc: Chris Wilson
Reviewed-by: Chris Wilson
---
tests/i915/gem_exec_parallel.c | 26 ++
1 file changed, 14
On Wed, 22 May 2019, Chris Wilson wrote:
> For convenience in avoiding inline spaghetti, keep the type definition
> as a separate header.
>
> Signed-off-by: Chris Wilson
> Reviewed-by: Matthew Auld
> Acked-by: Rodrigo Vivi
> Acked-by: Jani Nikula
Yup, applies to the entire series.
BR,
Jani.
Quoting Mika Kuoppala (2019-05-22 15:32:34)
> Chris Wilson writes:
> > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_context.c
> > b/drivers/gpu/drm/i915/gem/i915_gem_context.c
> > index 5016a3e1f863..a9608d9ced6a 100644
> > --- a/drivers/gpu/drm/i915/gem/i915_gem_context.c
> > +++
Quoting Jani Nikula (2019-05-22 15:50:29)
> intel_mark_busy(), intel_mark_idle(), and skl_cdclk_get_vco() no longer
> exist.
And there are some obsolete comments for them as well, git grep
mark_busy.
-Chris
___
Intel-gfx mailing list
Quoting Tvrtko Ursulin (2019-05-22 14:32:39)
> From: Tvrtko Ursulin
>
> commit c5d3e39caa456b1e061644b739131f2b54c84c08
> Author: Tvrtko Ursulin
> Date: Wed May 22 10:00:54 2019 +0100
>
> drm/i915: Engine discovery query
>
> Signed-off-by: Tvrtko Ursulin
Acked-by: Chris Wilson
1 - 100 of 216 matches
Mail list logo