From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e23smtp09.au.ibm.com (e23smtp09.au.ibm.com [202.81.31.142]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 7281A1A0B96 for ; Thu, 12 Feb 2015 22:13:56 +1100 (AEDT) Received: from /spool/local by e23smtp09.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 12 Feb 2015 21:13:54 +1000 Received: from d23relay07.au.ibm.com (d23relay07.au.ibm.com [9.190.26.37]) by d23dlp01.au.ibm.com (Postfix) with ESMTP id 7A4122CE8050 for ; Thu, 12 Feb 2015 22:13:50 +1100 (EST) Received: from d23av01.au.ibm.com (d23av01.au.ibm.com [9.190.234.96]) by d23relay07.au.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id t1CBDfqS42401834 for ; Thu, 12 Feb 2015 22:13:50 +1100 Received: from d23av01.au.ibm.com (localhost [127.0.0.1]) by d23av01.au.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id t1CBDGKY026391 for ; Thu, 12 Feb 2015 22:13:16 +1100 Message-ID: <54DC8A39.3070401@linux.vnet.ibm.com> Date: Thu, 12 Feb 2015 16:40:49 +0530 From: Neelesh Gupta MIME-Version: 1.0 To: Anshuman Khandual , mpe@ellerman.id.au, linuxppc-dev@lists.ozlabs.org, benh@kernel.crashing.org Subject: Re: [PATCH 2/2] opal: Add message notifier unregister function References: <20150211062616.3279.71632.stgit@localhost.localdomain> <20150211062712.3279.93469.stgit@localhost.localdomain> <54DB35A7.8030609@linux.vnet.ibm.com> In-Reply-To: <54DB35A7.8030609@linux.vnet.ibm.com> Content-Type: text/plain; charset=UTF-8; format=flowed List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 02/11/2015 04:27 PM, Anshuman Khandual wrote: > On 02/11/2015 11:57 AM, Neelesh Gupta wrote: >> Provide an unregister interface for the opal message notifiers >> to be called when not needed like during driver unload/remove. > Why only for unload/remove, you can also use it in cases where you need > to abort because of any other error soon after successful registration > for an OPAL message. I didn't mean *only* driver unload/remove.. gave a simple use case of why do we need this change. Neelesh. >