Re: [PATCH v14 06/12] swiotlb: Use is_swiotlb_force_bounce for swiotlb data bouncing

2021-06-24 Thread Qian Cai
On 6/24/2021 7:48 AM, Will Deacon wrote: > Ok, diff below which attempts to tackle the offset issue I mentioned as > well. Qian Cai -- please can you try with these changes? This works fine. > > Will > > --->8 > > diff --git a/include/linux/swiotlb.h b/incl

Re: [PATCH v14 06/12] swiotlb: Use is_swiotlb_force_bounce for swiotlb data bouncing

2021-06-23 Thread Qian Cai
On 6/18/2021 11:40 PM, Claire Chang wrote: > Propagate the swiotlb_force into io_tlb_default_mem->force_bounce and > use it to determine whether to bounce the data or not. This will be > useful later to allow for different pools. > > Signed-off-by: Claire Chang > Reviewed-by: Christoph Hellwig

Re: [PATCH v14 06/12] swiotlb: Use is_swiotlb_force_bounce for swiotlb data bouncing

2021-06-23 Thread Qian Cai
On 6/23/2021 2:37 PM, Will Deacon wrote: > On Wed, Jun 23, 2021 at 12:39:29PM -0400, Qian Cai wrote: >> >> >> On 6/18/2021 11:40 PM, Claire Chang wrote: >>> Propagate the swiotlb_force into io_tlb_default_mem->force_bounce and >>> use it to determine wh

Re: [PATCH v2] drm/rcar-du: DRM_RCAR_WRITEBACK depends on DRM

2020-08-22 Thread Qian Cai
l changes ? Laurent, this patch is still not in the latest linux-next. Can you take a look? > > On Mon, Jun 08, 2020 at 06:14:43AM +0300, Laurent Pinchart wrote: > > On Sun, Jun 07, 2020 at 10:53:40PM -0400, Qian Cai wrote: > > > There is no need to select DRM_RCAR_WRIT

Re: [PATCH] mm: Track mmu notifiers in fs_reclaim_acquire/release

2020-06-23 Thread Qian Cai
On Sun, Jun 21, 2020 at 10:01:03PM +0200, Daniel Vetter wrote: > On Sun, Jun 21, 2020 at 08:07:08PM +0200, Daniel Vetter wrote: > > On Sun, Jun 21, 2020 at 7:42 PM Qian Cai wrote: > > > > > > On Wed, Jun 10, 2020 at 09:41:01PM +0200, Daniel Vetter wrote: > > >

Re: [PATCH] mm: Track mmu notifiers in fs_reclaim_acquire/release

2020-06-23 Thread Qian Cai
> On Jun 23, 2020, at 6:13 PM, Daniel Vetter wrote: > > Ok I tested this. I can't use your script to repro because > - I don't have a setup with xfs, and the splat points at an issue in xfs > - reproducing lockdep splats in shrinker callbacks is always a bit tricky What’s xfs setup are you tal

Re: [PATCH 01/18] mm: Track mmu notifiers in fs_reclaim_acquire/release

2020-06-22 Thread Qian Cai
On Sun, Jun 21, 2020 at 07:28:40PM +0200, Daniel Vetter wrote: > On Sun, Jun 21, 2020 at 7:01 PM Qian Cai wrote: > > > > On Thu, Jun 04, 2020 at 10:12:07AM +0200, Daniel Vetter wrote: > > > fs_reclaim_acquire/release nicely catch recursion issues when > > > all

Re: [PATCH] mm: Track mmu notifiers in fs_reclaim_acquire/release

2020-06-22 Thread Qian Cai
On Sun, Jun 21, 2020 at 10:01:03PM +0200, Daniel Vetter wrote: > On Sun, Jun 21, 2020 at 08:07:08PM +0200, Daniel Vetter wrote: > > On Sun, Jun 21, 2020 at 7:42 PM Qian Cai wrote: > > > > > > On Wed, Jun 10, 2020 at 09:41:01PM +0200, Daniel Vetter wrote: > > >

