From owner-freebsd-current@FreeBSD.ORG Wed Dec 5 04:26:05 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 9E46261C for ; Wed, 5 Dec 2012 04:26:05 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from mail-pa0-f54.google.com (mail-pa0-f54.google.com [209.85.220.54]) by mx1.freebsd.org (Postfix) with ESMTP id 63DAA8FC14 for ; Wed, 5 Dec 2012 04:26:04 +0000 (UTC) Received: by mail-pa0-f54.google.com with SMTP id bi5so3293215pad.13 for ; Tue, 04 Dec 2012 20:26:04 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=sender:subject:mime-version:content-type:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to:x-mailer :x-gm-message-state; bh=zSclfivWd6DsOjYRXlsDATpcr+WmsU7Ss3TG1lwrWy8=; b=iBOcvpaUVOBg/Q87R9fXejjq9Sxk3gjqbxspdxi60Q0UwbeGbHKW05Y6gvRujoqc42 d98dpSvrCdQEXOIbQB0/KWOqd8UBRyI7V/cSuRPYHs5fq7esy9cXzWbzyJnBWwoL8yR9 6DKrcgYmMIwt/qxSNNIYt4OTgRlCHd6eKo/6cgUwDmzNnn4Klf1CeTL8uYjCLXIixYOo PRP0fzX8hOLWSM9BhQARPMBx6op2x4kDWY2baOdVUAtPITDaZZuXsOqT59z5S3tjbV3+ IOICJUkkhIJxhQu7I9TkFxsnLxdfHRHqtx1rusNG2a3OJ8ugymEmS7grpvMyTCG+MTul lYJQ== Received: by 10.68.143.162 with SMTP id sf2mr44746650pbb.137.1354681564404; Tue, 04 Dec 2012 20:26:04 -0800 (PST) Received: from [10.0.0.53] (50-78-194-198-static.hfc.comcastbusiness.net. [50.78.194.198]) by mx.google.com with ESMTPS id bb2sm1951514pab.34.2012.12.04.20.26.01 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 04 Dec 2012 20:26:03 -0800 (PST) Sender: Warner Losh Subject: Re: Call for testers, users with scsi cards Mime-Version: 1.0 (Apple Message framework v1085) Content-Type: text/plain; charset=us-ascii From: Warner Losh In-Reply-To: Date: Tue, 4 Dec 2012 21:25:56 -0700 Content-Transfer-Encoding: quoted-printable Message-Id: <54F59F93-91A1-44B6-93F6-AB94F3BF2CC2@bsdimp.com> References: <1354658563.87661.23.camel@revolution.hippie.lan> To: Jeff Roberson X-Mailer: Apple Mail (2.1085) X-Gm-Message-State: ALoCoQmbbgJzzQkNhSvtnQCYAC63xe1J/ja71AbpYoweP/iiGmmLllF5Ox4HL4yNNjGfFDBfkiXI Cc: Ian Lepore , 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: Wed, 05 Dec 2012 04:26:05 -0000 On Dec 4, 2012, at 5:58 PM, Jeff Roberson wrote: > On Tue, 4 Dec 2012, Ian Lepore wrote: >=20 >> On Tue, 2012-12-04 at 14:49 -0700, Warner Losh wrote: >>> On Dec 4, 2012, at 2:36 PM, Jeff Roberson wrote: >>>=20 >>>> http://people.freebsd.org/~jeff/loadccb.diff >>>>=20 >>>> 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. >>>=20 >>> 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... >>>=20 >>> Warner >>=20 >> Hmmm, if it's only used by cam, why isn't it in cam/ rather than = kern/ ? >=20 > kib pointed out drivers that use ccbs but do not depend on cam. I = also intend to consolidate many of the busdma_load_* functions into this = subr_busdma.c eventually. I will add a load_bio and things like = load_uio and load_mbuf don't need to be re-implemented for every = machine. I will define a MD function that allows you to add virtual or = physical segments piecemeal (as they all currently have) so that = function may be called for each member in the uio, mbuf, ccb, or bio. Sounds like a good reason to me... Look forward to it... Warner > Thanks, > Jeff >=20 >>=20 >> -- Ian >>=20