Re: [RFC PATCH v1] dma-buf: heaps: move the verification of heap_flags to the corresponding heap

2024-06-03 Thread Hailong Liu
On 6/4/2024 2:06 AM, John Stultz wrote: > On Mon, Jun 3, 2024 at 10:21 AM Hailong Liu wrote: >> On Mon, 03. Jun 09:01, John Stultz wrote: >>> On Mon, Jun 3, 2024 at 4:40 AM wrote: From: "Hailong.Liu" This help module use heap_flags to determine the type of dma-buf, so th

Re: [PATCH v11 2/8] drm/i915: Define and compute Transcoder CMRR registers

2024-06-03 Thread Nautiyal, Ankit K
On 6/3/2024 11:18 AM, Mitul Golani wrote: Add register definitions for Transcoder Fixed Average Vtotal mode/CMRR function, with the necessary bitfields. Compute these registers when CMRR is enabled, extending Adaptive refresh rate capabilities. --v2: - Use intel_de_read64_2x32 in intel_vrr_get

Re: [PATCH v11 1/8] drm/i915: Separate VRR related register definitions

2024-06-03 Thread Nautiyal, Ankit K
On 6/3/2024 11:18 AM, Mitul Golani wrote: Move VRR related register definitions to a separate file called intel_vrr_regs.h. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 1 + drivers/gpu/drm/i915/display/intel_vrr_regs.h | 113 ++ driver

Re: [PATCH] kernel/resource: optimize find_next_iomem_res

2024-06-03 Thread Chia-I Wu
On Mon, Jun 3, 2024 at 12:24 AM Ilpo Järvinen wrote: > > On Sun, 2 Jun 2024, Andy Shevchenko wrote: > > > On Fri, May 31, 2024 at 02:31:45PM -0700, Chia-I Wu wrote: > > > On Fri, May 31, 2024 at 1:57 AM Andy Shevchenko < > > > andriy.shevche...@linux.intel.com> wrote: > > > > On Thu, May 30, 2024

Re: [PATCH 3/6] dt-bindings: gpu: powervr-rogue: Add MediaTek MT8173 GPU

2024-06-03 Thread Chen-Yu Tsai
On Fri, May 31, 2024 at 10:25 PM Adam Ford wrote: > > On Fri, May 31, 2024 at 8:37 AM Frank Binns wrote: > > > > Hi ChenYu, > > > > On Thu, 2024-05-30 at 16:35 +0800, Chen-Yu Tsai wrote: > > > The MediaTek MT8173 comes with a PowerVR Rogue GX6250, which is one > > > of the Series6XT GPUs, another

Re: [PATCH 3/6] dt-bindings: gpu: powervr-rogue: Add MediaTek MT8173 GPU

2024-06-03 Thread Chen-Yu Tsai
On Fri, May 31, 2024 at 9:37 PM Frank Binns wrote: > > Hi ChenYu, > > On Thu, 2024-05-30 at 16:35 +0800, Chen-Yu Tsai wrote: > > The MediaTek MT8173 comes with a PowerVR Rogue GX6250, which is one > > of the Series6XT GPUs, another sub-family of the Rogue family. > > I've added Adam Ford who sent

Re: [PATCH v11 14/26] locking/lockdep, cpu/hotplus: Use a weaker annotation in AP thread

2024-06-03 Thread Byungchul Park
On Fri, Jan 26, 2024 at 06:30:02PM +0100, Thomas Gleixner wrote: > > Furthermore, now that Dept was introduced, false positive alarms was > > reported by that. Replaced it with try lock annotation. > > I still have zero idea what this is about. Lockdep is working on lock/unlock, while dept is wor

[PATCH] drm/v3d: Fix compile with CONFIG_WERROR

2024-06-03 Thread Ashutosh Dixit
Fix compile with CONFIG_WERROR by explicitly computing the max number of 7.1 and 4.2 counters. In file included from ../drivers/gpu/drm/v3d/v3d_drv.h:14, from ../drivers/gpu/drm/v3d/v3d_bo.c:26: ../drivers/gpu/drm/v3d/v3d_performance_counters.h:118:43: \ error: ‘v3d_v42_pe

[PATCH v1] drm/bridge: it6505: update usleep_range for RC circuit charge time

2024-06-03 Thread kuro
From: Kuro Chung The spec of timing between IVDD/OVDD and SYSRTEN is 10ms, but SYSRSTN RC circuit need at least 25ms for rising time, update for match spec Signed-off-by: Kuro Chung Signed-off-by: Hermes Wu --- drivers/gpu/drm/bridge/ite-it6505.c | 2 +- 1 file changed, 1 insertion(+), 1 dele

Re: [PATCH v2] drm/client: Detect when ACPI lid is closed during initialization

