Differential D27265

Use the standard method for localizing of MSI-X table bar.
ClosedPublic
Actions

Authored by mmel on Nov 18 2020, 12:56 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Sep 15, 6:45 PM

	Unknown Object (File)
	Sun, Sep 6, 2:34 AM

	Unknown Object (File)
	Sat, Sep 5, 10:06 PM

	Unknown Object (File)
	Sat, Sep 5, 10:26 AM

	Unknown Object (File)
	Fri, Sep 4, 11:07 PM

	Unknown Object (File)
	Fri, Sep 4, 6:11 PM

	Unknown Object (File)
	Thu, Sep 3, 5:09 AM

	Unknown Object (File)
	Wed, Sep 2, 10:07 PM

Subscribers

Details

Reviewers

None

Group Reviewers

Intel Networking

Commits

rS368698: Use the standard method for localizing of MSI-X table bar.

Summary

Current way, hardcoded value plus heuristic is not conform to the PCI(e)
specification and fails on systems where MSI-X bar is not initialized by
BIOS/ACPI (many arm or arm64 systems for example).
Instead, use the standard PCI(e) capability for determining of
MSIX table bar address.

MFC after: 3 weeks

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

mmel created this revision.Nov 18 2020, 12:56 PM

Herald added a subscriber: imp. · View Herald TranscriptNov 18 2020, 12:56 PM

mmel requested review of this revision.Nov 18 2020, 12:56 PM

erj edited reviewers, added: Intel Networking; removed: erj.Nov 24 2020, 9:31 PM

erj added a subscriber: Intel Networking.

erj removed a subscriber: Intel Networking.

ping.

This revision was not accepted when it landed; it landed in state Needs Review.Dec 16 2020, 2:39 PM

Closed by commit rS368698: Use the standard method for localizing of MSI-X table bar. (authored by mmel). · Explain Why

This revision was automatically updated to reflect the committed changes.

mmel added a commit: rS368698: Use the standard method for localizing of MSI-X table bar..

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

e1000/

1 line

6 lines

Diff 80762

head/sys/dev/e1000/if_em.h

Loading...

head/sys/dev/e1000/if_em.c

Loading...