Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Daniel Bristot de Oliveira
On 9/24/20 10:27 AM, pet...@infradead.org wrote: > So my current todo list is: > > - Change RT PULL > - Change DL PULL > - Add migrate_disable() tracer; exactly like preempt/irqoff, except >measuring task-runtime instead of cpu-time. > - Add a mode that measures actual interference. > -

Re: [PATCH 37/45] drm/ttm: add a helper to allocate a temp tt for copies.

2020-09-24 Thread Dave Airlie
On Thu, 24 Sep 2020 at 22:42, Christian König wrote: > > Am 24.09.20 um 07:18 schrieb Dave Airlie: > > From: Dave Airlie > > > > All the accel moves do the same pattern here, provide a helper > > And exactly that pattern I want to get away from. Currently this is just refactoring out the helper

[git pull] drm fixes for 5.9-rc7

2020-09-24 Thread Dave Airlie
Hi Linus, Due to the dax merge fail in rc6, this has two backmerges, Intel pulled in a later point in time to fix their CI systems, I also pulled in an earlier point to fix my local builds. Otherwise fairly quiet, a couple of i915 fixes, one dma-buf fix, one vc4 and two sun4i changes. I

Re: [PATCH 07/10] drm/amdgpu/ttm: handle tt moves properly.

2020-09-24 Thread Christian König
Am 24.09.20 um 02:46 schrieb Dave Airlie: On Thu, 24 Sep 2020 at 00:46, Christian König wrote: Am 23.09.20 um 05:04 schrieb Dave Airlie: From: Dave Airlie The core move code currently handles use_tt moves, for amdgpu this was being handled also in the driver, but not using the same paths.

Re: [PATCH 06/45] drm/vmwgfx: move null mem checks outside move notifies

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie Both fns checked mem == NULL, just move the check outside. Signed-off-by: Dave Airlie Acked-by: Christian König --- drivers/gpu/drm/vmwgfx/vmwgfx_bo.c | 3 --- drivers/gpu/drm/vmwgfx/vmwgfx_resource.c | 2 +-

Re: [PATCH 05/45] drm/nouveau/ttm: handle ttm moves properly.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie The idea is to flip the core over to calling the driver always, so add support for moves here. Signed-off-by: Dave Airlie Acked-by: Christian König --- drivers/gpu/drm/nouveau/nouveau_bo.c | 12 1 file changed,

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

2020-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2020 at 1:01 PM Ville Syrjälä wrote: > > On Thu, Sep 24, 2020 at 01:10:56PM +0300, Pekka Paalanen wrote: > > On Thu, 24 Sep 2020 10:04:12 +0200 > > Daniel Vetter wrote: > > > > > On Thu, Sep 24, 2020 at 9:41 AM Pekka Paalanen > > > wrote: > > > > > > > > On Wed, 23 Sep 2020

Re: [PATCH 04/45] drm/radeon/ttm: handle ttm moves properly

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie The core move code currently handles use_tt moves, for radeon this was being handled also in the driver, but not using the same paths. If moving between TT/SYSTEM (all the use_tt paths on radeon) use the core move function.

Re: [PATCH v2] MAINTAINERS: add Dan Murphy as TP LP8xxx drivers maintainer

2020-09-24 Thread Dan Murphy
Hello On 9/22/20 10:28 AM, Krzysztof Kozlowski wrote: Milo Kim's email in TI bounces with permanent error (550: Invalid recipient). Last email from him on LKML was in 2017. Move Milo Kim to credits and add Dan Murphy from TI to look after: - TI LP855x backlight driver, - TI LP8727 charger

[Bug 198551] amdgpu error on shutdown or gpu intense game

2020-09-24 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=198551 Gerry Binder (gerrybinde...@gmail.com) changed: What|Removed |Added CC|

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

2020-09-24 Thread Pekka Paalanen
On Wed, 23 Sep 2020 22:01:25 +0200 Daniel Vetter wrote: > On Wed, Sep 23, 2020 at 9:17 PM Marius Vlad wrote: > > > > On Wed, Sep 23, 2020 at 05:18:52PM +0200, Daniel Vetter wrote: > > > When doing an atomic modeset with ALLOW_MODESET drivers are allowed to > > > pull in arbitrary other

Re: [PATCH 30/45] drm/ttm: add a new invalidate notify callback.

2020-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2020 at 03:18:30PM +1000, Dave Airlie wrote: > From: Dave Airlie > > Signed-off-by: Dave Airlie A bikeshed, but why not just call this ->invalidate? ->move_notify needed the _notify because we already had the ->move callback, but there's not invalidate that also needs a

Re: [PATCH 09/11] drm/amdgpu: switch over to the new pin interface

2020-09-24 Thread Nirmoy
Tested-by: Nirmoy Das On 9/22/20 3:32 PM, Christian König wrote: Stop using TTM_PL_FLAG_NO_EVICT. Signed-off-by: Christian König --- .../gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 5 ++- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c| 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_display.c

Re: [Nouveau] [PATCH 0/6] drm/nouveau: Support sync FDs and sync objects

