kern_kcov: replace vm_page_next() with iterator
ClosedPublic
Actions

Authored by dougm on Apr 18 2025, 4:03 PM.

Tags

None

Referenced Files

	F163097012: D49889.id153870.diff
	Mon, Jul 20, 1:34 AM

	F163092460: D49889.diff
	Mon, Jul 20, 12:34 AM

	Unknown Object (File)
	Sat, Jun 20, 3:35 PM

	Unknown Object (File)
	Jun 5 2026, 5:30 AM

	Unknown Object (File)
	May 26 2026, 1:41 AM

	Unknown Object (File)
	May 17 2026, 5:23 PM

	Unknown Object (File)
	May 17 2026, 5:20 PM

	Unknown Object (File)
	May 17 2026, 3:19 AM

View All 62 Files

Subscribers

None

Details

Reviewers

kib

Summary

Use VM_RADIX_FORALL, and drop a use of vm_page_next(), in kcov_free().

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

dougm requested review of this revision.Apr 18 2025, 4:03 PM

dougm created this revision.

kib added inline comments.Apr 18 2025, 4:05 PM

sys/kern/kern_kcov.c
410	What is npages? Should it be &pages?

Pass an iterator pointer, not an iterator, as an argument to VM_FOR macros. Fix the iter argument used in kern_kcov.c.

I suggest to do it as two commits.

This revision is now accepted and ready to land.Apr 18 2025, 4:52 PM

Closed with:

commit 4568f5a286a212fcceaf9a9bb90eabc104829f49
Author: Doug Moore <dougm@FreeBSD.org>
Date: Fri Apr 18 12:18:00 2025 -0500

but without proper Differential Revision label.

Revision Contents
Changeset List

Path

Size

sys/

kern/

kern_kcov.c

9 lines

vm/

vm_radix.h

8 lines

Diff 153873

View Options

sys/kern/kern_kcov.c