diff options
| author | Amerigo Wang <amwang@redhat.com> | 2011-04-27 11:33:27 +0800 |
|---|---|---|
| committer | Simon Horman <horms@verge.net.au> | 2011-04-27 15:15:17 +0900 |
| commit | 600f18e344e8eb887042574ceea28a484a1da12f (patch) | |
| tree | 150aafbd3911f98b4b3267acf0483eb33e72caac /kexec/crashdump-elf.c | |
| parent | 37e6c4752822beb0feb00e4c2a462000550e105b (diff) | |
kexec: remove duplicated backup_src_start field from struct crash_elf_info
Vivek pointed out that we have duplicated ->backup_src_start
in struct crash_elf_info and struct kexec_info.
This patch removes the ->backup_src_start and ->backup_src_end
from struct crash_elf_info.
I tested it on both i686 and ppc64, and used a test case from
Dave Anderson to confirm the backup region is correct on i686.
Signed-off-by: WANG Cong <amwang@redhat.com>
Cc: Vivek Goyal <vgoyal@redhat.com>
Signed-off-by: Simon Horman <horms@verge.net.au>
Diffstat (limited to 'kexec/crashdump-elf.c')
| -rw-r--r-- | kexec/crashdump-elf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kexec/crashdump-elf.c b/kexec/crashdump-elf.c index 954d670..8d82db9 100644 --- a/kexec/crashdump-elf.c +++ b/kexec/crashdump-elf.c @@ -227,8 +227,8 @@ int FUNC(struct kexec_info *info, phdr->p_flags = PF_R|PF_W|PF_X; phdr->p_offset = mstart; - if (mstart == elf_info->backup_src_start - && mend == elf_info->backup_src_end) + if (mstart == info->backup_src_start + && (mend - mstart + 1) == info->backup_src_size) phdr->p_offset = info->backup_start; /* We already prepared the header for kernel text. Map |