Update NNPI backend to 0.5.2.5.

[omromano]

Summary:
Update of NNPI Backend to v0.5.2.5.

[facebook-github-bot]

Hi @omromano!

Thank you for your pull request and welcome to our community.We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file.

In order for us to review and merge your code, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks!

[omromano]

@jfix71 @arunm-git Please take a look.

Thanks
CC: @jsubag

[jfix71]

LGTM!

[facebook-github-bot]

@hl475 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

[facebook-github-bot]

@hl475 merged this pull request in 0d8034b.

[hl475]

Thanks @omromano for the PR! This PR has been landed into Facebook internal with some additional change:

1. NNPIHostManagerTest.cpp: We blacked list testStaticAssignmentP2PandDRTConcurrent/0, testStaticAssignmentDeviceResidentTensorOnly/0, testStaticAssignmentP2PandDRT/0, testStaticAssignmentP2POnly/0, testSaturateHost/0, ConcurrentAddRemoveUnique/0, whereas the original PR has only ConcurrentAddRemoveUnique/0 in the blacked list. These tests are causing hang during our test NNPIHostManagerTest.cpp.
2. We revert the change in importer.cpp. The change in importer.cpp passes NNPI/OperatorStatelessTest.rowwiseQuantizedFCTestSymmetric/0 test on our B0 hosts, but fail on A0 hosts.

If you can please help us take a look so we can fix it in the next release, that would be really appreciated.

cc @arunm-git

[omromano]

Thanks @hl475, we will address these issue in the next PR.