Page MenuHomeFreeBSD

Update MPI headers to version 42.

Authored by slm on Apr 25 2016, 10:09 PM.



Update MPI headers to version 42.

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

slm updated this revision to Diff 15592.Apr 25 2016, 10:09 PM
slm retitled this revision from to Update MPI headers to version 42..
slm updated this object.
slm edited the test plan for this revision. (Show Details)
slm added reviewers: ken, scottl, ambrisko, asomers, mav, allanjude.
slm set the repository for this revision to rS FreeBSD src repository.
asomers requested changes to this revision.Apr 25 2016, 10:57 PM
asomers edited edge metadata.
asomers added inline comments.
651 ↗(On Diff #15592)

Exceeds 80 chars per line here and several other places.

1437 ↗(On Diff #15592)

Line 1436 got duped.

This revision now requires changes to proceed.Apr 25 2016, 10:57 PM
scottl added inline comments.Apr 26 2016, 1:45 AM
651 ↗(On Diff #15592)

Since this is common platform header code, I don't think that we should be too concerned about style compliance, especially since the existing style is established and the additions do not alter it.

slm added inline comments.Apr 26 2016, 5:12 PM
651 ↗(On Diff #15592)

I agree with Scott on this. These headers are used in all of our drivers, apps, FW, etc. I'd like to keep them as close as possible to our actual released headers that we keep internally. The FreeBSD driver headers are modified a little bit from the released versions. Some licensing stuff is added and some newer items are removed to keep them from getting out into the public too soon. Other than that, these are the actual header files. I can remove that dup'ed line however.

ken accepted this revision.Apr 26 2016, 5:53 PM
ken edited edge metadata.

I agree with Steve and Scott. The MPI headers need to be largely the same between platforms. Doing otherwise will just lead to unnecessary effort.

1437 ↗(On Diff #15592)

I agree, this will probably cause a compile error.

slm updated this revision to Diff 15745.Apr 29 2016, 9:34 PM
slm edited edge metadata.
slm removed rS FreeBSD src repository as the repository for this revision.

Removed duplicated define.

asomers accepted this revision.May 2 2016, 3:02 PM
asomers edited edge metadata.

I'll accept this revision even with the long lines. But they should still be fixed in Avago's source version. Some of them reach 160 characters.

This revision is now accepted and ready to land.May 2 2016, 3:02 PM
This revision was automatically updated to reflect the committed changes.