From owner-cvs-all@FreeBSD.ORG  Thu Jul 27 16:52:33 2006
Return-Path: <owner-cvs-all@FreeBSD.ORG>
X-Original-To: cvs-all@FreeBSD.org
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 1C88D16A4DE;
	Thu, 27 Jul 2006 16:52:33 +0000 (UTC) (envelope-from nate@root.org)
Received: from ylpvm01.prodigy.net (ylpvm01-ext.prodigy.net [207.115.57.32])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 2E56A43DD7;
	Thu, 27 Jul 2006 16:52:23 +0000 (GMT) (envelope-from nate@root.org)
X-ORBL: [67.119.74.222]
Received: from [10.0.0.53] (adsl-67-119-74-222.dsl.sntc01.pacbell.net
	[67.119.74.222])
	by ylpvm01.prodigy.net (8.13.7 out spool5000 dk/8.13.7) with ESMTP id
	k6RGqAUi032475; Thu, 27 Jul 2006 12:52:11 -0400
Message-ID: <44C8E964.1060106@root.org>
Date: Thu, 27 Jul 2006 09:27:16 -0700
From: Nate Lawson <nate@root.org>
User-Agent: Thunderbird 1.5.0.2 (X11/20060501)
MIME-Version: 1.0
To: Pawel Jakub Dawidek <pjd@FreeBSD.org>
References: <20060725190716.C73E016A595@hub.freebsd.org>
In-Reply-To: <20060725190716.C73E016A595@hub.freebsd.org>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/crypto/via padlock.h
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 27 Jul 2006 16:52:33 -0000

Pawel Jakub Dawidek wrote:
> pjd         2006-07-25 19:06:54 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/crypto/via       padlock.h 
>   Log:
>   Modify PADLOCK_ALIGN() macro, so when the given address is already 16 bytes
>   aligned, it will be used directly, not 'address + 16'.
>   
>   Revision  Changes    Path
>   1.3       +1 -1      src/sys/crypto/via/padlock.h
> 
> 
> Index: src/sys/crypto/via/padlock.h
> diff -u src/sys/crypto/via/padlock.h:1.2 src/sys/crypto/via/padlock.h:1.3
> --- src/sys/crypto/via/padlock.h:1.2	Tue Jul 25 19:04:26 2006
> +++ src/sys/crypto/via/padlock.h	Tue Jul 25 19:06:54 2006
> @@ -72,7 +72,7 @@
>  	TAILQ_ENTRY(padlock_session) ses_next;
>  };
>  
> -#define PADLOCK_ALIGN(p)	(void *)((char *)(p) - ((uintptr_t)(p) % 16) + 16)
> +#define PADLOCK_ALIGN(p)	(void *)((uintptr_t)(p) - (((uintptr_t)(p) - 1) % 16) + 15)
>  
>  int	padlock_cipher_setup(struct padlock_session *ses,
>  	    struct cryptoini *encini);

I think you want roundup2() since 16 is a power of 2, see sys/param.h

-- 
Nate