Re: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-07 Thread Christian König
Hi guys, maybe it becomes clearer to understand when you see this as two different things: 1. The current location where the buffer is. 2. If the data inside the buffer can be accessed. The current location is returned back by dma_buf_map_attachment() and the result of it can be used to fill

Re: [PATCH 2/8] drm/ttm: move ttm_set_memory.h out of include

2020-10-07 Thread Christian König
Am 05.10.20 um 17:01 schrieb Ruhl, Michael J: -Original Message- From: dri-devel On Behalf Of Christian König Sent: Thursday, October 1, 2020 7:28 AM To: dri-devel@lists.freedesktop.org; ray.hu...@amd.com; airl...@gmail.com; dan...@ffwll.ch Subject: [PATCH 2/8] drm/ttm: move

[Bug 208981] trace with B550I AORUS PRO AX and AMD Ryzen 5 PRO 4650G

2020-10-07 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=208981 Tino Mettler (tino+ker...@tikei.de) changed: What|Removed |Added CC|

Re: [PATCH 4/8] drm/ttm: rename TTM caching enum

2020-10-07 Thread Christian König
Am 05.10.20 um 17:05 schrieb Ruhl, Michael J: -Original Message- From: dri-devel On Behalf Of Christian König Sent: Thursday, October 1, 2020 7:28 AM To: dri-devel@lists.freedesktop.org; ray.hu...@amd.com; airl...@gmail.com; dan...@ffwll.ch Subject: [PATCH 4/8] drm/ttm: rename TTM

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Jason Gunthorpe
On Mon, Oct 05, 2020 at 08:36:00PM -0700, Andrew Morton wrote: > On Mon, 5 Oct 2020 14:47:47 -0300 Jason Gunthorpe wrote: > > > Andrew please let me know if you need a resend > > Andrew is rather confused. > > Can we please identify the minimal patch(es) which are needed for 5.9 > and -stable?

Re: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-07 Thread Jason Gunthorpe
On Tue, Oct 06, 2020 at 11:22:14AM +0200, Daniel Vetter wrote: > > For reinstanting the pages you need: > > - dma_resv_lock, this prevents anyone else from issuing new moves or > anything like that > - dma_resv_get_excl + dma_fence_wait to wait for any pending moves to > finish. gpus

Re: [PATCH 2/4] dt-bindings: Use 'additionalProperties' instead of 'unevaluatedProperties'

2020-10-07 Thread Jonathan Cameron
On Mon, 5 Oct 2020 13:38:28 -0500 Rob Herring wrote: > In cases where we don't reference another schema, 'additionalProperties' > can be used instead. This is preferred for now as 'unevaluatedProperties' > support isn't implemented yet. > > In a few cases, this means adding some missing

Re: [PATCH v5 80/80] ARM: dts: bcm2711: Enable the display pipeline

2020-10-07 Thread Maxime Ripard
Hi Dave, On Fri, Oct 02, 2020 at 04:57:05PM +0100, Dave Stevenson wrote: > Hi Maxime > > On Fri, 2 Oct 2020 at 16:19, Maxime Ripard wrote: > > > > Hi Tim, > > > > On Thu, Oct 01, 2020 at 11:15:46AM +0100, Tim Gover wrote: > > > hdmi_enable_4k60=1 causes the firmware to select 3.3 GHz for the

Re: [PATCH v2] drm/msm/dp: add opp_table corner voting support base on dp_ink_clk rate

2020-10-07 Thread Stephen Boyd
Quoting Rajendra Nayak (2020-10-06 00:31:41) > > On 10/4/2020 3:56 AM, Kuogee Hsieh wrote: > > Set link rate by using OPP set rate api so that CX level will be set > > accordingly based on the link rate. > > > > Changes in v2: > > -- remove dev from dp_ctrl_put() parameters > > -- address review

Re: [PATCH 2/4] dt-bindings: Use 'additionalProperties' instead of 'unevaluatedProperties'

2020-10-07 Thread Guenter Roeck
On 10/5/20 11:38 AM, Rob Herring wrote: > In cases where we don't reference another schema, 'additionalProperties' > can be used instead. This is preferred for now as 'unevaluatedProperties' > support isn't implemented yet. > > In a few cases, this means adding some missing property definitions

Re: [PATCH 2/3] drm/msm: add DRM_MSM_GEM_SYNC_CACHE for non-coherent cache maintenance

2020-10-07 Thread Jonathan Marek
On 10/6/20 3:23 AM, Christoph Hellwig wrote: On Mon, Oct 05, 2020 at 10:35:43AM -0400, Jonathan Marek wrote: The cache synchronization doesn't have anything to do with IOMMU (for example: cache synchronization would be useful in cases where drm/msm doesn't use IOMMU). It has to do with doing

[PATCH 1/2] drm/i915/dpcd_bl: Skip testing control capability with force DPCD quirk

2020-10-07 Thread Kai-Heng Feng
HP DreamColor panel needs to be controlled via AUX interface. However, it has both DP_EDP_BACKLIGHT_BRIGHTNESS_AUX_SET_CAP and DP_EDP_BACKLIGHT_BRIGHTNESS_PWM_PIN_CAP set, so it fails to pass intel_dp_aux_display_control_capable() test. Skip the test if the panel has force DPCD quirk.

Re: [PATCH 1/4] dt-bindings: Add missing 'unevaluatedProperties'

2020-10-07 Thread Mathieu Poirier
On Mon, 5 Oct 2020 at 12:38, Rob Herring wrote: > > This doesn't yet do anything in the tools, but make it explicit so we can > check either 'unevaluatedProperties' or 'additionalProperties' is present > in schemas. > > 'unevaluatedProperties' is appropriate when including another schema (via >

