Re: [PATCH v2 2/3] drm/msm/disp/dpu1: add helper to know if display is pluggable

2022-11-16 Thread Dmitry Baryshkov
On 16/11/2022 17:30, Kalyan Thota wrote: Since DRM encoder type for few encoders can be similar (like eDP and DP) find out if the interface supports HPD from encoder bridge to differentiate between builtin and pluggable displays. Changes in v1: - add connector type in the disp_info (Dmitry) -

[PATCH 1/2] drm/tests: client: Remove extra blank lines

2022-11-16 Thread Maxime Ripard
Some extra blank lines slipped through, remove them. Fixes: 8fc0380f6ba7 ("drm/client: Add some tests for drm_connector_pick_cmdline_mode()") Signed-off-by: Maxime Ripard --- drivers/gpu/drm/tests/drm_client_modeset_test.c | 2 -- 1 file changed, 2 deletions(-) diff --git

Re: [PATCH v2 2/3] drm/msm/disp/dpu1: add helper to know if display is pluggable

2022-11-16 Thread Dmitry Baryshkov
On 16/11/2022 18:11, Abhinav Kumar wrote: On 11/16/2022 7:08 AM, Dmitry Baryshkov wrote: On 16/11/2022 17:30, Kalyan Thota wrote: Since DRM encoder type for few encoders can be similar (like eDP and DP) find out if the interface supports HPD from encoder bridge to differentiate between

[PATCH 2/2] drm/tests: helpers: Add SPDX header

2022-11-16 Thread Maxime Ripard
The SPDX header is missing, let's add it and fix the corresponding checkpatch warning. Suggested-by: Maíra Canal Fixes: 44a3928324e9 ("drm/tests: Add Kunit Helpers") Signed-off-by: Maxime Ripard --- drivers/gpu/drm/tests/drm_kunit_helpers.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [PATCH v2 2/3] drm/msm/disp/dpu1: add helper to know if display is pluggable

2022-11-16 Thread Dmitry Baryshkov
On 16/11/2022 18:35, Abhinav Kumar wrote: On 11/16/2022 7:18 AM, Dmitry Baryshkov wrote: On 16/11/2022 18:11, Abhinav Kumar wrote: On 11/16/2022 7:08 AM, Dmitry Baryshkov wrote: On 16/11/2022 17:30, Kalyan Thota wrote: Since DRM encoder type for few encoders can be similar (like eDP and

Re: [PATCH 2/2] drm/tests: helpers: Add SPDX header

2022-11-16 Thread Maíra Canal
Hi Maxime, On 11/16/22 12:18, Maxime Ripard wrote: > The SPDX header is missing, let's add it and fix the corresponding > checkpatch warning. > > Suggested-by: Maíra Canal > Fixes: 44a3928324e9 ("drm/tests: Add Kunit Helpers") > Signed-off-by: Maxime Ripard Could you add the

Re: [PATCH v2 2/3] drm/etnaviv: allocate unique ID per drm_file

2022-11-16 Thread Lucas Stach
Am Freitag, dem 16.09.2022 um 17:12 +0200 schrieb Lucas Stach: > Allows to easily track if several fd are pointing to the same > execution context due to being dup'ed. > I would appreciate it if someone could review path 2 and 3 of this series. I would like to include it in my next upstream PR.

[PATCH RESEND v4 0/2] Use devm helpers for regulator get and enable

2022-11-16 Thread Matti Vaittinen
Simplify couple of drivers by using the new devm_regulator_*get_enable*() Found these patches when doing some clean-up for my local git. Seems like these two fell through the cracks while other were merged. So, this is a respin of subset of original series v4. These patches were previously

[PATCH RESEND v4 1/2] gpu: drm: sii902x: Use devm_regulator_bulk_get_enable()

2022-11-16 Thread Matti Vaittinen
Simplify using devm_regulator_bulk_get_enable() Signed-off-by: Matti Vaittinen Acked-by: Robert Foss --- I am doing a clean-up for my local git and encountered this one. Respinning as it seems this one fell through the cracks. --- drivers/gpu/drm/bridge/sii902x.c | 26

[PATCH RESEND v4 2/2] gpu: drm: meson: Use devm_regulator_*get_enable*()

2022-11-16 Thread Matti Vaittinen
Simplify using the devm_regulator_get_enable_optional(). Also drop the seemingly unused struct member 'hdmi_supply'. Signed-off-by: Matti Vaittinen --- I am doing a clean-up for my local git and encountered this one. Respinning as it seems this one fell through the cracks. ---

Re: [PATCH i-g-t 8/8] gputop: Basic vendor agnostic GPU top tool

2022-11-16 Thread Philipp Zabel
On Fr, 2022-11-11 at 15:58 +, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > Rudimentary vendor agnostic example of how lib_igt_drm_clients can be used > to display a sorted by card and usage list of processes using GPUs. > > Borrows a bit of code from intel_gpu_top but for now omits the

[Bug 216624] The system freezes when it reaches the screen to ask password for LUKS

2022-11-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=216624 --- Comment #13 from The Linux kernel's regression tracker (Thorsten Leemhuis) (regressi...@leemhuis.info) --- FWIW, the nouveau developers might not see this report here, they want bugs filed here:

Re: [PATCH 2/3] drm/tests: helpers: Add module infos

2022-11-16 Thread Maxime Ripard
On Wed, Nov 16, 2022 at 01:32:51PM +0200, Jani Nikula wrote: > On Wed, 16 Nov 2022, Maíra Canal wrote: > > Hi Maxime, > > > > On 11/16/22 06:17, Maxime Ripard wrote: > >> The MODULE_LICENSE macro is missing from the kunit helpers file, thus > >> leading to a build error. > >> > >> Let's

Re: (subset) [PATCH 2/3] drm/tests: helpers: Add module infos

2022-11-16 Thread Maxime Ripard
On Wed, 16 Nov 2022 10:17:11 +0100, Maxime Ripard wrote: > The MODULE_LICENSE macro is missing from the kunit helpers file, thus > leading to a build error. > > Let's introduce it along with MODULE_AUTHOR. > > Applied to local tree (tmp). Thanks! Maxime

[PATCH 3/7] drm/cirrus: Decouple fbdev bpp value from color depth

2022-11-16 Thread Thomas Zimmermann
Cirrus has a preferred color depth of 16 bit; also use it as fbdev bpp value. Don't use the color depth directly. It has a different meaning than bpp and both cannot be used interchangeably. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/tiny/cirrus.c | 2 +- 1 file changed, 1

[PATCH 7/7] drm/fb-helper: Don't use the preferred depth for the BPP default

2022-11-16 Thread Thomas Zimmermann
If no preferred value for bits-per-pixel has been given, fall back to 32. Never use the preferred depth. The color depth is the number of color/alpha bits per pixel, while bpp is the overall number of bits in most cases. Most noteworthy, XRGB has a depth of 24 and a bpp value of 32. Using

[PATCH 4/7] drm/ofdrm: Set preferred depth from format of scanout buffer

2022-11-16 Thread Thomas Zimmermann
Set the preferred depth from the format of the scanout buffer. The value cannot be hardcoded, as the scanout buffer's is only known at runtime. Keeping the existing switch statement just duplicates the driver's existing logic for format detection. Also remove the FIXME comment from the call to

[PATCH 1/7] drm/hisilicon/hibmc: Fix preferred depth and bpp

2022-11-16 Thread Thomas Zimmermann
Set the preferred color depth to 24 bits and the fbdev bpp to 32 bits. This will signal XRGB as default format to clients. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 6/7] drm/solomon: Set preferred color depth and bpp to the correct values

