Re: [Mesa-dev] Freenode fallout

2021-05-21 Thread Wladislav Artsimovich
Indeed, a lot of communities are moving to Libera. However, centralization is what caused the mess with freenode in the first place, thus communities spreading to different services may not be such a bad idea. As long as the channels themselves are not split, that is. On 21/05/2021 15:33,

Re: [PATCH 11/11] drm/tiny: drm_gem_simple_display_pipe_prepare_fb is the default

2021-05-21 Thread Noralf Trønnes
Den 21.05.2021 11.09, skrev Daniel Vetter: > Goes through all the drivers and deletes the default hook since it's > the default now. > > Signed-off-by: Daniel Vetter Acked-by: Noralf Trønnes

Re: [Freedreno] [RFC PATCH 00/13] drm/msm: Add Display Stream Compression Support

2021-05-21 Thread Jeffrey Hugo
On Fri, May 21, 2021 at 6:50 AM Vinod Koul wrote: > > Display Stream Compression (DSC) compresses the display stream in host which > is later decoded by panel. This series enables this for Qualcomm msm driver. > This was tested on Google Pixel3 phone which use LGE SW43408 panel. > > The changes

Re: [PATCH 11/11] drm/tiny: drm_gem_simple_display_pipe_prepare_fb is the default

2021-05-21 Thread David Lechner
On 5/21/21 4:09 AM, Daniel Vetter wrote: Goes through all the drivers and deletes the default hook since it's the default now. Acked-by: David Lechner

Re: [PATCH] drm/amdgpu: Fix inconsistent indenting

2021-05-21 Thread Christian König
Am 21.05.21 um 11:50 schrieb Jiapeng Chong: Eliminate the follow smatch warning: drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c:449 sdma_v5_0_ring_emit_mem_sync() warn: inconsistent indenting. Reported-by: Abaci Robot Signed-off-by: Jiapeng Chong Reviewed-by: Christian König ---

Re: Freenode fallout

2021-05-21 Thread Simon Ser
On Friday, May 21st, 2021 at 1:49 AM, Lyude Paul wrote: > After considering Libera and OFTC as options, the board settled on > recommending OFTC. The primary reason for this is because OFTC is > associated with our parent foundation SPI, and has a long and well known > history of involvement

Re: [RFC PATCH 02/13] dt-bindings: msm/dsi: Document Display Stream Compression (DSC) parameters

2021-05-21 Thread Vinod Koul
On 21-05-21, 08:18, Rob Herring wrote: > On Fri, May 21, 2021 at 7:50 AM Vinod Koul wrote: > > > > DSC enables streams to be compressed before we send to panel. This > > requires DSC enabled encoder and a panel to be present. So we add this > > information in board DTS and find if DSC can be

Re: [Linaro-mm-sig] [PATCH 04/11] drm/panfrost: Fix implicit sync

2021-05-21 Thread Daniel Stone
On Fri, 21 May 2021 at 14:09, Christian König wrote: > Am 21.05.21 um 14:54 schrieb Daniel Stone: > > If you're curious, the interface definitions are in the csf/ directory > > in the 'Bifrost kernel driver' r30p0 download you can get from the Arm > > developer site. Unfortunately the exact

[PATCH] drm/fb-helper: improve DRM fbdev emulation device names

2021-05-21 Thread Javier Martinez Canillas
Framebuffer devices that are registered by DRM drivers for fbdev emulation have a "drmfb" suffix in their name. But makes them to be quite confusing if a driver already has "drm" in its name: $ cat /proc/fb 0 rockchipdrmdrmfb $ cat /proc/fb 0 simpledrmdrmfb Instead, let's just add a "-fb"

Re: [RFC PATCH 02/13] dt-bindings: msm/dsi: Document Display Stream Compression (DSC) parameters

2021-05-21 Thread Rob Herring
On Fri, May 21, 2021 at 7:50 AM Vinod Koul wrote: > > DSC enables streams to be compressed before we send to panel. This > requires DSC enabled encoder and a panel to be present. So we add this > information in board DTS and find if DSC can be enabled and the > parameters required to configure

Re: [Linaro-mm-sig] [PATCH 04/11] drm/panfrost: Fix implicit sync

2021-05-21 Thread Christian König
Am 21.05.21 um 14:54 schrieb Daniel Stone: On Fri, 21 May 2021 at 13:28, Christian König wrote: Am 21.05.21 um 14:22 schrieb Daniel Stone: Yeah, the 'second-generation Valhall' GPUs coming later this year / early next year are starting to get pretty weird. Firmware-mediated job scheduling out

Re: [PATCH] hgafb: fix probe function

2021-05-21 Thread Greg Kroah-Hartman
On Mon, May 17, 2021 at 11:50:32AM +0300, Dan Carpenter wrote: > There is a reversed if statement in this probe function so the driver is > completely broken. > > Fixes: dc13cac4862c ("video: hgafb: fix potential NULL pointer dereference") > Signed-off-by: Dan Carpenter > --- >

