From mboxrd@z Thu Jan 1 00:00:00 1970 From: boris.ostrovsky@oracle.com (Boris Ostrovsky) Date: Mon, 19 Oct 2015 14:24:03 -0400 Subject: [PATCH v3 2/3] xen, cpu_hotplug: call device_offline instead of cpu_down In-Reply-To: <1445273735-15758-2-git-send-email-stefano.stabellini@eu.citrix.com> References: <1445273735-15758-2-git-send-email-stefano.stabellini@eu.citrix.com> Message-ID: <56253543.8080007@oracle.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 10/19/2015 12:55 PM, Stefano Stabellini wrote: > When offlining a cpu, instead of cpu_down, call device_offline, which > also takes care of updating the cpu.dev.offline field. This keeps the > sysfs file /sys/devices/system/cpu/cpuN/online, up to date. Also move > the call to disable_hotplug_cpu, because it makes more sense to have it > there. > > Signed-off-by: Stefano Stabellini > CC: konrad.wilk at oracle.com > CC: boris.ostrovsky at oracle.com > CC: david.vrabel at citrix.com > --- > drivers/xen/cpu_hotplug.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/xen/cpu_hotplug.c b/drivers/xen/cpu_hotplug.c > index 43de1f5..78804b5 100644 > --- a/drivers/xen/cpu_hotplug.c > +++ b/drivers/xen/cpu_hotplug.c > @@ -18,6 +18,8 @@ static void enable_hotplug_cpu(int cpu) > > static void disable_hotplug_cpu(int cpu) > { > + if (cpu_online(cpu)) > + device_offline(get_cpu_device(cpu)); > if (cpu_present(cpu)) > xen_arch_unregister_cpu(cpu); Do we want to call device_online() when plugging cpu in? And take sysfs lock? -boris > > @@ -55,7 +57,6 @@ static void vcpu_hotplug(unsigned int cpu) > enable_hotplug_cpu(cpu); > break; > case 0: > - (void)cpu_down(cpu); > disable_hotplug_cpu(cpu); > break; > default: From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Ostrovsky Subject: Re: [PATCH v3 2/3] xen, cpu_hotplug: call device_offline instead of cpu_down Date: Mon, 19 Oct 2015 14:24:03 -0400 Message-ID: <56253543.8080007@oracle.com> References: <1445273735-15758-2-git-send-email-stefano.stabellini@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1445273735-15758-2-git-send-email-stefano.stabellini@eu.citrix.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Stefano Stabellini , xen-devel@lists.xensource.com Cc: david.vrabel@citrix.com, linux-arm-kernel@lists.infradead.org, konrad.wilk@oracle.com List-Id: xen-devel@lists.xenproject.org On 10/19/2015 12:55 PM, Stefano Stabellini wrote: > When offlining a cpu, instead of cpu_down, call device_offline, which > also takes care of updating the cpu.dev.offline field. This keeps the > sysfs file /sys/devices/system/cpu/cpuN/online, up to date. Also move > the call to disable_hotplug_cpu, because it makes more sense to have it > there. > > Signed-off-by: Stefano Stabellini > CC: konrad.wilk@oracle.com > CC: boris.ostrovsky@oracle.com > CC: david.vrabel@citrix.com > --- > drivers/xen/cpu_hotplug.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/xen/cpu_hotplug.c b/drivers/xen/cpu_hotplug.c > index 43de1f5..78804b5 100644 > --- a/drivers/xen/cpu_hotplug.c > +++ b/drivers/xen/cpu_hotplug.c > @@ -18,6 +18,8 @@ static void enable_hotplug_cpu(int cpu) > > static void disable_hotplug_cpu(int cpu) > { > + if (cpu_online(cpu)) > + device_offline(get_cpu_device(cpu)); > if (cpu_present(cpu)) > xen_arch_unregister_cpu(cpu); Do we want to call device_online() when plugging cpu in? And take sysfs lock? -boris > > @@ -55,7 +57,6 @@ static void vcpu_hotplug(unsigned int cpu) > enable_hotplug_cpu(cpu); > break; > case 0: > - (void)cpu_down(cpu); > disable_hotplug_cpu(cpu); > break; > default: