Explicitly use the ULL suffix to specify a large constant.

src/platform/vboot_reference/tests/big_firmware_tests.c

Index: src/platform/vboot_reference/tests/big_firmware_tests.c
diff --git a/src/platform/vboot_reference/tests/big_firmware_tests.c b/src/platform/vboot_reference/tests/big_firmware_tests.c
index 4079adde03ed51e6e0005cf5bbaac2bcf068f463..0b0d200b7458efaf97808e6553636a242cdeb516 100644
--- a/src/platform/vboot_reference/tests/big_firmware_tests.c
+++ b/src/platform/vboot_reference/tests/big_firmware_tests.c
@@ -16,7 +16,7 @@
 #include "utility.h"
 
 /* Choose a firmware size greater than the range of 32-bits unsigned. */
-#define BIG_FIRMWARE_SIZE ((uint64_t) 0x100000000)
+#define BIG_FIRMWARE_SIZE ((uint64_t) 0x100000000ULL)

[Will Drewry, 2010/03/30 14:40:14]

Have you run into any weirdness else where in the firmware code (not just
tests)?

If so, you may want to add some compile-time checking to ensure that
constants/max/etc aren't being truncated in unexpected ways.  E.g., this could
be done as: 
#define BIG_FIRMWARE_SIZE U64(0x100000000)

and have it include a platform.h (or in utility.h) that contains some basic
sanity checking:
#if __WORDSIZE == 64                                                            
#  define U64(x) ((uint64_t)(x ## UL))                                          
#elif __WORDSIZE == 32                                                          
#  define U64(x) ((uint64_t)(x ## ULL))                                         
#else                                                                           
#  error "Unsupported compilation target platform"                              
#endif                                                                          

If it seemed practical, you could also expand it to support U32, U16, etc to
ensure that all values are as declared.   However that may be overkill. (Of
course, we could also expand coverage to do any other type and math checking
throughout the code)

 
 #define ROOT_KEY_BASE_NAME "testkeys/key_rsa8192"
 #define FIRMWARE_KEY_BASE_NAME "testkeys/key_rsa1024"