Re: [Linaro-mm-sig] [PATCH 04/11] drm/panfrost: Fix implicit sync

2021-05-21 Thread Daniel Stone
On Fri, 21 May 2021 at 13:28, Christian König wrote: > Am 21.05.21 um 14:22 schrieb Daniel Stone: > > Yeah, the 'second-generation Valhall' GPUs coming later this year / > > early next year are starting to get pretty weird. Firmware-mediated > > job scheduling out of multiple queues, userspace

[RFC PATCH 13/13] drm/msm/dsi: Pass DSC params to drm_panel

2021-05-21 Thread Vinod Koul
When DSC is enabled, we need to pass the DSC parameters to panel driver as well, so add a dsc parameter in panel and set it when DSC is enabled Signed-off-by: Vinod Koul --- drivers/gpu/drm/msm/dsi/dsi_host.c | 5 + include/drm/drm_panel.h| 7 +++ 2 files changed, 12

[RFC PATCH 12/13] drm/msm/dsi: Add support for DSC configuration

2021-05-21 Thread Vinod Koul
When DSC is enabled, we need to configure DSI registers accordingly and configure the respective stream compression registers. Add support to calculate the register setting based on DSC params and timing information and configure these registers. Signed-off-by: Vinod Koul ---

[RFC PATCH 11/13] drm/msm/disp/dpu1: Add support for DSC in topology

2021-05-21 Thread Vinod Koul
For DSC to work we typically need a 2,2,1 configuration. This should suffice for resolutions upto 4k. For more resolutions like 8k this won't work. Furthermore, we can use 1 DSC encoder in lesser resulutions, but that is not power efficient according to Abhinav, so it is recommended to always use

[RFC PATCH 10/13] drm/msm/disp/dpu1: Add support for DSC in encoder

2021-05-21 Thread Vinod Koul
When DSC is enabled in DT, we need to configure the encoder for DSC configuration, calculate DSC parameters for the given timing. This patch adds that support by adding dpu_encoder_prep_dsc() which is invoked when DSC is enabled in DT Signed-off-by: Vinod Koul ---

[RFC PATCH 09/13] drm/msm/disp/dpu1: Dont use DSC with mode_3d

2021-05-21 Thread Vinod Koul
We cannot enable mode_3d when we are using the DSC. So pass configuration to detect DSC is enabled and not enable mode_3d when we are using DSC We add a helper dpu_encoder_helper_get_dsc_mode() to detect dsc enabled and pass this to .setup_intf_cfg() Signed-off-by: Vinod Koul ---

[RFC PATCH 09/13] drm/msm/disp/dpu1: Don't use DSC with mode_3d

2021-05-21 Thread Vinod Koul
We cannot enable mode_3d when we are using the DSC. So pass configuration to detect DSC is enabled and not enable mode_3d when we are using DSC We add a helper dpu_encoder_helper_get_dsc_mode() to detect dsc enabled and pass this to .setup_intf_cfg() Signed-off-by: Vinod Koul ---

[RFC PATCH 08/13] drm/msm/disp/dpu1: Add DSC support in hw_ctl

2021-05-21 Thread Vinod Koul
Later gens of hardware have DSC bits moved to hw_ctl, so configure these bits so that DSC would work there as well Signed-off-by: Vinod Koul --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git

[RFC PATCH 07/13] drm/msm/disp/dpu1: Add DSC for SDM845 to hw_catalog

2021-05-21 Thread Vinod Koul
This add SDM845 DSC blocks into hw_catalog Signed-off-by: Vinod Koul --- .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c| 22 +++ 1 file changed, 22 insertions(+) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c index

[RFC PATCH 06/13] drm/msm/disp/dpu1: Add DSC support in RM

2021-05-21 Thread Vinod Koul
This add the bits in RM to enable the DSC blocks Signed-off-by: Vinod Koul --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 1 + drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 32 + drivers/gpu/drm/msm/disp/dpu1/dpu_rm.h | 1 + 3 files changed, 34 insertions(+) diff --git

[RFC PATCH 05/13] drm/msm/dsi: add support for dsc data

2021-05-21 Thread Vinod Koul
DSC needs some configuration from device tree, add support to read and store these params and add DSC structures in msm_drv Signed-off-by: Vinod Koul --- drivers/gpu/drm/msm/dsi/dsi_host.c | 171 + drivers/gpu/drm/msm/msm_drv.h | 32 ++ 2 files changed, 203

[RFC PATCH 05/13] drm/msm/disp/dpu1: Add support for DSC in pingpong block

2021-05-21 Thread Vinod Koul
In SDM845, DSC can be enabled by writing to pingpong block registers, so add support for DSC in hw_pp Signed-off-by: Vinod Koul --- .../gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.c | 32 +++ .../gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.h | 14 2 files changed, 46

[RFC PATCH 04/13] drm/msm/disp/dpu1: Add support for DSC