Re: [Freedreno] [RESEND] Requests For Proposals for hosting XDC2021 are now open

2020-10-07 Thread Samuel Iglesias Gonsálvez
Deadline is November 1st, just in a few weeks! Don't forget to submit your XDC 2021 proposal to bo...@foundation.x.org . Sam On Thu, 2020-09-03 at 12:16 -0400, Lyude Paul wrote: > (Including a bunch more emails in the To: that got missed the first > time) > > Hello everyone! > > The X.org

Re: [PATCH 1/4] dt-bindings: Add missing 'unevaluatedProperties'

2020-10-07 Thread Ulf Hansson
On Mon, 5 Oct 2020 at 20:38, Rob Herring wrote: > > This doesn't yet do anything in the tools, but make it explicit so we can > check either 'unevaluatedProperties' or 'additionalProperties' is present > in schemas. > > 'unevaluatedProperties' is appropriate when including another schema (via >

Re: [PATCH rdma-next v5 0/4] Dynamicaly allocate SG table from the pages

2020-10-07 Thread Jason Gunthorpe
On Tue, Oct 06, 2020 at 12:41:22PM +0200, Daniel Vetter wrote: > On Mon, Oct 05, 2020 at 08:56:50PM -0300, Jason Gunthorpe wrote: > > On Sun, Oct 04, 2020 at 06:43:36PM +0300, Leon Romanovsky wrote: > > > This series extends __sg_alloc_table_from_pages to allow chaining of > > > new pages to

Re: [PATCH] drm/i915: Force DPCD backlight mode for HP CML 2020 system

2020-10-07 Thread Kai-Heng Feng
> On Apr 8, 2020, at 15:23, Jani Nikula wrote: > > On Tue, 07 Apr 2020, Kai-Heng Feng wrote: >> There's another Samsung OLED panel needs to use DPCD aux interface to >> control backlight. > > Acked-by: Jani Nikula David, Can you please merge this patch? Thanks. Kai-Heng > >> >>

Re: [PATCH 1/4] dt-bindings: Add missing 'unevaluatedProperties'

2020-10-07 Thread Guenter Roeck
On 10/5/20 11:38 AM, Rob Herring wrote: > This doesn't yet do anything in the tools, but make it explicit so we can > check either 'unevaluatedProperties' or 'additionalProperties' is present > in schemas. > > 'unevaluatedProperties' is appropriate when including another schema (via > '$ref') and

Re: [PATCH 4/4] dt-bindings: Explicitly allow additional properties in common schemas

2020-10-07 Thread Jonathan Cameron
On Mon, 5 Oct 2020 13:38:30 -0500 Rob Herring wrote: > In order to add meta-schema checks for additional/unevaluatedProperties > being present, all schema need to make this explicit. As common/shared > schema are included by other schemas, they should always allow for > additionalProperties. >

[PATCH] ALSA: hda/i915 - fix list corruption with concurrent probes

2020-10-07 Thread Kai Vehmanen
From: Takashi Iwai Current hdac_i915 uses a static completion instance to wait for i915 driver to complete the component bind. This design is not safe if multiple HDA controllers are active and communicating with different i915 instances, and can lead to list corruption and failed audio driver

Re: [PATCH 13/14] drm/msm: Drop struct_mutex in shrinker path

2020-10-07 Thread Hillf Danton
On Mon, 5 Oct 2020 20:40:12 Rob Clark wrote: > On Mon, Oct 5, 2020 at 5:44 PM Hillf Danton wrote: > > On Mon, 5 Oct 2020 18:17:01 Kristian H. Kristensen wrote: > > > On Mon, Oct 5, 2020 at 4:02 PM Daniel Vetter wrote: > > > > > > > > On Mon, Oct 05, 2020 at 05:24:19PM +0800, Hillf Danton

Re: [PATCH 4/4] dt-bindings: Explicitly allow additional properties in common schemas

2020-10-07 Thread Ulf Hansson
On Mon, 5 Oct 2020 at 20:38, Rob Herring wrote: > > In order to add meta-schema checks for additional/unevaluatedProperties > being present, all schema need to make this explicit. As common/shared > schema are included by other schemas, they should always allow for > additionalProperties. > >

Re: [PATCH 4/4] dt-bindings: Explicitly allow additional properties in common schemas

2020-10-07 Thread Dmitry Torokhov
On Mon, Oct 05, 2020 at 01:38:30PM -0500, Rob Herring wrote: > In order to add meta-schema checks for additional/unevaluatedProperties > being present, all schema need to make this explicit. As common/shared > schema are included by other schemas, they should always allow for >

[PATCH 1/4] soc / drm: mediatek: Move DDP component defines into mtk-mmsys.h

2020-10-07 Thread Enric Balletbo i Serra
From: Yongqiang Niu MMSYS is the driver which controls the routing of these DDP components, so the definition of the mtk_ddp_comp_id enum should be placed in mtk-mmsys.h Signed-off-by: Yongqiang Niu Reviewed-by: Chun-Kuang Hu Signed-off-by: Enric Balletbo i Serra --- This patch was

[PATCH 2/2] drm/dp: HP DreamColor panel brigntness fix

2020-10-07 Thread Kai-Heng Feng
HP DreamColor panel, which is used by new HP ZBook Studio, needs to use DPCD to control brightness. Signed-off-by: Kai-Heng Feng --- drivers/gpu/drm/drm_dp_helper.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/drm_dp_helper.c b/drivers/gpu/drm/drm_dp_helper.c index

Re: linux-next: build failure after merge of the hmm tree

