Re: [PATCH] drm/radeon: Convert get_user_pages() --> pin_user_pages()

2020-05-29 Thread John Hubbard
On 2020-05-28 23:49, Souptick Joarder wrote: ... This is what case 3 was *intended* to cover, but it looks like case 3 needs to be written a little better. I'll attempt that, and Cc you on the actual patch to -mm. (I think we also need a case 5 for an unrelated scenario, too, so it's time.) The

[Bug 207833] Brightness control not working on ASUS TUF FA506IU (AMD Ryzen 7 4800H / Nvidia GTX 1660 Ti)

2020-05-29 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=207833 Mateusz (drag...@op.pl) changed: What|Removed |Added CC||drag...@op.pl --- Comment #1 fr

[PATCH 07/13] test_sysctl: use new sysctl subdir helper register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
This simplifies the code considerably. The following coccinelle SmPL grammar rule was used to transform this code. // pycocci sysctl-subdir.cocci lib/test_sysctl.c @c1@ expression E1; identifier subdir, sysctls; @@ static struct ctl_table subdir[] = { { .procname = E1,

[PATCH 01/13] sysctl: add new register_sysctl_subdir() helper

2020-05-29 Thread Luis Chamberlain
Often enough all we need to do is create a subdirectory so that we can stuff sysctls underneath it. However, *if* that directory was already created early on the boot sequence we really have no need to use the full boiler plate code for it, we can just use local variables to help us guide sysctl to

[PATCH 00/13] sysctl: spring cleaning

2020-05-29 Thread Luis Chamberlain
Me and Xiaoming are working on some kernel/sysctl.c spring cleaning. During a recent linux-next merge conflict it became clear that the kitchen sink on kernel/sysctl.c creates too many conflicts, and so we need to do away with stuffing everyone's knobs on this one file. This is part of that work.

[PATCH 10/13] eventpoll: simplify sysctl declaration with register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
From: Xiaoming Ni Move epoll_table sysctl to fs/eventpoll.c and remove the clutter out of kernel/sysctl.c by using register_sysctl_subdir().. Signed-off-by: Xiaoming Ni Signed-off-by: Luis Chamberlain --- fs/eventpoll.c | 10 +- include/linux/poll.h | 2 -- include/linux/sy

[PATCH 03/13] hpet: use new sysctl subdir helper register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
This simplifies the code considerably. The following coccinelle SmPL grammar rule was used to transform this code. // pycocci sysctl-subdir.cocci drivers/char/hpet.c @c1@ expression E1; identifier subdir, sysctls; @@ static struct ctl_table subdir[] = { { .procname = E1,

[PATCH 06/13] ocfs2: use new sysctl subdir helper register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
This simplifies the code considerably. The following coccinelle SmPL grammar rule was used to transform this code. // pycocci sysctl-subdir.cocci fs/ocfs2/stackglue.c @c1@ expression E1; identifier subdir, sysctls; @@ static struct ctl_table subdir[] = { { .procname = E1,

[PATCH 05/13] macintosh/mac_hid.c: use new sysctl subdir helper register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
This simplifies the code considerably. The following coccinelle SmPL grammar rule was used to transform this code. // pycocci sysctl-subdir.cocci drivers/macintosh/mac_hid.c @c1@ expression E1; identifier subdir, sysctls; @@ static struct ctl_table subdir[] = { { .procnam

[PATCH 04/13] i915: use new sysctl subdir helper register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
This simplifies the code considerably. The following coccinelle SmPL grammar rule was used to transform this code. // pycocci sysctl-subdir.cocci drivers/gpu/drm/i915/i915_perf.c @c1@ expression E1; identifier subdir, sysctls; @@ static struct ctl_table subdir[] = { { .pr

[PATCH 08/13] inotify: simplify sysctl declaration with register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
From: Xiaoming Ni move inotify_user sysctl to inotify_user.c and use the new register_sysctl_subdir() helper. Signed-off-by: Xiaoming Ni Signed-off-by: Luis Chamberlain --- fs/notify/inotify/inotify_user.c | 11 ++- include/linux/inotify.h | 3 --- kernel/sysctl.c

[PATCH 13/13] fs: move binfmt_misc sysctl to its own file

2020-05-29 Thread Luis Chamberlain
This moves the binfmt_misc sysctl to its own file to help remove clutter from kernel/sysctl.c. Signed-off-by: Luis Chamberlain --- fs/binfmt_misc.c | 1 + kernel/sysctl.c | 7 --- 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/fs/binfmt_misc.c b/fs/binfmt_misc.c index f69a043

[PATCH 02/13] cdrom: use new sysctl subdir helper register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
This simplifies the code considerably. The following coccinelle SmPL grammar rule was used to transform this code. // pycocci sysctl-subdir.cocci drivers/cdrom/cdrom.c @c1@ expression E1; identifier subdir, sysctls; @@ static struct ctl_table subdir[] = { { .procname = E1

[PATCH 11/13] random: simplify sysctl declaration with register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
From: Xiaoming Ni Move random_table sysctl from kernel/sysctl.c to drivers/char/random.c and use register_sysctl_subdir() to help remove the clutter out of kernel/sysctl.c. Signed-off-by: Xiaoming Ni Signed-off-by: Luis Chamberlain --- drivers/char/random.c | 14 -- include/linux

