Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756937AbYBNQLI (ORCPT ); Thu, 14 Feb 2008 11:11:08 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752196AbYBNQKx (ORCPT ); Thu, 14 Feb 2008 11:10:53 -0500 Received: from accolon.hansenpartnership.com ([76.243.235.52]:52958 "EHLO accolon.hansenpartnership.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750807AbYBNQKw (ORCPT ); Thu, 14 Feb 2008 11:10:52 -0500 Subject: Re: [PATCH] gdth: bugfix for the at-exit problems From: James Bottomley To: Boaz Harrosh Cc: Sven =?ISO-8859-1?Q?K=F6hler?= , Christoph Hellwig , Jeff Garzik , linux-scsi , linux-kernel@vger.kernel.org, Joerg Dorchain , Jon Chelton , Stefan Priebe - allied internet ag In-Reply-To: <47B42CF4.2050008@panasas.com> References: <47A19E26.30107@panasas.com> <47B1D7A8.8010108@panasas.com> <47B1DA2A.1060904@panasas.com> <1202917468.3109.5.camel@localhost.localdomain> <47B312B3.3010200@panasas.com> <47B31BDE.2030408@panasas.com> <1202921122.3109.31.camel@localhost.localdomain> <47B31FC2.4040206@panasas.com> <1202922226.3109.36.camel@localhost.localdomain> <47B324E7.9040502@panasas.com> <1202924198.3109.52.camel@localhost.localdomain> <47B41CBD.8040701@panasas.com> <47B42CF4.2050008@panasas.com> Content-Type: text/plain Date: Thu, 14 Feb 2008 10:10:43 -0600 Message-Id: <1203005443.3189.23.camel@localhost.localdomain> Mime-Version: 1.0 X-Mailer: Evolution 2.12.3 (2.12.3-1.fc8) Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 842 Lines: 23 On Thu, 2008-02-14 at 13:58 +0200, Boaz Harrosh wrote: > This is a bugfix for the 2.6.24.x stable releases. > > gdth_exit would first remove all cards then stop the timer > and would not sync with the timer function. This caused a crash > in gdth_timer() when module was unloaded. > So del_timer_sync the timer before we delete the cards. > > also the reboot notifier function would crash. So unify > the exit and halt functions with a gdth_shutdown() that's > called by both. The patch looks fine now, thanks. Can we actually get a tester just to make sure there's nothing I missed. James -- 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/