2020-10-07 Thread Jason Gunthorpe
On Tue, Oct 06, 2020 at 08:35:08PM +1100, Stephen Rothwell wrote: > Hi all, > > After merging the hmm tree, today's linux-next build (arm > multi_v7_defconfig) failed like this: > > > Caused by commit > > 07da1223ec93 ("lib/scatterlist: Add support in dynamic allocation of SG > table from

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Jason Gunthorpe
On Tue, Oct 06, 2020 at 08:23:23AM +0200, Daniel Vetter wrote: > On Tue, Oct 6, 2020 at 1:41 AM Jason Gunthorpe wrote: > > > > On Tue, Oct 06, 2020 at 12:43:31AM +0200, Daniel Vetter wrote: > > > > > > iow I think I can outright delete the frame vector stuff. > > > > > > Ok this doesn't work,

Re: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-07 Thread Jason Gunthorpe
On Tue, Oct 06, 2020 at 07:24:30PM +0200, Daniel Vetter wrote: > On Tue, Oct 6, 2020 at 6:34 PM Daniel Vetter wrote: > > > > On Tue, Oct 06, 2020 at 12:49:56PM -0300, Jason Gunthorpe wrote: > > > On Tue, Oct 06, 2020 at 11:22:14AM +0200, Daniel Vetter wrote: > > > > > > > > For reinstanting the

[PATCH 0/4] soc: mediatek: Prepare MMSYS for DDP routing using tables

2020-10-07 Thread Enric Balletbo i Serra
Dear all, The following series are intended to prepare the mtk-mmsys driver to allow different DDP (Data Display Path) routing tables per SoC. Note that the series has been tested only on MT8173 platform and could break the display on MT2701 and MT2712 based devices. I kindly ask for someone

Re: [PATCH] dt-bindings: mxsfb: Add compatible for i.MX8MM

2020-10-07 Thread Marek Vasut
On 10/6/20 11:15 PM, Rob Herring wrote: > On Sun, 04 Oct 2020 00:48:01 +0200, Marek Vasut wrote: >> NXP's i.MX8MM has an LCDIF as well. >> >> Signed-off-by: Marek Vasut >> Cc: Fabio Estevam >> Cc: Guido Günther >> Cc: Lucas Stach >> Cc: NXP Linux Team >> Cc: Rob Herring >> Cc: Shawn Guo >>

Re: [PATCH v2 0/3] drm: commit_work scheduling

2020-10-07 Thread Qais Yousef
On 10/05/20 16:24, Rob Clark wrote: [...] > > RT planning and partitioning is not easy task for sure. You might want to > > consider using affinities too to get stronger guarantees for some tasks and > > prevent cross-talking. > > There is some cgroup stuff that is pinning SF and some other

Re: [PATCH] drm/i915: Force DPCD backlight mode for HP Spectre x360 Convertible 13t-aw100

2020-10-07 Thread Kai-Heng Feng
> On Apr 8, 2020, at 15:22, Jani Nikula wrote: > > On Tue, 07 Apr 2020, Kai-Heng Feng wrote: >>> On Mar 27, 2020, at 19:03, Kai-Heng Feng >>> wrote: >>> >>> Hi, >>> On Mar 23, 2020, at 13:35, Kai-Heng Feng wrote: There's another OLED panel needs to use DPCD aux

Re: [PATCH 1/4] dt-bindings: Add missing 'unevaluatedProperties'

2020-10-07 Thread Dmitry Torokhov
On Mon, Oct 05, 2020 at 01:38:27PM -0500, Rob Herring wrote: > This doesn't yet do anything in the tools, but make it explicit so we can > check either 'unevaluatedProperties' or 'additionalProperties' is present > in schemas. > > 'unevaluatedProperties' is appropriate when including another

Re: [PATCH 4/4] dt-bindings: Explicitly allow additional properties in common schemas

2020-10-07 Thread Guenter Roeck
On 10/5/20 11:38 AM, Rob Herring wrote: > In order to add meta-schema checks for additional/unevaluatedProperties > being present, all schema need to make this explicit. As common/shared > schema are included by other schemas, they should always allow for > additionalProperties. > >

BUG: unable to handle kernel paging request in cfb_imageblit

2020-10-07 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:22fbc037 Merge tag 'for-linus' of git://git.kernel.org/pub.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=133731eb90 kernel config: https://syzkaller.appspot.com/x/.config?x=4e672827d2ffab1f

Re: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-07 Thread Jason Gunthorpe
On Tue, Oct 06, 2020 at 08:17:05PM +0200, Daniel Vetter wrote: > So on the gpu we pipeline this all. So step 4 doesn't happen on the > cpu, but instead we queue up a bunch of command buffers so that the > gpu writes these pagetables (and the flushes tlbs and then does the > actual stuff userspace

Re: [PATCH rdma-next v5 0/4] Dynamicaly allocate SG table from the pages

2020-10-07 Thread Daniel Vetter
On Wed, Oct 7, 2020 at 9:22 AM Jason Gunthorpe wrote: > On Tue, Oct 06, 2020 at 12:41:22PM +0200, Daniel Vetter wrote: > > On Mon, Oct 05, 2020 at 08:56:50PM -0300, Jason Gunthorpe wrote: > > > On Sun, Oct 04, 2020 at 06:43:36PM +0300, Leon Romanovsky wrote: > > > > This series extends

Re: [PATCH 8/8] drm/ttm: nuke caching placement flags

