diff options
author | Aron Griffis <aron@hp.com> | 2008-01-08 22:29:38 -0500 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2008-02-04 15:35:06 -0800 |
commit | 410ab512e5c5716287a399145df0905c1dcddb04 (patch) | |
tree | 7e56cec14488041fab537b118bb237fc2b59f7eb | |
parent | 965e7c8affeca27f7e5de75c97954e74d3b8052d (diff) | |
download | op-kernel-dev-410ab512e5c5716287a399145df0905c1dcddb04.zip op-kernel-dev-410ab512e5c5716287a399145df0905c1dcddb04.tar.gz |
[IA64] efi.c Add /* never reached */ annotation
As written, this loop could be for (;;) instead of do while (md). The tests
inside the loop always result in a return so the loop never terminates normally.
Signed-off-by: Aron Griffis <aron@hp.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
-rw-r--r-- | arch/ia64/kernel/efi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/ia64/kernel/efi.c b/arch/ia64/kernel/efi.c index 269f4f4..d59134d 100644 --- a/arch/ia64/kernel/efi.c +++ b/arch/ia64/kernel/efi.c @@ -762,7 +762,7 @@ efi_mem_attribute (unsigned long phys_addr, unsigned long size) if (!md || (md->attribute & ~EFI_MEMORY_RUNTIME) != attr) return 0; } while (md); - return 0; + return 0; /* never reached */ } u64 @@ -798,7 +798,7 @@ kern_mem_attribute (unsigned long phys_addr, unsigned long size) if (!md || md->attribute != attr) return 0; } while (md); - return 0; + return 0; /* never reached */ } EXPORT_SYMBOL(kern_mem_attribute); |