<font size=2 face="sans-serif">I can't think of a way this can fail.</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">Vasant Hegde <hegdevasant@linux.vnet.ibm.com>,
skiboot@lists.ozlabs.org</font><br><font size=1 color=#5f5f5f face="sans-serif">Cc:      
 </font><font size=1 face="sans-serif">Dan Crowell <dcrowell@us.ibm.com></font><br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">05/25/2017 06:34 PM</font><br><font size=1 color=#5f5f5f face="sans-serif">Subject:    
   </font><font size=1 face="sans-serif">Re: [Skiboot]
[PATCH RFC 07/12] opal-prd: use get_ipoll_mask() callback</font><br><hr noshade><br><br><br><tt><font size=2>Hi Vasant,<br><br>>> @@ -1515,6 +1516,13 @@ static int run_attn_loop(struct opal_prd_ctx
*ctx)<br>>>          }<br>>>      }<br>>><br>>> +    if (hservice_runtime->get_ipoll_events) {<br>>> +        pr_debug("HBRT: calling get_ipoll_events");<br>>> +        opal_prd_ipoll = call_get_ipoll_events();<br>> <br>> So if get_ipoll_events fails then we endup passing same value. But<br>> looking into interface definition, I don't see a way to validate the<br>> output. May be validate for negative values?<br><br>This function can't fail (there isn't a return value to indicate an<br>error), and it returns an unsigned value. Besides, we can't check for<br>negative, as valid values currently have the top-bit set anyway...<br><br>That's okay though, as the HBRT implementation will just likely just be<br>a simple return <value>.<br><br>Dan: if that's not the case (and you do need a facility to indicate an<br>error), then we could define 0xff..ff as an error condition. However, we<br>would have no option but to exit from the daemon anyway, so you could<br>get the same behaviour by calling callbacks->assert()..<br><br>Cheers,<br><br><br>Jeremy<br><br></font></tt><br><br><BR>