In this patch, add validity checking of pin configuration from VBT;
If invalid, print log, ignore the configuration.
Without this patch, in case of invalid pin configured by VBT, panic is hit:
[ 45.778820] WARN_ON(!intel_gmbus_is_valid_pin(dev_priv, pin))
[ 45.778820] [ cut here ]
== Series Details ==
Series: drm/i915: enable to read CSB and CSB write pointer from HWSP in GVT-g
VM (rev2)
URL : https://patchwork.freedesktop.org/series/31789/
State : success
== Summary ==
shard-hswtotal:2501 pass:1413 dwarn:0 dfail:0 fail:9 skip:1079
time:9529s
== Logs ==
On Fri, 13 Oct 2017, Dave Airlie wrote:
> On 13 October 2017 at 01:23, Jani Nikula wrote:
>> On Wed, 11 Oct 2017, Jani Nikula wrote:
>>> Hi Dave, more v4.15 features.
>>
>> Okay, so I suck and there's still one more batch to come after this. I'm
>> a bit out of rhythm here. When do you want the
Lets try it and see what breaks!
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_fb_helper.c | 30 +++---
1 file changed, 11 insertions(+), 19 deletions(-)
diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
index 6a31d13f2f81..0414da9
On 10/12/2017 11:47 AM, Sagar Arun Kamble wrote:
On 10/12/2017 11:20 AM, Sagar Arun Kamble wrote:
On 10/11/2017 8:50 PM, Michal Wajdeczko wrote:
On Wed, 11 Oct 2017 10:53:58 +0200, Sagar Arun Kamble
wrote:
GuC interrupts are currently enabled by Logging and disabled in
different
scena
== Series Details ==
Series: HAX: Do not restore mode through fbcon
URL : https://patchwork.freedesktop.org/series/31893/
State : warning
== Summary ==
Series 31893v1 HAX: Do not restore mode through fbcon
https://patchwork.freedesktop.org/api/1.0/series/31893/revisions/1/mbox/
Test chamelium
Quoting Chris Wilson (2017-10-12 23:57:38)
> Quoting Daniele Ceraolo Spurio (2017-10-12 23:27:27)
> > +igt_cork_t *igt_cork_new(int fd);
>
> _new does not imply plugged.
>
> > +void igt_cork_signal(igt_cork_t *cork);
>
> When have you signaled a cork?
>
> > +void igt_cork_free(int fd, igt_cork_
On Thu, 12 Oct 2017, Rodrigo Vivi wrote:
> On Thu, Oct 12, 2017 at 09:30:36PM +, James Ausmus wrote:
>> Rename DP_AUX_CH_CTL_TIME_OUT_1600us to DP_AUX_CH_CTL_TIME_OUT_MAX, as
>> the meaning of the (3 << 26) value varies per platform, but it's always the
>> maximum timeout for that platform. Pr
On Fri, 2017-10-13 at 09:17 +0800, Li, Weinan Z wrote:
> On 10/12/2017 4:16 PM, Chris Wilson wrote:
> > Quoting Weinan Li (2017-10-12 07:50:08)
> > > @@ -398,6 +395,9 @@ static bool csb_force_mmio(struct drm_i915_private
> > > *i915)
> > > if (intel_vtd_active())
> > >
Quoting Weinan Li (2017-10-13 02:46:16)
> Let GVT-g VM read the CSB and CSB write pointer from virtual HWSP, not all
> the host support this feature, need to check the BIT(3) of caps in PVINFO.
>
> v3 : Remove unnecessary comments.
> v4 : Separate VM enable patch with GVT-g implementation patch du
Keep subject as "drm/i915" as this is generic change. Also I saw
i915_runtime_pm_status debugfs output not updated.
Could you please check.
On 10/11/2017 12:02 AM, Sujaritha Sundaresan wrote:
Unifying the various seq_puts messages in debugfs to the simplest one for
feature support.
v2: Clar
On Thu, 12 Oct 2017, Shashank Sharma wrote:
> From the CI builds, its been observed that during a driver
> reload/insert, dp dual mode read function sometimes fails to
> read from LSPCON device over i2c-over-aux channel.
>
> This patch:
> - adds some delay and few retries, allowing a scope for the
On Tue, 10 Oct 2017, "Sharma, Shashank" wrote:
> Regards
>
> Shashank
>
>
> On 10/10/2017 3:42 PM, Saarinen, Jani wrote:
>> Hi,
>>
>>> -Original Message-
>>> From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf
>>> Of Shashank Sharma
>>> Sent: tiistai 10. lokakuuta 20
On Thu, 2017-10-12 at 19:16 +, Michal Wajdeczko wrote:
> Suggested-by: Joonas Lahtinen
Suggested-by, Reported-by and other merit tags should be in the patches
where they get recorded when the series is merged.
Regards, Joonas
--
Joonas Lahtinen
Open Source Technology Center
Intel Corporatio
On 10/11/2017 12:02 AM, Sujaritha Sundaresan wrote:
We currently have two module parameters that control GuC: "enable_guc_loading" and
"enable_guc_submission".
Whenever we need i915_modparams.enable_guc_submission=1, we also need
enable_guc_loading=1.
We also need enable_guc_loading=1 when we
On Fri, 13 Oct 2017 08:49:22 +0200, Sagar Arun Kamble
wrote:
On 10/13/2017 4:24 AM, Michal Wajdeczko wrote:
Checking GuC firmware size can be done in GuC specific code
right before DMA copy as it is unlikely to fail anyway.
v2: rebased
Signed-off-by: Michal Wajdeczko
Cc: Sagar Arun Kamb
On Fri, 13 Oct 2017 08:54:14 +0200, Sagar Arun Kamble
wrote:
On 10/13/2017 4:24 AM, Michal Wajdeczko wrote:
Guc status message printed right after firmware upload may be too
optimistic, as we may fail on subsequent steps. Move that message
to the end of intel_uc_init_hw where we know the s
On Fri, 13 Oct 2017 08:54:14 +0200, Sagar Arun Kamble
wrote:
On 10/13/2017 4:24 AM, Michal Wajdeczko wrote:
Guc status message printed right after firmware upload may be too
optimistic, as we may fail on subsequent steps. Move that message
to the end of intel_uc_init_hw where we know the s
On 10/13/2017 3:44 PM, Michal Wajdeczko wrote:
On Fri, 13 Oct 2017 08:54:14 +0200, Sagar Arun Kamble
wrote:
On 10/13/2017 4:24 AM, Michal Wajdeczko wrote:
Guc status message printed right after firmware upload may be too
optimistic, as we may fail on subsequent steps. Move that message
t
Favor for_each_pipe() macro when looping through pipes.
This patch is related to miscellenous fixes to reduce dependency on
I915_MAX_PIPES.
https://patchwork.freedesktop.org/series/30336/
Signed-off-by: Mika Kahola
---
drivers/gpu/drm/i915/intel_dp_mst.c | 14 --
1 file changed, 8
On Thu, Oct 12, 2017 at 08:59:20PM +0200, Daniel Vetter wrote:
> On Wed, Oct 11, 2017 at 07:04:47PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Add a .get_hw_state() method for planes, returning true or false
> > depending on whether the plane is enabled. Use it to rewrite the
> >
On Thu, Oct 12, 2017 at 09:06:24PM +0200, Daniel Vetter wrote:
> On Wed, Oct 11, 2017 at 07:04:49PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Rename enum plane to enum old_plane_id to make it clear that it only
> > applies to pre-SKL platforms.
> >
> > v2: Reorder patches
> >
On Thu, Oct 12, 2017 at 09:17:40PM +0200, Daniel Vetter wrote:
> On Wed, Oct 11, 2017 at 07:04:52PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > The only relevant difference between i9xx_get_initial_plane_config() and
> > ironlake_get_initial_plane_config() is the HSW/BDW TILEOFF h
On Thu, Oct 12, 2017 at 09:21:07PM +0200, Daniel Vetter wrote:
> On Wed, Oct 11, 2017 at 07:04:53PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Stop using the old for_each_intel_plane_in_state() type iteration
> > macro and replace it with for_each_new_intel_plane_in_state().
> >
On Thu, Oct 12, 2017 at 09:38:20PM +0200, Daniel Vetter wrote:
> On Wed, Oct 11, 2017 at 07:04:54PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Eliminate crtc->plane since it's pretty much a layering violation.
> > We can always get the plane via crtc->primary if we actually need
== Series Details ==
Series: drm/i915: Get rid of hardcoded pipes
URL : https://patchwork.freedesktop.org/series/31900/
State : success
== Summary ==
Series 31900v1 drm/i915: Get rid of hardcoded pipes
https://patchwork.freedesktop.org/api/1.0/series/31900/revisions/1/mbox/
fi-bdw-5557u t
Couple of tests are using either determining submission method, or
pretty printing. Let's move those to helpers in lib.
Signed-off-by: Michał Winiarski
Cc: Arkadiusz Hiler
Cc: Chris Wilson
Cc: Katarzyna Dec
Cc: Petri Latvala
---
lib/igt_aux.c | 18 +
lib/igt_aux.h
Couple of tests are using either determining scheduler capabilities or
pretty printing. Let's move those to helpers in lib. We can also keep
the value obtained from getparam static.
v2: Break the trend of expanding ioctl_wrappers
Signed-off-by: Michał Winiarski
Cc: Arkadiusz Hiler
Cc: Chris Wil
Another example of something that is used across different tests, and
should be moved to lib.
v2: Break the trend of expanding ioctl_wrappers
Signed-off-by: Michał Winiarski
Cc: Arkadiusz Hiler
Cc: Chris Wilson
Cc: Katarzyna Dec
Cc: Petri Latvala
---
lib/i915/gem_context.c| 40 +
We'd like to make ioctl_wrappers a bit thinner, and we plan to add new
helpers in the following patch. Let's move context related helpers before
adding more content.
Signed-off-by: Michał Winiarski
Cc: Arkadiusz Hiler
Cc: Chris Wilson
Cc: Katarzyna Dec
Cc: Petri Latvala
---
.../intel-gpu-too
After GuC code reorg some documentation was left in wrong
place. Move it closer to corresponding definitions.
v2: use consistent name for the GuC (Sagar)
Signed-off-by: Michal Wajdeczko
Cc: Joonas Lahtinen
Cc: Sagar Arun Kamble
Reviewed-by: Sagar Arun Kamble
---
drivers/gpu/drm/i915/intel_gu
GuC status message printed right after firmware upload may be too
optimistic, as we may fail on subsequent steps. Move that message
to the end of intel_uc_init_hw where we know the status for sure.
v2: use dev_info (Chris)
Signed-off-by: Michal Wajdeczko
Cc: Joonas Lahtinen
Cc: Sagar Arun Kambl
Functions should be defined in their use order.
Signed-off-by: Michal Wajdeczko
Cc: Joonas Lahtinen
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/intel_guc_fw.c | 81 +++--
1 file changed, 41 insertions(+), 40 deletions(-)
diff --git a/drivers/gpu/drm/i915
We want to keep ucode xfer functions separate from other
initialization. Once separated, add explicit forcewake.
v2: use BLITTER domain only and add comment (Daniele)
Suggested-by: Joonas Lahtinen
Signed-off-by: Michal Wajdeczko
Cc: Joonas Lahtinen
Cc: Sagar Arun Kamble
Cc: Daniele Ceraolo Sp
Remaining functions in intel_guc_loader.c were focused around
GuC firmware. Rename them to match object-verb pattern and
rename file itself.
Suggested-by: Joonas Lahtinen
Signed-off-by: Michal Wajdeczko
Cc: Joonas Lahtinen
Cc: Sagar Arun Kamble
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i
Some of the error messages from DMC load were too generic and
may be confusing for the user. Lets explicitly add DMC words there.
Also as homepage of DMC firmware is same as for the GuC and Huc,
lets reuse URL definition.
Signed-off-by: Michal Wajdeczko
Cc: Chris Wilson
Cc: Imre Deak
Cc: Mika K
CC: Joonas Lahtinen
Cc: Chris Wilson
v1: was sent with wrong prefix "P v4"
v2: apply review comments from Chris/Daniele
v3: restore DRM_WARN, add firmware url
v4: update DMC messages, drop CI HAX
Michal Wajdeczko (14):
drm/i915: Move intel_guc_wopcm_size to intel_guc.c
drm/i915/guc: Move Gu
Existing code needs some style fixes. To avoid polluting
pure move patch, do it now as separate step.
Suggested-by: Chris Wilson
Signed-off-by: Michal Wajdeczko
Cc: Chris Wilson
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/intel_guc.c | 11 ++-
drivers/gpu/drm/i915/intel_gu
No functional change. Just place the function closer to related
definitions.
Signed-off-by: Michal Wajdeczko
Cc: Joonas Lahtinen
Cc: Sagar Arun Kamble
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/intel_huc.c | 72
1 file changed, 36 insertions(+)
Checking GuC firmware size can be done in GuC specific code
right before DMA copy as it is unlikely to fail anyway.
v2: rebased
Signed-off-by: Michal Wajdeczko
Cc: Sagar Arun Kamble
Cc: Joonas Lahtinen
Reviewed-by: Sagar Arun Kamble #1
---
drivers/gpu/drm/i915/intel_guc_fw.c | 7 ++-
dri
In case of firmware fetch failure we should help user
find latest firmware.
Suggested-by: Chris Wilson
Signed-off-by: Michal Wajdeczko
Cc: Chris Wilson
---
drivers/gpu/drm/i915/intel_uc_fw.c | 2 ++
drivers/gpu/drm/i915/intel_uc_fw.h | 2 ++
2 files changed, 4 insertions(+)
diff --git a/drive
In case of GuC firmware loading failure we were reporting
DRM_ERROR also for case when GuC loading was not strictly
required.
Signed-off-by: Michal Wajdeczko
Cc: Chris Wilson
Cc: Joonas Lahtinen
Cc: Sagar Arun Kamble
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/intel_uc.c | 9 +
Firmware loading for GuC and HuC are similar. Move common code
into generic function for maximum reuse.
v2: change message levels (Chris)
Signed-off-by: Michal Wajdeczko
Cc: Joonas Lahtinen
Cc: Sagar Arun Kamble
Cc: Anusha Srivatsa
Cc: Sujaritha Sundaresan
Cc: Chris Wilson
Reviewed-by: Chri
Time to remove less important info and make messages clear
and consistent.
v2: change some message levels (Chris)
v3: restore DRM_WARN (Chris)
Signed-off-by: Michal Wajdeczko
Cc: Sagar Arun Kamble
Cc: Joonas Lahtinen
Cc: Chris Wilson
Reviewed-by: Chris Wilson #2
---
drivers/gpu/drm/i915/int
Function intel_guc_wopcm_size didn't fit well in the old place.
With this move upcoming cleanup will be simpler.
Suggested-by: Joonas Lahtinen
Signed-off-by: Michal Wajdeczko
Cc: Joonas Lahtinen
Cc: Sagar Arun Kamble
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/intel_guc.c| 11
== Series Details ==
Series: HAX: Do not restore mode through fbcon
URL : https://patchwork.freedesktop.org/series/31893/
State : failure
== Summary ==
Test kms_chv_cursor_fail:
Subgroup pipe-C-256x256-bottom-edge:
pass -> INCOMPLETE (shard-hsw)
Subgroup p
On Thu, Oct 12, 2017 at 02:52:40PM +0100, Chris Wilson wrote:
> Quoting Imre Deak (2017-10-12 14:22:06)
> > The clone() system call with a larger executable (like /bin/sh) may have
>
> One thing to note it's not the target executable size that's
> problematic, it's ours. It is our mm that is being
On Thu, 2017-10-12 at 15:29 -0700, Daniele Ceraolo Spurio wrote:
> Respin with review fixes for meson build and gem_concurrent_all.
> I'll send the new tests for pread/pwrite suggested by Chris as a
> separated series as they depend on the other cleanup I've sent
> and this series can potentially b
On Fri, Oct 13, 2017 at 01:00:17PM +0200, Michał Winiarski wrote:
> Couple of tests are using either determining submission method, or
> pretty printing. Let's move those to helpers in lib.
>
> Signed-off-by: Michał Winiarski
> Cc: Arkadiusz Hiler
> Cc: Chris Wilson
>> Cc: Katarzyna Dec
Rev
== Series Details ==
Series: series starting with [1/4] lib: Extract helpers for determining
submission method
URL : https://patchwork.freedesktop.org/series/31904/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
58616272b23efce1e62a3ee0d37e13de6ffc012f igt
On Fri, Oct 13, 2017 at 01:00:20PM +0200, Michał Winiarski wrote:
> Another example of something that is used across different tests, and
> should be moved to lib.
>
> v2: Break the trend of expanding ioctl_wrappers
>
> Signed-off-by: Michał Winiarski
> Cc: Arkadiusz Hiler
> Cc: Chris Wilson
>
Quoting Michał Winiarski (2017-10-13 12:00:17)
> Couple of tests are using either determining submission method, or
> pretty printing. Let's move those to helpers in lib.
>
> Signed-off-by: Michał Winiarski
> Cc: Arkadiusz Hiler
> Cc: Chris Wilson
> Cc: Katarzyna Dec
> Cc: Petri Latvala
> ---
Reviewed-by:Katarzyna Dec ,
Cheers,
Kasia
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Reviewed-by: Katarzyna Dec
Cheers,
Kasia
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
The clone() system call has difficulty to make progress if interrupted
frequently by the signal helper process. At least on an APL, like in the
Bugzilla ticket below, this can introduce minutes of overhead to a
single system() call (leading to a global CI timeout). To get rid of the
overhead suspen
The clone() system call with a larger executable (like /bin/sh) may have
difficulty to make progress on some platforms if interrupted frequently.
So suspend the signal helper process for the duration of the syscall.
This is needed to solve an actual problem by the next patch.
v2:
- Clarify/fix cod
Quoting Michał Winiarski (2017-10-13 12:00:19)
> We'd like to make ioctl_wrappers a bit thinner, and we plan to add new
> helpers in the following patch. Let's move context related helpers before
> adding more content.
>
> Signed-off-by: Michał Winiarski
> Cc: Arkadiusz Hiler
> Cc: Chris Wilson
Quoting Michał Winiarski (2017-10-13 12:00:20)
> Another example of something that is used across different tests, and
> should be moved to lib.
>
> v2: Break the trend of expanding ioctl_wrappers
>
> Signed-off-by: Michał Winiarski
> Cc: Arkadiusz Hiler
> Cc: Chris Wilson
> Cc: Katarzyna Dec
== Series Details ==
Series: Firmware code reorg (rev3)
URL : https://patchwork.freedesktop.org/series/31846/
State : success
== Summary ==
Series 31846v3 Firmware code reorg
https://patchwork.freedesktop.org/api/1.0/series/31846/revisions/3/mbox/
fi-bdw-5557u total:289 pass:268 dwarn:0
Quoting Michal Wajdeczko (2017-10-13 12:11:33)
> diff --git a/drivers/gpu/drm/i915/intel_uc_fw.h
> b/drivers/gpu/drm/i915/intel_uc_fw.h
> index 5c01849..1d580fa 100644
> --- a/drivers/gpu/drm/i915/intel_uc_fw.h
> +++ b/drivers/gpu/drm/i915/intel_uc_fw.h
> @@ -27,6 +27,8 @@
>
> struct drm_i915_p
Quoting Michal Wajdeczko (2017-10-13 12:11:34)
> Some of the error messages from DMC load were too generic and
> may be confusing for the user. Lets explicitly add DMC words there.
> Also as homepage of DMC firmware is same as for the GuC and Huc,
> lets reuse URL definition.
>
> Signed-off-by: Mi
Quoting Chris Wilson (2017-10-13 12:50:53)
> Quoting Michal Wajdeczko (2017-10-13 12:11:33)
> > diff --git a/drivers/gpu/drm/i915/intel_uc_fw.h
> > b/drivers/gpu/drm/i915/intel_uc_fw.h
> > index 5c01849..1d580fa 100644
> > --- a/drivers/gpu/drm/i915/intel_uc_fw.h
> > +++ b/drivers/gpu/drm/i915/int
Since the introduction of debugfs/i915_drop_caches, we have offered the
ability to wait upon all outstanding batches. This is more efficient and
less error prone (one example is the use of context priorities, we have
to idle at the lowest in order not to jump over any low priority tasks
we want to
---
tests/gem_sync.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/gem_sync.c b/tests/gem_sync.c
index c9e2f014..e178db25 100644
--- a/tests/gem_sync.c
+++ b/tests/gem_sync.c
@@ -876,6 +876,8 @@ igt_main
}
for (e = intel_execution_engines; e->name; e++) {
+
General update to reflect current state of things.
Signed-off-by: Arkadiusz Hiler
---
tests/core_prop_blob.c | 2 --
tests/gem_busy.c | 4 ++--
tests/gem_close_race.c | 2 --
tests/gem_cpu_reloc.c | 1 -
tests/gem_ctx_exec.c | 4 ++--
tests/gem_ctx_switch.c | 3 +
Signed-off-by: Arkadiusz Hiler
---
tests/intel-ci/Makefile.am | 1 -
tests/intel-ci/README| 7 --
tests/intel-ci/fast-feedback-simulation.testlist | 91
3 files changed, 99 deletions(-)
delete mode 100644 tests/intel-ci
Supplement dmesg with the function call graph to try and help identify
why the suspend failed. On the other hand, it may be very noisy and
perturb the system due to its overhead...
Signed-off-by: Chris Wilson
---
lib/igt_aux.c | 15 +++
1 file changed, 11 insertions(+), 4 deletions(-
If the kernel has tracing support builtin, we can enable around
troublesome tests to obtain greater detail from the kernel that may
prove invaluable in debugging the problem.
Signed-off-by: Chris Wilson
---
lib/Makefile.sources | 2 +
lib/igt_core.c | 5 ++
lib/igt_ftrace.c | 181 +
is_mountpoint() asserts rather than report the error. Normally this
isn't a problem, except for atypical selftests.
Signed-off-by: Chris Wilson
---
lib/igt_debugfs.c | 33 -
1 file changed, 20 insertions(+), 13 deletions(-)
diff --git a/lib/igt_debugfs.c b/lib/ig
On Fri, Oct 13, 2017 at 02:22:26PM +0300, Joonas Lahtinen wrote:
> On Thu, 2017-10-12 at 15:29 -0700, Daniele Ceraolo Spurio wrote:
> > Respin with review fixes for meson build and gem_concurrent_all.
> > I'll send the new tests for pread/pwrite suggested by Chris as a
> > separated series as they
== Series Details ==
Series: series starting with [v2,1/2] aux: Suspend signal helper for shell
commands
URL : https://patchwork.freedesktop.org/series/31908/
State : failure
== Summary ==
IGT patchset tested on top of latest successful build
58616272b23efce1e62a3ee0d37e13de6ffc012f igt/gem_e
== Series Details ==
Series: series starting with [1/2] lib: Ask the kernel to quiesce the GPU
URL : https://patchwork.freedesktop.org/series/31909/
State : failure
== Summary ==
IGT patchset build failed on latest successful build
58616272b23efce1e62a3ee0d37e13de6ffc012f igt/gem_eio: Check ha
Quoting Chris Wilson (2017-10-13 13:02:58)
> If the kernel has tracing support builtin, we can enable around
> troublesome tests to obtain greater detail from the kernel that may
> prove invaluable in debugging the problem.
Michał pointed out that there is a ftrace-dump-on-oops flag, which could
v
Since the introduction of debugfs/i915_drop_caches, we have offered the
ability to wait upon all outstanding batches. This is more efficient and
less error prone (one example is the use of context priorities, we have
to idle at the lowest in order not to jump over any low priority tasks
we want to
---
tests/gem_sync.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/gem_sync.c b/tests/gem_sync.c
index c9e2f014..41cfe6c1 100644
--- a/tests/gem_sync.c
+++ b/tests/gem_sync.c
@@ -876,6 +876,8 @@ igt_main
}
for (e = intel_execution_engines; e->name; e++) {
+
== Series Details ==
Series: series starting with [1/2] tests: Clean up igt_skip_on_simulation() uses
URL : https://patchwork.freedesktop.org/series/31910/
State : failure
== Summary ==
IGT patchset tested on top of latest successful build
58616272b23efce1e62a3ee0d37e13de6ffc012f igt/gem_eio:
On Fri, Oct 13, 2017 at 01:02:57PM +0100, Chris Wilson wrote:
> is_mountpoint() asserts rather than report the error. Normally this
> isn't a problem, except for atypical selftests.
>
> Signed-off-by: Chris Wilson
Reviewed-by: Ville Syrjälä
> ---
> lib/igt_debugfs.c | 33 -
== Series Details ==
Series: series starting with [RFC,1/3] lib/debugfs: Make is_mountpoint()
non-fatal
URL : https://patchwork.freedesktop.org/series/31911/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
58616272b23efce1e62a3ee0d37e13de6ffc012f igt/gem_ei
This patch parse DSI backlight/cabc ports info from
VBT and save them inside local structure. This saved info
can be directly used while initializing DSI for different
platforms instead of parsing for each platform.
V2: Changes:
- Typo fix in commit message.
- Move up newly added port vari
This patch re-use already parsed DSI backlight/cabc ports
info for saving it inside struct intel_dsi rather than
parsing it at the time of DSI initialization.
V2: Remove backlight and cabc variable initialization (Jani N).
Reviewed-by: Jani Nikula
Signed-off-by: Madhav Chauhan
---
drivers/gpu/
== Series Details ==
Series: drm/i915: Get rid of hardcoded pipes
URL : https://patchwork.freedesktop.org/series/31900/
State : success
== Summary ==
Test kms_setmode:
Subgroup basic:
fail -> PASS (shard-hsw) fdo#99912
Test kms_atomic_transition:
Sub
== Series Details ==
Series: series starting with [1/2] lib: Ask the kernel to quiesce the GPU
URL : https://patchwork.freedesktop.org/series/31913/
State : failure
== Summary ==
IGT patchset tested on top of latest successful build
58616272b23efce1e62a3ee0d37e13de6ffc012f igt/gem_eio: Check h
Before modifying the ring register (RING_START, HEAD, TAIL, CTL) we
first make sure it is stopped (or else the hw may not resample the
registers). However, we do not need to let the hw restart until after we
have reprogrammed all the rings. This should help prevent situations
where pending operatio
Always try to stop the rings, even if the GPU reset itself has been
disabled (via modparam i915.reset). This should at least stop the hw
from spinning in the background consuming resources (e.g. power and
memory bandwidth) letting the system rest-in-peace.
References: https://bugs.freedesktop.org/
Chris Wilson writes:
> Before modifying the ring register (RING_START, HEAD, TAIL, CTL) we
> first make sure it is stopped (or else the hw may not resample the
> registers). However, we do not need to let the hw restart until after we
> have reprogrammed all the rings. This should help prevent si
== Series Details ==
Series: series starting with [1/4] lib: Extract helpers for determining
submission method
URL : https://patchwork.freedesktop.org/series/31904/
State : success
== Summary ==
Test kms_setmode:
Subgroup basic:
fail -> PASS (shard-hsw) fdo
Chris Wilson writes:
> Always try to stop the rings, even if the GPU reset itself has been
> disabled (via modparam i915.reset). This should at least stop the hw
> from spinning in the background consuming resources (e.g. power and
> memory bandwidth) letting the system rest-in-peace.
>
> Referen
== Series Details ==
Series: series starting with [1/2] drm/i915: Parse DSI backlight/cabc ports.
URL : https://patchwork.freedesktop.org/series/31914/
State : success
== Summary ==
Series 31914v1 series starting with [1/2] drm/i915: Parse DSI backlight/cabc
ports.
https://patchwork.freedeskt
From: Ville Syrjälä
Respin of the plane readout/sanitation series. I tried to address all of
Daniel's review comments, and tossed in the requested plane state
verification patch on top. Also dropped the primary plane windowing patch
for now.
Entire series available here:
git://github.com/vsyrjal
From: Ville Syrjälä
Add a .get_hw_state() method for planes, returning true or false
depending on whether the plane is enabled. Use it to rewrite the
plane enabled/disabled asserts in platform agnostic fashion.
We do lose the pre-gen4 plane<->pipe mapping checks, but since we're
supposed sanitiz
From: Ville Syrjälä
Replace the 0 and 1 with PLANE_A and PLANE_B in the pre-g4x wm code.
v2: s/old_plane_id/i9xx_plane_id/ (Daniel)
Cc: Daniel Vetter
Signed-off-by: Ville Syrjälä
Reviewed-by: Daniel Vetter
---
drivers/gpu/drm/i915/i915_drv.h | 3 ++-
drivers/gpu/drm/i915/intel_pm.c | 36 ++
From: Ville Syrjälä
Unify the plane disabling during state readout by pulling the code into
a new helper intel_plane_disable_noatomic(). We'll also read out the
state of all planes, so that we know which planes really need to be
diabled.
Additonally we change the plane<->pipe mapping sanitation
From: Ville Syrjälä
Rename enum plane to enum i9xx_plane_id to make it clear that it only
applies to pre-SKL platforms.
enum i9xx_plane_id is a global identifier, whereas enum plane_id is
per-pipe. We need the old global identifier to index the primary plane
(and the pre-g4x sprite C if we ever
From: Ville Syrjälä
The only relevant difference between i9xx_get_initial_plane_config() and
ironlake_get_initial_plane_config() is the HSW/BDW TILEOFF handling.
Add that to i9xx_get_initial_plane_config() and nuke
ironlake_get_initial_plane_config().
Signed-off-by: Ville Syrjälä
Reviewed-by: D
From: Ville Syrjälä
Check that the planes on each crtc undergoing a modeset are in the
state we expect them to be. For now we can only check whether each
plane is correctly enabled or disabled. In the future we may want
to expand the plane state readout to support a more through
verification.
Cc
From: Ville Syrjälä
Use enum pipe, enum plane_id, and enum i9xx_plane_id consistently in the
initial framebuffe readout.
v2: Use old_plane_id in the ilk code
v3: s/old_plane_id/i9xx_plane_id/ (Daniel)
Cc: Daniel Vetter
Signed-off-by: Ville Syrjälä
Reviewed-by: Daniel Vetter
---
drivers/gpu/
From: Ville Syrjälä
Eliminate crtc->plane since it's pretty much a layering violation.
We can always get the plane via crtc->primary if we actually need it.
The only ugly thing left is plane_to_crtc_mapping[], but that's
still needed by the pre-g4x watermark code.
v2: Removed a misplaced commen
From: Ville Syrjälä
Stop using the old for_each_intel_plane_in_state() type iteration
macro and replace it with for_each_new_intel_plane_in_state().
And similarly replace drm_atomic_get_existing_crtc_state() with
intel_atomic_get_new_crtc_state(). Switch over to intel_ types
as well to make the c
From: Ville Syrjälä
Since we now have a ->get_hw_state() method for planes, let's use
that during the initial plane fb readout.
Cc: Daniel Vetter
Suggested-by: Daniel Vetter
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/intel_display.c | 30 --
1 file chan
Op 12-10-17 om 14:05 schreef Daniel Vetter:
> Hi Dave,
>
> drm-misc-next-2017-10-12:
> More 4.15 drm-misc stuff:
>
> Cross-subsystem Changes:
> - bridge cleanup refactor (Benjamin Gaignard)
>
> Core Changes:
> - less surprising atomic iterators (Maarten), fixes an oops introduced
> in drm-next
>
1 - 100 of 225 matches
Mail list logo
