Re: [PATCH v2] mm: memblock: add more debug logs

2020-11-15 Thread Mike Rapoport
On Mon, Nov 16, 2020 at 10:14:04AM +0530, Faiyaz Mohammed wrote:
> It is useful to know the exact caller of memblock_phys_alloc_range() to
> track early memory reservations during development.
> 
> Currently, when memblock debugging is enabled, the allocations done with
> memblock_phys_alloc_range() are only reported at memblock_reserve():
> 
> [0.00] memblock_reserve: [0x00023fc6b000-0x00023fc6bfff] 
> memblock_alloc_range_nid+0xc0/0x188
> 
> Add memblock_dbg() to memblock_phys_alloc_range() to get details about
> its usage.
> 
> For example:
> 
> [0.00] memblock_phys_alloc_range: 4096 bytes align=0x1000 
> from=0x max_addr=0x 
> early_pgtable_alloc+0x24/0x178
> [0.00] memblock_reserve: [0x00023fc6b000-0x00023fc6bfff] 
> memblock_alloc_range_nid+0xc0/0x188
> 
> Signed-off-by: Faiyaz Mohammed 

Applied, thanks!



[PATCH v2] mm: memblock: add more debug logs

2020-11-15 Thread Faiyaz Mohammed
It is useful to know the exact caller of memblock_phys_alloc_range() to
track early memory reservations during development.

Currently, when memblock debugging is enabled, the allocations done with
memblock_phys_alloc_range() are only reported at memblock_reserve():

[0.00] memblock_reserve: [0x00023fc6b000-0x00023fc6bfff] 
memblock_alloc_range_nid+0xc0/0x188

Add memblock_dbg() to memblock_phys_alloc_range() to get details about
its usage.

For example:

[0.00] memblock_phys_alloc_range: 4096 bytes align=0x1000 
from=0x max_addr=0x 
early_pgtable_alloc+0x24/0x178
[0.00] memblock_reserve: [0x00023fc6b000-0x00023fc6bfff] 
memblock_alloc_range_nid+0xc0/0x188

Signed-off-by: Faiyaz Mohammed 
---
 mm/memblock.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/mm/memblock.c b/mm/memblock.c
index 049df41..f65af9f 100644
--- a/mm/memblock.c
+++ b/mm/memblock.c
@@ -1419,6 +1419,9 @@ phys_addr_t __init memblock_phys_alloc_range(phys_addr_t 
size,
 phys_addr_t start,
 phys_addr_t end)
 {
+   memblock_dbg("%s: %llu bytes align=0x%llx from=%pa max_addr=%pa %pS\n",
+   __func__, (u64)size, (u64)align, , ,
+   (void *)_RET_IP_);
return memblock_alloc_range_nid(size, align, start, end, NUMA_NO_NODE,
false);
 }
-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
member of the Code Aurora Forum, hosted by The Linux Foundation