From owner-freebsd-stable@FreeBSD.ORG Wed Oct 8 19:30:52 2014 Return-Path: Delivered-To: freebsd-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 2D6C5145 for ; Wed, 8 Oct 2014 19:30:52 +0000 (UTC) Received: from bigwig.baldwin.cx (bigwig.baldwin.cx [IPv6:2001:470:1f11:75::1]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 08FEAE76 for ; Wed, 8 Oct 2014 19:30:52 +0000 (UTC) Received: from ralph.baldwin.cx (pool-173-70-85-31.nwrknj.fios.verizon.net [173.70.85.31]) by bigwig.baldwin.cx (Postfix) with ESMTPSA id 09B57B99A for ; Wed, 8 Oct 2014 15:30:51 -0400 (EDT) From: John Baldwin To: freebsd-stable@freebsd.org Subject: [PATCH] Convert some timers in isp(4) from timeout(9) to callout(9) Date: Wed, 08 Oct 2014 15:02:23 -0400 Message-ID: <1927168.M9YyXzXvZZ@ralph.baldwin.cx> User-Agent: KMail/4.12.5 (FreeBSD/10.1-BETA2; KDE/4.12.5; amd64; ; ) MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.7 (bigwig.baldwin.cx); Wed, 08 Oct 2014 15:30:51 -0400 (EDT) X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 08 Oct 2014 19:30:52 -0000 This patch converts a few timers in isp(4) from timeout(9) to callout(9). It already used callout(9) for normal command timeouts. The patch is against HEAD but probably applies to 9 and 10 as well. http://people.freebsd.org/~jhb/patches/isp_callout.patch Note that I believe that this patch only affects the target mode of isp(4), so simply testing it as an HBA probably doesn't not exercise these code paths. I would still welcome testing as an HBA to ensure there are no regressions. If no one tests these I will not remove this driver. I may remove the target code if it is easily separable however. -- John Baldwin