On 4 August 2016 at 11:26, Brian Brooks <brian.bro...@linaro.org> wrote:
> Reviewed-by: Brian Brooks <brian.bro...@linaro.org> > Just wanted to follow up on this, this patch has also highlighted a bug in the original code and the thread below documents it, also a bug for it has been created https://bugs.linaro.org/show_bug.cgi?id=2457 However this patch is orthogonal to that problem which existed before this patch was created so I think we need to take it so that TgrM testing can continue. > > On 08/04 09:18:14, Mike Holmes wrote: > > +ret=0 > > + > > +run() > > +{ > > + echo odp_scheduling_run_proc starts with $1 worker threads > > + echo ===================================================== > > + > > + $PERFORMANCE/odp_scheduling${EXEEXT} --odph_proc -c $1 || ret=1 > > +} > > + > > +run 1 > > +run 8 > > + > > +exit $ret > > Seeing this randomly in both multithread and multiprocess modes: > > ../../../odp/platform/linux-generic/odp_queue.c:328:odp_queue_destroy():queue > "sched_00_07" not empty > ../../../odp/platform/linux-generic/odp_schedule.c:271:schedule_term_global():Queue > not empty > ../../../odp/platform/linux-generic/odp_schedule.c:294:schedule_term_global():Pool > destroy fail. > ../../../odp/platform/linux-generic/odp_init.c:188:_odp_term_global():ODP > schedule term failed. > ../../../odp/platform/linux-generic/odp_queue.c:170:odp_queue_term_global():Not > destroyed queue: sched_00_07 > ../../../odp/platform/linux-generic/odp_init.c:195:_odp_term_global():ODP > queue term failed. > ../../../odp/platform/linux-generic/odp_pool.c:149:odp_pool_term_global():Not > destroyed pool: odp_sched_pool > ../../../odp/platform/linux-generic/odp_pool.c:149:odp_pool_term_global():Not > destroyed pool: msg_pool > ../../../odp/platform/linux-generic/odp_init.c:202:_odp_term_global():ODP > buffer pool term failed. > ~/odp_incoming/odp_build/test/common_plat/performance$ echo $? > 0 > > Potentially two items: one for correctly returning the failure code, and > another related to teardown. Both beyond the scope of this patch which > LGTM. > -- Mike Holmes Technical Manager - Linaro Networking Group Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs "Work should be fun and collaborative, the rest follows"