Differential D4333

sfxge: retry VF vAdaptor allocation if it fails because of no EVB port yet
ClosedPublic
Actions

Authored by arybchik on Dec 1 2015, 7:57 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Apr 3, 8:59 PM

	Unknown Object (File)
	Jan 29 2024, 3:39 PM

	Unknown Object (File)
	Dec 26 2023, 9:40 PM

	Unknown Object (File)
	Dec 20 2023, 12:53 AM

	Unknown Object (File)
	Nov 28 2023, 2:10 PM

	Unknown Object (File)
	Nov 4 2023, 1:45 AM

	Unknown Object (File)
	Oct 28 2023, 3:19 AM

	Unknown Object (File)
	Oct 26 2023, 4:32 PM

View All 51 Files

Subscribers

Details

Reviewers

philip
np
gnn
bz

Commits

rS291590: sfxge: retry VF vAdaptor allocation if it fails because of no EVB port yet

Summary

After an MC reboot, a VF driver may reset before the PF driver has
finished bringing everything back up. This includes the VFs EVB port.
MC_CMD_VADAPTOR_ALLOC is the first MCDI call after an MC reboot to
require the EVB port, so if it fails with MC_CMD_ERR_NO_EVB_PORT,
retry the command a few times after waiting a while.

Submitted by: Mark Spender <mspender at solarflare.com>
Sponsored by: Solarflare Communications, Inc.
MFC after: 2 days

Test Plan

Driver build tested. Module loaded/unloaded.
sfxge FreeBSD driver does not support VFs yet, so it is just a preparation.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

arybchik updated this revision to Diff 10628.Dec 1 2015, 7:57 AM

arybchik retitled this revision from to sfxge: retry VF vAdaptor allocation if it fails because of no EVB port yet.

arybchik updated this object.

arybchik edited the test plan for this revision. (Show Details)

arybchik added reviewers: gnn, philip, bz, np.

Herald added a subscriber: imp. · View Herald TranscriptDec 1 2015, 7:57 AM

philip accepted this revision.Dec 1 2015, 8:33 AM

philip edited edge metadata.

This revision is now accepted and ready to land.Dec 1 2015, 8:33 AM

Closed by commit rS291590: sfxge: retry VF vAdaptor allocation if it fails because of no EVB port yet (authored by arybchik). · Explain WhyDec 1 2015, 3:55 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

sfxge/

common/

42 lines

Diff 10641

head/sys/dev/sfxge/common/hunt_nic.c

Loading...