2020-10-07 Thread Christian König
Am 05.10.20 um 18:17 schrieb Ruhl, Michael J: -Original Message- From: dri-devel On Behalf Of Christian König Sent: Thursday, October 1, 2020 7:28 AM To: dri-devel@lists.freedesktop.org; ray.hu...@amd.com; airl...@gmail.com; dan...@ffwll.ch Subject: [PATCH 8/8] drm/ttm: nuke caching

[PATCH] drm/i915/jsl: Split EHL/JSL platform info and PCI ids

2020-10-07 Thread Tejas Upadhyay
Recently we came across requirement to identify EHL and JSL platform to program them differently. Thus Split the basic platform definition, macros, and PCI IDs to differentiate between EHL and JSL platforms. Also, IS_ELKHARTLAKE is replaced with IS_JSL_EHL everywhere. Cc : Matt Roper Cc : Ville

Re: [PATCH] drm/fourcc: Add AXBXGXRX106106106106 format

2020-10-07 Thread Daniel Vetter
On Wed, Oct 7, 2020 at 11:29 AM Matteo Franchin wrote: > > Add ABGR format with 10-bit components packed in 64-bit per pixel. > This format can be used to handle > VK_FORMAT_R10X6G10X6B10X6A10X6_UNORM_4PACK16 on little-endian > architectures. > > Signed-off-by: Matteo Franchin So is this

RE: [PATCH 2/8] drm/ttm: move ttm_set_memory.h out of include

2020-10-07 Thread Ruhl, Michael J
>-Original Message- >From: Christian König >Sent: Wednesday, October 7, 2020 4:31 AM >To: Ruhl, Michael J ; dri- >de...@lists.freedesktop.org; ray.hu...@amd.com; airl...@gmail.com; >dan...@ffwll.ch >Subject: Re: [PATCH 2/8] drm/ttm: move ttm_set_memory.h out of include > >Am 05.10.20 um

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Daniel Vetter
On Wed, Oct 7, 2020 at 12:47 PM Marek Szyprowski wrote: > > Hi Daniel, > > On 03.10.2020 11:40, Daniel Vetter wrote: > >> After he three places above should use pin_user_pages_fast(), then > >> this whole broken API should be moved into videobuf2-memops.c and a > >> big fat "THIS DOESN'T WORK"

Re: [PATCH v2] drm/vkms: update todo

2020-10-07 Thread Rodrigo Siqueira
On 10/06, Melissa Wen wrote: > Drop issues already resolved in vkms: > > - CRC API Improvements to [1] add igt test to check extreme alpha values > and [2] alpha blending; > - [3] prime buffer sharing; > - [4] writeback support; > > On the other hand, we also found or thought about other

Re: [PATCH v3 2/7] drm/ttm: Add ttm_kmap_obj_to_dma_buf_map() for type conversion

2020-10-07 Thread Thomas Zimmermann
Hi Am 07.10.20 um 15:10 schrieb Daniel Vetter: > On Wed, Oct 7, 2020 at 2:57 PM Thomas Zimmermann wrote: >> >> Hi >> >> Am 02.10.20 um 11:58 schrieb Daniel Vetter: >>> On Wed, Sep 30, 2020 at 02:51:46PM +0200, Daniel Vetter wrote: On Wed, Sep 30, 2020 at 2:34 PM Christian König wrote:

[PATCH] drm/fb-helper: Add locking to sysrq handling

2020-10-07 Thread Daniel Vetter
We didn't take the kernel_fb_helper_lock mutex, which protects that code. While at it, simplify the code - inline the function (originally shared with kgdb I think) - drop the error tracking and all the complications - drop the pointless early out, it served nothing Signed-off-by: Daniel Vetter

Re: [PATCH 7/8] drm/ttm: use caching instead of placement for ttm_io_prot

2020-10-07 Thread Christian König
Am 05.10.20 um 17:51 schrieb Ruhl, Michael J: -Original Message- From: dri-devel On Behalf Of Christian König Sent: Thursday, October 1, 2020 7:28 AM To: dri-devel@lists.freedesktop.org; ray.hu...@amd.com; airl...@gmail.com; dan...@ffwll.ch Subject: [PATCH 7/8] drm/ttm: use caching

Re: [Intel-gfx] [PATCH] drm/i915/jsl: Split EHL/JSL platform info and PCI ids

2020-10-07 Thread Tvrtko Ursulin
On 07/10/2020 10:36, Tejas Upadhyay wrote: Recently we came across requirement to identify EHL and JSL platform to program them differently. Thus Split the basic platform definition, macros, and PCI IDs to differentiate between EHL and JSL platforms. Also, IS_ELKHARTLAKE is replaced with

Re: [PATCH] drm/fourcc: Add AXBXGXRX106106106106 format

2020-10-07 Thread Matteo Franchin
Hi Daniel, On Wed, Oct 07, 2020 at 12:06:34PM +0200, Daniel Vetter wrote: > On Wed, Oct 7, 2020 at 11:29 AM Matteo Franchin > wrote: > > > > Add ABGR format with 10-bit components packed in 64-bit per pixel. > > This format can be used to handle > > VK_FORMAT_R10X6G10X6B10X6A10X6_UNORM_4PACK16

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Marek Szyprowski
Hi Daniel, On 07.10.2020 14:01, Daniel Vetter wrote: > On Wed, Oct 7, 2020 at 12:47 PM Marek Szyprowski > wrote: >> On 03.10.2020 11:40, Daniel Vetter wrote: After he three places above should use pin_user_pages_fast(), then this whole broken API should be moved into videobuf2-memops.c

Re: [PATCH v3 2/7] drm/ttm: Add ttm_kmap_obj_to_dma_buf_map() for type conversion

