Quoting Joonas Lahtinen (2017-11-28 08:15:13)
> On Mon, 2017-11-27 at 16:54 +0200, Arkadiusz Hiler wrote:
> > Hey all,
> >
> > For some time already CI sends out 1-2 mails per series per (re)run, i.e.
> > BAT
> > results and "full IGT" results (if BAT has not failed).
> >
> > Recently we have ad
On Tue, Nov 28, 2017 at 11:06 AM, Chris Wilson wrote:
> Quoting Joonas Lahtinen (2017-11-28 08:15:13)
>> On Mon, 2017-11-27 at 16:54 +0200, Arkadiusz Hiler wrote:
>> > Hey all,
>> >
>> > For some time already CI sends out 1-2 mails per series per (re)run, i.e.
>> > BAT
>> > results and "full IGT"
Quoting Daniel Vetter (2017-11-28 10:08:56)
> On Tue, Nov 28, 2017 at 11:06 AM, Chris Wilson
> wrote:
> > Quoting Joonas Lahtinen (2017-11-28 08:15:13)
> >> On Mon, 2017-11-27 at 16:54 +0200, Arkadiusz Hiler wrote:
> >> > Hey all,
> >> >
> >> > For some time already CI sends out 1-2 mails per ser
---
integration-manifest | 28
1 file changed, 28 insertions(+)
create mode 100644 integration-manifest
diff --git a/integration-manifest b/integration-manifest
new file mode 100644
index ..2048d8f934c9
--- /dev/null
+++ b/integration-manifest
@@ -0,0 +1,
This reverts commit 2cffad7bad83157f89332872015f4305d2ac09ac.
Lost interrupts upon resume, characterised by
<0>[ 1806.220332] do_IRQ: 4.33 No irq handler for vecto
---
arch/x86/include/asm/apic.h | 1 -
arch/x86/include/asm/irq.h| 4 ++
arch/x86/kernel/apic/vector.c | 32 +-
On Sat, Nov 25, 2017 at 06:33:39PM +0100, Hans de Goede wrote:
> Ideally we could use the VBT for this, that would be simple, in
> intel_dsi_init() check dev_priv->vbt.dsi.config->rotation, set
> connector->display_info.panel_orientation accordingly and call
> drm_connector_init_panel_orientation_p
+ Li (for WOPCM)
On Mon, 2017-11-27 at 11:54 -0800, Sujaritha Sundaresan wrote:
> We currently have two module parameters that control GuC:
> "enable_guc_loading" and "enable_guc_submission". Whenever
> we need submission=1, we also need loading=1.We also need
> loading=1 when we want to want to v
On Sat, Nov 25, 2017 at 06:33:34PM +0100, Hans de Goede wrote:
> Hi All,
>
> Here is v6 of my series to add a "panel orientation" property to
> the drm-connector for the LCD panel to let userspace know about LCD
> panels which are not mounted upright, as well as detecting upside-down
> panels with
On Sat, Nov 25, 2017 at 08:27:31PM +0100, Hans de Goede wrote:
> This fixes the following make kerneldocs messages:
>
> ./include/drm/drm_mode_config.h:772: warning: No description found for
> parameter 'modifiers_property'
> ./include/drm/drm_mode_config.h:772: warning: Excess struct member
> '
Op 27-11-17 om 11:34 schreef Chris Wilson:
> Quoting Maarten Lankhorst (2017-11-27 10:18:46)
>> Op 27-11-17 om 10:45 schreef Chris Wilson:
>>> Previously we would enable hotplug polling on the outputs immediately
>>> upon construction. This would allow a very early hotplug event to
>>> trigger befo
On 11/28/2017 1:24 AM, Sujaritha Sundaresan wrote:
We currently have two module parameters that control GuC:
"enable_guc_loading" and "enable_guc_submission". Whenever
we need submission=1, we also need loading=1.We also need
loading=1 when we want to want to verify the HuC, which
is every time
== Series Details ==
Series: Revert "x86/irq: Simplify hotplug vector accounting"
URL : https://patchwork.freedesktop.org/series/34527/
State : warning
== Summary ==
Series 34527v1 Revert "x86/irq: Simplify hotplug vector accounting"
https://patchwork.freedesktop.org/api/1.0/series/34527/revis
From: Tvrtko Ursulin
In commit 0426c0465461 ("drm/i915/pmu: Only allow running on a single
CPU") I attempted to clarify the CPU hotplug logic in our PMU
implementation, but missed that a more logical error to return, when
attempting to initialize an event on a currently inactive CPU, is -EINVAL
r
From: Tvrtko Ursulin
i915 started returning -EINVAL for incorrect CPU.
Signed-off-by: Tvrtko Ursulin
---
tests/perf_pmu.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c
index 8d58ecea3528..46c39795a141 100644
--- a/tests/perf_pmu.c
Quoting Tvrtko Ursulin (2017-11-28 10:55:15)
> From: Tvrtko Ursulin
>
> In commit 0426c0465461 ("drm/i915/pmu: Only allow running on a single
> CPU") I attempted to clarify the CPU hotplug logic in our PMU
> implementation, but missed that a more logical error to return, when
> attempting to init
Quoting Tvrtko Ursulin (2017-11-28 10:55:36)
> From: Tvrtko Ursulin
>
> i915 started returning -EINVAL for incorrect CPU.
>
> Signed-off-by: Tvrtko Ursulin
Reviewed-by: Chris Wilson
-Chris
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
ht
Previously we would enable hotplug polling on the outputs immediately
upon construction. This would allow a very early hotplug event to
trigger before we had finishing setting up the driver to handle it.
Instead, move the output polling to the last step of registration, after
we have set up all han
This is just a very plain stress test that doesn't do any verification,
and is entirely duplicated by the other context tests. The test
currently leaks objects from every thread on every pass (which is
triggering an oom on smaller machines), while fixing it would be trivial,
it also is pointless as
== Series Details ==
Series: drm/i915/pmu: Return -EINVAL when selecting the inactive CPU
URL : https://patchwork.freedesktop.org/series/34529/
State : success
== Summary ==
Series 34529v1 drm/i915/pmu: Return -EINVAL when selecting the inactive CPU
https://patchwork.freedesktop.org/api/1.0/se
Some drivers like i915 start with crtc's enabled, but with deferred
fbcon setup they were no longer disabled as part of fbdev setup.
Headless units could no longer enter pc3 state because the crtc was
still enabled.
Fix this by calling restore_fbdev_mode when we would have called
it otherwise once
On Mon, Nov 27, 2017 at 09:10:37PM +0530, Sagar Arun Kamble wrote:
> I feel we generally tend to ignore the results mails for series that
> we are not actively involved on (although we might be interested in
> series itself). Also if number of revisions some series can undergo is
> high, this tende
== Series Details ==
Series: drm/i915: Enable hotplug polling after registering the outputs (rev2)
URL : https://patchwork.freedesktop.org/series/34445/
State : success
== Summary ==
Series 34445v2 drm/i915: Enable hotplug polling after registering the outputs
https://patchwork.freedesktop.org
On Tue, Nov 28, 2017 at 12:16:03PM +0100, Maarten Lankhorst wrote:
> Some drivers like i915 start with crtc's enabled, but with deferred
> fbcon setup they were no longer disabled as part of fbdev setup.
> Headless units could no longer enter pc3 state because the crtc was
> still enabled.
>
> Fix
Hey,
Op 21-11-17 om 15:02 schreef Juha-Pekka Heikkila:
> Gen10 onwards 90 and 270 degree rotations are supported for RGB565 format.
>
> v2 (Ville Syrjälä):
> As a side effect to keep bad-pixel-format test valid on all supported
> platforms it need to use DRM_FORMAT_C8 now.
>
> While at it clean up
== Series Details ==
Series: drm/fb_helper: Disable all crtc's when initial setup fails.
URL : https://patchwork.freedesktop.org/series/34532/
State : success
== Summary ==
Series 34532v1 drm/fb_helper: Disable all crtc's when initial setup fails.
https://patchwork.freedesktop.org/api/1.0/seri
== Series Details ==
Series: tests/perf_pmu: Sync invalid-init with i915 changes
URL : https://patchwork.freedesktop.org/series/34530/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
53f2b4b4ce2372da0b3b741d934dd5a9110ac1e2 igt/perf_pmu: Keep batch_duration_
== Series Details ==
Series: Revert "x86/irq: Simplify hotplug vector accounting"
URL : https://patchwork.freedesktop.org/series/34527/
State : failure
== Summary ==
Test prime_busy:
Subgroup wait-hang-bsd:
pass -> DMESG-WARN (shard-hsw)
Subgroup hang-bsd:
Quoting Patchwork (2017-11-28 12:22:22)
> == Series Details ==
>
> Series: Revert "x86/irq: Simplify hotplug vector accounting"
> URL : https://patchwork.freedesktop.org/series/34527/
> State : failure
And still the do_IRQ failures and what appears to be lost interrupts.
This was not the patch
== Series Details ==
Series: igt: Remove gem_ctx_basic (rev2)
URL : https://patchwork.freedesktop.org/series/34500/
State : warning
== Summary ==
IGT patchset tested on top of latest successful build
53f2b4b4ce2372da0b3b741d934dd5a9110ac1e2 igt/perf_pmu: Keep batch_duration_ns
as the minimum
From: Tvrtko Ursulin
Move the execlists specific setup out of intel_engine_setup_common. This
was supposed to be only for backend agnostic bits. At the same time rename
it to intel_engine_setup_execlist to follow the setup vs init naming
convetion we have.
Signed-off-by: Tvrtko Ursulin
---
dri
From: Tvrtko Ursulin
Will be adding a new per-engine flags shortly so it makes sense
to consolidate.
v2: Keep the original code flow in intel_engine_cleanup_cmd_parser.
(Joonas Lahtinen)
Signed-off-by: Tvrtko Ursulin
Suggested-by: Chris Wilson
Reviewed-by: Chris Wilson
Reviewed-by: Sagar
From: Tvrtko Ursulin
Sagar noticed the check can be consolidated between the engine stats
implementation and the PMU.
My first choice was a static inline helper but that got into include
ordering mess quickly fast so I went with a macro instead. At some point
we should perhaps looking into takin
From: Tvrtko Ursulin
Wire up the engine busy stats accounting to the GuC submission backend.
Since there is not context out interrupt we need to place the accounting
callbacks per-request.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/intel_guc_submission.c | 9 ++---
1 file chan
== Series Details ==
Series: drm/i915/pmu: Return -EINVAL when selecting the inactive CPU
URL : https://patchwork.freedesktop.org/series/34529/
State : failure
== Summary ==
Test gem_exec_capture:
Subgroup capture-blt:
pass -> DMESG-WARN (shard-snb) fdo#103950 +9
Quoting Tvrtko Ursulin (2017-11-28 12:41:27)
> From: Tvrtko Ursulin
>
> Move the execlists specific setup out of intel_engine_setup_common. This
> was supposed to be only for backend agnostic bits. At the same time rename
> it to intel_engine_setup_execlist to follow the setup vs init naming
> co
Quoting Tvrtko Ursulin (2017-11-28 12:41:29)
> diff --git a/drivers/gpu/drm/i915/intel_guc_submission.c
> b/drivers/gpu/drm/i915/intel_guc_submission.c
> index cf1cc2cb6722..a8e63779de79 100644
> --- a/drivers/gpu/drm/i915/intel_guc_submission.c
> +++ b/drivers/gpu/drm/i915/intel_guc_submission.c
Quoting Chris Wilson (2017-11-28 11:01:47)
> @@ -1266,6 +1271,13 @@ static void i915_driver_unregister(struct
> drm_i915_private *dev_priv)
> intel_fbdev_unregister(dev_priv);
> intel_audio_deinit(dev_priv);
>
> + /*
> +* After flushing the fbdev (incl. a late async
From: Tvrtko Ursulin
Sagar noticed the check can be consolidated between the engine stats
implementation and the PMU.
My first choice was a static inline helper but that got into include
ordering mess quickly fast so I went with a macro instead. At some point
we should perhaps looking into takin
From: Tvrtko Ursulin
Wire up the engine busy stats accounting to the GuC submission backend.
Since there is not context out interrupt we need to place the accounting
callbacks per-request.
v2: Rebase.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/intel_guc_submission.c | 4 ++--
1 f
== Series Details ==
Series: drm/i915: Enable hotplug polling after registering the outputs (rev2)
URL : https://patchwork.freedesktop.org/series/34445/
State : success
== Summary ==
Blacklisted hosts:
shard-apltotal:2509 pass:1582 dwarn:15 dfail:11 fail:13 skip:883
time:11204s
sha
On 28/11/2017 12:48, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2017-11-28 12:41:27)
From: Tvrtko Ursulin
Move the execlists specific setup out of intel_engine_setup_common. This
was supposed to be only for backend agnostic bits. At the same time rename
it to intel_engine_setup_execlist to f
== Series Details ==
Series: drm/fb_helper: Disable all crtc's when initial setup fails.
URL : https://patchwork.freedesktop.org/series/34532/
State : success
== Summary ==
Blacklisted hosts:
shard-hswtotal:2662 pass:1528 dwarn:8 dfail:2 fail:8 skip:1116
time:9259s
shard-kbl
== Series Details ==
Series: series starting with [1/4] drm/i915: Move execlists setup out of common
(rev3)
URL : https://patchwork.freedesktop.org/series/34541/
State : success
== Summary ==
Series 34541v3 series starting with [1/4] drm/i915: Move execlists setup out of
common
https://patch
Quoting Chris Wilson (2017-11-28 13:02:09)
> Quoting Chris Wilson (2017-11-28 11:01:47)
> > @@ -1266,6 +1271,13 @@ static void i915_driver_unregister(struct
> > drm_i915_private *dev_priv)
> > intel_fbdev_unregister(dev_priv);
> > intel_audio_deinit(dev_priv);
> >
> > + /*
Hi,
On 28-11-17 11:27, Daniel Vetter wrote:
On Sat, Nov 25, 2017 at 06:33:34PM +0100, Hans de Goede wrote:
Hi All,
Here is v6 of my series to add a "panel orientation" property to
the drm-connector for the LCD panel to let userspace know about LCD
panels which are not mounted upright, as well
On Mon, 27 Nov 2017, Sinan Kaya wrote:
> pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as
> where a PCI device is present. This restricts the device drivers to be
> reused for other domain numbers.
>
> Getting ready to remove pci_get_bus_and_slot() function in favor of
> pc
== Series Details ==
Series: tests/perf_pmu: Sync invalid-init with i915 changes
URL : https://patchwork.freedesktop.org/series/34530/
State : success
== Summary ==
Blacklisted hosts:
shard-apltotal:2348 pass:1453 dwarn:20 dfail:18 fail:10 skip:838
time:10677s
shard-hswtota
Hi,
On 28-11-17 11:28, Daniel Vetter wrote:
On Sat, Nov 25, 2017 at 08:27:31PM +0100, Hans de Goede wrote:
This fixes the following make kerneldocs messages:
./include/drm/drm_mode_config.h:772: warning: No description found for
parameter 'modifiers_property'
./include/drm/drm_mode_config.h:7
Testing the texture read performance shows that the same tuning for
the SQ credits is needed on GLK as on BXT/APL. This has been also
confirmed by Altug from the HW team.
V4: Rebase + fix
Signed-off-by: Valtteri Rantala
---
drivers/gpu/drm/i915/intel_engine_cs.c | 16 +---
1 file cha
Hi,
Pulled these in. There was a pretty big conflict to drm-tip which I
solved. Please give it some additional testing as I only compile
tested.
Regards, Joonas
On Tue, 2017-11-28 at 17:31 +0800, Zhenyu Wang wrote:
> Hi,
>
> Pls pull gvt fixes for 4.15-rc2, which contains one backport
> from f2
Ideally we'd create the key through a macro at the real callers and
pass it all the way down. This would give us better coverage for cases
where a bunch of kthreads are created for the same thing.
But this gets the job done meanwhile and unblocks our CI. Refining
later on is always possible.
Signe
On Mon, Nov 27, 2017 at 03:14:21PM -0800, Lucas De Marchi wrote:
> On Thu, Nov 23, 2017 at 7:21 AM, Ville Syrjälä
> wrote:
> > On Wed, Nov 22, 2017 at 10:55:14AM -0800, Lucas De Marchi wrote:
> >> WA Display #1178 is meant to fix Aux channel voltage swing too low with
> >> some type C dongles. Alt
Quoting Patchwork (2017-11-28 13:08:29)
> == Series Details ==
>
> Series: drm/i915: Enable hotplug polling after registering the outputs (rev2)
> URL : https://patchwork.freedesktop.org/series/34445/
> State : success
Maarten acked v2, so applied. At the worst it's an improvement which
we'll h
On Mon, Nov 27, 2017 at 11:57:46AM -0500, Sinan Kaya wrote:
> pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as
> where a PCI device is present. This restricts the device drivers to be
> reused for other domain numbers.
>
> Getting ready to remove pci_get_bus_and_slot() functi
Hi Hans,
I love your patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v4.15-rc1 next-20171128]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits
On 11/28/2017 1:40 AM, Chris Wilson wrote:
Quoting john.c.harri...@intel.com (2017-11-28 09:10:59)
From: John Harrison
The call to enable signaling was occuring after the request had been
sent to the GuC for execution on the hardware. That means that it is
possible for the request to actually
Instead of trying different seq_puts messages, lets use common
-ENODEV error code to indicate missing/unsupported feature.
Suggested-by: Chris Wilson
Signed-off-by: Michal Wajdeczko
Cc: Chris Wilson
Cc: Joonas Lahtinen
Cc: Sagar Arun Kamble
Cc: Sujaritha Sundaresan
---
drivers/gpu/drm/i915/
== Series Details ==
Series: drm/i915/glk: Apply WaProgramL3SqcReg1DefaultForPerf for GLK too (rev4)
URL : https://patchwork.freedesktop.org/series/33772/
State : success
== Summary ==
Series 33772v4 drm/i915/glk: Apply WaProgramL3SqcReg1DefaultForPerf for GLK too
https://patchwork.freedesktop
== Series Details ==
Series: series starting with [1/4] drm/i915: Move execlists setup out of common
(rev3)
URL : https://patchwork.freedesktop.org/series/34541/
State : success
== Summary ==
Blacklisted hosts:
shard-hswtotal:2643 pass:1514 dwarn:6 dfail:2 fail:17 skip:1103
time
== Series Details ==
Series: lockdep: finer-grained completion key for kthread
URL : https://patchwork.freedesktop.org/series/34547/
State : failure
== Summary ==
CHK include/config/kernel.release
CHK include/generated/uapi/linux/version.h
CHK include/generated/utsrelease.h
From: Ville Syrjälä
i830_disable_pipe() gets called from the power well code, and thus
we're already holding the power domain mutex. That means we can't
call plane->get_hw_state() as it will also try to grab the
same mutex and will thus deadlock.
Replace the assert_plane() calls (which calls ->g
Quoting Tvrtko Ursulin (2017-11-28 13:07:54)
>
> On 28/11/2017 12:48, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2017-11-28 12:41:27)
> >> From: Tvrtko Ursulin
> >>
> >> Move the execlists specific setup out of intel_engine_setup_common. This
> >> was supposed to be only for backend agnostic
== Series Details ==
Series: drm/i915: Unifying debugfs return codes for unsupported features
URL : https://patchwork.freedesktop.org/series/34552/
State : failure
== Summary ==
Series 34552v1 drm/i915: Unifying debugfs return codes for unsupported features
https://patchwork.freedesktop.org/ap
On 11/28/2017 07:42 AM, Michal Wajdeczko wrote:
Instead of trying different seq_puts messages, lets use common
-ENODEV error code to indicate missing/unsupported feature.
I agree that this is the simplest way to unify
Suggested-by: Chris Wilson
Signed-off-by: Michal Wajdeczko
Cc: Chris Wil
== Series Details ==
Series: drm/i915: Fix deadlock in i830_disable_pipe()
URL : https://patchwork.freedesktop.org/series/34553/
State : success
== Summary ==
Series 34553v1 drm/i915: Fix deadlock in i830_disable_pipe()
https://patchwork.freedesktop.org/api/1.0/series/34553/revisions/1/mbox/
On 11/28/2017 10:30 AM, Ville Syrjälä wrote:
>> +dev_priv->bridge_dev =
>> +pci_get_domain_bus_and_slot(domain, 0, PCI_DEVFN(0, 0));
> Maybe just pci_get_slot(pdev->bus, PCI_DEVFN(0, 0)) ?
>
> I guess if we want to be pedantic we could go for:
>
> bus = pci_find_host_bridge(pdev->
On Tue, 28 Nov 2017 17:07:27 +0100, Patchwork
wrote:
== Series Details ==
Series: drm/i915: Unifying debugfs return codes for unsupported features
URL : https://patchwork.freedesktop.org/series/34552/
State : failure
== Summary ==
Series 34552v1 drm/i915: Unifying debugfs return codes fo
Hi Hans,
I love your patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v4.15-rc1 next-20171128]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits
In commit 3f6ae7b19 ("igt/kms_frontbuffer_tracking: Keep the debugfs
dir around") we introduced custom variant of __igt_debugfs_read function
that fires assert when debugfs returns an error. Replace that assert
with proper error handling to allow use of errors like -ENODEV.
Signed-off-by: Michal W
On 28/11/2017 11:14, Patchwork wrote:
== Series Details ==
Series: drm/i915/pmu: Return -EINVAL when selecting the inactive CPU
URL : https://patchwork.freedesktop.org/series/34529/
State : success
== Summary ==
Series 34529v1 drm/i915/pmu: Return -EINVAL when selecting the inactive CPU
htt
cross-release ftl
Cc: Tvrtko Ursulin
Cc: Marta Lofstedt
References: https://bugs.freedesktop.org/show_bug.cgi?id=103707
Signed-off-by: Daniel Vetter
---
kernel/locking/lockdep_internals.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/locking/lockdep_internals.h
b/
Ideally we'd create the key through a macro at the real callers and
pass it all the way down. This would give us better coverage for cases
where a bunch of kthreads are created for the same thing.
But this gets the job done meanwhile and unblocks our CI. Refining
later on is always possible.
v2:
-
On 11/28/2017 07:42 AM, Michal Wajdeczko wrote:
Instead of trying different seq_puts messages, lets use common
-ENODEV error code to indicate missing/unsupported feature.
I agree that this is the simplest way to unify the message.
Regards,
Sujaritha
Suggested-by: Chris Wilson
Signed-off-by
On 26/11/2017 12:20, Chris Wilson wrote:
Looking at the distribution of i915_wait_request for a set of GL
benchmarks, we see:
broadwell# python bcc/tools/funclatency.py -u i915_wait_request
usecs : count distribution
0 -> 1 : 29184|
On 27/11/2017 10:10, Chris Wilson wrote:
An interesting discussion regarding "hybrid interrupt polling" for NVMe
came to the conclusion that the ideal busyspin before sleeping was half
of the expected request latency (and better if it was already halfway
through that request). This suggested tha
Quoting Daniel Vetter (2017-11-28 17:07:07)
> cross-release ftl
Fwiw, this isn't cross-release but us reloading the module many times,
creating a whole host of new lockclasses. Even more fun is when the
module gets a slightly different address and the new lock address hashes
into an old lock...
I
== Series Details ==
Series: tests/kms_frontbuffer_tracking: Correctly handle debugfs errors
URL : https://patchwork.freedesktop.org/series/34555/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
5b3619f3751ecef55fa993984ddb59a458fcebed igt: Remove gem_ctx_ba
On 28/11/2017 16:04, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2017-11-28 13:07:54)
On 28/11/2017 12:48, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2017-11-28 12:41:27)
From: Tvrtko Ursulin
Move the execlists specific setup out of intel_engine_setup_common. This
was supposed to be only
Quoting Michal Wajdeczko (2017-11-28 17:01:15)
> In commit 3f6ae7b19 ("igt/kms_frontbuffer_tracking: Keep the debugfs
> dir around") we introduced custom variant of __igt_debugfs_read function
> that fires assert when debugfs returns an error. Replace that assert
> with proper error handling to all
On Tue, Nov 28, 2017 at 09:51:13AM +0100, Michal Wajdeczko wrote:
> On Tue, 28 Nov 2017 07:50:52 +0100, Jonathan Neuschäfer
> wrote:
>
> > Sphinx complains that it can't find intel_guc_loader.c, and rightly so:
> > The file has been renamed.
> >
> > Fixes: e8668bbcb0f9 ("drm/i915/guc: Rename int
On Tue, 28 Nov 2017, Daniel Vetter wrote:
On Tue, Nov 28, 2017 at 12:16:03PM +0100, Maarten Lankhorst wrote:
Some drivers like i915 start with crtc's enabled, but with deferred
fbcon setup they were no longer disabled as part of fbdev setup.
Headless units could no longer enter pc3 state becaus
Sphinx complains that it can't find intel_guc_loader.c, and rightly so:
The file has been renamed.
Fixes: e8668bbcb0f9 ("drm/i915/guc: Rename intel_guc_loader.c to
intel_guc_fw.c")
Cc: Michal Wajdeczko
Signed-off-by: Jonathan Neuschäfer
---
Documentation/gpu/i915.rst | 4 ++--
1 file changed,
On Monday, November 27, 2017 11:58 AM, Sinan Kaya wrote:
>
> pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as
> where a PCI device is present. This restricts the device drivers to be
> reused for other domain numbers.
I think that this will be useful. I introduced multi doma
Quoting Tvrtko Ursulin (2017-11-28 17:29:25)
>
> On 28/11/2017 16:04, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2017-11-28 13:07:54)
> >>
> >> On 28/11/2017 12:48, Chris Wilson wrote:
> >>> Quoting Tvrtko Ursulin (2017-11-28 12:41:27)
> From: Tvrtko Ursulin
>
> Move the execl
From: Tvrtko Ursulin
Short series to clean up the code and bit.
Thanks to Lionel for pointing out the convenient /sys/dev/char/:
symlink.
Tvrtko Ursulin (3):
lib/igt_sysfs: Remove support for finding sysfs path with no device
opened
lib/igt_sysfs: igt_sysfs_path only works for master no
From: Tvrtko Ursulin
There is no need to iterate over /sys/class/drm/card* directories looking
for the one which matches our major and minor, when we can directly find
the right one via the /sys/dev/char/: symlink.
Signed-off-by: Tvrtko Ursulin
Suggested-by: Lionel Landwerlin
---
lib/igt_sysf
From: Tvrtko Ursulin
DRM code defines the minor range for master nodes as 0-63, so express that
explicitly near the top of the function.
Signed-off-by: Tvrtko Ursulin
---
lib/igt_sysfs.c | 4
1 file changed, 4 insertions(+)
diff --git a/lib/igt_sysfs.c b/lib/igt_sysfs.c
index dd057dc0494
From: Tvrtko Ursulin
It looks like no callers actually use this so remove it to simplify.
Signed-off-by: Tvrtko Ursulin
---
lib/igt_sysfs.c | 39 ---
1 file changed, 16 insertions(+), 23 deletions(-)
diff --git a/lib/igt_sysfs.c b/lib/igt_sysfs.c
index e7c6
On 20/11/2017 12:23, Lionel Landwerlin wrote:
> This enables userspace to discover the engines available on the GPU.
> Here is the layout on a Skylake GT4:
>
> /sys/devices/pci:00/:00:02.0/drm/card0/gt
On this one I think Joonas had a concern that it is difficult for
userspace to get to
== Series Details ==
Series: series starting with [1/2] lockdep: finer-grained completion key for
kthread
URL : https://patchwork.freedesktop.org/series/34556/
State : success
== Summary ==
Series 34556v1 series starting with [1/2] lockdep: finer-grained completion key
for kthread
https://pa
== Series Details ==
Series: drm/i915/guc: Fix doc reference to intel_guc_fw.c
URL : https://patchwork.freedesktop.org/series/34557/
State : success
== Summary ==
Series 34557v1 drm/i915/guc: Fix doc reference to intel_guc_fw.c
https://patchwork.freedesktop.org/api/1.0/series/34557/revisions/1
== Series Details ==
Series: Simplify hunting for drm card sysfs root
URL : https://patchwork.freedesktop.org/series/34559/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
380cc811486ba3fefbe3ebe4761afa7e169dcd3e tests/perf_pmu: Sync invalid-init with
i915
Hi Hans,
I love your patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v4.15-rc1 next-20171128]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits
Quoting Tvrtko Ursulin (2017-11-28 18:09:55)
> From: Tvrtko Ursulin
>
> It looks like no callers actually use this so remove it to simplify.
We just added more users to the equivalent for debugfs, and accepting -1
here was simply to have consistency with our debugfs api.
If that makes any diffe
Quoting Tvrtko Ursulin (2017-11-28 18:09:56)
> From: Tvrtko Ursulin
>
> DRM code defines the minor range for master nodes as 0-63, so express that
> explicitly near the top of the function.
We do use rendernodes, so we had better support them correctly. I
thought the premise of /sys/dev/char/$ma
Quoting Tvrtko Ursulin (2017-11-28 18:17:54)
>
> On 20/11/2017 12:23, Lionel Landwerlin wrote:
> > This enables userspace to discover the engines available on the GPU.
> > Here is the layout on a Skylake GT4:
> >
> > /sys/devices/pci:00/:00:02.0/drm/card0/gt
>
> On this one I think Joona
Quoting Ville Syrjala (2017-11-28 15:48:53)
> From: Ville Syrjälä
>
> i830_disable_pipe() gets called from the power well code, and thus
> we're already holding the power domain mutex. That means we can't
> call plane->get_hw_state() as it will also try to grab the
> same mutex and will thus dead
Hi Hans,
I love your patch! Yet something to improve:
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.15-rc1 next-20171128]
[cannot apply to linus/master]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https
Cc: Ville Syrjälä
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/intel_display.c | 2 +-
drivers/gpu/drm/i915/intel_hdmi.c| 2 +-
drivers/gpu/drm/i915/intel_pm.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drive
Cc: Ville Syrjälä
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/intel_hdmi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_hdmi.c
b/drivers/gpu/drm/i915/intel_hdmi.c
index 691600ce48c4..c42a6c672b73 100644
--- a/drivers/gpu/drm/i915/int
1 - 100 of 135 matches
Mail list logo