[PATCH 09/13] firmware_loader: simplify sysctl declaration with register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
From: Xiaoming Ni Move the firmware config sysctl table to fallback_table.c and use the new register_sysctl_subdir() helper. This removes the clutter from kernel/sysctl.c. Signed-off-by: Xiaoming Ni Signed-off-by: Luis Chamberlain --- drivers/base/firmware_loader/fallback.c | 4 d

[PATCH 12/13] sysctl: add helper to register empty subdir

2020-05-29 Thread Luis Chamberlain
The way to create a subdirectory from the base set of directories is a bit obscure, so provide a helper which makes this clear, and also helps remove boiler plate code required to do this work. Signed-off-by: Luis Chamberlain --- include/linux/sysctl.h | 7 +++ kernel/sysctl.c| 16 +

[Bug 207833] Brightness control not working on ASUS TUF FA506IU (AMD Ryzen 7 4800H / Nvidia GTX 1660 Ti)

2020-05-29 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=207833 --- Comment #2 from julen.pa...@outlook.es --- (In reply to Mateusz from comment #1) > Does it work better after closing lid and reopen it? (the system goes > suspend) > I have an ASUS Zephyrus G14 with similar specs and this is the case, after >

Re: [PATCH] xf86drm: add drmOpenByFB

2020-05-29 Thread Pekka Paalanen
On Thu, 28 May 2020 17:46:08 +0800 Chih-Wei Huang wrote: > The main problem we're trying to solve is to > find the DRM device of the primary framebuffer (fb0). Hi, I would say that is a completely wrong starting point. Please, let fbdev die in peace/pieces. Do not make any new code that relies

Re: [PATCH 01/13] sysctl: add new register_sysctl_subdir() helper

2020-05-29 Thread Jani Nikula
On Fri, 29 May 2020, Luis Chamberlain wrote: > Often enough all we need to do is create a subdirectory so that > we can stuff sysctls underneath it. However, *if* that directory > was already created early on the boot sequence we really have no > need to use the full boiler plate code for it, we c

Re: [PATCH 13/13] fs: move binfmt_misc sysctl to its own file