Re: [PATCH 01/18] mm: Track mmu notifiers in fs_reclaim_acquire/release

2020-06-22 Thread Qian Cai
On Thu, Jun 04, 2020 at 10:12:07AM +0200, Daniel Vetter wrote: > fs_reclaim_acquire/release nicely catch recursion issues when > allocating GFP_KERNEL memory against shrinkers (which gpu drivers tend > to use to keep the excessive caches in check). For mmu notifier > recursions we do have lockdep a

Re: [PATCH] mm: Track mmu notifiers in fs_reclaim_acquire/release

2020-06-22 Thread Qian Cai
On Wed, Jun 10, 2020 at 09:41:01PM +0200, Daniel Vetter wrote: > fs_reclaim_acquire/release nicely catch recursion issues when > allocating GFP_KERNEL memory against shrinkers (which gpu drivers tend > to use to keep the excessive caches in check). For mmu notifier > recursions we do have lockdep a

[PATCH v2] drm/rcar-du: DRM_RCAR_WRITEBACK depends on DRM

2020-06-08 Thread Qian Cai
There is no need to select DRM_RCAR_WRITEBACK if DRM=n which just make the generated .config a bit ugly. # ARM devices # # end of ARM devices CONFIG_DRM_RCAR_WRITEBACK=y # # Frame buffer Devices Let DRM_RCAR_WRITEBACK depend on DRM_RCAR_DU instead. Signed-off-by: Qian Cai --- drivers

[PATCH] drm/rcar-du: DRM_RCAR_WRITEBACK depends on DRM

2020-06-08 Thread Qian Cai
There is no need to select DRM_RCAR_WRITEBACK if DRM=n which just make the generated .config a bit ugly. # ARM devices # # end of ARM devices CONFIG_DRM_RCAR_WRITEBACK=y # # Frame buffer Devices Signed-off-by: Qian Cai --- drivers/gpu/drm/rcar-du/Kconfig | 1 + 1 file changed, 1

Re: [PATCH] drm/rcar-du: DRM_RCAR_WRITEBACK depends on DRM

2020-06-08 Thread Qian Cai
> On Jun 7, 2020, at 10:02 PM, Laurent Pinchart > wrote: > > How about depending on DRM_RCAR_DU instead, as DRM_RCAR_WRITEBACK is > used to select compilation of rcar_du_writeback.c that is part of the > rcar-du driver ? Sure. I’ll send a v2. ___ dr

Re: [PATCH v2] drm/rcar-du: DRM_RCAR_WRITEBACK depends on DRM

2020-06-08 Thread Qian Cai
> On Jun 7, 2020, at 11:16 PM, Laurent Pinchart > wrote: > > I forgot to mention, I think the subject line should be > > drm/rcar-du: Make DRM_RCAR_WRITEBACK depend on DRM_RCAR_DU > > Could you please let me know if you're OK with these two small changes ? Yes, I am fine with those. __

[tip: locking/core] locking/lockdep: Remove unused @nested argument from lock_release()

2019-10-09 Thread tip-bot2 for Qian Cai
The following commit has been merged into the locking/core branch of tip: Commit-ID: 5facae4f3549b5cf7c0e10ec312a65ffd43b5726 Gitweb: https://git.kernel.org/tip/5facae4f3549b5cf7c0e10ec312a65ffd43b5726 Author:Qian Cai AuthorDate:Thu, 19 Sep 2019 12:09:40 -04:00 Committer

Re: [PATCH -next] treewide: remove unused argument in lock_release()

2019-10-01 Thread Qian Cai
On Mon, 2019-09-30 at 09:29 +0200, Peter Zijlstra wrote: > On Thu, Sep 19, 2019 at 12:09:40PM -0400, Qian Cai wrote: > > Since the commit b4adfe8e05f1 ("locking/lockdep: Remove unused argument > > in __lock_release"), @nested is no longer used in lock_release(),

