Page MenuHomeFreeBSD

Refactor MPR/MPS interrupt setup code.

Authored by scottl on Jan 19 2017, 10:57 AM.



Split the interrupt setup code into two parts: allocation and
configuration. Do the allocation before requesting the IOCFacts message.
This triggers the LSI firmware to recognize the multiqueue should be
enabled if available. As a side-effect, fetch the driver tunables as
early as possible.

Diff Detail

rS FreeBSD src repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

scottl retitled this revision from to Refactor MPR/MPS interrupt setup code..Jan 19 2017, 10:57 AM
scottl updated this object.
scottl edited the test plan for this revision. (Show Details)
scottl added reviewers: slm, ken, imp.
scottl updated this revision to Diff 24192.
slm added inline comments.Jan 19 2017, 6:23 PM
237 ↗(On Diff #24192)

Setting sc->msi_msgs to 0 in the 'else' part may not do anything since you set it again before exiting. 'msgs' may or may not be initialized above, so it actually looks like it's possible to set msi_msgs to an unknown value before exit.

scottl edited edge metadata.Jan 19 2017, 9:23 PM
scottl updated this revision to Diff 24215.

Fix an uninitialized use of msi_msgs.

slm edited edge metadata.Jan 19 2017, 9:49 PM
slm accepted this revision.
This revision is now accepted and ready to land.Jan 19 2017, 9:49 PM