diff options
author | Jesper Juhl <juhl-lkml@dif.dk> | 2005-06-16 15:14:00 -0700 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2005-06-16 16:27:14 -0700 |
commit | 986a80d5c154808cc78170584670324a22fd8219 (patch) | |
tree | 11cc27ed20f65d1693ad2883a8606fbd7822888d /include/linux/efi.h | |
parent | 4845f3333765b732aa2d7ea6d72fd03cfec4fbf3 (diff) | |
download | kernel_samsung_espresso10-986a80d5c154808cc78170584670324a22fd8219.zip kernel_samsung_espresso10-986a80d5c154808cc78170584670324a22fd8219.tar.gz kernel_samsung_espresso10-986a80d5c154808cc78170584670324a22fd8219.tar.bz2 |
[PATCH] avoid signed vs unsigned comparison in efi_range_is_wc()
warning when building with gcc -W :
include/linux/efi.h: In function `efi_range_is_wc':
include/linux/efi.h:320: warning: comparison between signed and unsigned
It looks to me like a significantly large 'len' passed in could cause the
loop to never end. Isn't it safer to make 'i' an unsigned long as well?
Like this little patch below (which of course also kills the warning) :
Signed-off-by: Jesper Juhl <juhl-lkml@dif.dk>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'include/linux/efi.h')
-rw-r--r-- | include/linux/efi.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/efi.h b/include/linux/efi.h index 047e722..73781ec 100644 --- a/include/linux/efi.h +++ b/include/linux/efi.h @@ -315,7 +315,7 @@ extern struct efi_memory_map memmap; */ static inline int efi_range_is_wc(unsigned long start, unsigned long len) { - int i; + unsigned long i; for (i = 0; i < len; i += (1UL << EFI_PAGE_SHIFT)) { unsigned long paddr = __pa(start + i); |