From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760310AbaGEA22 (ORCPT ); Fri, 4 Jul 2014 20:28:28 -0400 Received: from mail.eperm.de ([89.247.134.16]:41777 "EHLO mail.eperm.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753766AbaGEA20 (ORCPT ); Fri, 4 Jul 2014 20:28:26 -0400 X-AuthUser: sm@eperm.de From: Stephan Mueller To: Joe Perches Cc: Dan Carpenter , Stephen Rothwell , Herbert Xu , kbuild test robot , kbuild@01.org, linux-crypto@vger.kernel.org, Randy Dunlap , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/4] DRBG: Fix format string for debugging statements Date: Sat, 05 Jul 2014 02:27:27 +0200 Message-ID: <2812156.9vUu6vDFVa@myon.chronox.de> User-Agent: KMail/4.12.5 (Linux/3.14.9-200.fc20.x86_64; KDE/4.12.5; x86_64; ; ) In-Reply-To: <1404519849.6384.32.camel@joe-AO725> References: <4927386.WYHrRe3NJm@myon.chronox.de> <3976291.2Ne0xJphcZ@myon.chronox.de> <1404519849.6384.32.camel@joe-AO725> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am Freitag, 4. Juli 2014, 17:24:09 schrieb Joe Perches: Hi Joe, > On Sat, 2014-07-05 at 02:15 +0200, Stephan Mueller wrote: > > Am Freitag, 4. Juli 2014, 17:09:33 schrieb Joe Perches: > > > On Sat, 2014-07-05 at 01:57 +0200, Stephan Mueller wrote: > > > > And I also get the same output. Yet I am not sure how that code can be > > > > compared to the code in the kernel. > > > > > > What that code shows is that the ARRAY_SIZE > > > type is size_t. > > > > > > The difference is ARRAY_SIZE in the kernel > > > should be output with %zu. > > > > Using %zu works without a warning on my 64 bit machine. So you are saying > > that ARRAY_SIZE will always be size_t on every architecture? If yes, I > > will update my patch. > > Hi again. > > ARRAY_SIZE is size_t in all architectures. Thank you very much, this is the answer I was looking for :-) I will update the patch accordingly. -- Ciao Stephan