Description Karl Wechsler 2010-05-16 14:08:12 EDT

I think we could do this in compatible way and should do this before 3.20 ...

The 'WaitStatus' enum is inconsistent with the return type of the wait function (Int).

I think that this:

    /*!
     *  ======== WaitStatus ========
     *  Error codes returned by Semaphore_pend
     */
    enum WaitStatus {
        WaitStatus_ERROR = -1,
        WaitStatus_TIMEOUT = 0,
        WaitStatus_SUCCESS = 1
    };

should be changed to this:

const Int ERROR = -1;
const Int TIMEOUT = 0;
const Int SUCCESS = 1;

I think this change would be compatible.  Note that we have a mis-mosh (sp?) of return codes in assorted implementations of ISync in the BIOS tree.   Some are returning direct value from Semaphore_pend (true/false).  Others are returning (0) and (1) (instead of the enums above).  It is all working out because success is (1) which matches TRUE but we should clean this up in ISync and the SyncSemaphore, etc. in xdc.runtime.knl.  And then clean it up in the BIOS code.

I think we could do this in compatible way and should do this before 3.20.