[Intel-gfx] [PATCH] drm/i915: Add relocation exceptions for two other platforms

2021-06-16 Thread Zbigniew Kempczyński
For topic/intel-for-CI branch only. We have established previously we stop using relocations starting from gen12 platforms with Tigerlake as an exception. We keep this statement but we want to enable relocations conditionally for Alderlake S+P under require_force_probe flag set. Keeping

[Intel-gfx] ✓ Fi.CI.BAT: success for Explicity steer l3bank multicast reads when necessary (rev5)

2021-06-16 Thread Patchwork
== Series Details == Series: Explicity steer l3bank multicast reads when necessary (rev5) URL : https://patchwork.freedesktop.org/series/91485/ State : success == Summary == CI Bug Log - changes from CI_DRM_10233 -> Patchwork_20394 Summary

[Intel-gfx] linux-next: manual merge of the drm tree with the drm-misc-fixes tree

2021-06-16 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the drm tree got a conflict in: drivers/gpu/drm/vc4/vc4_hdmi.c between commit: 9984d6664ce9 ("drm/vc4: hdmi: Make sure the controller is powered in detect") from the drm-misc-fixes tree and commit: 6800234ceee0 ("drm/vc4: hdmi: Convert to gpiod")

Re: [Intel-gfx] [PATCH 2/3] drm/i915/guc: Update firmware to v62.0.0

2021-06-16 Thread Daniele Ceraolo Spurio
On 6/15/2021 5:13 PM, Matthew Brost wrote: From: Michal Wajdeczko Most of the changes to the 62.0.0 firmware revolved around CTB communication channel. Conform to the new (stable) CTB protocol. v2: (Michal) Add values back to kernel DOC for actions (Docs) Add 'CT buffer' back in

Re: [Intel-gfx] [PATCH 8/8] drm/i915: Move submission tasklet to i915_sched_engine

2021-06-16 Thread Daniele Ceraolo Spurio
On 6/15/2021 3:43 PM, Matthew Brost wrote: The submission tasklet operates on i915_sched_engine, thus it is the correct place for it. v3: (Jason Ekstrand) Change sched_engine->engine to a void* private data pointer Add kernel doc v4: (Daniele) Update private_data comment Set

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Explicity steer l3bank multicast reads when necessary (rev5)

2021-06-16 Thread Patchwork
== Series Details == Series: Explicity steer l3bank multicast reads when necessary (rev5) URL : https://patchwork.freedesktop.org/series/91485/ State : warning == Summary == $ dim checkpatch origin/drm-tip 311242e9e0f0 drm/i915: extract steered reg access to common function -:90:

[Intel-gfx] [PATCH v3 3/3] drm/i915: Add support for explicit L3BANK steering

2021-06-16 Thread Matt Roper
Because Render Power Gating restricts us to just a single subslice as a valid steering target for reads of multicast registers in a SUBSLICE range, the default steering we setup at init may not lead to a suitable target for L3BANK multicast register. In cases where it does not, use explicit

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/6] drm/i915/display/psr: Handle SU Y granularity

2021-06-16 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915/display/psr: Handle SU Y granularity URL : https://patchwork.freedesktop.org/series/91594/ State : failure == Summary == CI Bug Log - changes from CI_DRM_10233_full -> Patchwork_20393_full

[Intel-gfx] ✗ Fi.CI.IGT: failure for Explicity steer l3bank multicast reads when necessary (rev4)

2021-06-16 Thread Patchwork
== Series Details == Series: Explicity steer l3bank multicast reads when necessary (rev4) URL : https://patchwork.freedesktop.org/series/91485/ State : failure == Summary == CI Bug Log - changes from CI_DRM_10230_full -> Patchwork_20391_full

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: return DRIVER_NAME for the fence driver name

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915: return DRIVER_NAME for the fence driver name URL : https://patchwork.freedesktop.org/series/91573/ State : success == Summary == CI Bug Log - changes from CI_DRM_10230_full -> Patchwork_20388_full

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/gt: Fix duplicate included intel_region_lmem.h

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915/gt: Fix duplicate included intel_region_lmem.h URL : https://patchwork.freedesktop.org/series/91571/ State : success == Summary == CI Bug Log - changes from CI_DRM_10230_full -> Patchwork_20387_full

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/6] drm/i915/display/psr: Handle SU Y granularity

2021-06-16 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915/display/psr: Handle SU Y granularity URL : https://patchwork.freedesktop.org/series/91594/ State : success == Summary == CI Bug Log - changes from CI_DRM_10233 -> Patchwork_20393

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/6] drm/i915/display/psr: Handle SU Y granularity

2021-06-16 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915/display/psr: Handle SU Y granularity URL : https://patchwork.freedesktop.org/series/91594/ State : warning == Summary == $ dim checkpatch origin/drm-tip 47efae2a069f drm/i915/display/psr: Handle SU Y granularity a03bc073ba13

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/dp_mst: Add missing drm parameters to recently added call to drm_dbg_kms()

2021-06-16 Thread Patchwork
== Series Details == Series: drm/dp_mst: Add missing drm parameters to recently added call to drm_dbg_kms() URL : https://patchwork.freedesktop.org/series/91592/ State : failure == Summary == Applying: drm/dp_mst: Add missing drm parameters to recently added call to drm_dbg_kms() Using

