@@ -16,7 +16,7 @@ unsigned long long kvm__arch_get_kern_offset(struct kvm *kvm, int fd)
struct arm64_image_header header;
off_t cur_offset;
ssize_t size;
- const char *warn_str;
+ const char *debug_str;
/* the 32bit kernel offset is a well known value */
if (kvm->cfg.arch.aarch32_guest)
@@ -25,8 +25,8 @@ unsigned long long kvm__arch_get_kern_offset(struct kvm *kvm, int fd)
cur_offset = lseek(fd, 0, SEEK_CUR);
if (cur_offset == (off_t)-1 ||
lseek(fd, 0, SEEK_SET) == (off_t)-1) {
- warn_str = "Failed to seek in kernel image file";
- goto fail;
+ debug_str = "Failed to seek in kernel image file";
+ goto default_offset;
}
size = xread(fd, &header, sizeof(header));
@@ -36,16 +36,16 @@ unsigned long long kvm__arch_get_kern_offset(struct kvm *kvm, int fd)
lseek(fd, cur_offset, SEEK_SET);
if (memcmp(&header.magic, ARM64_IMAGE_MAGIC, sizeof(header.magic))) {
- warn_str = "Kernel image magic not matching";
- goto fail;
+ debug_str = "Kernel image magic not matching";
+ goto default_offset;
}
if (le64_to_cpu(header.image_size))
return le64_to_cpu(header.text_offset);
- warn_str = "Image size is 0";
-fail:
- pr_warning("%s, assuming TEXT_OFFSET to be 0x80000", warn_str);
+ debug_str = "Image size is 0";
+default_offset:
+ pr_debug("%s, assuming TEXT_OFFSET to be 0x80000", debug_str);
return 0x80000;
}
kvmtool complains loudly when it parses the kernel header and doesn't find what it expects, but unless it outright fails to read the kernel image, it will copy the image in the guest memory at the default offset of 0x80000. There's no technical reason to stop the user from loading payloads other than a Linux kernel with the --kernel option. These payloads can behave just like a kernel and can use an initrd (which is not possible with --firmware), but don't have the kernel header (like kvm-unit-tests), and the warnings kvmtool emites can be confusing for this type of payloads. Change the warnings to debug statements, which can be enabled via the --debug kvmtool command line option, to make them disappear for these cases where they aren't really relevant. Signed-off-by: Alexandru Elisei <alexandru.elisei@arm.com> --- arm/aarch64/kvm.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-)