tree 441e166cef78f0f8ed645ca8314ad7d5240f576e
parent dec822771b0174a01e72d7641d08e44461b6a82f
author Atish Patra <atish.patra@wdc.com> 1610408701 -0800
committer Palmer Dabbelt <palmerdabbelt@google.com> 1610775346 -0800
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQJHBAABCgAxFiEEKzw3R0RoQ7JKlDp6LhMZ81+7GIkFAmACezITHHBhbG1lckBk
 YWJiZWx0LmNvbQAKCRAuExnzX7sYiZ0ND/9dwrCiHvpFQapwxz/8CqnQtmvySary
 Wyzy9z+tOirdipL5WzIfr9Ew1waSEaY9fWO0MearGGpVvQSulVTOK5GvxU7lImCe
 LVF0iq1mznlXyWL24pneZInS9F872F0GEI4uCpoV7kFDINGB0OYUwd/xVt5wR3Or
 tf2X1atw7G9Oj3gWUVHwKfJ/W7Y9DnJK2W8zD2OTS2A5+VMimeBcjrDGYD/puZiY
 c5mTke3quMK/DGrsq/Aa1IUTQnX0gxiey+0AJZ0axXp5GMbFqq8JyqJQSuMzFXjl
 523NV5ivF2Pb1HDKEYSbcjCLit5G/75aqJHL9O+fo6Tu1sI/y88U+GeKbaHv/gDq
 pfxVLwE5ySBv4PirCP01COycYxPGPS45x3RVvZzkliNnOmo4hQbF+V4aEDHqYJMw
 6El+fG3rG3jXFQP9RJlzaGVvP0db4vTBxqeU+/lURn7JwTor26ZoKLmfoc9XcdkH
 n+/RViP/iWgaqm+jRT+uD+F60IrbRQxzM+FJQqVW4tdPSd89OVmKk2eN4C2meWQT
 pOuWo2aq7p/nUYoIWCZijmuJZW/GTmtYZti18FSqnoC7tEzgl1bLGCXYkj4kaOJl
 ZmT95PoQRqULnAXvw4ZDEub0weFGpens1dEq+WqFLTiZlSLQgYCNp8eXAQQNTxTD
 Y1bISQJ9Qjkwtw==
 =spwe
 -----END PGP SIGNATURE-----

RISC-V: Do not allocate memblock while iterating reserved memblocks

Currently, resource tree allocates memory blocks while iterating on the
list. It leads to following kernel warning because memblock allocation
also invokes memory block reservation API.

[    0.000000] ------------[ cut here ]------------
[    0.000000] WARNING: CPU: 0 PID: 0 at kernel/resource.c:795
__insert_resource+0x8e/0xd0
[    0.000000] Modules linked in:
[    0.000000] CPU: 0 PID: 0 Comm: swapper Not tainted
5.10.0-00022-ge20097fb37e2-dirty #549
[    0.000000] epc: c00125c2 ra : c001262c sp : c1c01f50
[    0.000000]  gp : c1d456e0 tp : c1c0a980 t0 : ffffcf20
[    0.000000]  t1 : 00000000 t2 : 00000000 s0 : c1c01f60
[    0.000000]  s1 : ffffcf00 a0 : ffffff00 a1 : c1c0c0c4
[    0.000000]  a2 : 80c12b15 a3 : 80402000 a4 : 80402000
[    0.000000]  a5 : c1c0c0c4 a6 : 80c12b15 a7 : f5faf600
[    0.000000]  s2 : c1c0c0c4 s3 : c1c0e000 s4 : c1009a80
[    0.000000]  s5 : c1c0c000 s6 : c1d48000 s7 : c1613b4c
[    0.000000]  s8 : 00000fff s9 : 80000200 s10: c1613b40
[    0.000000]  s11: 00000000 t3 : c1d4a000 t4 : ffffffff

This is also unnecessary as we can pre-compute the total memblocks required
for each memory region and allocate it before the loop. It save precious
boot time not going through memblock allocation code every time.

Fixes: 00ab027a3b82 ("RISC-V: Add kernel image sections to the resource tree")

Reviewed-by: Anup Patel <anup@brainfault.org>
Tested-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Atish Patra <atish.patra@wdc.com>
Signed-off-by: Palmer Dabbelt <palmerdabbelt@google.com>