Avoid calling blessed() more often than needed.
ClosedPublic
Actions

Authored by markj on Aug 2 2019, 3:25 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Dec 28, 2:09 AM

	Unknown Object (File)
	Fri, Dec 27, 12:05 PM

	Unknown Object (File)
	Oct 11 2024, 4:55 PM

	Unknown Object (File)
	Sep 23 2024, 1:13 AM

	Unknown Object (File)
	Sep 22 2024, 9:21 PM

	Unknown Object (File)
	Sep 19 2024, 11:53 AM

	Unknown Object (File)
	Sep 18 2024, 7:39 PM

	Unknown Object (File)
	Sep 18 2024, 7:17 PM

View All 47 Files

Subscribers

imp

Details

Reviewers

jhb

Commits

rS350540: Only check the blessings table for known LORs.

Summary

Only call it once we've detected a reversal and have set a flag in the
corresponding matrix entries. The badstacks handler now must also omit
stacks corresponding to blessed lock pairs.

Test Plan

Checked that we don't log blessed LORs in badstacks.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Aug 2 2019, 3:25 PM

Harbormaster completed remote builds in B25643: Diff 60391.Aug 2 2019, 3:25 PM

markj edited the test plan for this revision. (Show Details)Aug 2 2019, 3:25 PM

markj added a reviewer: jhb.

jhb accepted this revision.Aug 2 2019, 3:46 PM

This revision is now accepted and ready to land.Aug 2 2019, 3:46 PM

Closed by commit rS350540: Only check the blessings table for known LORs. (authored by markj). · Explain WhyAug 2 2019, 6:01 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rS350540: Only check the blessings table for known LORs..

Herald added a subscriber: imp. · View Herald TranscriptAug 2 2019, 6:01 PM

lwhsu mentioned this in D21134: Avoid calling blessed() more often than needed..Aug 3 2019, 9:47 AM

Revision Contents
Changeset List

Path

Size

head/

sys/

kern/

subr_witness.c

19 lines

Diff 60395

View Options

head/sys/kern/subr_witness.c

Avoid calling blessed() more often than needed.ClosedPublicActions