[Intel-gfx] [PATCH 4/6] drm/i915/xelpd: Handle PSR2 SDP indication in the prior scanline

2021-06-16 Thread José Roberto de Souza
In some modes there is not enough time during hblank to transmit PSR2 SDP plus the pixels CRC SDP, if such case happens PSR2 needs to be disabled. But eDP spec 1.4b allows to transmit PSR2 SDP in a prior scanline alone and than later the CRC SDP, allowing PSR2 to be enabled in those hblank

[Intel-gfx] [PATCH 6/6] drm/i915/display/adl_p: Implement PSR changes

2021-06-16 Thread José Roberto de Souza
Implements changes around PSR for alderlake-P: - EDP_SU_TRACK_ENABLE was removed and bit 30 now has other function - Some bits of PSR2_MAN_TRK_CTL moved and SF_PARTIAL_FRAME_UPDATE was removed setting SU_REGION_START/END_ADDR will do this job - SU_REGION_START/END_ADDR have now line granularity

[Intel-gfx] [PATCH 5/6] drm/i915/display/adl_p: Implement Wa_16011303918

2021-06-16 Thread José Roberto de Souza
PSR2 is not compatible with DC3CO or VRR in this stepping, so not enabling PSR2 if VRR will be enabled or not enabling DC3CO if PSR2 is possible. BSpec: 54369 Cc: Gwan-gyeong Mun Cc: Matt Atwood Signed-off-by: José Roberto de Souza --- drivers/gpu/drm/i915/display/intel_psr.c | 12

[Intel-gfx] [PATCH 1/6] drm/i915/display/psr: Handle SU Y granularity

2021-06-16 Thread José Roberto de Souza
We were only handling X and width granularity, what was causing issues when sink had a granularity different than 4. While at it, renaming su_x_granularity to su_w_granularity to better match reality. Cc: Gwan-gyeong Mun Signed-off-by: José Roberto de Souza ---

[Intel-gfx] [PATCH 3/6] drm/i915/display/adl_p: Implement Wa_16011168373

2021-06-16 Thread José Roberto de Souza
Another WA that is required for PSR2. BSpec: 54369 Cc: Gwan-gyeong Mun Cc: Matt Atwood Signed-off-by: José Roberto de Souza --- drivers/gpu/drm/i915/display/intel_psr.c | 15 +++ drivers/gpu/drm/i915/i915_reg.h | 8 2 files changed, 23 insertions(+) diff --git

[Intel-gfx] [PATCH 2/6] drm/i915/display/adl_p: Implement Wa_22012278275

2021-06-16 Thread José Roberto de Souza
The PSR2_CTL io buffer wake and fast wake values do not match expected in pre production hardware, so here adding a table that matches with HW to program it with values that HW expect. Cc: Gwan-gyeong Mun Signed-off-by: José Roberto de Souza --- drivers/gpu/drm/i915/display/intel_psr.c | 29

Re: [Intel-gfx] [PATCH 08/10] vfio/mtty: Convert to use vfio_register_group_dev()