2022-11-16 Thread Thomas Zimmermann
Set the preferred color depth to 24 bits and the fbdev bpp to 32 bits. This will signal XRGB as default format to clients. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/solomon/ssd130x.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 5/7] drm/simpledrm: Set preferred depth from format of scanout buffer

2022-11-16 Thread Thomas Zimmermann
Set the preferred depth from the format of the scanout buffer. The value cannot be hardcoded, as the scanout buffer's is only known at runtime. Also derive the fbdev emulations bpp value from the scanout format. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/tiny/simpledrm.c | 4 ++-- 1

[PATCH 2/7] drm/logicvc: Fix preferred fbdev cpp

2022-11-16 Thread Thomas Zimmermann
Logicvc can have different values for the preferred color depth. Set the fbdev bpp value depending on the runtime value. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/logicvc/logicvc_drm.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git

RE: [v1] drm/msm/disp/dpu1: add color management support for the crtc

2022-11-16 Thread Kalyan Thota
>-Original Message- >From: Dmitry Baryshkov >Sent: Wednesday, November 16, 2022 8:37 PM >To: Kalyan Thota ; Kalyan Thota (QUIC) >; dri-devel@lists.freedesktop.org; linux-arm- >m...@vger.kernel.org; freedr...@lists.freedesktop.org; >devicet...@vger.kernel.org >Cc:

[PATCH v2 0/3] add color management support for the crtc

2022-11-16 Thread Kalyan Thota
Add color management support for the crtc provided there are enough dspps that can be allocated from the catalog Kalyan Thota (3): drm/msm/disp/dpu1: pin 1 crtc to 1 encoder drm/msm/disp/dpu1: add helper to know if display is pluggable drm/msm/disp/dpu1: add color management support for the

[PATCH v2 2/3] drm/msm/disp/dpu1: add helper to know if display is pluggable

2022-11-16 Thread Kalyan Thota
Since DRM encoder type for few encoders can be similar (like eDP and DP) find out if the interface supports HPD from encoder bridge to differentiate between builtin and pluggable displays. Changes in v1: - add connector type in the disp_info (Dmitry) - add helper functions to know encoder type -

[PATCH v2 3/3] drm/msm/disp/dpu1: add color management support for the crtc

2022-11-16 Thread Kalyan Thota
Add color management support for the crtc provided there are enough dspps that can be allocated from the catalog. Changes in v1: - cache color enabled state in the dpu crtc obj (Dmitry) - simplify dspp allocation while creating crtc (Dmitry) - register for color when crtc is created (Dmitry)

[PATCH v2 1/3] drm/msm/disp/dpu1: pin 1 crtc to 1 encoder

2022-11-16 Thread Kalyan Thota
Pin each crtc with one encoder. This arrangement will disallow crtc switching between encoders and also will facilitate to advertise certain features on crtc based on encoder type. Changes in v1: - use drm_for_each_encoder macro while iterating through encoder list (Dmitry) Changes in v2: -

Re: [PATCH] drm/i915/guc: add the GSC CS to the GuC capture list

2022-11-16 Thread Teres Alexis, Alan Previn
I'm assuming that you have verified that the GuC ADS code is calling for the registration of the GSC capture register list accordingly and for the correct tile. That said: Reviewed-by: Alan Previn On Thu, 2022-11-10 at 16:18 -0800, Ceraolo Spurio, Daniele wrote: > For the GSC engine we only

Re: [PATCH 1/1] drm/i915/mtl: Enable Idle Messaging for GSC CS