2020-10-07 Thread Thomas Zimmermann
Hi Am 02.10.20 um 11:58 schrieb Daniel Vetter: > On Wed, Sep 30, 2020 at 02:51:46PM +0200, Daniel Vetter wrote: >> On Wed, Sep 30, 2020 at 2:34 PM Christian König >> wrote: >>> >>> Am 30.09.20 um 11:47 schrieb Daniel Vetter: On Wed, Sep 30, 2020 at 10:34:31AM +0200, Christian König wrote:

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Tomasz Figa
On Sat, Oct 3, 2020 at 1:31 AM Jason Gunthorpe wrote: > > On Fri, Oct 02, 2020 at 08:16:48PM +0200, Daniel Vetter wrote: > > On Fri, Oct 2, 2020 at 8:06 PM Jason Gunthorpe wrote: > > > On Fri, Oct 02, 2020 at 07:53:03PM +0200, Daniel Vetter wrote: > > > > For $reasons I've stumbled over this

Re: [PATCH] drm/fourcc: Add AXBXGXRX106106106106 format

2020-10-07 Thread Matteo Franchin
Hi Ville, On Wed, Oct 07, 2020 at 01:35:39PM +0300, Ville Syrj�l� wrote: > On Wed, Oct 07, 2020 at 10:27:25AM +0100, Matteo Franchin wrote: > > Add ABGR format with 10-bit components packed in 64-bit per pixel. > > This format can be used to handle > >

[PATCH] drm/msm/dsi: save PLL registers across first PHY reset

2020-10-07 Thread benl-kernelpatches
From: Benjamin Li Take advantage of previously-added support for persisting PLL registers across DSI PHY disable/enable cycles (see 328e1a6 'drm/msm/dsi: Save/Restore PLL status across PHY reset') to support persisting across the very first DSI PHY enable at boot. The bootloader may have left

Re: [PATCH v3 2/7] drm/ttm: Add ttm_kmap_obj_to_dma_buf_map() for type conversion

2020-10-07 Thread Daniel Vetter
On Wed, Oct 7, 2020 at 2:57 PM Thomas Zimmermann wrote: > > Hi > > Am 02.10.20 um 11:58 schrieb Daniel Vetter: > > On Wed, Sep 30, 2020 at 02:51:46PM +0200, Daniel Vetter wrote: > >> On Wed, Sep 30, 2020 at 2:34 PM Christian König > >> wrote: > >>> > >>> Am 30.09.20 um 11:47 schrieb Daniel

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Tomasz Figa
On Wed, Oct 7, 2020 at 2:44 PM Jason Gunthorpe wrote: > > On Wed, Oct 07, 2020 at 02:33:56PM +0200, Marek Szyprowski wrote: > > Well, it was in vb2_get_vma() function, but now I see that it has been > > lost in fb639eb39154 and 6690c8c78c74 some time ago... > > There is no guarentee that holding

Re: [PATCH v2 1/7] dt-bindings: display: mxsfb: Convert binding to YAML

2020-10-07 Thread Lucas Stach
On Mi, 2020-10-07 at 10:32 +0200, Marek Vasut wrote: > On 10/7/20 3:24 AM, Laurent Pinchart wrote: > [...] > > +properties: > > + compatible: > > +enum: > > + - fsl,imx23-lcdif > > + - fsl,imx28-lcdif > > + - fsl,imx6sx-lcdif > > + - fsl,imx8mq-lcdif > > There is no

Re: [PATCH 6/8] drm/ttm: add caching state to ttm_bus_placement

2020-10-07 Thread Christian König
Am 05.10.20 um 17:39 schrieb Ruhl, Michael J: -Original Message- From: dri-devel On Behalf Of Christian König Sent: Thursday, October 1, 2020 7:28 AM To: dri-devel@lists.freedesktop.org; ray.hu...@amd.com; airl...@gmail.com; dan...@ffwll.ch Subject: [PATCH 6/8] drm/ttm: add caching

devfreq and panfrost on Allwinner H6

2020-10-07 Thread Tomeu Vizoso
Hi Clément, Have just noticed that my Pine H64 board hangs when I try to set the performance governor for the GPU devfreq. Is this a known bug? Thanks, Tomeu ___ dri-devel mailing list dri-devel@lists.freedesktop.org

[PATCH] drm/fourcc: Add AXBXGXRX106106106106 format

2020-10-07 Thread Matteo Franchin
Add ABGR format with 10-bit components packed in 64-bit per pixel. This format can be used to handle VK_FORMAT_R10X6G10X6B10X6A10X6_UNORM_4PACK16 on little-endian architectures. Signed-off-by: Matteo Franchin --- drivers/gpu/drm/drm_fourcc.c | 1 + include/uapi/drm/drm_fourcc.h | 7 +++ 2

Re: [PATCH 2/8] drm/ttm: move ttm_set_memory.h out of include

2020-10-07 Thread Christian König
Am 07.10.20 um 13:46 schrieb Ruhl, Michael J: -Original Message- From: Christian König Sent: Wednesday, October 7, 2020 4:31 AM To: Ruhl, Michael J ; dri- de...@lists.freedesktop.org; ray.hu...@amd.com; airl...@gmail.com; dan...@ffwll.ch Subject: Re: [PATCH 2/8] drm/ttm: move

[PATCH] drm/ttm: remove ttm_bo_unmap_virtual_locked declaration

2020-10-07 Thread Christian König
That was missed during the cleanup. Signed-off-by: Christian König --- include/drm/ttm/ttm_bo_driver.h | 9 - 1 file changed, 9 deletions(-) diff --git a/include/drm/ttm/ttm_bo_driver.h b/include/drm/ttm/ttm_bo_driver.h index 9897a16c0a9d..b58dedce7079 100644 ---