2020-09-24 Thread Thierry Reding
On Wed, Sep 23, 2020 at 05:21:24PM +0200, Daniel Vetter wrote: > On Wed, Sep 23, 2020 at 11:18:53AM +0200, Thierry Reding wrote: > > On Fri, Aug 28, 2020 at 12:40:10PM +0200, Thierry Reding wrote: > > > From: Thierry Reding > > > > > > Hi, > > > > > > This series implements a new IOCTL to

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

2020-09-24 Thread Pekka Paalanen
On Thu, 24 Sep 2020 10:04:12 +0200 Daniel Vetter wrote: > On Thu, Sep 24, 2020 at 9:41 AM Pekka Paalanen wrote: > > > > On Wed, 23 Sep 2020 22:01:25 +0200 > > Daniel Vetter wrote: > > > > > On Wed, Sep 23, 2020 at 9:17 PM Marius Vlad > > > wrote: > > > > > > > > On Wed, Sep 23, 2020 at

[Bug 209373] Pixels

2020-09-24 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=209373 --- Comment #1 from Michel Dänzer (mic...@daenzer.net) --- (In reply to Jens R. from comment #0) > kern :warn : [ 2.381846] Hardware name: LENOVO 20NFGE/20NFGE, BIOS > R11ET32W (1.12 ) 12/23/2019 I'm writing this on the exact same

[PULL] drm-misc-fixes

2020-09-24 Thread Maarten Lankhorst
drm-misc-fixes-2020-09-24: drm-misc-fixes for v5.9: - Single null pointer deref fix for dma-buf. The following changes since commit 74ea06164cda81dc80e97790164ca533fd7e3087: drm/sun4i: mixer: Extend regmap max_register (2020-09-10 13:08:48 +0200) are available in the Git repository at:

Re: [PATCH 07/45] drm/vmwgfx: add a move callback.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This just copies the fallback to vmwgfx, I'm going to iterate on this a bit until it's not the same as the fallback path. Signed-off-by: Dave Airlie --- drivers/gpu/drm/vmwgfx/vmwgfx_ttm_buffer.c | 19 ++- 1 file

Re: [PATCH 08/45] drm/vram_helper: implement a ttm move callback.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This will always do memcpy moves. Signed-off-by: Dave Airlie Acked-by: Christian König --- drivers/gpu/drm/drm_gem_vram_helper.c | 21 + 1 file changed, 21 insertions(+) diff --git

Re: [PATCH 12/45] drm/ttm: drop free old node wrapper.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This isn't really used anymore, if drivers needs it later, just add back an inline wrapper. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo_util.c | 11 +++

Re: [PATCH 13/45] drm/ttm: use new move interface for known system->ttm moves

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie In all 3 drivers there is a case where the driver knows the bo is in SYSTEM so don't call the api that checks that. Signed-off-by: Dave Airlie In the long term I completely want to get rid of this nonsense because it means that

[PATCH] MAINTAINERS: mark FRAMEBUFFER LAYER as Orphan

2020-09-24 Thread Bartlomiej Zolnierkiewicz
It has been a fun ride since 2017 but unfortunately I don't have enough time to look after it properly anymore. Signed-off-by: Bartlomiej Zolnierkiewicz --- MAINTAINERS |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) Index: b/MAINTAINERS

[Bug 209373] New: Pixels

2020-09-24 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=209373 Bug ID: 209373 Summary: Pixels Product: Drivers Version: 2.5 Kernel Version: 5.8.0-1-amd64, but also 5.7 Hardware: x86-64 OS: Linux Tree: Mainline

Re: [PATCH 01/45] drm/qxl: drop unused code

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie Signed-off-by: Dave Airlie --- drivers/gpu/drm/qxl/qxl_ttm.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/gpu/drm/qxl/qxl_ttm.c b/drivers/gpu/drm/qxl/qxl_ttm.c index 2c35ca4270c6..5738be300078 100644 ---

Re: [PATCH 02/45] drm/ttm: handle the SYSTEM->TT path in same place as others.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This just consolidates the code making the flow easier to understand and also helps when moving move to the driver side. Signed-off-by: Dave Airlie --- drivers/gpu/drm/ttm/ttm_bo.c | 17 +++-- 1 file changed, 7

[Bug 209373] Pixels

2020-09-24 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=209373 --- Comment #2 from Jens R. (rapp.j...@gmail.com) --- I followed this description https://www.cyberciti.biz/faq/update-lenovo-bios-from-linux-usb-stick-pen/ for making a bootable stick and it didn't work on three different sticks.. -- You are

[Bug 209373] Pixels

2020-09-24 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=209373 --- Comment #3 from Michel Dänzer (mic...@daenzer.net) --- Yeah, the BIOS seems very picky about which partitions it boots from. With GParted or a similar tool, check that the partition which contains the EFI directory has at least the boot & esp

Re: [PATCH 09/45] drm/ttm: make move callback compulstory

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie All drivers should have a move callback now so make it compulsory. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-)

Re: [PATCH 11/45] drm/ttm: split out the move to system from move ttm code

2020-09-24 Thread Christian König
What should that be good for? Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie Signed-off-by: Dave Airlie --- drivers/gpu/drm/ttm/ttm_bo_util.c | 39 --- 1 file changed, 25 insertions(+), 14 deletions(-) diff --git

