Ah... sorry. This patchset shouldn't be here.
-Original Message-
From: Wang, Zhi A
Sent: Tuesday, September 12, 2017 8:40 AM
To: intel-gfx@lists.freedesktop.org
Cc: Wang, Zhi A
Subject: [PATCH v2 9/9] drm/i915/gvt: Do not allocate initial ring scan buffer
Theoretically, the largest bulk
== Series Details ==
Series: series starting with [v2,1/9] drm/i915/gvt: Make elsp_dwords in the
right order
URL : https://patchwork.freedesktop.org/series/30164/
State : success
== Summary ==
Series 30164v1 series starting with [v2,1/9] drm/i915/gvt: Make elsp_dwords in
the right order
http
"reserved" means reserve something from somewhere. Actually they are
buffers used by command scanner. Rename it to ring_scan_buffer.
v2:
- Remove the usage of an extra variable. (Zhenyu)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/cmd_parser.c | 16
drivers/gpu/drm/i9
Move vGPU workload cache initialization/de-initialization into
intel_vgpu_{setup, clean}_submission() since they are not specific to
execlist stuffs.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/execlist.c | 15 +--
drivers/gpu/drm/i915/gvt/scheduler.c | 24 +
To move workload related functions into scheduler.c, an expected way is
to collect all the init/clean functions related to vGPU workload
submission into fewer functions.
Rename intel_vgpu_{init, clean}_gvt_context() for above usage in future.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt
The context descriptors in elsp_dwords are stored in a reversed order and
the definition of context descriptor is also reversed. The revesred stuff
is hard to be used and might cause misunderstanding. Make them in the right
oder for following code re-factoring.
Tested on my SKL NUC.
Signed-off-by
Theoretically, the largest bulk of commands in the ring buffer of an
engine might be the first submission, which usually contains a lot
of commands to initialize the HW. After removing the initial allocation
of the ring scan buffer and let krealloc() do everything we need, we
still have a big chanc
The pointer points to the original memory can never take the return value
of krealloc().
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/cmd_parser.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/gvt/cmd_parser.c
b/drivers/gpu/drm/i915/
Move tlb_handle_pending into intel_vgpu_submssion since it belongs to a
part of vGPU submission stuffs
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/gvt.h | 2 +-
drivers/gpu/drm/i915/gvt/handlers.c | 2 +-
drivers/gpu/drm/i915/gvt/render.c| 3 ++-
drivers/gpu/drm/i915/gvt/sche
Move ring scan buffers into intel_vgpu_submission since they belongs to
a part of vGPU submission stuffs.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/cmd_parser.c | 11 ++-
drivers/gpu/drm/i915/gvt/execlist.c | 23 +--
drivers/gpu/drm/i915/gvt/gvt.h
Introduce intel_vgpu_submission to hold all members related to submission
in struct intel_vgpu before.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/execlist.c | 30 +---
drivers/gpu/drm/i915/gvt/gvt.h | 17 +
drivers/gpu/drm/i915/gvt/handlers.c | 2 +-
driver
On 9/11/2017 11:34 PM, Michal Wajdeczko wrote:
On Mon, Sep 11, 2017 at 11:32:23AM +0530, Sagar Arun Kamble wrote:
From: "Kamble, Sagar A"
With GuC v9, new type of Default/critical logging in GuC to enable
capturing minimal important logs in production systems efficiently.
This patch enables
On 9/11/2017 11:04 PM, Michal Wajdeczko wrote:
On Mon, Sep 11, 2017 at 11:32:24AM +0530, Sagar Arun Kamble wrote:
Disabling GuC interrupts involves access to GuC IRQ control registers
hence ensure device is RPM awake.
Cc: Michal Wajdeczko
Signed-off-by: Sagar Arun Kamble
---
drivers/gpu/d
== Series Details ==
Series: drm/i915/bxt: set min brightness from VBT
URL : https://patchwork.freedesktop.org/series/30163/
State : success
== Summary ==
Test kms_setmode:
Subgroup basic:
pass -> FAIL (shard-hsw) fdo#99912
Test kms_cursor_legacy:
Su
== Series Details ==
Series: drm/i915/bxt: set min brightness from VBT
URL : https://patchwork.freedesktop.org/series/30163/
State : success
== Summary ==
Series 30163v1 drm/i915/bxt: set min brightness from VBT
https://patchwork.freedesktop.org/api/1.0/series/30163/revisions/1/mbox/
Test cha
Min brightness value from vbt was missing for BXT platform.
This setting have to refer backlight ic spec to restrict
min backlight output. Without this restriction, driver would
allow to configure lower brightness value and violate
backlight ic requirement.
Fixes: 0fb890c01349 ("drm/i915/bxt: BLC
On Mon, 2017-09-11 at 16:25 +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> From: Chris Wilson
> From: Tvrtko Ursulin
> From: Dmitry Rogozhkin
>
> The first goal is to be able to measure GPU (and invidual ring) busyness
> without having to poll registers from userspace. (Which not onl
Hi,
Just tried v3 series. perf-stat works fine. From the IGT tests which I wrote
for i915 PMU
(https://patchwork.freedesktop.org/series/29313/) all pass (assuming
pmu.enabled will be exposed
in debugfs) except cpu_online subtest. And this is pretty interesting - see
details below.
Ok, be prepa
== Series Details ==
Series: drm/i915: Refresh VLV/CHV PSR comments on HW PSR_state machine.
URL : https://patchwork.freedesktop.org/series/30161/
State : success
== Summary ==
Test gem_eio:
Subgroup in-flight:
fail -> PASS (shard-hsw) fdo#102616
Test kms_cu
== Series Details ==
Series: drm/i915: Refresh VLV/CHV PSR comments on HW PSR_state machine.
URL : https://patchwork.freedesktop.org/series/30161/
State : success
== Summary ==
Series 30161v1 drm/i915: Refresh VLV/CHV PSR comments on HW PSR_state machine.
https://patchwork.freedesktop.org/api/
DK had pointed out a comment there was hard to understand, so I
tried to read back again and I couldn't understand that as well.
So let me re-phrase that in a way that anyone can understand
later, even myself.
Also fixed the comment block style.
Cc: Dhinakaran Pandiyan
Signed-off-by: Rodrigo Viv
On Fri, 2017-09-08 at 01:43 +, Pandiyan, Dhinakaran wrote:
> On Thu, 2017-09-07 at 16:00 -0700, Rodrigo Vivi wrote:
> > Continue on VLV PSR split with vfunc, let's also create one
> > for enabling source.
> >
> > Also since we are touching *_enable_source functions let's
> > fix a comment with
== Series Details ==
Series: meson: Build more tools and install more stuff
URL : https://patchwork.freedesktop.org/series/30155/
State : failure
== Summary ==
Test kms_cursor_legacy:
Subgroup flip-vs-cursor-varying-size:
pass -> FAIL (shard-hsw)
Test drv_su
== Series Details ==
Series: kms_rotation_crc: 90 degree flip test is not a stress test (rev5)
URL : https://patchwork.freedesktop.org/series/28308/
State : failure
== Summary ==
Test gem_exec_capture:
Subgroup capture-render:
pass -> INCOMPLETE (shard-hsw)
Test g
On Mon, 2017-09-11 at 20:52 +0100, Chris Wilson wrote:
> Quoting Tvrtko Ursulin (2017-09-11 16:34:08)
> >
> > On 11/09/2017 14:09, Michał Winiarski wrote:
> > > There's no reason to hide those tracepoints.
> > > Let's also remove the DRM_I915_LOW_LEVEL_TRACEPOINTS Kconfig option.
> >
> > No numbe
Quoting Tvrtko Ursulin (2017-09-11 16:34:08)
>
> On 11/09/2017 14:09, Michał Winiarski wrote:
> > There's no reason to hide those tracepoints.
> > Let's also remove the DRM_I915_LOW_LEVEL_TRACEPOINTS Kconfig option.
>
> No numbers from (micro-)bechmarks showing how small the impact of doing
> th
== Series Details ==
Series: series starting with [1/2] drm/i915/kbl: Remove unused Kabylake pci ids
URL : https://patchwork.freedesktop.org/series/30151/
State : success
== Summary ==
Test kms_atomic_transition:
Subgroup plane-all-transition-nonblocking:
fail ->
On Mon, 2017-09-11 at 16:33 +0300, Ville Syrjälä wrote:
> On Wed, Sep 06, 2017 at 05:14:58PM -0700, Dhinakaran Pandiyan wrote:
> > The POWER_DOWN_PHY and POWER_UP_PHY sideband message transactions allow
> > the source to reqest any node in a mst path or a whole path to be
> > powered down or up. Th
On Mon, Sep 11, 2017 at 08:51:51AM +0200, Daniel Vetter wrote:
> We're using this for outreachy, unfortunately someone already tried to
> look at a task that was done already :-(
>
> Update them all.
>
> Cc: Sean Paul
> Signed-off-by: Daniel Vetter
Applied to -misc-next
Thanks!
Sean
> ---
>
== Series Details ==
Series: drm/fb-helper: Use drm_file to get a dumb framebuffer
URL : https://patchwork.freedesktop.org/series/30136/
State : success
== Summary ==
Test kms_setmode:
Subgroup basic:
fail -> PASS (shard-hsw) fdo#99912
Test kms_atomic_transi
On Mon, Sep 11, 2017 at 01:19:15PM +, Shankar, Uma wrote:
>
>
> >-Original Message-
> >From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of
> >Daniel Vetter
> >Sent: Saturday, September 9, 2017 1:15 AM
> >To: Ville Syrjälä
> >Cc: intel-gfx@lists.freedesktop.o
== Series Details ==
Series: meson: Build more tools and install more stuff
URL : https://patchwork.freedesktop.org/series/30155/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
f9e0154630766b63617c64255a68e5129e233a4b igt/gem_evict_(alignment,everything):
On Mon, Sep 11, 2017 at 11:32:23AM +0530, Sagar Arun Kamble wrote:
> From: "Kamble, Sagar A"
>
> With GuC v9, new type of Default/critical logging in GuC to enable
> capturing minimal important logs in production systems efficiently.
> This patch enables this logging in GuC by default always. It
== Series Details ==
Series: kms_rotation_crc: 90 degree flip test is not a stress test (rev5)
URL : https://patchwork.freedesktop.org/series/28308/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
f9e0154630766b63617c64255a68e5129e233a4b igt/gem_evict_(align
On Mon, Sep 11, 2017 at 01:04:18PM +, Shankar, Uma wrote:
>
>
> >-Original Message-
> >From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com]
> >Sent: Friday, September 8, 2017 8:18 PM
> >To: Srinivas, Vidya
> >Cc: intel-gfx@lists.freedesktop.org; Kahola, Mika ;
> >Kamath, Sunil
On Mon, Sep 11, 2017 at 11:32:24AM +0530, Sagar Arun Kamble wrote:
> Disabling GuC interrupts involves access to GuC IRQ control registers
> hence ensure device is RPM awake.
>
> Cc: Michal Wajdeczko
> Signed-off-by: Sagar Arun Kamble
> ---
> drivers/gpu/drm/i915/intel_guc_log.c | 2 ++
> 1 fil
> -Original Message-
> From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel
> Vetter
> Sent: Monday, September 4, 2017 1:14 AM
> To: Sripada, Radhakrishna
> Cc: Vivi, Rodrigo ; Nikula, Jani
> ; intel-gfx@lists.freedesktop.org; Pandiyan,
> Dhinakaran ;
> nicholas.stom..
From: Ville Syrjälä
Signed-off-by: Ville Syrjälä
---
tools/meson.build | 5 +
1 file changed, 5 insertions(+)
diff --git a/tools/meson.build b/tools/meson.build
index b6052a0c71c3..66dd697973c3 100644
--- a/tools/meson.build
+++ b/tools/meson.build
@@ -57,6 +57,11 @@ endforeach
pkgdatad
From: Ville Syrjälä
Signed-off-by: Ville Syrjälä
---
tools/meson.build | 4
1 file changed, 4 insertions(+)
diff --git a/tools/meson.build b/tools/meson.build
index 0e64621d1d45..84b3467a6d72 100644
--- a/tools/meson.build
+++ b/tools/meson.build
@@ -75,6 +75,10 @@ executable('intel_reg',
From: Ville Syrjälä
Autotools called the binary 'intel-gpu-overlay', let's have meson use
that name as well. Also install it.
Signed-off-by: Ville Syrjälä
---
overlay/meson.build | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/overlay/meson.build b/overlay/meson.build
From: Ville Syrjälä
FIXME: install_subdir exclude_files knob seems to require meson 0.42+.
Is that acceptable?
Signed-off-by: Ville Syrjälä
---
tools/meson.build | 5 +
1 file changed, 5 insertions(+)
diff --git a/tools/meson.build b/tools/meson.build
index 9d08a0d1f986..b6052a0c71c3 1006
From: Ville Syrjälä
Signed-off-by: Ville Syrjälä
---
tests/meson.build | 7 +++
1 file changed, 7 insertions(+)
diff --git a/tests/meson.build b/tests/meson.build
index 37c4c4c24fa5..df315de2713c 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -301,3 +301,10 @@ foreach prog : han
From: Ville Syrjälä
Signed-off-by: Ville Syrjälä
---
tests/generate_testlist.sh | 9 ++---
tests/meson.build | 9 -
2 files changed, 14 insertions(+), 4 deletions(-)
diff --git a/tests/generate_testlist.sh b/tests/generate_testlist.sh
index 6ea78655daca..a0d53e9ff095 10075
From: Ville Syrjälä
Signed-off-by: Ville Syrjälä
---
tools/meson.build | 5 +
1 file changed, 5 insertions(+)
diff --git a/tools/meson.build b/tools/meson.build
index 66dd697973c3..bf24602396f1 100644
--- a/tools/meson.build
+++ b/tools/meson.build
@@ -57,6 +57,11 @@ endforeach
pkgdatad
From: Ville Syrjälä
Autotools didn't add the 'lib' prefix to the intel_aubdump.so filename.
Coerce meson to match that.
Signed-off-by: Ville Syrjälä
---
tools/meson.build | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/tools/meson.build b/tools/meson.build
index bf24602
From: Ville Syrjälä
Signed-off-by: Ville Syrjälä
---
tools/meson.build | 10 ++
1 file changed, 10 insertions(+)
diff --git a/tools/meson.build b/tools/meson.build
index 3b6d6376d2d6..9d08a0d1f986 100644
--- a/tools/meson.build
+++ b/tools/meson.build
@@ -55,6 +55,16 @@ foreach prog :
From: Ville Syrjälä
This series makes meson install a bunch of missing tools, and tries
to unify a bit more what meson vs. autotools installs.
Notable differences left after this series:
* asm/disasm missing from meson
* intel_aubdump script missing from meson
* some benchmarks/tests missing fro
From: Ville Syrjälä
autotools installs the tests into $libexecdir. Make meson do the same.
Signed-off-by: Ville Syrjälä
---
tests/meson.build | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/tests/meson.build b/tests/meson.build
index 4dd5a9c9d4c7..1f98f2a02bb9 10
From: Ville Syrjälä
Signed-off-by: Ville Syrjälä
---
tools/meson.build | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tools/meson.build b/tools/meson.build
index d2d4410e2130..3b6d6376d2d6 100644
--- a/tools/meson.build
+++ b/tools/meson.build
@@ -51,7 +51,8 @@ endif
f
From: Ville Syrjälä
We use IGT_DATADIR to refer to $pkgdatadir elsewhere, so let's have
intel_reg follow suit.
Signed-off-by: Ville Syrjälä
---
tools/Makefile.am | 2 +-
tools/intel_reg.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/Makefile.am b/tools/Makefile.
From: Tvrtko Ursulin
To the best of my recollection the page flipping test was added
simply to start exercising page flips with 90/270 rotation.
There is no need to do 60 flips which can take quite some time,
because we do 60 flips against each pipe and each fb geometry.
Also, calling this a st
On Mon, Sep 11, 2017 at 04:11:33PM +, Anuj Phogat wrote:
> See Mesa commits: ebc5ccf and b2dae9f
I believe we need to be in sync between multiple gfx stack components,
but I don't believe we should remove ids.
In the past we had cases where we noticed a product group using a listed
id to do
== Series Details ==
Series: series starting with [v6,1/2] drm/i915: Introduce INTEL_GEN_MASK
URL : https://patchwork.freedesktop.org/series/30124/
State : success
== Summary ==
Test perf:
Subgroup blocking:
fail -> PASS (shard-hsw) fdo#102252
fdo#102252 ht
== Series Details ==
Series: series starting with [1/2] drm/i915/kbl: Remove unused Kabylake pci ids
URL : https://patchwork.freedesktop.org/series/30151/
State : success
== Summary ==
Series 30151v1 series starting with [1/2] drm/i915/kbl: Remove unused Kabylake
pci ids
https://patchwork.fre
See Mesa commit 9c588ff
Cc: Matt Turner
Cc: Rodrigo Vivi
Signed-off-by: Anuj Phogat
---
intel/intel_chipset.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/intel/intel_chipset.h b/intel/intel_chipset.h
index 77a9ca6..6bd8ae2 100644
--- a/intel/intel_chipset.h
+++ b/
These PCI IDs are not used in any Kabylake SKUs.
See Mesa commits: ebc5ccf and b2dae9f
Cc: Matt Turner
Cc: Rodrigo Vivi
Signed-off-by: Anuj Phogat
---
intel/intel_chipset.h | 26 --
1 file changed, 4 insertions(+), 22 deletions(-)
diff --git a/intel/intel_chipset.h b/i
See Mesa commits: ebc5ccf and b2dae9f
Cc: Matt Turner
Cc: Rodrigo Vivi
Signed-off-by: Anuj Phogat
---
drivers/gpu/drm/i915/i915_pci.c | 1 -
include/drm/i915_pciids.h | 15 ++-
2 files changed, 2 insertions(+), 14 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_pci.c b/
See Mesa commit 9c588ff
Cc: Matt Turner
Cc: Rodrigo Vivi
Signed-off-by: Anuj Phogat
---
include/drm/i915_pciids.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h
index a1bf90e..1c29063 100644
--- a/include/drm/i915_pciid
== Series Details ==
Series: series starting with [01/11] drm/i915: Mark the userptr invalidate
workqueue as WQ_MEM_RECLAIM
URL : https://patchwork.freedesktop.org/series/30108/
State : warning
== Summary ==
Test perf:
Subgroup polling:
fail -> PASS (shard-
On 07/09/2017 18:07, Srivatsa, Anusha wrote:
-Original Message-
From: Tvrtko Ursulin [mailto:tvrtko.ursu...@linux.intel.com]
Sent: Thursday, September 7, 2017 1:49 AM
To: Srivatsa, Anusha ; intel-
g...@lists.freedesktop.org
Cc: Sundaresan, Sujaritha
Subject: Re: [Intel-gfx] [PATCH 1/2]
== Series Details ==
Series: i915 PMU and engine busy stats (rev3)
URL : https://patchwork.freedesktop.org/series/27488/
State : warning
== Summary ==
Series 27488v3 i915 PMU and engine busy stats
https://patchwork.freedesktop.org/api/1.0/series/27488/revisions/3/mbox/
Test chamelium:
On 11/09/2017 13:32, Joonas Lahtinen wrote:
Split INTEL_GEN_MASK out of IS_GEN macro, and make it usable
within static declarations (unlike combound statements).
Typo -> compound to fixup when merging.
Regards,
Tvrtko
Signed-off-by: Joonas Lahtinen
Cc: Jani Nikula
Cc: Chris Wilson
Revi
On 11/09/2017 14:09, Michał Winiarski wrote:
There's no reason to hide those tracepoints.
Let's also remove the DRM_I915_LOW_LEVEL_TRACEPOINTS Kconfig option.
No numbers from (micro-)bechmarks showing how small the impact of doing
this is? I thought John was compiling this data. It will be ju
From: Tvrtko Ursulin
Other kernel users might want to look at total GPU busyness
in order to implement things like package power distribution
algorithms more efficiently.
Signed-off-by: Tvrtko Ursulin
Cc: Ben Widawsky
Cc: Ben Widawsky
---
drivers/gpu/drm/i915/intel_engine_cs.c | 3 +++
1 fil
From: Tvrtko Ursulin
This reduces the cost of the software engine busyness tracking
to a single no-op instruction when there are no listeners.
v2: Rebase and some comments.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_pmu.c | 54 +++--
drivers/gpu/dr
From: Tvrtko Ursulin
Export the stats added in the previous patch in debugfs.
Number of active clients reading this data is tracked and the
static key is only enabled whilst there are some.
Userspace is intended to keep the file descriptor open, seeking
to the beginning of the file periodically
From: Tvrtko Ursulin
No functional change just something which will be handy in the
following patch.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/intel_lrc.c | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/driv
From: Tvrtko Ursulin
We can use engine busy stats instead of the MMIO sampling timer
for better efficiency.
As minimum this saves period * num_engines / sec mmio reads,
and in a better case, when only engine busy samplers are active,
it enables us to not kick off the sampling timer at all.
v2:
From: Tvrtko Ursulin
Will be used for exposing the PMU counters.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_drv.h | 8 +++-
drivers/gpu/drm/i915/intel_pm.c | 23 +--
2 files changed, 16 insertions(+), 15 deletions(-)
diff --git a/drivers/gpu/drm/i915/
From: Tvrtko Ursulin
Track total time requests have been executing on the hardware.
We add new kernel API to allow software tracking of time GPU
engines are spending executing requests.
Both per-engine and global API is added with the latter also
being exported for use by external users.
v2:
From: Tvrtko Ursulin
Code to be shared between debugfs and the PMU implementation.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_debugfs.c | 8 +---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/intel_pm.c | 14 ++
3 files changed, 16 inser
From: Tvrtko Ursulin
From: Chris Wilson
From: Tvrtko Ursulin
From: Dmitry Rogozhkin
The first goal is to be able to measure GPU (and invidual ring) busyness
without having to poll registers from userspace. (Which not only incurs
holding the forcewake lock indefinitely, perturbing the system,
From: Tvrtko Ursulin
If only a subset of events is enabled we can afford to suspend
the sampling timer when the GPU is idle and so save some cycles
and power.
v2: Rebase and limit timer even more.
v3: Rebase.
v4: Rebase.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_drv.h
From: Tvrtko Ursulin
Extract code from i915_energy_uJ (debugfs) so it can be used by
other callers in future patches.
v2: Rebase.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_debugfs.c | 17 +
drivers/gpu/drm/i915/i915_drv.h | 2 ++
drivers/gpu/drm/i915/int
From: Tvrtko Ursulin
Third spin of the i915 PMU series.
In this version I integrated some of the changes Dmitry made after clarifying
the perf API usage questions with Peter Zijlstra. Some other changes I
integrated in spirit but changed the implementation, and then finally there
are some new fi
On Mon, Sep 11, 2017 at 9:16 AM, Jani Nikula wrote:
> Add new environment variable $DIM_GPG_KEYID for configuring the GPG key
> ID of the key to use for signing tags for pull requests. The tags will
> be signed if the key ID is set, otherwise annotated tags will be used
> like before.
>
> The drm-
== Series Details ==
Series: igt/sw_sync: Fix up close(timeline) tests for unsignaled fences
URL : https://patchwork.freedesktop.org/series/30126/
State : failure
== Summary ==
IGT patchset tested on top of latest successful build
f9e0154630766b63617c64255a68e5129e233a4b igt/gem_evict_(alignme
Quoting Michał Winiarski (2017-09-11 15:17:31)
> To create an upper bound on number of GuC workitems, we need to change
> the way that requests are being submitted. Rather than submitting each
> request as an individual workitem, we can do coalescing in a similar way
> we're handlig execlist submis
== Series Details ==
Series: drm/fb-helper: Use drm_file to get a dumb framebuffer
URL : https://patchwork.freedesktop.org/series/30136/
State : success
== Summary ==
Series 30136v1 drm/fb-helper: Use drm_file to get a dumb framebuffer
https://patchwork.freedesktop.org/api/1.0/series/30136/rev
== Series Details ==
Series: series starting with [1/2] drm/i915/guc: Submit GuC workitems
containing coalesced requests
URL : https://patchwork.freedesktop.org/series/30135/
State : failure
== Summary ==
Series 30135v1 series starting with [1/2] drm/i915/guc: Submit GuC workitems
containing
Since the cma library now uses drm_framebuffer_create_dumb(),
drm_fbdev_cma_init() will provide a framebuffer with the dirty callback
set.
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/tinydrm/core/tinydrm-core.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/driver
Drop the unnecessary fbdev buffer offset, it can only be zero since
drm_fb_helper_fill_var() sets xoffset and yoffset to zero;
Cc: Laurent Pinchart
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_fb_cma_helper.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/d
No users left so remove drm_fbdev_cma_init_with_funcs().
Also remove example fbdev code in docs, there's nothing special now.
Cc: Laurent Pinchart
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_fb_cma_helper.c | 59 ++---
1 file changed, 3 insertions(+), 5
Use drm_framebuffer_create_dumb() to create a framebuffer for fbdev.
No need for error cleanup in drm_fbdev_cma_create(), since the one in
drm_fbdev_cma_init_with_funcs() will do fine.
Cc: Laurent Pinchart
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_fb_cma_helper.c | 44 +-
Allocate a drm_file so drivers can use drm_framebuffer_create_dumb()
to create a buffer. The framebuffer is reaped automtically in
drm_fb_helper_fini().
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_fb_helper.c | 22 --
include/drm/drm_fb_helper.h | 9 +
Export drm_dropmaster_ioctl() so in-kernel drm_file_alloc() users can
drop master.
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_auth.c | 1 +
drivers/gpu/drm/drm_internal.h | 2 --
include/drm/drm_auth.h | 2 ++
3 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/
drm_framebuffer_create_dumb() uses the dumb buffer API to create a
buffer that is attached to a framebuffer.
Useful for fbdev emulation.
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_framebuffer.c | 61 +++
include/drm/drm_framebuffer.h | 4 +++
2
From: David Herrmann
Rather than doing drm_file allocation/destruction right in the fops, lets
provide separate helpers. This decouples drm_file management from the
still-mandatory drm-fops. It prepares for use of drm_file without the
fops, both by possible separate fops implementations and APIs
Hi,
I want to start out by saying that this patchset is low priority for me
and if no one has interest or time to review this, that is just fine. I
was in the flow and just typed it out.
This patchset adds a way for fbdev emulation code to create a
framebuffer that is backed by a dumb buffer. drm
---
drivers/gpu/drm/i915/i915_params.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_params.c
b/drivers/gpu/drm/i915/i915_params.c
index 8ab003dca113..c9d72f1b8383 100644
--- a/drivers/gpu/drm/i915/i915_params.c
+++ b/drivers/gpu/drm/i915/i915_p
== Series Details ==
Series: series starting with [1/2] drm/i915/tracepoints: Don't compile-out
low-level tracepoints
URL : https://patchwork.freedesktop.org/series/30129/
State : warning
== Summary ==
Series 30129v1 series starting with [1/2] drm/i915/tracepoints: Don't
compile-out low-leve
To create an upper bound on number of GuC workitems, we need to change
the way that requests are being submitted. Rather than submitting each
request as an individual workitem, we can do coalescing in a similar way
we're handlig execlist submission ports. We also need to stop pretending
that we're
== Series Details ==
Series: series starting with [v6,1/2] drm/i915: Introduce INTEL_GEN_MASK
URL : https://patchwork.freedesktop.org/series/30124/
State : success
== Summary ==
Series 30124v1 series starting with [v6,1/2] drm/i915: Introduce INTEL_GEN_MASK
https://patchwork.freedesktop.org/ap
== Series Details ==
Series: series starting with [v5,1/2] drm/i915: Introduce INTEL_GEN_MASK
URL : https://patchwork.freedesktop.org/series/30113/
State : success
== Summary ==
Test perf:
Subgroup blocking:
pass -> FAIL (shard-hsw) fdo#102252
Test kms_setmo
On Wed, Sep 06, 2017 at 05:14:58PM -0700, Dhinakaran Pandiyan wrote:
> The POWER_DOWN_PHY and POWER_UP_PHY sideband message transactions allow
> the source to reqest any node in a mst path or a whole path to be
> powered down or up. This allows drivers to target a specific sink in the
> MST topolog
On Mon, Sep 11, 2017 at 04:15:59PM +0300, Jani Nikula wrote:
> Presumably an oversight while converting update-next-continue to use
> annotated tags.
>
> Cc: Daniel Vetter
> Signed-off-by: Jani Nikula
Reviewed-by: Ewelina Musial
> ---
> dim | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(
>-Original Message-
>From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of
>Daniel Vetter
>Sent: Saturday, September 9, 2017 1:15 AM
>To: Ville Syrjälä
>Cc: intel-gfx@lists.freedesktop.org; Srinivas, Vidya
>Subject: Re: [Intel-gfx] [PATCH] drm/i915: Enable scanli
== Series Details ==
Series: drm/i915: Skylake plane update/disable unifications [v3]
URL : https://patchwork.freedesktop.org/series/30122/
State : failure
== Summary ==
CHK include/config/kernel.release
CHK include/generated/uapi/linux/version.h
CHK include/generated/utsrele
Add new environment variable $DIM_GPG_KEYID for configuring the GPG key
ID of the key to use for signing tags for pull requests. The tags will
be signed if the key ID is set, otherwise annotated tags will be used
like before.
The drm-intel-testing tags are left unsigned, at least for now.
Cc: Dan
Presumably an oversight while converting update-next-continue to use
annotated tags.
Cc: Daniel Vetter
Signed-off-by: Jani Nikula
---
dim | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dim b/dim
index dbaeb1ec944d..23dfccb5513d 100755
--- a/dim
+++ b/dim
@@ -1589,7 +1589,7
1 - 100 of 188 matches
Mail list logo
