From patchwork Tue Nov 24 12:26:19 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Corentin Labbe X-Patchwork-Id: 7690101 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 7B157BF90C for ; Tue, 24 Nov 2015 12:29:25 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 810ED207A4 for ; Tue, 24 Nov 2015 12:29:24 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 8B09520797 for ; Tue, 24 Nov 2015 12:29:23 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1a1Cgt-0004mD-7D; Tue, 24 Nov 2015 12:27:23 +0000 Received: from mail-wm0-x22a.google.com ([2a00:1450:400c:c09::22a]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1a1Cgp-0004gi-1T for linux-arm-kernel@lists.infradead.org; Tue, 24 Nov 2015 12:27:20 +0000 Received: by wmec201 with SMTP id c201so24204562wme.1 for ; Tue, 24 Nov 2015 04:26:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id; bh=o2B5Ag0aYOsI1iIlfT+Zshm4NvOheMRw7DbFrFpVh6Q=; b=MatoVObmrJmj/xn/sNQ+kZI2rWFv397YUCivKqwruazVMw3lrU3iszN4ysV5vQII5w z2v+mMgC5720UCbfTM/DqEqpOElUuRwUUvRubrKfdG0sIG/EcyyN8+K8BjPeFVZgIQ9j KSb3RhxTGJdt29DnV4MSqo4AXxzuMdr/gAQ6ezeXnqiB6HfgB2yfC94HJ3l7H/jRg7Pp ezQ6JuwIOBD37TQuu5O5wUcOm0Idi3AdZu3syQ0aTDpWC+pNmb9HvWWYPVPe10uxSkIT wzaOhgA+NGSSbTc6xV8KPa/bw7WARw4SMw/ohRUWkkbiaSxvtiSmQ20H9V9g7A4no4fl rqdw== X-Received: by 10.28.134.199 with SMTP id i190mr24640292wmd.33.1448368017348; Tue, 24 Nov 2015 04:26:57 -0800 (PST) Received: from Red.local (ANice-651-1-376-123.w86-193.abo.wanadoo.fr. [86.193.8.123]) by smtp.googlemail.com with ESMTPSA id b82sm18113798wmf.9.2015.11.24.04.26.55 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 24 Nov 2015 04:26:56 -0800 (PST) From: LABBE Corentin To: clabbe.montjoie@gmail.com, linux-arm-kernel@lists.infradead.org, linux@arm.linux.org.uk, yamada.masahiro@socionext.com Subject: [PATCH RESEND] arm: fix the return value of find_first{_zero}_bit Date: Tue, 24 Nov 2015 13:26:19 +0100 Message-Id: <1448367991-12734-1-git-send-email-clabbe.montjoie@gmail.com> X-Mailer: git-send-email 2.4.10 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20151124_042719_375214_6AC64B99 X-CRM114-Status: GOOD ( 12.51 ) X-Spam-Score: -2.7 (--) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-kernel@vger.kernel.org MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-4.6 required=5.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, RCVD_IN_DNSWL_MED, RP_MATCHES_RCVD, T_DKIM_INVALID, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Building linux with W=1 on arm give me this warning; In file included from ../include/linux/nodemask.h:92:0, from ../include/linux/mmzone.h:16, from ../include/linux/gfp.h:5, from ../include/linux/kmod.h:22, from ../include/linux/module.h:13, from net/netfilter/xt_nat.mod.c:1: ../include/linux/bitmap.h: In function 'bitmap_empty': ../include/linux/bitmap.h:287:36: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] return find_first_bit(src, nbits) == nbits; ../include/linux/bitmap.h: In function 'bitmap_full': ../include/linux/bitmap.h:295:41: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] return find_first_zero_bit(src, nbits) == nbits; The problem is that find_first{_zero}_bit() return type is unsigned long on majority of arch. Since find_first{_zero}_bit cannot return negative value, unsigned long is the best return type. After some read, it seems that find_next_bit_[lb]e/_find_next_zero_bit_[lb]e are also incorectly set as return int. This patch fix the return type of all thoses functions. Signed-off-by: LABBE Corentin --- arch/arm/include/asm/bitops.h | 22 +++++++++++----------- arch/arm/lib/findbit.S | 8 ++++---- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/arch/arm/include/asm/bitops.h b/arch/arm/include/asm/bitops.h index e943e6c..868414b 100644 --- a/arch/arm/include/asm/bitops.h +++ b/arch/arm/include/asm/bitops.h @@ -159,18 +159,18 @@ extern int _test_and_change_bit(int nr, volatile unsigned long * p); /* * Little endian assembly bitops. nr = 0 -> byte 0 bit 0. */ -extern int _find_first_zero_bit_le(const void * p, unsigned size); -extern int _find_next_zero_bit_le(const void * p, int size, int offset); -extern int _find_first_bit_le(const unsigned long *p, unsigned size); -extern int _find_next_bit_le(const unsigned long *p, int size, int offset); +extern unsigned long _find_first_zero_bit_le(const void *p, unsigned size); +extern unsigned long _find_next_zero_bit_le(const void *p, int size, int offset); +extern unsigned long _find_first_bit_le(const unsigned long *p, unsigned size); +extern unsigned long _find_next_bit_le(const unsigned long *p, int size, int offset); /* * Big endian assembly bitops. nr = 0 -> byte 3 bit 0. */ -extern int _find_first_zero_bit_be(const void * p, unsigned size); -extern int _find_next_zero_bit_be(const void * p, int size, int offset); -extern int _find_first_bit_be(const unsigned long *p, unsigned size); -extern int _find_next_bit_be(const unsigned long *p, int size, int offset); +extern unsigned long _find_first_zero_bit_be(const void *p, unsigned size); +extern unsigned long _find_next_zero_bit_be(const void *p, int size, int offset); +extern unsigned long _find_first_bit_be(const unsigned long *p, unsigned size); +extern unsigned long _find_next_bit_be(const unsigned long *p, int size, int offset); #ifndef CONFIG_SMP /* @@ -317,19 +317,19 @@ static inline unsigned long __ffs(unsigned long x) #ifdef __ARMEB__ -static inline int find_first_zero_bit_le(const void *p, unsigned size) +static inline unsigned long find_first_zero_bit_le(const void *p, unsigned size) { return _find_first_zero_bit_le(p, size); } #define find_first_zero_bit_le find_first_zero_bit_le -static inline int find_next_zero_bit_le(const void *p, int size, int offset) +static inline unsigned long find_next_zero_bit_le(const void *p, int size, int offset) { return _find_next_zero_bit_le(p, size, offset); } #define find_next_zero_bit_le find_next_zero_bit_le -static inline int find_next_bit_le(const void *p, int size, int offset) +static inline unsigned long find_next_bit_le(const void *p, int size, int offset) { return _find_next_bit_le(p, size, offset); } diff --git a/arch/arm/lib/findbit.S b/arch/arm/lib/findbit.S index 7848780..b4f6aec 100644 --- a/arch/arm/lib/findbit.S +++ b/arch/arm/lib/findbit.S @@ -19,7 +19,7 @@ /* * Purpose : Find a 'zero' bit - * Prototype: int find_first_zero_bit(void *addr, unsigned int maxbit); + * Prototype: unsigned long find_first_zero_bit(void *addr, unsigned int maxbit); */ ENTRY(_find_first_zero_bit_le) teq r1, #0 @@ -40,7 +40,7 @@ ENDPROC(_find_first_zero_bit_le) /* * Purpose : Find next 'zero' bit - * Prototype: int find_next_zero_bit(void *addr, unsigned int maxbit, int offset) + * Prototype: unsigned long find_next_zero_bit(void *addr, unsigned int maxbit, int offset) */ ENTRY(_find_next_zero_bit_le) teq r1, #0 @@ -60,7 +60,7 @@ ENDPROC(_find_next_zero_bit_le) /* * Purpose : Find a 'one' bit - * Prototype: int find_first_bit(const unsigned long *addr, unsigned int maxbit); + * Prototype: unsigned long find_first_bit(const unsigned long *addr, unsigned int maxbit); */ ENTRY(_find_first_bit_le) teq r1, #0 @@ -81,7 +81,7 @@ ENDPROC(_find_first_bit_le) /* * Purpose : Find next 'one' bit - * Prototype: int find_next_zero_bit(void *addr, unsigned int maxbit, int offset) + * Prototype: unsigned long find_next_zero_bit(void *addr, unsigned int maxbit, int offset) */ ENTRY(_find_next_bit_le) teq r1, #0