Re: [PATCH] drm/ast: Reload gamma LUT after changing primary plane's color format

2020-09-24 Thread Thomas Zimmermann
Hi Am 23.09.20 um 17:30 schrieb Daniel Vetter: > On Tue, Sep 22, 2020 at 04:46:55PM +0200, Thomas Zimmermann wrote: >> The gamma LUT has to be reloaded after changing the primary plane's >> color format. This used to be done implicitly by the CRTC atomic_enable() >> helper after updating the

Re: [PATCH] drm: bridge: cdns-mhdp8546: fix compile warning

2020-09-24 Thread Tomi Valkeinen
On 23/09/2020 11:30, Tomi Valkeinen wrote: > On x64 we get: > > drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c:751:10: warning: > conversion from 'long unsigned int' to 'unsigned int' changes value from > '18446744073709551613' to '4294967293' [-Woverflow] > > The registers are 32 bit, so

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

2020-09-24 Thread Ville Syrjälä
On Thu, Sep 24, 2020 at 01:10:56PM +0300, Pekka Paalanen wrote: > On Thu, 24 Sep 2020 10:04:12 +0200 > Daniel Vetter wrote: > > > On Thu, Sep 24, 2020 at 9:41 AM Pekka Paalanen wrote: > > > > > > On Wed, 23 Sep 2020 22:01:25 +0200 > > > Daniel Vetter wrote: > > > > > > > On Wed, Sep 23, 2020

Re: [PATCH 03/45] drm/amdgpu/ttm: handle tt moves properly.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie The core move code currently handles use_tt moves, for amdgpu this was being handled also in the driver, but not using the same paths. If moving between TT/SYSTEM (all the use_tt paths on amdgpu) use the core move function.

Re: [Nouveau] [PATCH 0/6] drm/nouveau: Support sync FDs and sync objects

2020-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2020 at 12:05 PM Thierry Reding wrote: > > On Wed, Sep 23, 2020 at 05:21:24PM +0200, Daniel Vetter wrote: > > On Wed, Sep 23, 2020 at 11:18:53AM +0200, Thierry Reding wrote: > > > On Fri, Aug 28, 2020 at 12:40:10PM +0200, Thierry Reding wrote: > > > > From: Thierry Reding > > > >

Re: [PATCH 10/45] drm/ttm: refactor out common code to setup a new tt backed resource

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This factors out the code to set the new caching and for non-system tt populate and bind things. The same code was used twice in the move paths. Signed-off-by: Dave Airlie Reviewed-by: Christian König ---

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

2020-09-24 Thread Ville Syrjälä
On Thu, Sep 24, 2020 at 01:13:17PM +0200, Daniel Vetter wrote: > On Thu, Sep 24, 2020 at 1:01 PM Ville Syrjälä > wrote: > > > > On Thu, Sep 24, 2020 at 01:10:56PM +0300, Pekka Paalanen wrote: > > > On Thu, 24 Sep 2020 10:04:12 +0200 > > > Daniel Vetter wrote: > > > > > > > On Thu, Sep 24, 2020

Re: [PATCH] drm: bridge: cdns-mhdp8546: fix compile warning

2020-09-24 Thread Laurent Pinchart
Hi Tomi, Thank you for the patch. On Wed, Sep 23, 2020 at 11:30:57AM +0300, Tomi Valkeinen wrote: > On x64 we get: > > drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c:751:10: warning: > conversion from 'long unsigned int' to 'unsigned int' changes value from > '18446744073709551613' to

Re: [PATCHv2] dt-bindings: dp-connector: add binding for DisplayPort connector

2020-09-24 Thread Tomi Valkeinen
On 23/09/2020 23:00, Rob Herring wrote: > On Wed, Sep 23, 2020 at 11:15 AM Tomi Valkeinen wrote: >> >> Hi Rob, >> >> On 23/09/2020 19:17, Rob Herring wrote: >> * No eDP. There's really no "eDP connector", as it's always a custom made connection between the DP and the DP panel. So

[Bug 209373] Pixels

2020-09-24 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=209373 --- Comment #4 from Jens R. (rapp.j...@gmail.com) --- (In reply to Michel Dänzer from comment #3) > Yeah, the BIOS seems very picky about which partitions it boots from. With > GParted or a similar tool, check that the partition which contains

Re: [PATCH 00/45] TTM move refactoring

2020-09-24 Thread Dave Airlie
On Thu, 24 Sep 2020 at 15:19, Dave Airlie wrote: > > This refactors how TTM moves get called between core and drivers. > Full series is also: https://github.com/airlied/linux/tree/ttm-no-more-bind (it's based on top of Christian's last series). Dave.

Re: [PATCH v2 2/3] dma-buf: Use struct dma_buf_map in dma_buf_vmap() interfaces

2020-09-24 Thread Thomas Zimmermann
Hi Am 23.09.20 um 17:39 schrieb Daniel Vetter: > On Wed, Sep 23, 2020 at 02:32:04PM +0200, Thomas Zimmermann wrote: >> This patch updates dma_buf_vmap() and dma-buf's vmap callback to use >> struct dma_buf_map. >> >> The interfaces used to return a buffer address. This address now gets >> stored

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