2021-05-21 Thread Vinod Koul
Display Stream Compression (DSC) is one of the hw blocks in dpu, so add support by adding hw blocks for DSC Signed-off-by: Vinod Koul --- drivers/gpu/drm/msm/Makefile | 1 + .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h| 26 +++ drivers/gpu/drm/msm/disp/dpu1/dpu_hw_dsc.c

[RFC PATCH 04/13] drm/msm/disp/dpu1: Add support for DSC in pingpong block

2021-05-21 Thread Vinod Koul
In SDM845, DSC can be enabled by writing to pingpong block registers, so add support for DSC in hw_pp Signed-off-by: Vinod Koul --- .../gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.c | 32 +++ .../gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.h | 14 2 files changed, 46

[RFC PATCH 03/13] drm/msm/dsi: add support for dsc data

2021-05-21 Thread Vinod Koul
DSC needs some configuration from device tree, add support to read and store these params and add DSC structures in msm_drv Signed-off-by: Vinod Koul --- drivers/gpu/drm/msm/dsi/dsi_host.c | 170 + drivers/gpu/drm/msm/msm_drv.h | 32 ++ 2 files changed, 202

[RFC PATCH 03/13] drm/msm/disp/dpu1: Add support for DSC

2021-05-21 Thread Vinod Koul
Display Stream Compression (DSC) is one of the hw blocks in dpu, so add support by adding hw blocks for DSC Signed-off-by: Vinod Koul --- drivers/gpu/drm/msm/Makefile | 1 + .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h| 26 +++ drivers/gpu/drm/msm/disp/dpu1/dpu_hw_dsc.c

[RFC PATCH 02/13] dt-bindings: msm/dsi: Document Display Stream Compression (DSC) parameters

2021-05-21 Thread Vinod Koul
DSC enables streams to be compressed before we send to panel. This requires DSC enabled encoder and a panel to be present. So we add this information in board DTS and find if DSC can be enabled and the parameters required to configure DSC are added to binding document along with example

[RFC PATCH 01/13] drm/dsc: Add dsc pps header init function

2021-05-21 Thread Vinod Koul
We required a helper to create and set the dsc_dce_header, so add the dsc_dce_header and API drm_dsc_dsi_pps_header_init Signed-off-by: Vinod Koul --- drivers/gpu/drm/drm_dsc.c | 11 +++ include/drm/drm_dsc.h | 16 2 files changed, 27 insertions(+) diff --git

[RFC PATCH 00/13] drm/msm: Add Display Stream Compression Support

2021-05-21 Thread Vinod Koul
Display Stream Compression (DSC) compresses the display stream in host which is later decoded by panel. This series enables this for Qualcomm msm driver. This was tested on Google Pixel3 phone which use LGE SW43408 panel. The changes include adding DT properties for DSC then hardware blocks

Re: [PATCH 06/11] drm/: drm_gem_plane_helper_prepare_fb is now the default

2021-05-21 Thread Heiko Stübner
Am Freitag, 21. Mai 2021, 11:09:54 CEST schrieb Daniel Vetter: > No need to set it explicitly. > > Signed-off-by: Daniel Vetter > Cc: Laurentiu Palcu > Cc: Lucas Stach > Cc: Shawn Guo > Cc: Sascha Hauer > Cc: Pengutronix Kernel Team > Cc: Fabio Estevam > Cc: NXP Linux Team > Cc: Philipp

[PATCH -next] drm: Fix PM reference leak

2021-05-21 Thread Zou Wei
pm_runtime_get_sync will increment pm usage counter even it failed. Forgetting to putting operation will result in reference leak here. Fix it by replacing it with pm_runtime_resume_and_get to keep usage counter balanced. Reported-by: Hulk Robot Signed-off-by: Zou Wei ---

Re: [RFC 7/7] drm/i915: Expose client engine utilisation via fdinfo

2021-05-21 Thread Christian König
Am 21.05.21 um 14:26 schrieb Tvrtko Ursulin: On 20/05/2021 18:47, Daniel Vetter wrote: On Thu, May 20, 2021 at 6:31 PM Christian König wrote: Yeah, having the timestamp is a good idea as well.    drm-driver: i915 I think we should rather add something like printing

Re: [Linaro-mm-sig] [PATCH 04/11] drm/panfrost: Fix implicit sync

2021-05-21 Thread Christian König
Am 21.05.21 um 14:22 schrieb Daniel Stone: Hi, On Fri, 21 May 2021 at 10:10, Daniel Vetter wrote: Currently this has no practial relevance I think because there's not many who can pull off a setup with panfrost and another gpu in the same system. But the rules are that if you're setting an

Re: [RFC 7/7] drm/i915: Expose client engine utilisation via fdinfo

2021-05-21 Thread Tvrtko Ursulin
On 20/05/2021 18:47, Daniel Vetter wrote: On Thu, May 20, 2021 at 6:31 PM Christian König wrote: Yeah, having the timestamp is a good idea as well. drm-driver: i915 I think we should rather add something like printing file_operations->owner->name to the common fdinfo code. This way

