All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 5/7] MIPS: Cocci spatch "vma_pages"
  2017-09-20 22:29 [PATCH 0/7] Cocci spatch "vma_pages" - v4.14-rc1 Thomas Meyer
  2017-09-20 22:29   ` Thomas Meyer
@ 2017-09-20 22:29 ` Thomas Meyer
  2017-09-20 22:29 ` [PATCH 4/7] microblaze: " Thomas Meyer
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: ralf, linux-mips, linux-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/arch/mips/mm/dma-default.c b/arch/mips/mm/dma-default.c
--- a/arch/mips/mm/dma-default.c
+++ b/arch/mips/mm/dma-default.c
@@ -179,7 +179,7 @@ static int mips_dma_mmap(struct device *
 	void *cpu_addr, dma_addr_t dma_addr, size_t size,
 	unsigned long attrs)
 {
-	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
+	unsigned long user_count = vma_pages(vma);
 	unsigned long count = PAGE_ALIGN(size) >> PAGE_SHIFT;
 	unsigned long addr = (unsigned long)cpu_addr;
 	unsigned long off = vma->vm_pgoff;

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 6/7] KVM: PPC: Cocci spatch "vma_pages"
  2017-09-20 22:29 [PATCH 0/7] Cocci spatch "vma_pages" - v4.14-rc1 Thomas Meyer
@ 2017-09-20 22:29   ` Thomas Meyer
  2017-09-20 22:29 ` [PATCH 5/7] MIPS: " Thomas Meyer
                     ` (5 subsequent siblings)
  6 siblings, 0 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: agraf, benh, paulus, mpe, kvm-ppc, linuxppc-dev, linux-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/arch/powerpc/kvm/e500_mmu_host.c b/arch/powerpc/kvm/e500_mmu_host.c
--- a/arch/powerpc/kvm/e500_mmu_host.c
+++ b/arch/powerpc/kvm/e500_mmu_host.c
@@ -377,7 +377,7 @@ static inline int kvmppc_e500_shadow_map
 
 			start = vma->vm_pgoff;
 			end = start +
-			      ((vma->vm_end - vma->vm_start) >> PAGE_SHIFT);
+			      vma_pages(vma);
 
 			pfn = start + ((hva - vma->vm_start) >> PAGE_SHIFT);
 

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
  2017-09-20 22:29 [PATCH 0/7] Cocci spatch "vma_pages" - v4.14-rc1 Thomas Meyer
@ 2017-09-20 22:29   ` Thomas Meyer
  2017-09-20 22:29 ` [PATCH 5/7] MIPS: " Thomas Meyer
                     ` (5 subsequent siblings)
  6 siblings, 0 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: mark.yao, airlied, heiko, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
--- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
+++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
@@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
 {
 	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
 	unsigned int i, count = obj->size >> PAGE_SHIFT;
-	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
+	unsigned long user_count = vma_pages(vma);
 	unsigned long uaddr = vma->vm_start;
 	unsigned long offset = vma->vm_pgoff;
 	unsigned long end = user_count + offset;

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 1/7] ARM: dma-mapping: Cocci spatch "vma_pages"
  2017-09-20 22:29 [PATCH 0/7] Cocci spatch "vma_pages" - v4.14-rc1 Thomas Meyer
@ 2017-09-20 22:29   ` Thomas Meyer
  2017-09-20 22:29 ` [PATCH 5/7] MIPS: " Thomas Meyer
                     ` (5 subsequent siblings)
  6 siblings, 0 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: linux, linux-arm-kernel, linux-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/arch/arm/mm/dma-mapping.c b/arch/arm/mm/dma-mapping.c
--- a/arch/arm/mm/dma-mapping.c
+++ b/arch/arm/mm/dma-mapping.c
@@ -846,7 +846,7 @@ static int __arm_dma_mmap(struct device
 		 unsigned long attrs)
 {
 	int ret;
-	unsigned long nr_vma_pages = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
+	unsigned long nr_vma_pages = vma_pages(vma);
 	unsigned long nr_pages = PAGE_ALIGN(size) >> PAGE_SHIFT;
 	unsigned long pfn = dma_to_pfn(dev, dma_addr);
 	unsigned long off = vma->vm_pgoff;

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 4/7] microblaze: Cocci spatch "vma_pages"
  2017-09-20 22:29 [PATCH 0/7] Cocci spatch "vma_pages" - v4.14-rc1 Thomas Meyer
  2017-09-20 22:29   ` Thomas Meyer
  2017-09-20 22:29 ` [PATCH 5/7] MIPS: " Thomas Meyer
@ 2017-09-20 22:29 ` Thomas Meyer
  2017-09-25 13:37   ` Michal Simek
  2017-09-20 22:29   ` Thomas Meyer
                   ` (3 subsequent siblings)
  6 siblings, 1 reply; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: monstr, linux-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/arch/microblaze/kernel/dma.c b/arch/microblaze/kernel/dma.c
--- a/arch/microblaze/kernel/dma.c
+++ b/arch/microblaze/kernel/dma.c
@@ -165,7 +165,7 @@ int dma_direct_mmap_coherent(struct devi
 			     unsigned long attrs)
 {
 #ifdef CONFIG_MMU
-	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
+	unsigned long user_count = vma_pages(vma);
 	unsigned long count = PAGE_ALIGN(size) >> PAGE_SHIFT;
 	unsigned long off = vma->vm_pgoff;
 	unsigned long pfn;

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 0/7] Cocci spatch "vma_pages" - v4.14-rc1
@ 2017-09-20 22:29 Thomas Meyer
  2017-09-20 22:29   ` Thomas Meyer
                   ` (6 more replies)
  0 siblings, 7 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: linux-kernel

