On ti, 2017-01-31 at 13:15 +, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> dev, file and ctx are unused.
>
> Signed-off-by: Tvrtko Ursulin
Could either squash this to the previous commit, or have this before
previous commit, to avoid going through removed code in previous patch
(easier
On ti, 2017-01-31 at 13:15 +, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> Use a local variable for storing the request and engine
> and at the same time drop the engine field from
> i915_execbuffer_params since it is available from the
> request.
>
> textdata bss dec
On ti, 2017-01-31 at 13:15 +, Tvrtko Ursulin wrote:
> > From: Tvrtko Ursulin
>
> Instead of sprinkling around usage and initialization of
> i915_execbuffer_params we can consolidate it just before
> execbuf_submit for maintability and readability.
>
> That way we can also drop the memset sin
Hi,
On Tuesday 31 January 2017 09:26 PM, Ander Conselvan De Oliveira wrote:
On Tue, 2017-01-31 at 20:27 +0530, Mahesh Kumar wrote:
This patch adds IPC support for platforms. This patch enables IPC
only for BXT/KBL platform as for SKL recommendation is to keep it disabled.
IPC (Isochronous Prio
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 6 +++---
lib/igt_kms.h | 23 +++
2 files changed, 26 insertions(+), 3 deletions(-)
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 41acce45..142658a6 100644
--- a/lib/igt
Added the igt_pipe_get_last_out_fence() helper function
that wraps accesses to pipe->fence_out.
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 8
lib/igt_kms.h | 1 +
2 files changed, 9 insertions(+)
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index a1eaf411..b99b0398 100644
--- a/lib/i
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 34 +-
1 file changed, 17 insertions(+), 17 deletions(-)
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 4ba6316d..41acce45 100644
--- a/lib/igt_kms.c
+++ b/lib
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
tests/kms_atomic_transition.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
index 095af515..72429759 100644
--- a/t
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 16 +
tests/kms_atomic_transition.c | 153 --
2 files changed, 164 insertions(+), 5 deletions(-)
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 7bf3fa3a..
Add support dor the IN_FENCE_FD property to enable setting in fences for atomic
commits.
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 25 +
lib/igt_kms.h | 5 +
2 files changed, 30 insertions(+)
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 142658a6..9b60d74a 1
This series adds in/out fence testing to kms_atomic_transition test and makes
some minor cleanups.
This series can be found here:
https://git.collabora.com/cgit/user/robertfoss/intel-gpu-tools.git/log/?h=fences_$VER
Changes since v3:
Rebased on upstream/master
lib/igt_kms:
- Change out_
From: Gustavo Padovan
Add support for the OUT_FENCE_PTR property to enable setting out fences for
atomic commits.
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 23 ++-
lib/igt_kms.h | 6 +-
2 files changed, 27 insertions(+), 2 deletions
From: Gustavo Padovan
If the event never arrives we can timeout and end the test.
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
tests/kms_atomic_transition.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
ind
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
tests/kms_atomic.c | 123 -
1 file changed, 37 insertions(+), 86 deletions(-)
diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c
index e6d71c31..8df51ccd 1
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
Reviewed-by: Brian Starkey
---
tests/kms_atomic.c | 187 ++---
1 file changed, 177 insertions(+), 10 deletions(-)
diff --git a/tests/kms_atomic.c b/tests/kms_atomic
On 2017-01-31 11:52 AM, Brian Starkey wrote:
On Mon, Jan 30, 2017 at 08:58:47PM -0500, Robert Foss wrote:
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 3 +++
tests/kms_atomic_transition.c | 48
++
Hi Arthur,
[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on next-20170131]
[cannot apply to v4.10-rc6]
[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/Arthur
== Series Details ==
Series: Still call-traces after suspend-resume (pm? i915? cpu/hotplug?)
URL : https://patchwork.freedesktop.org/series/18871/
State : success
== Summary ==
Series 18871v1 Still call-traces after suspend-resume (pm? i915? cpu/hotplug?)
https://patchwork.freedesktop.org/api/
On 2017-01-31 11:52 AM, Brian Starkey wrote:
On Mon, Jan 30, 2017 at 08:58:46PM -0500, Robert Foss wrote:
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 35 ++
tests/kms_atomic_transition.c | 148
+
I added some printk()s all over and gathered a bit more information
about what's going on. It looks like the display doesn't work until the
drm connector code cleans up the *old* connector. For some reason, it
isn't motivated to do that until I go to the console and back.
In this case, the displ
== Series Details ==
Series: drm/i915: Get correct display clock on 945gm (rev3)
URL : https://patchwork.freedesktop.org/series/18693/
State : success
== Summary ==
Series 18693v3 drm/i915: Get correct display clock on 945gm
https://patchwork.freedesktop.org/api/1.0/series/18693/revisions/3/mb
On Mon, Jan 30, 2017 at 11:44 PM, Rafael J. Wysocki
wrote:
> On 1/24/2017 2:33 AM, Sedat Dilek wrote:
>>
>> On Fri, Dec 30, 2016 at 3:02 PM, Rafael J. Wysocki
>> wrote:
>>>
>>> On Fri, Dec 30, 2016 at 12:40 PM, Sedat Dilek
>>> wrote:
Hi,
I have already reported this issue in
Hi Arthur,
[auto build test ERROR on drm-intel/for-linux-next]
[also build test ERROR on next-20170131]
[cannot apply to v4.10-rc6]
[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/Arthur-Heymans
This is according to Mobile Intel® 945 Express Chipset
Family datasheet.
Signed-off-by: Arthur Heymans
---
drivers/gpu/drm/i915/i915_reg.h | 2 +-
drivers/gpu/drm/i915/intel_display.c | 27 +--
2 files changed, 26 insertions(+), 3 deletions(-)
diff --git a/drivers/
== Series Details ==
Series: drm/i915: Get correct display clock on 945gm (rev2)
URL : https://patchwork.freedesktop.org/series/18693/
State : failure
== Summary ==
CC drivers/acpi/acpica/utpredef.o
CC [M] drivers/gpu/drm/i915/i915_perf.o
CC [M] drivers/gpu/drm/i915/i915_oa_hsw.o
This is according to Mobile Intel® 945 Express Chipset
Family datasheet.
Signed-off-by: Arthur Heymans
---
drivers/gpu/drm/i915/i915_reg.h | 2 +-
drivers/gpu/drm/i915/intel_display.c | 29 ++---
2 files changed, 27 insertions(+), 4 deletions(-)
diff --git a/driver
On 2017-01-31 11:50 AM, Brian Starkey wrote:
This one lgtm, just need to swap all the uint64_t out_fence_ptrs for
int32_t.
-Brian
Fixed in v4.
Rob.
On Mon, Jan 30, 2017 at 08:58:44PM -0500, Robert Foss wrote:
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert
On 2017-01-31 11:49 AM, Brian Starkey wrote:
On Mon, Jan 30, 2017 at 08:58:43PM -0500, Robert Foss wrote:
From: Gustavo Padovan
Add support for the OUT_FENCE_PTR property to enable setting out
fences for
atomic commits.
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt
== Series Details ==
Series: DisplayPort audio support on Cherrytrail
URL : https://patchwork.freedesktop.org/series/18864/
State : success
== Summary ==
Series 18864v1 DisplayPort audio support on Cherrytrail
https://patchwork.freedesktop.org/api/1.0/series/18864/revisions/1/mbox/
fi-bdw-55
From: Pierre-Louis Bossart
Now the pipe that is being used is passed over i915 notification, we
can re-setup the relevant register offset depending on pipe assignments
during hotplug.
This allows playback on single port machines such Zotac Pi330 or
dual-port machines such as Dell Wyse 3040 box
S
From: Pierre-Louis Bossart
Enable unmute/mute amp notification. This doesn't seem to affect
HDMI support so this is done unconditionally.
An earlier version of this patch set a chicken bit at address 0x62F38
prior to the mute/unmute but this register doesn't seem to do anything
so this phase was
The LPE audio configuration depends on the pipe, thus we need to pass
the currently used pipe. It's now embedded in struct
intel_hdmi_lpe_audio_eld as well as port id.
Signed-off-by: Takashi Iwai
---
drivers/gpu/drm/i915/i915_drv.h| 2 +-
drivers/gpu/drm/i915/intel_audio.c | 6 +++--
From: Pierre-Louis Bossart
Imported from legacy patches
Note: the new code doesn't assume a modified ELD but
an explicit notification that DP is present. It appears
that the i915 code does change the ELD so we could use
the ELD-based tests to check for DP audio
Signed-off-by: Pierre-Louis Bossa
The pipe gets cleared to -1 for non-MST before the ELD audio
notification due to the MST audio support. This makes sense for
HD-audio that received the MST handling, but it's useless for LPE
audio. Handle the MST pipe hack conditionally only for HD-audio.
Reported-by: Pierre-Louis Bossart
Signe
From: Pierre-Louis Bossart
If DisplayPort is detected, pass flag and link rate to audio driver
Signed-off-by: Pierre-Louis Bossart
Signed-off-by: Takashi Iwai
---
drivers/gpu/drm/i915/i915_drv.h| 3 ++-
drivers/gpu/drm/i915/intel_audio.c | 19 +++
drivers/gpu/drm/
This allows the LPE HDMI driver to clean up its global variable
reference.
Also drop to pass the eld pointer because the connection status and
the ELD bytes can be retrieved from the attached pdata.
Signed-off-by: Takashi Iwai
---
drivers/gpu/drm/i915/intel_lpe_audio.c | 3 +--
include/drm/int
Hi,
the following patches enable DisplayPort Audio on Cherrytrail machines
when applied on top of my topic/intel-lpe-audio branch. Tests of DP
audio were run on Dell Wyse 3040. The regression test were performed
on Baytrail (Compute Stick) and Cherrytrail (Zotac PI330) in HDMI
mode. On Cherrytr
On Tue, Jan 31, 2017 at 12:28:40PM -0800, Linus Torvalds wrote:
> On Tue, Jan 31, 2017 at 1:21 AM, Maarten Lankhorst
> wrote:
> >
> > This is marked for rc6 because it seems the issue is triggerable on
> > mainline and resulting in an oops.
>
> So I did apply my obvious "avoid the oops and just w
On Tue, Jan 31, 2017 at 1:21 AM, Maarten Lankhorst
wrote:
>
> This is marked for rc6 because it seems the issue is triggerable on
> mainline and resulting in an oops.
So I did apply my obvious "avoid the oops and just warn about it"
patch: commit 39cb2c9a316e ("drm/i915: Check for NULL i915_vma i
On Tue, Jan 31, 2017 at 02:31:32PM -0500, Sean Paul wrote:
> On Tue, Jan 31, 2017 at 07:01:44PM +0100, Daniel Vetter wrote:
> > For the experiement we have right now Eric (with vc4) and Sean Paul
> > (with rockchip and zte) volunteering, and Gerd (entire pile of qemu
> > drivers) and Boris (atmel)
On Tue, Jan 31, 2017 at 07:01:44PM +0100, Daniel Vetter wrote:
> For the experiement we have right now Eric (with vc4) and Sean Paul
> (with rockchip and zte) volunteering, and Gerd (entire pile of qemu
> drivers) and Boris (atmel) are also considering to participate. I
> think that's enough to get
Hello,
On Tue, Jan 31, 2017 at 10:03:26AM +0100, Maarten Lankhorst wrote:
> Op 31-01-17 om 09:09 schreef Uwe Kleine-König:
> > From: Chris Wilson
> >
> > As the introduced comment admits this is clearly a workaround, but for
> > me this is the only known way to make my Lenovo X201 work without
>
-Kumar/Enable-IPC-WM-related-WA-s/20170131-230708
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All
For the experiement we have right now Eric (with vc4) and Sean Paul
(with rockchip and zte) volunteering, and Gerd (entire pile of qemu
drivers) and Boris (atmel) are also considering to participate. I
think that's enough to get started and figure things out as we go.
I tried to summarize the main
Hi Dave,
2 patches to fix the oops Dave Hanse reported, plus a double kfree fix
Maarten discovered while backporting the fix for Linus.
For Linus' vma tracking oops the plan is to send you a dedicated pull with
the 2 patches we need, but since it's tricky we're letting CI beat on it a
bit more.
-Kumar/Enable-IPC-WM-related-WA-s/20170131-230708
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: x86_64-rhel (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
All
On Mon, Jan 30, 2017 at 08:58:41PM -0500, Robert Foss wrote:
Added the igt_pipe_get_last_out_fence() helper function
that wraps accesses to pipe->fence_out.
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 8
lib/igt_kms.h | 1 +
2 files changed, 9 insertions(+)
diff --git a/lib/igt_kms.c
On Mon, Jan 30, 2017 at 08:58:47PM -0500, Robert Foss wrote:
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 3 +++
tests/kms_atomic_transition.c | 48 ++-
2 files changed, 32 insertions
On Mon, Jan 30, 2017 at 08:58:46PM -0500, Robert Foss wrote:
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 35 ++
tests/kms_atomic_transition.c | 148 ++
2 files changed, 169 insertions(+), 14 deletio
This one lgtm, just need to swap all the uint64_t out_fence_ptrs for
int32_t.
-Brian
On Mon, Jan 30, 2017 at 08:58:44PM -0500, Robert Foss wrote:
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
tests/kms_atomic.c | 187
On Mon, Jan 30, 2017 at 08:58:43PM -0500, Robert Foss wrote:
From: Gustavo Padovan
Add support for the OUT_FENCE_PTR property to enable setting out fences for
atomic commits.
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 26 +-
lib/igt_k
On Mon, Jan 30, 2017 at 08:58:42PM -0500, Robert Foss wrote:
Add support dor the IN_FENCE_FD property to enable setting in fences for atomic
commits.
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 20
lib/igt_kms.h | 5 +
2 files changed, 25 insertions(+)
diff --git a/l
On Mon, Jan 30, 2017 at 05:54:14PM -, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [v2,1/3] drm/i915: Make
> intel_detect_preproduction_hw easier to extend
> URL : https://patchwork.freedesktop.org/series/18758/
> State : success
>
> == Summary ==
>
> Series 18
On Tue, Jan 31, 2017 at 12:13:47PM +0200, Joonas Lahtinen wrote:
> On ma, 2017-01-30 at 13:47 +, Chris Wilson wrote:
> > Just do a quick check that the stolen memory address range doesn't
> > overflow our chosen integer type.
> >
> > v2: Add add_overflows() to utils with the promise that gcc7
On Tue, 2017-01-31 at 20:27 +0530, Mahesh Kumar wrote:
> This patch adds IPC support for platforms. This patch enables IPC
> only for BXT/KBL platform as for SKL recommendation is to keep it disabled.
> IPC (Isochronous Priority Control) is the hardware feature, which
> dynamically controls the mem
On 2017-01-31 05:18 AM, Chris Wilson wrote:
On Mon, Jan 30, 2017 at 08:58:36PM -0500, Robert Foss wrote:
This series adds in/out fence testing to kms_atomic_transition test and makes
some minor cleanups.
This series is rebased ontop of the dyn_n_planes_v3 series.
This series can be found he
The purpose of the current placeholder gvt is to reload the module with
gvt enabled. As a reload, it should be last after the basic reload
checks.
Signed-off-by: Chris Wilson
---
tests/intel-ci/fast-feedback.testlist | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/intel
== Series Details ==
Series: Enable IPC & WM related WA's
URL : https://patchwork.freedesktop.org/series/18842/
State : success
== Summary ==
Series 18842v1 Enable IPC & WM related WA's
https://patchwork.freedesktop.org/api/1.0/series/18842/revisions/1/mbox/
fi-bdw-5557u total:247 pass:
== Series Details ==
Series: do_execbuffer tidy
URL : https://patchwork.freedesktop.org/series/18838/
State : warning
== Summary ==
Series 18838v1 do_execbuffer tidy
https://patchwork.freedesktop.org/api/1.0/series/18838/revisions/1/mbox/
Test kms_force_connector_basic:
Subgroup force
This patch adds IPC support for platforms. This patch enables IPC
only for BXT/KBL platform as for SKL recommendation is to keep it disabled.
IPC (Isochronous Priority Control) is the hardware feature, which
dynamically controls the memory read priority of Display.
When IPC is enabled, plane read
This patch adds support to decode system memory bandwidth
which will be used for arbitrated display memory percentage
calculation in GEN9 based system.
Changes from v1:
- Address comments from Paulo
- implement decode function for SKL/KBL also
Changes from v2:
- Rewrite the code as per HW team
This patch enables workarounds related to display arbitrated memory
bandwidth only if it's necessary. WA's are applicable for all GEN9
based platforms.
Changes since v1:
- Rebase on top of Paulo's patch series
Changes since v2:
- Address review comments
- Rebase/rework as per other patch change
This series include remaining patches from following series
to enable IPC and Enable/update memory BW related WA's for WM.
https://patchwork.freedesktop.org/series/15562/
Mahesh Kumar (3):
drm/i915/bxt: Enable IPC support
drm/i915: Decode system memory bandwidth
drm/i915/gen9: WM memory band
On Mon, 30 Jan 2017, Ville Syrjälä wrote:
> On Mon, Jan 30, 2017 at 10:00:54AM +0200, Jani Nikula wrote:
>> On Tue, 24 Jan 2017, Jani Nikula wrote:
>> > After all these years intel_bios_reader and intel_bios_dumper still
>> > manage to confuse me. Read or dump, which one decodes. Rename
>> > inte
On Fri, 27 Jan 2017, Jani Nikula wrote:
> The function is not that big, but it's also not used for anything
> performance critical. Make it a normal function.
>
> As a side effect, this apparently makes sparse smarter about what it's
> doing, and gets rid of the warning:
>
> ./include/drm/drm_colo
On Tue, Jan 31, 2017 at 01:15:36PM +, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> I've noticed a few opportunities to improve the readability of this functions
> and then kept spotting more and more which can be removed or compacted.
>
> Eventually ended up with removing i915_execbuffer
From: Tvrtko Ursulin
dev, file and ctx are unused.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_gem_execbuffer.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
index ee7b7bd17e29..
From: Tvrtko Ursulin
Signed-off-by: Tvrtko Ursulin
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_gem_execbuffer.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
index 57a
From: Tvrtko Ursulin
Remove ctx_id, ggtt and vm since they are single use.
textdata bss dec hex filename
1085338 263982628 1114364 1100fc i915.ko.0
1085290 263982628 1114316 1100cc i915.ko.1
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_gem
From: Tvrtko Ursulin
Instead of sprinkling around usage and initialization of
i915_execbuffer_params we can consolidate it just before
execbuf_submit for maintability and readability.
That way we can also drop the memset since it becomes
easy to spot we initialize all the fields.
textd
From: Tvrtko Ursulin
Now that it only contains three parameters we can pass them
directly to execbuf_submit just as well.
No effect on generated binary, just a source reduction.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_gem_execbuffer.c | 39 --
1
From: Tvrtko Ursulin
I've noticed a few opportunities to improve the readability of this functions
and then kept spotting more and more which can be removed or compacted.
Eventually ended up with removing i915_execbuffer_params completely. But I think
it's OK since the plan between when it was a
From: Tvrtko Ursulin
Also redundant since it is stored in the request.
textdata bss dec hex filename
1085354 263982628 1114380 11010c i915.ko.0
1085338 263982628 1114364 1100fc i915.ko.1
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_gem_exe
From: Tvrtko Ursulin
Just spotted a few lines which fit in 80 chars and were split.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_gem_execbuffer.c | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
b/drive
From: Tvrtko Ursulin
Not the whole args struct needs to be passed in.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_gem_execbuffer.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
b/drivers/gpu/drm/i
From: Tvrtko Ursulin
file_priv is what it needs to pass on to gen8_dispatch_bsd_engine so
simplify things by passing it straight away.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_gem_execbuffer.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/d
From: Tvrtko Ursulin
Use a local variable for storing the request and engine
and at the same time drop the engine field from
i915_execbuffer_params since it is available from the
request.
textdata bss dec hex filename
1085402 263982628 1114428 11013c i915.ko.0
108
On Tue, Jan 31, 2017 at 01:12:05PM +0100, Rafael J. Wysocki wrote:
> On 1/31/2017 1:02 PM, Imre Deak wrote:
> >On Tue, Jan 31, 2017 at 12:39:35PM +0100, Rafael J. Wysocki wrote:
> >>On 1/31/2017 11:58 AM, Imre Deak wrote:
> >>>Hi Rafael,
> >>Hi,
> >>
> >>>On Mon, Jan 30, 2017 at 11:44:37PM +0100, R
On Tue, Jan 31, 2017 at 10:04:09AM -0200, Gustavo Padovan wrote:
> Hi Maarten,
>
> 2017-01-31 Maarten Lankhorst :
>
> > drm_atomic_helper_page_flip and drm_atomic_ioctl set their own events
> > in crtc_state->event. But when it's set the event is freed in 2 places.
> >
> > Solve this by only fre
On to, 2017-01-19 at 11:41 +, Chris Wilson wrote:
> Allocate objects with varying number of pages (which should hopefully
> consist of a mixture of contiguous page chunks and so coalesced sg
> lists) and check that the sg walkers in insert_pages cope.
>
> Signed-off-by: Chris Wilson
> +sta
On 1/31/2017 1:02 PM, Imre Deak wrote:
On Tue, Jan 31, 2017 at 12:39:35PM +0100, Rafael J. Wysocki wrote:
On 1/31/2017 11:58 AM, Imre Deak wrote:
Hi Rafael,
Hi,
On Mon, Jan 30, 2017 at 11:44:37PM +0100, Rafael J. Wysocki wrote:
On 1/24/2017 2:33 AM, Sedat Dilek wrote:
On Fri, Dec 30, 2016
Hi Maarten,
2017-01-31 Maarten Lankhorst :
> drm_atomic_helper_page_flip and drm_atomic_ioctl set their own events
> in crtc_state->event. But when it's set the event is freed in 2 places.
>
> Solve this by only freeing the event in the atomic ioctl when it
> allocated its own event.
>
> This h
On Tue, Jan 31, 2017 at 12:39:35PM +0100, Rafael J. Wysocki wrote:
> On 1/31/2017 11:58 AM, Imre Deak wrote:
> >Hi Rafael,
>
> Hi,
>
> >On Mon, Jan 30, 2017 at 11:44:37PM +0100, Rafael J. Wysocki wrote:
> >>On 1/24/2017 2:33 AM, Sedat Dilek wrote:
> >>>On Fri, Dec 30, 2016 at 3:02 PM, Rafael J. W
On to, 2017-01-19 at 11:41 +, Chris Wilson wrote:
> Mock testing to ensure we can create and lookup partial VMA.
>
> Signed-off-by: Chris Wilson
> +static bool assert_partial(struct drm_i915_gem_object *obj,
> + struct i915_vma *vma,
> + unsign
On 1/31/2017 11:58 AM, Imre Deak wrote:
Hi Rafael,
Hi,
On Mon, Jan 30, 2017 at 11:44:37PM +0100, Rafael J. Wysocki wrote:
On 1/24/2017 2:33 AM, Sedat Dilek wrote:
On Fri, Dec 30, 2016 at 3:02 PM, Rafael J. Wysocki wrote:
On Fri, Dec 30, 2016 at 12:40 PM, Sedat Dilek wrote:
Hi,
I have a
On Tue, Jan 31, 2017 at 01:21:15PM +0200, Petri Latvala wrote:
> On Tue, Jan 31, 2017 at 09:54:21AM +, Chris Wilson wrote:
> > Ping? And any feedback from the earlier ringfill-fds?
> > -Chris
>
>
> Sent ack on the ringfill test. This one has been queued for a test
> round at farm2.
>
> For t
== Series Details ==
Series: drm/i915: The return of i915_gpu_info to debugfs
URL : https://patchwork.freedesktop.org/series/18833/
State : failure
== Summary ==
Series 18833v1 drm/i915: The return of i915_gpu_info to debugfs
https://patchwork.freedesktop.org/api/1.0/series/18833/revisions/1/m
On Tue, Jan 31, 2017 at 11:04:49AM +, Matthew Auld wrote:
> On 31 January 2017 at 10:46, Chris Wilson wrote:
> > If we abort the i915_gem_internal get_pages, we mark the failing sg as
> > the last. However, that means we iterate upto and including the failing
> > sg element and results us in t
On Tue, Jan 31, 2017 at 09:54:21AM +, Chris Wilson wrote:
> Ping? And any feedback from the earlier ringfill-fds?
> -Chris
Sent ack on the ringfill test. This one has been queued for a test
round at farm2.
For those following along and smelling chances of getting their
$favoritefeatureofthed
On Tue, Jan 24, 2017 at 01:02:23PM +0200, Petri Latvala wrote:
> On Tue, Jan 24, 2017 at 09:31:39AM +, Chris Wilson wrote:
> > On Wed, Jan 18, 2017 at 08:54:10PM +, Chris Wilson wrote:
> > > Execlists introduces a new wrinkle to filling rings, in that each
> > > context has an independent s
On 31 January 2017 at 10:46, Chris Wilson wrote:
> If we abort the i915_gem_internal get_pages, we mark the failing sg as
> the last. However, that means we iterate upto and including the failing
> sg element and results us in trying to free the unallocated sg_page().
s/us in/in us/
>
> Signed-of
Looks ok.
Acked-by: Mika Kahola
On Wed, 2017-01-25 at 19:43 +0530, Vidya Srinivas wrote:
> From: Uma Shankar
>
> Enable MIPI IO WA for BXT DSI as per bspec and
> program the DSI regulators.
>
> v2: Moved IO enable to pre-enable as per Mika's
> review comments. Also reused the existing registe
Hi Rafael,
On Mon, Jan 30, 2017 at 11:44:37PM +0100, Rafael J. Wysocki wrote:
> On 1/24/2017 2:33 AM, Sedat Dilek wrote:
> >On Fri, Dec 30, 2016 at 3:02 PM, Rafael J. Wysocki wrote:
> >>On Fri, Dec 30, 2016 at 12:40 PM, Sedat Dilek wrote:
> >>>Hi,
> >>>
> >>>I have already reported this issue in
On to, 2017-01-19 at 11:41 +, Chris Wilson wrote:
> Simple test to exercise creation and lookup of VMA within an object.
>
> Signed-off-by: Chris Wilson
> +static bool assert_vma(struct i915_vma *vma,
> + struct drm_i915_gem_object *obj,
> + struct i91
If we abort the i915_gem_internal get_pages, we mark the failing sg as
the last. However, that means we iterate upto and including the failing
sg element and results us in trying to free the unallocated sg_page().
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_gem_internal.c | 7 +
Once upon a time before we had automated GPU state capture upon hangs,
we had intel_gpu_dump. Now we come almost full circle and reinstate that
view of the current GPU queues and registers by using the error capture
facility to snapshot the GPU state when debugfs/.../i915_gpu_info is
opened - which
== Series Details ==
Series: drm/atomic: Fix double free in drm_atomic_state_default_clear
URL : https://patchwork.freedesktop.org/series/18826/
State : success
== Summary ==
Series 18826v1 drm/atomic: Fix double free in drm_atomic_state_default_clear
https://patchwork.freedesktop.org/api/1.0/
On Mon, Jan 30, 2017 at 08:58:36PM -0500, Robert Foss wrote:
> This series adds in/out fence testing to kms_atomic_transition test and makes
> some minor cleanups.
>
> This series is rebased ontop of the dyn_n_planes_v3 series.
>
> This series can be found here:
> https://git.collabora.com/cgit/
On ma, 2017-01-30 at 13:47 +, Chris Wilson wrote:
> Just do a quick check that the stolen memory address range doesn't
> overflow our chosen integer type.
>
> v2: Add add_overflows() to utils with the promise that gcc7 can do this
> better than C and then maybe it will have a proper definition
Gentle remainder - could you kindly check the patch please? Thank you.
> -Original Message-
> From: Srinivas, Vidya
> Sent: Wednesday, January 25, 2017 7:43 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Shankar, Uma ; Nikula, Jani
> ; Syrjala, Ville ; Kahola,
> Mika ; Srinivas, Vidya
> S
1 - 100 of 117 matches
Mail list logo