git: 9ad8116b5461 - main - kerneldump: fix incorrect SETSIZE to BIT_COPY_STORE_REL when livedump

From: Ka Ho Ng <khng_at_FreeBSD.org>
Date: Sat, 23 Nov 2024 16:35:11 UTC
The branch main has been updated by khng:

URL: https://cgit.FreeBSD.org/src/commit/?id=9ad8116b54616a5e8cbcdd2e1432893dce0c608c

commit 9ad8116b54616a5e8cbcdd2e1432893dce0c608c
Author:     Ka Ho Ng <khng@FreeBSD.org>
AuthorDate: 2024-11-23 16:35:07 +0000
Commit:     Ka Ho Ng <khng@FreeBSD.org>
CommitDate: 2024-11-23 16:35:07 +0000

    kerneldump: fix incorrect SETSIZE to BIT_COPY_STORE_REL when livedump
    
    Also replace malloc/free with BITSET_ALLOC/BITSET_FREE macros.
    
    Sponsored by:   Juniper Networks, Inc.
    MFC after:      1 week
    Reviewed by:    markj
    Differential Revision:  https://reviews.freebsd.org/D47708
---
 sys/kern/kern_dump.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/sys/kern/kern_dump.c b/sys/kern/kern_dump.c
index 40355c14ba0f..67c5844013be 100644
--- a/sys/kern/kern_dump.c
+++ b/sys/kern/kern_dump.c
@@ -465,7 +465,6 @@ minidumpsys(struct dumperinfo *di, bool livedump)
 	struct minidumpstate state;
 	struct msgbuf mb_copy;
 	char *msg_ptr;
-	size_t sz;
 	int error;
 
 	if (livedump) {
@@ -510,9 +509,10 @@ minidumpsys(struct dumperinfo *di, bool livedump)
 		msgbuf_duplicate(msgbufp, &mb_copy, msg_ptr);
 		state.msgbufp = &mb_copy;
 
-		sz = BITSET_SIZE(vm_page_dump_pages);
-		state.dump_bitset = malloc(sz, M_TEMP, M_WAITOK);
-		BIT_COPY_STORE_REL(sz, vm_page_dump, state.dump_bitset);
+		state.dump_bitset = BITSET_ALLOC(vm_page_dump_pages, M_TEMP,
+		    M_WAITOK);
+		BIT_COPY_STORE_REL(vm_page_dump_pages, vm_page_dump,
+		    state.dump_bitset);
 	} else {
 		KASSERT(dumping, ("minidump invoked outside of doadump()"));
 
@@ -524,7 +524,7 @@ minidumpsys(struct dumperinfo *di, bool livedump)
 	error = cpu_minidumpsys(di, &state);
 	if (livedump) {
 		free(msg_ptr, M_TEMP);
-		free(state.dump_bitset, M_TEMP);
+		BITSET_FREE(state.dump_bitset, M_TEMP);
 	}
 
 	return (error);