From bdcafee412ab6eec58f6b315e16e57fe3dad002f Mon Sep 17 00:00:00 2001 From: Sergey Bugaev Date: Wed, 5 Jul 2023 17:16:37 +0300 Subject: vm: Make vm_object_coalesce return new object and offset vm_object_coalesce() callers used to rely on the fact that it always merged the next_object into prev_object, potentially destroying next_object and leaving prev_object the result of the whole operation. After ee65849bec5da261be90f565bee096abb4117bdd "vm: Allow coalescing null object with an internal object", this is no longer true, since in case of prev_object == VM_OBJECT_NULL and next_object != VM_OBJECT_NULL, the overall result is next_object, not prev_object. The current callers are prepared to deal with this since they handle this case seprately anyway, but the following commit will introduce another caller that handles both cases in the same code path. So, declare the way vm_object_coalesce() coalesces the two objects its implementation detail, and make it return the resulting object and the offset into it explicitly. This simplifies the callers, too. Message-Id: <20230705141639.85792-2-bugaevc@gmail.com> --- vm/vm_object.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'vm/vm_object.h') diff --git a/vm/vm_object.h b/vm/vm_object.h index 46328a38..9c17541f 100644 --- a/vm/vm_object.h +++ b/vm/vm_object.h @@ -247,7 +247,9 @@ extern boolean_t vm_object_coalesce( vm_offset_t prev_offset, vm_offset_t next_offset, vm_size_t prev_size, - vm_size_t next_size); + vm_size_t next_size, + vm_object_t *new_object, /* OUT */ + vm_offset_t *new_offset); /* OUT */ extern void vm_object_pager_wakeup(ipc_port_t pager); -- cgit v1.2.3