From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Olsa Subject: Re: [PATCH] ACPI/Processor: Rework processor throttling with work_on_cpu() Date: Fri, 21 Feb 2014 18:07:10 +0100 Message-ID: <20140221170709.GC1121@krava.brq.redhat.com> References: <20140220151721.GB32719@redhat.com> <1392960945-30741-1-git-send-email-tianyu.lan@intel.com> <20140221100630.GA1121@krava.brq.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mx1.redhat.com ([209.132.183.28]:26624 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755947AbaBURHR (ORCPT ); Fri, 21 Feb 2014 12:07:17 -0500 Content-Disposition: inline In-Reply-To: <20140221100630.GA1121@krava.brq.redhat.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Lan Tianyu Cc: tj@kernel.org, oleg@redhat.com, lenb@kernel.org, rjw@rjwysocki.net, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org On Fri, Feb 21, 2014 at 11:06:30AM +0100, Jiri Olsa wrote: > On Fri, Feb 21, 2014 at 01:35:45PM +0800, Lan Tianyu wrote: > > acpi_processor_set_throttling() uses set_cpus_allowed_ptr() to make > > sure struct acpi_processor->acpi_processor_set_throttling() callback > > run on associated cpu. But the function maybe called in a worker which > > has been bound to a cpu. The patch is to replace set_cpus_allowed_ptr() > > with work_on_cpu(). > > testing the new patch.. so far so good ;-) ook.. survived whole day under the test workload without the warning ;-) jirka