Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261802AbVEZVdu (ORCPT ); Thu, 26 May 2005 17:33:50 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261801AbVEZVdo (ORCPT ); Thu, 26 May 2005 17:33:44 -0400 Received: from cpu1185.adsl.bellglobal.com ([207.236.110.166]:9995 "EHLO mail.rtr.ca") by vger.kernel.org with ESMTP id S261800AbVEZVdX (ORCPT ); Thu, 26 May 2005 17:33:23 -0400 Message-ID: <42964099.6000207@pobox.com> Date: Thu, 26 May 2005 17:33:13 -0400 From: Mark Lord Organization: Real-Time Remedies Inc. User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.7) Gecko/20050420 Debian/1.7.7-2 X-Accept-Language: en, en-us MIME-Version: 1.0 To: Jeff Garzik Cc: "linux-ide@vger.kernel.org" , Linux Kernel , Jens Axboe , Al Viro Subject: Re: [PATCH] libata: fix use-after-free during driver unload/unplug References: <42962379.5000206@pobox.com> In-Reply-To: <42962379.5000206@pobox.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 937 Lines: 28 Hi Jeff! Jeff Garzik wrote: > > * add ->host_stop_prewalk() hook, use it in sata_qstor.c (hi Mark). > sata_qstor appears to require the host-stop-before-port-stop ordering > that existed prior to applying the attached patch. Mmm.. I'm a little bit rusty here, but I don't think qstor cares about the order, so long as ports are marked with ATA_FLAG_PORT_DISABLED before invoking port_stop(). I've tried to allow disabling/enabling individual ports on-the-fly as needed, even though it never really happens in practice. So host_stop() kills the whole chip, whereas port_stop() I took to mean just one of the four SATA ports. Cheers -- Mark Lord Real-Time Remedies Inc. mlord@pobox.com - 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/