> Why not just do > > if (cpu == WORK_CPU_UNBOUND) > cpu = raw_smp_processor_id(); > > if (!(wq->flags...) { > ... > if (gcwq && gcwq->cpu != WORK_CPU_UNBOUND) > lcpu = gcwq->cpu; > else > lcpu = cpu; > } > ... > > add_timer_on(timer, cpu);
I re-look at code deeply and find that dwork->cpu is related to tracing information. If we do above change, tracing information is bias. > Also, can you please base the patches on top of the following git > branch? > > git://git.kernel.org/pub/scm/linux/kernel/git/tj/wq.git for-3.7 Sure! -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/