From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx0a-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3rp8jp21j0zDqwf for ; Tue, 12 Jul 2016 01:52:14 +1000 (AEST) Received: from pps.filterd (m0098419.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.16.0.11/8.16.0.11) with SMTP id u6BFmkMi095884 for ; Mon, 11 Jul 2016 11:52:11 -0400 Received: from e34.co.us.ibm.com (e34.co.us.ibm.com [32.97.110.152]) by mx0b-001b2d01.pphosted.com with ESMTP id 243edcrd9e-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Mon, 11 Jul 2016 11:52:11 -0400 Received: from localhost by e34.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 11 Jul 2016 09:52:10 -0600 Subject: Re: [PATCH 3/3] powerpc/pseries: Use kernel hotplug queue for PowerVM hotplug events To: John Allen , Michael Ellerman , linuxppc-dev@lists.ozlabs.org References: <4cf3edf2-fc24-5989-ae5d-b451a2323cf2@linux.vnet.ibm.com> <4467d582-954a-23ca-af5e-75ddc1c99cee@linux.vnet.ibm.com> Cc: Michael Roth From: Nathan Fontenot Date: Mon, 11 Jul 2016 10:52:05 -0500 MIME-Version: 1.0 In-Reply-To: <4467d582-954a-23ca-af5e-75ddc1c99cee@linux.vnet.ibm.com> Content-Type: text/plain; charset=utf-8 Message-Id: <5783C0A5.102@linux.vnet.ibm.com> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 07/07/2016 10:05 AM, John Allen wrote: > The sysfs interface used to handle PowerVM hotplug events should use the > hotplug queue as well. PRRN events will soon be placing many hotplug > events on the queue at once and we will need ordinary hotplug events to > use the queue as well in order to ensure these events will still be handled > and that proper serialization is maintained during the PRRN event. > > Signed-off-by: John Allen Reviewed-by: Nathan Fontenot > --- > arch/powerpc/platforms/pseries/dlpar.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/arch/powerpc/platforms/pseries/dlpar.c b/arch/powerpc/platforms/pseries/dlpar.c > index 66a77d7..4748124 100644 > --- a/arch/powerpc/platforms/pseries/dlpar.c > +++ b/arch/powerpc/platforms/pseries/dlpar.c > @@ -489,7 +489,9 @@ static ssize_t dlpar_store(struct class *class, struct class_attribute *attr, > goto dlpar_store_out; > } > > - rc = handle_dlpar_errorlog(hp_elog); > + init_completion(&hotplug_done); > + queue_hotplug_event(hp_elog, &hotplug_done, &rc); > + wait_for_completion(&hotplug_done); > > dlpar_store_out: > kfree(hp_elog); > > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/linuxppc-dev >