kern_kcov: replace vm_page_next() with iterator
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F167661272: D49889.id153873.diff
	Sun, Aug 23, 3:15 PM

	Unknown Object (File)
	Thu, Aug 13, 6:44 AM

	Unknown Object (File)
	Wed, Aug 12, 4:59 PM

	Unknown Object (File)
	Wed, Aug 12, 7:05 AM

	Unknown Object (File)
	Wed, Aug 12, 12:47 AM

	Unknown Object (File)
	Tue, Aug 11, 7:50 PM

	Unknown Object (File)
	Mon, Aug 10, 7:45 PM

	Unknown Object (File)
	Mon, Aug 10, 11:25 AM

View All 79 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