Use vma_pages function on vma object instead of explicit computation.

Found by coccinelle spatch "api/vma_pages.cocci"

Run against version v4.14-rc1

Let me know when you as a maintainer are not interested in these kind of patches.
I can exclude you by path; e.g. all findings in "drivers/scsi" will never
be reported again by this semi-automatic program runs.

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 3/7] metag: Cocci spatch "vma_pages"
  2017-09-20 22:29 [PATCH 0/7] Cocci spatch "vma_pages" - v4.14-rc1 Thomas Meyer
                   ` (3 preceding siblings ...)
  2017-09-20 22:29   ` Thomas Meyer
@ 2017-09-20 22:29 ` Thomas Meyer
  2017-09-20 22:29   ` Thomas Meyer
  2017-09-20 22:29   ` Thomas Meyer
  6 siblings, 0 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: linux-metag, linux-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/arch/metag/kernel/dma.c b/arch/metag/kernel/dma.c
--- a/arch/metag/kernel/dma.c
+++ b/arch/metag/kernel/dma.c
@@ -342,7 +342,7 @@ static int metag_dma_mmap(struct device
 	else
 		vma->vm_page_prot = pgprot_noncached(vma->vm_page_prot);
 
-	user_size = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
+	user_size = vma_pages(vma);
 
 	spin_lock_irqsave(&consistent_lock, flags);
 	c = metag_vm_region_find(&consistent_head, (unsigned long)cpu_addr);

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 2/7] arm64: Cocci spatch "vma_pages"
  2017-09-20 22:29 [PATCH 0/7] Cocci spatch "vma_pages" - v4.14-rc1 Thomas Meyer
@ 2017-09-20 22:29   ` Thomas Meyer
  2017-09-20 22:29 ` [PATCH 5/7] MIPS: " Thomas Meyer
                     ` (5 subsequent siblings)
  6 siblings, 0 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: catalin.marinas, will.deacon, linux-arm-kernel, linux-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/arch/arm64/mm/dma-mapping.c b/arch/arm64/mm/dma-mapping.c
