Skip to content

Commit

Permalink
Fix missing sbe traces in errorlog
Browse files Browse the repository at this point in the history
There is a size limitation for build field
in sf_magic_cookie_t in fsp-trace. Max size is 128 characters.
But with SBE release now moving to jenkins, the path of SBE
directory is long(172 chars). This fails the sbeStringFIle parsing.

Change-Id: I75cc3b4028bef592f45980773be03ea2e40e7b52
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54098
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
  • Loading branch information
Shakeebbk authored and sgupta2m committed Feb 15, 2018
1 parent 0aae9a8 commit ffa97b5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/tools/trace/ppetracepp.C
Original file line number Diff line number Diff line change
Expand Up @@ -921,7 +921,7 @@ if (!hashtab.empty())
time_t tt = time(NULL);
sprintf(buf, "%s", asctime(localtime(&tt)));
buf[strlen(buf)-1] = '\0'; // chop off extra newline
fprintf(TRC, "#FSP_TRACE_v2|||%s|||BUILD:%s", buf, pwd.c_str());
fprintf(TRC, "#FSP_TRACE_v2|||%s|||BUILD:%.120s", buf, pwd.c_str());

string srch_str = "||";
srch_str += source;
Expand Down
2 changes: 1 addition & 1 deletion src/tools/trace/tracehash.pl
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@
if (defined $dir)
{

$build = $build . $dir; # default to put at top of string file
$build = $build . substr($dir, 0, 120); # default to put at top of string file
if($collect)
{
collect_files($dir);
Expand Down

0 comments on commit ffa97b5

Please sign in to comment.