Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add support for Intel SDE 9.9.0 #980

Merged
merged 17 commits into from
Jun 28, 2022
Merged

Add support for Intel SDE 9.9.0 #980

merged 17 commits into from
Jun 28, 2022

Conversation

yi-tseng-intel
Copy link
Contributor

@yi-tseng-intel yi-tseng-intel commented Jun 9, 2022 •
edited

@yi-tseng-intel
Support SDE 9.9.0 and removes old SDE and stratum_bf
a18af8b 
 - Start from SDE 9.9.0, we need to add p4lang/target-syslibs and
p4lang/target-utils when building the SDE.
 - This PR also removes the stratum_bf target since we only support
stratum_bfrt after version 9.7.0
@yi-tseng-intel yi-tseng-intel marked this pull request as draft June 9, 2022 04:59
@codecov
Copy link

codecov bot commented Jun 23, 2022 •
edited

Codecov Report

Merging #980 (5aaead2) into main (87c447d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #980   +/-   ##
=======================================
  Coverage   79.39%   79.39%           
=======================================
  Files         342      342           
  Lines       31165    31165           
=======================================
  Hits        24742    24742           
  Misses       6423     6423

@yi-tseng-intel yi-tseng-intel marked this pull request as ready for review June 23, 2022 23:46
@yi-tseng-intel yi-tseng-intel changed the title Support SDE 9.9.0 and removes old SDE and stratum_bf Support SDE 9.9.0 and removes old SDE Jun 24, 2022
@yi-tseng-intel yi-tseng-intel changed the title Support SDE 9.9.0 and removes old SDE Support SDE 9.9.0 and removes old SDE versions Jun 24, 2022
@pudelkoM pudelkoM mentioned this pull request Jun 27, 2022
Merged
@pudelkoM pudelkoM changed the title Support SDE 9.9.0 and removes old SDE versions Add support for Intel SDE 9.9.0 Jun 27, 2022
@yi-tseng-intel yi-tseng-intel mentioned this pull request Jun 27, 2022
Merged
@pudelkoM
Copy link
Member

I'm hitting a compile error for the SDE:

Command: BSP=/home/max/bf-reference-bsp-9.9.0.tgz stratum/hal/bin/barefoot/docker/build-stratum-bf-container.sh /home/max/bf-sde-9.9.0.tgz /home/max/stratum/linux-4.14.49-OpenNetworkLinux.tar.xz

/tmp/tmp.9mGpEYT17N/src/hashtbl/hashtbl.c:16:43: fatal error: target-sys/bf_sal/bf_sys_intf.h: No such file or directory
 #include <target-sys/bf_sal/bf_sys_intf.h>
                                           ^
compilation terminated.
src/CMakeFiles/target_sysutil_o.dir/build.make:86: recipe for target 'src/CMakeFiles/target_sysutil_o.dir/hashtbl/hashtbl.c.o' failed
make[2]: *** [src/CMakeFiles/target_sysutil_o.dir/hashtbl/hashtbl.c.o] Error 1
CMakeFiles/Makefile2:130: recipe for target 'src/CMakeFiles/target_sysutil_o.dir/all' failed
make[1]: *** [src/CMakeFiles/target_sysutil_o.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....

@yi-tseng-intel
Fix build-bf-sde.sh
342438f 
Use standalone build for target-utils
@pudelkoM
Copy link
Member

Seems to fixed now. Made a full Stratum-bfrt build that started on a HW switch.

pudelkoM
pudelkoM previously approved these changes Jun 28, 2022
View reviewed changes
pudelkoM
pudelkoM previously approved these changes Jun 28, 2022
View reviewed changes
@pudelkoM pudelkoM merged commit d40842e into stratum:main Jun 28, 2022
@pudelkoM pudelkoM mentioned this pull request Jun 28, 2022
Merged
ffoulkes added a commit to ipdk-io/stratum-dev that referenced this pull request Jan 18, 2024
@ffoulkes
Add support for Intel SDE 9.9.0 (stratum#980)
8d15235 
Signed-off-by: Derek G Foster <derek.foster@intel.com>
ffoulkes added a commit to ipdk-io/stratum-dev that referenced this pull request Jan 18, 2024
@ffoulkes
Add support for Intel SDE 9.9.0 (stratum#980)
dc85f67 
Signed-off-by: Derek G Foster <derek.foster@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pudelkoM pudelkoM pudelkoM approved these changes

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@yi-tseng-intel @pudelkoM