On Fri, Aug 11, 2017 at 4:39 PM, Rodrigo Vivi wrote:
> WC is apparently not an option for CNL+ on GTT here.
> Trying to use it we get hard hangs.
>
> Credits-to: Ben Widawsky
forgot to CC relavant people for possible reviews:
Cc: Joonas
Cc: Imre
Cc: Ben
> Signed-off-by: Rodrigo Vivi
> ---
> d
== Series Details ==
Series: drm/i915/cnl: Avoid ioremap_wc on Cannonlake as well.
URL : https://patchwork.freedesktop.org/series/28702/
State : success
== Summary ==
Series 28702v1 drm/i915/cnl: Avoid ioremap_wc on Cannonlake as well.
https://patchwork.freedesktop.org/api/1.0/series/28702/rev
== Series Details ==
Series: series starting with [1/6] drm/i915/gen10+: use the SKL code for
reading WM latencies (rev2)
URL : https://patchwork.freedesktop.org/series/28586/
State : success
== Summary ==
Series 28586v2 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/se
WC is apparently not an option for CNL+ on GTT here.
Trying to use it we get hard hangs.
Credits-to: Ben Widawsky
Signed-off-by: Rodrigo Vivi
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c
b/driver
Hi Daniel,
On Tuesday 25 Jul 2017 10:01:16 Daniel Vetter wrote:
> Atomic drivers only use the property value store for immutable (i.e.
> can't be set by userspace, but the kernel can still adjust it)
> properties. The only tricky part is the removal of the update in
> drm_atomic_helper_update_lega
merged to dinq. thanks for the reviews and ideas
On Fri, Aug 11, 2017 at 11:26 AM, Rodrigo Vivi wrote:
> The idea is to have an unique place to decide the pin-port
> per platform.
>
> So let's create this function now without any functional
> change. Just adding together code from hdmi and dp tog
== Series Details ==
Series: series starting with [1/2] drm/i915: Simplify hpd pin to port (rev4)
URL : https://patchwork.freedesktop.org/series/28261/
State : success
== Summary ==
Series 28261v4 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/series/28261/revisions/4/mb
merged to dinq.
thanks for the review.
On Thu, Aug 10, 2017 at 3:45 PM, Rodrigo Vivi wrote:
> Different from SKL we don't need ctrl1 and cfgcr2, but
> we need to dump cfgcr0 and cfgcr1 instead.
>
> v2: rebase and commit message
>
> Cc: Clint Taylor
> Cc: Mika Kahola
> Signed-off-by: Rodrigo Viv
== Series Details ==
Series: series starting with [v2,1/2] drm/dp: Bit definition for D3 power state
that keeps AUX fully powered (rev2)
URL : https://patchwork.freedesktop.org/series/28667/
State : success
== Summary ==
Series 28667v2 Series without cover letter
https://patchwork.freedesktop
The idea is to have an unique place to decide the pin-port
per platform.
So let's create this function now without any functional
change. Just adding together code from hdmi and dp together.
v2: Add missing pin for port A.
v3: Fix typo on subject.
Avoid behaviour change so add WARN_ON and ret
We will soon need to make that pin port association per
platform, so let's try to simplify it beforehand.
Also we are moving the backwards port to pin
here as well so let's use a standardized way.
One extra possibility here would be to add a
MISSING_CASE along with PORT_NONE, but I don't want
to
DPCD 600h - SET_POWER & SET_DP_PWR_VOLTAGE defines power state
101 = Set Main-Link for local Sink device and all downstream Sink
devices to D3 (power-down mode), keep AUX block fully powered, ready to
reply within a Response Timeout period of 300us.
This state is useful in a MST dock + MST monito
On Fri, Aug 11, 2017 at 06:03:24PM +0300, Jani Nikula wrote:
> We don't generally apply patches to the drm-intel-fixes or
> drm-intel-next-fixes trees, we cherry-pick instead, so the aliases are
> unnecessary.
great! this always confused me honestly, but I thought you were
the one using this someh
On Fri, Aug 11, 2017 at 06:03:25PM +0300, Jani Nikula wrote:
> dim list-aliases lists "list-aliases" as being an alias of
> "list-aliases". This is because the temporary subcmd variable reference
> is erroneously included in the list. Unset it.
I got confused with to many list-aliases list-aliases
Reviewed-by: Rodrigo Vivi
On Fri, Aug 11, 2017 at 06:03:23PM +0300, Jani Nikula wrote:
> Demote dim_cherry_pick_branch to an internal function. It's too
> specialized (at least for now) to be useful for anything other than as a
> helper for the cherry-pick-fixes and cherry-pick-next-fixes subc
Reviewed-by: Rodrigo Vivi
On Fri, Aug 11, 2017 at 06:03:22PM +0300, Jani Nikula wrote:
> Reported by 'make mancheck'.
>
> Signed-off-by: Jani Nikula
> ---
> dim.rst | 4
> 1 file changed, 4 insertions(+)
>
> diff --git a/dim.rst b/dim.rst
> index 8b4653aacbda..802c776e03f9 100644
> ---
On 08/11/2017 12:49 AM, Lofstedt, Marta wrote:
-Original Message-
From: Taylor, Clinton A
Sent: Thursday, August 10, 2017 8:50 PM
To: intel-gfx@lists.freedesktop.org
Cc: Taylor, Clinton A ; Vetter, Daniel
; Lofstedt, Marta
Subject: [PATCH v4 i-g-t] tests/kms: increase max threshold t
On Fri, Aug 11, 2017 at 02:24:19PM +, Deucher, Alexander wrote:
> > -Original Message-
> > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf
> > Of Cihangir Akturk
> > Sent: Friday, August 11, 2017 8:33 AM
> > Cc: de...@driverdev.osuosl.org; linux-arm-...@vger.kerne
dim list-aliases lists "list-aliases" as being an alias of
"list-aliases". This is because the temporary subcmd variable reference
is erroneously included in the list. Unset it.
Signed-off-by: Jani Nikula
---
dim | 3 +++
1 file changed, 3 insertions(+)
diff --git a/dim b/dim
index d9820ac87358
Demote dim_cherry_pick_branch to an internal function. It's too
specialized (at least for now) to be useful for anything other than as a
helper for the cherry-pick-fixes and cherry-pick-next-fixes subcommands.
As a side effect, fixes mancheck for missing documentation for the
subcommand.
Signed-o
Reported by 'make mancheck'.
Signed-off-by: Jani Nikula
---
dim.rst | 4
1 file changed, 4 insertions(+)
diff --git a/dim.rst b/dim.rst
index 8b4653aacbda..802c776e03f9 100644
--- a/dim.rst
+++ b/dim.rst
@@ -204,6 +204,10 @@ apply [*git am arguments*]
--
**apply-b
On Fri, Aug 11, 2017 at 11:36:15AM +0100, Chris Wilson wrote:
> Quoting Daniel Vetter (2017-08-11 09:04:18)
> > On Fri, Aug 11, 2017 at 09:23:27AM +0200, Daniel Vetter wrote:
> > > In our snb farm in CI we have plenty of underruns, but not enough
> > > stolen memory to enable fbc. Which means every
>On Fri, 11 Aug 2017, Daniel Vetter wrote:
> > Cc: "Tahvanainen, Jari"
> > Signed-off-by: Daniel Vetter
> > ---
> > dim | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/dim b/dim
> > index 2b377cb3a3f3..4a4c1adb2c68 100755
> > --- a/dim
> > +++ b/dim
> > @@ -89,7 +
On Fri, 11 Aug 2017, Daniel Vetter wrote:
> Cc: "Tahvanainen, Jari"
> Signed-off-by: Daniel Vetter
> ---
> dim | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dim b/dim
> index 2b377cb3a3f3..4a4c1adb2c68 100755
> --- a/dim
> +++ b/dim
> @@ -89,7 +89,7 @@ addr_intel_gfx_
> -Original Message-
> From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf
> Of Cihangir Akturk
> Sent: Friday, August 11, 2017 8:33 AM
> Cc: de...@driverdev.osuosl.org; linux-arm-...@vger.kernel.org; intel-
> g...@lists.freedesktop.org; linux-ker...@vger.kernel.org; dri-
Cc: "Tahvanainen, Jari"
Signed-off-by: Daniel Vetter
---
dim | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dim b/dim
index 2b377cb3a3f3..4a4c1adb2c68 100755
--- a/dim
+++ b/dim
@@ -89,7 +89,7 @@ addr_intel_gfx_maintainer1="Daniel Vetter
"
addr_intel_gfx_maintainer2="Jani
From: Colin Ian King
The structure intel_sprite_plane_funcs is local to the source
and does not need to be in global scope, so make it static.
Cleans up sparse warning:
symbol 'intel_sprite_plane_funcs' was not declared. Should it be static?
Signed-off-by: Colin Ian King
---
drivers/gpu/drm/i
== Series Details ==
Series: series starting with [1/2] drm: add retries for lspcon status check
URL : https://patchwork.freedesktop.org/series/28684/
State : success
== Summary ==
Series 28684v1 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/series/28684/revisions/1/mbo
Acked-by: Marta Lofstedt
> -Original Message-
> From: Maarten Lankhorst [mailto:maarten.lankho...@linux.intel.com]
> Sent: Thursday, August 10, 2017 3:42 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Maarten Lankhorst ; Lofstedt,
> Marta
> Subject: [PATCH i-g-t] tests/kms_mmap_write_crc
Our current logic to read LSPCON's current mode, stops retries and
breaks wait-loop, if it gets LSPCON_MODE_INVALID as return from the
core function. This doesn't allow us to try reading the mode again.
This patch removes this condition and allows retries reading
the currnt mode until timeout.
Th
It's an observation during some CI tests that few LSPCON chips
respond slow while system is under load, and need some delay
while reading current mode status using i2c-over-aux channel.
This patch:
- Adds few retries and delays before declaring a read
failure from LSPCON hardware.
- Changes the
Quoting Jani Nikula (2017-08-11 14:15:50)
> On Fri, 11 Aug 2017, Chris Wilson wrote:
> > Many times an error may occur before the start of igt, leaving the
> > system in a less-than-optimal debugging state (e.g. an oops turning off
> > lockdep). Flag such occasions by checking /proc/sys/kernel/tai
== Series Details ==
Series: igt: Add a test to precheck status of kernel taints
URL : https://patchwork.freedesktop.org/series/28683/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
1385b31d9371fae02af2fd8adb0d9ea86a5bb0f2 tests/igt_command_line: Ignore sub
On Fri, 11 Aug 2017, Chris Wilson wrote:
> Many times an error may occur before the start of igt, leaving the
> system in a less-than-optimal debugging state (e.g. an oops turning off
> lockdep). Flag such occasions by checking /proc/sys/kernel/tainted.
Hmm, which flag is set for unsafe module pa
Many times an error may occur before the start of igt, leaving the
system in a less-than-optimal debugging state (e.g. an oops turning off
lockdep). Flag such occasions by checking /proc/sys/kernel/tainted.
Signed-off-by: Chris Wilson
---
tests/Makefile.sources | 1 +
tests/kernel_taint.c | 7
== Series Details ==
Series: series starting with [1/4] lib: Add some syncobj helpers
URL : https://patchwork.freedesktop.org/series/28666/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
1385b31d9371fae02af2fd8adb0d9ea86a5bb0f2 tests/igt_command_line: Ignor
Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
and drm_*_unreference() helpers.
drm_*_reference() and drm_*_unreference() functions are just
compatibility alias for drm_*_get() and drm_*_put() and should not be
used by new code. So convert all users of compatibility functions
Changes since v2:
- Patch series is based on *drm-misc-next* as suggested by Sean Paul.
- Dropped patch 05 (drm/atmel-hlcdc) and patch 25 (drm/vc4) from v2,
since they were already pulled in the drm-misc-next
Changes since v1:
- This time patches were generated with coccinelle instead of my o
== Series Details ==
Series: tests/kms: increase max threshold time for edid read (rev5)
URL : https://patchwork.freedesktop.org/series/28399/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
1385b31d9371fae02af2fd8adb0d9ea86a5bb0f2 tests/igt_command_line: Ig
On 10/08/17 18:57, Chris Wilson wrote:
Another case where we need to call sysfs_attr_init() to setup the
internal lockdep class prior to use:
[9.325229] BUG: key 880168bc7bb0 not in .data!
[9.325240] DEBUG_LOCKS_WARN_ON(1)
[9.325250] [ cut here ]
[9.32
== Series Details ==
Series: drm/i915/vbt: ignore extraneous child devices for a port
URL : https://patchwork.freedesktop.org/series/28681/
State : success
== Summary ==
Series 28681v1 drm/i915/vbt: ignore extraneous child devices for a port
https://patchwork.freedesktop.org/api/1.0/series/286
Quoting Joonas Lahtinen (2017-08-11 08:34:02)
> On ke, 2017-07-26 at 14:25 +0100, Chris Wilson wrote:
> > - if (vm->pt_kmap_wc)
> > - set_pages_array_wb(vm->free_pages.pages,
> > - pagevec_count(&vm->free_pages));
> > + /* When we use WC, f
== Series Details ==
Series: drm/i915: Split obj->cache_coherent to track r/w (rev4)
URL : https://patchwork.freedesktop.org/series/28641/
State : success
== Summary ==
Series 28641v4 drm/i915: Split obj->cache_coherent to track r/w
https://patchwork.freedesktop.org/api/1.0/series/28641/revisi
Ever since we've parsed VBT child devices, starting from 6acab15a7b0d
("drm/i915: use the HDMI DDI buffer translations from VBT"), we've
ignored the child device information if more than one child device
references the same port. The rationale for this seems lost in time.
Since commit 311a20949f04
== Series Details ==
Series: drm/i915: Handle full s64 precision for wait-ioctl (rev2)
URL : https://patchwork.freedesktop.org/series/28420/
State : success
== Summary ==
Series 28420v2 drm/i915: Handle full s64 precision for wait-ioctl
https://patchwork.freedesktop.org/api/1.0/series/28420/re
Quoting Chris Wilson (2017-08-11 11:11:31)
> diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> index 5fa44767c29e..9d808838a1ba 100644
> --- a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> +++ b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> @@ -
Another month, another story in the cache coherency saga. This time, we
come to the realisation that i915_gem_object_is_coherent() has been
reporting whether we can read from the target without requiring a cache
invalidate; but we were using it in places for testing whether we could
write into the
The wait-ioctl is optionally supplied a timeout with nanosecond
precision in a s64 field. We use nsecs_to_jiffies64() to convert that
into the jiffies consumed by the scheduler, but internally
nsecs_to_jiffies64() does not guard against overflow (as it's purpose is
for use by the scheduler and not
== Series Details ==
Series: drm/i915: Split obj->cache_coherent to track r/w (rev3)
URL : https://patchwork.freedesktop.org/series/28641/
State : failure
== Summary ==
Series 28641v3 drm/i915: Split obj->cache_coherent to track r/w
https://patchwork.freedesktop.org/api/1.0/series/28641/revisi
Quoting Daniel Vetter (2017-08-11 09:04:18)
> On Fri, Aug 11, 2017 at 09:23:27AM +0200, Daniel Vetter wrote:
> > In our snb farm in CI we have plenty of underruns, but not enough
> > stolen memory to enable fbc. Which means every time there's an
> > underrun the no_fbc_reason swichtes to something
Quoting Joonas Lahtinen (2017-08-11 10:51:26)
> Configurations like virtualized environments may support only 48 bit
> ppGTT without supporting 32 bit ppGTT. Support this by disconnecting
> the relationship of the two feature bits.
Did the gvt patches land in dinq? After that, I say we just kill t
== Series Details ==
Series: drm/i915: Disconnect 32 and 48 bit ppGTT support
URL : https://patchwork.freedesktop.org/series/28676/
State : success
== Summary ==
Series 28676v1 drm/i915: Disconnect 32 and 48 bit ppGTT support
https://patchwork.freedesktop.org/api/1.0/series/28676/revisions/1/m
Another month, another story in the cache coherency saga. This time, we
come to the realisation that i915_gem_object_is_coherent() has been
reporting whether we can read from the target without requiring a cache
invalidate; but we were using it in places for testing whether we could
write into the
Paulo, my currently conclusion in
https://bugs.freedesktop.org/show_bug.cgi?id=101623 is that the more than 2
second wait for enable_fbs only occurs when changing between draw domains,
typically between blt and mmap_cpu.
To me this appear to be way too long time, but I am no expert here.
I don
Another month, another story in the cache coherency saga. This time, we
come to the realisation that i915_gem_object_is_coherent() has been
reporting whether we can read from the target without requiring a cache
invalidate; but we were using it in places for testing whether we could
write into the
== Series Details ==
Series: drm/i915/fbc: only update no_fbc_reason when active
URL : https://patchwork.freedesktop.org/series/28674/
State : success
== Summary ==
Series 28674v1 drm/i915/fbc: only update no_fbc_reason when active
https://patchwork.freedesktop.org/api/1.0/series/28674/revisio
== Series Details ==
Series: series starting with [1/2] drm/dp: Bit definition for D3 power state
that keeps AUX fully powered
URL : https://patchwork.freedesktop.org/series/28667/
State : success
== Summary ==
Series 28667v1 Series without cover letter
https://patchwork.freedesktop.org/api/1
Configurations like virtualized environments may support only 48 bit
ppGTT without supporting 32 bit ppGTT. Support this by disconnecting
the relationship of the two feature bits.
Cc: Tina Zhang
Cc: Chris Wilson
Cc: Zhi Wang
Signed-off-by: Joonas Lahtinen
---
drivers/gpu/drm/i915/i915_gem_gtt
On to, 2017-08-10 at 07:41 +0800, Tina Zhang wrote:
> Enable the guest i915 full ppgtt functionality when host can provide this
> capability. vgt_caps is introduced to guest i915 driver to get the vgpu
> capabilities from the device model. VGT_CPAS_FULL_PPGTT is one of the
> capabilities type to le
On Wed, 26 Jul 2017, Chris Wilson wrote:
> We use WC pages for coherent writes into the ppGTT on !llc
> architectuures. However, to create a WC page requires a stop_machine(),
> i.e. is very slow. To compensate we currently keep a per-vm cache of
> recently freed pages, but we still see the slow s
== Series Details ==
Series: drm/i915/cnl: Dump the right pll registers when dumping pipe config.
URL : https://patchwork.freedesktop.org/series/28665/
State : success
== Summary ==
Series 28665v1 drm/i915/cnl: Dump the right pll registers when dumping pipe
config.
https://patchwork.freedeskt
On Thu, Aug 03, 2017 at 01:52:55PM -0400, Alex Deucher wrote:
> On Thu, Aug 3, 2017 at 9:54 AM, Daniel Vetter wrote:
> > On Thu, Aug 3, 2017 at 1:17 AM, Daniel Vetter
> > wrote:
> >> On Wed, Aug 2, 2017 at 10:50 PM, Alex Deucher
> >> wrote:
> >>> On Wed, Aug 2, 2017 at 7:56 AM, Daniel Vetter
== Series Details ==
Series: series starting with drm/i915/cnl: Add allowed DP rates for Cannonlake.
(rev2)
URL : https://patchwork.freedesktop.org/series/26952/
State : success
== Summary ==
Series 26952v2 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/series/26952/rev
== Series Details ==
Series: drm/i915: Split pin mapping into per platform functions (rev2)
URL : https://patchwork.freedesktop.org/series/27965/
State : success
== Summary ==
Series 27965v2 drm/i915: Split pin mapping into per platform functions
https://patchwork.freedesktop.org/api/1.0/serie
On Fri, 11 Aug 2017, Daniel Vetter wrote:
> On Thu, Aug 10, 2017 at 05:08:12PM +0300, Jani Nikula wrote:
>> Thou shalt not push dim patches before passing 'make check'.
>
> On the series:
>
> Reviewed-by: Daniel Vetter
Thanks, pushed.
BR,
Jani.
>
>>
>> Cc: Daniel Vetter
>> Cc: Maarten Lankho
On Thu, Aug 10, 2017 at 01:26:47PM +0300, Petri Latvala wrote:
> The current documentation for tests is limited to a single string per
> test binary. This patch adds support for documenting individual
> subtests.
>
> The syntax for subtest documentation is:
>
>igt_document_subtest("Frob knobs
== Series Details ==
Series: drm/i915: Enable FBC for non X-tiled FBs (rev6)
URL : https://patchwork.freedesktop.org/series/21264/
State : success
== Summary ==
Series 21264v6 drm/i915: Enable FBC for non X-tiled FBs
https://patchwork.freedesktop.org/api/1.0/series/21264/revisions/6/mbox/
Tes
On to, 2017-08-10 at 17:20 +0100, Chris Wilson wrote:
> Another month, another story in the cache coherency saga. This time, we
> come to the realisation that i915_gem_object_is_coherent() has been
> reporting whether we can read from the target without requiring a cache
> invalidate; but we were u
On Fri, Aug 11, 2017 at 09:23:27AM +0200, Daniel Vetter wrote:
> In our snb farm in CI we have plenty of underruns, but not enough
> stolen memory to enable fbc. Which means every time there's an
> underrun the no_fbc_reason swichtes to something that makes
> kms_frontbuffer_tracking fail instead o
On Thu, Aug 10, 2017 at 05:08:12PM +0300, Jani Nikula wrote:
> Thou shalt not push dim patches before passing 'make check'.
On the series:
Reviewed-by: Daniel Vetter
>
> Cc: Daniel Vetter
> Cc: Maarten Lankhorst
> Cc: Benjamin Gaignard
>
>
> Jani Nikula (4):
> dim: Fix shellcheck SC1083
On Thu, Aug 10, 2017 at 02:20:27PM +0100, Chris Wilson wrote:
> Quoting Katarzyna Dec (2017-08-10 14:06:15)
> > In addition to checking whether the frequency is in correct range
> > for certain scenario, we can also verify whether PM interrupts are
> > masked correctly.
>
> What does correctly mea
Hello Xu Han,
The patch 5cf5fe8f729b: "drm/i915/gvt: Update MMIO handle policy to
compatible KBL platform." from Mar 29, 2017, leads to the following
static checker warning:
drivers/gpu/drm/i915/gvt/handlers.c:2855 init_skl_mmio_info()
'0x6 | 0x4' has '0x4' set on both sides
driv
> -Original Message-
> From: Taylor, Clinton A
> Sent: Thursday, August 10, 2017 8:50 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Taylor, Clinton A ; Vetter, Daniel
> ; Lofstedt, Marta
> Subject: [PATCH v4 i-g-t] tests/kms: increase max threshold time for edid
> read
>
> From: Clint
On ke, 2017-07-26 at 14:26 +0100, Chris Wilson wrote:
> We free objects in bulk after they wait for their RCU grace period.
> Currently, we take struct_mutex and unbind all the objects. This can lead
> to a long lock duration during which time those objects have their pages
> unfreeable (i.e. the s
On ke, 2017-07-26 at 14:25 +0100, Chris Wilson wrote:
> We use WC pages for coherent writes into the ppGTT on !llc
> architectuures. However, to create a WC page requires a stop_machine(),
> i.e. is very slow. To compensate we currently keep a per-vm cache of
> recently freed pages, but we still se
Op 10-08-17 om 18:20 schreef Chris Wilson:
> Another month, another story in the cache coherency saga. This time, we
> come to the realisation that i915_gem_object_is_coherent() has been
> reporting whether we can read from the target without requiring a cache
> invalidate; but we were using it in
In our snb farm in CI we have plenty of underruns, but not enough
stolen memory to enable fbc. Which means every time there's an
underrun the no_fbc_reason swichtes to something that makes
kms_frontbuffer_tracking fail instead of skip, adding massive amounts
of additional noise to igt test runs.
M
On la, 2017-08-05 at 20:47 +0100, Chris Wilson wrote:
> Quoting Chris Wilson (2017-08-05 20:19:24)
> > +++ b/drivers/gpu/drm/i915/i915_drv.h
> > @@ -4144,6 +4144,12 @@ static inline unsigned long
> > msecs_to_jiffies_timeout(const unsigned int m)
> >
> > static inline unsigned long nsecs_to_
On la, 2017-08-05 at 20:19 +0100, Chris Wilson wrote:
> The wait-ioctl is optionally supplied a timeout with nanosecond
> precision in a s64 field. We use nsecs_to_jiffies64() to convert that
> into the jiffies consumed by the scheduler, but internally
> nsecs_to_jiffies64() does not guard against
79 matches
Mail list logo