radix_trie: pass fewer params to node_get
ClosedPublic
Actions

Authored by dougm on Jun 23 2023, 8:50 AM.

Tags

None

Referenced Files

	F174846733: D40723.id123789.diff
	Tue, Oct 6, 11:50 AM

	F174846682: D40723.id123669.diff
	Tue, Oct 6, 11:50 AM

	F174846469: D40723.id123851.diff
	Tue, Oct 6, 11:47 AM

	F174845808: D40723.diff
	Tue, Oct 6, 11:38 AM

	Unknown Object (File)
	Mon, Sep 28, 7:11 PM

	Unknown Object (File)
	Sun, Sep 27, 5:04 PM

	Unknown Object (File)
	Sun, Sep 27, 4:10 PM

	Unknown Object (File)
	Sun, Sep 27, 3:52 PM

View All Files

Subscribers

imp

Details

Reviewers

alc
markj

Commits

rGda72505f9c6a: radix_trie: pass fewer params to node_get

Summary

Let node_get calculate it's own owner value. Don't pass the count parameter, since it's always 2. Save 16 bytes in insert(). Move, without modifying, slot and trimkey to handle use-before-declaration problem.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

dougm requested review of this revision.Jun 23 2023, 8:50 AM

dougm created this revision.

This patch needs a refresh after the trimkey patch.

markj accepted this revision.Jun 27 2023, 2:00 PM

markj added inline comments.

sys/kern/subr_pctrie.c
101	Perhaps remove the extra newline here since you're moving this function anyway.

This revision is now accepted and ready to land.Jun 27 2023, 2:00 PM

Closed by commit rGda72505f9c6a: radix_trie: pass fewer params to node_get (authored by dougm). · Explain WhyJun 27 2023, 5:22 PM

This revision was automatically updated to reflect the committed changes.

dougm added a commit: rGda72505f9c6a: radix_trie: pass fewer params to node_get.

Herald added a subscriber: imp. · View Herald TranscriptJun 27 2023, 5:22 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

subr_pctrie.c

48 lines

vm/

vm_radix.c

45 lines

Diff 123789

View Options

sys/kern/subr_pctrie.c