Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762252AbXFARH5 (ORCPT ); Fri, 1 Jun 2007 13:07:57 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1760879AbXFARHu (ORCPT ); Fri, 1 Jun 2007 13:07:50 -0400 Received: from wa-out-1112.google.com ([209.85.146.179]:63869 "EHLO wa-out-1112.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760833AbXFARHt (ORCPT ); Fri, 1 Jun 2007 13:07:49 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=R7lN2/UsBXbI9k2wlbjptTPnzN5haZV8tJ6HE6SIewU/t2MsPZcJkwHr38abBReBh6keviJtyl3U1S53Mrn2fCRCrcaN6WuY/4g8ME0Q1E3D1FsyKY0ecxEwEyY6bjGRrNovjMpur0sPl/tWPKSeZ6gPreRs2AKT5QvYlzC90eQ= Message-ID: <86802c440706011007r7c9162c3w1fe0ba1952361f19@mail.gmail.com> Date: Fri, 1 Jun 2007 10:07:48 -0700 From: "Yinghai Lu" To: vgoyal@in.ibm.com Subject: Re: [PATCH] aacraid: fix shutdown handler to also disable interrupts. Cc: "Salyzyn, Mark" , linux-scsi@vger.kernel.org, "Andrew Morton" , "Eric W. Biederman" , "Michal Piotrowski" , "Linux Kernel Mailing List" In-Reply-To: <20070601110845.GC20714@in.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20070601110845.GC20714@in.ibm.com> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1669 Lines: 36 On 6/1/07, Vivek Goyal wrote: > On Wed, May 30, 2007 at 11:59:13AM -0400, Salyzyn, Mark wrote: > > Moves quiesce, thread and interrupt shutdown into aacraid drivers' > > .shutdown handler. This fix to the aac_shutdown handler will remove the > > superfluous reset of the adapter during a (clean) kexec. > > > > This fix may mitigate the active investigation 'kexec and aacraid > > broken' but it is unlikely to affect the root cause (issue likely > > present in both kexec and kdump). This patch reduces the chance the > > problem will occur with a kexec. The fix for root cause is currently > > expected to be the minimum value check to the aacraid.startup_timeout > > driver variable after an adapter reset within aacraid_commit_reset.patch > > submitted on 05/22/2007 and awaiting testing by Yinghai to confirm. > > > > This attached patch is against current scsi-misc-2.6 > > > > ObligatoryDisclaimer: Please accept my condolences regarding Outlook's > > handling of patch attachments. > > > > Signed-off-by: Mark Salyzyn > > > > Thanks Mark. This does fix the issue of unnecessary reset of aacraid > adapter over kexec on my machine. > i'm little confused about that. this patch is some clear shutdown, so even next start will have tight condition will not try to reset the adapter fw. right Mark? Maybe the driver could be smart to find out if it need to reset adaptec fw. YH - 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/