From owner-freebsd-current@FreeBSD.ORG Thu Mar 12 02:16:34 2015 Return-Path: Delivered-To: current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 147F32A4; Thu, 12 Mar 2015 02:16:34 +0000 (UTC) Received: from mithlond.kdm.org (mithlond.kdm.org [70.56.43.85]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "A1-33714", Issuer "A1-33714" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id AD17A1C0; Thu, 12 Mar 2015 02:16:33 +0000 (UTC) Received: from mithlond.kdm.org (localhost [127.0.0.1]) by mithlond.kdm.org (8.14.9/8.14.9) with ESMTP id t2C278HE072013 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Wed, 11 Mar 2015 20:07:08 -0600 (MDT) (envelope-from ken@mithlond.kdm.org) Received: (from ken@localhost) by mithlond.kdm.org (8.14.9/8.14.9/Submit) id t2C278hx072012; Wed, 11 Mar 2015 20:07:08 -0600 (MDT) (envelope-from ken) Date: Wed, 11 Mar 2015 20:07:07 -0600 From: "Kenneth D. Merry" To: Harald Schmalzbauer Subject: Re: sa(4) driver changes available for test Message-ID: <20150312020707.GA71962@mithlond.kdm.org> References: <20150214003232.GA63990@mithlond.kdm.org> <20150219001347.GA57416@mithlond.kdm.org> <54FAFD72.4080704@omnilan.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <54FAFD72.4080704@omnilan.de> User-Agent: Mutt/1.5.23 (2014-03-12) X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.4.3 (mithlond.kdm.org [127.0.0.1]); Wed, 11 Mar 2015 20:07:08 -0600 (MDT) X-Spam-Status: No, score=-2.9 required=5.0 tests=ALL_TRUSTED,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mithlond.kdm.org Cc: current@freebsd.org, scsi@freebsd.org X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.18-1 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: Thu, 12 Mar 2015 02:16:34 -0000 On Sat, Mar 07, 2015 at 14:30:26 +0100, Harald Schmalzbauer wrote: > Bez?glich Kenneth D. Merry's Nachricht vom 19.02.2015 01:13 (localtime): > > I have updated the patches. > > > > I have removed the XPT_DEV_ADVINFO changes from the patches to head, since > > I committed those separately. > > > > I have (hopefully) fixed the build for the stable/10 patches by MFCing > > dependencies. (One of them mav did for me, thanks!) > > > > Rough draft commit message: > > > > http://people.freebsd.org/~ken/sa_changes_commitmsg.20150218.1.txt > > > > The patches against FreeBSD/head as of SVN revision 278975: > > > > http://people.freebsd.org/~ken/sa_changes.20150218.1.txt > > > > And (untested) patches against FreeBSD stable/10 as of SVN revision 278974: > > > > http://people.freebsd.org/~ken/sa_changes.stable_10.20150218.1.txt > > Hello, > > on 26/02/2105, r278964 seems to be part from the sa_changes patchset. > Do you have a sa_changes.stable_10.20150226 ready? I haven't done it yet, sorry. > Or is it just a matter of exluding all parts, comitted with r278964 > from the patchset? > I've done so in the mean while: > ftp://ftp.omnilan.de/pub/FreeBSD/OmniLAN/misc/sa_changes.stable_10.20150226.fudge.patch > Thanks! > Noticed that in sys/dev/mps/mps_sas.c 'cdai.flags' gets conditionally > (#if __FreeBSD_version >= 1100061) the new "CDAI_FLAG_NONE", while in > sbin/camcontrol/camcontrol.c, this is unconditionally used. Haven't > really looked at the code, mostly because my skills wouldN#t allow me to > answer this qusteion myself, but is that versioncheck in mps_sas.c still > vaild with the rest of the sa_driver-changes? Yes, that's intentional. The mps(4) and mpr(4) drivers are also maintained outside the tree by LSI/Avago. I usually try to put version checks in there, so that things work when they try to compile against earlier releases. Otherwise they'd be putting in the same checks themselves. It is easier to do them when the changes go in the tree. camcontrol(8), on the other hand, is only maintained in the FreeBSD tree. So it only ever needs to build against the FreeBSD branch it is in. Ken -- Kenneth Merry ken@FreeBSD.ORG