Re: [PATCH -next] treewide: remove unused argument in lock_release()

2019-09-20 Thread Qian Cai
On Fri, 2019-09-20 at 10:38 +0100, Will Deacon wrote: > On Thu, Sep 19, 2019 at 12:09:40PM -0400, Qian Cai wrote: > > Since the commit b4adfe8e05f1 ("locking/lockdep: Remove unused argument > > in __lock_release"), @nested is no longer used in lock_release(), so > >

[PATCH -next] treewide: remove unused argument in lock_release()

2019-09-19 Thread Qian Cai
Since the commit b4adfe8e05f1 ("locking/lockdep: Remove unused argument in __lock_release"), @nested is no longer used in lock_release(), so remove it from all lock_release() calls and friends. Signed-off-by: Qian Cai --- drivers/gpu/drm/drm_connector.c | 2 +- drive

Re: [RFC PATCH] gpu: no need to compile drm/ if CONFIG_DRM=n

2019-09-11 Thread Qian Cai
> On Sep 11, 2019, at 1:37 PM, Maxime Ripard wrote: > > Hi, > > Le mer. 11 sept. 2019 à 19:35, Qian Cai a écrit : > The commit c0e09200dc08 ("drm: reorganise drm tree to be more future > proof.") changed the behavior from only compiling drm/ if CONFIG_DRM=y &

[RFC PATCH] gpu: no need to compile drm/ if CONFIG_DRM=n

