Skip to content

Align kvrealloc() with krealloc()

Danilo Krummrich linux-mm lkml rust-for-linux 2 versions – Applied

Status from this message: [merged mm-stable] mm-kvmalloc-align-kvrealloc-with-krealloc.patch removed from -mm tree

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest2 patches 25 repliesApplied 2/2
  2. v12 patches 7 repliesSuperseded

Changes in v2 from the cover letter

- remove unnecessary extern and move __realloc_size to a new line for vrealloc_noprof() and kvrealloc_noprof()
- drop EXPORT_SYMBOL for vrealloc_noprof()
- rename to_kmalloc_flags() to kmalloc_gfp_adjust()
- fix missing NULL check in vrealloc_noprof()
- rephrase TODO comments in vrealloc_noprof()

Patches in v2 All 2 applied

#Subject and filesSizeRepliesApplied
1/2mm: vmalloc: implement vrealloc()include/linux/vmalloc.h mm/vmalloc.c+63 -013 repliesApplied
2/2mm: kvmalloc: align kvrealloc() with krealloc()arch/arm64/kvm/nested.c arch/powerpc/platforms/pseries/papr-vpd.c drivers/gpu/drm/drm_exec.c +5 more+66 -4410 repliesApplied

Reviews trailers per patch

All 2 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchMichal HockoVlastimil Babka
SeriesA-b
1/2A-b
2/2A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/2] Align kvrealloc() with krealloc()

Hi,

Besides the obvious (and desired) difference between krealloc() and kvrealloc(), there is some inconsistency in their function signatures and behavior:

- krealloc() frees the memory when the requested size is zero, whereas kvrealloc() simply returns a pointer to the existing allocation.

- krealloc() behaves like kmalloc() if a NULL pointer is passed, whereas kvrealloc() does not accept a NULL pointer at all and, if passed, would fault instead.

- krealloc() is self-contained, whereas kvrealloc() relies on the caller to provide the size of the previous allocation.

Inconsistent behavior throughout allocation APIs is error prone, hence make kvrealloc() behave like krealloc(), which seems superior in all mentioned aspects.

In order to be able to get rid of kvrealloc()'s oldsize parameter, introduce vrealloc() and make use of it in kvrealloc().

Making use of vrealloc() in kvrealloc() also provides oppertunities to grow (and shrink) allocations more efficiently. For instance, vrealloc() can be optimized to allocate and map additional pages to grow the allocation or unmap and free unused pages to shrink the allocation.

Besides the above, those functions are required by Rust's allocator abstractons
[1] (rework based on this series in [2]). With Vec or KVec respectively, potentially growing (and shrinking) data structures are rather common.

The patches of this series can also be found in [3].

[1] https://lore.kernel.org/lkml/[email protected]/
[2] https://git.kernel.org/pub/scm/linux/kernel/git/dakr/linux.git/log/?h=rust/mm
[3] https://git.kernel.org/pub/scm/linux/kernel/git/dakr/linux.git/log/?h=mm/krealloc

Changes in v2:
- remove unnecessary extern and move __realloc_size to a new line for vrealloc_noprof() and kvrealloc_noprof()
- drop EXPORT_SYMBOL for vrealloc_noprof()
- rename to_kmalloc_flags() to kmalloc_gfp_adjust()
- fix missing NULL check in vrealloc_noprof()
- rephrase TODO comments in vrealloc_noprof()

Danilo Krummrich (2):
mm: vmalloc: implement vrealloc()
mm: kvmalloc: align kvrealloc() with krealloc()

 arch/arm64/kvm/nested.c                   |  1 -
 arch/powerpc/platforms/pseries/papr-vpd.c |  5 +-
 drivers/gpu/drm/drm_exec.c                |  3 +-
 fs/xfs/xfs_log_recover.c                  |  2 +-
 include/linux/slab.h                      |  4 +-
 include/linux/vmalloc.h                   |  4 +
 kernel/resource.c                         |  3 +-
 lib/fortify_kunit.c                       |  3 +-
 mm/util.c                                 | 89 +++++++++++++++--------
 mm/vmalloc.c                              | 59 +++++++++++++++
 10 files changed, 129 insertions(+), 44 deletions(-)

base-commit: 933069701c1b507825b514317d4edd5d3fd9d417

Signature · 1 line
2.45.2