2024-06-03 Thread kernel test robot
Hi Mario, kernel test robot noticed the following build errors: [auto build test ERROR on drm-misc/drm-misc-next] [also build test ERROR on drm/drm-next drm-exynos/exynos-drm-next drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-tip/drm-tip linus/master v6.10-rc2 next-20240603] [If

[PATCH v2 22/22] iommu: Remove iommu_domain_alloc()

2024-06-03 Thread Lu Baolu
The iommu_domain_alloc() interface is no longer used in the tree anymore. Remove it to avoid dead code. There is increasing demand for supporting multiple IOMMU drivers, and this is the last bus-based thing standing in the way of that. Signed-off-by: Lu Baolu --- include/linux/iommu.h | 6

[PATCH v2 21/22] iommu: Remove iommu_present()

2024-06-03 Thread Lu Baolu
The iommu_present() interface is no longer used in the tree anymore. Remove it to avoid dead code. Signed-off-by: Lu Baolu --- include/linux/iommu.h | 6 -- drivers/iommu/iommu.c | 25 - 2 files changed, 31 deletions(-) diff --git a/include/linux/iommu.h b/include/l

[PATCH v2 20/22] drm/tegra: Remove call to iommu_domain_alloc()

2024-06-03 Thread Lu Baolu
Commit <17de3f5fdd35> ("iommu: Retire bus ops") removes iommu ops from the bus structure. The iommu subsystem no longer relies on bus for operations. So iommu_domain_alloc() interface is no longer relevant. Normally, iommu_paging_domain_alloc() could be a replacement for iommu_domain_alloc() if th

[PATCH v2 19/22] drm/rockchip: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
Commit <421be3ee36a4> ("drm/rockchip: Refactor IOMMU initialisation") has refactored rockchip_drm_init_iommu() to pass a device that the domain is allocated for. Replace iommu_domain_alloc() with iommu_paging_domain_alloc() to retire the former. Signed-off-by: Lu Baolu --- drivers/gpu/drm/rockch

[PATCH v2 17/22] ARM: dma-mapping: Pass device to arm_iommu_create_mapping()

2024-06-03 Thread Lu Baolu
From: Robin Murphy All users of ARM IOMMU mappings create them for a particular device, so change the interface to accept the device rather than forcing a vague indirection through a bus type. This prepares for making a similar change to iommu_domain_alloc() itself. Signed-off-by: Robin Murphy

[PATCH v2 18/22] ARM: dma-mapping: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
Since arm_iommu_create_mapping() now accepts the device, let's replace iommu_domain_alloc() with iommu_paging_domain_alloc() to retire the former. Signed-off-by: Lu Baolu --- arch/arm/mm/dma-mapping.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/arm/mm/dma-mappi

[PATCH v2 15/22] RDMA/usnic: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
usnic_uiom_alloc_pd() allocates a paging domain for a given device. In this case, iommu_domain_alloc(dev->bus) is equivalent to  iommu_paging_domain_alloc(dev). Replace it as iommu_domain_alloc() has been deprecated. Signed-off-by: Lu Baolu --- drivers/infiniband/hw/usnic/usnic_uiom.c | 6 +++---

[PATCH v2 16/22] iommu/vt-d: Add helper to allocate paging domain

2024-06-03 Thread Lu Baolu
The domain_alloc_user operation is currently implemented by allocating a paging domain using iommu_domain_alloc(). This is because it needs to fully initialize the domain before return. Add a helper to do this to avoid using iommu_domain_alloc(). Signed-off-by: Lu Baolu --- drivers/iommu/intel/i

[PATCH v2 14/22] soc/fsl/qbman: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
An iommu domain is allocated in portal_set_cpu() and is attached to pcfg->dev in the same function. Use iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu --- drivers/soc/fsl/qbman/qman_portal.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drive

[PATCH v2 13/22] remoteproc: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
An iommu domain is allocated in rproc_enable_iommu() and is attached to rproc->dev.parent in the same function. Use iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu --- drivers/remoteproc/remoteproc_core.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH v2 12/22] wifi: ath11k: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
An iommu domain is allocated in ath11k_ahb_fw_resources_init() and is attached to ab_ahb->fw.dev in the same function. Use iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu --- drivers/net/wireless/ath/ath11k/ahb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(

[PATCH v2 11/22] ath10k: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
An iommu domain is allocated in ath10k_fw_init() and is attached to ar_snoc->fw.dev in the same function. Use iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu --- drivers/net/wireless/ath/ath10k/snoc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --gi

[PATCH v2 10/22] media: venus: firmware: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
An iommu domain is allocated in venus_firmware_init() and is attached to core->fw.dev in the same function. Use iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu --- drivers/media/platform/qcom/venus/firmware.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) d

[PATCH v2 09/22] media: nvidia: tegra: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
An iommu domain is allocated in tegra_vde_iommu_init() and is attached to vde->dev. Use iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu --- drivers/media/platform/nvidia/tegra-vde/iommu.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/

[PATCH v2 07/22] drm/nouveau/tegra: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
In nvkm_device_tegra_probe_iommu(), a paging domain is allocated for @dev and attached to it on success. Use iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu --- drivers/gpu/drm/nouveau/nvkm/engine/device/tegra.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH v2 08/22] gpu: host1x: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
An iommu domain is allocated in host1x_iommu_attach() and is attached to host->dev. Use iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu --- drivers/gpu/host1x/dev.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/host1x/dev.c b/driv

[PATCH v2 06/22] drm/msm: Use iommu_paging_domain_alloc()

2024-06-03 Thread Lu Baolu
The domain allocated in msm_iommu_new() is for the @dev. Replace iommu_domain_alloc() with iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu --- drivers/gpu/drm/msm/msm_iommu.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/msm/m

[PATCH v2 05/22] iommu: Add iommu_paging_domain_alloc() interface

2024-06-03 Thread Lu Baolu
Commit <17de3f5fdd35> ("iommu: Retire bus ops") removes iommu ops from bus. The iommu subsystem no longer relies on bus for operations. So the bus parameter in iommu_domain_alloc() is no longer relevant. Add a new interface named iommu_paging_domain_alloc(), which explicitly indicates the allocati

[PATCH v2 04/22] vhost-vdpa: Use iommu_user_domain_alloc()

