Mohan,
Good patch. There is one minor problem in this debug printf below after
call to saHpiRdrGet(sessionid,resourceid, entryid,&nextentryid, &rdr);
It tries to print rv instead of rv_rdr. rv is not set at this point.
Should this be rv_rdr?
- while ((rv == SA_OK) && (entryid != SAHPI_LAST_ENTRY))
+ rv_rdr = SA_OK;
+ while ((rv_rdr == SA_OK) && (entryid != SAHPI_LAST_ENTRY))
{
- rv = saHpiRdrGet(sessionid,resourceid, entryid,&nextentryid,
&rdr);
+ rv_rdr = saHpiRdrGet(sessionid,resourceid,
entryid,&nextentryid, &rdr);
if (fdebug) printf("saHpiRdrGet[%x] rv = %d\n",entryid,rv);
Thanks,
-- Shuah
On Wed, 2009-08-12 at 17:41 +0000, mo...@fc wrote:
> Hello All,
>
> Patch for the bug 2800731 (hpiinv stops if saHpiIdrAreaHeaderGet errors)
> is attached for review. This patch basically makes sure hpiinv continues
> to go through the next RPT entry even if an area of an RDR returns an
> error. Please review this patch. I will wait till Aug 21 to receive
> feedback.
>
> Thanks
> Mohan
>
>
>
--
------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
trial. Simplify your report design, integration and deployment - and focus on
what you do best, core application coding. Discover what's new with
Crystal Reports now. http://p.sf.net/sfu/bobj-july
_______________________________________________
Openhpi-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openhpi-devel