Re: [PATCH 04/11] drm/panfrost: Fix implicit sync

2021-05-21 Thread Daniel Stone
Hi, On Fri, 21 May 2021 at 10:10, Daniel Vetter wrote: > Currently this has no practial relevance I think because there's not > many who can pull off a setup with panfrost and another gpu in the > same system. But the rules are that if you're setting an exclusive > fence, indicating a gpu write

Re: [PATCH 06/11] drm/: drm_gem_plane_helper_prepare_fb is now the default

2021-05-21 Thread Paul Cercueil
Hi Daniel, Le ven., mai 21 2021 at 11:09:54 +0200, Daniel Vetter a écrit : No need to set it explicitly. Signed-off-by: Daniel Vetter Cc: Laurentiu Palcu Cc: Lucas Stach Cc: Shawn Guo Cc: Sascha Hauer Cc: Pengutronix Kernel Team Cc: Fabio Estevam Cc: NXP Linux Team Cc: Philipp Zabel

Re: [Intel-gfx] [RFC 2/2] drm/doc/rfc: i915 new parallel submission uAPI plan

2021-05-21 Thread Tvrtko Ursulin
On 20/05/2021 20:41, Daniel Vetter wrote: On Thu, May 20, 2021 at 11:57:44AM +0100, Tvrtko Ursulin wrote: On 20/05/2021 10:54, Daniel Vetter wrote: On Wed, May 19, 2021 at 7:19 PM Matthew Brost wrote: On Wed, May 19, 2021 at 01:10:04PM +0200, Daniel Vetter wrote: On Tue, May 18, 2021 at

Re: [Intel-gfx] [RFC 2/2] drm/doc/rfc: i915 new parallel submission uAPI plan

2021-05-21 Thread Tvrtko Ursulin
On 19/05/2021 00:58, Matthew Brost wrote: Add entry fpr i915 new parallel submission uAPI plan. v2: (Daniel Vetter): - Expand logical order explaination - Add dummy header - Only allow N BBs in execbuf IOCTL - Configure parallel submission per slot not per gem context Cc:

Re: [PATCH 0/8] drm + usb-type-c: Add support for out-of-band hotplug notification (v3)

2021-05-21 Thread Heikki Krogerus
On Tue, May 11, 2021 at 10:05:26AM +0300, Heikki Krogerus wrote: > On Wed, May 05, 2021 at 06:24:07PM +0200, Hans de Goede wrote: > > Hi All, > > > > Here is v3 of my patchset making DP over Type-C work on devices where the > > Type-C controller does not drive the HPD pin on the GPU, but instead

Re: [igt-dev] [PATCH i-g-t] tests/kms_big_fb: Wait for vblank before collecting CRC

2021-05-21 Thread Juha-Pekka Heikkila
Hi Vidya, on which machines this would help? I see there's many vblanks already being waited. There's igt_display_commit2 which probably will block and even if it didn't there's igt_pipe_crc_collect_crc(..) where crc calculation is started after flip and then get one crc before disabling crc

Re: [PATCH 01/11] drm/amdgpu: Comply with implicit fencing rules

2021-05-21 Thread Christian König
Am 21.05.21 um 11:09 schrieb Daniel Vetter: Docs for struct dma_resv are fairly clear: "A reservation object can have attached one exclusive fence (normally associated with write operations) or N shared fences (read operations)."

Re: [PATCH] drm/panfrost: Fix the panfrost_mmu_map_fault_addr() error path

2021-05-21 Thread Steven Price
On 21/05/2021 10:38, Boris Brezillon wrote: > Make sure all bo->base.pages entries are either NULL or pointing to a > valid page before calling drm_gem_shmem_put_pages(). > > Reported-by: Tomeu Vizoso > Cc: > Fixes: 187d2929206e ("drm/panfrost: Add support for GPU heap allocations") >

[PATCH v17 3/4] dt-bindings: msm: dsi: add yaml schemas for DSI PHY bindings

2021-05-21 Thread Krishna Manikandan
Add YAML schema for the device tree bindings for DSI PHY. Signed-off-by: Krishna Manikandan Changes in v1: - Merge dsi-phy.yaml and dsi-phy-10nm.yaml (Stephen Boyd) - Remove qcom,dsi-phy-regulator-ldo-mode (Stephen Boyd) - Add clock cells properly (Stephen Boyd) - Remove unnecessary

[PATCH v17 2/4] dt-bindings: msm: dsi: add yaml schemas for DSI bindings

2021-05-21 Thread Krishna Manikandan
Add YAML schema for the device tree bindings for DSI Signed-off-by: Krishna Manikandan Changes in v1: - Separate dsi controller bindings to a separate patch (Stephen Boyd) - Merge dsi-common-controller.yaml and dsi-controller-main.yaml to a single file (Stephen Boyd) - Drop

