From owner-freebsd-scsi@FreeBSD.ORG Sun Oct 24 16:53:10 2010 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 1686D106566C; Sun, 24 Oct 2010 16:53:10 +0000 (UTC) (envelope-from bruce@cran.org.uk) Received: from muon.cran.org.uk (muon.cran.org.uk [IPv6:2a01:348:0:15:5d59:5c40:0:1]) by mx1.freebsd.org (Postfix) with ESMTP id C8D938FC19; Sun, 24 Oct 2010 16:53:09 +0000 (UTC) Received: from muon.cran.org.uk (localhost [127.0.0.1]) by muon.cran.org.uk (Postfix) with ESMTP id 328F5E7F6E; Sun, 24 Oct 2010 17:53:09 +0100 (BST) Received: from unknown (client-81-107-142-135.midd.adsl.virginmedia.com [81.107.142.135]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by muon.cran.org.uk (Postfix) with ESMTPSA; Sun, 24 Oct 2010 17:53:08 +0100 (BST) Date: Sun, 24 Oct 2010 17:53:06 +0100 From: Bruce Cran To: Alexander Motin Message-ID: <20101024175306.00003c66@unknown> In-Reply-To: <4CC4633D.4070605@FreeBSD.org> References: <201009161742.24228.tijl@coosemans.org> <201009161619.o8GGJAmv035378@lurza.secnetix.de> <20101018155944.GA12425@freebsd.org> <868w1r92rf.fsf@ds4.des.no> <20101021122110.GA65490@freebsd.org> <86zku77mj6.fsf@ds4.des.no> <20101021130730.GA72290@freebsd.org> <86r5fj7gin.fsf@ds4.des.no> <20101024011854.GB78293@freebsd.org> <4CC4633D.4070605@FreeBSD.org> X-Mailer: Claws Mail 3.7.6 (GTK+ 2.16.6; i586-pc-mingw32msvc) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Alexander Best , freebsd-scsi@freebsd.org, Oliver Fromme , freebsd-hackers@freebsd.org Subject: Re: Summary: Re: Spin down HDD after disk sync or before power off X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 24 Oct 2010 16:53:10 -0000 On Sun, 24 Oct 2010 19:47:57 +0300 Alexander Motin wrote: > Comparing two ways implementing spindown, I've recalled that both of > them using xpt_polled_action() method, which depends on working > controller polling operation. So they could be almost equaly not good. > But the method present in HEAD now is more universal. Looking on fact > that need of spindown is not so obvious for SCSI devices (in SAN > environments), we can just make kern.cam.power_down tunable a bitmask > of supported protocols for now. Patch is attached. I've just committed the patch to move the functionality into ada(4). Should it be reverted? -- Bruce Cran