2022-11-16 Thread Belgaumkar, Vinay
On 11/15/2022 5:44 AM, Badal Nilawar wrote: From: Vinay Belgaumkar By defaut idle mesaging is disabled for GSC CS so to unblock RC6 entry on media tile idle messaging need to be enabled. v2: - Fix review comments (Vinay) - Set GSC idle hysterisis to 5 us (Badal) Bspec: 71496 Cc:

Re: [RESEND] drm/edid/firmware: stop using a throwaway platform device

2022-11-16 Thread Matthieu CHARETTE
Thank you everyone for your work! Matthieu. On Wed, Nov 16 2022 at 03:32:01 PM +0200, Jani Nikula wrote: On Wed, 16 Nov 2022, Thomas Zimmermann wrote: Hi Am 14.11.22 um 12:17 schrieb Jani Nikula: We've used a temporary platform device for firmware EDID loading since it was introduced

Re: [PATCH mm-unstable v1 20/20] mm: rename FOLL_FORCE to FOLL_PTRACE

2022-11-16 Thread Linus Torvalds
On Wed, Nov 16, 2022 at 2:30 AM David Hildenbrand wrote: > > Let's make it clearer that functionality provided by FOLL_FORCE is > really only for ptrace access. I'm not super-happy about this one. I do understand the "let's rename the bit so that no new user shows up". And it's true that the

[PATCH 0/7] drm: Fix the color-depth/bpp confusion

2022-11-16 Thread Thomas Zimmermann
A number of drivers mix up the meaning of bits-per-pixel and color depth. For each of them, set the correct values. As a rule of thumb, the color depth is the number of color and alpha bits that affect image composition. The bpp value is the color depth in the pixel plus the filler bits. The

Re: [PATCH v3 05/10] dt-bindings: display: bridge: Add MHDP DP for i.MX8MQ

2022-11-16 Thread Rob Herring
On Tue, Nov 08, 2022 at 09:00:08PM +0800, Sandor Yu wrote: > Add bindings for i.MX8MQ MHDP DisplayPort. > > Signed-off-by: Sandor Yu > --- > .../display/bridge/cdns,mhdp-imx8mq-dp.yaml | 59 +++ > 1 file changed, 59 insertions(+) > create mode 100644 >

Re: [PATCH v1] dt-bindings: display: Convert fsl,imx-fb.txt to dt-schema

2022-11-16 Thread Philipp Zabel
On Thu, Nov 10, 2022 at 10:49:45AM +0100, Uwe Kleine-König wrote: [...] > new file mode 100644 > index ..c3cf6f92a766 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/imx/fsl,imx-lcdc.yaml > @@ -0,0 +1,110 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)

RE: [v1] drm/msm/disp/dpu1: add color management support for the crtc

2022-11-16 Thread Kalyan Thota
>-Original Message- >From: Dmitry Baryshkov >Sent: Saturday, November 12, 2022 4:13 AM >To: Kalyan Thota (QUIC) ; dri- >de...@lists.freedesktop.org; linux-arm-...@vger.kernel.org; >freedr...@lists.freedesktop.org; devicet...@vger.kernel.org >Cc: linux-ker...@vger.kernel.org;

Re: [v1] drm/msm/disp/dpu1: add color management support for the crtc

2022-11-16 Thread Dmitry Baryshkov
On 16/11/2022 17:29, Kalyan Thota wrote: -Original Message- From: Dmitry Baryshkov Sent: Saturday, November 12, 2022 4:13 AM To: Kalyan Thota (QUIC) ; dri- de...@lists.freedesktop.org; linux-arm-...@vger.kernel.org; freedr...@lists.freedesktop.org; devicet...@vger.kernel.org Cc:

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Never return 0 if request wait succeeds

2022-11-16 Thread Janusz Krzysztofik
On Wednesday, 16 November 2022 15:42:46 CET Andrzej Hajda wrote: > On 16.11.2022 12:25, Janusz Krzysztofik wrote: > > According to the docs of i915_request_wait_timeout(), its return value > > "may be zero if the request is unfinished after the timeout expires." > > However, 0 is also returned

[PATCH printk v5 00/40] reduce console_lock scope

2022-11-16 Thread John Ogness
This is v5 of a series to prepare for threaded/atomic printing. v4 is here [0]. This series focuses on reducing the scope of the BKL console_lock. It achieves this by switching to SRCU and a dedicated mutex for console list iteration and modification, respectively. The console_lock will no longer

[PATCH printk v5 32/40] printk, xen: fbfront: create/use safe function for forcing preferred

2022-11-16 Thread John Ogness
With commit 9e124fe16ff2("xen: Enable console tty by default in domU if it's not a dummy") a hack was implemented to make sure that the tty console remains the console behind the /dev/console device. The main problem with the hack is that, after getting the console pointer to the tty console, it

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Never return 0 if request wait succeeds

2022-11-16 Thread Andrzej Hajda
On 16.11.2022 12:25, Janusz Krzysztofik wrote: According to the docs of i915_request_wait_timeout(), its return value "may be zero if the request is unfinished after the timeout expires." However, 0 is also returned when the request is found finished right after the timeout has expired. Since

Re: [PATCH 2/3] drm/tests: helpers: Add module infos

2022-11-16 Thread Jani Nikula
On Wed, 16 Nov 2022, Maxime Ripard wrote: > On Wed, Nov 16, 2022 at 01:32:51PM +0200, Jani Nikula wrote: >> On Wed, 16 Nov 2022, Maíra Canal wrote: >> > Hi Maxime, >> > >> > On 11/16/22 06:17, Maxime Ripard wrote: >> >> The MODULE_LICENSE macro is missing from the kunit helpers file, thus >> >>

