From: SourceForge.net <no...@so...> - 2011-12-23 00:09:02
|
Bugs item #3463092, was opened at 2011-12-20 14:02 Message generated for change (Comment added) made by dr_mohan You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=532251&aid=3463092&group_id=71730 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: HP c-Class Plugin Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: dr_mohan (dr_mohan) Assigned to: dr_mohan (dr_mohan) Summary: hotswap timeout functions are not spec compliant Initial Comment: saHpiAutoInsertTimeoutGet() and saHpiAutoExtractTimeoutGet() functions can return only SA_OK, SA_ERR_HPI_CAPABILITY, SA_ERR_HPI_INVALID_PARAMS and they need to return the Timeout. Further the infrastructure calls the set time out even when the function can return only SA_ERR_HPI_READ_ONLY. ---------------------------------------------------------------------- >Comment By: dr_mohan (dr_mohan) Date: 2011-12-22 16:09 Message: Please review the attached patch ---------------------------------------------------------------------- Comment By: dr_mohan (dr_mohan) Date: 2011-12-20 17:26 Message: It is plugin specific as well as infrastructure also. saHpiAutoInsertTimeoutSet needs to be implemented even if the plugin does not support it with SA_ERR_HPI_READ_ONLY as output. So if the set_autoinsert_timeout need not be called if ai_timeout is 0 and the timeout is read only, if it is called with ai_timeout of 0 then SA_ERR_HPI_READ_ONLY then SA_ERR_HPI_READ_ONLY is a valid return, not a critical error. ---------------------------------------------------------------------- Comment By: Anton Pak (avpak) Date: 2011-12-20 14:06 Message: Is it a plug-in specific ticket? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=532251&aid=3463092&group_id=71730 |