2020-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2020 at 9:41 AM Pekka Paalanen wrote: > > On Wed, 23 Sep 2020 22:01:25 +0200 > Daniel Vetter wrote: > > > On Wed, Sep 23, 2020 at 9:17 PM Marius Vlad > > wrote: > > > > > > On Wed, Sep 23, 2020 at 05:18:52PM +0200, Daniel Vetter wrote: > > > > When doing an atomic modeset with

Re: [Intel-gfx] [PATCH rdma-next v3 1/2] lib/scatterlist: Add support in dynamic allocation of SG table from pages

2020-09-24 Thread Tvrtko Ursulin
On 22/09/2020 09:39, Leon Romanovsky wrote: From: Maor Gottlieb Extend __sg_alloc_table_from_pages to support dynamic allocation of SG table from pages. It should be used by drivers that can't supply all the pages at one time. This function returns the last populated SGE in the table. Users

Re: [PATCH 1/2] drm/dp: add subheadings to DPCD address definitions

2020-09-24 Thread Jani Nikula
On Fri, 18 Sep 2020, "Navare, Manasi" wrote: > On Fri, Sep 18, 2020 at 02:40:16PM +0300, Jani Nikula wrote: >> Add the subheadings from the DP spec. No functional changes. >> >> Signed-off-by: Jani Nikula > > Looks good to me > > Reviewed-by: Manasi Navare Thanks, pushed both to

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

2020-09-24 Thread Daniel Vetter
On Wed, Sep 23, 2020 at 07:33:17PM -0700, Rob Clark wrote: > On Wed, Sep 23, 2020 at 8:25 AM Daniel Vetter wrote: > > > > On Tue, Sep 22, 2020 at 07:48:10AM -0700, Rob Clark wrote: > > > On Mon, Sep 21, 2020 at 11:59 PM Daniel Vetter wrote: > > > > > > > > On Mon, Sep 21, 2020 at 5:16 PM Rob

[PATCH v2 2/2] drm/vc4: crtc: Keep the previously assigned HVS FIFO

2020-09-24 Thread Maxime Ripard
The HVS FIFOs are currently assigned each time we have an atomic_check for all the enabled CRTCs. However, if we are running multiple outputs in parallel and we happen to disable the first (by index) CRTC, we end up changing the assigned FIFO of the second CRTC without disabling and reenabling

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Thomas Gleixner
On Wed, Sep 23 2020 at 17:12, Steven Rostedt wrote: > On Wed, 23 Sep 2020 22:55:54 +0200 > Then scratch the idea of having anonymous local_lock() and just bring > local_lock in directly? Then have a kmap local lock, which would only > block those that need to do a kmap. That's still going to end

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Thomas Gleixner
On Wed, Sep 23 2020 at 12:19, peterz wrote: > On Mon, Sep 21, 2020 at 09:27:57PM +0200, Thomas Gleixner wrote: >> Alternatively this could of course be solved with per CPU page tables >> which will come around some day anyway I fear. > > Previously (with PTI) we looked at making the entire kernel

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Thomas Gleixner
On Wed, Sep 23 2020 at 10:40, peterz wrote: > Right, so I'm concerned. migrate_disable() wrecks pretty much all > Real-Time scheduler theory we have, and PREEMPRT_RT bringing it in is > somewhat ironic. It's even more ironic that the approach of PREEMPT_RT has been 'pragmatic ignorance of theory'

Re: [PATCH] drm/rockchip: skip probed failed device