RE: [PATCH] drm/ttm: remove ttm_bo_unmap_virtual_locked declaration

2020-10-07 Thread Ruhl, Michael J
>-Original Message- >From: Christian König >Sent: Wednesday, October 7, 2020 4:52 AM >To: dri-devel@lists.freedesktop.org; airl...@gmail.com; >ray.hu...@amd.com; Ruhl, Michael J >Subject: [PATCH] drm/ttm: remove ttm_bo_unmap_virtual_locked >declaration > >That was missed during the

Re: [PATCH] drm/fourcc: Add AXBXGXRX106106106106 format

2020-10-07 Thread Ville Syrjälä
On Wed, Oct 07, 2020 at 10:27:25AM +0100, Matteo Franchin wrote: > Add ABGR format with 10-bit components packed in 64-bit per pixel. > This format can be used to handle > VK_FORMAT_R10X6G10X6B10X6A10X6_UNORM_4PACK16 on little-endian > architectures. > > Signed-off-by: Matteo Franchin > --- >

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Marek Szyprowski
Hi Daniel, On 03.10.2020 11:40, Daniel Vetter wrote: >> After he three places above should use pin_user_pages_fast(), then >> this whole broken API should be moved into videobuf2-memops.c and a >> big fat "THIS DOESN'T WORK" stuck on it. >> >> videobuf2 should probably use P2P DMA buf for this

Re: [PATCH] drm/ttm: remove ttm_bo_unmap_virtual_locked declaration

2020-10-07 Thread Christian König
Am 07.10.20 um 13:43 schrieb Ruhl, Michael J: -Original Message- From: Christian König Sent: Wednesday, October 7, 2020 4:52 AM To: dri-devel@lists.freedesktop.org; airl...@gmail.com; ray.hu...@amd.com; Ruhl, Michael J Subject: [PATCH] drm/ttm: remove ttm_bo_unmap_virtual_locked

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Daniel Vetter
On Wed, Oct 7, 2020 at 2:48 PM Tomasz Figa wrote: > > On Wed, Oct 7, 2020 at 2:44 PM Jason Gunthorpe wrote: > > > > On Wed, Oct 07, 2020 at 02:33:56PM +0200, Marek Szyprowski wrote: > > > Well, it was in vb2_get_vma() function, but now I see that it has been > > > lost in fb639eb39154 and

Re: [PATCH v3 2/7] drm/ttm: Add ttm_kmap_obj_to_dma_buf_map() for type conversion

2020-10-07 Thread Christian König
Am 07.10.20 um 15:20 schrieb Thomas Zimmermann: Hi Am 07.10.20 um 15:10 schrieb Daniel Vetter: On Wed, Oct 7, 2020 at 2:57 PM Thomas Zimmermann wrote: Hi Am 02.10.20 um 11:58 schrieb Daniel Vetter: On Wed, Sep 30, 2020 at 02:51:46PM +0200, Daniel Vetter wrote: On Wed, Sep 30, 2020 at 2:34

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Tomasz Figa
On Wed, Oct 7, 2020 at 3:06 PM Jason Gunthorpe wrote: > > On Wed, Oct 07, 2020 at 02:58:33PM +0200, Daniel Vetter wrote: > > On Wed, Oct 7, 2020 at 2:48 PM Tomasz Figa wrote: > > > > > > On Wed, Oct 7, 2020 at 2:44 PM Jason Gunthorpe wrote: > > > > > > > > On Wed, Oct 07, 2020 at 02:33:56PM

Re: [PATCH v2 1/7] dt-bindings: display: mxsfb: Convert binding to YAML

2020-10-07 Thread Laurent Pinchart
Hi Marek, On Wed, Oct 07, 2020 at 10:55:24AM +0200, Marek Vasut wrote: > On 10/7/20 10:43 AM, Lucas Stach wrote: > > On Mi, 2020-10-07 at 10:32 +0200, Marek Vasut wrote: > >> On 10/7/20 3:24 AM, Laurent Pinchart wrote: > >> [...] > >>> +properties: > >>> + compatible: > >>> +enum: > >>> +

[PATCH 09/14] drm/amd/pm: Replace one-element array with flexible-array in struct phm_samu_clock_voltage_dependency_table

2020-10-07 Thread Gustavo A. R. Silva
There is a regular need in the kernel to provide a way to declare having a dynamically sized set of trailing elements in a structure. Kernel code should always use “flexible array members”[1] for these cases. The older style of one-element or zero-length arrays should no longer be used[2].

Re: [PATCH v2 2/7] dt-bindings: display: mxsfb: Add and fix compatible strings

2020-10-07 Thread Rob Herring
On Wed, 07 Oct 2020 04:24:33 +0300, Laurent Pinchart wrote: > Additional compatible strings have been added in DT source for the > i.MX6SL, i.MX6SLL, i.MX6UL and i.MX7D without updating the bindings. > Most of the upstream DT sources use the fsl,imx28-lcdif compatible > string, which mostly

[PATCH 08/14] drm/amd/pm: Replace one-element array with flexible-array in struct phm_cac_leakage_table

2020-10-07 Thread Gustavo A. R. Silva
There is a regular need in the kernel to provide a way to declare having a dynamically sized set of trailing elements in a structure. Kernel code should always use “flexible array members”[1] for these cases. The older style of one-element or zero-length arrays should no longer be used[2].

[PATCH v3 08/20] gpu: host1x: Implement /dev/host1x device node

