tablet/grub
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

mm: Avoid complex heap growth math in hot path

Browse Source 
We do a lot of math about heap growth in hot path of grub_memalign().
However, the result is only used if out of memory is encountered, which
is seldom.

This patch moves these calculations away from hot path. These
calculations are now only done if out of memory is encountered. This
change can also help compiler to optimize integer overflow checks away.

Signed-off-by: Zhang Boyang <zhangboyang.id@gmail.com>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
This commit is contained in:
Zhang Boyang 2023-01-29 19:49:33 +08:00 committed by Daniel Kiper
parent 21869baec1
commit 65bc459630
1 changed files with 20 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
grub-core/kern/mm.c
View File

@ -467,20 +467,7 @@ grub_memalign (grub_size_t align, grub_size_t size)
if (size > ~(grub_size_t) align)
goto fail;
/*
* Pre-calculate the necessary size of heap growth (if applicable),
* with region management overhead taken into account.
*/
if (grub_add (size + align, GRUB_MM_MGMT_OVERHEAD, &grow))
goto fail;
/* Preallocate some extra space if heap growth is small. */
grow = grub_max (grow, GRUB_MM_HEAP_GROW_EXTRA);
/* Align up heap growth to make it friendly to CPU/MMU. */
if (grow > ~(grub_size_t) (GRUB_MM_HEAP_GROW_ALIGN - 1))
goto fail;
grow = ALIGN_UP (grow, GRUB_MM_HEAP_GROW_ALIGN);
grow = size + align;
/* We currently assume at least a 32-bit grub_size_t,
so limiting allocations to <adress space size> - 1MiB
@ -510,6 +497,25 @@ grub_memalign (grub_size_t align, grub_size_t size)
/* Request additional pages, contiguous */
count++;
/*
* Calculate the necessary size of heap growth (if applicable),
* with region management overhead taken into account.
*/
if (grub_add (grow, GRUB_MM_MGMT_OVERHEAD, &grow))
goto fail;
/* Preallocate some extra space if heap growth is small. */
grow = grub_max (grow, GRUB_MM_HEAP_GROW_EXTRA);
/* Align up heap growth to make it friendly to CPU/MMU. */
if (grow > ~(grub_size_t) (GRUB_MM_HEAP_GROW_ALIGN - 1))
goto fail;
grow = ALIGN_UP (grow, GRUB_MM_HEAP_GROW_ALIGN);
/* Do the same sanity check again. */
if (grow > ~(grub_size_t) 0x100000)
goto fail;
if (grub_mm_add_region_fn != NULL &&
grub_mm_add_region_fn (grow, GRUB_MM_ADD_REGION_CONSECUTIVE) == GRUB_ERR_NONE)
goto again;