<tt><font size=2>> be at the beginning of the puts() string?</font></tt><br><font size=2 face="sans-serif">Yes, those are always at the beginning.</font><br><br><tt><font size=2>> Would it be worthwhile removing the strcasestr(..,
"error"/"warning"),</font></tt><br><font size=2 face="sans-serif">Yeah, not sure what you'll get with
those.</font><br><br><tt><font size=2>> Or is FAIL_MRK more of a warning? Or a notice
("normal but significant condition")?</font></tt><br><font size=2 face="sans-serif">I've honestly never seen a F> in
any trace I've looked at.  I suspect we copied these in from the FSP
and just never used it.  I would equate FAIL with ERR</font><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">Frederic Bonnard <FREDERIC@fr.ibm.com>,
skiboot@lists.ozlabs.org</font><br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">06/03/2017 12:20 AM</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>>>  - 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>> I'm good with that.  We just need to get PE to update their FFDC<br>> collection scripts to grab the new file once it shows up.<br>> <br>> <br>> I would suggest you add a couple more cases in your compares.  We've
got<br>> these constant in HB that are used a lot (though not all over) as
the<br>> first 2 characters in our traces.  <br>> #define ERR_MRK   "E>"<br>> #define FAIL_MRK  "F>"<br>> #define WARN_MRK  "W>"<br>> #define INFO_MRK  "I>"<br><br>Oh, that's great then. If one of these is present, is it going to<br>be at the beginning of the puts() string?<br><br>Would it be worthwhile removing the strcasestr(.., "error"/"warning"),<br>and *just* using these instead? That would prevent the potential for<br>false classification.<br><br>We'd probably map both FAIL_MRK and ERR_MRK to LOG_ERROR. Or is FAIL_MRK<br>more of a warning? Or a notice ("normal but significant condition")?<br><br>Cheers,<br><br><br>Jeremy<br><br></font></tt><br><br><BR>