Page MenuHomeFreeBSD
Differential D31381

devclass_alloc_unit: move "at" hint test to after device-in-use test
ClosedPublic
Actions

Authored by adam_fenn.io on Aug 2 2021, 12:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 7, 7:12 PM
Unknown Object (File)
Sun, Sep 6, 1:12 AM
Unknown Object (File)
Sat, Sep 5, 12:34 PM
Unknown Object (File)
Fri, Sep 4, 3:37 AM
Unknown Object (File)
Thu, Sep 3, 12:29 PM
Unknown Object (File)
Tue, Sep 1, 10:51 PM
Unknown Object (File)
Tue, Sep 1, 2:57 PM
Unknown Object (File)
Tue, Sep 1, 11:54 AM
View All Files
Subscribers
allanjude
imp
kevans
Contributor Reviews (src)

Details

Reviewers
imp
Commits
rG8ca384eb1d42: devclass_alloc_unit: move "at" hint test to after device-in-use test
Summary

Only perform this expensive operation when the unit number is a
potential candidate (i.e. not already in use), thereby reducing device
scan time on systems with many devices, unit numbers, and drivers.

Sponsored by: NetApp, Inc.
Sponsored by: Klara, Inc.
X-NetApp-PR: #61

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

adam_fenn.io created this revision.Aug 2 2021, 12:46 PM
adam_fenn.io held this revision as a draft.
Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptAug 2 2021, 12:46 PM
Harbormaster completed remote builds in B40803: Diff 93105.Aug 2 2021, 12:46 PM
adam_fenn.io published this revision for review.Aug 2 2021, 12:49 PM
imp accepted this revision.Aug 2 2021, 2:30 PM

This looks good. Need someone to commit it?

This revision is now accepted and ready to land.Aug 2 2021, 2:30 PM
kevans added a subscriber: kevans.Aug 2 2021, 4:15 PM
In D31381#707302, @imp wrote:

This looks good. Need someone to commit it?

I'll grab it

Closed by commit rG8ca384eb1d42: devclass_alloc_unit: move "at" hint test to after device-in-use test (authored by adam_fenn.io, committed by kevans). · Explain WhyAug 2 2021, 4:30 PM
This revision was automatically updated to reflect the committed changes.
kevans added a commit: rG8ca384eb1d42: devclass_alloc_unit: move "at" hint test to after device-in-use test.

Revision Contents
Changeset List

PathSize
sys/
kern/
8 lines

Diff 93128

View Options

sys/kern/subr_bus.c

Loading...