2020-09-24 Thread Jian-Hong Pan
Heiko Stübner 於 2020年9月23日 週三 下午7:16寫道: > > Am Mittwoch, 23. September 2020, 13:05:26 CEST schrieb Robin Murphy: > > On 2020-09-23 07:59, Jian-Hong Pan wrote: > > > The cdn-dp sub driver probes the device failed on PINEBOOK Pro. > > > > > > kernel: cdn-dp fec0.dp: [drm:cdn_dp_probe

[PATCH] linux-firmware: Update firmware for Cadence MHDP8546 DP bridge

2020-09-24 Thread Swapnil Jakhade
Update firmware for Cadence MHDP8546 DP bridge to version 2.0.0. The firmware source code now complies with MISRA2012 and HIS rules and directives. Also, there are some improvements in AUX channel communication handling. Signed-off-by: Swapnil Jakhade --- WHENCE | 4 ++--

Re: [PATCH V4] drm/dp_mst: Retrieve extended DPCD caps for topology manager

2020-09-24 Thread Koba Ko
Thanks a lot. On Wed, Sep 23, 2020, 10:18 PM Lyude Paul wrote: > On Wed, 2020-09-23 at 10:16 +0800, Koba Ko wrote: > > Thanks for the review. > > Sorry for that I thought the review tag should be appended by myself. > > One thing to confirm with you, will you or I push this patch to drm-misc- >

Re: [PATCH 1/2] drm/vc4: kms: Assign a FIFO to enabled CRTCs instead of active

2020-09-24 Thread Maxime Ripard
On Fri, Sep 18, 2020 at 04:43:20PM +0100, Dave Stevenson wrote: > Hi Maxime > > Thanks for the patch. > > On Fri, 18 Sep 2020 at 15:59, Maxime Ripard wrote: > > > > The HVS has three FIFOs that can be assigned to a number of PixelValves > > through a mux. > > > > However, changing that FIFO

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Thomas Gleixner
On Mon, Sep 21 2020 at 21:27, Thomas Gleixner wrote: > On Mon, Sep 21 2020 at 09:24, Linus Torvalds wrote: >> On Mon, Sep 21, 2020 at 12:39 AM Thomas Gleixner wrote: >> Maybe we really *could* call this new kmap functionality something >> like "kmap_percpu()" (or maybe "local" is good enough),

[PATCH v2 1/2] drm/vc4: crtc: Rework a bit the CRTC state code

2020-09-24 Thread Maxime Ripard
The current CRTC state reset hook in vc4 allocates a vc4_crtc_state structure as a drm_crtc_state, and relies on the fact that vc4_crtc_state embeds drm_crtc_state as its first member, and therefore can be safely casted. However, this is pretty fragile especially since there's no check for this

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Thomas Gleixner
On Wed, Sep 23 2020 at 11:52, Steven Rostedt wrote: > On Wed, 23 Sep 2020 10:40:32 +0200 > pet...@infradead.org wrote: > >> However, with migrate_disable() we can have each task preempted in a >> migrate_disable() region, worse we can stack them all on the _same_ CPU >> (super ridiculous odds,

[PULL] drm-intel-fixes

2020-09-24 Thread Jani Nikula
Hi Dave & Daniel - Just a couple of simple fixes. With Daniel's irc ack I backmerged Linus' tree at an arbitrary commit due to a build failure in v5.9-rc6 that blocked CI. drm-intel-fixes-2020-09-24: drm/i915 fixes for v5.9-rc7: - Fix selftest reference to stack data out of scope - Fix GVT

Re: [PATCH v2] MAINTAINERS: add Dan Murphy as TP LP8xxx drivers maintainer

2020-09-24 Thread Krzysztof Kozlowski
On Wed, 23 Sep 2020 at 22:01, Jonathan Cameron wrote: > > On Wed, 23 Sep 2020 11:53:33 -0500 > Dan Murphy wrote: > > > Hello > > > > On 9/22/20 10:28 AM, Krzysztof Kozlowski wrote: > > > Milo Kim's email in TI bounces with permanent error (550: Invalid > > > recipient). Last email from him on

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread peterz
On Wed, Sep 23, 2020 at 11:52:51AM -0400, Steven Rostedt wrote: > On Wed, 23 Sep 2020 10:40:32 +0200 > pet...@infradead.org wrote: > > > However, with migrate_disable() we can have each task preempted in a > > migrate_disable() region, worse we can stack them all on the _same_ CPU > > (super

Re: [PATCH 35/45] drm/vram-helper: move to invalidate callback.

2020-09-24 Thread Thomas Zimmermann
Hi Am 24.09.20 um 07:18 schrieb Dave Airlie: > From: Dave Airlie > > Signed-off-by: Dave Airlie > --- > drivers/gpu/drm/drm_gem_vram_helper.c | 16 ++-- > 1 file changed, 6 insertions(+), 10 deletions(-) > > diff --git a/drivers/gpu/drm/drm_gem_vram_helper.c >

[PATCH] drm/bridge: tc358764: restore connector support

2020-09-24 Thread Marek Szyprowski
This patch restores DRM connector registration in the TC358764 bridge driver and restores usage of the old drm_panel_* API, thus allows dynamic panel registration. This fixes panel operation on Exynos5250-based Arndale board. This is equivalent to the revert of the following commits: 1644127f83bc

Re: [PATCH 01/11] drm/ttm: add ttm_bo_pin()/ttm_bo_unpin() v2

2020-09-24 Thread Daniel Vetter
On Wed, Sep 23, 2020 at 01:01:14PM +1000, Dave Airlie wrote: > On Tue, 22 Sep 2020 at 23:32, Christian König > wrote: > > > > As an alternative to the placement flag add a > > pin count to the ttm buffer object. > > These all look good to me, nice cleanup. > > For the series: > Reviewed-by:

Re: [PATCH] drm/msm/dp: fix incorrect function prototype of dp_debug_get()

2020-09-24 Thread Stephen Boyd
Quoting Abhinav Kumar (2020-09-23 16:24:48) > Fix the incorrect function prototype for dp_debug_get() > in the dp_debug module to address compilation warning. > > Fixes: f913454aae8e ("drm/msm/dp: move debugfs node to > /sys/kernel/debug/dri/*/") > Reported-by: kernel test robot >

Re: [PATCH v4 11/23] device-dax: Kill dax_kmem_res

2020-09-24 Thread David Hildenbrand
On 23.09.20 23:41, Dan Williams wrote: > On Wed, Sep 23, 2020 at 1:04 AM David Hildenbrand wrote: >> >> On 08.09.20 17:33, Joao Martins wrote: >>> [Sorry for the late response] >>> >>> On 8/21/20 11:06 AM, David Hildenbrand wrote: On 03.08.20 07:03, Dan Williams wrote: > @@ -37,109

Re: [PATCH 1/2] drm/vc4: kms: Assign a FIFO to enabled CRTCs instead of active

2020-09-24 Thread Hoegeun Kwon
Hi Maxime, On 9/18/20 11:59 PM, Maxime Ripard wrote: > The HVS has three FIFOs that can be assigned to a number of PixelValves > through a mux. > > However, changing that FIFO requires that we disable and then enable the > pixelvalve, so we want to assign FIFOs to all the enabled CRTCs, and not >

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Steven Rostedt
On Thu, 24 Sep 2020 08:57:52 +0200 Thomas Gleixner wrote: > > Now as for migration disabled nesting, at least now we would have > > groupings of this, and perhaps the theorists can handle that. I mean, > > how is this much different that having a bunch of tasks blocked on a > > mutex with the

Re: [PATCH 37/45] drm/ttm: add a helper to allocate a temp tt for copies.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie All the accel moves do the same pattern here, provide a helper And exactly that pattern I want to get away from. See what happens if we (for example) have a VRAM -> SYSTEM move is the following: 1. TTM allocates a new

Re: [PATCH 26/45] drm/ttm: remove bind/unbind callbacks.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie If a driver wants to bind/unbind then it should implement a move callback. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c| 8 +++--

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Steven Rostedt
On Thu, 24 Sep 2020 14:42:41 +0200 Peter Zijlstra wrote: > On Thu, Sep 24, 2020 at 08:32:41AM -0400, Steven Rostedt wrote: > > Anyway, instead of blocking. What about having a counter of number of > > migrate disabled tasks per cpu, and when taking a migrate_disable(), and > > there's > >

Re: [PATCH 0/3] Prevent out-of-bounds access for built-in font data buffers

2020-09-24 Thread Greg Kroah-Hartman
On Thu, Sep 24, 2020 at 09:38:22AM -0400, Peilin Ye wrote: > Hi all, > > syzbot has reported [1] a global out-of-bounds read issue in > fbcon_get_font(). A malicious user may resize `vc_font.height` to a large > value in vt_ioctl(), causing fbcon_get_font() to overflow our built-in > font data

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Peter Zijlstra
On Thu, Sep 24, 2020 at 08:32:41AM -0400, Steven Rostedt wrote: > Anyway, instead of blocking. What about having a counter of number of > migrate disabled tasks per cpu, and when taking a migrate_disable(), and > there's > already another task with migrate_disabled() set, and the current task has

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Peter Zijlstra
On Thu, Sep 24, 2020 at 09:51:38AM -0400, Steven Rostedt wrote: > > It turns out, that getting selected for pull-balance is exactly that > > condition, and clearly a migrate_disable() task cannot be pulled, but we > > can use that signal to try and pull away the running task that's in the > >

Re: [PATCH 14/45] drm/ttm: add move old to system to drivers.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie Uninline ttm_bo_move_ttm. Eventually want to unhook the unbind out. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 9 - drivers/gpu/drm/nouveau/nouveau_bo.c

Re: [PATCH] MAINTAINERS: mark FRAMEBUFFER LAYER as Orphan

2020-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2020 at 1:25 PM Bartlomiej Zolnierkiewicz wrote: > > It has been a fun ride since 2017 but unfortunately I don't have > enough time to look after it properly anymore. > > Signed-off-by: Bartlomiej Zolnierkiewicz Thanks a lot for all the work you put in in picking up patches and

Re: [PATCH 15/45] drm/ttm: push copy unbind into drivers.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This uninlines ttm_bo_move_old_to_system into 3 places Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 7 ++- drivers/gpu/drm/nouveau/nouveau_bo.c| 7 +--

Re: [PATCH 30/45] drm/ttm: add a new invalidate notify callback.

2020-09-24 Thread Christian König
Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie Signed-off-by: Dave Airlie NAK, completely unnecessary. We should rather do the remaining accounting in the already existing release_notify() callback. That makes much more sense and if I'm not completely mistaken could

Re: [PATCH v4 11/23] device-dax: Kill dax_kmem_res

2020-09-24 Thread Dan Williams
On Thu, Sep 24, 2020 at 12:26 AM David Hildenbrand wrote: > > On 23.09.20 23:41, Dan Williams wrote: > > On Wed, Sep 23, 2020 at 1:04 AM David Hildenbrand wrote: > >> > >> On 08.09.20 17:33, Joao Martins wrote: > >>> [Sorry for the late response] > >>> > >>> On 8/21/20 11:06 AM, David

[PATCH v2 4/6] drm/dp: Add LTTPR helpers

2020-09-24 Thread Imre Deak
Add the helpers and register definitions needed to read out the common and per-PHY LTTPR capabilities and perform link training in the LTTPR non-transparent mode. v2: - Add drm_dp_dpcd_read_phy_link_status() and DP_PHY_LTTPR() here instead of adding these to i915. (Ville) Cc:

[PATCH v2 0/6] drm/i915: Add support for LTTPR non-transparent link training mode

2020-09-24 Thread Imre Deak
This patchset is v2 of [1], addressing the comments from Ville and an issue in the last patch with the per-PHY capability readout (needs to be done after switching to non-transparent mode). [1] https://patchwork.freedesktop.org/series/81968/ Cc: dri-devel@lists.freedesktop.org Cc: Ville Syrjälä

Re: [PATCH -next] drm/panel: simplify the return expression of td028ttec1_prepare

2020-09-24 Thread Sam Ravnborg
Hi Qinglang On Mon, Sep 21, 2020 at 09:10:18PM +0800, Qinglang Miao wrote: > Simplify the return expression. > > Signed-off-by: Qinglang Miao Thanks, applied to drm-misc-next. Sam > --- > drivers/gpu/drm/panel/panel-tpo-td028ttec1.c | 7 +-- > 1 file changed, 1 insertion(+), 6

Re: [PATCH v2] drm/panel: otm8009a: allow using non-continuous dsi clock

2020-09-24 Thread Sam Ravnborg
On Tue, Sep 22, 2020 at 09:42:53AM +0200, Yannick Fertre wrote: > From: Antonio Borneo > > The panel is able to work when dsi clock is non-continuous, thus > the system power consumption can be reduced using such feature. > > Add MIPI_DSI_CLOCK_NON_CONTINUOUS to panel's mode_flags. > > Changes

Re: [PATCH v2] drm/panel: otm8009a: remove hack to force commands in HS

2020-09-24 Thread Sam Ravnborg
On Tue, Sep 22, 2020 at 09:42:42AM +0200, Yannick Fertre wrote: > From: Antonio Borneo > > The panel is able to receive commands in LP. The current hack to > force backlight commands in HS was due to workaround an incorrect > settings on DSI controller that prevents sending LP commands while >

Re: [PATCH v4 11/23] device-dax: Kill dax_kmem_res

2020-09-24 Thread David Hildenbrand
On 24.09.20 15:54, Dan Williams wrote: > On Thu, Sep 24, 2020 at 12:26 AM David Hildenbrand wrote: >> >> On 23.09.20 23:41, Dan Williams wrote: >>> On Wed, Sep 23, 2020 at 1:04 AM David Hildenbrand wrote: On 08.09.20 17:33, Joao Martins wrote: > [Sorry for the late response] >

Re: [PATCH] drm/msm/dp: fix incorrect function prototype of dp_debug_get()

2020-09-24 Thread abhinavk
Hi Stephen Thanks for the review. On 2020-09-23 23:32, Stephen Boyd wrote: Quoting Abhinav Kumar (2020-09-23 16:24:48) Fix the incorrect function prototype for dp_debug_get() in the dp_debug module to address compilation warning. Fixes: f913454aae8e ("drm/msm/dp: move debugfs node to

Re: [RFC PATCH v1 3/3] dt-binding: display: Require two rests on mantix panel

2020-09-24 Thread Sam Ravnborg
Hi Guido. On Mon, Sep 21, 2020 at 06:55:52PM +0200, Guido Günther wrote: > We need to reset both for the panel to show an image. > > Signed-off-by: Guido Günther > --- > .../bindings/display/panel/mantix,mlaf057we51-x.yaml | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) >

Re: [PATCH 1/2] drm/bridge: sii902x: Enable I/O and core VCC supplies if present

2020-09-24 Thread Fabio Estevam
Hi Alexandre, On Thu, Sep 24, 2020 at 5:16 PM Alexandru Gagniuc wrote: > + ret = regulator_enable(sii902x->cvcc12); > + if (ret < 0) { > + dev_err(dev, "Failed to enable cvcc12 supply: %d\n", ret); > + regulator_disable(sii902x->iovcc); > +

Re: [PATCH 2/3] drm/ingenic: Reset pixclock rate when parent clock rate changes

2020-09-24 Thread Sam Ravnborg
Hi Paul. On Tue, Sep 15, 2020 at 02:38:17PM +0200, Paul Cercueil wrote: > Old Ingenic SoCs can overclock very well, up to +50% of their nominal > clock rate, whithout requiring overvolting or anything like that, just > by changing the rate of the main PLL. Unfortunately, all clocks on the >

Re: [PATCH] drm/stm: dsi: Avoid printing errors for -EPROBE_DEFER

2020-09-24 Thread Sam Ravnborg
Hi Yannick On Fri, Sep 18, 2020 at 01:46:24PM +0200, Yannick Fertre wrote: > Don't print error when probe deferred error is returned. > > Signed-off-by: Yannick Fertre > --- > drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git

Re: [PATCH v4 11/23] device-dax: Kill dax_kmem_res

2020-09-24 Thread Dan Williams
[..] > > I'm not suggesting to busy the whole "virtio" range, just the portion > > that's about to be passed to add_memory_driver_managed(). > > I'm afraid I don't get your point. For virtio-mem: > > Before: > > 1. Create virtio0 container resource > > 2. (somewhen in the future)

Re: [PATCH v4 11/23] device-dax: Kill dax_kmem_res

2020-09-24 Thread David Hildenbrand
> Am 24.09.2020 um 23:26 schrieb Dan Williams : > > [..] >>> I'm not suggesting to busy the whole "virtio" range, just the portion >>> that's about to be passed to add_memory_driver_managed(). >> >> I'm afraid I don't get your point. For virtio-mem: >> >> Before: >> >> 1. Create virtio0

Re: [PATCH] MAINTAINERS: mark FRAMEBUFFER LAYER as Orphan

2020-09-24 Thread Sam Ravnborg
Hi Bartlomiej On Thu, Sep 24, 2020 at 01:25:30PM +0200, Bartlomiej Zolnierkiewicz wrote: > It has been a fun ride since 2017 but unfortunately I don't have > enough time to look after it properly anymore. Thanks for all your work on fbdev, and other stuff. I hope you have fun in the current job

Re: [PATCH 10/11] x86/xen: open code alloc_vm_area in arch_gnttab_valloc

2020-09-24 Thread boris . ostrovsky
On 9/24/20 9:58 AM, Christoph Hellwig wrote: > Replace the last call to alloc_vm_area with an open coded version using > an iterator in struct gnttab_vm_area instead of the triple indirection > magic in alloc_vm_area. > > Signed-off-by: Christoph Hellwig Reviewed-by: Boris Ostrovsky

Re: [PATCH 09/11] xen/xenbus: use apply_to_page_range directly in xenbus_map_ring_pv

2020-09-24 Thread boris . ostrovsky
On 9/24/20 9:58 AM, Christoph Hellwig wrote: > Replacing alloc_vm_area with get_vm_area_caller + apply_page_range > allows to fill put the phys_addr values directly instead of doing > another loop over all addresses. > > Signed-off-by: Christoph Hellwig Reviewed-by: Boris Ostrovsky -boris

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

2020-09-24 Thread Steven Rostedt
On Thu, 24 Sep 2020 19:55:10 +0200 Thomas Gleixner wrote: > On Thu, Sep 24 2020 at 08:32, Steven Rostedt wrote: > > On Thu, 24 Sep 2020 08:57:52 +0200 > > Thomas Gleixner wrote: > > > >> > Now as for migration disabled nesting, at least now we would have > >> > groupings of this, and perhaps

Re: [PATCH] drm/panel: otm8009a: remove hack to force commands in HS

2020-09-24 Thread Sam Ravnborg
Hi Yannick On Fri, Sep 18, 2020 at 01:47:18PM +0200, Yannick Fertre wrote: > From: Antonio Borneo > > The panel is able to receive commands in LP. The current hack to > force backlight commands in HS was due to workaround an incorrect > settings on DSI controller that prevents sending LP

Re: [PATCH] drm/panel: otm8009a: allow using non-continuous dsi clock

2020-09-24 Thread Sam Ravnborg
On Fri, Sep 18, 2020 at 01:47:26PM +0200, Yannick Fertre wrote: > From: Antonio Borneo > > The panel is able to work when dsi clock is non-continuous, thus > the system power consumption can be reduced using such feature. > > Add MIPI_DSI_CLOCK_NON_CONTINUOUS to panel's mode_flags. > >

Re: [PATCH 1/3] drm/ingenic: Add support for 30-bit modes

2020-09-24 Thread Sam Ravnborg
On Tue, Sep 15, 2020 at 02:38:16PM +0200, Paul Cercueil wrote: > Starting from the JZ4760 SoC, the primary and overlay planes support > 30-bit pixel modes (10 bits per color component). Add support for these > in the ingenic-drm driver. > > Signed-off-by: Paul Cercueil Acked-by: Sam Ravnborg >

Re: [PATCH] MAINTAINERS: Add Thomas as reviewer for ast, mgag200 and udl

2020-09-24 Thread Sam Ravnborg
Hi Thomas. On Tue, Sep 15, 2020 at 09:17:08AM +0200, Thomas Zimmermann wrote: > I'm adding myself as reviewer for ast, mgag200 and udl. I've already > been keeping these drivers in shape for a while. In my mind you are maintainer of these and not just reviewer. I had expected you to take the

[gabbayo:habanalabs-next 4/5] WARNING: modpost: vmlinux.o(.text+0x1db5e10): Section mismatch in reference from the function tfrc_calc_x() to the variable .init.text:.L0

2020-09-24 Thread kernel test robot
tree: git://people.freedesktop.org/~gabbayo/linux habanalabs-next head: bbfc1bab1edc92023b1e144c1dceb341b31dc834 commit: 6ba70b882d6343bc2c86a6d09fbcad2c8170b6a6 [4/5] habanalabs: add notice of device not idle config: riscv-randconfig-r031-20200923 (attached as .config) compiler:

Re: [PATCH] drm/panel: rm68200: allow using non-continuous dsi clock

2020-09-24 Thread Sam Ravnborg
Hi Yannick On Fri, Sep 18, 2020 at 01:47:36PM +0200, Yannick Fertre wrote: > The panel is able to work when dsi clock is non-continuous, thus > the system power consumption can be reduced using such feature. > > Add MIPI_DSI_CLOCK_NON_CONTINUOUS to panel's mode_flags. > > Signed-off-by: Antonio

  1   2   >