Shouldn't this wait until the coremask patch is in and then be based on it, using the new cpumask names?
On Mon, Jan 12, 2015 at 3:43 PM, Mike Holmes <[email protected]> wrote: > Allow the complier to check for NULL args. > > Signed-off-by: Mike Holmes <[email protected]> > --- > platform/linux-generic/include/api/odp_coremask.h | 18 +++++++++++------- > 1 file changed, 11 insertions(+), 7 deletions(-) > > diff --git a/platform/linux-generic/include/api/odp_coremask.h > b/platform/linux-generic/include/api/odp_coremask.h > index c9331fd..b52f977 100644 > --- a/platform/linux-generic/include/api/odp_coremask.h > +++ b/platform/linux-generic/include/api/odp_coremask.h > @@ -21,6 +21,7 @@ extern "C" { > > > #include <odp_std_types.h> > +#include <odp_hints.h> > > /** @addtogroup odp_scheduler > * Core mask operations. > @@ -51,7 +52,8 @@ typedef struct odp_coremask_t { > * > * @note Supports currently only core indexes upto 63 > */ > -void odp_coremask_from_str(const char *str, odp_coremask_t *mask); > +void odp_coremask_from_str(const char *str, odp_coremask_t *mask) > + ODP_NONNULL(1, 2); > > /** > * Write core mask as a string of hexadecimal digits > @@ -64,7 +66,8 @@ void odp_coremask_from_str(const char *str, > odp_coremask_t *mask); > * > * @note Supports currently only core indexes upto 63 > */ > -void odp_coremask_to_str(char *str, int len, const odp_coremask_t *mask); > +void odp_coremask_to_str(char *str, int len, const odp_coremask_t *mask) > + ODP_NONNULL(1, 3); > > > /** > @@ -87,7 +90,8 @@ void odp_coremask_to_str(char *str, int len, const > odp_coremask_t *mask); > * > * @note Supports currently only core indexes upto 63 > */ > -void odp_coremask_from_u64(const uint64_t *u64, int num, odp_coremask_t > *mask); > +void odp_coremask_from_u64(const uint64_t *u64, int num, odp_coremask_t > *mask) > + ODP_NONNULL(1, 3); > > /** > * Clear entire mask > @@ -103,14 +107,14 @@ static inline void odp_coremask_zero(odp_coremask_t > *mask) > * @param core Core number > * @param mask add core number in core mask > */ > -void odp_coremask_set(int core, odp_coremask_t *mask); > +void odp_coremask_set(int core, odp_coremask_t *mask) ODP_NONNULL(2); > > /** > * Remove core from mask > * @param core Core number > * @param mask clear core number from core mask > */ > -void odp_coremask_clr(int core, odp_coremask_t *mask); > +void odp_coremask_clr(int core, odp_coremask_t *mask) ODP_NONNULL(2); > > /** > * Test if core is a member of mask > @@ -118,14 +122,14 @@ void odp_coremask_clr(int core, odp_coremask_t > *mask); > * @param mask Core mask to check if core num set or not > * @return non-zero if set otherwise 0 > */ > -int odp_coremask_isset(int core, const odp_coremask_t *mask); > +int odp_coremask_isset(int core, const odp_coremask_t *mask) > ODP_NONNULL(2); > > /** > * Count number of cores in mask > * @param mask Core mask > * @return coremask count > */ > -int odp_coremask_count(const odp_coremask_t *mask); > +int odp_coremask_count(const odp_coremask_t *mask) ODP_NONNULL(1); > > > > -- > 2.1.0 > > > _______________________________________________ > lng-odp mailing list > [email protected] > http://lists.linaro.org/mailman/listinfo/lng-odp >
_______________________________________________ lng-odp mailing list [email protected] http://lists.linaro.org/mailman/listinfo/lng-odp