2020-10-07 Thread Mikko Perttunen
Add the /dev/host1x device node, implementing the following functionality: - Reading syncpoint values - Allocating syncpoints (providing syncpoint FDs) - Incrementing syncpoints (based on syncpoint FD) Signed-off-by: Mikko Perttunen --- v3: * Pass process name as syncpoint name when allocating

[PATCH v3 13/20] gpu: host1x: Reset max value when freeing a syncpoint

2020-10-07 Thread Mikko Perttunen
With job recovery becoming optional, syncpoints may have a mismatch between their value and max value when freed. As such, when freeing, set the max value to the current value of the syncpoint so that it is in a sane state for the next user. Signed-off-by: Mikko Perttunen --- v3: * Use

[PATCH v3 20/20] drm/tegra: Add job firewall

2020-10-07 Thread Mikko Perttunen
Add a firewall that validates jobs before submission to ensure they don't do anything they aren't allowed to do, like accessing memory they should not access. The firewall is functionality-wise a copy of the firewall already implemented in gpu/host1x. It is copied here as it makes more sense for

[PATCH v3 12/20] gpu: host1x: Add support for syncpoint waits in CDMA pushbuffer

2020-10-07 Thread Mikko Perttunen
Add support for inserting syncpoint waits in the CDMA pushbuffer. These waits need to be done in HOST1X class, while gather submitted by the application execute in engine class. Support is added by converting the gather list of job into a command list that can include both gathers and waits. When

[PATCH v3 16/20] drm/tegra: Boot VIC during runtime PM resume

2020-10-07 Thread Mikko Perttunen
With the new UAPI implementation, engines are powered on and off when there are active jobs, and the core code handles channel allocation. To accommodate that, boot the engine as part of runtime PM instead of using the open_channel callback, which is not used by the new submit path.

[PATCH v3 00/20] Host1x/TegraDRM UAPI

2020-10-07 Thread Mikko Perttunen
Hi all, here's the third revision of the Host1x/TegraDRM UAPI proposal. The open issues from RFCv2 should be resolved now, so I'm dropping the RFC tag. The series is still only tested with Tegra186 so I'm hoping for people with devices with other chips to test this out. The test suite[1] has

[PATCH v3 05/20] gpu: host1x: Use HW-equivalent syncpoint expiration check

2020-10-07 Thread Mikko Perttunen
Make syncpoint expiration checks always use the same logic used by the hardware. This ensures that there are no race conditions that could occur because of the hardware triggering a syncpoint interrupt and then the driver disagreeing. One situation where this could occur is if a job incremented a

[PATCH v3 06/20] gpu: host1x: Cleanup and refcounting for syncpoints

2020-10-07 Thread Mikko Perttunen
Add reference counting for allocated syncpoints to allow keeping them allocated while jobs are referencing them. Additionally, clean up various places using syncpoint IDs to use host1x_syncpt pointers instead. Signed-off-by: Mikko Perttunen --- drivers/gpu/drm/tegra/dc.c | 4 +-

[PATCH v3 02/20] gpu: host1x: Allow syncpoints without associated client

2020-10-07 Thread Mikko Perttunen
Syncpoints don't need to be associated with any client, so remove the property, and expose host1x_syncpt_alloc. This will allow allocating syncpoints without prior knowledge of the engine that it will be used with. Signed-off-by: Mikko Perttunen --- v3: * Clean up host1x_syncpt_alloc signature

[PATCH v3 17/20] drm/tegra: Set resv fields when importing/exporting GEMs

2020-10-07 Thread Mikko Perttunen
To allow sharing of implicit fences when exporting/importing dma_buf objects, set the 'resv' fields when importing or exporting GEM objects. Signed-off-by: Mikko Perttunen --- drivers/gpu/drm/tegra/gem.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/tegra/gem.c

[PATCH v3 10/20] gpu: host1x: Add no-recovery mode

2020-10-07 Thread Mikko Perttunen
Add a new property for jobs to enable or disable recovery i.e. CPU increments of syncpoints to max value on job timeout. This allows for a more solid model for hanged jobs, where userspace doesn't need to guess if a syncpoint increment happened because the job completed, or because job timeout was

[PATCH v3 11/20] gpu: host1x: Add job release callback

2020-10-07 Thread Mikko Perttunen
Add a callback field to the job structure, to be called just before the job is to be freed. This allows the job's submitter to clean up any of its own state, like decrement runtime PM refcounts. Signed-off-by: Mikko Perttunen --- drivers/gpu/host1x/job.c | 3 +++ include/linux/host1x.h | 4

[PATCH v3 15/20] drm/tegra: Add new UAPI to header

2020-10-07 Thread Mikko Perttunen
Update the tegra_drm.h UAPI header, adding the new proposed UAPI. The old staging UAPI is left in for now, with minor modification to avoid name collisions. Signed-off-by: Mikko Perttunen --- v3: * Remove timeout field * Inline the syncpt_incrs array to the submit structure * Remove WRITE_RELOC

[PATCH v3 18/20] drm/tegra: Allocate per-engine channel in core code

2020-10-07 Thread Mikko Perttunen
To avoid duplication, allocate the per-engine shared channel in the core code instead. Once MLOCKs are implemented on Host1x side, we can also update this to avoid allocating a shared channel when MLOCKs are enabled. Signed-off-by: Mikko Perttunen --- drivers/gpu/drm/tegra/drm.c | 11

[PATCH v3 19/20] drm/tegra: Implement new UAPI

2020-10-07 Thread Mikko Perttunen
Implement the new UAPI, and bump the TegraDRM major version. Signed-off-by: Mikko Perttunen --- v3: * Remove WRITE_RELOC. Relocations are now patched implicitly when patching is needed. * Directly call PM runtime APIs on devices instead of using power_on/power_off callbacks. * Remove

