From owner-cvs-all@FreeBSD.ORG Fri Mar 11 23:28:34 2005 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5F55B16A4CE; Fri, 11 Mar 2005 23:28:34 +0000 (GMT) Received: from ebb.errno.com (ebb.errno.com [66.127.85.87]) by mx1.FreeBSD.org (Postfix) with ESMTP id E8C6B43D2F; Fri, 11 Mar 2005 23:28:33 +0000 (GMT) (envelope-from sam@errno.com) Received: from [66.127.85.89] ([66.127.85.89]) (authenticated bits=0) by ebb.errno.com (8.12.9/8.12.6) with ESMTP id j2BNSXms074827 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Fri, 11 Mar 2005 15:28:33 -0800 (PST) (envelope-from sam@errno.com) Message-ID: <423229D3.6090308@errno.com> Date: Fri, 11 Mar 2005 15:29:23 -0800 From: Sam Leffler Organization: Errno Consulting User-Agent: Mozilla Thunderbird 1.0 (Macintosh/20041206) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Hajimu UMEMOTO References: <200503111626.j2BGQAvL012546@repoman.freebsd.org> <4231D748.4080405@errno.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/crypto/rijndael rijndael-alg-fst.c rijndael-alg-fst.h rijndael-api-fst.c rijndael-api-fst.h rijndael.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Mar 2005 23:28:34 -0000 Hajimu UMEMOTO wrote: > Hi, > > >>>>>>On Fri, 11 Mar 2005 09:37:12 -0800 >>>>>>Sam Leffler said: > > > sam> Why did you make this change and others before I could respond? I told > sam> you I agreed with your overall goal but that there were specific isuses > sam> we needed to resolve before you pulled the trigger. Less than 24 hours > sam> after your last note to me you committed the changes. > > Sorry, I was too premature. > > sam> This change in particular I'm not happy with because--as I explained to > sam> you--I am uncertain if there are user applications that include files > sam> from /usr/include/crypto that may be affected. > > I changed to install crypto/rijndael/rijndael.h as > /usr/include/crypto/rijndael.h. The difference from > opencrypto/rijndael.h are: > > - export four internal functions. I realized it is required for > gbde(8). > > - definitions of RIJNDAEL_MAXKC, RIJNDAEL_MAXKB and RIJNDAEL_MAXNR. > These are correspond to MAXKC, MAXKB and MAXNR in > opencrypto/rijndael.h. > > - rijndael_set_key() always sets up full context. opencrypto's one > has a flag to set up context for encryption only. > > I confirmed that only gbde(8) refers rindael in src/sys, in our tree. > Are you mind 3rd party application? > > sam> As we discussed privately there is no immediate benefit to what you are > sam> doing. In fact because of these changes we are diverging from BOTH kame > sam> and openbsd code bases. I'm willing to do that but only after > sam> understanding the consequences. > > This integration will be done by moving opencrypto/* into crypto/. I > think there is no conflict between opencrypto/ and crypto/, now. Can > I go ahead? Or, once, should I backout my changes? See my previous note (sent moments before this arrived)--I want you to give me a chance properly review what you are doing. My original import of the OCF had a merged crypto+opencrypto structure. I ended up NOT bringing this into CVS for several reasons. I am still not convinced that what you are doing is the right approach. For now please leave things unchanged (i.e. don't backout your rijndael mods). Sam