Re: [PATCH 3.6-rc6] cpufreq/powernow-k8: workqueue user shouldn'tmigrate the kworker to another CPU

From: Borislav Petkov
Date: Mon Sep 17 2012 - 17:22:00 EST


On Mon, Sep 17, 2012 at 01:53:55PM -0700, Tejun Heo wrote:
> On Mon, Sep 17, 2012 at 10:36:54PM +0200, Borislav Petkov wrote:
> > Wouldn't it be much simpler to carve out the piece after
> > set_cpus_allowed_ptr(), put it in a sub-function called
> > __powernowk8_target() and call it with smp_call_function_single instead
> > of defining another work item?
> >
> > Would the workqueue code handle that or are there any other issues?
>
> The function grabs a mutex. smp_call_function wouldn't be too happy
> about that.

Yes indeed.

Thanks.

--
Regards/Gruss,
Boris.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/