git: 57a7f61b13c3 - stable/14 - kerneldump: fix incorrect SETSIZE to BIT_COPY_STORE_REL when livedump
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 30 Nov 2024 06:54:07 UTC
The branch stable/14 has been updated by khng: URL: https://cgit.FreeBSD.org/src/commit/?id=57a7f61b13c3c9c171fa791466dce4df704629c0 commit 57a7f61b13c3c9c171fa791466dce4df704629c0 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-30 06:53:49 +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 (cherry picked from commit 9ad8116b54616a5e8cbcdd2e1432893dce0c608c) --- 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 3afaee9acb7e..a3ca716b2861 100644 --- a/sys/kern/kern_dump.c +++ b/sys/kern/kern_dump.c @@ -466,7 +466,6 @@ minidumpsys(struct dumperinfo *di, bool livedump) struct minidumpstate state; struct msgbuf mb_copy; char *msg_ptr; - size_t sz; int error; if (livedump) { @@ -511,9 +510,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()")); @@ -525,7 +525,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);