From owner-freebsd-current@FreeBSD.ORG Tue Dec 4 22:03:07 2012 Return-Path: Delivered-To: current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id A383C75F for ; Tue, 4 Dec 2012 22:03:07 +0000 (UTC) (envelope-from freebsd@damnhippie.dyndns.org) Received: from duck.symmetricom.us (duck.symmetricom.us [206.168.13.214]) by mx1.freebsd.org (Postfix) with ESMTP id 274818FC13 for ; Tue, 4 Dec 2012 22:02:54 +0000 (UTC) Received: from damnhippie.dyndns.org (daffy.symmetricom.us [206.168.13.218]) by duck.symmetricom.us (8.14.5/8.14.5) with ESMTP id qB4M2jiE017375 for ; Tue, 4 Dec 2012 15:02:46 -0700 (MST) (envelope-from freebsd@damnhippie.dyndns.org) Received: from [172.22.42.240] (revolution.hippie.lan [172.22.42.240]) by damnhippie.dyndns.org (8.14.3/8.14.3) with ESMTP id qB4M2hON047904; Tue, 4 Dec 2012 15:02:43 -0700 (MST) (envelope-from freebsd@damnhippie.dyndns.org) Subject: Re: Call for testers, users with scsi cards From: Ian Lepore To: Warner Losh In-Reply-To: References: Content-Type: text/plain; charset="us-ascii" Date: Tue, 04 Dec 2012 15:02:43 -0700 Message-ID: <1354658563.87661.23.camel@revolution.hippie.lan> Mime-Version: 1.0 X-Mailer: Evolution 2.32.1 FreeBSD GNOME Team Port Content-Transfer-Encoding: 7bit Cc: Jeff Roberson , current@freebsd.org X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Dec 2012 22:03:07 -0000 On Tue, 2012-12-04 at 14:49 -0700, Warner Losh wrote: > On Dec 4, 2012, at 2:36 PM, Jeff Roberson wrote: > > > http://people.freebsd.org/~jeff/loadccb.diff > > > > This patch consolidates all of the functions that map cam control blocks for DMA into one central function. This change is a precursor to adding new features to the I/O stack. It is mostly mechanical. If you are running current on a raid or scsi card, especially if it is a lesser used one, I would really like you to apply this patch and report back any problems. If it works you should notice nothing. If it doesn't work you will probably panic immediately on I/O or otherwise no I/O will happen. > > I haven't tested it yet. My only comment from reading it though would be to make subr_busdma.c be dependent on cam, since it can only used from cam. We've grown sloppy about noting these dependencies in the tree... > > Warner Hmmm, if it's only used by cam, why isn't it in cam/ rather than kern/ ? -- Ian