On 7/7/25 07:27, Lorenzo Stoakes wrote: > We const-ify the vrm flags parameter to indicate this will never change. > > We rename resize_is_valid() to remap_is_valid(), as this function does not > only apply to cases where we resize, so it's simply confusing to refer to > that here. > > We remove the BUG() from mremap_at(), as we should not BUG() unless we are > certain it'll result in system instability. > > We rename vrm_charge() to vrm_calc_charge() to make it clear this simply > calculates the charged number of pages rather than actually adjusting any > state. > > We update the comment for vrm_implies_new_addr() to explain that > MREMAP_DONTUNMAP does not require a set address, but will always be moved. > > Additionally consistently use 'res' rather than 'ret' for result values. > > No functional change intended. > > Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@xxxxxxxxxx> Reviewed-by: Vlastimil Babka <vbabka@xxxxxxx>