2021-06-16 Thread Kirti Wankhede
-static int mtty_reset(struct mdev_device *mdev) +static int mtty_reset(struct mdev_state *mdev_stte) Nit pick: s/mdev_stte/mdev_state +static const struct vfio_device_ops mtty_dev_ops = { + .name = "vfio-mdev", I think name should be different that 'vfio-mdev', probably

Re: [Intel-gfx] [PATCH 07/10] vfio/mdev: Allow the mdev_parent_ops to specify the device driver to bind

2021-06-16 Thread Kirti Wankhede
On 6/15/2021 7:05 PM, Christoph Hellwig wrote: From: Jason Gunthorpe This allows a mdev driver to opt out of using vfio_mdev.c, instead the driver will provide a 'struct mdev_driver' and register directly with the driver core. Much of mdev_parent_ops becomes unused in this mode: -

Re: [Intel-gfx] [PATCH 06/10] vfio/mdev: Remove CONFIG_VFIO_MDEV_DEVICE

2021-06-16 Thread Kirti Wankhede
On 6/15/2021 7:05 PM, Christoph Hellwig wrote: From: Jason Gunthorpe For some reason the vfio_mdev shim mdev_driver has its own module and kconfig. As the next patch requires access to it from mdev.ko merge the two modules together and remove VFIO_MDEV_DEVICE. A later patch deletes this

Re: [Intel-gfx] [PATCH 02/10] driver core: Better distinguish probe errors in really_probe

2021-06-16 Thread Kirti Wankhede
On 6/15/2021 7:05 PM, Christoph Hellwig wrote: really_probe tries to special case errors from ->probe, but due to all other initialization added to the function over time now a lot of internal errors hit that code path as well. Untangle that by adding a new probe_err local variable and apply

Re: [Intel-gfx] [PATCH 29/31] drm/i915/gem: Roll all of context creation together

2021-06-16 Thread kernel test robot
Hi Jason, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm-intel/for-linux-next] [also build test WARNING on drm-tip/drm-tip drm-exynos/exynos-drm-next next-20210616] [cannot apply to tegra-drm/drm/tegra/for-next drm/drm-next v5.13-rc6] [If your patch

[Intel-gfx] ✓ Fi.CI.BAT: success for Explicity steer l3bank multicast reads when necessary (rev4)

2021-06-16 Thread Patchwork
== Series Details == Series: Explicity steer l3bank multicast reads when necessary (rev4) URL : https://patchwork.freedesktop.org/series/91485/ State : success == Summary == CI Bug Log - changes from CI_DRM_10230 -> Patchwork_20391 Summary

Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Force a TypeC PHY disconnect during suspend/shutdown

2021-06-16 Thread Vudum, Lakshminarayana
Re-reported. -Original Message- From: Deak, Imre Sent: Wednesday, June 16, 2021 9:20 AM To: intel-gfx@lists.freedesktop.org; Chris Chiu ; Shankar, Uma ; Vudum, Lakshminarayana Subject: Re: ✗ Fi.CI.IGT: failure for drm/i915: Force a TypeC PHY disconnect during suspend/shutdown On

Re: [Intel-gfx] [PATCH] drm/dp_mst: Add missing drm parameters to recently added call to drm_dbg_kms()

2021-06-16 Thread Lyude Paul
Reviewed-by: Lyude Paul Will go ahead and push this to drm-misc-next-fixes, thanks On Wed, 2021-06-16 at 12:44 -0700, José Roberto de Souza wrote: > Commit 3769e4c0af5b ("drm/dp_mst: Avoid to mess up payload table by > ports in stale topology") added to calls to drm_dbg_kms() but it > missed

[Intel-gfx] [PATCH] drm/dp_mst: Add missing drm parameters to recently added call to drm_dbg_kms()

2021-06-16 Thread José Roberto de Souza
Commit 3769e4c0af5b ("drm/dp_mst: Avoid to mess up payload table by ports in stale topology") added to calls to drm_dbg_kms() but it missed the first parameter, the drm device breaking the build. Fixes: 3769e4c0af5b ("drm/dp_mst: Avoid to mess up payload table by ports in stale topology") Cc:

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Explicity steer l3bank multicast reads when necessary (rev4)

2021-06-16 Thread Patchwork
== Series Details == Series: Explicity steer l3bank multicast reads when necessary (rev4) URL : https://patchwork.freedesktop.org/series/91485/ State : warning == Summary == $ dim checkpatch origin/drm-tip 4c22a61b7efa drm/i915: extract steered reg access to common function -:90:

[Intel-gfx] ✗ Fi.CI.BUILD: failure for series starting with [v5,1/7] drm/i915/ttm: add ttm_buddy_man

2021-06-16 Thread Patchwork
== Series Details == Series: series starting with [v5,1/7] drm/i915/ttm: add ttm_buddy_man URL : https://patchwork.freedesktop.org/series/91589/ State : failure == Summary == Applying: drm/i915/ttm: add ttm_buddy_man .git/rebase-apply/patch:845: new blank line at EOF. + warning: 1 line adds

[Intel-gfx] ✗ Fi.CI.BUILD: failure for drm/i915/ttm: Calculate the object placement at get_pages time (rev2)

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915/ttm: Calculate the object placement at get_pages time (rev2) URL : https://patchwork.freedesktop.org/series/91586/ State : failure == Summary == Applying: drm/i915/ttm: Calculate the object placement at get_pages time Using index info to reconstruct a

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: return DRIVER_NAME for the fence driver name

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915: return DRIVER_NAME for the fence driver name URL : https://patchwork.freedesktop.org/series/91573/ State : success == Summary == CI Bug Log - changes from CI_DRM_10230 -> Patchwork_20388 Summary

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gt: Fix duplicate included intel_region_lmem.h

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915/gt: Fix duplicate included intel_region_lmem.h URL : https://patchwork.freedesktop.org/series/91571/ State : success == Summary == CI Bug Log - changes from CI_DRM_10230 -> Patchwork_20387 Summary

[Intel-gfx] [PULL] drm-misc-next-fixes

2021-06-16 Thread Thomas Zimmermann
Hi Dave and Daniel, here's this week's PR for drm-misc-next-fixes. Best regards Thomas drm-misc-next-fixes-2021-06-16: Short summary of fixes pull: * hyperv: advertise the correct formatmodifiers for its primary plane * dp_mst: VCPI fixes to make it work with StarTech hub The following

Re: [Intel-gfx] [PATCH 5/5] DONOTMERGE: dma-buf: Get rid of dma_fence_get_rcu_safe

2021-06-16 Thread kernel test robot
Hi Jason, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm-intel/for-linux-next] [also build test WARNING on drm-tip/drm-tip drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next linus/master v5.13-rc6 next-20210616] [cannot apply to drm/drm-next

[Intel-gfx] [PATCH v3 2/3] drm/i915: Add GT support for multiple types of multicast steering

2021-06-16 Thread Matt Roper
Although most of our multicast registers are replicated per-subslice, we also have a small number of multicast registers that are replicated per-l3 bank instead. For both types of multicast registers we need to make sure we steer reads of these registers to a valid instance. Ideally we'd like to

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Force a TypeC PHY disconnect during suspend/shutdown

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915: Force a TypeC PHY disconnect during suspend/shutdown URL : https://patchwork.freedesktop.org/series/91345/ State : success == Summary == CI Bug Log - changes from CI_DRM_10205_full -> Patchwork_20334_full

Re: [Intel-gfx] [PATCH v2.1 2/3] drm/i915: Add GT support for multiple types of multicast steering

2021-06-16 Thread Matt Roper
On Wed, Jun 16, 2021 at 10:24:48AM -0400, Rodrigo Vivi wrote: > On Tue, Jun 15, 2021 at 05:42:12PM -0700, Matt Roper wrote: > > Although most of our multicast registers are replicated per-subslice, we > > also have a small number of multicast registers that are replicated > > per-l3 bank instead.

Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Force a TypeC PHY disconnect during suspend/shutdown

2021-06-16 Thread Imre Deak
On Thu, Jun 10, 2021 at 09:28:31PM +, Patchwork wrote: > == Series Details == > > Series: drm/i915: Force a TypeC PHY disconnect during suspend/shutdown > URL : https://patchwork.freedesktop.org/series/91345/ > State : failure Thanks for the report, testing and review. Pushed to

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/gem: Remove duplicated call to ops->pread

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915/gem: Remove duplicated call to ops->pread URL : https://patchwork.freedesktop.org/series/91561/ State : success == Summary == CI Bug Log - changes from CI_DRM_10226_full -> Patchwork_20383_full Summary

Re: [Intel-gfx] [PATCH 06/31] drm/i915: Drop the CONTEXT_CLONE API (v2)

2021-06-16 Thread kernel test robot
to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Jason-Ekstrand/drm-i915-gem-ioctl-clean-ups-v6/20210616-151016 base: git

[Intel-gfx] [PATCH v5 4/7] drm/i915/ttm: pass along the I915_BO_ALLOC_CONTIGUOUS

2021-06-16 Thread Matthew Auld
Currently we just ignore the I915_BO_ALLOC_CONTIGUOUS flag, which is fine since everything is already contiguous with the ttm range manager. However in the next patch we want to switch over to the ttm buddy manager, where allocations are by default not contiguous. v2(Thomas): - Forward

[Intel-gfx] [PATCH v5 6/7] drm/i915/ttm: switch over to ttm_buddy_man

2021-06-16 Thread Matthew Auld
Move back to the buddy allocator for managing device local memory, and restore the lost mock selftests. Keep around the range manager related bits, since we likely need this for managing stolen at some point. For stolen we also don't need to reserve anything so no need to support a generic reserve

[Intel-gfx] [PATCH v5 7/7] drm/i915/ttm: restore min_page_size behaviour

2021-06-16 Thread Matthew Auld
We now have bo->page_alignment which perfectly describes what we need if we have min page size restrictions for lmem. We can also drop the flag here, since this is the default behaviour for all objects. v2(Thomas): - bo->page_alignment is in page units Signed-off-by: Matthew Auld Cc: Thomas

[Intel-gfx] [PATCH v5 3/7] drm/i915/ttm: Calculate the object placement at get_pages time

2021-06-16 Thread Matthew Auld
From: Thomas Hellström Instead of relying on a static placement, calculate at get_pages() time. This should work for LMEM regions and system for now. For stolen we need to take preallocated range into account. That will if needed be added later. Signed-off-by: Thomas Hellström Reviewed-by:

[Intel-gfx] [PATCH v5 2/7] drm/i915/ttm: add i915_sg_from_buddy_resource

2021-06-16 Thread Matthew Auld
We need to be able to build an sg table from our list of buddy blocks, so that we can later plug this into our ttm backend, and replace our use of the range manager. Signed-off-by: Matthew Auld Cc: Thomas Hellström Reviewed-by: Thomas Hellström --- drivers/gpu/drm/i915/i915_scatterlist.c | 80

[Intel-gfx] [PATCH v5 5/7] drm/i915/ttm: remove node usage in our naming

2021-06-16 Thread Matthew Auld
Now that ttm_resource_manager just returns a generic ttm_resource we don't need to reference the mm_node stuff anymore which mostly only makes sense for drm_mm_node. In the next few patches we want switch over to the ttm_buddy_man which is just another type of ttm_resource so reflect that in the

[Intel-gfx] [PATCH v5 1/7] drm/i915/ttm: add ttm_buddy_man

2021-06-16 Thread Matthew Auld
Add back our standalone i915_buddy allocator and integrate it into a ttm_resource_manager. This will plug into our ttm backend for managing device local-memory in the next couple of patches. v2(Thomas): - Return -ENOSPC from the buddy; ttm expects this in order to trigger eviction -

[Intel-gfx] [PATCH] drm/i915/ttm: Calculate the object placement at get_pages time

2021-06-16 Thread Thomas Hellström
Instead of relying on a static placement, calculate at get_pages() time. This should work for LMEM regions and system for now. For stolen we need to take preallocated range into account. That will if needed be added later. Signed-off-by: Thomas Hellström Reviewed-by: Matthew Auld --- v2: -

Re: [Intel-gfx] [PATCH 0/3] Update firmware to v62.0.0

2021-06-16 Thread Mike Lothian
Hi Is there a place where we can download these new firmware images? Cheers Mike On Wed, 16 Jun 2021 at 00:55, Matthew Brost wrote: > As part of enabling GuC submission [1] we need to update to the latest > and greatest firmware. This series does that. All backwards > compatibility breaking

[Intel-gfx] [PATCH] drm/i915/ttm: Calculate the object placement at get_pages time

2021-06-16 Thread Thomas Hellström
Instead of relying on a static placement, calculate at get_pages() time. This should work for LMEM regions and system for now. For stolen we need to take preallocated range into account. That will if needed be added later. Signed-off-by: Thomas Hellström Reviewed-by: Matthew Auld --- v2: -

[Intel-gfx] [PATCH v4 7/7] drm/i915/ttm: restore min_page_size behaviour

2021-06-16 Thread Matthew Auld
We now have bo->page_alignment which perfectly describes what we need if we have min page size restrictions for lmem. We can also drop the flag here, since this is the default behaviour for all objects. v2(Thomas): - bo->page_alignment is in page units Signed-off-by: Matthew Auld Cc: Thomas

[Intel-gfx] [PATCH v4 6/7] drm/i915/ttm: switch over to ttm_buddy_man

2021-06-16 Thread Matthew Auld
Move back to the buddy allocator for managing device local memory, and restore the lost mock selftests. Keep around the range manager related bits, since we likely need this for managing stolen at some point. For stolen we also don't need to reserve anything so no need to support a generic reserve

[Intel-gfx] [PATCH v4 5/7] drm/i915/ttm: remove node usage in our naming

2021-06-16 Thread Matthew Auld
Now that ttm_resource_manager just returns a generic ttm_resource we don't need to reference the mm_node stuff anymore which mostly only makes sense for drm_mm_node. In the next few patches we want switch over to the ttm_buddy_man which is just another type of ttm_resource so reflect that in the

[Intel-gfx] [PATCH v4 3/7] drm/i915/ttm: Calculate the object placement at get_pages time

2021-06-16 Thread Matthew Auld
From: Thomas Hellström Instead of relying on a static placement, calculate at get_pages() time. This should work for LMEM regions and system for now. For stolen we need to take preallocated range into account. That well be added later. Instead of relying on a static placement, calculate at

[Intel-gfx] [PATCH v4 4/7] drm/i915/ttm: pass along the I915_BO_ALLOC_CONTIGUOUS

2021-06-16 Thread Matthew Auld
Currently we just ignore the I915_BO_ALLOC_CONTIGUOUS flag, which is fine since everything is already contiguous with the ttm range manager. However in the next patch we want to switch over to the ttm buddy manager, where allocations are by default not contiguous. v2(Thomas): - Forward

[Intel-gfx] [PATCH v4 2/7] drm/i915/ttm: add i915_sg_from_buddy_resource

2021-06-16 Thread Matthew Auld
We need to be able to build an sg table from our list of buddy blocks, so that we can later plug this into our ttm backend, and replace our use of the range manager. Signed-off-by: Matthew Auld Cc: Thomas Hellström Reviewed-by: Thomas Hellström --- drivers/gpu/drm/i915/i915_scatterlist.c | 80

[Intel-gfx] [PATCH v4 1/7] drm/i915/ttm: add ttm_buddy_man

2021-06-16 Thread Matthew Auld
Add back our standalone i915_buddy allocator and integrate it into a ttm_resource_manager. This will plug into our ttm backend for managing device local-memory in the next couple of patches. v2(Thomas): - Return -ENOSPC from the buddy; ttm expects this in order to trigger eviction -

Re: [Intel-gfx] [PATCH 1/3] drm/i915/guc: Add fetch of hwconfig table

2021-06-16 Thread kernel test robot
to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/John-C-Harrison-Intel-com/Add-support-for-querying-hw-info-that-UMDs-need/20210616-141805 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: x86_64

Re: [Intel-gfx] [PATCH v2.1 3/3] drm/i915: Add support for explicit L3BANK steering

2021-06-16 Thread Rodrigo Vivi
On Tue, Jun 15, 2021 at 05:42:13PM -0700, Matt Roper wrote: > Because Render Power Gating restricts us to just a single subslice as a > valid steering target for reads of multicast registers in a SUBSLICE > range, the default steering we setup at init may not lead to a suitable > target for L3BANK

Re: [Intel-gfx] [PATCH] drm/i915: allow DG1 autoprobe for CONFIG_BROKEN

2021-06-16 Thread Matthew Auld
On Mon, 14 Jun 2021 at 10:22, Matthew Auld wrote: > > Purely for CI so we can get some pre-merge results for DG1. This is > especially useful for cross driver TTM changes where CI can hopefully > catch regressions. This is similar to how we already handle the DG1 > specific uAPI, which are also

Re: [Intel-gfx] [PATCH v2.1 2/3] drm/i915: Add GT support for multiple types of multicast steering

2021-06-16 Thread Rodrigo Vivi
On Tue, Jun 15, 2021 at 05:42:12PM -0700, Matt Roper wrote: > Although most of our multicast registers are replicated per-subslice, we > also have a small number of multicast registers that are replicated > per-l3 bank instead. For both types of multicast registers we need to > make sure we steer

Re: [Intel-gfx] [PATCH] drm/i915: Add relocation exceptions for two other platforms

2021-06-16 Thread Rodrigo Vivi
On Wed, Jun 16, 2021 at 11:48:02AM +0200, Zbigniew Kempczyński wrote: > For topic/intel-for-CI branch only. > > We have established previously we stop using relocations starting > from gen12 platforms with Tigerlake as an exception. We keep this > statement but we want to enable relocations

Re: [Intel-gfx] [PATCH] drm/i915: return DRIVER_NAME for the fence driver name

2021-06-16 Thread Daniel Vetter
On Wed, Jun 16, 2021 at 01:28:33PM +0100, Matthew Auld wrote: > The first tracepoint for a request is trace_dma_fence_init which is > called in the ctor before we have properly setup the request->engine. So > if it's a non-recycled request the rq->engine might be NULL, or some > garbage value,

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Be more gentle with exiting non-persistent context (rev6)

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915: Be more gentle with exiting non-persistent context (rev6) URL : https://patchwork.freedesktop.org/series/89644/ State : failure == Summary == CI Bug Log - changes from CI_DRM_10226_full -> Patchwork_20386_full

Re: [Intel-gfx] [PATCH 4/5] dma-buf: Stop using SLAB_TYPESAFE_BY_RCU in selftests

2021-06-16 Thread kernel test robot
-for-dma_fence/20210616-154432 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: sparc-randconfig-s032-20210615 (attached as .config) compiler: sparc-linux-gcc (GCC) 9.3.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin

[Intel-gfx] [PATCH] drm/i915: return DRIVER_NAME for the fence driver name

2021-06-16 Thread Matthew Auld
The first tracepoint for a request is trace_dma_fence_init which is called in the ctor before we have properly setup the request->engine. So if it's a non-recycled request the rq->engine might be NULL, or some garbage value, which leads to a crash. Since we are not permitted to use

Re: [Intel-gfx] [PATCH 07/10] vfio/mdev: Allow the mdev_parent_ops to specify the device driver to bind

2021-06-16 Thread Jason Gunthorpe
On Tue, Jun 15, 2021 at 04:11:29PM +0200, Greg Kroah-Hartman wrote: > On Tue, Jun 15, 2021 at 03:35:16PM +0200, Christoph Hellwig wrote: > > From: Jason Gunthorpe > > > > This allows a mdev driver to opt out of using vfio_mdev.c, instead the > > driver will provide a 'struct mdev_driver' and

[Intel-gfx] [PATCH] drm/i915/gt: Fix duplicate included intel_region_lmem.h

2021-06-16 Thread Jiapeng Chong
Clean up the following includecheck warning: ./drivers/gpu/drm/i915/gt/intel_region_lmem.c: intel_region_lmem.h is included more than once. No functional change. Reported-by: Abaci Robot Signed-off-by: Jiapeng Chong --- drivers/gpu/drm/i915/gt/intel_region_lmem.c | 1 - 1 file changed, 1

Re: [Intel-gfx] Allow mdev drivers to directly create the vfio_device (v3)

2021-06-16 Thread Jason Gunthorpe
On Tue, Jun 15, 2021 at 01:35:49PM -0600, Alex Williamson wrote: > On Tue, 15 Jun 2021 15:35:09 +0200 > Christoph Hellwig wrote: > > > This is my alternative take on this series from Jason: > > > > https://lore.kernel.org/dri-devel/87czsszi9i@redhat.com/T/ > > > > The mdev/vfio parts are

Re: [Intel-gfx] [PATCH v12 00/12] Restricted DMA

2021-06-16 Thread Will Deacon
Hi Claire, On Wed, Jun 16, 2021 at 02:21:45PM +0800, Claire Chang wrote: > This series implements mitigations for lack of DMA access control on > systems without an IOMMU, which could result in the DMA accessing the > system memory at unexpected times and/or unexpected addresses, possibly >

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Add relocation exceptions for two other platforms (rev7)

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915: Add relocation exceptions for two other platforms (rev7) URL : https://patchwork.freedesktop.org/series/89594/ State : success == Summary == CI Bug Log - changes from CI_DRM_10226_full -> Patchwork_20385_full

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Be more gentle with exiting non-persistent context (rev6)

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915: Be more gentle with exiting non-persistent context (rev6) URL : https://patchwork.freedesktop.org/series/89644/ State : success == Summary == CI Bug Log - changes from CI_DRM_10226 -> Patchwork_20386

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/gem: Remove duplicated call to ops->pread

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915/gem: Remove duplicated call to ops->pread URL : https://patchwork.freedesktop.org/series/91561/ State : failure == Summary == CI Bug Log - changes from CI_DRM_10226_full -> Patchwork_20383_full Summary

Re: [Intel-gfx] [PATCH v4 03/12] drm/i915: Introduce a ww transaction helper

2021-06-16 Thread Thomas Hellström
On 6/16/21 1:00 PM, Maarten Lankhorst wrote: Op 15-06-2021 om 15:14 schreef Thomas Hellström: Introduce a for_i915_gem_ww(){} utility to help make the code around a ww transaction more readable. Signed-off-by: Thomas Hellström Reviewed-by: Matthew Auld ---

Re: [Intel-gfx] [PATCH v4 03/12] drm/i915: Introduce a ww transaction helper

2021-06-16 Thread Maarten Lankhorst
Op 15-06-2021 om 15:14 schreef Thomas Hellström: > Introduce a for_i915_gem_ww(){} utility to help make the code > around a ww transaction more readable. > > Signed-off-by: Thomas Hellström > Reviewed-by: Matthew Auld > --- > drivers/gpu/drm/i915/i915_gem_ww.h | 31

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Add relocation exceptions for two other platforms (rev7)

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915: Add relocation exceptions for two other platforms (rev7) URL : https://patchwork.freedesktop.org/series/89594/ State : success == Summary == CI Bug Log - changes from CI_DRM_10226 -> Patchwork_20385

Re: [Intel-gfx] [PATCH] drm/i915/gem: Remove duplicated call to ops->pread

2021-06-16 Thread Matthew Auld
On Wed, 16 Jun 2021 at 10:04, Daniel Vetter wrote: > > Between > > commit ae30af84edb5b7cc95485922e43afd909a892e1b > Author: Maarten Lankhorst > Date: Tue Mar 23 16:50:00 2021 +0100 > > drm/i915: Disable userptr pread/pwrite support. > > and > > commit

Re: [Intel-gfx] [PATCH 3/3] drm/i915/uapi: Add query for L3 bank count

2021-06-16 Thread Daniel Vetter
On Thu, Jun 10, 2021 at 10:46 PM wrote: > > From: John Harrison > > Various UMDs need to know the L3 bank count. So add a query API for it. Please link to both the igt test submission for this (there's not even a Test-with: on the cover letter) and the merge requests for the various UMD which

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Be more gentle with exiting non-persistent context (rev5)

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915: Be more gentle with exiting non-persistent context (rev5) URL : https://patchwork.freedesktop.org/series/89644/ State : success == Summary == CI Bug Log - changes from CI_DRM_10226 -> Patchwork_20384

[Intel-gfx] [PATCH v5] drm/i915: Be more gentle with exiting non-persistent context

2021-06-16 Thread Tvrtko Ursulin
From: Tvrtko Ursulin When a non-persistent context exits we currently mark it as banned in order to trigger fast termination of any outstanding GPU jobs it may have left running. In doing so we apply a very strict 1ms limit in which the left over job has to preempt before we issues an engine

Re: [Intel-gfx] [PATCH 3/3] drm/i915/uapi: Add query for L3 bank count

2021-06-16 Thread Tvrtko Ursulin
On 10/06/2021 21:46, john.c.harri...@intel.com wrote: From: John Harrison Various UMDs need to know the L3 bank count. So add a query API for it. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/intel_gt.c | 15 +++ drivers/gpu/drm/i915/gt/intel_gt.h | 1 +

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Be more gentle with exiting non-persistent context (rev5)

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915: Be more gentle with exiting non-persistent context (rev5) URL : https://patchwork.freedesktop.org/series/89644/ State : warning == Summary == $ dim checkpatch origin/drm-tip e16557e3eeac drm/i915: Be more gentle with exiting non-persistent context -:66:

[Intel-gfx] [PATCH] drm/i915: Add relocation exceptions for two other platforms

2021-06-16 Thread Zbigniew Kempczyński
For topic/intel-for-CI branch only. We have established previously we stop using relocations starting from gen12 platforms with Tigerlake as an exception. We keep this statement but we want to enable relocations conditionally for Alderlake S+P under require_force_probe flag set. Keeping

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gem: Remove duplicated call to ops->pread

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915/gem: Remove duplicated call to ops->pread URL : https://patchwork.freedesktop.org/series/91561/ State : success == Summary == CI Bug Log - changes from CI_DRM_10226 -> Patchwork_20383 Summary ---

Re: [Intel-gfx] [PATCH 1/3] drm/i915/guc: Add fetch of hwconfig table

2021-06-16 Thread kernel test robot
to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/John-C-Harrison-Intel-com/Add-support-for-querying-hw-info-that-UMDs-need/20210616-141805 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: i386-defconfig

[Intel-gfx] [PATCH v5] drm/i915: Be more gentle with exiting non-persistent context

2021-06-16 Thread Tvrtko Ursulin
From: Tvrtko Ursulin When a non-persistent context exits we currently mark it as banned in order to trigger fast termination of any outstanding GPU jobs it may have left running. In doing so we apply a very strict 1ms limit in which the left over job has to preempt before we issues an engine

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gem: Remove duplicated call to ops->pread

2021-06-16 Thread Patchwork
== Series Details == Series: drm/i915/gem: Remove duplicated call to ops->pread URL : https://patchwork.freedesktop.org/series/91561/ State : warning == Summary == $ dim checkpatch origin/drm-tip 8887e9f43783 drm/i915/gem: Remove duplicated call to ops->pread -:11: ERROR:GIT_COMMIT_ID: Please

Re: [Intel-gfx] [PATCH] drm/i915: Force a TypeC PHY disconnect during suspend/shutdown

2021-06-16 Thread kernel test robot
'--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Imre-Deak/drm-i915-Force-a-TypeC-PHY-disconnect-during-suspend-shutdown/20210616-12 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: x86_64-randconfig

[Intel-gfx] [PATCH] drm/i915/gem: Remove duplicated call to ops->pread

2021-06-16 Thread Daniel Vetter
Between commit ae30af84edb5b7cc95485922e43afd909a892e1b Author: Maarten Lankhorst Date: Tue Mar 23 16:50:00 2021 +0100 drm/i915: Disable userptr pread/pwrite support. and commit 0049b688459b846f819b6e51c24cd0781fcfde41 Author: Matthew Auld Date: Thu Nov 5 15:49:33 2020 +

Re: [Intel-gfx] [PATCH v12 09/12] swiotlb: Add restricted DMA alloc/free support

2021-06-16 Thread Christoph Hellwig
On Wed, Jun 16, 2021 at 02:21:54PM +0800, Claire Chang wrote: > Add the functions, swiotlb_{alloc,free} and is_swiotlb_for_alloc to > support the memory allocation from restricted DMA pool. > > The restricted DMA pool is preferred if available. > > Note that since coherent allocation needs

Re: [Intel-gfx] [PATCH v12 06/12] swiotlb: Use is_swiotlb_force_bounce for swiotlb data bouncing

2021-06-16 Thread Christoph Hellwig
On Wed, Jun 16, 2021 at 02:21:51PM +0800, Claire Chang wrote: > Propagate the swiotlb_force into io_tlb_default_mem->force_bounce and > use it to determine whether to bounce the data or not. This will be > useful later to allow for different pools. > > Signed-off-by: Claire Chang Looks good,

[Intel-gfx] ✗ Fi.CI.BUILD: failure for Restricted DMA

2021-06-16 Thread Patchwork
== Series Details == Series: Restricted DMA URL : https://patchwork.freedesktop.org/series/91553/ State : failure == Summary == Applying: swiotlb: Refactor swiotlb init functions Applying: swiotlb: Refactor swiotlb_create_debugfs Applying: swiotlb: Set dev->dma_io_tlb_mem to the swiotlb pool

[Intel-gfx] ✗ Fi.CI.BUILD: failure for Restricted DMA

2021-06-16 Thread Patchwork
== Series Details == Series: Restricted DMA URL : https://patchwork.freedesktop.org/series/91548/ State : failure == Summary == Applying: swiotlb: Refactor swiotlb init functions Applying: swiotlb: Refactor swiotlb_create_debugfs Applying: swiotlb: Set dev->dma_io_tlb_mem to the swiotlb pool

Re: [Intel-gfx] New uAPI for color management proposal and feedback request

2021-06-16 Thread Maxime Ripard
Hi Pekka, On Mon, Jun 07, 2021 at 11:06:32AM +0300, Pekka Paalanen wrote: > On Mon, 7 Jun 2021 09:48:05 +0200 > Maxime Ripard wrote: > > > I've started to implement this for the raspberrypi some time ago. > > > > https://github.com/raspberrypi/linux/pull/4201 > > > > It's basically two

[Intel-gfx] ✗ Fi.CI.IGT: failure for Explicity steer l3bank multicast reads when necessary (rev3)

2021-06-16 Thread Patchwork
== Series Details == Series: Explicity steer l3bank multicast reads when necessary (rev3) URL : https://patchwork.freedesktop.org/series/91485/ State : failure == Summary == CI Bug Log - changes from CI_DRM_10226_full -> Patchwork_20380_full

Re: [Intel-gfx] [PATCH 07/10] vfio/mdev: Allow the mdev_parent_ops to specify the device driver to bind

2021-06-16 Thread Greg Kroah-Hartman
On Tue, Jun 15, 2021 at 09:00:40PM -0300, Jason Gunthorpe wrote: > On Tue, Jun 15, 2021 at 04:11:29PM +0200, Greg Kroah-Hartman wrote: > > On Tue, Jun 15, 2021 at 03:35:16PM +0200, Christoph Hellwig wrote: > > > From: Jason Gunthorpe > > > > > > This allows a mdev driver to opt out of using

Re: [Intel-gfx] [PATCH v11 00/12] Restricted DMA

2021-06-16 Thread Claire Chang
v12: https://lore.kernel.org/patchwork/cover/1447254/ On Wed, Jun 16, 2021 at 11:52 AM Claire Chang wrote: > > This series implements mitigations for lack of DMA access control on > systems without an IOMMU, which could result in the DMA accessing the > system memory at unexpected times and/or

[Intel-gfx] [PATCH v12 12/12] of: Add plumbing for restricted DMA pool

2021-06-16 Thread Claire Chang
If a device is not behind an IOMMU, we look up the device node and set up the restricted DMA when the restricted-dma-pool is presented. Signed-off-by: Claire Chang --- drivers/of/address.c| 33 + drivers/of/device.c | 3 +++ drivers/of/of_private.h | 6

  1   2   >