Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S267353AbUITUot (ORCPT ); Mon, 20 Sep 2004 16:44:49 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S267346AbUITUot (ORCPT ); Mon, 20 Sep 2004 16:44:49 -0400 Received: from fmr04.intel.com ([143.183.121.6]:38794 "EHLO caduceus.sc.intel.com") by vger.kernel.org with ESMTP id S267345AbUITUoo (ORCPT ); Mon, 20 Sep 2004 16:44:44 -0400 Date: Mon, 20 Sep 2004 13:44:32 -0700 From: Keshavamurthy Anil S To: Bjorn Helgaas Cc: Keshavamurthy Anil S , Dmitry Torokhov , acpi-devel@lists.sourceforge.net, "Brown, Len" , LHNS list , Linux IA64 , Linux Kernel Subject: Re: [ACPI] PATCH-ACPI based CPU hotplug[1/6]-ACPI core enhancement support Message-ID: <20040920134432.A16611@unix-os.sc.intel.com> Reply-To: Keshavamurthy Anil S References: <20040920092520.A14208@unix-os.sc.intel.com> <200409201326.44946.dtor_core@ameritech.net> <20040920120128.A15677@unix-os.sc.intel.com> <200409201426.31672.bjorn.helgaas@hp.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <200409201426.31672.bjorn.helgaas@hp.com>; from bjorn.helgaas@hp.com on Mon, Sep 20, 2004 at 02:26:31PM -0600 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1477 Lines: 29 On Mon, Sep 20, 2004 at 02:26:31PM -0600, Bjorn Helgaas wrote: > On Monday 20 September 2004 1:01 pm, Keshavamurthy Anil S wrote: > > On Mon, Sep 20, 2004 at 01:26:44PM -0500, Dmitry Torokhov wrote: > > > Also, introducing recursion (depth does not seem to be limited here) is > > > not a good idea IMHO - better convert it into iteration to avoid stack > > > problems down teh road. > > Humm, I guess recursion should be fine and even though the code does not have > > an explicit limit, the ACPI namespace describing the Ejectable device will limit the > > number of recursible devices. And I believe this won;t be more than 3 to 4 level depth. > > Hence recursion is fine here. > > > > If you still strongly believe that recursion is not the right choice here, > > let me know and I will convert it to iteration. > > I'm also in favor of removing the recursion, if only because it > allows local analysis. I.e., a correctness argument based solely > on the code in the patch is much more useful than one that relies > on properties of an external and mostly unknown ACPI namespace. Okay, I am convinced, I will post an updated patch to address this soon. Again thanks for the feedback and more comments are welcome:) -Anil - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/