linux-kernel - [PATCH] [34/50] i386: Fix argument signedness warnings

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] [thread-next>] [day] [month] [year] [list]

Message-Id: <20070921223234.0A71513DCD@wotan.suse.de>
Date:	Sat, 22 Sep 2007 00:32:33 +0200 (CEST)
From:	Andi Kleen <ak@...e.de>
To:	satyam@...radead.org, patches@...-64.org,
	linux-kernel@...r.kernel.org
Subject: [PATCH] [34/50] i386: Fix argument signedness warnings


From: Satyam Sharma <satyam@...radead.org>


These build warnings:

In file included from include/asm/thread_info.h:16,
from include/linux/thread_info.h:21,
from include/linux/preempt.h:9,
from include/linux/spinlock.h:49,
from include/linux/vmalloc.h:4,
from arch/i386/boot/compressed/misc.c:14:
include/asm/processor.h: In function .$B!F.(Jcpuid_count.$B!G.(J:
include/asm/processor.h:615: warning: pointer targets in passing argument 1 of .$B!F.(Jnative_cpuid.$B!G.(J differ in signedness
include/asm/processor.h:615: warning: pointer targets in passing argument 2 of .$B!F.(Jnative_cpuid.$B!G.(J differ in signedness
include/asm/processor.h:615: warning: pointer targets in passing argument 3 of .$B!F.(Jnative_cpuid.$B!G.(J differ in signedness
include/asm/processor.h:615: warning: pointer targets in passing argument 4 of .$B!F.(Jnative_cpuid.$B!G.(J differ in signedness

come because the arguments have been specified as pointers to (signed) int
types, not unsigned. So let's specify those as unsigned. Do some codingstyle
here and there while at it.

Signed-off-by: Satyam Sharma <satyam@...radead.org>
Signed-off-by: Andi Kleen <ak@...e.de>

---

 include/asm-i386/processor.h |    9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

Index: linux/include/asm-i386/processor.h
===================================================================
--- linux.orig/include/asm-i386/processor.h
+++ linux/include/asm-i386/processor.h
@@ -595,7 +595,9 @@ static inline void load_esp0(struct tss_
  * clear %ecx since some cpus (Cyrix MII) do not set or clear %ecx
  * resulting in stale register contents being returned.
  */
-static inline void cpuid(unsigned int op, unsigned int *eax, unsigned int *ebx, unsigned int *ecx, unsigned int *edx)
+static inline void cpuid(unsigned int op,
+			 unsigned int *eax, unsigned int *ebx,
+			 unsigned int *ecx, unsigned int *edx)
 {
 	*eax = op;
 	*ecx = 0;
@@ -603,8 +605,9 @@ static inline void cpuid(unsigned int op
 }
 
 /* Some CPUID calls want 'count' to be placed in ecx */
-static inline void cpuid_count(int op, int count, int *eax, int *ebx, int *ecx,
-			       int *edx)
+static inline void cpuid_count(unsigned int op, int count,
+			       unsigned int *eax, unsigned int *ebx,
+			       unsigned int *ecx, unsigned int *edx)
 {
 	*eax = op;
 	*ecx = count;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/