[PATCH v17 1/4] dt-bindings: msm: disp: add yaml schemas for DPU bindings

2021-05-21 Thread Krishna Manikandan
MSM Mobile Display Subsystem (MDSS) encapsulates sub-blocks like DPU display controller, DSI etc. Add YAML schema for DPU device tree bindings. Signed-off-by: Krishna Manikandan Changes in v2: - Changed dpu to DPU (Sam Ravnborg) - Fixed indentation issues (Sam Ravnborg) - Added

[PATCH v17 4/4] dt-bindings: msm/dp: Add bindings of MSM DisplayPort controller

2021-05-21 Thread Krishna Manikandan
Add bindings for Snapdragon DisplayPort controller driver. Signed-off-by: Chandan Uddaraju Signed-off-by: Vara Reddy Signed-off-by: Tanmay Shah Signed-off-by: Kuogee Hsieh Signed-off-by: Krishna Manikandan Changes in V2: -Provide details about sel-gpio Changes in V4: -Provide details about

[PATCH] drm/amdgpu: Fix inconsistent indenting

2021-05-21 Thread Jiapeng Chong
Eliminate the follow smatch warning: drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c:449 sdma_v5_0_ring_emit_mem_sync() warn: inconsistent indenting. Reported-by: Abaci Robot Signed-off-by: Jiapeng Chong --- drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c | 13 ++--- 1 file changed, 6 insertions(+), 7

Re: [PATCH 01/11] drm/amdgpu: Comply with implicit fencing rules

2021-05-21 Thread Bas Nieuwenhuizen
On Fri, May 21, 2021 at 11:10 AM Daniel Vetter wrote: > > Docs for struct dma_resv are fairly clear: > > "A reservation object can have attached one exclusive fence (normally > associated with write operations) or N shared fences (read > operations)." > >

Re: [PATCH v5 0/7] dma-buf: Performance improvements for system heap & a system-uncached implementation

2021-05-21 Thread Lee Jones
On Tue, 10 Nov 2020 at 03:49, John Stultz wrote: > Hey All, > So just wanted to send my last revision of my patch series > of performance optimizations to the dma-buf system heap. > > This series reworks the system heap to use sgtables, and then > consolidates the pagelist method from the

Re: [PATCH 06/11] drm/: drm_gem_plane_helper_prepare_fb is now the default

2021-05-21 Thread Lucas Stach
Am Freitag, dem 21.05.2021 um 11:09 +0200 schrieb Daniel Vetter: > No need to set it explicitly. > > Signed-off-by: Daniel Vetter > Cc: Laurentiu Palcu > Cc: Lucas Stach For dcss and imx-drm: Acked-by: Lucas Stach > Cc: Shawn Guo > Cc: Sascha Hauer > Cc: Pengutronix Kernel Team > Cc:

[PATCH] drm/panfrost: Fix the panfrost_mmu_map_fault_addr() error path

2021-05-21 Thread Boris Brezillon
Make sure all bo->base.pages entries are either NULL or pointing to a valid page before calling drm_gem_shmem_put_pages(). Reported-by: Tomeu Vizoso Cc: Fixes: 187d2929206e ("drm/panfrost: Add support for GPU heap allocations") Signed-off-by: Boris Brezillon ---

Re: [RFC] Add DMA_RESV_USAGE flags

2021-05-21 Thread Bas Nieuwenhuizen
On Thu, May 20, 2021 at 9:15 PM Daniel Vetter wrote: > > On Thu, May 20, 2021 at 9:04 PM Jason Ekstrand wrote: > > > > On Thu, May 20, 2021 at 12:23 PM Jason Ekstrand > > wrote: > > > > > > On Thu, May 20, 2021 at 5:50 AM Christian König > > > wrote: > > > > > > > > Am 20.05.21 um 09:55

Re: [PATCH 08/11] drm/vram-helpers: Create DRM_GEM_VRAM_PLANE_HELPER_FUNCS

2021-05-21 Thread tiantao (H)
在 2021/5/21 17:09, Daniel Vetter 写道: Like we have for the shadow helpers too, and roll it out to drivers. Signed-off-by: Daniel Vetter Cc: Dave Airlie Cc: Thomas Zimmermann Cc: Hans de Goede Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: David Airlie Cc: Daniel Vetter Cc: Tian Tao Cc:

Re: [PATCH 02/11] drm/panfrost: Remove sched_lock

2021-05-21 Thread Lucas Stach
Am Freitag, dem 21.05.2021 um 11:09 +0200 schrieb Daniel Vetter: > Scheduler takes care of its own locking, dont worry. For everything else > there's reservation locking on each bo. > > So seems to be entirely redundnant and just a bit in the way. I haven't read all the surrounding code, but

[PATCH] drm/gem: Tiny kernel clarification for drm_gem_fence_array_add

2021-05-21 Thread Daniel Vetter
Spotted while trying to convert panfrost to these. Signed-off-by: Daniel Vetter Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Thomas Zimmermann Cc: David Airlie Cc: Daniel Vetter --- drivers/gpu/drm/drm_gem.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/drm_gem.c