2020-05-29 Thread Kees Cook
On Fri, May 29, 2020 at 07:41:08AM +, Luis Chamberlain wrote: > This moves the binfmt_misc sysctl to its own file to help remove > clutter from kernel/sysctl.c. > > Signed-off-by: Luis Chamberlain > --- > fs/binfmt_misc.c | 1 + > kernel/sysctl.c | 7 --- > 2 files changed, 1 insertion(

Re: [PATCH 12/13] sysctl: add helper to register empty subdir

2020-05-29 Thread Kees Cook
On Fri, May 29, 2020 at 07:41:07AM +, Luis Chamberlain wrote: > The way to create a subdirectory from the base set of directories > is a bit obscure, so provide a helper which makes this clear, and > also helps remove boiler plate code required to do this work. > > Signed-off-by: Luis Chamberl

Re: [PATCH] drm/mm: add ig_frag selftest

2020-05-29 Thread Nirmoy
On 5/28/20 6:04 PM, Chris Wilson wrote: Quoting Nirmoy Das (2020-05-28 14:05:56) This patch introduces fragmentation in the address range and measures time taken by 10k insertions for each modes. ig_frag() will fail if one of the mode takes more than 1 sec. Output: [ 37.326723] drm_mm: ig

Re: [PATCH] drm/mm: add ig_frag selftest

2020-05-29 Thread Nirmoy
On 5/28/20 8:49 PM, Christian König wrote: Am 28.05.20 um 15:05 schrieb Nirmoy Das: This patch introduces fragmentation in the address range and measures time taken by 10k insertions for each modes. ig_frag() will fail if one of the mode takes more than 1 sec. Output: [   37.326723] drm_mm:

Re: [PATCH] omapfb: Fix reference count leak in display_init_sysfs.

2020-05-29 Thread Greg Kroah-Hartman
On Thu, May 28, 2020 at 02:44:23PM -0500, wu000...@umn.edu wrote: > From: Qiushi Wu > > kobject_init_and_add() takes reference even when it fails. > If this function returns an error, kobject_put() must be called to > properly clean up the memory associated with the object. > Because function oma

Re: [PATCH 06/13] ocfs2: use new sysctl subdir helper register_sysctl_subdir()

2020-05-29 Thread Kees Cook
On Fri, May 29, 2020 at 07:41:01AM +, Luis Chamberlain wrote: > This simplifies the code considerably. The following coccinelle > SmPL grammar rule was used to transform this code. > > // pycocci sysctl-subdir.cocci fs/ocfs2/stackglue.c > > @c1@ > expression E1; > identifier subdir, sysctls;

Re: [PATCH i-g-t] panfrost: Test labeling functionality

2020-05-29 Thread Steven Price
On 28/05/2020 14:38, Rohan Garg wrote: Introduce tests to cover the new generic labeling ioctl's being reviewed here [1]. Signed-off-by: Rohan Garg [1] https://patchwork.freedesktop.org/series/77267/ Signed-off-by: Rohan Garg A few comments below. --- include/drm-uapi/drm.h| 23 ++

Re: Re:[PATCH] drm: fix setting of plane_mask in pan_display_atomic() function for linux-4.4

2020-05-29 Thread Ville Syrjälä
On Wed, May 27, 2020 at 09:33:27PM +0800, chenxb_99...@126.com wrote: > From: Xuebing Chen > > On Mon, May 25, 2020 at 04:34:28PM +0200, Daniel Vetter wrote: > > On Sat, May 23, 2020 at 11:49:07AM +0800, chenxb_99...@126.com wrote: > > > From: Xuebing Chen > > > > > > The provides drm_for_each

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-05-29 Thread Simon Ser
On Thursday, May 28, 2020 5:49 PM, Marek Olšák wrote: > On most hardware, there is a minimum pitch alignment for linear and > any greater multiple of the alignment is fine. > > On Navi, the pitch in bytes for linear must be > align(width * bpp / 8, 256). That's because the hw computes the pitch >

Re: [PATCH 09/13] firmware_loader: simplify sysctl declaration with register_sysctl_subdir()

2020-05-29 Thread Greg KH
On Fri, May 29, 2020 at 07:41:04AM +, Luis Chamberlain wrote: > From: Xiaoming Ni > > Move the firmware config sysctl table to fallback_table.c and use the > new register_sysctl_subdir() helper. This removes the clutter from > kernel/sysctl.c. > > Signed-off-by: Xiaoming Ni > Signed-off-by:

Re: [PATCH 11/13] random: simplify sysctl declaration with register_sysctl_subdir()

2020-05-29 Thread Greg KH
On Fri, May 29, 2020 at 07:41:06AM +, Luis Chamberlain wrote: > From: Xiaoming Ni > > Move random_table sysctl from kernel/sysctl.c to drivers/char/random.c > and use register_sysctl_subdir() to help remove the clutter out of > kernel/sysctl.c. > > Signed-off-by: Xiaoming Ni > Signed-off-by

Re: [igt-dev] [PATCH i-g-t] panfrost: Test labeling functionality

2020-05-29 Thread Petri Latvala
On Thu, May 28, 2020 at 03:38:35PM +0200, Rohan Garg wrote: > Introduce tests to cover the new generic labeling ioctl's > being reviewed here [1]. > > Signed-off-by: Rohan Garg > > [1] https://patchwork.freedesktop.org/series/77267/ > > Signed-off-by: Rohan Garg > --- > include/drm-uapi/drm.h

Re: [PATCH] drm: use drm_dev_has_vblank more

2020-05-29 Thread Daniel Vetter
On Wed, May 27, 2020 at 02:02:12PM +0200, Thomas Zimmermann wrote: > > Am 27.05.20 um 13:11 schrieb Daniel Vetter: > > For historical reasons it's called dev->num_crtcs, which is rather > > confusing ever since kms was added. But now we have a nice helper, so > > let's use it for better readabilit

[PATCH v2] drm: drm_fourcc: add NV15, Q410, Q401 YUV formats

2020-05-29 Thread Ben Davis
DRM_FORMAT_NV15 is a 2 plane format suitable for linear and 16x16 block-linear memory layouts. The format is similar to P010 with 4:2:0 sub-sampling but has no padding between components. Instead, luminance and chrominance samples are grouped into 4s so that each group is packed into an integer num

Re: [Intel-gfx] [PATCH 06/13] ocfs2: use new sysctl subdir helper register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
On Fri, May 29, 2020 at 01:23:19AM -0700, Kees Cook wrote: > On Fri, May 29, 2020 at 07:41:01AM +, Luis Chamberlain wrote: > > This simplifies the code considerably. The following coccinelle > > SmPL grammar rule was used to transform this code. > > > > // pycocci sysctl-subdir.cocci fs/ocfs2/

Re: [PATCH v2] drm: drm_fourcc: add NV15, Q410, Q401 YUV formats

2020-05-29 Thread Daniel Vetter
On Fri, May 29, 2020 at 12:47:26PM +0100, Ben Davis wrote: > DRM_FORMAT_NV15 is a 2 plane format suitable for linear and 16x16 Is there a specific modifier for this 16x16 block linear format? Feels a bit confusing, would be great to add that to the commit message so usage is clearer. We have DRM_F

Re: [PATCH 2/2] drm/atomic-helper: reset vblank on crtc reset

2020-05-29 Thread Daniel Vetter
On Wed, May 27, 2020 at 11:47:57AM +0200, Daniel Vetter wrote: > Only when vblanks are supported ofc. > > Some drivers do this already, but most unfortunately missed it. This > opens up bugs after driver load, before the crtc is enabled for the > first time. syzbot spotted this when loading vkms a

Re: [PATCH 2/2] drm/atomic-helper: reset vblank on crtc reset

2020-05-29 Thread Boris Brezillon
On Wed, 27 May 2020 11:47:57 +0200 Daniel Vetter wrote: > Only when vblanks are supported ofc. > > Some drivers do this already, but most unfortunately missed it. This > opens up bugs after driver load, before the crtc is enabled for the > first time. syzbot spotted this when loading vkms as a s

Re: [PATCH 09/13] firmware_loader: simplify sysctl declaration with register_sysctl_subdir()

2020-05-29 Thread Luis Chamberlain
On Fri, May 29, 2020 at 12:26:13PM +0200, Greg KH wrote: > On Fri, May 29, 2020 at 07:41:04AM +, Luis Chamberlain wrote: > > From: Xiaoming Ni > > > > Move the firmware config sysctl table to fallback_table.c and use the > > new register_sysctl_subdir() helper. This removes the clutter from >

Re: [PATCH 01/13] sysctl: add new register_sysctl_subdir() helper

2020-05-29 Thread Luis Chamberlain
On Fri, May 29, 2020 at 11:13:21AM +0300, Jani Nikula wrote: > On Fri, 29 May 2020, Luis Chamberlain wrote: > > Often enough all we need to do is create a subdirectory so that > > we can stuff sysctls underneath it. However, *if* that directory > > was already created early on the boot sequence we

Re: [PATCH 05/15] drm/panfrost: use spinlock instead of atomic

2020-05-29 Thread Robin Murphy
On 2020-05-10 17:55, Clément Péron wrote: Convert busy_count to a simple int protected by spinlock. A little more reasoning might be nice. Signed-off-by: Clément Péron --- [...] diff --git a/drivers/gpu/drm/panfrost/panfrost_devfreq.h b/drivers/gpu/drm/panfrost/panfrost_devfreq.h index 06

Re: [PATCH 05/15] drm/panfrost: use spinlock instead of atomic

2020-05-29 Thread Steven Price
On 29/05/2020 13:35, Clément Péron wrote: Hi Robin, On Fri, 29 May 2020 at 14:20, Robin Murphy wrote: On 2020-05-10 17:55, Clément Péron wrote: Convert busy_count to a simple int protected by spinlock. A little more reasoning might be nice. I have follow the modification requested for li

[Bug 207833] Brightness control not working on ASUS TUF FA506IU (AMD Ryzen 7 4800H / Nvidia GTX 1660 Ti)

2020-05-29 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=207833 Alex Deucher (alexdeuc...@gmail.com) changed: What|Removed |Added CC||alexdeuc...@gmail.c

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-05-29 Thread Alex Deucher
On Fri, May 29, 2020 at 4:59 AM Simon Ser wrote: > > On Thursday, May 28, 2020 5:49 PM, Marek Olšák wrote: > > > On most hardware, there is a minimum pitch alignment for linear and > > any greater multiple of the alignment is fine. > > > > On Navi, the pitch in bytes for linear must be > > align(

Re: [git pull] drm fixes for 5.7-rc8/final

2020-05-29 Thread Rodrigo Vivi
On Fri, May 29, 2020 at 12:15:27PM +1000, Dave Airlie wrote: > On Fri, 29 May 2020 at 12:02, Dave Airlie wrote: > > > > On Fri, 29 May 2020 at 11:49, Linus Torvalds > > wrote: > > > > > > On Thu, May 28, 2020 at 5:21 PM Dave Airlie wrote: > > > > > > > > Seems to have wound down nicely, a couple

Re: [PATCH] drm/shmem-helpers: Simplify dma-buf importing

2020-05-29 Thread Boris Brezillon
Hi Daniel, On Wed, 20 May 2020 20:02:32 +0200 Daniel Vetter wrote: > - Ditch the ->pages array > - Make it a private gem bo, which means no shmem object, which means > fireworks if anyone calls drm_gem_object_get_pages. But we've just > made sure that's all covered. > > v2: Rebase > > Acke

Re: [PATCH] drm/shmem-helpers: Simplify dma-buf importing

2020-05-29 Thread Boris Brezillon
On Fri, 29 May 2020 15:34:28 +0200 Boris Brezillon wrote: > Hi Daniel, > > On Wed, 20 May 2020 20:02:32 +0200 > Daniel Vetter wrote: > > > - Ditch the ->pages array > > - Make it a private gem bo, which means no shmem object, which means > > fireworks if anyone calls drm_gem_object_get_pages

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-05-29 Thread Daniel Vetter
On Fri, May 29, 2020 at 3:29 PM Alex Deucher wrote: > > On Fri, May 29, 2020 at 4:59 AM Simon Ser wrote: > > > > On Thursday, May 28, 2020 5:49 PM, Marek Olšák wrote: > > > > > On most hardware, there is a minimum pitch alignment for linear and > > > any greater multiple of the alignment is fine

Re: [PATCH v6] drm/ioctl: Add a ioctl to set and get a label on GEM objects

2020-05-29 Thread Rohan Garg
Hey Eric! On jueves, 28 de mayo de 2020 20:45:24 (CEST) Eric Anholt wrote: > On Thu, May 28, 2020 at 10:06 AM Rohan Garg wrote: > > DRM_IOCTL_HANDLE_SET_LABEL lets you label buffers associated > > with a handle, making it easier to debug issues in userspace > > applications. > > > > DRM_IOCTL_H

Re: [PATCH] drm/shmem-helpers: Simplify dma-buf importing

2020-05-29 Thread Boris Brezillon
On Wed, 20 May 2020 20:02:32 +0200 Daniel Vetter wrote: > @@ -695,36 +702,16 @@ drm_gem_shmem_prime_import_sg_table(struct drm_device > *dev, > struct sg_table *sgt) > { > size_t size = PAGE_ALIGN(attach->dmabuf->size); > - size_t npages = size >> PA

Re: [PATCH 0/9] shmem helper untangling

2020-05-29 Thread Boris Brezillon
On Mon, 11 May 2020 11:35:45 +0200 Daniel Vetter wrote: > Hi all, > > I've started this a while ago, with the idea to move shmem helpers over > to dma_resv_lock. Big prep work for that was to untangle the layering > between functions called by drivers, and functions used to implement > drm_gem_o

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-05-29 Thread Daniel Stone
Hi Alex, On Fri, 29 May 2020 at 14:29, Alex Deucher wrote: > On Fri, May 29, 2020 at 4:59 AM Simon Ser wrote: > > OK. In this case I think it's fine to make the DMA-BUF import fail, as > > we've suggested on IRC. The more-or-less planned fix for these buffer > > sharing issues is to revive the b

[PATCH] drm/shmem-helpers: Simplify dma-buf importing

2020-05-29 Thread Daniel Vetter
- Ditch the ->pages array - Make it a private gem bo, which means no shmem object, which means fireworks if anyone calls drm_gem_object_get_pages. But we've just made sure that's all covered. v2: Rebase v3: I forgot to remove the page_count mangling from the free path too. Noticed by Boris wh

Re: [PATCH 1/1] drm/mm: fix hole size comparison

2020-05-29 Thread Chris Wilson
Quoting Nirmoy Das (2020-05-29 15:04:01) > Fixes: 0cdea4455acd350a ("drm/mm: optimize rb_hole_addr rbtree search") Hmm. Indeed. Now, do you have a test case on your end that hits this? One of us should add one for drm-mm. -Chris ___ dri-devel mailing lis

Re: [PATCH 1/1] drm/mm: fix hole size comparison

2020-05-29 Thread Christian König
Am 29.05.20 um 16:09 schrieb Chris Wilson: Quoting Nirmoy Das (2020-05-29 15:04:01) Fixes: 0cdea4455acd350a ("drm/mm: optimize rb_hole_addr rbtree search") Hmm. Indeed. Now, do you have a test case on your end that hits this? No, I just got time to double read through this and stumbled over i

Re: [PATCH] xf86drm: add drmOpenByFB

2020-05-29 Thread Alex Deucher
On Fri, May 29, 2020 at 3:49 AM Pekka Paalanen wrote: > > On Thu, 28 May 2020 17:46:08 +0800 > Chih-Wei Huang wrote: > > > The main problem we're trying to solve is to > > find the DRM device of the primary framebuffer (fb0). > > Hi, > > I would say that is a completely wrong starting point. Plea

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-05-29 Thread Alex Deucher
On Fri, May 29, 2020 at 9:58 AM Daniel Stone wrote: > > Hi Alex, > > On Fri, 29 May 2020 at 14:29, Alex Deucher wrote: > > On Fri, May 29, 2020 at 4:59 AM Simon Ser wrote: > > > OK. In this case I think it's fine to make the DMA-BUF import fail, as > > > we've suggested on IRC. The more-or-less

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-05-29 Thread Daniel Stone
On Fri, 29 May 2020 at 15:29, Alex Deucher wrote: > Maybe I'm over thinking this. I just don't want to get into a > situation where we go through a lot of effort to add modifier support > and then performance ends up being worse than it is today in a lot of > cases. I'm genuinely curious: what d

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-05-29 Thread Alex Deucher
On Fri, May 29, 2020 at 10:32 AM Daniel Stone wrote: > > On Fri, 29 May 2020 at 15:29, Alex Deucher wrote: > > Maybe I'm over thinking this. I just don't want to get into a > > situation where we go through a lot of effort to add modifier support > > and then performance ends up being worse than

Re: [PATCH] drm/shmem-helpers: Simplify dma-buf importing

2020-05-29 Thread Boris Brezillon
On Fri, 29 May 2020 16:05:42 +0200 Daniel Vetter wrote: > - Ditch the ->pages array > - Make it a private gem bo, which means no shmem object, which means > fireworks if anyone calls drm_gem_object_get_pages. But we've just > made sure that's all covered. > > v2: Rebase > > v3: I forgot to

Re: [Intel-gfx] [PATCH 06/13] ocfs2: use new sysctl subdir helper register_sysctl_subdir()

2020-05-29 Thread Kees Cook
On Fri, May 29, 2020 at 11:49:12AM +, Luis Chamberlain wrote: > Yikes, sense, you're right. Nope, I left the random config tests to > 0day. Will fix, thanks! Yeah, I do the same for randconfig, but I always do an "allmodconfig" build before sending stuff. It's a good smoke test. -- Kees Cook

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-05-29 Thread Daniel Stone
On Fri, 29 May 2020 at 15:36, Alex Deucher wrote: > On Fri, May 29, 2020 at 10:32 AM Daniel Stone wrote: > > On Fri, 29 May 2020 at 15:29, Alex Deucher wrote: > > > Maybe I'm over thinking this. I just don't want to get into a > > > situation where we go through a lot of effort to add modifier

[PATCH v7 0/6] drm/meson: add support for Amlogic Video FBC

2020-05-29 Thread Neil Armstrong
Amlogic uses a proprietary lossless image compression protocol and format for their hardware video codec accelerators, either video decoders or video input encoders. It considerably reduces memory bandwidth while writing and reading frames in memory. The underlying storage is considered to be 3 c

[PATCH v7 4/6] drm/meson: overlay: setup overlay for Amlogic FBC Memory Saving mode

2020-05-29 Thread Neil Armstrong
Setup the Amlogic FBC decoder for the VD1 video overlay plane to use a different superblock size for the Memory Saving mode. Tested-by: Kevin Hilman Signed-off-by: Neil Armstrong --- drivers/gpu/drm/meson/meson_overlay.c | 25 +++-- 1 file changed, 23 insertions(+), 2 deleti

[PATCH v7 5/6] drm/meson: overlay: setup overlay for Amlogic FBC Scatter Memory layout

2020-05-29 Thread Neil Armstrong
Setup the Amlogic FBC decoder for the VD1 video overlay plane to use read the FBC header as Scatter Memory layout reference. Tested-by: Kevin Hilman Signed-off-by: Neil Armstrong --- drivers/gpu/drm/meson/meson_overlay.c | 53 ++- 1 file changed, 35 insertions(+), 18 del

[PATCH v7 1/6] drm/fourcc: Add modifier definitions for describing Amlogic Video Framebuffer Compression

2020-05-29 Thread Neil Armstrong
Amlogic uses a proprietary lossless image compression protocol and format for their hardware video codec accelerators, either video decoders or video input encoders. It considerably reduces memory bandwidth while writing and reading frames in memory. The underlying storage is considered to be 3 c

[PATCH v7 6/6] drm/meson: crtc: handle commit of Amlogic FBC frames

2020-05-29 Thread Neil Armstrong
Since the VD1 Amlogic FBC decoder is now configured by the overlay driver, commit the right registers to decode the Amlogic FBC frame. Tested-by: Kevin Hilman Signed-off-by: Neil Armstrong --- drivers/gpu/drm/meson/meson_crtc.c | 118 + 1 file changed, 88 insertions(

[PATCH v7 2/6] drm/meson: add Amlogic Video FBC registers

2020-05-29 Thread Neil Armstrong
Add the registers of the VPU VD1 Amlogic FBC decoder module, and routing register. Tested-by: Kevin Hilman Signed-off-by: Neil Armstrong --- drivers/gpu/drm/meson/meson_registers.h | 22 ++ 1 file changed, 22 insertions(+) diff --git a/drivers/gpu/drm/meson/meson_registers.

[PATCH v7 3/6] drm/meson: overlay: setup overlay for Amlogic FBC

2020-05-29 Thread Neil Armstrong
Setup the Amlogic FBC decoder for the VD1 video overlay plane. The VD1 Amlogic FBC decoder is integrated in the pipeline like the YUV pixel reading/formatter but used a direct memory address instead. This adds support for the basic layout, and needs to calculate the content body size since the he

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-05-29 Thread Alex Deucher
On Fri, May 29, 2020 at 11:03 AM Daniel Stone wrote: > > On Fri, 29 May 2020 at 15:36, Alex Deucher wrote: > > On Fri, May 29, 2020 at 10:32 AM Daniel Stone wrote: > > > On Fri, 29 May 2020 at 15:29, Alex Deucher wrote: > > > > Maybe I'm over thinking this. I just don't want to get into a > >

Re: [RFC PATCH 1/1] drm/mm: add ig_frag selftest

2020-05-29 Thread Nirmoy
This works correctly most of the times but sometimes 20k insertions can take more than 8 times of 10k insertion time. Regards, Nirmoy On 5/29/20 6:33 PM, Nirmoy Das wrote: This patch introduces fragmentation in the address range and measures time taken by 10k and 20k insertions. ig_frag() wi

Re: [Linux-stm32] [PATCH v8 08/10] drm: stm: dw-mipi-dsi: let the bridge handle the HW version check

2020-05-29 Thread Philippe CORNU
Hi Adrian, and thank you very much for the patchset. Thank you also for having tested it on STM32F769 and STM32MP1. Sorry for the late response, Yannick and I will review it as soon as possible and we will keep you posted. Note: Do not hesitate to put us in copy for the next version (philippe.co.

Re: [RFC PATCH 1/1] drm/mm: add ig_frag selftest

2020-05-29 Thread Chris Wilson
Quoting Nirmoy (2020-05-29 16:40:53) > This works correctly most of the times but sometimes > > 20k insertions can take more than 8 times of 10k insertion time. The pressure is on to improve then :) > Regards, > > Nirmoy > > On 5/29/20 6:33 PM, Nirmoy Das wrote: > > This patch introduces fragm

Re: [PATCH v6 4/9] dt-bindings: display: panel: Add ilitek ili9341 panel bindings

2020-05-29 Thread Rob Herring
On Wed, 27 May 2020 15:27:28 +0800, dillon.min...@gmail.com wrote: > From: dillon min > > Add documentation for "ilitek,ili9341" panel. > > Signed-off-by: dillon min > --- > .../bindings/display/panel/ilitek,ili9341.yaml | 69 > ++ > 1 file changed, 69 insertions(+) >

Re: [PATCH v8 0/8] Add support for devices in the Energy Model

2020-05-29 Thread Rafael J. Wysocki
On Fri, May 29, 2020 at 5:01 PM Lukasz Luba wrote: > > Hi Rafael, > > > On 5/27/20 10:58 AM, Lukasz Luba wrote: > > Hi all, > > > > Background of this version: > > This is the v8 of the patch set and is has smaller scope. I had to split > > the series into two: EM changes and thermal changes due t

Re: [PATCH v3 4/5] arm64: dts: sun50i-a64-pinephone: Enable LCD support on PinePhone

2020-05-29 Thread Pavel Machek
Hi! > PinePhone uses PWM backlight and a XBD599 LCD panel over DSI for > display. > > Backlight levels curve was optimized by Martijn Braam using a > lux meter. If it was possible to preserve lux values for individual settings in the comment somewhere... that would be nice :-). One day, it woul

[RFC PATCH v5 0/6] Exynos: Simple QoS for exynos-bus using interconnect

2020-05-29 Thread Sylwester Nawrocki
This patchset adds interconnect API support for the Exynos SoC "samsung, exynos-bus" compatible devices, which already have their corresponding exynos-bus driver in the devfreq subsystem. Complementing the devfreq driver with an interconnect functionality allows to ensure the QoS requirements o

[RFC PATCH v5 6/6] drm: exynos: mixer: Add interconnect support

2020-05-29 Thread Sylwester Nawrocki
From: Marek Szyprowski This patch adds interconnect support to exynos-mixer. The mixer works the same as before when CONFIG_INTERCONNECT is 'n'. For proper operation of the video mixer block we need to ensure the interconnect busses like DMC or LEFTBUS provide enough bandwidth so as to avoid DMA

[RFC PATCH v5 3/6] PM / devfreq: exynos-bus: Add registration of interconnect child device

2020-05-29 Thread Sylwester Nawrocki
This patch adds registration of a child platform device for the exynos interconnect driver. It is assumed that the interconnect provider will only be needed when #interconnect-cells property is present in the bus DT node, hence the child device will be created only when such a property is present.

[RFC PATCH v5 5/6] ARM: dts: exynos: Add interconnects to Exynos4412 mixer

2020-05-29 Thread Sylwester Nawrocki
From: Artur Świgoń This patch adds an 'interconnects' property to Exynos4412 DTS in order to declare the interconnect path used by the mixer. Please note that the 'interconnect-names' property is not needed when there is only one path in 'interconnects', in which case calling of_icc_get() with a

[RFC PATCH v5 2/6] interconnect: Add generic interconnect driver for Exynos SoCs

2020-05-29 Thread Sylwester Nawrocki
This patch adds a generic interconnect driver for Exynos SoCs in order to provide interconnect functionality for each "samsung,exynos-bus" compatible device. The SoC topology is a graph (or more specifically, a tree) and its edges are specified using the 'samsung,interconnect-parent' in the DT. Du

[RFC PATCH v5 1/6] dt-bindings: exynos-bus: Add documentation for interconnect properties

2020-05-29 Thread Sylwester Nawrocki
Add documentation for new optional properties in the exynos bus nodes: samsung,interconnect-parent, #interconnect-cells. These properties allow to specify the SoC interconnect structure which then allows the interconnect consumer devices to request specific bandwidth requirements. Signed-off-by: A

[RFC PATCH v5 4/6] ARM: dts: exynos: Add interconnect properties to Exynos4412 bus nodes

2020-05-29 Thread Sylwester Nawrocki
This patch adds the following properties for Exynos4412 interconnect bus nodes: - samsung,interconnect-parent: to declare connections between nodes in order to guarantee PM QoS requirements between nodes; - #interconnect-cells: required by the interconnect framework. Note that #interconnect-c

Re: [PATCH v6] drm/ioctl: Add a ioctl to set and get a label on GEM objects

2020-05-29 Thread Eric Anholt
On Fri, May 29, 2020 at 6:44 AM Rohan Garg wrote: > > Hey Eric! > > On jueves, 28 de mayo de 2020 20:45:24 (CEST) Eric Anholt wrote: > > On Thu, May 28, 2020 at 10:06 AM Rohan Garg > wrote: > > > DRM_IOCTL_HANDLE_SET_LABEL lets you label buffers associated > > > with a handle, making it easier to

[PATCH v3 2/6] drm/client: Add drm_client_modeset_disable()

2020-05-29 Thread Noralf Trønnes
Add a way for clients to disable all outputs. Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_client_modeset.c | 20 include/drm/drm_client.h | 1 + 2 files changed, 21 insertions(+) diff --git a/drivers/gpu/drm/drm_client_modeset.c b/drivers/gpu/drm/drm

[PATCH v3 3/6] drm/client: Add a way to set modeset, properties and rotation

2020-05-29 Thread Noralf Trønnes
This adds functions for clients that need more control over the configuration than what's setup by drm_client_modeset_probe(). Connector, fb and display mode can be set using drm_client_modeset_set(). Plane rotation can be set using drm_client_modeset_set_rotation() and other properties using drm_c

[PATCH v3 4/6] drm: Add Generic USB Display driver

2020-05-29 Thread Noralf Trønnes
This adds a generic USB display driver with the intention that it can be used with future USB interfaced low end displays/adapters. The Linux gadget device driver will serve as the canonical device implementation. The following DRM properties are supported: - Plane rotation - Connector TV properti

[PATCH v3 6/6] usb: gadget: function: Add Generic USB Display support

2020-05-29 Thread Noralf Trønnes
This adds the gadget side support for the Generic USB Display. It presents a DRM display device as a USB Display configured through configfs. The display is implemented as a vendor type USB interface with one bulk out endpoint. The protocol is implemented using control requests. lz4 compressed fra

[PATCH v3 0/6] Generic USB Display driver

2020-05-29 Thread Noralf Trønnes
Hi, A while back I had the idea to turn a Raspberry Pi Zero into a $5 USB to HDMI/SDTV/DSI/DPI display adapter. This series adds a USB host driver and a device/gadget driver to achieve that. The reason for calling it 'Generic' is so anyone can make a USB display/adapter against this driver, all

[PATCH v3 1/6] drm/client: Add drm_client_init_from_id()

2020-05-29 Thread Noralf Trønnes
drm_client_init_from_id() provides a way for clients to add a client based on the minor. drm_client_register() is changed to return whether it was registered or not depending on the unplugged status of the DRM device. Its only caller drm_fbdev_generic_setup() runs inside probe() so it doesn't have

[PATCH v3 5/6] drm/gud: Add functionality for the USB gadget side

2020-05-29 Thread Noralf Trønnes
Since the USB gadget/device has to reach into the DRM internals, part of the code is placed in the DRM subsystem as a separate module. All calls into this module runs in process context and are serialized, except one function that runs in interrupt context. Since both the gadget side and the DRM s

Re: [PATCH v3 104/105] dt-bindings: display: vc4: hdmi: Add BCM2711 HDMI controllers bindings

2020-05-29 Thread Rob Herring
On Wed, May 27, 2020 at 05:49:14PM +0200, Maxime Ripard wrote: > The HDMI controllers found in the BCM2711 SoC need some adjustments to the > bindings, especially since the registers have been shuffled around in more > register ranges. > > Cc: Rob Herring > Cc: devicet...@vger.kernel.org > Signed

[PATCH] drm/selftests/mm: reduce per-function stack usage

2020-05-29 Thread Arnd Bergmann
The check_reserve_boundaries() function has a large array on the stack, over 500 bytes. It gets inlined into __igt_reserve, which has multiple other large structures as well but stayed just under the stack size warning limit of 1024 bytes until one more member got added to struct drm_mm_node, causi

Re: [PATCH] drm/selftests/mm: reduce per-function stack usage

2020-05-29 Thread Chris Wilson
Quoting Arnd Bergmann (2020-05-29 21:15:26) > The check_reserve_boundaries() function has a large array on the stack, > over 500 bytes. It gets inlined into __igt_reserve, which has multiple > other large structures as well but stayed just under the stack size > warning limit of 1024 bytes until on

Re: [PATCH] drm/selftests/mm: reduce per-function stack usage

2020-05-29 Thread Arnd Bergmann
On Fri, May 29, 2020 at 10:26 PM Chris Wilson wrote: > Quoting Arnd Bergmann (2020-05-29 21:15:26) > > > > diff --git a/drivers/gpu/drm/selftests/test-drm_mm.c > > b/drivers/gpu/drm/selftests/test-drm_mm.c > > index 9aabe82dcd3a..30108c330db8 100644 > > --- a/drivers/gpu/drm/selftests/test-drm_m

Re: [RFC PATCH 1/1] drm/mm: add ig_frag selftest

2020-05-29 Thread Nirmoy
On 5/29/20 5:52 PM, Chris Wilson wrote: Quoting Nirmoy (2020-05-29 16:40:53) This works correctly most of the times but sometimes I have to take my word back. In another machine,  20k insertions in best mode takes 6-9 times more than 10k insertions, all most all the time. evict, bottom-up

Re: [git pull] drm fixes for 5.7 final (apologies release)

2020-05-29 Thread pr-tracker-bot
The pull request you sent on Fri, 29 May 2020 12:37:51 +1000: > git://anongit.freedesktop.org/drm/drm tags/drm-fixes-2020-05-29-1 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/86e43b8bf0e6b3897e504cdb9230fd063ecd4452 Thank you! -- Deet-doot-dot, I am a bot. https:/

Re: [PATCH 0/8] Convert the intel iommu driver to the dma-iommu api

2020-05-29 Thread Marek Szyprowski
Hi Logan, On 29.05.2020 21:05, Logan Gunthorpe wrote: > On 2020-05-29 6:45 a.m., Christoph Hellwig wrote: >> On Thu, May 28, 2020 at 06:00:44PM -0600, Logan Gunthorpe wrote: This issue is most likely in the i915 driver and is most likely caused by the driver not respecting the return va

Re: [PATCH] drm/selftests/mm: reduce per-function stack usage

2020-05-29 Thread Chris Wilson
Quoting Arnd Bergmann (2020-05-29 21:43:47) > On Fri, May 29, 2020 at 10:26 PM Chris Wilson > wrote: > > Quoting Arnd Bergmann (2020-05-29 21:15:26) > > > > > > > diff --git a/drivers/gpu/drm/selftests/test-drm_mm.c > > > b/drivers/gpu/drm/selftests/test-drm_mm.c > > > index 9aabe82dcd3a..30108

[PATCH 1/2] drm: vmwgfx: remove drm_driver::master_set() return typ

2020-05-29 Thread Emil Velikov
The function always returns zero (success). Ideally we'll remove it all together - although that's requires a little more work. For now, we can drop the return type and simplify the drm core code surrounding it. Cc: David Airlie Cc: Daniel Vetter Cc: VMware Graphics Cc: Roland Scheidegger Sig

[PATCH 2/2] drm/auth: drop unnessesary variable assignments

2020-05-29 Thread Emil Velikov
The variables are already the exact same value or will be overwritten shortly afterwords. In either case there's no functional difference. Cc: David Airlie Cc: Daniel Vetter Signed-off-by: Emil Velikov --- drivers/gpu/drm/drm_auth.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff

  1   2   >