diff options
author | Dave Young <dyoung@redhat.com> | 2014-02-06 14:30:44 +0800 |
---|---|---|
committer | Simon Horman <horms@verge.net.au> | 2014-02-06 16:29:23 +0900 |
commit | 507e210daf047a5ef98de680151ace745297d82e (patch) | |
tree | db260af5f7e6a8f747acf5bf38f823cfdd3128ef /kexec/add_buffer.c | |
parent | eb05abb060191940a0efc81e9fdce69aecd9e8b1 (diff) |
kernel image probe function return value checking fix
Currently kexec will use the kernel image type when probe function return
value >=0. It looks odd, but previously it works. Since commit bf06cf2095
it does not work anymore.
During my testing for arm zImage, in 2nd kernel the atags pointer and the
machine_id are not valid, I did a lot of debugging in kernel, finally I found
this is caused by a kexec tools bug instead.
Because uImage will be probed before zImage, also the uImage probe return 1
instead of -1 since bf06cf2095, thus kexec will mistakenly think it is uImage.
Fix this issue by regarding it's valid only when probe return 0.
Signed-off-by: Dave Young <dyoung@redhat.com>
Signed-off-by: Simon Horman <horms@verge.net.au>
Diffstat (limited to 'kexec/add_buffer.c')
0 files changed, 0 insertions, 0 deletions