Re: KASAN: slab-out-of-bounds Read in fbcon_get_font

2019-12-04 Thread syzbot
syzbot has bisected this bug to: commit 2de50e9674fc4ca3c6174b04477f69eb26b4ee31 Author: Russell Currey Date: Mon Feb 8 04:08:20 2016 + powerpc/powernv: Remove support for p5ioc2 bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=127a042ae0 start commit: 76bb8b05

cirrusfb: divide errors in cirrusfb_check_var/cirrusfb_check_pixclock/cirrusfb_set_par_foo

2019-12-04 Thread Dmitry Vyukov
Hello, syzkaller has found 3 of divide errors in the cirrusfb driver. Kernel is on c5db92909bedd Add linux-next specific files for 20191202. divide error: [#1] PREEMPT SMP KASAN CPU: 0 PID: 8133 Comm: syz-executor.5 Not tainted 5.4.0-next-20191202+ #13 Hardware name: QEMU Standard PC (Q35 +

Re: [PATCH] drm/modes: remove unused variables

2019-12-04 Thread Benjamin GAIGNARD
On 12/4/19 10:35 AM, Thomas Zimmermann wrote: > Hi > > Am 19.11.19 um 14:47 schrieb Benjamin Gaignard: >> When compiling with W=1 few warnings about unused variables show up. >> This patch removes all the involved variables. >> >> Signed-off-by: Benjamin Gaignard >> --- >>

RE: [PATCH] drm/edid: fixup EDID 1.3 and 1.4 judge reduced-blanking timings logic

2019-12-04 Thread allen.chen
Hi Jani Nikula Thanks for your suggestion and I have replied one comment below. -Original Message- From: Jani Nikula [mailto:jani.nik...@linux.intel.com] Sent: Tuesday, December 03, 2019 4:02 PM To: Allen Chen (陳柏宇) Cc: Jau-Chih Tseng (曾昭智); maxime.rip...@bootlin.com;

Re: [PATCH 2/2] drm/tegra: Do not implement runtime PM

2019-12-04 Thread Dmitry Osipenko
03.12.2019 19:27, Thierry Reding пишет: > From: Thierry Reding > > The Tegra DRM driver heavily relies on the implementations for runtime > suspend/resume to be called at specific times. Unfortunately, there are > some cases where that doesn't work. One example is if the user disables > runtime

Re: KASAN: slab-out-of-bounds Read in fbcon_get_font

2019-12-04 Thread Andrey Ryabinin
On 12/4/19 9:33 AM, Dmitry Vyukov wrote: > On Tue, Dec 3, 2019 at 11:37 PM Daniel Vetter wrote: >> >> On Tue, Dec 3, 2019 at 11:25 PM syzbot >> wrote: >>> >>> Hello, >>> >>> syzbot found the following crash on: >>> >>> HEAD commit:76bb8b05 Merge tag 'kbuild-v5.5' of

Re: KASAN: slab-out-of-bounds Read in fbcon_get_font

2019-12-04 Thread Tetsuo Handa
Hello. syzbot is reporting that memory allocation size at fbcon_set_font() is too small because font's height is rounded up from 10 to 16 after memory allocation. -- diff --git a/drivers/video/fbdev/core/fbcon.c b/drivers/video/fbdev/core/fbcon.c index c9235a2f42f8..68fe66e435d3 100644

KASAN: vmalloc-out-of-bounds Write in bitfill_aligned

2019-12-04 Thread syzbot
Hello, syzbot found the following crash on: HEAD commit:63de3747 Merge tag 'tag-chrome-platform-for-v5.5' of git:/.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=1040fc41e0 kernel config: https://syzkaller.appspot.com/x/.config?x=1d189d07c6717979

Re: [RFC PATCH v2 09/11] devfreq: exynos-bus: Add interconnect functionality to exynos-bus

2019-12-04 Thread Chanwoo Choi
Hi Artur, On 12/3/19 2:05 AM, Artur Świgoń wrote: > Hi Chanwoo, > > On Wed, 2019-09-25 at 16:03 +0900, Chanwoo Choi wrote: >> Hi, >> >> I need the time to dig the ICC framework >> to understand them detailed. After that, I'll review this. > > Any updates on this topic? I'm sorry for delaying

KASAN: slab-out-of-bounds Read in soft_cursor

2019-12-04 Thread syzbot
Hello, syzbot found the following crash on: HEAD commit:63de3747 Merge tag 'tag-chrome-platform-for-v5.5' of git:/.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=14819aeae0 kernel config: https://syzkaller.appspot.com/x/.config?x=1d189d07c6717979

Re: [v1,1/2] drm/mediatek: Fixup external display black screen issue

2019-12-04 Thread Hsin-Yi Wang
On Wed, Nov 27, 2019 at 1:17 AM wrote: > > From: Yongqiang Niu > > Problem: > overlay hangup when external display hotplut test > > Fix: > disable overlay when crtc disable > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 39 >

Re: [PATCH 2/2] dt-bindings: display: Add obsolete note to "ti, tilcdc, panel" binding

2019-12-04 Thread H. Nikolaus Schaller
> Am 04.12.2019 um 18:53 schrieb Jyri Sarha : > > The "ti,tilcdc,panel" binding should not be used anymore, since tilcdc > is fully capable of using generic drm panels like panel-simple and > panel-common binding. Is there an example how to convert? As far as I can see all

Re: [EXT] [PATCH v8 2/2] drm/bridge: Add NWL MIPI DSI host controller support

2019-12-04 Thread Robert Chiras
On Mi, 2019-12-04 at 13:06 +0100, Guido Günther wrote: > Hi Robert, > On Tue, Dec 03, 2019 at 09:50:03AM +, Robert Chiras wrote: > > > > Hi Guido, > > > > Since your last revision sent, I've done more tests here and found > > a > > few more improvements that could be added to this driver. >

Re: [v1,1/2] drm/mediatek: Fixup external display black screen issue

2019-12-04 Thread Yongqiang Niu
On Thu, 2019-12-05 at 14:15 +0800, Hsin-Yi Wang wrote: > On Wed, Nov 27, 2019 at 1:17 AM wrote: > > > > From: Yongqiang Niu > > > > Problem: > > overlay hangup when external display hotplut test > > > > Fix: > > disable overlay when crtc disable > > > > Signed-off-by: Yongqiang Niu > > --- > >

Re: [PATCH 2/2] dt-bindings: display: Add obsolete note to "ti,tilcdc,panel" binding

2019-12-04 Thread Jyri Sarha
On 05/12/2019 09:06, H. Nikolaus Schaller wrote: > >> Am 04.12.2019 um 18:53 schrieb Jyri Sarha : >> >> The "ti,tilcdc,panel" binding should not be used anymore, since tilcdc >> is fully capable of using generic drm panels like panel-simple and >> panel-common binding. > > Is there an example

Re: [PATCH v3 6/6] drm/mediatek: apply CMDQ control flow

2019-12-04 Thread CK Hu
Hi, Bibby: On Wed, 2019-12-04 at 17:44 +0800, Bibby Hsieh wrote: > Unlike other SoCs, MT8183 does not have "shadow" > registers for performaing an atomic video mode > set or page flip at vblank/vsync. > > The CMDQ (Commend Queue) in MT8183 is used to help > update all relevant display controller

Re: [PATCH v3 5/6] drm/mediatek: support CMDQ interface in ddp component

2019-12-04 Thread CK Hu
Hi, Bibby: On Wed, 2019-12-04 at 17:44 +0800, Bibby Hsieh wrote: > The CMDQ (Command Queue) in MT8183 is used to help > update all relevant display controller registers > with critical time limation. > This patch add cmdq interface in ddp_comp interface, > let all ddp_comp interface can support

Re: [PATCH v3 3/6] drm/mediatek: update cursors by using async atomic update

2019-12-04 Thread CK Hu
Hi, Bibby: On Wed, 2019-12-04 at 17:44 +0800, Bibby Hsieh wrote: > Support to async updates of cursors by using the new atomic > interface for that. > > Signed-off-by: Bibby Hsieh > --- > drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 75 +--- >

Re: [PATCH v3 05/12] video: fbdev: make fbops member of struct fb_info a const pointer

2019-12-04 Thread kbuild test robot
to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Jani-Nikula/video-drm-etc-constify-fbops-in-struct-fb_info/20191204-055320 base: git

Re: [PATCH 5/5] arm: dts: sc7180: Add A618 gpu dt blob

2019-12-04 Thread Matthias Kaehlcke
Hi Sharat, on which tree is this patch based on? It does not apply against qcom/arm64-for-5.6-to-be-rebased. In one of my repos which has a non-upstream Qualcomm tree as remote git can make sense of the hashes in the index line, however the parent of your patch looks quite different from the

Re: [PATCH] MAINTAINERS: Match on dma_buf|fence|resv anywhere

2019-12-04 Thread Alex Deucher
On Wed, Dec 4, 2019 at 4:51 PM Daniel Vetter wrote: > > I've spent a bit too much time reviewing all kinds of users all over > the kernel for this buffer sharing infrastructure. And some of it is > at least questionable. > > Make sure we at least see when this stuff flies by. > > Acked-by: Dave

Re: [RFC 06/13] drm/i915/svm: Page table mirroring support

2019-12-04 Thread Jerome Glisse
On Tue, Dec 03, 2019 at 11:19:43AM -0800, Niranjan Vishwanathapura wrote: > On Tue, Nov 26, 2019 at 06:32:52PM +, Jason Gunthorpe wrote: > > On Mon, Nov 25, 2019 at 11:33:27AM -0500, Jerome Glisse wrote: > > > On Fri, Nov 22, 2019 at 11:33:12PM +, Jason Gunthorpe wrote: > > > > On Fri, Nov

[PATCH] MAINTAINERS: Match on dma_buf|fence|resv anywhere

2019-12-04 Thread Daniel Vetter
I've spent a bit too much time reviewing all kinds of users all over the kernel for this buffer sharing infrastructure. And some of it is at least questionable. Make sure we at least see when this stuff flies by. Acked-by: Dave Airlie Signed-off-by: Daniel Vetter Cc: Sumit Semwal Cc: Mauro

[PULL] drm-misc-next-fixes

2019-12-04 Thread Sean Paul
Hi Dave & Daniel, A handful of fixes this week, all straightforward. drm-misc-next-fixes-2019-12-04: mgag200- Fix hw with broken 'startadd' support (Thomas) mst- Avoid skipping payloads in payload deletion loop (Wayne) omap- Fix dma_addr refcounting (Tomi) Cc: Wayne Lin Cc: Tomi Valkeinen

Re: [PATCH v9] drm/dp_mst: Add PBN calculation for DSC modes

2019-12-04 Thread Mikita Lipski
On 12/4/19 11:45 AM, Jani Nikula wrote: On Tue, 03 Dec 2019, wrote: From: David Francis With DSC, bpp can be fractional in multiples of 1/16. Can be? I worry a bit that "bpp" can either be integer or fixed point depending on other variables. I admit I haven't followed up on this too

Re: [PATCH v2] drm/dp_mst: Correct the bug in drm_dp_update_payload_part1()

2019-12-04 Thread Lyude Paul
Just pushed to drm-misc-next-fixes as e5a6ca27eb72c67533ddfc11c06df84beaa167fa, thanks! On Tue, 2019-12-03 at 12:24 +0800, Wayne Lin wrote: > [Why] > If the payload_state is DP_PAYLOAD_DELETE_LOCAL in series, current > code doesn't delete the payload at current index and just move the > index to

Re: [PATCH 2/2] drm/panel: simple: Add Rocktech RK101II01D-CT panel

2019-12-04 Thread Tomi Valkeinen
On 28/11/2019 18:45, Jyri Sarha wrote: Add support for Rocktech RK101II01D-CT, 10.1" 1280x800 TFT with LVDS interface, LED backlight and integrated Goodix GT928 capacitive touch panel. Signed-off-by: Jyri Sarha --- drivers/gpu/drm/panel/panel-simple.c | 32 1

Re: [PATCH] dt-bindings: display: Convert a bunch of panels to DT schema

2019-12-04 Thread Sam Ravnborg
Hi Rob. On Mon, Dec 02, 2019 at 08:38:39AM -0600, Rob Herring wrote: > On Sat, Nov 30, 2019 at 1:43 PM Sam Ravnborg wrote: > > > > Hi Rob. > > > > Thanks for doing this boring, trivial and tiresome task. > > It was somewhat scripted at least... > > > On Tue, Nov 19, 2019 at 05:13:09PM -0600,

Re: [PATCH v2 1/4] dt-bindings: drm/msm/gpu: document second interconnect

2019-12-04 Thread Rob Herring
On Thu, 21 Nov 2019 20:26:42 -0500, Brian Masney wrote: > Some A3xx and all A4xx Adreno GPUs do not have GMEM inside the GPU core > and must use the On Chip MEMory (OCMEM) in order to be functional. > There's a separate interconnect path that needs to be setup to OCMEM. > Let's document this

Re: [PATCH v6 03/15] drm/bridge: tc358767: Simplify polling in tc_link_training()

2019-12-04 Thread Tomi Valkeinen
On 04/12/2019 20:27, Tomi Valkeinen wrote: Hi Andrey, On 19/06/2019 08:27, Andrey Smirnov wrote: @@ -748,22 +748,19 @@ static int tc_set_video_mode(struct tc_data *tc,   static int tc_wait_link_training(struct tc_data *tc)   { -    u32 timeout = 1000;   u32 value;   int ret; -    do {

[PATCH] drm/gma500: Call psb_driver_{load, unload}() before registering device

2019-12-04 Thread Thomas Zimmermann
The load/unload callbacks in struct drm_driver are deprecated. Remove them and call functions explicitly. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/gma500/psb_drv.c | 43 +++- 1 file changed, 37 insertions(+), 6 deletions(-) diff --git

Re: [PATCH v6 03/15] drm/bridge: tc358767: Simplify polling in tc_link_training()

2019-12-04 Thread Tomi Valkeinen
Hi Andrey, On 19/06/2019 08:27, Andrey Smirnov wrote: @@ -748,22 +748,19 @@ static int tc_set_video_mode(struct tc_data *tc, static int tc_wait_link_training(struct tc_data *tc) { - u32 timeout = 1000; u32 value; int ret; - do { - udelay(1); -

[PATCH v4] drm: Add support for DP 1.4 Compliance edid corruption test

2019-12-04 Thread Jerry (Fangzhi) Zuo
Unlike DP 1.2 edid corruption test, DP 1.4 requires to calculate real CRC value of the last edid data block, and write it back. Current edid CRC calculates routine adds the last CRC byte, and check if non-zero. This behavior is not accurate; actually, we need to return the actual CRC value when

[PATCH 1/2] drm/tilcdc: Remove obsolete bundled tilcdc tfp410 driver

2019-12-04 Thread Jyri Sarha
Remove obsolete bundled tfp410 driver with its "ti,tilcdc,tfp410" devicetree binding. No platform has ever used this driver in the mainline kernel and if anybody connects tfp410 to tilcdc he or she should use the generic drm tfp410 bridge driver. Signed-off-by: Jyri Sarha ---

[PATCH 2/2] dt-bindings: display: Add obsolete note to "ti, tilcdc, panel" binding

2019-12-04 Thread Jyri Sarha
The "ti,tilcdc,panel" binding should not be used anymore, since tilcdc is fully capable of using generic drm panels like panel-simple and panel-common binding. However, the obsolete binding is still widely used in many mainline supported platforms that I do not have access to and who knows how

[PATCH 0/2] drm/tilcdc: Remove "ti, tilcdc, tfp410" and mark "ti, tilcdc, panel" obsolete

2019-12-04 Thread Jyri Sarha
Nobody should any more use the old tilcdc bundled drivers anymore. The driver is fully capable of using the generic drm bridge and panel drivers. However, the obsolete "ti,tilcdc,panel" binding is still widely used in many mainline supported platforms that I do not have access to and who knows

Re: [PATCH] drm/omap: fix dma_addr refcounting

2019-12-04 Thread Tomi Valkeinen
On 02/12/2019 14:36, Laurent Pinchart wrote: Hi Tomi, Thank you for the patch. On Thu, Nov 14, 2019 at 10:03:43AM +0200, Tomi Valkeinen wrote: cec4fa7511ef7a73eb635834e9d85b25a5b47a98 ("drm/omap: use refcount API to track the number of users of dma_addr") changed omap_gem.c to use refcounting

Re: [PATCH v3] drm/dp_mst: Fix W=1 warnings

2019-12-04 Thread Jani Nikula
On Thu, 28 Nov 2019, Benjamin Gaignard wrote: > Fix the warnings that show up with W=1. > They are all about unused but set variables. > If functions returns are not used anymore make them void. > > Signed-off-by: Benjamin Gaignard > --- > CC: Jani Nikula > > changes in version 3: > - remove

Re: [PATCH v9] drm/dp_mst: Add PBN calculation for DSC modes

2019-12-04 Thread Jani Nikula
On Tue, 03 Dec 2019, wrote: > From: David Francis > > With DSC, bpp can be fractional in multiples of 1/16. Can be? I worry a bit that "bpp" can either be integer or fixed point depending on other variables. I admit I haven't followed up on this too much, but how widespread it is? It seems

Re: [PATCH 3/7] drm/udl: Vmap framebuffer after all tests succeeded in damage handling

2019-12-04 Thread Thomas Zimmermann
Hi Sam Am 04.12.19 um 15:25 schrieb Sam Ravnborg: > Hi Thomas. > > I did a casual browse of the patches. > Looks like some nice cleanup. > > On Wed, Dec 04, 2019 at 02:24:26PM +0100, Thomas Zimmermann wrote: >> We now do the fast tests before the potentially expensive vmap operation. >> >>

Re: [PATCH 6/8] drm: Add a drm_get_unmapped_area() helper

2019-12-04 Thread VMware
On 12/4/19 3:40 PM, Christian König wrote: Am 04.12.19 um 13:32 schrieb Thomas Hellström (VMware): On 12/4/19 1:08 PM, Christian König wrote: Am 04.12.19 um 12:36 schrieb Thomas Hellström (VMware): On 12/4/19 12:11 PM, Christian König wrote: Am 03.12.19 um 14:22 schrieb Thomas Hellström

Re: [PATCH v2 2/2] drm/ttm: Fix vm page protection handling

2019-12-04 Thread VMware
On 12/4/19 4:26 PM, Michal Hocko wrote: On Wed 04-12-19 16:19:27, Thomas Hellström (VMware) wrote: On 12/4/19 3:42 PM, Michal Hocko wrote: On Wed 04-12-19 15:36:58, Thomas Hellström (VMware) wrote: On 12/4/19 3:35 PM, Michal Hocko wrote: On Wed 04-12-19 15:16:09, Thomas Hellström (VMware)

[PATCH] OMAP: DSS2: remove non-zero check on variable r

2019-12-04 Thread Colin King
From: Colin Ian King Variable r is being initialized to zero, so the check of a non-zero rv is redundant and can be removed. It appears that the previous case statements set r to be -EINVAL and the "Fallthrough" comment afterwards suggested it was going to fall through to this non-zero check

Re: [PATCH v2 2/2] drm/ttm: Fix vm page protection handling

2019-12-04 Thread Michal Hocko
On Wed 04-12-19 16:19:27, Thomas Hellström (VMware) wrote: > On 12/4/19 3:42 PM, Michal Hocko wrote: > > On Wed 04-12-19 15:36:58, Thomas Hellström (VMware) wrote: > > > On 12/4/19 3:35 PM, Michal Hocko wrote: > > > > On Wed 04-12-19 15:16:09, Thomas Hellström (VMware) wrote: > > > > > On 12/4/19

Re: [PATCH v2 2/2] drm/ttm: Fix vm page protection handling

2019-12-04 Thread VMware
On 12/4/19 3:42 PM, Michal Hocko wrote: On Wed 04-12-19 15:36:58, Thomas Hellström (VMware) wrote: On 12/4/19 3:35 PM, Michal Hocko wrote: On Wed 04-12-19 15:16:09, Thomas Hellström (VMware) wrote: On 12/4/19 2:52 PM, Michal Hocko wrote: On Tue 03-12-19 11:48:53, Thomas Hellström (VMware)

Re: [PATCH v2 2/2] drm/ttm: Fix vm page protection handling

2019-12-04 Thread Michal Hocko
On Wed 04-12-19 15:36:58, Thomas Hellström (VMware) wrote: > On 12/4/19 3:35 PM, Michal Hocko wrote: > > On Wed 04-12-19 15:16:09, Thomas Hellström (VMware) wrote: > > > On 12/4/19 2:52 PM, Michal Hocko wrote: > > > > On Tue 03-12-19 11:48:53, Thomas Hellström (VMware) wrote: > > > > > From:

Re: [PATCH 6/8] drm: Add a drm_get_unmapped_area() helper

2019-12-04 Thread Christian König
Am 04.12.19 um 13:32 schrieb Thomas Hellström (VMware): On 12/4/19 1:08 PM, Christian König wrote: Am 04.12.19 um 12:36 schrieb Thomas Hellström (VMware): On 12/4/19 12:11 PM, Christian König wrote: Am 03.12.19 um 14:22 schrieb Thomas Hellström (VMware): From: Thomas Hellstrom This helper

Re: [PATCH v2 2/2] drm/ttm: Fix vm page protection handling

2019-12-04 Thread VMware
On 12/4/19 3:35 PM, Michal Hocko wrote: On Wed 04-12-19 15:16:09, Thomas Hellström (VMware) wrote: On 12/4/19 2:52 PM, Michal Hocko wrote: On Tue 03-12-19 11:48:53, Thomas Hellström (VMware) wrote: From: Thomas Hellstrom TTM graphics buffer objects may, transparently to user-space, move

Re: [PATCH v2 2/2] drm/ttm: Fix vm page protection handling

2019-12-04 Thread Michal Hocko
On Wed 04-12-19 15:16:09, Thomas Hellström (VMware) wrote: > On 12/4/19 2:52 PM, Michal Hocko wrote: > > On Tue 03-12-19 11:48:53, Thomas Hellström (VMware) wrote: > > > From: Thomas Hellstrom > > > > > > TTM graphics buffer objects may, transparently to user-space, move > > > between IO and

Re: [PATCH] drm/tilcdc: Switch to using GPIO descriptors

2019-12-04 Thread Sam Ravnborg
Hi Jyri. On Tue, Dec 03, 2019 at 08:29:45PM +0200, Jyri Sarha wrote: > On 03/12/2019 15:09, Linus Walleij wrote: > > The TI LCDC picks a GPIO line from the device tree to use > > for DPMS power on/off. We can switch this to use a GPIO > > descriptor pretty easily. Make sure to request the GPIO >

Re: [PATCH 3/7] drm/udl: Vmap framebuffer after all tests succeeded in damage handling

2019-12-04 Thread Sam Ravnborg
Hi Thomas. I did a casual browse of the patches. Looks like some nice cleanup. On Wed, Dec 04, 2019 at 02:24:26PM +0100, Thomas Zimmermann wrote: > We now do the fast tests before the potentially expensive vmap operation. > > Signed-off-by: Thomas Zimmermann > --- >

Re: [PATCH v2 2/2] drm/ttm: Fix vm page protection handling

2019-12-04 Thread VMware
On 12/4/19 2:52 PM, Michal Hocko wrote: On Tue 03-12-19 11:48:53, Thomas Hellström (VMware) wrote: From: Thomas Hellstrom TTM graphics buffer objects may, transparently to user-space, move between IO and system memory. When that happens, all PTEs pointing to the old location are zapped

Re: [PATCH 7/8] drm/ttm: Introduce a huge page aligning TTM range manager.

2019-12-04 Thread Christian König
Am 04.12.19 um 14:18 schrieb Thomas Hellström (VMware): On 12/4/19 1:16 PM, Christian König wrote: Am 04.12.19 um 12:45 schrieb Thomas Hellström (VMware): On 12/4/19 12:13 PM, Christian König wrote: Am 03.12.19 um 14:22 schrieb Thomas Hellström (VMware): From: Thomas Hellstrom Using huge

Re: [PATCH v2 2/2] drm/ttm: Fix vm page protection handling

2019-12-04 Thread Michal Hocko
On Tue 03-12-19 11:48:53, Thomas Hellström (VMware) wrote: > From: Thomas Hellstrom > > TTM graphics buffer objects may, transparently to user-space, move > between IO and system memory. When that happens, all PTEs pointing to the > old location are zapped before the move and then faulted in

Re: [PATCH] drm: Funnel drm logs to tracepoints

2019-12-04 Thread Sean Paul
On Wed, Dec 04, 2019 at 02:02:51PM +0100, Daniel Vetter wrote: > On Wed, Dec 4, 2019 at 10:54 AM Pekka Paalanen wrote: > > > > On Wed, 4 Dec 2019 10:14:11 +0100 > > Daniel Vetter wrote: > > > > > On Wed, Dec 4, 2019 at 8:33 AM Pekka Paalanen wrote: > > > > > > > > On Tue, 3 Dec 2019 22:20:14

Re: [PATCH v2 1/2] mm: Add and export vmf_insert_mixed_prot()

2019-12-04 Thread Michal Hocko
On Tue 03-12-19 11:48:52, Thomas Hellström (VMware) wrote: > From: Thomas Hellstrom > > The TTM module today uses a hack to be able to set a different page > protection than struct vm_area_struct::vm_page_prot. To be able to do > this properly, add and export vmf_insert_mixed_prot(). A new

Re: [PATCH v4 00/11] drm: Add support for bus-format negotiation

2019-12-04 Thread Neil Armstrong
On 03/12/2019 19:19, Laurent Pinchart wrote: > Hi Boris, > > On Tue, Dec 03, 2019 at 03:15:04PM +0100, Boris Brezillon wrote: >> This patch series aims at adding support for runtime bus-format >> negotiation between all elements of the >> 'encoder -> bridges -> connector/display' section of the

Re: [PATCH v12 1/7] dt-bindings: sun6i-dsi: Document A64 MIPI-DSI controller

2019-12-04 Thread Maxime Ripard
On Tue, Dec 03, 2019 at 07:18:10PM +0530, Jagan Teki wrote: > The MIPI DSI controller in Allwinner A64 is similar to A33. > > But unlike A33, A64 doesn't have DSI_SCLK gating so it is valid > to have separate compatible for A64 on the same driver. > > DSI_SCLK uses mod clock-names on dt-bindings,

[PATCH] drm/mgag200: Debug-print unique revisions id on G200 SE

2019-12-04 Thread Thomas Zimmermann
The behavior of MGA G200 SE depends on the rev id. Print the id when debugging is enabled. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/mgag200/mgag200_main.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c

[PATCH 0/7] drm/udl: Prepare damage handler for simple-pipe KMS

2019-12-04 Thread Thomas Zimmermann
Udl's damage-handling code requires clean up before switching the driver to simple-pipe helpers. Patches 1, 2 and 7 remove unused state variables and statistics. Patches 3 to 5 reorganizes the damage handler to be more readable. Patch 6 moves dma-buf begin/end calls into the damage handler, so it

[PATCH 4/7] drm/udl: Move clip-rectangle code out of udl_handle_damage()

2019-12-04 Thread Thomas Zimmermann
Computing the clip rectable in a separate helper function makes the damage-handler code more readable. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/udl/udl_fb.c | 48 1 file changed, 32 insertions(+), 16 deletions(-) diff --git

[PATCH 6/7] drm/udl: Begin/end access to imported buffers in damage-handler

2019-12-04 Thread Thomas Zimmermann
The damage-handler code now invokes dma_buf_{begin,end}_access() for imported buffers. These calls were missing from the page-flip and modesetting code paths. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/udl/udl_fb.c | 38 ++-- 1 file changed, 19

[PATCH 1/7] drm/udl: Remove unused statistics counters

2019-12-04 Thread Thomas Zimmermann
None of the udl driver's statistics counters is used anywhere. Remove them. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/udl/udl_drv.h | 4 drivers/gpu/drm/udl/udl_fb.c | 14 +- 2 files changed, 1 insertion(+), 17 deletions(-) diff --git

[PATCH 7/7] drm/udl: Remove field lost_pixels from struct udl_device

2019-12-04 Thread Thomas Zimmermann
The field lost_pixels in struct udl_device was supposed to signal an error during USB transfers of the framebuffer data. The driver would have to schedule a re-transfer at a later point. This code was never implemented. Remove lost_pixels and return regular error codes instead. Signed-off-by:

[PATCH 5/7] drm/udl: Move log-cpp code out of udl_damage_handler()

2019-12-04 Thread Thomas Zimmermann
Computing the cpp value's logarithm in a separate helper function makes the damage-handler code more readable. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/udl/udl_fb.c | 17 - 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/udl/udl_fb.c

[PATCH 3/7] drm/udl: Vmap framebuffer after all tests succeeded in damage handling

2019-12-04 Thread Thomas Zimmermann
We now do the fast tests before the potentially expensive vmap operation. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/udl/udl_fb.c | 19 +++ drivers/gpu/drm/udl/udl_transfer.c | 1 - 2 files changed, 7 insertions(+), 13 deletions(-) diff --git

[PATCH 2/7] drm/udl: Don't track number of identical and sent pixels per line

2019-12-04 Thread Thomas Zimmermann
A call to udl_render_hline() returns the number of identical and sent pixels. None of these values is used. Remove the parameters. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/udl/udl_drv.h | 3 +-- drivers/gpu/drm/udl/udl_fb.c | 6 +- drivers/gpu/drm/udl/udl_transfer.c |

Re: [PATCH 7/8] drm/ttm: Introduce a huge page aligning TTM range manager.

2019-12-04 Thread VMware
On 12/4/19 1:16 PM, Christian König wrote: Am 04.12.19 um 12:45 schrieb Thomas Hellström (VMware): On 12/4/19 12:13 PM, Christian König wrote: Am 03.12.19 um 14:22 schrieb Thomas Hellström (VMware): From: Thomas Hellstrom Using huge page-table entries require that the start of a buffer

Re: [PATCH v1 05/26] drm/panel: add drm_connector argument to get_modes()

2019-12-04 Thread Laurent Pinchart
Hi Guido, On Wed, Dec 04, 2019 at 01:08:04PM +0100, Guido Günther wrote: > On Mon, Dec 02, 2019 at 08:32:09PM +0100, Sam Ravnborg wrote: > > Today the bridge creates the drm_connector, but that is planned > > to be moved to the display drivers. > > Do you have a reference for that move at hand?

Re: [PATCH v2 01/28] drm: Introduce drm_bridge_init()

2019-12-04 Thread Daniel Vetter
On Wed, Dec 04, 2019 at 11:48:02AM +, Mihail Atanassov wrote: > A simple convenience function to initialize the struct drm_bridge. The > goal is to standardize initialization for any bridge registered with > drm_bridge_add() so that we can later add device links for consumers of > those

Re: [PATCH] drm: Funnel drm logs to tracepoints

2019-12-04 Thread Daniel Vetter
On Wed, Dec 4, 2019 at 10:54 AM Pekka Paalanen wrote: > > On Wed, 4 Dec 2019 10:14:11 +0100 > Daniel Vetter wrote: > > > On Wed, Dec 4, 2019 at 8:33 AM Pekka Paalanen wrote: > > > > > > On Tue, 3 Dec 2019 22:20:14 +0100 > > > Daniel Vetter wrote: > > > > > > > On Tue, Dec 3, 2019 at 8:10 PM

[GIT PULL] drm/tegra: Fixes for v5.5-rc1

2019-12-04 Thread Thierry Reding
Hi Dave, The following changes since commit 84db889e6d827eefc3dde130fec8382d2dcb23ac: drm/tegra: Unconditionally select IOMMU_IOVA (2019-11-01 10:49:29 +0100) are available in the Git repository at: git://anongit.freedesktop.org/tegra/linux tags/drm/tegra/for-5.5-rc1-fixes for you to

Re: [PATCH 6/8] drm: Add a drm_get_unmapped_area() helper

2019-12-04 Thread VMware
On 12/4/19 1:08 PM, Christian König wrote: Am 04.12.19 um 12:36 schrieb Thomas Hellström (VMware): On 12/4/19 12:11 PM, Christian König wrote: Am 03.12.19 um 14:22 schrieb Thomas Hellström (VMware): From: Thomas Hellstrom This helper is used to align user-space buffer object addresses to

Re: [PATCH 7/8] drm/ttm: Introduce a huge page aligning TTM range manager.

2019-12-04 Thread Christian König
Am 04.12.19 um 12:45 schrieb Thomas Hellström (VMware): On 12/4/19 12:13 PM, Christian König wrote: Am 03.12.19 um 14:22 schrieb Thomas Hellström (VMware): From: Thomas Hellstrom Using huge page-table entries require that the start of a buffer object is huge page size aligned. So introduce a

Re: [PATCH 6/8] drm: Add a drm_get_unmapped_area() helper

2019-12-04 Thread Christian König
Am 04.12.19 um 12:36 schrieb Thomas Hellström (VMware): On 12/4/19 12:11 PM, Christian König wrote: Am 03.12.19 um 14:22 schrieb Thomas Hellström (VMware): From: Thomas Hellstrom This helper is used to align user-space buffer object addresses to huge page boundaries, minimizing the chance of

Re: [PATCH v1 05/26] drm/panel: add drm_connector argument to get_modes()

2019-12-04 Thread Guido Günther
Hi Sam, On Mon, Dec 02, 2019 at 08:32:09PM +0100, Sam Ravnborg wrote: > Today the bridge creates the drm_connector, but that is planned > to be moved to the display drivers. Do you have a reference for that move at hand? Cheers, -- Guido > To facilitate this, update drm_panel_funcs.get_modes()

Re: [EXT] [PATCH v8 2/2] drm/bridge: Add NWL MIPI DSI host controller support

2019-12-04 Thread Guido Günther
Hi Robert, On Tue, Dec 03, 2019 at 09:50:03AM +, Robert Chiras wrote: > Hi Guido, > > Since your last revision sent, I've done more tests here and found a > few more improvements that could be added to this driver. > See inline. > > On Lu, 2019-12-02 at 20:35 +0100, Guido Günther wrote: > >

Re: [PATCH v1 18/26] drm/panel: rocktech-jh057n00900: use drm_panel backlight support

2019-12-04 Thread Guido Günther
Hi, On Mon, Dec 02, 2019 at 08:32:22PM +0100, Sam Ravnborg wrote: > Use the backlight support in drm_panel to simplify the driver. > While touching the include files sort them > and divide them up in blocks. > > Signed-off-by: Sam Ravnborg > Cc: "Guido Günther" > Cc: Purism Kernel Team > Cc:

[PATCH v2 03/28] drm/bridge/analogix: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. The setting of bridge->of_node by drm_bridge_init() in analogix_dp_core.c is safe, since ->of_node isn't used directly and the bridge isn't published with drm_bridge_add(). Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/analogix/analogix-anx6345.c | 5 ++---

[PATCH v2 00/28] drm/bridge: Consolidate initialization

2019-12-04 Thread Mihail Atanassov
Hi all, I've dropped the fun parts of this series since they need more work, but figured drm_bridge_init() as a concept is still valuable on its own (and I think I'll need it to roll out device links for registered bridges), so here goes. v2: - expanded commit messages and added some extra

[PATCH v2 22/28] drm/i2c: tda998x: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/i2c/tda998x_drv.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c index a63790d32d75..f7dfa694aff7 100644 ---

[PATCH v2 26/28] drm: rcar-du: lvds: Don't set drm_bridge private pointer

2019-12-04 Thread Mihail Atanassov
No functional change: it's not used anywhere. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/rcar-du/rcar_lvds.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c index ac1f29bacfcb..168a718cbcbd

[PATCH v2 07/28] drm/bridge: megachips-stdpxxxx-ge-b850v3-fw: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/megachips-stdp-ge-b850v3-fw.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/megachips-stdp-ge-b850v3-fw.c

[PATCH v2 25/28] drm: rcar-du: lvds: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/rcar-du/rcar_lvds.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c index 8c6c172bbf2e..ac1f29bacfcb 100644 ---

[PATCH v2 21/28] drm/exynos: mic: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change: no logic depends on driver_private being NULL, so it's safe to set it earlier in exynos_mic_probe. Acked-by: Inki Dae Tested-by: Inki Dae Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/exynos/exynos_drm_mic.c | 8 +--- 1 file changed, 1 insertion(+), 7

[PATCH v2 28/28] drm/msm: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change: drm_bridge_init() sets bridge->of_node, but that's not used by msm anywhere, and the bridges aren't published with drm_bridge_add() for it to matter. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/msm/dsi/dsi_manager.c | 4 ++-- drivers/gpu/drm/msm/edp/edp_bridge.c

[PATCH v2 11/28] drm/bridge: sii902x: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/sii902x.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/sii902x.c b/drivers/gpu/drm/bridge/sii902x.c index b70e8c5cf2e1..2a9db621484d 100644 ---

[PATCH v2 24/28] drm/mediatek: hdmi: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/mediatek/mtk_hdmi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_hdmi.c b/drivers/gpu/drm/mediatek/mtk_hdmi.c index f684947c5243..9761a80674d9 100644 ---

[PATCH v2 23/28] drm/mcde: dsi: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Acked-by: Linus Walleij Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/mcde/mcde_dsi.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/mcde/mcde_dsi.c b/drivers/gpu/drm/mcde/mcde_dsi.c index 42fff811653e..d9b9253acccf 100644 ---

[PATCH v2 14/28] drm/bridge/synopsys: dw-hdmi: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c index dbe38a54870b..6c71ffc9df5a

[PATCH v2 18/28] drm/bridge: thc63: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/thc63lvd1024.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/bridge/thc63lvd1024.c b/drivers/gpu/drm/bridge/thc63lvd1024.c index 3d74129b2995..abe806db5f4d 100644 ---

[PATCH v2 12/28] gpu: drm: bridge: sii9234: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/sii9234.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/sii9234.c b/drivers/gpu/drm/bridge/sii9234.c index f81f81b7051f..bfd3832baa1a 100644 ---

[PATCH v2 08/28] drm/bridge: nxp-ptn3460: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/nxp-ptn3460.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/nxp-ptn3460.c b/drivers/gpu/drm/bridge/nxp-ptn3460.c index 57ff01339559..2656a188b434 100644 ---

[PATCH v2 05/28] drm/bridge: dumb-vga-dac: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/dumb-vga-dac.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/bridge/dumb-vga-dac.c b/drivers/gpu/drm/bridge/dumb-vga-dac.c index cc33dc411b9e..896f27272e38 100644 ---

[PATCH v2 27/28] drm/sti: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. v2: - Also apply drm_bridge_init() in sti_hdmi.c and sti_hda.c (Sam, Benjamin) Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/sti/sti_dvo.c | 4 +--- drivers/gpu/drm/sti/sti_hda.c | 3 +-- drivers/gpu/drm/sti/sti_hdmi.c | 3 +-- 3 files changed, 3

[PATCH v2 17/28] drm/bridge: tc358767: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/tc358767.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c index 8029478ffebb..7846c1925cbb 100644 ---

[PATCH v2 09/28] drm/bridge: panel: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/panel.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c index f4e293e7cf64..91d68d0337cc 100644 ---

[PATCH v2 19/28] drm/bridge: ti-sn65dsi86: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/ti-sn65dsi86.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c b/drivers/gpu/drm/bridge/ti-sn65dsi86.c index 43abf01ebd4c..4e236b46f913 100644 ---

[PATCH v2 13/28] drm/bridge: sil_sii8620: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/sil-sii8620.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/sil-sii8620.c b/drivers/gpu/drm/bridge/sil-sii8620.c index 4c0eef406eb1..482dc2291350 100644 ---

[PATCH v2 20/28] drm/bridge: ti-tfp410: Use drm_bridge_init()

2019-12-04 Thread Mihail Atanassov
No functional change. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/ti-tfp410.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/ti-tfp410.c b/drivers/gpu/drm/bridge/ti-tfp410.c index aa3198dc9903..377eca989ff3 100644 ---

  1   2   >