cbfstool: Add '-p' option for padding

This patch adds '-p' to the 'add' command. It allows the add
command to specify the size of the padding added with the file
being added. This is useful to reserve an extra space in case
the file is too big to be relocated.

BUG=b:68660966
BRANCH=none
TEST=emerge-fizz coreboot &&
cbfstool image.bin add -n ecrw -f EC_RW.bin -p 0x10 ...
Verify image.bin has extra space in the file header.

Change-Id: I64bc54fd10a453b4da467bc69d9590e61b0f7ead
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://review.coreboot.org/22239
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
This commit is contained in:
Daisuke Nojiri 2017-10-30 17:38:04 -07:00 committed by Aaron Durbin
parent 9adcbfe486
commit ff906fb95b
2 changed files with 29 additions and 6 deletions

View File

@ -106,6 +106,7 @@ struct cbfs_file_attribute {
#define CBFS_FILE_ATTR_TAG_HASH 0x68736148 #define CBFS_FILE_ATTR_TAG_HASH 0x68736148
#define CBFS_FILE_ATTR_TAG_POSITION 0x42435350 /* PSCB */ #define CBFS_FILE_ATTR_TAG_POSITION 0x42435350 /* PSCB */
#define CBFS_FILE_ATTR_TAG_ALIGNMENT 0x42434c41 /* ALCB */ #define CBFS_FILE_ATTR_TAG_ALIGNMENT 0x42434c41 /* ALCB */
#define CBFS_FILE_ATTR_TAG_PADDING 0x47444150 /* PDNG */
struct cbfs_file_attr_compression { struct cbfs_file_attr_compression {
uint32_t tag; uint32_t tag;

View File

@ -71,6 +71,7 @@ static struct param {
uint32_t cbfsoffset; uint32_t cbfsoffset;
uint32_t cbfsoffset_assigned; uint32_t cbfsoffset_assigned;
uint32_t arch; uint32_t arch;
uint32_t padding;
bool u64val_assigned; bool u64val_assigned;
bool fill_partial_upward; bool fill_partial_upward;
bool fill_partial_downward; bool fill_partial_downward;
@ -421,6 +422,18 @@ static int cbfs_add_component(const char *filename,
} }
} }
if (param.padding) {
const uint32_t hs = sizeof(struct cbfs_file_attribute);
uint32_t size = MAX(hs, param.padding);
INFO("Padding %d bytes\n", size);
struct cbfs_file_attribute *attr =
(struct cbfs_file_attribute *)cbfs_add_file_attr(
header, CBFS_FILE_ATTR_TAG_PADDING,
size);
if (attr == NULL)
return -1;
}
if (IS_TOP_ALIGNED_ADDRESS(offset)) if (IS_TOP_ALIGNED_ADDRESS(offset))
offset = convert_to_from_top_aligned(param.image_region, offset = convert_to_from_top_aligned(param.image_region,
-offset); -offset);
@ -1139,12 +1152,12 @@ static int cbfs_truncate(void)
} }
static const struct command commands[] = { static const struct command commands[] = {
{"add", "H:r:f:n:t:c:b:a:yvA:gh?", cbfs_add, true, true}, {"add", "H:r:f:n:t:c:b:a:p:yvA:gh?", cbfs_add, true, true},
{"add-flat-binary", "H:r:f:n:l:e:c:b:vA:gh?", cbfs_add_flat_binary, {"add-flat-binary", "H:r:f:n:l:e:c:b:p:vA:gh?", cbfs_add_flat_binary,
true, true}, true, true},
{"add-payload", "H:r:f:n:t:c:b:C:I:vA:gh?", cbfs_add_payload, {"add-payload", "H:r:f:n:t:c:b:C:I:p:vA:gh?", cbfs_add_payload,
true, true}, true, true},
{"add-stage", "a:H:r:f:n:t:c:b:P:S:yvA:gh?", cbfs_add_stage, {"add-stage", "a:H:r:f:n:t:c:b:P:S:p:yvA:gh?", cbfs_add_stage,
true, true}, true, true},
{"add-int", "H:r:i:n:b:vgh?", cbfs_add_integer, true, true}, {"add-int", "H:r:i:n:b:vgh?", cbfs_add_integer, true, true},
{"add-master-header", "H:r:vh?", cbfs_add_master_header, true, true}, {"add-master-header", "H:r:vh?", cbfs_add_master_header, true, true},
@ -1187,6 +1200,7 @@ static struct option long_options[] = {
{"machine", required_argument, 0, 'm' }, {"machine", required_argument, 0, 'm' },
{"name", required_argument, 0, 'n' }, {"name", required_argument, 0, 'n' },
{"offset", required_argument, 0, 'o' }, {"offset", required_argument, 0, 'o' },
{"padding", required_argument, 0, 'p' },
{"page-size", required_argument, 0, 'P' }, {"page-size", required_argument, 0, 'P' },
{"size", required_argument, 0, 's' }, {"size", required_argument, 0, 's' },
{"top-aligned", required_argument, 0, 'T' }, {"top-aligned", required_argument, 0, 'T' },
@ -1267,7 +1281,7 @@ static void usage(char *name)
"COMMANDs:\n" "COMMANDs:\n"
" add [-r image,regions] -f FILE -n NAME -t TYPE [-A hash] \\\n" " add [-r image,regions] -f FILE -n NAME -t TYPE [-A hash] \\\n"
" [-c compression] [-b base-address | -a alignment] \\\n" " [-c compression] [-b base-address | -a alignment] \\\n"
" [-y|--xip if TYPE is FSP] " " [-p padding size] [-y|--xip if TYPE is FSP] "
"Add a component\n" "Add a component\n"
" add-payload [-r image,regions] -f FILE -n NAME [-A hash] \\\n" " add-payload [-r image,regions] -f FILE -n NAME [-A hash] \\\n"
" [-c compression] [-b base-address] \\\n" " [-c compression] [-b base-address] \\\n"
@ -1505,6 +1519,14 @@ int main(int argc, char **argv)
return 1; return 1;
} }
break; break;
case 'p':
param.padding = strtoul(optarg, &suffix, 0);
if (!*optarg || (suffix && *suffix)) {
ERROR("Invalid pad size '%s'.\n",
optarg);
return 1;
}
break;
case 'P': case 'P':
param.pagesize = strtoul(optarg, &suffix, 0); param.pagesize = strtoul(optarg, &suffix, 0);
if (!*optarg || (suffix && *suffix)) { if (!*optarg || (suffix && *suffix)) {