2019-09-11 Thread Qian Cai
The commit c0e09200dc08 ("drm: reorganise drm tree to be more future proof.") changed the behavior from only compiling drm/ if CONFIG_DRM=y to always compiling drm/. This restores the behavior, so people don't need to waste time compiling stuff they don't need. Fixes: c0e09200dc08 ("drm: reorganis

Re: [PATCH] gpu/drm: fix a -Wstringop-truncation warning

2019-09-09 Thread Qian Cai
Emil, this seems has been stalled again from DRM maintainers. Just to see if you have some "magic" to unsilence it like the last time... On Fri, 2019-08-23 at 14:37 -0400, Qian Cai wrote: > In file included from ./include/linux/bitmap.h:9, > from ./include/li

Re: [PATCH] gpu/drm: fix a -Wstringop-truncation warning

2019-09-05 Thread Qian Cai
Ping. Please take a look at this trivial patch. On Fri, 2019-08-23 at 14:37 -0400, Qian Cai wrote: > In file included from ./include/linux/bitmap.h:9, >  from ./include/linux/cpumask.h:12, >  from ./arch/x86/include/asm/cpumask.h:5, >

[PATCH] gpu/drm: fix a -Wstringop-truncation warning

2019-08-25 Thread Qian Cai
~~~ Fix it by using strscpy() which will always return a valid string, and doesn't unnecessarily force the tail of the destination buffer to be zeroed. Signed-off-by: Qian Cai --- drivers/gpu/drm/drm_property.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --g

Re: [Bug 204407] New: Bad page state in process Xorg

2019-08-05 Thread Qian Cai
On Fri, 2019-08-02 at 13:33 -0700, Matthew Wilcox wrote: > On Fri, Aug 02, 2019 at 01:23:06PM -0700, Andrew Morton wrote: > > > [259701.387365] BUG: Bad page state in process Xorg  pfn:2a300 > > > [259701.393593] page:eaa8c000 refcount:0 mapcount:-128 > > > mapping: index:0x

[PATCH] drm: silence variable 'conn' set but not used

2019-07-22 Thread Qian Cai
: warning: variable 'conn' set but not used [-Wunused-but-set-variable] Silence it by marking "conn" __maybe_unused. Signed-off-by: Qian Cai --- drivers/gpu/drm/drm_framebuffer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_frameb

[PATCH] gpu/drm: fix a few kernel-doc "/**" mark warnings

2019-07-15 Thread Qian Cai
The opening comment mark "/**" is reserved for kernel-doc comments, so it will generate warnings for comments that are not kernel-doc with "make W=1". For example, drivers/gpu/drm/drm_memory.c:2: warning: Cannot understand * \file drm_memory.c Signed-off-by: Qian Cai

Re: [PATCH v3] gpu/drm_memory: fix a few warnings

2019-07-12 Thread Qian Cai
Maybe one of the non-DRM maintainers (Andrew, Thomas or Greg) who cares a bit about SPDX can pick this up. It occurs to me none of DRM maintainers cares about this as there is no feedback from any of them for months since v1. On Tue, 2019-07-09 at 11:13 -0400, Qian Cai wrote: > The open

[PATCH v3] gpu/drm_memory: fix a few warnings

2019-07-09 Thread Qian Cai
ates the MIT license further down in the source file. WARNING: Missing or malformed SPDX-License-Identifier tag in line 1 Fix it by adding the MIT SPDX identifier without touching the boilerplate language. Suggested-by: Thomas Gleixner Signed-off-by: Qian Cai --- v3: Keep the boilerplate lang

Re: [PATCH v2] gpu/drm_memory: fix a few warnings

2019-07-08 Thread Qian Cai
On Mon, 2019-07-08 at 15:21 -0400, Ilia Mirkin wrote: > On Mon, Jul 8, 2019 at 2:06 PM Qian Cai wrote: > > > > The opening comment mark "/**" is reserved for kernel-doc comments, so > > it will generate a warning with "make W=1". > > >

[PATCH v2] gpu/drm_memory: fix a few warnings

2019-07-08 Thread Qian Cai
ates the MIT license further down in the source file. WARNING: Missing or malformed SPDX-License-Identifier tag in line 1 It becomes redundant to add both an SPDX identifier and have a description of the license in the comment block at the top, so remove the later. Signed-off-by: Qian Cai

Re: [RESEND PATCH] gpu/drm_memory: fix a few warnings

2019-06-21 Thread Qian Cai
Ping. On Fri, 2019-06-07 at 15:00 -0400, Qian Cai wrote: > The opening comment mark "/**" is reserved for kernel-doc comments, so > it will generate a warning with "make W=1". > > drivers/gpu/drm/drm_memory.c:2: warning: Cannot understand  * \file > drm_memor

[RESEND PATCH] gpu/drm_memory: fix a few warnings

2019-06-07 Thread Qian Cai
ates the MIT license further down in the source file. WARNING: Missing or malformed SPDX-License-Identifier tag in line 1 Signed-off-by: Qian Cai --- drivers/gpu/drm/drm_memory.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_memory.c b/drivers/gpu/dr

[RESEND PATCH v2] drm/vmwgfx: fix a warning due to missing dma_parms

2019-06-03 Thread Qian Cai
Mware Virtual GPU") Suggested-by: Thomas Hellstrom Signed-off-by: Qian Cai --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c index 4ff11a0077e1..5f690429eb89 100644 --- a/drive

[PATCH] gpu/drm_memory: fix a few warnings

2019-05-31 Thread Qian Cai
se-Identifier tag in line 1 Signed-off-by: Qian Cai --- drivers/gpu/drm/drm_memory.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_memory.c b/drivers/gpu/drm/drm_memory.c index 132fef8ff1b6..683042c8ee2c 100644 --- a/drivers/gpu/drm/drm_memory.c +

[PATCH v2] drm/vmwgfx: fix a warning due to missing dma_parms

2019-05-24 Thread Qian Cai
Mware Virtual GPU") Signed-off-by: Qian Cai --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c index bf6c3500d363..e10fe109ee48 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c +++

[PATCH] drm/vmwgfx: fix a warning due to missing dma_parms

2019-05-23 Thread Qian Cai
: fb1d9738ca05 ("drm/vmwgfx: Add DRM driver for VMware Virtual GPU") Signed-off-by: Qian Cai --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c b/drivers/gpu/drm/vmwgfx/vmwgfx_d