aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAron Griffis <aron@hp.com>2008-01-08 22:29:38 -0500
committerTony Luck <tony.luck@intel.com>2008-02-04 18:35:06 -0500
commit410ab512e5c5716287a399145df0905c1dcddb04 (patch)
tree7e56cec14488041fab537b118bb237fc2b59f7eb
parent965e7c8affeca27f7e5de75c97954e74d3b8052d (diff)
[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.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/ia64/kernel/efi.c b/arch/ia64/kernel/efi.c
index 269f4f4adfed..d59134d7e73c 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)
762 if (!md || (md->attribute & ~EFI_MEMORY_RUNTIME) != attr) 762 if (!md || (md->attribute & ~EFI_MEMORY_RUNTIME) != attr)
763 return 0; 763 return 0;
764 } while (md); 764 } while (md);
765 return 0; 765 return 0; /* never reached */
766} 766}
767 767
768u64 768u64
@@ -798,7 +798,7 @@ kern_mem_attribute (unsigned long phys_addr, unsigned long size)
798 if (!md || md->attribute != attr) 798 if (!md || md->attribute != attr)
799 return 0; 799 return 0;
800 } while (md); 800 } while (md);
801 return 0; 801 return 0; /* never reached */
802} 802}
803EXPORT_SYMBOL(kern_mem_attribute); 803EXPORT_SYMBOL(kern_mem_attribute);
804 804