From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 6628BB6F80 for ; Tue, 30 Aug 2011 16:20:33 +1000 (EST) Subject: Re: [PATCH] PSeries: Cancel RTAS event scan before firmware flash From: Benjamin Herrenschmidt To: Suzuki Poulose In-Reply-To: <4E5C806C.1070809@in.ibm.com> References: <20110727120801.10429.7276.stgit@localhost6.localdomain6> <1314684237.2488.77.camel@pasglop> <4E5C806C.1070809@in.ibm.com> Content-Type: text/plain; charset="UTF-8" Date: Tue, 30 Aug 2011 16:19:58 +1000 Message-ID: <1314685198.2488.83.camel@pasglop> Mime-Version: 1.0 Cc: mikey@neuling.org, "Ravi K. Nittala" , sbest@us.ibm.com, antonb@au1.ibm.com, subrata.modak@in.ibm.com, ranittal@linux.vnet.ibm.com, linuxppc-dev@lists.ozlabs.org, divya.vikas@in.ibm.com List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, 2011-08-30 at 11:47 +0530, Suzuki Poulose wrote: > > > > The flash operation is performed in the reboot path at the very end. > So, even if we restart the event scan, the thread may not be able to > process > the events. Hence we thought we would leave it stopped. > > Again, we do not have much expertise in deciding which is the best > thing to do. > We could resume the event scan, if you think that is needed. > > Thanks for the review. No that's ok, I'll merge the patch as-is then. Cheers, Ben.