2024-06-03 Thread Lu Baolu
Replace iommu_domain_alloc() with iommu_user_domain_alloc(). Signed-off-by: Lu Baolu --- drivers/vhost/vdpa.c | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/drivers/vhost/vdpa.c b/drivers/vhost/vdpa.c index 63a53680a85c..d15673cb05f2 100644 --- a/drivers/vhost

[PATCH v2 03/22] vfio/type1: Use iommu_user_domain_alloc()

2024-06-03 Thread Lu Baolu
Replace iommu_domain_alloc() with iommu_user_domain_alloc(). Signed-off-by: Lu Baolu --- drivers/vfio/vfio_iommu_type1.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/vfio/vfio_iommu_type1.c b/drivers/vfio/vfio_iommu_type1.c index 3a0218171cfa..1d553f7f7c26 1

[PATCH v2 02/22] iommufd: Use iommu_user_domain_alloc()

2024-06-03 Thread Lu Baolu
Replace iommu_domain_alloc() with iommu_user_domain_alloc(). Signed-off-by: Lu Baolu --- drivers/iommu/iommufd/hw_pagetable.c | 20 +--- 1 file changed, 5 insertions(+), 15 deletions(-) diff --git a/drivers/iommu/iommufd/hw_pagetable.c b/drivers/iommu/iommufd/hw_pagetable.c ind

[PATCH v2 01/22] iommu: Add iommu_user_domain_alloc() interface

2024-06-03 Thread Lu Baolu
Commit <909f4abd1097> ("iommu: Add new iommu op to create domains owned by userspace") added a dedicated iommu op to allocate a user domain. While IOMMUFD has already made use of this callback, other frameworks like vfio/type1 and vDPA still use the paging domain allocation interface. Add a new in

[PATCH v2 00/22] iommu: Refactoring domain allocation interface

2024-06-03 Thread Lu Baolu
The IOMMU subsystem has undergone some changes, including the removal of iommu_ops from the bus structure. Consequently, the existing domain allocation interface, which relies on a bus type argument, is no longer relevant: struct iommu_domain *iommu_domain_alloc(struct bus_type *bus) This ser

Re: [PATCH 00/20] iommu: Refactoring domain allocation interface

2024-06-03 Thread Baolu Lu
On 6/3/24 9:35 PM, Jason Gunthorpe wrote: On Wed, May 29, 2024 at 08:02:12PM +0800, Baolu Lu wrote: drivers/infiniband/hw/usnic/usnic_uiom.c:   pd->domain = domain = iommu_domain_alloc(dev->bus); This series leave those cases unchanged and keep iommu_domain_alloc() for their usage. But new

Re: [PATCH] drm/i915/gt: Fix potential UAF by revoke of fence registers

2024-06-03 Thread Andi Shyti
Hi Janusz, On Mon, Jun 03, 2024 at 09:54:45PM +0200, Janusz Krzysztofik wrote: > CI has been sporadically reporting the following issue triggered by > igt@i915_selftest@live@hangcheck on ADL-P and similar machines: > > <6> [414.049203] i915: Running > intel_hangcheck_live_selftests/igt_reset_evi

Re: [PATCH] drm/i915/gt: Delete the live_hearbeat_fast selftest

2024-06-03 Thread Andi Shyti
Hi Krzysztof, On Mon, Jun 03, 2024 at 06:20:22PM +0200, Niemiec, Krzysztof wrote: > The test is trying to push the heartbeat frequency to the limit, which > might sometimes fail. Such a failure does not provide valuable > information, because it does not indicate that there is something > necessar

[PATCH] drm/msm/adreno: Add support for Adreno 505 GPU

