From owner-p4-projects@FreeBSD.ORG Tue May 16 01:37:58 2006 Return-Path: X-Original-To: p4-projects@freebsd.org Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 9EDF916AAB5; Tue, 16 May 2006 01:37:58 +0000 (UTC) X-Original-To: perforce@FreeBSD.org Delivered-To: perforce@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 6069016AAB3 for ; Tue, 16 May 2006 01:37:58 +0000 (UTC) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.FreeBSD.org (Postfix) with ESMTP id CA76E43D46 for ; Tue, 16 May 2006 01:37:55 +0000 (GMT) (envelope-from scottl@samsco.org) Received: from [192.168.254.14] (imini.samsco.home [192.168.254.14]) (authenticated bits=0) by pooker.samsco.org (8.13.4/8.13.4) with ESMTP id k4G1bmEX044694; Mon, 15 May 2006 19:37:53 -0600 (MDT) (envelope-from scottl@samsco.org) Message-ID: <44692CEC.1000108@samsco.org> Date: Mon, 15 May 2006 19:37:48 -0600 From: Scott Long User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.7) Gecko/20050416 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Ian Dowse References: <200605160231.aa63988@nowhere.iedowse.com> In-Reply-To: <200605160231.aa63988@nowhere.iedowse.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-1.4 required=3.8 tests=ALL_TRUSTED autolearn=failed version=3.1.1 X-Spam-Checker-Version: SpamAssassin 3.1.1 (2006-03-10) on pooker.samsco.org Cc: Perforce Change Reviews Subject: Re: PERFORCE change 97221 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 16 May 2006 01:37:59 -0000 Ian Dowse wrote: > In message <200605160001.k4G01hBX053502@repoman.freebsd.org>, Scott Long writes > : > >> Add callout storage in the CCB for MPSAFE drivers to use. > > > FYI, just in case it's of any use to you, there is a patch at > > http://people.freebsd.org/~iedowse/cam_xccb.diff > > that was an attempt from a while back to separate the userland and > kernel CCB structures to avoid breaking the ABI when making such > changes. I wasn't sure if this was a sensible way to go for CCBs > though, and the resulting code is a bit gross, so I didn't take it > any further. > > Ian What's going into P4 right now is just a 'get it done' effort. When it comes time to backport this to RELENG_6, I need to decide what kernel and userland APIs are worth breaking and what ones are not. However, 7.0 will represent a rather large API (and design) change for CAM, and trying to use compat shims will probably be more effort than it's worth. With that in mind, I don't like gratuitous userland changes either, so I'll keep your patch on the back-burner until it's time to decide what to break and what to keep. Scott