On Thu, 05 Jul 2018, Daniel Vetter wrote:
> On Thu, Jul 5, 2018 at 3:53 PM, Jani Nikula wrote:
>> On Wed, 27 Jun 2018, Daniel Vetter wrote:
>>> On Wed, Jun 27, 2018 at 5:13 PM, Jani Nikula wrote:
As a rule of thumb, don't change patches while committing.
Cc: Imre Deak
Signe
== Series Details ==
Series: series starting with [01/27] drm/i915: Squelch very verbose error
logging
URL : https://patchwork.freedesktop.org/series/46051/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
47eefccc61dd drm/i915: Squelch very verbose error logging
-:10: WARNING:CO
== Series Details ==
Series: series starting with [01/27] drm/i915: Squelch very verbose error
logging
URL : https://patchwork.freedesktop.org/series/46051/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Commit: drm/i915: Squelch very verbose error logging
+
+Error in reading or en
> -Original Message-
> From: Nikula, Jani
> Sent: Thursday, July 5, 2018 6:55 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Nikula, Jani ; Chauhan, Madhav
> ; Daniel Vetter ; Chris
> Wilson ; Ville Syrjälä
>
> Subject: [PATCH v3 1/3] drm/i915/dsi: rename the current DSI files based on
> f
== Series Details ==
Series: series starting with [01/27] drm/i915: Squelch very verbose error
logging
URL : https://patchwork.freedesktop.org/series/46051/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4441 -> Patchwork_9560 =
== Summary - SUCCESS ==
No regressions fou
On Fri, 06 Jul 2018, "Chauhan, Madhav" wrote:
>> -Original Message-
>> From: Nikula, Jani
>> Sent: Thursday, July 5, 2018 6:55 PM
>> To: intel-gfx@lists.freedesktop.org
>> Cc: Nikula, Jani ; Chauhan, Madhav
>> ; Daniel Vetter ; Chris
>> Wilson ; Ville Syrjälä
>>
>> Subject: [PATCH v3 1/3]
== Series Details ==
Series: ICELAKE DSI DRIVER (rev4)
URL : https://patchwork.freedesktop.org/series/44823/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4441 -> Patchwork_9561 =
== Summary - SUCCESS ==
No regressions found.
External URL:
https://patchwork.freedeskt
Hey Jakob,
On Thu, 5 Jul 2018 at 14:32, Jakob Bornecrantz wrote:
> So from a VR compositor getting blocked like this is a no-go as the
> user would quickly throw EPUKE. The situation is compounded by the
> fact that the VR compositor has no idea what the display compositor is
> doing with regards
== Series Details ==
Series: ICELAKE DSI DRIVER (rev4)
URL : https://patchwork.freedesktop.org/series/44823/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4438_full -> Patchwork_9549_full =
== Summary - FAILURE ==
Serious unknown changes coming with Patchwork_9549_full a
On Thu, 05 Jul 2018, Lucas De Marchi wrote:
> Instead of defining all registers twice, define just a PCH_GPIO_BASE
> that has the same address as PCH_GPIO_A and use that to calculate all
> the others. This also brings VLV and !HAS_GMCH_DISPLAY in line, doing
> the same thing.
>
> Signed-off-by: Lu
On 6 July 2018 at 07:53, Chris Wilson wrote:
> We test the GPU handling of huge pages by submitting requests that write
> into a huge page, but if the GPU is irrecoverably wedged we cannot
> submit any requests. As the test expectedly fails, skip over it.
>
> Signed-off-by: Chris Wilson
Reviewed-
Borrow the idea from net_dim.h to simplify the common determination of
how many bits in a particular type (sizeof(type) * BITS_PER_BYTE).
Suggested-by: Ville Syrjälä
Signed-off-by: Chris Wilson
Cc: Ville Syrjälä
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Tvrtko Ursulin
---
drivers/gpu/drm/i915
On 6 July 2018 at 07:53, Chris Wilson wrote:
> As we keep VMA around until the object is destroyed, when testing
> partial tiling we instantiate many, many VMA (as the object is huge
> allowing for many different partial regions). We test elsewhere our
> handling of populating large objects with a
== Series Details ==
Series: drm/i915: Introduce BITS_PER_TYPE
URL : https://patchwork.freedesktop.org/series/46055/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
c827b8e3ce5f drm/i915: Introduce BITS_PER_TYPE
-:30: WARNING:AVOID_BUG: Avoid crashing the kernel - try using WARN_
On 06/07/2018 09:44, Chris Wilson wrote:
Borrow the idea from net_dim.h to simplify the common determination of
how many bits in a particular type (sizeof(type) * BITS_PER_BYTE).
Suggested-by: Ville Syrjälä
Signed-off-by: Chris Wilson
Cc: Ville Syrjälä
Cc: Jani Nikula
Cc: Joonas Lahtinen
C
On 6 July 2018 at 07:53, Chris Wilson wrote:
> Having found the error causing the IGT test to fail, downgrade the
> verbose logging so that we stop flooding the syslogs as we deliberately
> provoke it many thousands of time during selftests.
>
> References: 10195b1e4411 ("drm/i915: Show vma alloca
On 05/07/2018 20:26, Chris Wilson wrote:
As we want to compare a templated tiling pattern against the target_bo,
we need to know that the swizzling is compatible. Or else the two
tiling pattern may differ due to underlying page address that we cannot
know, and so the test may sporadically fail.
On 6 July 2018 at 07:53, Chris Wilson wrote:
> If the GPU is irrecoverably broken, we can not use it to dirty memory
> and check for cache coherency with the CPU. All we can do is simply skip
> over the GPU subtests and focus on the CPU domains (WC, WB) cache
> management.
>
> Bugzilla: https://bu
On Fri, 06 Jul 2018, Chris Wilson wrote:
> Borrow the idea from net_dim.h to simplify the common determination of
> how many bits in a particular type (sizeof(type) * BITS_PER_BYTE).
Nice. Follow-up, have that included in bitops.h?
BR,
Jani.
>
> Suggested-by: Ville Syrjälä
> Signed-off-by: Chr
== Series Details ==
Series: drm/i915/selftests: Skip live context execution test without logical
contexts
URL : https://patchwork.freedesktop.org/series/46015/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4438_full -> Patchwork_9550_full =
== Summary - WARNING ==
Mino
On 6 July 2018 at 07:53, Chris Wilson wrote:
> If the GPU is wedged, we cannot make the object busy as trying to
> submit a request will generate -EIO. Skip to the end of the test.
>
> Signed-off-by: Chris Wilson
Reviewed-by: Matthew Auld
___
Intel-gfx
== Series Details ==
Series: drm/i915: Introduce BITS_PER_TYPE
URL : https://patchwork.freedesktop.org/series/46055/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4443 -> Patchwork_9562 =
== Summary - SUCCESS ==
No regressions found.
External URL:
https://patchwork.f
On 6 July 2018 at 07:53, Chris Wilson wrote:
> If the GPU is irrecoverably wedge, we cannot submit any request and so
> all of the request selftests will expectedly fail. Skip over them.
>
> Signed-off-by: Chris Wilson
Reviewed-by: Matthew Auld
___
Int
On 6 July 2018 at 07:53, Chris Wilson wrote:
> If the GPU is irrecoverably wedged, we cannot submit any request and
> therefore cannot query the register state of the context (which is done
> using the GPU command stream). So skip over the test as it expectedly
> fails.
>
> Signed-off-by: Chris Wi
On 6 July 2018 at 07:53, Chris Wilson wrote:
> If the GPU is irrecoverably wedged, we cannot submit any requests and so
> cannot make the GTT busy in order to test evicting active objects. As
> this expectedly fails, skip over the test.
>
> Signed-off-by: Chris Wilson
Reviewed-by: Matthew Auld
_
On Thu, 05 Jul 2018, Madhav Chauhan wrote:
> This patch defines AUX lane registers for PORT_PCS_DW1,
> PORT_TX_DW2, PORT_TX_DW4, PORT_TX_DW5 used during
> dsi enabling.
>
> v2: Review comments from Jani N:
> - Define _ICL_PORT_PCS_DW1_AUX_A for consistency
> - Three spaces for bitfield def
On 6 July 2018 at 07:53, Chris Wilson wrote:
> If the GPU is terminally wedged we cannot submit any requests into a
> context, completely unfulfilling our purpose of doing so. As this
> expectedly fails, skip over the test.
>
> Signed-off-by: Chris Wilson
Reviewed-by: Matthew Auld
__
On Thu, Jul 5, 2018 at 10:40 PM, Ville Syrjälä
wrote:
> On Thu, Jul 05, 2018 at 03:58:36PM +0800, Chris Chiu wrote:
>> Hi,
>> We have few ASUS laptops X705FD (The new WiskyLake), X560UD (intel
>> i5-8250U), X530UN (intel i7-8550U) share the same problem, which is
>> the HDMI connector status s
On Tue, 03 Jul 2018, Zhenyu Wang wrote:
> Hi,
>
> Here's two gvt fixes for 4.18. One is for guest warning to change
> virtual transcoder mode for DVI to align with our virtual display,
> and one to fix possible partial GGTT entry update from guest.
Pulled, and already included in a pull request t
Avoid looking at the magical engines[RCS] to decide if the HW and driver
supports logical contexts, and instead record that knowledge during
initialisation.
Signed-off-by: Chris Wilson
Cc: Matthew Auld
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/i915_gem_context.c
If the HW (or driver) doesn't support logical contexts, don't pretend we
gain anything from trying to execute GPU commands with them. At best it
reports -ENODEV, which is an unhelpful failure that we should just skip.
v2: Be more specific and check the driver/engine caps for logical (HW)
context s
If the HW (or driver) doesn't support logical contexts, don't pretend we
gain anything from trying to execute GPU commands with them. At best it
reports -ENODEV, which is an unhelpful failure that we should just skip.
v2: Be more specific and check the driver/engine caps for logical (HW)
context s
Quoting Chris Wilson (2018-07-06 11:14:42)
> If the HW (or driver) doesn't support logical contexts, don't pretend we
> gain anything from trying to execute GPU commands with them. At best it
> reports -ENODEV, which is an unhelpful failure that we should just skip.
>
> v2: Be more specific and ch
On 6 July 2018 at 11:14, Chris Wilson wrote:
> Avoid looking at the magical engines[RCS] to decide if the HW and driver
> supports logical contexts, and instead record that knowledge during
> initialisation.
>
> Signed-off-by: Chris Wilson
> Cc: Matthew Auld
Reviewed-by: Matthew Auld
__
Using a VMA on more than one timeline concurrently is the exception
rather than the rule (using it concurrently on multiple engines). As we
expect to only use one active tracker, store the most recently used
tracker inside the i915_vma itself and only fallback to the rbtree if
we need a second or m
Currently all callers are responsible for adding the vma to the active
timeline and then exporting its fence. Combine the two operations into
i915_vma_move_to_active() to move all the extra handling from the
callers to the single site.
Signed-off-by: Chris Wilson
Reviewed-by: Tvrtko Ursulin
---
Handling such a late error in request construction is tricky, but to
accommodate future patches which may allocate here, we potentially could
err. To handle the error after already adjusting global state to track
the new request, we must finish and submit the request. But we don't
want to use the r
i915_vma_move_to_active() has grown beyond its execbuf origins, and
should take its rightful place in i915_vma.c as a method for i915_vma!
Signed-off-by: Chris Wilson
Reviewed-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_drv.h| 3 --
drivers/gpu/drm/i915/i915_gem_execbuffer.c |
In the next patch, we will want to be able to use more flexible request
timelines that can hop between engines. From the vma pov, we can then
not rely on the binding of this request to an engine and so can not
ensure that different requests are ordered through a per-engine
timeline, and so we must
In the next patch, we will want to start skipping requests on failing to
complete their payloads. So export the utility function current used to
make requests inoperable following a failed gpu reset.
Signed-off-by: Chris Wilson
Reviewed-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_gem.c
On 6 July 2018 at 11:19, Chris Wilson wrote:
> If the HW (or driver) doesn't support logical contexts, don't pretend we
> gain anything from trying to execute GPU commands with them. At best it
> reports -ENODEV, which is an unhelpful failure that we should just skip.
>
> v2: Be more specific and
On Thu, Jul 05, 2018 at 02:29:59PM -0700, Rodrigo Vivi wrote:
> On Thu, Jul 05, 2018 at 07:43:52PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Rather than looping over all the ports and picking the encoder based on
> > the port, let's just loop over all the encoders instead. Gets
== Series Details ==
Series: drm/i915: Hotplug cleanups and whanot
URL : https://patchwork.freedesktop.org/series/46022/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4438_full -> Patchwork_9552_full =
== Summary - WARNING ==
Minor unknown changes coming with Patchwork_9
On 06/07/2018 11:39, Chris Wilson wrote:
Using a VMA on more than one timeline concurrently is the exception
rather than the rule (using it concurrently on multiple engines). As we
expect to only use one active tracker, store the most recently used
tracker inside the i915_vma itself and only fal
On Fri, 2018-06-01 at 21:39 +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> There's not much point in following the primary vs. sprite split
> for the SKL+ universal plane init code. The only difference is
> of our own doing in the form of the .check_plane(). Let's make
> a small exception
== Series Details ==
Series: series starting with [1/2] drm/i915: Record logical context support in
driver caps (rev2)
URL : https://patchwork.freedesktop.org/series/46065/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
3d66e5f3e2e2 drm/i915: Record logical context support in d
== Series Details ==
Series: series starting with [1/2] drm/i915: Record logical context support in
driver caps (rev2)
URL : https://patchwork.freedesktop.org/series/46065/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Commit: drm/i915: Record logical context support in driver cap
If the GPU is irrecoverably wedged, we can not execute any requests
making testing execlists (request execution) pointless. Skip!
Signed-off-by: Chris Wilson
Cc: Matthew Auld
---
drivers/gpu/drm/i915/selftests/intel_lrc.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/i9
If we have just completed a WC write, we must ensure that the WCB (Write
Combining Buffer) is flushed out to main memory before we can expect to
see the results. This is especially important when mixing WC with GTT as
the physical paths are different and cachelines are not naturally flushed.
Testc
== Series Details ==
Series: drm/i915/selftests: Detect unknown swizzling correctly
URL : https://patchwork.freedesktop.org/series/46026/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4438_full -> Patchwork_9553_full =
== Summary - WARNING ==
Minor unknown changes coming
== Series Details ==
Series: series starting with [1/2] drm/i915: Record logical context support in
driver caps (rev2)
URL : https://patchwork.freedesktop.org/series/46065/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_ -> Patchwork_9563 =
== Summary - SUCCESS ==
No
== Series Details ==
Series: series starting with [1/6] drm/i915: Refactor export_fence() after
i915_vma_move_to_active()
URL : https://patchwork.freedesktop.org/series/46067/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
478556e2da64 drm/i915: Refactor export_fence() after i9
== Series Details ==
Series: series starting with [1/6] drm/i915: Refactor export_fence() after
i915_vma_move_to_active()
URL : https://patchwork.freedesktop.org/series/46067/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Commit: drm/i915: Refactor export_fence() after i915_vma_mo
On 6 July 2018 at 12:54, Chris Wilson wrote:
> If we have just completed a WC write, we must ensure that the WCB (Write
> Combining Buffer) is flushed out to main memory before we can expect to
> see the results. This is especially important when mixing WC with GTT as
> the physical paths are diff
On 6 July 2018 at 12:45, Chris Wilson wrote:
> If the GPU is irrecoverably wedged, we can not execute any requests
> making testing execlists (request execution) pointless. Skip!
>
> Signed-off-by: Chris Wilson
> Cc: Matthew Auld
Reviewed-by: Matthew Auld
___
== Series Details ==
Series: series starting with [1/6] drm/i915: Refactor export_fence() after
i915_vma_move_to_active()
URL : https://patchwork.freedesktop.org/series/46067/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_ -> Patchwork_9564 =
== Summary - FAILURE ==
On Fri, 2018-06-01 at 20:00 +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Another version of these cleansup. Last time there was some kind of
> smtp
> fail when sending and patchwork got confused at the wonky threading.
> So
> best to repost fully I thought. Additionally I had to resolve
Quoting Patchwork (2018-07-06 13:20:19)
> == Series Details ==
>
> Series: series starting with [1/6] drm/i915: Refactor export_fence() after
> i915_vma_move_to_active()
> URL : https://patchwork.freedesktop.org/series/46067/
> State : failure
>
> == Summary ==
>
> = CI Bug Log - changes from
As we propagate back the error to the caller for them to handle, we do
not need the lowest level spitting out a redundant warning upon an
allocation failure inside dma_map_page().
Signed-off-by: Chris Wilson
Cc: Matthew Auld
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 12
1 file chan
We already maually control the CPU cache for our page table directories,
so we can tell the dma mapper to skip doing it as well.
Signed-off-by: Chris Wilson
Cc: Matthew Auld
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_g
Using a VMA on more than one timeline concurrently is the exception
rather than the rule (using it concurrently on multiple engines). As we
expect to only use one active tracker, store the most recently used
tracker inside the i915_vma itself and only fallback to the rbtree if
we need a second or m
On 06/07/2018 12:54, Chris Wilson wrote:
If we have just completed a WC write, we must ensure that the WCB (Write
Combining Buffer) is flushed out to main memory before we can expect to
see the results. This is especially important when mixing WC with GTT as
the physical paths are different and
On 6 July 2018 at 13:26, Chris Wilson wrote:
> As we propagate back the error to the caller for them to handle, we do
> not need the lowest level spitting out a redundant warning upon an
> allocation failure inside dma_map_page().
>
> Signed-off-by: Chris Wilson
> Cc: Matthew Auld
Reviewed-by: M
== Series Details ==
Series: drm/i915/selftests: Skip live_execlists if the GPU is terminally wedged
URL : https://patchwork.freedesktop.org/series/46069/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_ -> Patchwork_9565 =
== Summary - SUCCESS ==
No regressions found.
== Series Details ==
Series: drm/i915: Flush the WCB following a WC write
URL : https://patchwork.freedesktop.org/series/46070/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
1442df0746fe drm/i915: Flush the WCB following a WC write
-:25: WARNING:MEMORY_BARRIER: memory barrier w
== Series Details ==
Series: drm/i915/selftests: Destroy partial tiling vma after use
URL : https://patchwork.freedesktop.org/series/46031/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4438_full -> Patchwork_9554_full =
== Summary - FAILURE ==
Serious unknown changes co
Limit the GTT size we try and allocate to ensure that it fits within RAM
and does not trigger the oomkiller indiscriminately.
Signed-off-by: Chris Wilson
Cc: Matthew Auld
---
drivers/gpu/drm/i915/selftests/i915_gem_gtt.c | 20 ---
1 file changed, 13 insertions(+), 7 deletions(-)
On 6 July 2018 at 13:26, Chris Wilson wrote:
> We already maually control the CPU cache for our page table directories,
> so we can tell the dma mapper to skip doing it as well.
>
> Signed-off-by: Chris Wilson
> Cc: Matthew Auld
Reviewed-by: Matthew Auld
== Series Details ==
Series: drm/i915: Flush the WCB following a WC write
URL : https://patchwork.freedesktop.org/series/46070/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_ -> Patchwork_9566 =
== Summary - SUCCESS ==
No regressions found.
External URL:
https://
On Thu, Jul 05, 2018 at 04:33:50PM +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915/ddi: Simplify get_encoder_power_domains()
> URL : https://patchwork.freedesktop.org/series/45980/
> State : success
Pushed to -dinq, thanks for the review.
>
> == Summary ==
>
> = CI Bug Lo
Op 18-04-18 om 17:37 schreef Ville Syrjälä:
> On Wed, Apr 18, 2018 at 02:51:21PM +0200, Maarten Lankhorst wrote:
>> This reduces the amount of spam when you debug a CRC reading
>> program.
>>
>> Signed-off-by: Maarten Lankhorst
>> ---
>> drivers/gpu/drm/drm_debugfs_crc.c | 9 -
>> include
On 6 July 2018 at 13:53, Chris Wilson wrote:
> Limit the GTT size we try and allocate to ensure that it fits within RAM
> and does not trigger the oomkiller indiscriminately.
>
> Signed-off-by: Chris Wilson
> Cc: Matthew Auld
Reviewed-by: Matthew Auld
___
On Fri, 29 Jun 2018, Hans de Goede wrote:
> Hi,
>
> On 19-06-18 22:18, Hans de Goede wrote:
>> Hi All,
>>
>> This patch-set is the result of the work I've been doing recently to
>> give people a smooth "flickerfree" boot experience where the display
>> keeps displaying the logo put there by the f
On 06/07/2018 07:53, Chris Wilson wrote:
In the next patch, we will want a third distinct class of timeline that
may overlap with the current pair of client and engine timeline classes.
Rather than use the ad hoc markup of SINGLE_DEPTH_NESTING, initialise
the different timeline classes with an e
On 04/07/2018 15:38, Chris Wilson wrote:
Mark up gem_pwrite_pread's dependence on a functioning GPU, by calling
igt_require_gem in its setup fixture.
Signed-off-by: Chris Wilson
---
tests/gem_pwrite_pread.c | 1 +
tests/gem_tiled_partial_pwrite_pread.c | 1 +
2 files changed,
== Series Details ==
Series: drm/i915/psr: Split sink status into a separate debugfs node
URL : https://patchwork.freedesktop.org/series/45952/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4438_full -> Patchwork_9555_full =
== Summary - WARNING ==
Minor unknown changes
On Tue, Jun 19, 2018 at 10:18:27PM +0200, Hans de Goede wrote:
> On BYT and CHT the GOP sometimes initializes the pclk at a (slightly)
> different frequency then the pclk which we've calculated.
>
> This commit makes the DSI code read-back the pclk set by the GOP and
> if that is within a reasonab
Replace the magic bit with the proper symbolic name for instructing
MI_STORE_DWORD_IMM to use a virtual address (on gen3) or the global GTT
address (still virtual!) on gen4+.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/selftests/huge_pages.c | 4 ++--
drivers/gpu/drm/i915/selfte
We always want to use a virtual address (i.e. use the GTT) for
MI_STORE_DWORD_IMM, but forgot the ever so important flag in
live_hangcheck for gen3.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/selftests/intel_hangcheck.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
== Series Details ==
Series: series starting with [1/3] drm: Extract __setplane_check() (rev3)
URL : https://patchwork.freedesktop.org/series/45589/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4438_full -> Patchwork_9557_full =
== Summary - WARNING ==
Minor unknown cha
The caching of SGT's is actually quite harmful and should probably removed
altogether when all drivers are audited.
Start by providing a separate DMA-buf export implementation in amdgpu. This is
also a prerequisite of unpinned DMA-buf handling.
v2: fix unintended recursion, remove debugging lefto
To allow a smooth transition from pinning buffer objects to dynamic
invalidation we first start to cache the sg_table for an attachment
unless the driver explicitly says to not do so.
Signed-off-by: Christian König
---
drivers/dma-buf/dma-buf.c | 24
include/linux/dma-bu
Instead of relying on the DRM functions just implement our own import
functions. This prepares support for taking care of unpinned DMA-buf.
v2: enable for all exporters, not just amdgpu, fix invalidation
handling, lock reservation object while setting callback
v3: change to new dma_buf attach
That is now done by the DMA-buf helpers instead.
Signed-off-by: Christian König
---
drivers/gpu/drm/drm_prime.c | 78 +++--
1 file changed, 18 insertions(+), 60 deletions(-)
diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c
index 186d
Next try of prework for unpinned DMA-buf operation.
Only send to intel-gfx to trigger unit tests on the following patches.
Christian.
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Add function variants which can be called with the reservation lock
already held.
v2: reordered, add lockdep asserts, fix kerneldoc
v3: rebased on sgt caching
Signed-off-by: Christian König
---
drivers/dma-buf/dma-buf.c | 63 +++
include/linux/dma-buf
First step towards unpinned DMA buf operation.
I've checked the DRM drivers to potential locking of the reservation
object, but essentially we need to audit all implementations of the
dma_buf _ops for this to work.
v2: reordered
v3: rebased on sgt caching
Signed-off-by: Christian König
---
dri
On Fri, Jul 06, 2018 at 03:23:22PM +0100, Chris Wilson wrote:
> Replace the magic bit with the proper symbolic name for instructing
> MI_STORE_DWORD_IMM to use a virtual address (on gen3) or the global GTT
> address (still virtual!) on gen4+.
>
> Signed-off-by: Chris Wilson
> ---
> drivers/gpu/d
On Fri, Jul 06, 2018 at 03:23:23PM +0100, Chris Wilson wrote:
> We always want to use a virtual address (i.e. use the GTT) for
> MI_STORE_DWORD_IMM, but forgot the ever so important flag in
> live_hangcheck for gen3.
Reviewed-by: Ville Syrjälä
>
> Signed-off-by: Chris Wilson
> ---
> drivers/g
Quoting Ville Syrjälä (2018-07-06 15:47:25)
> On Fri, Jul 06, 2018 at 03:23:22PM +0100, Chris Wilson wrote:
> > diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_coherency.c
> > b/drivers/gpu/drm/i915/selftests/i915_gem_coherency.c
> > index cb9eef1635e1..294c58aba2c1 100644
> > --- a/drivers/g
== Series Details ==
Series: series starting with [1/2] drm/i915/gtt: Suppress warnings for
dma_map_page
URL : https://patchwork.freedesktop.org/series/46072/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4446 -> Patchwork_9567 =
== Summary - SUCCESS ==
No regressions f
== Series Details ==
Series: series starting with [1/6] drm/i915: Refactor export_fence() after
i915_vma_move_to_active() (rev2)
URL : https://patchwork.freedesktop.org/series/46067/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
c404016bcc1d drm/i915: Refactor export_fence() a
== Series Details ==
Series: series starting with [1/6] drm/i915: Refactor export_fence() after
i915_vma_move_to_active() (rev2)
URL : https://patchwork.freedesktop.org/series/46067/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Commit: drm/i915: Refactor export_fence() after i915
== Series Details ==
Series: series starting with [1/6] drm/i915: Refactor export_fence() after
i915_vma_move_to_active() (rev2)
URL : https://patchwork.freedesktop.org/series/46067/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4446 -> Patchwork_9568 =
== Summary - SUCCES
== Series Details ==
Series: drm/i915/selftests: Limit live_gtt allocation test to fit within RAM
URL : https://patchwork.freedesktop.org/series/46075/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Commit: drm/i915/selftests: Limit live_gtt allocation test to fit within RAM
+driver
== Series Details ==
Series: drm/i915/selftests: Limit live_gtt allocation test to fit within RAM
URL : https://patchwork.freedesktop.org/series/46075/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4446 -> Patchwork_9569 =
== Summary - SUCCESS ==
No regressions found.
The patch adds support of preempt-to-idle requesting by setting a proper
bit within Execlist Control Register, and receiving preemption result from
Context Status Buffer.
Preemption in previous gens required a special batch buffer to be executed,
so the Command Streamer never preempted to idle dir
== Series Details ==
Series: series starting with [1/6] dma-buf: add caching of sg_table
URL : https://patchwork.freedesktop.org/series/46080/
State : failure
== Summary ==
Applying: dma-buf: add caching of sg_table
Applying: drm: remove prime sg_table caching
Applying: dma-buf: add dma_buf_(u
== Series Details ==
Series: series starting with [1/2] drm/i915/selftests: Replace magic 1<<22 with
MI_USE_GGTT/MI_MEM_VIRTUAL
URL : https://patchwork.freedesktop.org/series/46078/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4446 -> Patchwork_9570 =
== Summary - SUCCESS
== Series Details ==
Series: drm/i915/gen11: Preempt-to-idle support in execlists. (rev5)
URL : https://patchwork.freedesktop.org/series/40747/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
90f487fed124 drm/i915/gen11: Preempt-to-idle support in execlists.
-:141: CHECK:COMPARIS
1 - 100 of 149 matches
Mail list logo
