linux-kernel - Re: [PATCH 0/2] crypto: sunxi-ss: fix 64-bit compilation

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <568F9A39.9030400@arm.com>
Date:	Fri, 8 Jan 2016 11:15:05 +0000
From:	Andre Przywara <andre.przywara@....com>
To:	Herbert Xu <herbert@...dor.apana.org.au>
Cc:	Corentin Labbe <clabbe.montjoie@...il.com>,
	maxime.ripard@...e-electrons.com, Chen-Yu Tsai <wens@...e.org>,
	arnd@...db.de, "David S . Miller" <davem@...emloft.net>,
	linux-sunxi@...glegroups.com, linux-arm-kernel@...ts.infradead.org,
	linux-kernel@...r.kernel.org, linux-crypto@...r.kernel.org
Subject: Re: [PATCH 0/2] crypto: sunxi-ss: fix 64-bit compilation

Hi Herbert,

On 08/01/16 09:59, Herbert Xu wrote:
> On Thu, Jan 07, 2016 at 03:58:16PM +0000, Andre Przywara wrote:
>> Hi,
>>
>> these two patches provide a different approach to an issue I tried
>> to fix lately [1].
>> Instead of casting everything I now promote local types to size_t, so
>> that the min3() arguments naturally match in type.
>> As size_t is defined as "unsigned int" on 32-bit architectures
>> anyway, that actually does not change anything there, but instead
>> provides a clean approach to get it compiled for arm64.
>>
>> I split this up because 1/2 seems much cleaner to me than 2/2, so we
>> can have a separate discussion/merge process on this.
> 
> If this is meant for the crypto tree it needs to go to the linux-crypto
> list.

Oh dear, I planned on adding the list, but eventually forgot it. Sorry
for that, I will resend it to linux-crypto.

Thanks for pointing this out!

Cheers,
Andre.