diff options
| author | Matt Evans <matt@ozlabs.org> | 2010-05-14 14:15:09 +1000 |
|---|---|---|
| committer | Simon Horman <horms@verge.net.au> | 2010-05-19 12:18:29 +0900 |
| commit | 436f1376e1621aeba54d8253244955df4865d5e9 (patch) | |
| tree | ab557c57d939167e8a3217b960e7a54b95fcf18a /kexec/arch/arm/kexec-zImage-arm.c | |
| parent | b1dcc08ea79ca96256a9ae6ef59e5ea4e217ef1d (diff) | |
kexec-tools: Fix option/argument parsing
The argument parsing is currently a bit broken as main()'s getopt_long()
knows nothing about either the architecture-specific options or, even
more specifically, the architecture-and-loader-specific options.
This patch introduces new #defines for all architectures,
KEXEC_ALL_OPTIONS and KEXEC_ALL_OPT_STR. These contain all possible
options for a given build, and the getopt_long() passes in main() and
arch_process_options() will now recognise arch- and loader-specific
options; these will not be re-ordered in argv[], there is no confusion
over which argv[] entry is the kernel filename, and using '--opt=foo' and
'--opt foo' both work.
All architectures have command line options (and #define OPT_BLAHs)
consolidated into their include/arch/option.h files. x86_64 builds
parts of i386/ as well, so now both share a single option.h file (with
a symlink).
Signed-off-by: Matt Evans <matt@ozlabs.org>
Signed-off-by: Simon Horman <horms@verge.net.au>
Diffstat (limited to 'kexec/arch/arm/kexec-zImage-arm.c')
| -rw-r--r-- | kexec/arch/arm/kexec-zImage-arm.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/kexec/arch/arm/kexec-zImage-arm.c b/kexec/arch/arm/kexec-zImage-arm.c index 8e27956..88a6c29 100644 --- a/kexec/arch/arm/kexec-zImage-arm.c +++ b/kexec/arch/arm/kexec-zImage-arm.c @@ -222,8 +222,7 @@ int zImage_arm_load(int argc, char **argv, const char *buf, off_t len, off_t ramdisk_length; off_t ramdisk_offset; int opt; -#define OPT_APPEND 'a' -#define OPT_RAMDISK 'r' + /* See options.h -- add any more there, too. */ static const struct option options[] = { KEXEC_ARCH_OPTIONS { "command-line", 1, 0, OPT_APPEND }, |