[PATCH 09/11] drm/omap: Follow implicit fencing in prepare_fb

2021-05-21 Thread Daniel Vetter
I guess no one ever tried running omap together with lima or panfrost, not even sure that's possible. Anyway for consistency, fix this. Signed-off-by: Daniel Vetter Cc: Tomi Valkeinen --- drivers/gpu/drm/omapdrm/omap_plane.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

[PATCH 11/11] drm/tiny: drm_gem_simple_display_pipe_prepare_fb is the default

2021-05-21 Thread Daniel Vetter
Goes through all the drivers and deletes the default hook since it's the default now. Signed-off-by: Daniel Vetter Cc: Joel Stanley Cc: Andrew Jeffery Cc: "Noralf Trønnes" Cc: Linus Walleij Cc: Emma Anholt Cc: David Lechner Cc: Kamlesh Gurudasani Cc: Oleksandr Andrushchenko Cc: Daniel

[PATCH 07/11] drm/armada: Remove prepare/cleanup_fb hooks

2021-05-21 Thread Daniel Vetter
All they do is refcount the fb, which the atomic helpers already do. This is was necessary with the legacy helpers and I guess just carry over in the conversion. drm_plane_state always has a full reference for its ->fb pointer during its entire lifetime, see

[PATCH 05/11] drm/atomic-helper: make drm_gem_plane_helper_prepare_fb the default

2021-05-21 Thread Daniel Vetter
There's a bunch of atomic drivers who don't do this quite correctly, luckily most of them aren't in wide use or people would have noticed the tearing. By making this the default we avoid the constant audit pain and can additionally remove a ton of lines from vfuncs for a bit more clarity in

[PATCH 10/11] drm/simple-helper: drm_gem_simple_display_pipe_prepare_fb as default

2021-05-21 Thread Daniel Vetter
It's tedious to review this all the time, and my audit showed that arcpgu actually forgot to set this. Make this the default and stop worrying. Again I sprinkled WARN_ON_ONCE on top to make sure we don't have strange combinations of hooks: cleanup_fb without prepare_fb doesn't make sense, and

[PATCH 08/11] drm/vram-helpers: Create DRM_GEM_VRAM_PLANE_HELPER_FUNCS

2021-05-21 Thread Daniel Vetter
Like we have for the shadow helpers too, and roll it out to drivers. Signed-off-by: Daniel Vetter Cc: Dave Airlie Cc: Thomas Zimmermann Cc: Hans de Goede Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: David Airlie Cc: Daniel Vetter Cc: Tian Tao Cc: Laurent Pinchart ---

[PATCH 04/11] drm/panfrost: Fix implicit sync

2021-05-21 Thread Daniel Vetter
Currently this has no practial relevance I think because there's not many who can pull off a setup with panfrost and another gpu in the same system. But the rules are that if you're setting an exclusive fence, indicating a gpu write access in the implicit fencing system, then you need to wait for

[PATCH 06/11] drm/: drm_gem_plane_helper_prepare_fb is now the default

2021-05-21 Thread Daniel Vetter
No need to set it explicitly. Signed-off-by: Daniel Vetter Cc: Laurentiu Palcu Cc: Lucas Stach Cc: Shawn Guo Cc: Sascha Hauer Cc: Pengutronix Kernel Team Cc: Fabio Estevam Cc: NXP Linux Team Cc: Philipp Zabel Cc: Paul Cercueil Cc: Chun-Kuang Hu Cc: Matthias Brugger Cc: Neil Armstrong

[PATCH 03/11] drm/panfrost: Use xarray and helpers for depedency tracking

2021-05-21 Thread Daniel Vetter
More consistency and prep work for the next patch. Aside: I wonder whether we shouldn't just move this entire xarray business into the scheduler so that not everyone has to reinvent the same wheels. Cc'ing some scheduler people for this too. Cc: "Christian König" Cc: Luben Tuikov Cc: Alex

[PATCH 01/11] drm/amdgpu: Comply with implicit fencing rules

2021-05-21 Thread Daniel Vetter
Docs for struct dma_resv are fairly clear: "A reservation object can have attached one exclusive fence (normally associated with write operations) or N shared fences (read operations)." https://dri.freedesktop.org/docs/drm/driver-api/dma-buf.html#reservation-objects Furthermore a review across

[PATCH 02/11] drm/panfrost: Remove sched_lock

2021-05-21 Thread Daniel Vetter
Scheduler takes care of its own locking, dont worry. For everything else there's reservation locking on each bo. So seems to be entirely redundnant and just a bit in the way. Signed-off-by: Daniel Vetter Cc: Rob Herring Cc: Tomeu Vizoso Cc: Steven Price Cc: Alyssa Rosenzweig ---

[PATCH v2] drm/exynos: Use pm_runtime_resume_and_get() to replace open coding