--- a/arch/arm64/mm/dma-mapping.c
+++ b/arch/arm64/mm/dma-mapping.c
@@ -303,8 +303,7 @@ static int __swiotlb_mmap_pfn(struct vm_
 			      unsigned long pfn, size_t size)
 {
 	int ret = -ENXIO;
-	unsigned long nr_vma_pages = (vma->vm_end - vma->vm_start) >>
-					PAGE_SHIFT;
+	unsigned long nr_vma_pages = vma_pages(vma);
 	unsigned long nr_pages = PAGE_ALIGN(size) >> PAGE_SHIFT;
 	unsigned long off = vma->vm_pgoff;
 

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
@ 2017-09-20 22:29   ` Thomas Meyer
  0 siblings, 0 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: linux-arm-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
--- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
+++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
@@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
 {
 	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
 	unsigned int i, count = obj->size >> PAGE_SHIFT;
-	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
+	unsigned long user_count = vma_pages(vma);
 	unsigned long uaddr = vma->vm_start;
 	unsigned long offset = vma->vm_pgoff;
 	unsigned long end = user_count + offset;

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 2/7] arm64: Cocci spatch "vma_pages"
@ 2017-09-20 22:29   ` Thomas Meyer
  0 siblings, 0 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: linux-arm-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/arch/arm64/mm/dma-mapping.c b/arch/arm64/mm/dma-mapping.c
--- a/arch/arm64/mm/dma-mapping.c
+++ b/arch/arm64/mm/dma-mapping.c
@@ -303,8 +303,7 @@ static int __swiotlb_mmap_pfn(struct vm_
 			      unsigned long pfn, size_t size)
 {
 	int ret = -ENXIO;
-	unsigned long nr_vma_pages = (vma->vm_end - vma->vm_start) >>
-					PAGE_SHIFT;
+	unsigned long nr_vma_pages = vma_pages(vma);
 	unsigned long nr_pages = PAGE_ALIGN(size) >> PAGE_SHIFT;
 	unsigned long off = vma->vm_pgoff;
 

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 1/7] ARM: dma-mapping: Cocci spatch "vma_pages"
@ 2017-09-20 22:29   ` Thomas Meyer
  0 siblings, 0 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: linux-arm-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/arch/arm/mm/dma-mapping.c b/arch/arm/mm/dma-mapping.c
--- a/arch/arm/mm/dma-mapping.c
+++ b/arch/arm/mm/dma-mapping.c
@@ -846,7 +846,7 @@ static int __arm_dma_mmap(struct device
 		 unsigned long attrs)
 {
 	int ret;
-	unsigned long nr_vma_pages = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
+	unsigned long nr_vma_pages = vma_pages(vma);
 	unsigned long nr_pages = PAGE_ALIGN(size) >> PAGE_SHIFT;
 	unsigned long pfn = dma_to_pfn(dev, dma_addr);
 	unsigned long off = vma->vm_pgoff;

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 6/7] KVM: PPC: Cocci spatch "vma_pages"
@ 2017-09-20 22:29   ` Thomas Meyer
  0 siblings, 0 replies; 30+ messages in thread
From: Thomas Meyer @ 2017-09-20 22:29 UTC (permalink / raw)
  To: agraf, benh, paulus, mpe, kvm-ppc, linuxppc-dev, linux-kernel

Use vma_pages function on vma object instead of explicit computation.
Found by coccinelle spatch "api/vma_pages.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/arch/powerpc/kvm/e500_mmu_host.c b/arch/powerpc/kvm/e500_mmu_host.c
--- a/arch/powerpc/kvm/e500_mmu_host.c
+++ b/arch/powerpc/kvm/e500_mmu_host.c
@@ -377,7 +377,7 @@ static inline int kvmppc_e500_shadow_map
 
 			start = vma->vm_pgoff;
 			end = start +
-			      ((vma->vm_end - vma->vm_start) >> PAGE_SHIFT);
+			      vma_pages(vma);
 
 			pfn = start + ((hva - vma->vm_start) >> PAGE_SHIFT);
 

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
  2017-09-20 22:29   ` Thomas Meyer
  (?)
@ 2017-09-21  1:02     ` Mark yao
  -1 siblings, 0 replies; 30+ messages in thread
From: Mark yao @ 2017-09-21  1:02 UTC (permalink / raw)
  To: Thomas Meyer, airlied, heiko, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

On 2017年09月21日 06:29, Thomas Meyer wrote:
> Use vma_pages function on vma object instead of explicit computation.
> Found by coccinelle spatch "api/vma_pages.cocci"
>
> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> ---
Looks good for me:
Acked-by: Mark Yao <mark.yao@rock-chips.com>

>
> diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
>   {
>   	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
>   	unsigned int i, count = obj->size >> PAGE_SHIFT;
> -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> +	unsigned long user_count = vma_pages(vma);
>   	unsigned long uaddr = vma->vm_start;
>   	unsigned long offset = vma->vm_pgoff;
>   	unsigned long end = user_count + offset;
>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rockchip
>
>
>

-- 
Mark Yao

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
@ 2017-09-21  1:02     ` Mark yao
  0 siblings, 0 replies; 30+ messages in thread
From: Mark yao @ 2017-09-21  1:02 UTC (permalink / raw)
  To: Thomas Meyer, airlied, heiko, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

On 2017年09月21日 06:29, Thomas Meyer wrote:
> Use vma_pages function on vma object instead of explicit computation.
> Found by coccinelle spatch "api/vma_pages.cocci"
>
> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> ---
Looks good for me:
Acked-by: Mark Yao <mark.yao@rock-chips.com>

>
> diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
>   {
>   	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
>   	unsigned int i, count = obj->size >> PAGE_SHIFT;
> -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> +	unsigned long user_count = vma_pages(vma);
>   	unsigned long uaddr = vma->vm_start;
>   	unsigned long offset = vma->vm_pgoff;
>   	unsigned long end = user_count + offset;
>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rockchip
>
>
>

-- 
Mark Yao


_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
@ 2017-09-21  1:02     ` Mark yao
  0 siblings, 0 replies; 30+ messages in thread
From: Mark yao @ 2017-09-21  1:02 UTC (permalink / raw)
  To: linux-arm-kernel

On 2017?09?21? 06:29, Thomas Meyer wrote:
> Use vma_pages function on vma object instead of explicit computation.
> Found by coccinelle spatch "api/vma_pages.cocci"
>
> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> ---
Looks good for me:
Acked-by: Mark Yao <mark.yao@rock-chips.com>

>
> diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
>   {
>   	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
>   	unsigned int i, count = obj->size >> PAGE_SHIFT;
> -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> +	unsigned long user_count = vma_pages(vma);
>   	unsigned long uaddr = vma->vm_start;
>   	unsigned long offset = vma->vm_pgoff;
>   	unsigned long end = user_count + offset;
>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rockchip
>
>
>

-- 
?ark Yao

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 4/7] microblaze: Cocci spatch "vma_pages"
  2017-09-20 22:29 ` [PATCH 4/7] microblaze: " Thomas Meyer
@ 2017-09-25 13:37   ` Michal Simek
  0 siblings, 0 replies; 30+ messages in thread
From: Michal Simek @ 2017-09-25 13:37 UTC (permalink / raw)
  To: Thomas Meyer, linux-kernel


[-- Attachment #1.1: Type: text/plain, Size: 1076 bytes --]

On 21.9.2017 00:29, Thomas Meyer wrote:
> Use vma_pages function on vma object instead of explicit computation.
> Found by coccinelle spatch "api/vma_pages.cocci"
> 
> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> ---
> 
> diff -u -p a/arch/microblaze/kernel/dma.c b/arch/microblaze/kernel/dma.c
> --- a/arch/microblaze/kernel/dma.c
> +++ b/arch/microblaze/kernel/dma.c
> @@ -165,7 +165,7 @@ int dma_direct_mmap_coherent(struct devi
>  			     unsigned long attrs)
>  {
>  #ifdef CONFIG_MMU
> -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> +	unsigned long user_count = vma_pages(vma);
>  	unsigned long count = PAGE_ALIGN(size) >> PAGE_SHIFT;
>  	unsigned long off = vma->vm_pgoff;
>  	unsigned long pfn;
> 

Applied to mb/next.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
  2017-09-21  1:02     ` Mark yao
  (?)
@ 2017-09-26  5:12       ` Daniel Vetter
  -1 siblings, 0 replies; 30+ messages in thread
From: Daniel Vetter @ 2017-09-26  5:12 UTC (permalink / raw)
  To: Mark yao
  Cc: Thomas Meyer, airlied, heiko, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

On Thu, Sep 21, 2017 at 09:02:22AM +0800, Mark yao wrote:
> On 2017年09月21日 06:29, Thomas Meyer wrote:
> > Use vma_pages function on vma object instead of explicit computation.
> > Found by coccinelle spatch "api/vma_pages.cocci"
> > 
> > Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> > ---
> Looks good for me:
> Acked-by: Mark Yao <mark.yao@rock-chips.com>

Once more a maintainer who acks a patch and doesn't push it. This is
really confusing, who exactly do you expect to handle this patch for you?

Please push to drm-misc-next (also for future patches), thanks.
-Daniel

> 
> > 
> > diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
> >   {
> >   	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
> >   	unsigned int i, count = obj->size >> PAGE_SHIFT;
> > -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> > +	unsigned long user_count = vma_pages(vma);
> >   	unsigned long uaddr = vma->vm_start;
> >   	unsigned long offset = vma->vm_pgoff;
> >   	unsigned long end = user_count + offset;
> > 
> > _______________________________________________
> > Linux-rockchip mailing list
> > Linux-rockchip@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-rockchip
> > 
> > 
> > 
> 
> -- 
> Mark Yao
> 
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
@ 2017-09-26  5:12       ` Daniel Vetter
  0 siblings, 0 replies; 30+ messages in thread
From: Daniel Vetter @ 2017-09-26  5:12 UTC (permalink / raw)
  To: Mark yao
  Cc: linux-kernel, dri-devel, linux-rockchip, linux-arm-kernel, Thomas Meyer

On Thu, Sep 21, 2017 at 09:02:22AM +0800, Mark yao wrote:
> On 2017年09月21日 06:29, Thomas Meyer wrote:
> > Use vma_pages function on vma object instead of explicit computation.
> > Found by coccinelle spatch "api/vma_pages.cocci"
> > 
> > Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> > ---
> Looks good for me:
> Acked-by: Mark Yao <mark.yao@rock-chips.com>

Once more a maintainer who acks a patch and doesn't push it. This is
really confusing, who exactly do you expect to handle this patch for you?

Please push to drm-misc-next (also for future patches), thanks.
-Daniel

> 
> > 
> > diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
> >   {
> >   	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
> >   	unsigned int i, count = obj->size >> PAGE_SHIFT;
> > -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> > +	unsigned long user_count = vma_pages(vma);
> >   	unsigned long uaddr = vma->vm_start;
> >   	unsigned long offset = vma->vm_pgoff;
> >   	unsigned long end = user_count + offset;
> > 
> > _______________________________________________
> > Linux-rockchip mailing list
> > Linux-rockchip@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-rockchip
> > 
> > 
> > 
> 
> -- 
> Mark Yao
> 
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
@ 2017-09-26  5:12       ` Daniel Vetter
  0 siblings, 0 replies; 30+ messages in thread
From: Daniel Vetter @ 2017-09-26  5:12 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Sep 21, 2017 at 09:02:22AM +0800, Mark yao wrote:
> On 2017?09?21? 06:29, Thomas Meyer wrote:
> > Use vma_pages function on vma object instead of explicit computation.
> > Found by coccinelle spatch "api/vma_pages.cocci"
> > 
> > Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> > ---
> Looks good for me:
> Acked-by: Mark Yao <mark.yao@rock-chips.com>

Once more a maintainer who acks a patch and doesn't push it. This is
really confusing, who exactly do you expect to handle this patch for you?

Please push to drm-misc-next (also for future patches), thanks.
-Daniel

> 
> > 
> > diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
> >   {
> >   	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
> >   	unsigned int i, count = obj->size >> PAGE_SHIFT;
> > -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> > +	unsigned long user_count = vma_pages(vma);
> >   	unsigned long uaddr = vma->vm_start;
> >   	unsigned long offset = vma->vm_pgoff;
> >   	unsigned long end = user_count + offset;
> > 
> > _______________________________________________
> > Linux-rockchip mailing list
> > Linux-rockchip at lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-rockchip
> > 
> > 
> > 
> 
> -- 
> ?ark Yao
> 
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
  2017-09-26  5:12       ` Daniel Vetter
  (?)
@ 2017-09-26  6:33         ` Mark yao
  -1 siblings, 0 replies; 30+ messages in thread
From: Mark yao @ 2017-09-26  6:33 UTC (permalink / raw)
  To: Thomas Meyer, airlied, heiko, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

On 2017年09月26日 13:12, Daniel Vetter wrote:
> On Thu, Sep 21, 2017 at 09:02:22AM +0800, Mark yao wrote:
>> On 2017年09月21日 06:29, Thomas Meyer wrote:
>>> Use vma_pages function on vma object instead of explicit computation.
>>> Found by coccinelle spatch "api/vma_pages.cocci"
>>>
>>> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
>>> ---
>> Looks good for me:
>> Acked-by: Mark Yao <mark.yao@rock-chips.com>
> Once more a maintainer who acks a patch and doesn't push it. This is
> really confusing, who exactly do you expect to handle this patch for you?
>
> Please push to drm-misc-next (also for future patches), thanks.
> -Daniel

Hi Daniel
I Saw the patch title is "[PATCH 7/7]", I guessed it's one of a series of patches and maybe it can pushed by series.

Ok, Pushed it to drm-misc-next.

>
>>> diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
>>>    {
>>>    	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
>>>    	unsigned int i, count = obj->size >> PAGE_SHIFT;
>>> -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
>>> +	unsigned long user_count = vma_pages(vma);
>>>    	unsigned long uaddr = vma->vm_start;
>>>    	unsigned long offset = vma->vm_pgoff;
>>>    	unsigned long end = user_count + offset;
>>>
>>> _______________________________________________
>>> Linux-rockchip mailing list
>>> Linux-rockchip@lists.infradead.org
>>> http://lists.infradead.org/mailman/listinfo/linux-rockchip
>>>
>>>
>>>
>> -- 
>> Mark Yao
>>
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel


-- 
Mark Ya

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
@ 2017-09-26  6:33         ` Mark yao
  0 siblings, 0 replies; 30+ messages in thread
From: Mark yao @ 2017-09-26  6:33 UTC (permalink / raw)
  To: Thomas Meyer, airlied, heiko, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

On 2017年09月26日 13:12, Daniel Vetter wrote:
> On Thu, Sep 21, 2017 at 09:02:22AM +0800, Mark yao wrote:
>> On 2017年09月21日 06:29, Thomas Meyer wrote:
>>> Use vma_pages function on vma object instead of explicit computation.
>>> Found by coccinelle spatch "api/vma_pages.cocci"
>>>
>>> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
>>> ---
>> Looks good for me:
>> Acked-by: Mark Yao <mark.yao@rock-chips.com>
> Once more a maintainer who acks a patch and doesn't push it. This is
> really confusing, who exactly do you expect to handle this patch for you?
>
> Please push to drm-misc-next (also for future patches), thanks.
> -Daniel

Hi Daniel
I Saw the patch title is "[PATCH 7/7]", I guessed it's one of a series of patches and maybe it can pushed by series.

Ok, Pushed it to drm-misc-next.

>
>>> diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
>>>    {
>>>    	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
>>>    	unsigned int i, count = obj->size >> PAGE_SHIFT;
>>> -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
>>> +	unsigned long user_count = vma_pages(vma);
>>>    	unsigned long uaddr = vma->vm_start;
>>>    	unsigned long offset = vma->vm_pgoff;
>>>    	unsigned long end = user_count + offset;
>>>
>>> _______________________________________________
>>> Linux-rockchip mailing list
>>> Linux-rockchip@lists.infradead.org
>>> http://lists.infradead.org/mailman/listinfo/linux-rockchip
>>>
>>>
>>>
>> -- 
>> Mark Yao
>>
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel


-- 
Mark Ya


_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
@ 2017-09-26  6:33         ` Mark yao
  0 siblings, 0 replies; 30+ messages in thread
From: Mark yao @ 2017-09-26  6:33 UTC (permalink / raw)
  To: linux-arm-kernel

On 2017?09?26? 13:12, Daniel Vetter wrote:
> On Thu, Sep 21, 2017 at 09:02:22AM +0800, Mark yao wrote:
>> On 2017?09?21? 06:29, Thomas Meyer wrote:
>>> Use vma_pages function on vma object instead of explicit computation.
>>> Found by coccinelle spatch "api/vma_pages.cocci"
>>>
>>> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
>>> ---
>> Looks good for me:
>> Acked-by: Mark Yao <mark.yao@rock-chips.com>
> Once more a maintainer who acks a patch and doesn't push it. This is
> really confusing, who exactly do you expect to handle this patch for you?
>
> Please push to drm-misc-next (also for future patches), thanks.
> -Daniel

Hi Daniel
I Saw the patch title is "[PATCH 7/7]", I guessed it's one of a series of patches and maybe it can pushed by series.

Ok, Pushed it to drm-misc-next.

>
>>> diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
>>>    {
>>>    	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
>>>    	unsigned int i, count = obj->size >> PAGE_SHIFT;
>>> -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
>>> +	unsigned long user_count = vma_pages(vma);
>>>    	unsigned long uaddr = vma->vm_start;
>>>    	unsigned long offset = vma->vm_pgoff;
>>>    	unsigned long end = user_count + offset;
>>>
>>> _______________________________________________
>>> Linux-rockchip mailing list
>>> Linux-rockchip at lists.infradead.org
>>> http://lists.infradead.org/mailman/listinfo/linux-rockchip
>>>
>>>
>>>
>> -- 
>> ?ark Yao
>>
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel


-- 
?ark Ya

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
  2017-09-26  6:33         ` Mark yao
  (?)
@ 2017-09-26  8:08           ` Daniel Vetter
  -1 siblings, 0 replies; 30+ messages in thread
From: Daniel Vetter @ 2017-09-26  8:08 UTC (permalink / raw)
  To: Mark yao
  Cc: Thomas Meyer, airlied, heiko, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

On Tue, Sep 26, 2017 at 02:33:07PM +0800, Mark yao wrote:
> On 2017年09月26日 13:12, Daniel Vetter wrote:
> > On Thu, Sep 21, 2017 at 09:02:22AM +0800, Mark yao wrote:
> > > On 2017年09月21日 06:29, Thomas Meyer wrote:
> > > > Use vma_pages function on vma object instead of explicit computation.
> > > > Found by coccinelle spatch "api/vma_pages.cocci"
> > > > 
> > > > Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> > > > ---
> > > Looks good for me:
> > > Acked-by: Mark Yao <mark.yao@rock-chips.com>
> > Once more a maintainer who acks a patch and doesn't push it. This is
> > really confusing, who exactly do you expect to handle this patch for you?
> > 
> > Please push to drm-misc-next (also for future patches), thanks.
> > -Daniel
> 
> Hi Daniel
> I Saw the patch title is "[PATCH 7/7]", I guessed it's one of a series of patches and maybe it can pushed by series.
> 
> Ok, Pushed it to drm-misc-next.

Hm right, but I only see 7/7 here. Either way, except when the author asks
for a preferred tree it's best if you just pick things up right away. And
if you're unsure, just ask instead of risking that a patch drops through
the cracks.
-Daniel

> 
> > 
> > > > diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > > > --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > > > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > > > @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
> > > >    {
> > > >    	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
> > > >    	unsigned int i, count = obj->size >> PAGE_SHIFT;
> > > > -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> > > > +	unsigned long user_count = vma_pages(vma);
> > > >    	unsigned long uaddr = vma->vm_start;
> > > >    	unsigned long offset = vma->vm_pgoff;
> > > >    	unsigned long end = user_count + offset;
> > > > 
> > > > _______________________________________________
> > > > Linux-rockchip mailing list
> > > > Linux-rockchip@lists.infradead.org
> > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip
> > > > 
> > > > 
> > > > 
> > > -- 
> > > Mark Yao
> > > 
> > > 
> > > _______________________________________________
> > > dri-devel mailing list
> > > dri-devel@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> 
> 
> -- 
> Mark Ya
> 
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
@ 2017-09-26  8:08           ` Daniel Vetter
  0 siblings, 0 replies; 30+ messages in thread
From: Daniel Vetter @ 2017-09-26  8:08 UTC (permalink / raw)
  To: Mark yao
  Cc: linux-kernel, dri-devel, linux-rockchip, linux-arm-kernel, Thomas Meyer

On Tue, Sep 26, 2017 at 02:33:07PM +0800, Mark yao wrote:
> On 2017年09月26日 13:12, Daniel Vetter wrote:
> > On Thu, Sep 21, 2017 at 09:02:22AM +0800, Mark yao wrote:
> > > On 2017年09月21日 06:29, Thomas Meyer wrote:
> > > > Use vma_pages function on vma object instead of explicit computation.
> > > > Found by coccinelle spatch "api/vma_pages.cocci"
> > > > 
> > > > Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> > > > ---
> > > Looks good for me:
> > > Acked-by: Mark Yao <mark.yao@rock-chips.com>
> > Once more a maintainer who acks a patch and doesn't push it. This is
> > really confusing, who exactly do you expect to handle this patch for you?
> > 
> > Please push to drm-misc-next (also for future patches), thanks.
> > -Daniel
> 
> Hi Daniel
> I Saw the patch title is "[PATCH 7/7]", I guessed it's one of a series of patches and maybe it can pushed by series.
> 
> Ok, Pushed it to drm-misc-next.

Hm right, but I only see 7/7 here. Either way, except when the author asks
for a preferred tree it's best if you just pick things up right away. And
if you're unsure, just ask instead of risking that a patch drops through
the cracks.
-Daniel

> 
> > 
> > > > diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > > > --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > > > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > > > @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
> > > >    {
> > > >    	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
> > > >    	unsigned int i, count = obj->size >> PAGE_SHIFT;
> > > > -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> > > > +	unsigned long user_count = vma_pages(vma);
> > > >    	unsigned long uaddr = vma->vm_start;
> > > >    	unsigned long offset = vma->vm_pgoff;
> > > >    	unsigned long end = user_count + offset;
> > > > 
> > > > _______________________________________________
> > > > Linux-rockchip mailing list
> > > > Linux-rockchip@lists.infradead.org
> > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip
> > > > 
> > > > 
> > > > 
> > > -- 
> > > Mark Yao
> > > 
> > > 
> > > _______________________________________________
> > > dri-devel mailing list
> > > dri-devel@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> 
> 
> -- 
> Mark Ya
> 
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
@ 2017-09-26  8:08           ` Daniel Vetter
  0 siblings, 0 replies; 30+ messages in thread
From: Daniel Vetter @ 2017-09-26  8:08 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Sep 26, 2017 at 02:33:07PM +0800, Mark yao wrote:
> On 2017?09?26? 13:12, Daniel Vetter wrote:
> > On Thu, Sep 21, 2017 at 09:02:22AM +0800, Mark yao wrote:
> > > On 2017?09?21? 06:29, Thomas Meyer wrote:
> > > > Use vma_pages function on vma object instead of explicit computation.
> > > > Found by coccinelle spatch "api/vma_pages.cocci"
> > > > 
> > > > Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> > > > ---
> > > Looks good for me:
> > > Acked-by: Mark Yao <mark.yao@rock-chips.com>
> > Once more a maintainer who acks a patch and doesn't push it. This is
> > really confusing, who exactly do you expect to handle this patch for you?
> > 
> > Please push to drm-misc-next (also for future patches), thanks.
> > -Daniel
> 
> Hi Daniel
> I Saw the patch title is "[PATCH 7/7]", I guessed it's one of a series of patches and maybe it can pushed by series.
> 
> Ok, Pushed it to drm-misc-next.

Hm right, but I only see 7/7 here. Either way, except when the author asks
for a preferred tree it's best if you just pick things up right away. And
if you're unsure, just ask instead of risking that a patch drops through
the cracks.
-Daniel

> 
> > 
> > > > diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > > > --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > > > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > > > @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
> > > >    {
> > > >    	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
> > > >    	unsigned int i, count = obj->size >> PAGE_SHIFT;
> > > > -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> > > > +	unsigned long user_count = vma_pages(vma);
> > > >    	unsigned long uaddr = vma->vm_start;
> > > >    	unsigned long offset = vma->vm_pgoff;
> > > >    	unsigned long end = user_count + offset;
> > > > 
> > > > _______________________________________________
> > > > Linux-rockchip mailing list
> > > > Linux-rockchip at lists.infradead.org
> > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip
> > > > 
> > > > 
> > > > 
> > > -- 
> > > ?ark Yao
> > > 
> > > 
> > > _______________________________________________
> > > dri-devel mailing list
> > > dri-devel at lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> 
> 
> -- 
> ?ark Ya
> 
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 7/7] drm/rockchip: Cocci spatch "vma_pages"
  2017-09-26  8:08           ` Daniel Vetter
  (?)
  (?)
@ 2017-09-26  9:06           ` Mark yao
  -1 siblings, 0 replies; 30+ messages in thread
From: Mark yao @ 2017-09-26  9:06 UTC (permalink / raw)
  To: Thomas Meyer, airlied, heiko, dri-devel, linux-arm-kernel,
	linux-rockchip, linux-kernel

On 2017年09月26日 16:08, Daniel Vetter wrote:
> On Tue, Sep 26, 2017 at 02:33:07PM +0800, Mark yao wrote:
>> On 2017年09月26日 13:12, Daniel Vetter wrote:
>>> On Thu, Sep 21, 2017 at 09:02:22AM +0800, Mark yao wrote:
>>>> On 2017年09月21日 06:29, Thomas Meyer wrote:
>>>>> Use vma_pages function on vma object instead of explicit computation.
>>>>> Found by coccinelle spatch "api/vma_pages.cocci"
>>>>>
>>>>> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
>>>>> ---
>>>> Looks good for me:
>>>> Acked-by: Mark Yao <mark.yao@rock-chips.com>
>>> Once more a maintainer who acks a patch and doesn't push it. This is
>>> really confusing, who exactly do you expect to handle this patch for you?
>>>
>>> Please push to drm-misc-next (also for future patches), thanks.
>>> -Daniel
>> Hi Daniel
>> I Saw the patch title is "[PATCH 7/7]", I guessed it's one of a series of patches and maybe it can pushed by series.
>>
>> Ok, Pushed it to drm-misc-next.
> Hm right, but I only see 7/7 here. Either way, except when the author asks
> for a preferred tree it's best if you just pick things up right away. And
> if you're unsure, just ask instead of risking that a patch drops through
> the cracks.
> -Daniel

Okay, I'll keep it in mind next time.

Thanks
Mark

>
>>>>> diff -u -p a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>>>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>>>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>>>> @@ -220,7 +220,7 @@ static int rockchip_drm_gem_object_mmap_
>>>>>     {
>>>>>     	struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj);
>>>>>     	unsigned int i, count = obj->size >> PAGE_SHIFT;
>>>>> -	unsigned long user_count = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
>>>>> +	unsigned long user_count = vma_pages(vma);
>>>>>     	unsigned long uaddr = vma->vm_start;
>>>>>     	unsigned long offset = vma->vm_pgoff;
>>>>>     	unsigned long end = user_count + offset;
>>>>>
>>>>> _______________________________________________
>>>>> Linux-rockchip mailing list
>>>>> Linux-rockchip@lists.infradead.org
>>>>> http://lists.infradead.org/mailman/listinfo/linux-rockchip
>>>>>
>>>>>
>>>>>
>>>> -- 
>>>> Mark Yao
>>>>
>>>>
>>>> _______________________________________________
>>>> dri-devel mailing list
>>>> dri-devel@lists.freedesktop.org
>>>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>>
>> -- 
>> Mark Ya
>>
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 2/7] arm64: Cocci spatch "vma_pages"
  2017-09-20 22:29   ` Thomas Meyer
@ 2017-09-26 11:57     ` Robin Murphy
  -1 siblings, 0 replies; 30+ messages in thread
From: Robin Murphy @ 2017-09-26 11:57 UTC (permalink / raw)
  To: Thomas Meyer, catalin.marinas, will.deacon, linux-arm-kernel,
	linux-kernel

On 20/09/17 23:29, Thomas Meyer wrote:
> Use vma_pages function on vma object instead of explicit computation.
> Found by coccinelle spatch "api/vma_pages.cocci"

Reviewed-by: Robin Murphy <robin.murphy@arm.com>

I guess this can be picked up independently of the other patches?

Robin.

> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> ---
> 
> diff -u -p a/arch/arm64/mm/dma-mapping.c b/arch/arm64/mm/dma-mapping.c
> --- a/arch/arm64/mm/dma-mapping.c
> +++ b/arch/arm64/mm/dma-mapping.c
> @@ -303,8 +303,7 @@ static int __swiotlb_mmap_pfn(struct vm_
>  			      unsigned long pfn, size_t size)
>  {
>  	int ret = -ENXIO;
> -	unsigned long nr_vma_pages = (vma->vm_end - vma->vm_start) >>
> -					PAGE_SHIFT;
> +	unsigned long nr_vma_pages = vma_pages(vma);
>  	unsigned long nr_pages = PAGE_ALIGN(size) >> PAGE_SHIFT;
>  	unsigned long off = vma->vm_pgoff;
>  
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 2/7] arm64: Cocci spatch "vma_pages"
@ 2017-09-26 11:57     ` Robin Murphy
  0 siblings, 0 replies; 30+ messages in thread
From: Robin Murphy @ 2017-09-26 11:57 UTC (permalink / raw)
  To: linux-arm-kernel

On 20/09/17 23:29, Thomas Meyer wrote:
> Use vma_pages function on vma object instead of explicit computation.
> Found by coccinelle spatch "api/vma_pages.cocci"

Reviewed-by: Robin Murphy <robin.murphy@arm.com>

I guess this can be picked up independently of the other patches?

Robin.

> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> ---
> 
> diff -u -p a/arch/arm64/mm/dma-mapping.c b/arch/arm64/mm/dma-mapping.c
> --- a/arch/arm64/mm/dma-mapping.c
> +++ b/arch/arm64/mm/dma-mapping.c
> @@ -303,8 +303,7 @@ static int __swiotlb_mmap_pfn(struct vm_
>  			      unsigned long pfn, size_t size)
>  {
>  	int ret = -ENXIO;
> -	unsigned long nr_vma_pages = (vma->vm_end - vma->vm_start) >>
> -					PAGE_SHIFT;
> +	unsigned long nr_vma_pages = vma_pages(vma);
>  	unsigned long nr_pages = PAGE_ALIGN(size) >> PAGE_SHIFT;
>  	unsigned long off = vma->vm_pgoff;
>  
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 6/7] KVM: PPC: Cocci spatch "vma_pages"
  2017-09-20 22:29   ` Thomas Meyer
@ 2017-10-19  3:46     ` Paul Mackerras
  -1 siblings, 0 replies; 30+ messages in thread
From: Paul Mackerras @ 2017-10-19  3:46 UTC (permalink / raw)
  To: Thomas Meyer; +Cc: agraf, benh, mpe, kvm-ppc, linuxppc-dev, linux-kernel

On Thu, Sep 21, 2017 at 12:29:36AM +0200, Thomas Meyer wrote:
> Use vma_pages function on vma object instead of explicit computation.
> Found by coccinelle spatch "api/vma_pages.cocci"
> 
> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>

Thanks, applied to my kvm-ppc-next branch, with the headline "KVM:
PPC: BookE: Use vma_pages function".

Paul.

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 6/7] KVM: PPC: Cocci spatch "vma_pages"
@ 2017-10-19  3:46     ` Paul Mackerras
  0 siblings, 0 replies; 30+ messages in thread
From: Paul Mackerras @ 2017-10-19  3:46 UTC (permalink / raw)
  To: Thomas Meyer; +Cc: agraf, benh, mpe, kvm-ppc, linuxppc-dev, linux-kernel

On Thu, Sep 21, 2017 at 12:29:36AM +0200, Thomas Meyer wrote:
> Use vma_pages function on vma object instead of explicit computation.
> Found by coccinelle spatch "api/vma_pages.cocci"
> 
> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>

Thanks, applied to my kvm-ppc-next branch, with the headline "KVM:
PPC: BookE: Use vma_pages function".

Paul.

^ permalink raw reply	[flat|nested] 30+ messages in thread

end of thread, other threads:[~2017-10-19  3:49 UTC | newest]

Thread overview: 30+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-20 22:29 [PATCH 0/7] Cocci spatch "vma_pages" - v4.14-rc1 Thomas Meyer
2017-09-20 22:29 ` [PATCH 1/7] ARM: dma-mapping: Cocci spatch "vma_pages" Thomas Meyer
2017-09-20 22:29   ` Thomas Meyer
2017-09-20 22:29 ` [PATCH 5/7] MIPS: " Thomas Meyer
2017-09-20 22:29 ` [PATCH 4/7] microblaze: " Thomas Meyer
2017-09-25 13:37   ` Michal Simek
2017-09-20 22:29 ` [PATCH 7/7] drm/rockchip: " Thomas Meyer
2017-09-20 22:29   ` Thomas Meyer
2017-09-21  1:02   ` Mark yao
2017-09-21  1:02     ` Mark yao
2017-09-21  1:02     ` Mark yao
2017-09-26  5:12     ` Daniel Vetter
2017-09-26  5:12       ` Daniel Vetter
2017-09-26  5:12       ` Daniel Vetter
2017-09-26  6:33       ` Mark yao
2017-09-26  6:33         ` Mark yao
2017-09-26  6:33         ` Mark yao
2017-09-26  8:08         ` Daniel Vetter
2017-09-26  8:08           ` Daniel Vetter
2017-09-26  8:08           ` Daniel Vetter
2017-09-26  9:06           ` Mark yao
2017-09-20 22:29 ` [PATCH 3/7] metag: " Thomas Meyer
2017-09-20 22:29 ` [PATCH 2/7] arm64: " Thomas Meyer
2017-09-20 22:29   ` Thomas Meyer
2017-09-26 11:57   ` Robin Murphy
2017-09-26 11:57     ` Robin Murphy
2017-09-20 22:29 ` [PATCH 6/7] KVM: PPC: " Thomas Meyer
2017-09-20 22:29   ` Thomas Meyer
2017-10-19  3:46   ` Paul Mackerras
2017-10-19  3:46     ` Paul Mackerras

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.