<tt><font size=2>>  - all log output will go to a new log file,
/var/log/opal-prd.log<br>><br>>  - log messages of LOG_WARNING and LOG_ERROR *also* go to the
standard<br>>    syslog paths (generally /var/log/syslog).<br></font></tt><br><font size=2 face="sans-serif">I'm good with that.  We just need
to get PE to update their FFDC collection scripts to grab the new file
once it shows up.</font><br><br><br><font size=2 face="sans-serif">I would suggest you add a couple more
cases in your compares.  We've got these constant in HB that are used
a lot (though not all over) as the first 2 characters in our traces.  </font><br><font size=2 face="sans-serif">#define ERR_MRK   "E>"</font><br><font size=2 face="sans-serif">#define FAIL_MRK  "F>"</font><br><font size=2 face="sans-serif">#define WARN_MRK  "W>"</font><br><font size=2 face="sans-serif">#define INFO_MRK  "I>"</font><br><br><font size=2 face="sans-serif"><br>--<br>Dan Crowell<br>Senior Software Engineer - Power Systems Enablement Firmware<br>IBM Rochester: t/l 553-2987<br>dcrowell@us.ibm.com</font><br><br><br><br><font size=1 color=#5f5f5f face="sans-serif">From:      
 </font><font size=1 face="sans-serif">Jeremy Kerr <jk@ozlabs.org></font><br><font size=1 color=#5f5f5f face="sans-serif">To:      
 </font><font size=1 face="sans-serif">Daniel M Crowell <dcrowell@us.ibm.com></font><br><font size=1 color=#5f5f5f face="sans-serif">Cc:      
 </font><font size=1 face="sans-serif">skiboot@lists.ozlabs.org,
Frederic Bonnard <FREDERIC@fr.ibm.com></font><br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">06/02/2017 10:15 PM</font><br><font size=1 color=#5f5f5f face="sans-serif">Subject:    
   </font><font size=1 face="sans-serif">Re: [PATCH RFC
05/12] opal-prd: implement basic heuristic for hservice_puts loglevels</font><br><hr noshade><br><br><br><tt><font size=2>Hi Dan,<br><br>> So what will the actual result of these changes be?  Is this
going to<br>> change where our messages show up?<br><br>This only changes the priority of the messages logged. Currently, we log<br>all HBRT output at LOG_INFO priority. After this change, we will log<br>HBRT messages that contain the string "error" as LOG_ERROR and
those<br>that contain "warn" as LOG_WARNING (both are higher priorities
than<br>LOG_INFO).<br><br>So, nothing will change in terms of log locations at this stage.<br><br>However, once this is done, I'd like to propose a second change, to the<br>distributions' packing of opal-prd. The idea would be that:<br><br>  - all log output will go to a new log file, /var/log/opal-prd.log<br><br>  - log messages of LOG_WARNING and LOG_ERROR *also* go to the standard<br>    syslog paths (generally /var/log/syslog).<br><br>This means that we get all the HBRT output stored, but only include<br>higher-priority messages in the standard system logs.<br><br>How does that sound?<br><br>Cheers,<br><br><br>Jeremy<br><br></font></tt><br><br><BR>