2021-05-21 Thread Tian Tao
use pm_runtime_resume_and_get() to replace pm_runtime_get_sync and pm_runtime_put_noidle. Signed-off-by: Tian Tao --- v2: drop unnecessary change about if condition. --- drivers/gpu/drm/exynos/exynos_drm_mic.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

Re: [RFC PATCH 4/5] drm/ttm: Document and optimize ttm_bo_pipeline_gutting()

2021-05-21 Thread Thomas Hellström
On 5/21/21 10:21 AM, Christian König wrote: Am 20.05.21 um 17:09 schrieb Thomas Hellström: If the bo is idle when calling ttm_bo_pipeline_gutting(), we unnecessarily create a ghost object and push it out to delayed destroy. Fix this by adding a path for idle, and document the function. Also

Re: [PATCH] drm/ttm: Skip swapout if ttm object is not populated

2021-05-21 Thread Christian König
Am 21.05.21 um 10:31 schrieb xinhui pan: Swapping a ttm object which has no backend pages makes no sense. Suggested-by: Christian König Signed-off-by: xinhui pan Reviewed-by: Christian König Going to add a CC: stable and pushing that to drm-misc-fixes in a minute. ---

Re: [Intel-gfx] [Mesa-dev] [RFC 2/2] drm/doc/rfc: i915 new parallel submission uAPI plan

2021-05-21 Thread Christian König
Am 20.05.21 um 23:38 schrieb Jason Ekstrand: On Thu, May 20, 2021 at 10:46 AM Matthew Brost wrote: On Thu, May 20, 2021 at 01:11:59PM +0200, Christian König wrote: Am 19.05.21 um 18:51 schrieb Matthew Brost: On Wed, May 19, 2021 at 01:45:39PM +0200, Christian König wrote: Oh, yeah we call

[PATCH] drm/ttm: Skip swapout if ttm object is not populated

2021-05-21 Thread xinhui pan
Swapping a ttm object which has no backend pages makes no sense. Suggested-by: Christian König Signed-off-by: xinhui pan --- drivers/gpu/drm/ttm/ttm_device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/ttm/ttm_device.c b/drivers/gpu/drm/ttm/ttm_device.c

Re: [RFC PATCH 3/5] drm/ttm: Use drm_memcpy_from_wc for TTM bo moves

2021-05-21 Thread Thomas Hellström
On 5/21/21 10:10 AM, Christian König wrote: Am 20.05.21 um 17:09 schrieb Thomas Hellström: Use fast wc memcpy for reading out of wc memory for TTM bo moves. Cc: Dave Airlie Cc: Christian König Cc: Daniel Vetter Signed-off-by: Thomas Hellström Oh, yes I really wanted to have that in TTM

Re: [PATCH] drm/doc: Includ fence chain api

2021-05-21 Thread Christian König
Am 21.05.21 um 10:24 schrieb Daniel Vetter: We have this nice kerneldoc, but forgot to include it. Well I didn't forgot it, I just didn't had time to double check that it is bug free :) Signed-off-by: Daniel Vetter Cc: Sumit Semwal Cc: "Christian König" Cc: linux-me...@vger.kernel.org

[PATCH] drm/doc: Includ fence chain api

2021-05-21 Thread Daniel Vetter
We have this nice kerneldoc, but forgot to include it. Signed-off-by: Daniel Vetter Cc: Sumit Semwal Cc: "Christian König" Cc: linux-me...@vger.kernel.org Cc: linaro-mm-...@lists.linaro.org --- Documentation/driver-api/dma-buf.rst | 9 + 1 file changed, 9 insertions(+) diff --git

Re: [RFC PATCH 4/5] drm/ttm: Document and optimize ttm_bo_pipeline_gutting()

2021-05-21 Thread Christian König
Am 20.05.21 um 17:09 schrieb Thomas Hellström: If the bo is idle when calling ttm_bo_pipeline_gutting(), we unnecessarily create a ghost object and push it out to delayed destroy. Fix this by adding a path for idle, and document the function. Also avoid having the bo end up in a bad state

Re: [RFC PATCH 3/5] drm/ttm: Use drm_memcpy_from_wc for TTM bo moves

2021-05-21 Thread Christian König
Am 20.05.21 um 17:09 schrieb Thomas Hellström: Use fast wc memcpy for reading out of wc memory for TTM bo moves. Cc: Dave Airlie Cc: Christian König Cc: Daniel Vetter Signed-off-by: Thomas Hellström Oh, yes I really wanted to have that in TTM for quite some time. But I'm wondering if we

Re: [RFC PATCH 1/5] drm/ttm: Add a generic TTM memcpy move for page-based iomem

2021-05-21 Thread Christian König
Am 20.05.21 um 17:09 schrieb Thomas Hellström: The internal ttm_bo_util memcpy uses ioremap functionality, and while it probably might be possible to use it for copying in- and out of sglist represented io memory, using io_mem_reserve() / io_mem_free() callbacks, that would cause problems with