Re: [PATCH v2 2/3] drm/msm/disp/dpu1: add helper to know if display is pluggable

2022-11-16 Thread Abhinav Kumar
On 11/16/2022 7:18 AM, Dmitry Baryshkov wrote: On 16/11/2022 18:11, Abhinav Kumar wrote: On 11/16/2022 7:08 AM, Dmitry Baryshkov wrote: On 16/11/2022 17:30, Kalyan Thota wrote: Since DRM encoder type for few encoders can be similar (like eDP and DP) find out if the interface supports HPD

Re: [PATCH v3 02/10] dt-bindings: display: bridge: Add MHDP HDMI for i.MX8MQ

2022-11-16 Thread Rob Herring
On Tue, Nov 08, 2022 at 09:00:05PM +0800, Sandor Yu wrote: > Add bindings for i.MX8MQ MHDP HDMI. > > Signed-off-by: Sandor Yu > --- > .../display/bridge/cdns,mhdp-imx8mq-hdmi.yaml | 59 +++ > 1 file changed, 59 insertions(+) > create mode 100644 >

Re: [PATCH 1/2] drm/tests: client: Remove extra blank lines

2022-11-16 Thread Maíra Canal
On 11/16/22 12:18, Maxime Ripard wrote: > Some extra blank lines slipped through, remove them. > > Fixes: 8fc0380f6ba7 ("drm/client: Add some tests for > drm_connector_pick_cmdline_mode()") > Signed-off-by: Maxime Ripard Reviewed-by: Maíra Canal Best Regards, - Maíra Canal > --- >

RE: [v1] drm/msm/disp/dpu1: pin 1 crtc to 1 encoder

2022-11-16 Thread Kalyan Thota
>-Original Message- >From: Dmitry Baryshkov >Sent: Saturday, November 12, 2022 3:51 AM >To: Kalyan Thota (QUIC) ; dri- >de...@lists.freedesktop.org; linux-arm-...@vger.kernel.org; >freedr...@lists.freedesktop.org; devicet...@vger.kernel.org >Cc: linux-ker...@vger.kernel.org;

Re: [PATCH v2 2/3] drm/etnaviv: allocate unique ID per drm_file

2022-11-16 Thread Tvrtko Ursulin
On 16/11/2022 13:18, Philipp Zabel wrote: On Fri, Sep 16, 2022 at 05:12:04PM +0200, Lucas Stach wrote: Allows to easily track if several fd are pointing to the same execution context due to being dup'ed. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnaviv/etnaviv_drv.c | 3 +++

Re: [PATCH v2 01/12] dt-bindings: display: msm: Add qcom,sm8350-dpu binding

2022-11-16 Thread Rob Herring
On Tue, 15 Nov 2022 14:30:54 +0100, Robert Foss wrote: > Mobile Display Subsystem (MDSS) encapsulates sub-blocks > like DPU display controller, DSI etc. Add YAML schema for DPU device > tree bindings > > Signed-off-by: Robert Foss > --- > .../bindings/display/msm/qcom,sm8350-dpu.yaml | 120

Re: [PATCH v2 2/3] drm/msm/disp/dpu1: add helper to know if display is pluggable

2022-11-16 Thread Abhinav Kumar
On 11/16/2022 7:08 AM, Dmitry Baryshkov wrote: On 16/11/2022 17:30, Kalyan Thota wrote: Since DRM encoder type for few encoders can be similar (like eDP and DP) find out if the interface supports HPD from encoder bridge to differentiate between builtin and pluggable displays. Changes in v1:

Re: (subset) [PATCH 1/3] drm/tests: Include helpers header

2022-11-16 Thread Maxime Ripard
On Wed, 16 Nov 2022 10:17:10 +0100, Maxime Ripard wrote: > The kunit helpers code weren't including its header, leading to a > warning that no previous prototype had been defined for public > functions. > > Include the matching header to fix the warning. > > > [...] Applied to local tree

Re: [PATCH v3] drm/mediatek: Add AFBC support to Mediatek DRM driver

