Skip to content
This repository has been archived by the owner on Jan 3, 2023. It is now read-only.

Link just bfn sai lib and shorten linking command by removing other libraries linked at compile time #6

Merged
merged 3 commits into from
Jul 11, 2018
Merged

Link just bfn sai lib and shorten linking command by removing other libraries linked at compile time #6

merged 3 commits into from
Jul 11, 2018

Conversation

mkbalani
Copy link

@mkbalani mkbalani commented Jul 5, 2018

  • all unresolved symbols resolved at runtime

@mkbalani
Link only bfn sai lib
16ef14e 
* all unresolved symbols resolved at runtime
@mkbalani mkbalani requested a review from krambn July 5, 2018 08:10
@mkbalani
Copy link
Author

mkbalani commented Jul 5, 2018

rpath option seems not to be working. Tried some other linker options, which didn't help either. With this change was able to get past syncd compilation. Entire sonic build is yet to complete and I am yet to test the image. But I am positive that this change should work.

@mkbalani mkbalani merged commit 82760ff into bf-master Jul 11, 2018
NStetskovych-zz pushed a commit that referenced this pull request Nov 20, 2018
@shruthi9 @lguohan
Dataplane telemetry support in SONiC (sonic-net#328)
c2c5e7c 
* initial barefoot checkin october 2017

* Revert "Merge branch 'master' of https://github.com/Azure/sonic-sairedis into rel_6_0"

This reverts commit c48a7a2, reversing
changes made to aa5bf64.

* missed integration diffs

* Added new attr type support to sairedis. Also, some fixes for compilation issues

* Changes to add new DTel api support in sairedis

* Add new file to generate Dtel specific SAI stub API

* Missed adding a file in the last commit

* Fix ref point for SAI

* Updated SAI repo to point to dtel_exp

* Changes to handle new additions to DTel experimental SAI. Not compiled yet

* handle platform specific lins in different directory (@runtime)

* force order of library path to look for platform dir before lib dir

* Change SAI branch refpoint

* Update SAI submodule refpoint

* enable fast-boot for barefoot platforms

* Update ref point for SAI

* SONiC changes due to DTel experimental SAI changes

* allow Makefile to build for other platforms - restore it original for non-bfn platforms

* allow clean build

* Revert "allow clean build"

This reverts commit adfdb86.

* Revert "allow Makefile to build for other platforms - restore it original for non-bfn platforms"

This reverts commit d6b0ca3.

* makefile cleanup towards upstream

* Support for platforms based on Barefoot Networks' device (sonic-net#304)

* search for exact string - newer onie versions match multiple lines

* Will need to revert this

Should work without this change. Untested for now

* SONiC sairedis changes needed to work with SAIv1.3

* Fix SAI path in gitmodules and add a comment

* Remove sai thrift build hack

* enable fast-boot for barefoot platforms

* enable fast-boot for barefoot platforms

* Add missing sai rpc hdr file path for bfn

* Update SAI ref points

* Multi p4 profile support for bfn sde (#5)

* Remove code duplicated in merge

* Keep fn in same order as azure master (#8)

* Address review comment

* Link just bfn sai lib and shorten linking command by removing other libraries linked at compile time (#6)

* Link only bfn sai lib

* all unresolved symbols resolved at runtime

* Preload some bfn libraries on syncd startup

* Build fix

* also remove commented code

* Convert tabs to spaces

* Fix alignment

* Address upstream comment

* BFN: link with just sai library for saisdkdump

* Address review comments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@krambn krambn Awaiting requested review from krambn

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mkbalani