[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <160554375807.96595.16142008590130221699.stgit@devnote2>
Date: Tue, 17 Nov 2020 01:22:38 +0900
From: Masami Hiramatsu <mhiramat@...nel.org>
To: Steven Rostedt <rostedt@...dmis.org>,
Linus Torvalds <torvalds@...ux-foundation.org>
Cc: Chen Yu <yu.c.chen@...el.com>, Chen Yu <yu.chen.surf@...il.com>,
Masami Hiramatsu <mhiramat@...nel.org>,
LKML <linux-kernel@...r.kernel.org>,
Ingo Molnar <mingo@...nel.org>
Subject: [PATCH v2 0/2] tools/bootconfig: Align the bootconfig applied initrd
Hi,
This is the 2nd version of the bootconfig tool update to align
the total size of initrd + bootconfig to 4.
In this version, I've fixed to count the footer size into the
total file size.
To adjust the file size, the bootconfig tool adds padding null
characters in between the boot configuration data and the footer.
This series include Documentation update. The changing points are
- The bootconfig applied initrd image size is aligned to 4.
- To insert the padding null ('\0') bytes, the size in the footer
can be bigger than the actual bootconfig file size.
- But the max size of the boot configuration file is same, because
the max size doesn't include the last null characters.
For this version I decided to keep 4 bytes aligned rather than
longer size. It will be easy to expand it to e.g. 32 bytes (in this
series, we just need to update 3 locations of the code), but I
couldn't find any reason to expand it. Only I could found was
that the grub can align the filesize to 4, and U-Boot/EDK2 will
not change it. So I couldn't say what is the best size.
Anyway, I updated the documentation too, which clearly says that
the above changing points, and if the bootloader pass the wrong
size, kernel will not find bootconfig from the initrd.
Thank you,
---
Masami Hiramatsu (2):
tools/bootconfig: Align the bootconfig applied initrd image size to 4
Documentation: bootconfig: Update file format on initrd image
tools/bootconfig/main.c | 36 ++++++++++++++++++++++++++++++-----
tools/bootconfig/test-bootconfig.sh | 6 +++++-
2 files changed, 36 insertions(+), 6 deletions(-)
--
Masami Hiramatsu (Linaro) <mhiramat@...nel.org>
Powered by blists - more mailing lists