[PATCH v3 09/20] gpu: host1x: DMA fences and userspace fence creation

2020-10-07 Thread Mikko Perttunen
Add an implementation of dma_fences based on syncpoints. Syncpoint interrupts are used to signal fences. Additionally, after software signaling has been enabled, a 30 second timeout is started. If the syncpoint threshold is not reached within this period, the fence is signalled with an -ETIMEDOUT

[PATCH v3 14/20] gpu: host1x: Reserve VBLANK syncpoints at initialization

2020-10-07 Thread Mikko Perttunen
On T20-T148 chips, the bootloader can set up a boot splash screen with DC configured to increment syncpoint 26/27 at VBLANK. Because of this we shouldn't allow these syncpoints to be allocated until DC has been reset and will no longer increment them in the background. As such, on these chips,

[PATCH v3 04/20] gpu: host1x: Remove cancelled waiters immediately

2020-10-07 Thread Mikko Perttunen
Before this patch, cancelled waiters would only be cleaned up once their threshold value was reached. Make host1x_intr_put_ref process the cancellation immediately to fix this. Signed-off-by: Mikko Perttunen --- drivers/gpu/host1x/intr.c | 14 +- 1 file changed, 9 insertions(+), 5

[PATCH v3 03/20] gpu: host1x: Show number of pending waiters in debugfs

2020-10-07 Thread Mikko Perttunen
Show the number of pending waiters in the debugfs status file. This is useful for testing to verify that waiters do not leak or accumulate incorrectly. Signed-off-by: Mikko Perttunen --- drivers/gpu/host1x/debug.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git

[PATCH v3 07/20] gpu: host1x: Introduce UAPI header

2020-10-07 Thread Mikko Perttunen
Add the userspace interface header, specifying interfaces for allocating and accessing syncpoints from userspace, and for creating sync_file based fences based on syncpoint thresholds. Signed-off-by: Mikko Perttunen --- include/uapi/linux/host1x.h | 134 1

[PATCH v3 01/20] gpu: host1x: Use different lock classes for each client

2020-10-07 Thread Mikko Perttunen
To avoid false lockdep warnings, give each client lock a different lock class, passed from the initialization site by macro. Signed-off-by: Mikko Perttunen --- drivers/gpu/host1x/bus.c | 7 --- include/linux/host1x.h | 9 - 2 files changed, 12 insertions(+), 4 deletions(-) diff

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Daniel Vetter
On Wed, Oct 7, 2020 at 3:34 PM Tomasz Figa wrote: > > On Wed, Oct 7, 2020 at 3:06 PM Jason Gunthorpe wrote: > > > > On Wed, Oct 07, 2020 at 02:58:33PM +0200, Daniel Vetter wrote: > > > On Wed, Oct 7, 2020 at 2:48 PM Tomasz Figa wrote: > > > > > > > > On Wed, Oct 7, 2020 at 2:44 PM Jason

[PATCH 03/14] drm/amd/pm: Replace one-element array with flexible-array in struct phm_clock_array

2020-10-07 Thread Gustavo A. R. Silva
There is a regular need in the kernel to provide a way to declare having a dynamically sized set of trailing elements in a structure. Kernel code should always use “flexible array members”[1] for these cases. The older style of one-element or zero-length arrays should no longer be used[2].

[PATCH 04/14] drm/amd/pm: Replace one-element array with flexible-array in struct phm_uvd_clock_voltage_dependency_table

2020-10-07 Thread Gustavo A. R. Silva
There is a regular need in the kernel to provide a way to declare having a dynamically sized set of trailing elements in a structure. Kernel code should always use “flexible array members”[1] for these cases. The older style of one-element or zero-length arrays should no longer be used[2].

[PATCH 05/14] drm/amd/pm: Replace one-element array with flexible-array in struct phm_acp_clock_voltage_dependency_table

2020-10-07 Thread Gustavo A. R. Silva
There is a regular need in the kernel to provide a way to declare having a dynamically sized set of trailing elements in a structure. Kernel code should always use “flexible array members”[1] for these cases. The older style of one-element or zero-length arrays should no longer be used[2].

Re: [PATCH v2 0/3] drm: commit_work scheduling

2020-10-07 Thread Rob Clark
On Wed, Oct 7, 2020 at 3:36 AM Qais Yousef wrote: > > On 10/06/20 13:04, Rob Clark wrote: > > On Tue, Oct 6, 2020 at 3:59 AM Qais Yousef wrote: > > > > > > On 10/05/20 16:24, Rob Clark wrote: > > > > > > [...] > > > > > > > > RT planning and partitioning is not easy task for sure. You might > >

[PATCH 02/14] drm/amd/pm: Replace one-element array with flexible-array member in struct vi_dpm_table

2020-10-07 Thread Gustavo A. R. Silva
There is a regular need in the kernel to provide a way to declare having a dynamically sized set of trailing elements in a structure. Kernel code should always use “flexible array members”[1] for these cases. The older style of one-element or zero-length arrays should no longer be used[2]. Use a

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-07 Thread Tomasz Figa
On Wed, Oct 7, 2020 at 4:09 PM Daniel Vetter wrote: > > On Wed, Oct 7, 2020 at 3:34 PM Tomasz Figa wrote: > > > > On Wed, Oct 7, 2020 at 3:06 PM Jason Gunthorpe wrote: > > > > > > On Wed, Oct 07, 2020 at 02:58:33PM +0200, Daniel Vetter wrote: > > > > On Wed, Oct 7, 2020 at 2:48 PM Tomasz Figa

  1   2   >