Re: [PATCH 1/4] dma-buf: add dma_fence_array_for_each (v2)

2021-05-21 Thread Christian König
Am 20.05.21 um 21:00 schrieb Jason Ekstrand: From: Christian König Add a helper to iterate over all fences in a dma_fence_array object. v2 (Jason Ekstrand) - Return NULL from dma_fence_array_first if head == NULL. This matches the iterator behavior of dma_fence_chain_for_each in that

Re: [Linaro-mm-sig] [RFC 1/3] dma-fence: Add boost fence op

2021-05-21 Thread Christian König
Am 20.05.21 um 19:08 schrieb Daniel Vetter: [SNIP] AH! So we are basically telling the fence backend that we have just missed an event we waited for. So what we want to know is how long the frontend wanted to wait instead of how long the backend took for rendering. tbh I'm not sure the

Re: [RFC] Add DMA_RESV_USAGE flags

2021-05-21 Thread Christian König
Am 20.05.21 um 21:14 schrieb Daniel Vetter: On Thu, May 20, 2021 at 9:04 PM Jason Ekstrand wrote: On Thu, May 20, 2021 at 12:23 PM Jason Ekstrand wrote: On Thu, May 20, 2021 at 5:50 AM Christian König wrote: Am 20.05.21 um 09:55 schrieb Daniel Vetter: On Wed, May 19, 2021 at 5:48 PM

Re: [RFC] Add DMA_RESV_USAGE flags

2021-05-21 Thread Christian König
Am 20.05.21 um 19:23 schrieb Jason Ekstrand: [SNIP] I'd argue then that making amdgpu poll semantics match those of other drivers is a pre-requisite for the new ioctl, otherwise it seems unlikely that the ioctl will be widely adopted. This seems backwards, because that means useful

RE: [Intel-gfx] NVIDIA GPU fallen off the bus after exiting s2idle

2021-05-21 Thread Saarinen, Jani
Hi, > -Original Message- > From: Intel-gfx On Behalf Of Chris > Chiu > Sent: perjantai 21. toukokuuta 2021 7.02 > To: Rafael J. Wysocki > Cc: Brown, Len ; Karol Herbst ; Linux > PM ; Linux PCI ; > Westerberg, Mika ; Rafael J. Wysocki > ; dri-devel ; Bjorn > Helgaas > ;

Re: [PATCH 38/38] drm/amd/amdgpu/smuio_v13_0: Realign 'smuio_v13_0_is_host_gpu_xgmi_supported()' header

2021-05-21 Thread Lee Jones
On Thu, 20 May 2021, Alex Deucher wrote: > Applied. Thanks! Thanks again Alex. > On Thu, May 20, 2021 at 8:03 AM Lee Jones wrote: > > > > Fixes the following W=1 kernel build warning(s): > > > > drivers/gpu/drm/amd/amdgpu/smuio_v13_0.c:99: warning: expecting prototype > > for

Re: [Nouveau] [Intel-gfx] [PATCH 0/7] Per client engine busyness

2021-05-21 Thread arabek
> Well if it becomes a problem fixing the debugfs "clients" file and > making it sysfs shouldn't be much of a problem later on. Why not to try using something in terms of perf / opensnoop or bpf to do the work. Should be optimal enough. ie.

[PATCH] drm/msm/dpu: remove unused variable cmd_enc

2021-05-21 Thread Austin Kim
After the call to to_dpu_encoder_phys_cmd() is made, 'cmd_enc' is not used. Where to_dpu_encoder_phys_cmd() is simply replaced with container_of(x, struct dpu_encoder_phys_cmd, base) by compiler. So it had better remove W=1 kernel build warning(s):

Re: [PATCH] video: hgafb: correctly handle card detect failure during probe

2021-05-21 Thread Anirudh Rayabharam
On Mon, May 17, 2021 at 12:57:14AM +0530, Anirudh Rayabharam wrote: > The return value of hga_card_detect() is not properly handled causing > the probe to succeed even though hga_card_detect() failed. Since probe > succeeds, hgafb_open() can be called which will end up operating on an > unmapped

Re: [PATCH v8 5/8] mm: Device exclusive memory access

2021-05-21 Thread Alistair Popple
On Tuesday, 18 May 2021 11:19:05 PM AEST Alistair Popple wrote: [...] > > > +/* > > > + * Restore a potential device exclusive pte to a working pte entry > > > + */ > > > +static vm_fault_t remove_device_exclusive_entry(struct vm_fault *vmf) > > > +{ > > > + struct page *page = vmf->page; >

Re: [git pull] drm fixes for 5.13-rc3

2021-05-21 Thread pr-tracker-bot
The pull request you sent on Fri, 21 May 2021 14:31:53 +1000: > git://anongit.freedesktop.org/drm/drm tags/drm-fixes-2021-05-21-1 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/79a106fc6585979022012e65a1e45e3d2d28b77b Thank you! -- Deet-doot-dot, I am a bot.

<    1   2