[Cbe-oss-dev] [PATCH 8/10 v2] MARS: workload queue mutex protection
Kazunori Asayama
asayama at sm.sony.co.jp
Mon Sep 1 12:48:54 EST 2008
Geoff Levand wrote:
> On 08/29/2008 06:26 PM, Yuji Mano wrote:
>> - MARS_CHECK_RET(*id < MARS_WORKLOAD_MAX, MARS_ERROR_LIMIT);
>> + MARS_CHECK_CLEANUP_RET(queue->block[block].bits[index].state ==
>> + MARS_WORKLOAD_STATE_NONE,
>> + mars_mutex_unlock((struct mars_mutex *)&queue->block[block]),
>> + MARS_ERROR_STATE);
>
> As I said in another mail, these macros cannot be traced and will make
> it very hard to find bugs, and so should be removed.
Mano-san plans to eliminate those macros by upcoming another patch.
Please wait for that?
--
(ASAYAMA Kazunori
(asayama at sm.sony.co.jp))
t
More information about the cbe-oss-dev
mailing list