2022-11-16 Thread Justin Green
Hi Chun-Kuang, > > + mtk_ovl_set_afbc(dev, cmdq_pkt, idx, is_afbc); > > mtk_ddp_write_relaxed(cmdq_pkt, con, >cmdq_reg, ovl->regs, > > DISP_REG_OVL_CON(idx)); > > - mtk_ddp_write_relaxed(cmdq_pkt, pitch, >cmdq_reg, ovl->regs, > > +

[PATCH] dt-bindings: msm/dsi: Don't require vdds-supply on 10nm PHY

2022-11-16 Thread Konrad Dybcio
On some SoCs (hello SM6350) vdds-supply is not wired to any smd-rpm or rpmh regulator, but instead powered by the VDD_MX/mx.lvl line, which is voted for in the DSI ctrl node. Signed-off-by: Konrad Dybcio --- Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml | 1 - 1 file changed,

Re: [PATCH v1] dt-bindings: display: Convert fsl,imx-fb.txt to dt-schema

2022-11-16 Thread Rob Herring
On Thu, Nov 10, 2022 at 10:49:45AM +0100, Uwe Kleine-König wrote: > This is a straight forward conversion. Note that fsl,imx-lcdc was picked > as the new name as this is the compatible that should supersede the > legacy fb binding. > > Signed-off-by: Uwe Kleine-König > --- > Hello, > > the

Re: [Intel-gfx] [PATCH] drm/fb-helper: Try to protect cleanup against delayed setup

2022-11-16 Thread Daniel Vetter
On Tue, Nov 15, 2022 at 10:30:01AM +0100, Andrzej Hajda wrote: > On 13.07.2021 15:59, Daniel Vetter wrote: > > Some vague evidences suggests this can go wrong. Try to prevent it by > > holding the right mutex and clearing ->deferred_setup to make sure we > > later on don't accidentally try to

Re: [PATCH v3 6/8] drm/msm/dpu: add support for MDP_TOP blackhole

2022-11-16 Thread Abhinav Kumar
On 11/16/2022 1:43 AM, Dmitry Baryshkov wrote: On 16/11/2022 12:29, Abhinav Kumar wrote: On 11/16/2022 1:18 AM, Dmitry Baryshkov wrote: On 16/11/2022 11:30, Abhinav Kumar wrote: On 11/16/2022 12:19 AM, Dmitry Baryshkov wrote: On 16/11/2022 10:50, Abhinav Kumar wrote: On 11/4/2022

Re: [PATCH 1/3] drm/tests: Include helpers header

2022-11-16 Thread Maíra Canal
Hi Maxime, On 11/16/22 06:17, Maxime Ripard wrote: > The kunit helpers code weren't including its header, leading to a > warning that no previous prototype had been defined for public > functions. > > Include the matching header to fix the warning. > > Fixes: 44a3928324e9 ("drm/tests: Add Kunit

Re: [PATCH 2/3] drm/tests: helpers: Add module infos

2022-11-16 Thread Maíra Canal
Hi Maxime, On 11/16/22 06:17, Maxime Ripard wrote: > The MODULE_LICENSE macro is missing from the kunit helpers file, thus > leading to a build error. > > Let's introduce it along with MODULE_AUTHOR. > > Fixes: 44a3928324e9 ("drm/tests: Add Kunit Helpers") > Reported-by: Stephen Rothwell >

Re: [PATCH 2/3] drm/tests: helpers: Add module infos

2022-11-16 Thread Jani Nikula
On Wed, 16 Nov 2022, Maíra Canal wrote: > Hi Maxime, > > On 11/16/22 06:17, Maxime Ripard wrote: >> The MODULE_LICENSE macro is missing from the kunit helpers file, thus >> leading to a build error. >> >> Let's introduce it along with MODULE_AUTHOR. >> >> Fixes: 44a3928324e9 ("drm/tests: Add

Re: [PATCH 1/2] drm/ttm: Clean up page shift operation

2022-11-16 Thread Christian König
Am 16.11.22 um 09:50 schrieb Somalapuram Amaranath: remove page shift operations as ttm_resource moved from num_pages to size_t size in bytes. Signed-off-by: Somalapuram Amaranath --- drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 4 +--- drivers/gpu/drm/ttm/ttm_range_manager.c| 2 +-

Re: [Intel-gfx] How is the progress for removing flush_scheduled_work() callers?

2022-11-16 Thread Ville Syrjälä
On Wed, Nov 16, 2022 at 12:08:27PM +0200, Jani Nikula wrote: > On Sun, 06 Nov 2022, Tetsuo Handa wrote: > > Like commit c4f135d643823a86 ("workqueue: Wrap flush_workqueue() using a > > macro") says, flush_scheduled_work() is dangerous and will be forbidden. > > We are on the way for removing all

Re: [PATCH v2 2/3] drm/etnaviv: allocate unique ID per drm_file

2022-11-16 Thread Philipp Zabel
On Fri, Sep 16, 2022 at 05:12:04PM +0200, Lucas Stach wrote: > Allows to easily track if several fd are pointing to the same > execution context due to being dup'ed. > > Signed-off-by: Lucas Stach > --- > drivers/gpu/drm/etnaviv/etnaviv_drv.c | 3 +++ > drivers/gpu/drm/etnaviv/etnaviv_drv.h | 1

Re: [PATCH 4/5] drm/fb-helper: Schedule deferred-I/O worker after writing to framebuffer

2022-11-16 Thread Daniel Vetter
On Tue, Nov 15, 2022 at 11:05:02AM +0100, Thomas Zimmermann wrote: > Hi > > Am 11.11.22 um 10:28 schrieb Daniel Vetter: > > On Thu, Nov 10, 2022 at 02:55:18PM +0100, Thomas Zimmermann wrote: > > > Schedule the deferred-I/O worker instead of the damage worker after > > > writing to the fbdev

Re: [RESEND] drm/edid/firmware: stop using a throwaway platform device

2022-11-16 Thread Thomas Zimmermann
Hi Am 14.11.22 um 12:17 schrieb Jani Nikula: We've used a temporary platform device for firmware EDID loading since it was introduced in commit da0df92b5731 ("drm: allow loading an EDID as firmware to override broken monitor"), but there's no explanation why. Using a temporary device does not

[PATCH mm-unstable v1 20/20] mm: rename FOLL_FORCE to FOLL_PTRACE

2022-11-16 Thread David Hildenbrand
Let's make it clearer that functionality provided by FOLL_FORCE is really only for ptrace access. Prevent accidental re-use in drivers by renaming FOLL_FORCE to FOLL_PTRACE: git grep -l 'FOLL_FORCE' | xargs sed -i 's/FOLL_FORCE/FOLL_PTRACE/g' In the future, we might want to use a separate set

[PATCH mm-unstable v1 19/20] habanalabs: remove FOLL_FORCE usage

2022-11-16 Thread David Hildenbrand
FOLL_FORCE is really only for ptrace access. As we unpin the pinned pages using unpin_user_pages_dirty_lock(true), the assumption is that all these pages are writable. FOLL_FORCE in this case seems to be due to copy-and-past from other drivers. Let's just remove it. Acked-by: Oded Gabbay Cc:

Re: [PATCH v8 09/14] drm: bridge: samsung-dsim: Add atomic_get_input_bus_fmts

2022-11-16 Thread Marek Vasut
On 11/16/22 09:07, Marek Szyprowski wrote: On 15.11.2022 13:00, Marek Vasut wrote: On 11/14/22 08:49, Jagan Teki wrote: On Sun, Nov 13, 2022 at 5:51 AM Marek Vasut wrote: On 11/10/22 19:38, Jagan Teki wrote: Finding the right input bus format throughout the pipeline is hard so add

Re: [PATCH mm-unstable v1 13/20] media: videobuf-dma-sg: remove FOLL_FORCE usage

2022-11-16 Thread Daniel Vetter
On Wed, Nov 16, 2022 at 11:26:52AM +0100, David Hildenbrand wrote: > GUP now supports reliable R/O long-term pinning in COW mappings, such > that we break COW early. MAP_SHARED VMAs only use the shared zeropage so > far in one corner case (DAXFS file with holes), which can be ignored > because GUP

Re: [PATCH mm-unstable v1 14/20] drm/etnaviv: remove FOLL_FORCE usage

2022-11-16 Thread Daniel Vetter
On Wed, Nov 16, 2022 at 11:26:53AM +0100, David Hildenbrand wrote: > GUP now supports reliable R/O long-term pinning in COW mappings, such > that we break COW early. MAP_SHARED VMAs only use the shared zeropage so > far in one corner case (DAXFS file with holes), which can be ignored > because GUP

[PATCH 2/3] drm/i915: Never return 0 on timeout when retiring requests

2022-11-16 Thread Janusz Krzysztofik
Users of intel_gt_retire_requests_timeout() expect 0 return value on success. However, we have no protection from passing back 0 potentially returned by dma_fence_wait_timeout() on timeout. Replace 0 with -ETIME before using timeout as return value. Fixes: f33a8a51602c ("drm/i915: Merge

[PATCH 1/3] drm/i915: Fix negative remaining time after retire requests

2022-11-16 Thread Janusz Krzysztofik
Commit b97060a99b01 ("drm/i915/guc: Update intel_gt_wait_for_idle to work with GuC") extended the API of intel_gt_retire_requests_timeout() with an extra argument 'remaining_timeout', intended for passing back unconsumed portion of requested timeout when 0 (success) is returned. However, when

[PATCH 3/3] drm/i915: Never return 0 if request wait succeeds

2022-11-16 Thread Janusz Krzysztofik
According to the docs of i915_request_wait_timeout(), its return value "may be zero if the request is unfinished after the timeout expires." However, 0 is also returned when the request is found finished right after the timeout has expired. Since the docs also state: "If the timeout is 0, it will

[PATCH 0/3] drm/i915: Fix timeout handling when retiring requests

2022-11-16 Thread Janusz Krzysztofik
Fixes for issues discovered via code review while working on https://gitlab.freedesktop.org/drm/intel/issues/7349. Reworked version of a series supposed to fix the same issues, sent before under the same subject. Since some solutions are significantly different than before, I'm not marking this

Re: [PATCH] mm/migrate_device: Return number of migrating pages in args->cpages

2022-11-16 Thread Alistair Popple
Hi Andrew, I realize it's getting late in the cycle but any chance you could take this one as well for v6.1-rc6? It fixes a minor bug introduced in v6.1-rc1 which could cause some drivers to think they can't migrate pages when they can. Thanks. Ralph Campbell writes: > On 11/10/22 16:51,

Re: [PATCH] drm/msm/dp: remove limitation of link rate at 5.4G to support HBR3

2022-11-16 Thread Dmitry Baryshkov
On 15/11/2022 21:43, Kuogee Hsieh wrote: On 11/9/2022 11:43 PM, Dmitry Baryshkov wrote: On 10/11/2022 02:47, Kuogee Hsieh wrote: On 11/2/2022 11:04 AM, Dmitry Baryshkov wrote: On 02/11/2022 20:28, Doug Anderson wrote: Hi, On Wed, Nov 2, 2022 at 10:23 AM Dmitry Baryshkov wrote: 1.

Re: [PATCH v8 09/14] drm: bridge: samsung-dsim: Add atomic_get_input_bus_fmts

2022-11-16 Thread Marek Szyprowski
On 16.11.2022 11:49, Marek Vasut wrote: > On 11/16/22 09:07, Marek Szyprowski wrote: >> On 15.11.2022 13:00, Marek Vasut wrote: >>> On 11/14/22 08:49, Jagan Teki wrote: On Sun, Nov 13, 2022 at 5:51 AM Marek Vasut wrote: > > On 11/10/22 19:38, Jagan Teki wrote: >> Finding the

Re: [PATCH v2 3/3] drm/etnaviv: export client GPU usage statistics via fdinfo

2022-11-16 Thread Philipp Zabel
On Fri, Sep 16, 2022 at 05:12:05PM +0200, Lucas Stach wrote: > This exposes a accumulated GPU active time per client via the > fdinfo infrastructure. > > Signed-off-by: Lucas Stach > --- > v2: > - fix code style > - switch to raw seq_printf > - leave some breadcrumbs about the output format >

Re: [PATCH 17/26] drm: rcar-du: Remove #ifdef guards for PM related functions

2022-11-16 Thread Kieran Bingham
Quoting Paul Cercueil (2022-11-07 17:52:47) > Use the DEFINE_SIMPLE_DEV_PM_OPS() and pm_sleep_ptr() macros to handle > the .suspend/.resume callbacks. > > These macros allow the suspend and resume functions to be automatically > dropped by the compiler when CONFIG_SUSPEND is disabled, without

Re: [PATCH 2/2] drm/gem: Remove BUG_ON in drm_gem_private_object_init

2022-11-16 Thread Arunpravin Paneer Selvam
Hi Amar, On 11/16/2022 2:20 PM, Somalapuram Amaranath wrote: ttm_resource allocate size in bytes i.e less than page size. Signed-off-by: Somalapuram Amaranath --- drivers/gpu/drm/drm_gem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_gem.c

Re: linux-next: manual merge of the drm-misc tree with the drm-misc-fixes tree

2022-11-16 Thread Christian König
Am 16.11.22 um 01:25 schrieb Stephen Rothwell: Hi all, On Wed, 16 Nov 2022 10:47:52 +1100 Stephen Rothwell wrote: Today's linux-next merge of the drm-misc tree got a conflict in: drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c between commit: eca13f3c67b6 ("drm/amdgpu: use the last IB as

Re: [PATCH] drm/etnaviv: switch to PFN mappings

2022-11-16 Thread Daniel Vetter
On Wed, Nov 16, 2022 at 10:17:02AM +0100, Lucas Stach wrote: > Am Freitag, dem 09.09.2022 um 11:20 +0200 schrieb Lucas Stach: > > There is no reason to use page based mappings, as the established > > mappings are special driver mappings anyways and should not be > > handled like normal pages. > >

[PATCH mm-unstable v1 10/20] RDMA/umem: remove FOLL_FORCE usage

2022-11-16 Thread David Hildenbrand
GUP now supports reliable R/O long-term pinning in COW mappings, such that we break COW early. MAP_SHARED VMAs only use the shared zeropage so far in one corner case (DAXFS file with holes), which can be ignored because GUP does not support long-term pinning in fsdax (see check_vma_flags()).

[PATCH mm-unstable v1 11/20] RDMA/usnic: remove FOLL_FORCE usage

2022-11-16 Thread David Hildenbrand
GUP now supports reliable R/O long-term pinning in COW mappings, such that we break COW early. MAP_SHARED VMAs only use the shared zeropage so far in one corner case (DAXFS file with holes), which can be ignored because GUP does not support long-term pinning in fsdax (see check_vma_flags()).

[PATCH mm-unstable v1 04/20] mm: add early FAULT_FLAG_UNSHARE consistency checks

2022-11-16 Thread David Hildenbrand
For now, FAULT_FLAG_UNSHARE only applies to anonymous pages, which implies a COW mapping. Let's hide FAULT_FLAG_UNSHARE early if we're not dealing with a COW mapping, such that we treat it like a read fault as documented and don't have to worry about the flag throughout all fault handlers. While

[PATCH mm-unstable v1 06/20] mm: rework handling in do_wp_page() based on private vs. shared mappings

2022-11-16 Thread David Hildenbrand
We want to extent FAULT_FLAG_UNSHARE support to anything mapped into a COW mapping (pagecache page, zeropage, PFN, ...), not just anonymous pages. Let's prepare for that by handling shared mappings first such that we can handle private mappings last. While at it, use folio-based functions instead

[PATCH mm-unstable v1 07/20] mm: don't call vm_ops->huge_fault() in wp_huge_pmd()/wp_huge_pud() for private mappings

2022-11-16 Thread David Hildenbrand
If we already have a PMD/PUD mapped write-protected in a private mapping and we want to break COW either due to FAULT_FLAG_WRITE or FAULT_FLAG_UNSHARE, there is no need to inform the file system just like on the PTE path. Let's just split (->zap) + fallback in that case. This is a preparation

[PATCH mm-unstable v1 08/20] mm: extend FAULT_FLAG_UNSHARE support to anything in a COW mapping

2022-11-16 Thread David Hildenbrand
Extend FAULT_FLAG_UNSHARE to break COW on anything mapped into a COW (i.e., private writable) mapping and adjust the documentation accordingly. FAULT_FLAG_UNSHARE will now also break COW when encountering the shared zeropage, a pagecache page, a PFNMAP, ... inside a COW mapping, by properly

[PATCH mm-unstable v1 02/20] selftests/vm: cow: basic COW tests for non-anonymous pages

2022-11-16 Thread David Hildenbrand
Let's add basic tests for COW with non-anonymous pages in private mappings: write access should properly trigger COW and result in the private changes not being visible through other page mappings. Especially, add tests for: * Zeropage * Huge zeropage * Ordinary pagecache pages via memfd and

[PATCH mm-unstable v1 05/20] mm: add early FAULT_FLAG_WRITE consistency checks

2022-11-16 Thread David Hildenbrand
Let's catch abuse of FAULT_FLAG_WRITE early, such that we don't have to care in all other handlers and might get "surprises" if we forget to do so. Write faults without VM_MAYWRITE don't make any sense, and our maybe_mkwrite() logic could have hidden such abuse for now. Write faults without

Re: [PATCH mm-unstable v1 16/20] mm/frame-vector: remove FOLL_FORCE usage

2022-11-16 Thread Daniel Vetter
On Wed, Nov 16, 2022 at 11:26:55AM +0100, David Hildenbrand wrote: > FOLL_FORCE is really only for ptrace access. According to commit > 707947247e95 ("media: videobuf2-vmalloc: get_userptr: buffers are always > writable"), get_vaddr_frames() currently pins all pages writable as a > workaround for

Re: [PATCH v3] udmabuf: add vmap and vunmap methods to udmabuf_ops

2022-11-16 Thread Dmitry Osipenko
On 11/15/22 23:04, Lukasz Wiecaszek wrote: > The reason behind that patch is associated with videobuf2 subsystem > (or more genrally with v4l2 framework) and user created > dma buffers (udmabuf). In some circumstances > when dealing with V4L2_MEMORY_DMABUF buffers videobuf2 subsystem > wants to

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Never return 0 on timeout when retiring requests

2022-11-16 Thread Andrzej Hajda
On 16.11.2022 12:25, Janusz Krzysztofik wrote: Users of intel_gt_retire_requests_timeout() expect 0 return value on success. However, we have no protection from passing back 0 potentially returned by dma_fence_wait_timeout() on timeout. Replace 0 with -ETIME before using timeout as return

Re: [PATCH v3 6/8] drm/msm/dpu: add support for MDP_TOP blackhole

2022-11-16 Thread Dmitry Baryshkov
On 16/11/2022 11:30, Abhinav Kumar wrote: On 11/16/2022 12:19 AM, Dmitry Baryshkov wrote: On 16/11/2022 10:50, Abhinav Kumar wrote: On 11/4/2022 6:03 AM, Dmitry Baryshkov wrote: On sm8450 a register block was removed from MDP TOP. Accessing it during snapshotting results in NoC errors /

Re: [PATCH] drm/etnaviv: switch to PFN mappings

2022-11-16 Thread Lucas Stach
Am Freitag, dem 09.09.2022 um 11:20 +0200 schrieb Lucas Stach: > There is no reason to use page based mappings, as the established > mappings are special driver mappings anyways and should not be > handled like normal pages. > > Be consistent with what other drivers do and use raw PFN based >

[PATCH 2/3] drm/tests: helpers: Add module infos

2022-11-16 Thread Maxime Ripard
The MODULE_LICENSE macro is missing from the kunit helpers file, thus leading to a build error. Let's introduce it along with MODULE_AUTHOR. Fixes: 44a3928324e9 ("drm/tests: Add Kunit Helpers") Reported-by: Stephen Rothwell Signed-off-by: Maxime Ripard ---

[PATCH 1/3] drm/tests: Include helpers header

2022-11-16 Thread Maxime Ripard
The kunit helpers code weren't including its header, leading to a warning that no previous prototype had been defined for public functions. Include the matching header to fix the warning. Fixes: 44a3928324e9 ("drm/tests: Add Kunit Helpers") Reported-by: kernel test robot Signed-off-by: Maxime

[PATCH 3/3] firmware: raspberrypi: Fix type assignment

2022-11-16 Thread Maxime Ripard
We silently cast an unsigned int into a __le32 which makes sparse complain. Moreover, we never actually convert endianness between the CPU's and the expected little-endian value. Fix both at once by calling cpu_to_le32(). Fixes: 40c31955e4e9 ("firmware: raspberrypi: Provide a helper to query a

Re: [PATCH v3 6/8] drm/msm/dpu: add support for MDP_TOP blackhole

2022-11-16 Thread Abhinav Kumar
On 11/16/2022 1:18 AM, Dmitry Baryshkov wrote: On 16/11/2022 11:30, Abhinav Kumar wrote: On 11/16/2022 12:19 AM, Dmitry Baryshkov wrote: On 16/11/2022 10:50, Abhinav Kumar wrote: On 11/4/2022 6:03 AM, Dmitry Baryshkov wrote: On sm8450 a register block was removed from MDP TOP.

Re: [PATCH v3 6/8] drm/msm/dpu: add support for MDP_TOP blackhole

2022-11-16 Thread Dmitry Baryshkov
On 16/11/2022 12:29, Abhinav Kumar wrote: On 11/16/2022 1:18 AM, Dmitry Baryshkov wrote: On 16/11/2022 11:30, Abhinav Kumar wrote: On 11/16/2022 12:19 AM, Dmitry Baryshkov wrote: On 16/11/2022 10:50, Abhinav Kumar wrote: On 11/4/2022 6:03 AM, Dmitry Baryshkov wrote: On sm8450 a

[PATCH mm-unstable v1 00/20] mm/gup: remove FOLL_FORCE usage from drivers (reliable R/O long-term pinning)

2022-11-16 Thread David Hildenbrand
For now, we did not support reliable R/O long-term pinning in COW mappings. That means, if we would trigger R/O long-term pinning in MAP_PRIVATE mapping, we could end up pinning the (R/O-mapped) shared zeropage or a pagecache page. The next write access would trigger a write fault and replace the

[PATCH mm-unstable v1 01/20] selftests/vm: anon_cow: prepare for non-anonymous COW tests

2022-11-16 Thread David Hildenbrand
Originally, the plan was to have a separate tests for testing COW of non-anonymous (e.g., shared zeropage) pages. Turns out, that we'd need a lot of similar functionality and that there isn't a really good reason to separate it. So let's prepare for non-anon tests by renaming to "cow".

[PATCH mm-unstable v1 03/20] selftests/vm: cow: R/O long-term pinning reliability tests for non-anon pages

2022-11-16 Thread David Hildenbrand
Let's test whether R/O long-term pinning is reliable for non-anonymous memory: when R/O long-term pinning a page, the expectation is that we break COW early before pinning, such that actual write access via the page tables won't break COW later and end up replacing the R/O-pinned page in the page

Re: [PATCH 2/2] drm/gem: Remove BUG_ON in drm_gem_private_object_init

2022-11-16 Thread Christian König
Am 16.11.22 um 10:20 schrieb Arunpravin Paneer Selvam: Hi Amar, On 11/16/2022 2:20 PM, Somalapuram Amaranath wrote: ttm_resource allocate size in bytes i.e less than page size. Signed-off-by: Somalapuram Amaranath ---   drivers/gpu/drm/drm_gem.c | 2 +-   1 file changed, 1 insertion(+), 1

  1   2   >