2024-06-03 Thread Barnabás Czémán
tatic inline int adreno_is_a506(const struct adreno_gpu *gpu) { return adreno_is_revn(gpu, 506); --- base-commit: 861a3cb5a2a8480d361fa6708da24747d6fa72fe change-id: 20240603-a505-e57389b61d2a Best regards, -- Barnabás Czémán

Re: [RFC PATCH v1] dma-buf: heaps: move the verification of heap_flags to the corresponding heap

2024-06-03 Thread Hailong Liu
On Mon, 03. Jun 09:01, John Stultz wrote: > On Mon, Jun 3, 2024 at 4:40 AM wrote: > > > > From: "Hailong.Liu" > > > > This help module use heap_flags to determine the type of dma-buf, > > so that some mechanisms can be used to speed up allocation, such as > > memory_pool, to optimize the allocati

[PATCH v1 1/1] treewide: Align match_string() with sysfs_match_string()

2024-06-03 Thread Andy Shevchenko
Make two APIs look similar. Hence convert match_string() to be a 2-argument macro. In order to avoid unneeded churn, convert all users as well. There is no functional change intended. Signed-off-by: Andy Shevchenko --- Compile tested with `make allyesconfig` and `make allmodconfig` on x86_64, ar

[RFC PATCH v1] dma-buf: heaps: move the verification of heap_flags to the corresponding heap

2024-06-03 Thread hailong.liu
From: "Hailong.Liu" This help module use heap_flags to determine the type of dma-buf, so that some mechanisms can be used to speed up allocation, such as memory_pool, to optimize the allocation time of dma-buf. Signed-off-by: Hailong.Liu --- drivers/dma-buf/dma-heap.c | 3 --- drivers

Re: [PATCH v1 1/1] treewide: Align match_string() with sysfs_match_string()

2024-06-03 Thread Bjorn Helgaas
On Sun, Jun 02, 2024 at 06:57:12PM +0300, Andy Shevchenko wrote: > Make two APIs look similar. Hence convert match_string() to be > a 2-argument macro. In order to avoid unneeded churn, convert > all users as well. There is no functional change intended. Looks nice, thanks for doing this. > diff

Re: [PATCH] drm/amd/display: Increase frame-larger-than warning limit

2024-06-03 Thread Nathan Chancellor
Hi Palmer, On Thu, May 30, 2024 at 07:57:42AM -0700, Palmer Dabbelt wrote: > From: Palmer Dabbelt > > I get a handful of build errors along the lines of > > > linux/drivers/gpu/drm/amd/amdgpu/../display/dc/dml/dcn32/display_mode_vba_32.c:58:13: > error: stack frame size (2352) exceeds lim

Re: [PATCH] Revert "drm/amdgpu: init iommu after amdkfd device init"

2024-06-03 Thread Armin Wolf
Am 23.05.24 um 19:30 schrieb Armin Wolf: This reverts commit 56b522f4668167096a50c39446d6263c96219f5f. A user reported that this commit breaks the integrated gpu of his notebook, causing a black screen. He was able to bisect the problematic commit and verified that by reverting it the notebook

Re: [PATCH v3 4/5] drm/mipi-dbi: Add support for DRM_FORMAT_RGB888

2024-06-03 Thread Dmitry Baryshkov
On Mon, Jun 03, 2024 at 01:21:35PM +0200, Noralf Trønnes via B4 Relay wrote: > From: Noralf Trønnes > > DRM_FORMAT_RGB888 is 24 bits per pixel and it would be natural to send it > on the SPI bus using a 24 bits per word transfer. The problem with this > is that not all SPI controllers support 24

Re: [PATCH v2] drm/msm/dpu: fix encoder irq wait skip

2024-06-03 Thread Abhinav Kumar
On 5/9/2024 12:40 PM, Barnabás Czémán wrote: The irq_idx is unsigned so it cannot be lower than zero, better to change the condition to check if it is equal with zero. It could not cause any issue because a valid irq index starts from one. Fixes: 5a9d50150c2c ("drm/msm/dpu: shift IRQ indices

Re: [PATCH v2 7/9] drm/msm/dpu: check for overflow in _dpu_crtc_setup_lm_bounds()

2024-06-03 Thread Dmitry Baryshkov
On Mon, Jun 03, 2024 at 02:24:27PM -0700, Abhinav Kumar wrote: > > > On 6/3/2024 2:16 PM, Dmitry Baryshkov wrote: > > On Mon, 3 Jun 2024 at 23:57, Abhinav Kumar > > wrote: > > > > > > > > > > > > On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: > > > > Check in _dpu_crtc_setup_lm_bounds() that C

Re: [PATCH 2/6] drm/bridge: tc358767: Use tc_pxl_pll_calc() to correct adjusted_mode clock

2024-06-03 Thread Marek Vasut
On 6/3/24 2:45 PM, Alexander Stein wrote: Hi, @@ -1631,6 +1643,18 @@ static int tc_edp_atomic_check(struct drm_bridge *bridge, struct drm_crtc_state *crtc_state, struct drm_connector_state *conn_state) { + struct tc_data *tc

Re: [PATCH 3/6] drm/bridge: tc358767: Drop line_pixel_subtract

2024-06-03 Thread Marek Vasut
On 6/3/24 2:18 PM, Alexander Stein wrote: Hi Marek, Hi, Am Freitag, 31. Mai 2024, 22:39:49 CEST schrieb Marek Vasut: This line_pixel_subtract is no longer needed now that the bridge can request and obtain specific pixel clock on input to the bridge, with clock frequency that matches the Pixe

Re: [PATCH v2 9/9] drm/msm/dpu: sync mode_config limits to the FB limits in dpu_plane.c

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: Lift mode_config limits set by the DPU driver to the actual FB limits as handled by the dpu_plane.c. Move 2*max_lm_width check where it belongs, to the drm_crtc_helper_funcs::mode_valid() callback. Signed-off-by: Dmitry Baryshkov --- drivers/gpu

Re: [PATCH v2 7/9] drm/msm/dpu: check for overflow in _dpu_crtc_setup_lm_bounds()

2024-06-03 Thread Abhinav Kumar
On 6/3/2024 2:16 PM, Dmitry Baryshkov wrote: On Mon, 3 Jun 2024 at 23:57, Abhinav Kumar wrote: On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: Check in _dpu_crtc_setup_lm_bounds() that CRTC width is not overflowing LM requirements. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/ms

Re: [PATCH] drm/msm/dpu: drop duplicate drm formats from wb2_formats arrays

2024-06-03 Thread Dmitry Baryshkov
On Fri, May 24, 2024 at 11:01:12PM +0800, Junhao Xie wrote: > There are duplicate items in wb2_formats_rgb and wb2_formats_rgb_yuv, > which cause weston assertions failed. > > weston: libweston/drm-formats.c:131: weston_drm_format_array_add_format: > Assertion `!weston_drm_format_array_find_format

Re: [PATCH v2 7/9] drm/msm/dpu: check for overflow in _dpu_crtc_setup_lm_bounds()

2024-06-03 Thread Dmitry Baryshkov
On Mon, 3 Jun 2024 at 23:57, Abhinav Kumar wrote: > > > > On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: > > Check in _dpu_crtc_setup_lm_bounds() that CRTC width is not overflowing > > LM requirements. > > > > Signed-off-by: Dmitry Baryshkov > > --- > > drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c |

Re: [PATCH v2 7/9] drm/msm/dpu: check for overflow in _dpu_crtc_setup_lm_bounds()

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: Check in _dpu_crtc_setup_lm_bounds() that CRTC width is not overflowing LM requirements. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff -

Re: [PATCH v3 3/5] drm/mipi-dbi: Make bits per word configurable for pixel transfers

2024-06-03 Thread Dmitry Baryshkov
On Mon, Jun 03, 2024 at 01:21:34PM +0200, Noralf Trønnes via B4 Relay wrote: > From: Noralf Trønnes > > This prepares for supporting other pixel formats than RGB565. Yes, the patch is pretty simple, however could you please expand the commit message by describing write_memory_bpw introduction.

Re: [PATCH v3 2/5] drm/mipi-dbi: Remove mipi_dbi_machine_little_endian()

2024-06-03 Thread Dmitry Baryshkov
On Mon, Jun 03, 2024 at 01:21:33PM +0200, Noralf Trønnes via B4 Relay wrote: > From: Noralf Trønnes > > mipi_dbi_machine_little_endian() should really have been called > mipi_dbi_framebuffer_little_endian() because that's the function it > performs. When I added support for these SPI displays I t

Re: [PATCH] drm: Combine identical if/elif code blocks

2024-06-03 Thread Thorsten Blum
On 16. May 2024, at 03:24, Luc Ma wrote: > On Wed, 15 May 2024 at 17:31, Thorsten Blum wrote: >> On 15. May 2024, at 11:22, Thorsten Blum wrote: >>> On 15. May 2024, at 09:43, Luc Ma wrote: On Tue, 14 May 2024 at 19:37, Thorsten Blum wrote: > > Merge the identical if/elif c

[PATCH] drm/i915/gt: Fix potential UAF by revoke of fence registers

2024-06-03 Thread Janusz Krzysztofik
CI has been sporadically reporting the following issue triggered by igt@i915_selftest@live@hangcheck on ADL-P and similar machines: <6> [414.049203] i915: Running intel_hangcheck_live_selftests/igt_reset_evict_fence ... <6> [414.068804] i915 :00:02.0: [drm] GT0: GUC: submission enabled <6> [4

Re: [PATCH v2 6/9] drm/msm/dpu: drop call to _dpu_crtc_setup_lm_bounds from atomic_begin

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: The dpu_crtc_atomic_check() already calls _dpu_crtc_setup_lm_bounds(). There is no need to call it from dpu_crtc_atomic_begin(). Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 2 -- 1 file changed, 2 deletions(-)

Re: [PATCH v2 4/9] drm/msm/dpu: move dpu_format_populate_plane_sizes to atomic_check

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: Move a call to dpu_format_populate_plane_sizes() to the atomic_check step, so that any issues with the FB layout can be reported as early as possible. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 12 ++-

Re: [PATCH 4/9] drm/msm/dpu: move dpu_format_populate_plane_sizes to atomic_check

2024-06-03 Thread Abhinav Kumar
On 4/22/2024 5:22 AM, Dmitry Baryshkov wrote: On Fri, Apr 19, 2024 at 07:37:44PM -0700, Abhinav Kumar wrote: On 4/19/2024 6:34 PM, Dmitry Baryshkov wrote: On Fri, Apr 19, 2024 at 05:14:01PM -0700, Abhinav Kumar wrote: On 3/19/2024 6:22 AM, Dmitry Baryshkov wrote: Move a call to dpu_for

Re: [PATCH v2 2/9] drm/msm/dpu: drop dpu_format_populate_layout from dpu_plane_sspp_atomic_update

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: The dpu_plane_prepare_fb() already calls dpu_format_populate_layout(). Store the generated layour in the plane state and drop this call from dpu_plane_sspp_update(). noticed a small typo (missed it in v1), layour --> layout. No need to resend,

Re: [PATCH v2 1/9] drm/msm/dpu: drop dpu_format_check_modified_format

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: The msm_kms_funcs::check_modified_format() callback is not used by the driver. Drop it completely. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c | 43 - drivers/gpu/drm/msm/disp/dpu1

Re: [PATCH 1/9] drm/msm/dpu: drop dpu_format_check_modified_format

2024-06-03 Thread Abhinav Kumar
On 4/22/2024 4:06 AM, Dmitry Baryshkov wrote: On Fri, Apr 19, 2024 at 07:32:35PM -0700, Abhinav Kumar wrote: On 4/19/2024 6:26 PM, Dmitry Baryshkov wrote: On Fri, Apr 19, 2024 at 04:43:20PM -0700, Abhinav Kumar wrote: On 3/19/2024 6:21 AM, Dmitry Baryshkov wrote: The msm_kms_funcs::che

Re: [RFC PATCH v1] dma-buf: heaps: move the verification of heap_flags to the corresponding heap

2024-06-03 Thread John Stultz
On Mon, Jun 3, 2024 at 10:21 AM Hailong Liu wrote: > On Mon, 03. Jun 09:01, John Stultz wrote: > > On Mon, Jun 3, 2024 at 4:40 AM wrote: > > > > > > From: "Hailong.Liu" > > > > > > This help module use heap_flags to determine the type of dma-buf, > > > so that some mechanisms can be used to spee

[PATCH] drm/xe/vm: Simplify if condition

2024-06-03 Thread Thorsten Blum
The if condition !A || A && B can be simplified to !A || B. Fixes the following Coccinelle/coccicheck warning reported by excluded_middle.cocci: WARNING !A || A && B is equivalent to !A || B Compile-tested only. Signed-off-by: Thorsten Blum --- drivers/gpu/drm/xe/xe_vm.c | 4 ++-- 1 f

Re: [PATCH v10 1/8] x86/vmware: Introduce VMware hypercall API

2024-06-03 Thread Borislav Petkov
On Wed, May 29, 2024 at 05:44:32PM -0700, Alexey Makhalov wrote: > While most of the vmware_hypercall callers are executed after alternative > patching applied, there are small amount of hypercalls running before that. > Only for them we have the logic of analyzing vmware_hypercall_mode as a > defa

Re: [PATCH 2/6] drm/nouveau: remove unused struct 'init_exec'

2024-06-03 Thread Dr. David Alan Gilbert
* li...@treblig.org (li...@treblig.org) wrote: > From: "Dr. David Alan Gilbert" > > 'init_exec' is unused since > commit cb75d97e9c77 ("drm/nouveau: implement devinit subdev, and new > init table parser") > Remove it. > > Signed-off-by: Dr. David Alan Gilbert Ping. > --- > drivers/gpu/drm/n

[linux-next:master] BUILD REGRESSION 861a3cb5a2a8480d361fa6708da24747d6fa72fe

2024-06-03 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: 861a3cb5a2a8480d361fa6708da24747d6fa72fe Add linux-next specific files for 20240603 Error/Warning reports: https://lore.kernel.org/oe-kbuild-all/202406031357.4t4jtalq-...@intel.com https

Re: [PATCH 3/6] drm/vmwgfx: remove unused struct 'vmw_stdu_dma'

2024-06-03 Thread Dr. David Alan Gilbert
* li...@treblig.org (li...@treblig.org) wrote: > From: "Dr. David Alan Gilbert" > > 'vmw_stdu_dma' is unused since > commit 39985eea5a6d ("drm/vmwgfx: Abstract placement selection") > Remove it. Ping. > Signed-off-by: Dr. David Alan Gilbert > --- > drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c | 5 ---

[PATCH] agp: add missing MODULE_DESCRIPTION() macros

2024-06-03 Thread Jeff Johnson
it(agp_via_cleanup); +MODULE_DESCRIPTION("VIA AGPGART routines"); MODULE_LICENSE("GPL"); MODULE_AUTHOR("Dave Jones"); --- base-commit: a693b9c95abd4947c2d06e05733de5d470ab6586 change-id: 20240603-md-agp-68a9e38017af

Re: [PATCH 00/14] Add initial support for the Rockchip RK3588 HDMI TX Controller

2024-06-03 Thread Maxime Ripard
On Mon, Jun 03, 2024 at 03:03:12PM GMT, Heiko Stuebner wrote: > Am Montag, 3. Juni 2024, 14:14:17 CEST schrieb Andy Yan: > > Hi Neil: > > > > On 6/3/24 16:55, Neil Armstrong wrote: > > > Hi Christian, > > > > > > On 01/06/2024 15:12, Cristian Ciocaltea wrote: > > >> The RK3588 SoC family integrat

[PATCH] drm/i915/gt: Delete the live_hearbeat_fast selftest

2024-06-03 Thread Niemiec, Krzysztof
The test is trying to push the heartbeat frequency to the limit, which might sometimes fail. Such a failure does not provide valuable information, because it does not indicate that there is something necessarily wrong with either the driver or the hardware. Remove the test to prevent random, unnec

Re: [RFC PATCH v1] dma-buf: heaps: move the verification of heap_flags to the corresponding heap

2024-06-03 Thread John Stultz
On Mon, Jun 3, 2024 at 4:40 AM wrote: > > From: "Hailong.Liu" > > This help module use heap_flags to determine the type of dma-buf, > so that some mechanisms can be used to speed up allocation, such as > memory_pool, to optimize the allocation time of dma-buf. This feels like it's trying to intr

Re: [DO NOT MERGE v8 21/36] dt-bindings: serial: renesas,scif: Add scif-sh7751.

2024-06-03 Thread Rob Herring (Arm)
On Wed, 29 May 2024 17:01:07 +0900, Yoshinori Sato wrote: > Add Renesas SH7751 SCIF. > > Signed-off-by: Yoshinori Sato > Reviewed-by: Geert Uytterhoeven > --- > Documentation/devicetree/bindings/serial/renesas,scif.yaml | 1 + > 1 file changed, 1 insertion(+) > Acked-by: Rob Herring (Arm)

Re: [DO NOT MERGE v8 19/36] dt-bindings: interrupt-controller: renesas,sh7751-irl-ext: Add json-schema

2024-06-03 Thread Rob Herring
On Wed, May 29, 2024 at 05:01:05PM +0900, Yoshinori Sato wrote: > Renesas SH7751 external interrupt encoder json-schema. > > Signed-off-by: Yoshinori Sato > --- > .../renesas,sh7751-irl-ext.yaml | 57 +++ > 1 file changed, 57 insertions(+) > create mode 100644 > D

Re: [PATCH net-next v10 02/14] net: page_pool: create hooks for custom page providers

2024-06-03 Thread Mina Almasry
On Mon, Jun 3, 2024 at 7:52 AM Pavel Begunkov wrote: > > On 6/3/24 15:17, Mina Almasry wrote: > > On Fri, May 31, 2024 at 10:35 PM Christoph Hellwig > > wrote: > >> > >> On Thu, May 30, 2024 at 08:16:01PM +, Mina Almasry wrote: > >>> I'm unsure if the discussion has been resolved yet. Sendin

Re: [PATCH v3 01/10] media: Add Chameleon v3 video interface driver

2024-06-03 Thread Hans Verkuil
On 03/06/2024 16:32, Paweł Anikiel wrote: > On Mon, Jun 3, 2024 at 9:57 AM Hans Verkuil wrote: >> >> On 07/05/2024 17:54, Paweł Anikiel wrote: >>> Add v4l2 driver for the video interface present on the Google >>> Chameleon v3. The Chameleon v3 uses the video interface to capture >>> a single video

Re: [PATCH net-next v10 02/14] net: page_pool: create hooks for custom page providers

2024-06-03 Thread Pavel Begunkov
On 6/3/24 15:17, Mina Almasry wrote: On Fri, May 31, 2024 at 10:35 PM Christoph Hellwig wrote: On Thu, May 30, 2024 at 08:16:01PM +, Mina Almasry wrote: I'm unsure if the discussion has been resolved yet. Sending the series anyway to get reviews/feedback on the (unrelated) rest of the ser

Re: [PATCH v3 00/12] drm/imx/ipuv3: switch LDB and parallel-display driver to use drm_bridge_connector

2024-06-03 Thread Chris Healy
On Mon, Jun 3, 2024 at 3:12 AM Dmitry Baryshkov wrote: > > On Sun, Jun 02, 2024 at 08:25:39PM -0700, Chris Healy wrote: > > On an i.MX53 QSB with HDMI daughter board, this patch series is: > > > > Tested-by: Chris Healy > > Thank you! I assume this is imx53-qsrb-hdmi ? Yes > > > > > HDMI output

Re: [PATCH v3 01/10] media: Add Chameleon v3 video interface driver

2024-06-03 Thread Paweł Anikiel
On Mon, Jun 3, 2024 at 9:57 AM Hans Verkuil wrote: > > On 07/05/2024 17:54, Paweł Anikiel wrote: > > Add v4l2 driver for the video interface present on the Google > > Chameleon v3. The Chameleon v3 uses the video interface to capture > > a single video source from a given HDMI or DP connector and

Re: [PATCH net-next v10 02/14] net: page_pool: create hooks for custom page providers

2024-06-03 Thread Mina Almasry
On Fri, May 31, 2024 at 10:35 PM Christoph Hellwig wrote: > > On Thu, May 30, 2024 at 08:16:01PM +, Mina Almasry wrote: > > I'm unsure if the discussion has been resolved yet. Sending the series > > anyway to get reviews/feedback on the (unrelated) rest of the series. > > As far as I'm concern

Re: [PATCH 01/20] drm/bridge: add dsi_lp11_notify mechanism

2024-06-03 Thread Michael Walle
[+ Marek ] Hi Dmitry, > > > Some bridges have very strict power-up reqirements. In this case, the > > > Toshiba TC358775. The reset has to be deasserted while *both* the DSI > > > clock and DSI data lanes are in LP-11 mode. After the reset is relased, > > > the bridge needs the DSI clock to actua

[PATCH v13 12/12] drm/ttm/tests: Use u32 and u64 over uint*_t types

2024-06-03 Thread Karolina Stolarek
Update the tests and helpers to use unsigned kernel types. Signed-off-by: Karolina Stolarek Suggested-by: Thomas Hellström --- drivers/gpu/drm/ttm/tests/ttm_bo_test.c | 12 ++-- drivers/gpu/drm/ttm/tests/ttm_kunit_helpers.c | 6 ++ drivers/gpu/drm/ttm/tests/ttm_kunit_helpers.

[PATCH v13 11/12] drm/ttm/tests: Correct modules' licenses

2024-06-03 Thread Karolina Stolarek
The test files are GPL and MIT, so update the definitions accordingly. Signed-off-by: Karolina Stolarek --- drivers/gpu/drm/ttm/tests/ttm_bo_test.c | 2 +- drivers/gpu/drm/ttm/tests/ttm_device_test.c | 2 +- drivers/gpu/drm/ttm/tests/ttm_kunit_helpers.c | 2 +- drivers/gpu/drm/ttm/tests/

[PATCH v13 10/12] drm/ttm/tests: Add TODO file

2024-06-03 Thread Karolina Stolarek
List improvements for the test suite with some notes. Signed-off-by: Karolina Stolarek Reviewed-by: Thomas Hellström --- drivers/gpu/drm/ttm/tests/TODO | 27 +++ 1 file changed, 27 insertions(+) create mode 100644 drivers/gpu/drm/ttm/tests/TODO diff --git a/drivers/gpu

[PATCH v13 09/12] drm/ttm/tests: Add tests for ttm_tt_populate

2024-06-03 Thread Karolina Stolarek
Add tests for functions that add and release pages to TTs. Test the swapin operation. Export ttm_tt_unpopulate, ttm_tt_swapin and ttm_tt_swapout symbols for testing purposes. Signed-off-by: Karolina Stolarek Reviewed-by: Somalapuram, Amaranath Tested-by: Somalapuram, Amaranath --- drivers/gpu/

[PATCH v13 08/12] drm/ttm/tests: Add eviction testing

2024-06-03 Thread Karolina Stolarek
Add tests for ttm_bo_validate that focus on BO eviction and swapout. Update device funcs definition with eviction-related callbacks. Add alternative funcs where evict_flags() routes eviction to a domain that can't allocate resources (dubbed "busy manager" in the tests). Extract the common path of t

[PATCH v13 07/12] drm/ttm/tests: Add test cases dependent on fence signaling

2024-06-03 Thread Karolina Stolarek
Add test cases that check how the state of dma fences in BO's reservation object influence the ttm_bo_validation() flow. Do similar tests for resource manager's move fence. Signed-off-by: Karolina Stolarek Reviewed-by: Somalapuram, Amaranath Tested-by: Somalapuram, Amaranath --- .../gpu/drm/tt

[PATCH v13 06/12] drm/ttm/tests: Add tests with mock resource managers

2024-06-03 Thread Karolina Stolarek
Add mock resource manager to test ttm_bo_validate() with non-system placements. Update KConfig entry to enable DRM Buddy allocator, used by the mock manager. Update move function to do more than just assign a resource. Signed-off-by: Karolina Stolarek Tested-by: Somalapuram, Amaranath --- drive

[PATCH v13 05/12] drm/ttm/tests: Test simple BO creation and validation

2024-06-03 Thread Karolina Stolarek
Add tests for ttm_bo_init_reserved() and ttm_bo_validate() that use sys manager. Define a simple move function in ttm_device_funcs. Expose destroy callback of the buffer object to make testing of ttm_bo_init_reserved() behaviour easier. Signed-off-by: Karolina Stolarek Reviewed-by: Matthew Auld

[PATCH v13 04/12] drm/ttm/tests: Use an init function from the helpers lib

2024-06-03 Thread Karolina Stolarek
Add a new helper function that also initializes the device. Use it in ttm_tt test suite and delete the local definition. Signed-off-by: Karolina Stolarek Reviewed-by: Matthew Auld Reviewed-by: Somalapuram, Amaranath --- drivers/gpu/drm/ttm/tests/ttm_kunit_helpers.c | 14 ++ drivers

[PATCH v13 03/12] drm/ttm/tests: Set DMA mask in KUnit device

2024-06-03 Thread Karolina Stolarek
In commit d393acce7b3f ("drm/tests: Switch to kunit devices"), DRM test helpers migrated away from using a dummy platform driver in favour of KUnit device. This means that DMA masks for the device are not set but are required by ttm_pool_alloc tests. Set the DMA mask for coherent mappings to unblo

[PATCH v13 02/12] drm/ttm/tests: Delete unnecessary config option

2024-06-03 Thread Karolina Stolarek
DRM KUnit helpers are selected automatically when TTM tests are enabled, so there's no need to do it directly in the .kunitconfig file. Signed-off-by: Karolina Stolarek Reviewed-by: Nirmoy Das --- drivers/gpu/drm/ttm/tests/.kunitconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/

[PATCH v13 01/12] drm/ttm/tests: Fix a warning in ttm_bo_unreserve_bulk

2024-06-03 Thread Karolina Stolarek
BOs in a bulk move have to share the same reservation object. That is not the case in the ttm_bo_unreserve_bulk subtest. Update ttm_bo_kunit_init() helper to accept dma_resv object so we can define buffer objects that share the same resv. Update calls to that helper accordingly. Fixes: 995279d280d

[PATCH v13 00/12] Improve test coverage of TTM

2024-06-03 Thread Karolina Stolarek
Introduce tests for ttm_bo_validate()/ttm_bo_init_validate() that exercise simple BO placement as well as eviction (including the case where the evict domain also requires eviction to fit the incoming buffer). Prepare KUnit helpers to handle such scenarios and add a mock VRAM manager. This series a

Re: [PATCH 00/20] iommu: Refactoring domain allocation interface

2024-06-03 Thread Jason Gunthorpe
On Wed, May 29, 2024 at 08:02:12PM +0800, Baolu Lu wrote: > > > drivers/infiniband/hw/usnic/usnic_uiom.c:   pd->domain = domain > > > = iommu_domain_alloc(dev->bus); > > > > > > This series leave those cases unchanged and keep iommu_domain_alloc() > > > for their usage. But new drivers should

Re: [PATCH 00/14] Add initial support for the Rockchip RK3588 HDMI TX Controller

2024-06-03 Thread neil . armstrong
Hi, On 03/06/2024 15:03, Heiko Stuebner wrote: Am Montag, 3. Juni 2024, 14:14:17 CEST schrieb Andy Yan: Hi Neil: On 6/3/24 16:55, Neil Armstrong wrote: Hi Christian, On 01/06/2024 15:12, Cristian Ciocaltea wrote: The RK3588 SoC family integrates a Quad-Pixel (QP) variant of the Synopsys Des

Re: [PATCH 00/14] Add initial support for the Rockchip RK3588 HDMI TX Controller

2024-06-03 Thread Heiko Stuebner
Am Montag, 3. Juni 2024, 14:14:17 CEST schrieb Andy Yan: > Hi Neil: > > On 6/3/24 16:55, Neil Armstrong wrote: > > Hi Christian, > > > > On 01/06/2024 15:12, Cristian Ciocaltea wrote: > >> The RK3588 SoC family integrates a Quad-Pixel (QP) variant of the > >> Synopsys DesignWare HDMI TX controlle

Re: [PATCH] drm/vmwgfx: Don't destroy Screen Target when CRTC is enabled but inactive

2024-06-03 Thread Martin Krastev
On Fri, May 31, 2024 at 11:34 PM Ian Forbes wrote: > > drm_crtc_helper_funcs::atomic_disable can be called even when the CRTC is > still enabled. This can occur when the mode changes or the CRTC is set as > inactive. > > In the case where the CRTC is being set as inactive we only want to > blank t

Re: [PATCH net-next v10 01/14] netdev: add netdev_rx_queue_restart()

2024-06-03 Thread Pavel Begunkov
On 5/30/24 21:16, Mina Almasry wrote: Add netdev_rx_queue_restart() function to netdev_rx_queue.h Signed-off-by: David Wei Signed-off-by: Mina Almasry --- ... diff --git a/net/core/netdev_rx_queue.c b/net/core/netdev_rx_queue.c new file mode 100644 index 0..b3899358e5a9c --- /de

  1   2   >