Re: [PATCH v3 2/4] mm: cma: add number of pages to debug message in cma_release()

2015-02-24 Thread Michal Nazarewicz
On Tue, Feb 24 2015, Stefan Strogin  wrote:
> It's more useful to print address and number of pages which are being 
> released,
> not only address.
>
> Signed-off-by: Stefan Strogin 

Acked-by: Michal Nazarewicz 

> ---
>  mm/cma.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/mm/cma.c b/mm/cma.c
> index 3a63c96..111bf62 100644
> --- a/mm/cma.c
> +++ b/mm/cma.c
> @@ -434,7 +434,7 @@ bool cma_release(struct cma *cma, struct page *pages, int 
> count)
>   if (!cma || !pages)
>   return false;
>  
> - pr_debug("%s(page %p)\n", __func__, (void *)pages);
> + pr_debug("%s(page %p, count %d)\n", __func__, (void *)pages, count);
>  
>   pfn = page_to_pfn(pages);
>  
> -- 
> 2.1.0
>

-- 
Best regards, _ _
.o. | Liege of Serenely Enlightened Majesty of  o' \,=./ `o
..o | Computer Science,  Michał “mina86” Nazarewicz(o o)
ooo +--ooO--(_)--Ooo--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v3 2/4] mm: cma: add number of pages to debug message in cma_release()

2015-02-24 Thread Michal Nazarewicz
On Tue, Feb 24 2015, Stefan Strogin s.stro...@partner.samsung.com wrote:
 It's more useful to print address and number of pages which are being 
 released,
 not only address.

 Signed-off-by: Stefan Strogin s.stro...@partner.samsung.com

Acked-by: Michal Nazarewicz min...@mina86.com

 ---
  mm/cma.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/mm/cma.c b/mm/cma.c
 index 3a63c96..111bf62 100644
 --- a/mm/cma.c
 +++ b/mm/cma.c
 @@ -434,7 +434,7 @@ bool cma_release(struct cma *cma, struct page *pages, int 
 count)
   if (!cma || !pages)
   return false;
  
 - pr_debug(%s(page %p)\n, __func__, (void *)pages);
 + pr_debug(%s(page %p, count %d)\n, __func__, (void *)pages, count);
  
   pfn = page_to_pfn(pages);
  
 -- 
 2.1.0


-- 
Best regards, _ _
.o. | Liege of Serenely Enlightened Majesty of  o' \,=./ `o
..o | Computer Science,  Michał “mina86” Nazarewicz(o o)
ooo +--m...@google.com--xmpp:min...@jabber.org--ooO--(_)--Ooo--
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/