cbfstool: Restructure around support for reading/writing portions of files
The buffer API that cbfstool uses to read and write files only directly supports
one-shot operations on whole files. This adds an intermediate partitioned_file
module that sits on top of the buffer system and has an awareness of FMAP
entries. It provides an easy way to get a buffer for an individual region of a
larger image file based on FMAP section name, as well as incrementally write
those smaller buffers back to the backing file at the appropriate offset. The
module has two distinct modes of operation:
- For new images whose layout is described exclusively by an FMAP section, all
the aforementioned functionality will be available.
- For images in the current format, where the CBFS master header serves as the
root of knowledge of the image's size and layout, the module falls back to a
legacy operation mode, where it only allows manipulation of the entire image
as one unit, but exposes this support through the same interface by mapping
the region named SECTION_NAME_PRIMARY_CBFS ("COREBOOT") to the whole file.
The tool is presently only ported onto the new module running in legacy mode:
higher-level support for true "partitioned" images will be forthcoming. However,
as part of this change, the crusty cbfs_image_from_file() and
cbfs_image_write_file() abstractions are removed and replaced with a single
cbfs_image function, cbfs_image_from_buffer(), as well as centralized image
reading/writing directly in cbfstool's main() function. This reduces the
boilerplate required to implement each new action, makes the create action much
more similar to the others, and will make implementing additional actions and
adding in support for the new format much easier.
BUG=chromium:470407
TEST=Build panther and nyan_big coreboot.rom images with and without this patch
and diff their hexdumps. Ensure that no differences occur at different locations
from the diffs between subsequent builds of an identical source tree. Then flash
a full new build onto nyan_big and watch it boot normally.
BRANCH=None
Change-Id: I25578c7b223bc8434c3074cb0dd8894534f8c500
Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Commit-Id: 7e1c96a48e7a27fc6b90289d35e6e169d5e7ad20
Original-Change-Id: Ia4a1a4c48df42b9ec2d6b9471b3a10eb7b24bb39
Original-Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/265581
Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/10134
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
2015-03-25 21:40:08 +01:00
|
|
|
/*
|
|
|
|
* partitioned_file.c, read and write binary file "partitions" described by FMAP
|
|
|
|
*
|
|
|
|
* Copyright (C) 2015 Google, Inc.
|
|
|
|
*
|
|
|
|
* This program is free software; you can redistribute it and/or modify
|
|
|
|
* it under the terms of the GNU General Public License as published by
|
|
|
|
* the Free Software Foundation; version 2 of the License.
|
|
|
|
*
|
|
|
|
* This program is distributed in the hope that it will be useful,
|
|
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
|
|
* GNU General Public License for more details.
|
|
|
|
*
|
|
|
|
* You should have received a copy of the GNU General Public License
|
|
|
|
* along with this program; if not, write to the Free Software
|
Remove address from GPLv2 headers
As per discussion with lawyers[tm], it's not a good idea to
shorten the license header too much - not for legal reasons
but because there are tools that look for them, and giving
them a standard pattern simplifies things.
However, we got confirmation that we don't have to update
every file ever added to coreboot whenever the FSF gets a
new lease, but can drop the address instead.
util/kconfig is excluded because that's imported code that
we may want to synchronize every now and then.
$ find * -type f -exec sed -i "s:Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, *MA[, ]*02110-1301[, ]*USA:Foundation, Inc.:" {} +
$ find * -type f -exec sed -i "s:Foundation, Inc., 51 Franklin Street, Suite 500, Boston, MA 02110-1335, USA:Foundation, Inc.:" {} +
$ find * -type f -exec sed -i "s:Foundation, Inc., 59 Temple Place[-, ]*Suite 330, Boston, MA *02111-1307[, ]*USA:Foundation, Inc.:" {} +
$ find * -type f -exec sed -i "s:Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.:Foundation, Inc.:" {} +
$ find * -type f
-a \! -name \*.patch \
-a \! -name \*_shipped \
-a \! -name LICENSE_GPL \
-a \! -name LGPL.txt \
-a \! -name COPYING \
-a \! -name DISCLAIMER \
-exec sed -i "/Foundation, Inc./ N;s:Foundation, Inc.* USA\.* *:Foundation, Inc. :;s:Foundation, Inc. $:Foundation, Inc.:" {} +
Change-Id: Icc968a5a5f3a5df8d32b940f9cdb35350654bef9
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-on: http://review.coreboot.org/9233
Tested-by: build bot (Jenkins)
Reviewed-by: Vladimir Serbinenko <phcoder@gmail.com>
2015-03-26 15:17:45 +01:00
|
|
|
* Foundation, Inc.
|
cbfstool: Restructure around support for reading/writing portions of files
The buffer API that cbfstool uses to read and write files only directly supports
one-shot operations on whole files. This adds an intermediate partitioned_file
module that sits on top of the buffer system and has an awareness of FMAP
entries. It provides an easy way to get a buffer for an individual region of a
larger image file based on FMAP section name, as well as incrementally write
those smaller buffers back to the backing file at the appropriate offset. The
module has two distinct modes of operation:
- For new images whose layout is described exclusively by an FMAP section, all
the aforementioned functionality will be available.
- For images in the current format, where the CBFS master header serves as the
root of knowledge of the image's size and layout, the module falls back to a
legacy operation mode, where it only allows manipulation of the entire image
as one unit, but exposes this support through the same interface by mapping
the region named SECTION_NAME_PRIMARY_CBFS ("COREBOOT") to the whole file.
The tool is presently only ported onto the new module running in legacy mode:
higher-level support for true "partitioned" images will be forthcoming. However,
as part of this change, the crusty cbfs_image_from_file() and
cbfs_image_write_file() abstractions are removed and replaced with a single
cbfs_image function, cbfs_image_from_buffer(), as well as centralized image
reading/writing directly in cbfstool's main() function. This reduces the
boilerplate required to implement each new action, makes the create action much
more similar to the others, and will make implementing additional actions and
adding in support for the new format much easier.
BUG=chromium:470407
TEST=Build panther and nyan_big coreboot.rom images with and without this patch
and diff their hexdumps. Ensure that no differences occur at different locations
from the diffs between subsequent builds of an identical source tree. Then flash
a full new build onto nyan_big and watch it boot normally.
BRANCH=None
Change-Id: I25578c7b223bc8434c3074cb0dd8894534f8c500
Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Commit-Id: 7e1c96a48e7a27fc6b90289d35e6e169d5e7ad20
Original-Change-Id: Ia4a1a4c48df42b9ec2d6b9471b3a10eb7b24bb39
Original-Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/265581
Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/10134
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
2015-03-25 21:40:08 +01:00
|
|
|
*/
|
|
|
|
|
|
|
|
#include "partitioned_file.h"
|
|
|
|
|
|
|
|
#include "cbfs_sections.h"
|
|
|
|
|
|
|
|
#include <assert.h>
|
|
|
|
#include <stdlib.h>
|
|
|
|
#include <string.h>
|
|
|
|
|
|
|
|
struct partitioned_file {
|
|
|
|
struct fmap *fmap;
|
|
|
|
struct buffer buffer;
|
|
|
|
FILE *stream;
|
|
|
|
};
|
|
|
|
|
|
|
|
static bool fill_ones_through(struct partitioned_file *file)
|
|
|
|
{
|
|
|
|
assert(file);
|
|
|
|
|
|
|
|
memset(file->buffer.data, 0xff, file->buffer.size);
|
|
|
|
return partitioned_file_write_region(file, &file->buffer);
|
|
|
|
}
|
|
|
|
|
|
|
|
static unsigned count_selected_fmap_entries(const struct fmap *fmap,
|
|
|
|
partitioned_file_fmap_selector_t callback, const void *arg)
|
|
|
|
{
|
|
|
|
assert(fmap);
|
|
|
|
assert(callback);
|
|
|
|
|
|
|
|
unsigned count = 0;
|
2015-05-15 08:14:09 +02:00
|
|
|
for (unsigned i = 0; i < fmap->nareas; ++i) {
|
|
|
|
if (callback(fmap->areas + i, arg))
|
cbfstool: Restructure around support for reading/writing portions of files
The buffer API that cbfstool uses to read and write files only directly supports
one-shot operations on whole files. This adds an intermediate partitioned_file
module that sits on top of the buffer system and has an awareness of FMAP
entries. It provides an easy way to get a buffer for an individual region of a
larger image file based on FMAP section name, as well as incrementally write
those smaller buffers back to the backing file at the appropriate offset. The
module has two distinct modes of operation:
- For new images whose layout is described exclusively by an FMAP section, all
the aforementioned functionality will be available.
- For images in the current format, where the CBFS master header serves as the
root of knowledge of the image's size and layout, the module falls back to a
legacy operation mode, where it only allows manipulation of the entire image
as one unit, but exposes this support through the same interface by mapping
the region named SECTION_NAME_PRIMARY_CBFS ("COREBOOT") to the whole file.
The tool is presently only ported onto the new module running in legacy mode:
higher-level support for true "partitioned" images will be forthcoming. However,
as part of this change, the crusty cbfs_image_from_file() and
cbfs_image_write_file() abstractions are removed and replaced with a single
cbfs_image function, cbfs_image_from_buffer(), as well as centralized image
reading/writing directly in cbfstool's main() function. This reduces the
boilerplate required to implement each new action, makes the create action much
more similar to the others, and will make implementing additional actions and
adding in support for the new format much easier.
BUG=chromium:470407
TEST=Build panther and nyan_big coreboot.rom images with and without this patch
and diff their hexdumps. Ensure that no differences occur at different locations
from the diffs between subsequent builds of an identical source tree. Then flash
a full new build onto nyan_big and watch it boot normally.
BRANCH=None
Change-Id: I25578c7b223bc8434c3074cb0dd8894534f8c500
Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Commit-Id: 7e1c96a48e7a27fc6b90289d35e6e169d5e7ad20
Original-Change-Id: Ia4a1a4c48df42b9ec2d6b9471b3a10eb7b24bb39
Original-Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/265581
Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/10134
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
2015-03-25 21:40:08 +01:00
|
|
|
++count;
|
|
|
|
}
|
|
|
|
return count;
|
|
|
|
}
|
|
|
|
|
|
|
|
static partitioned_file_t *reopen_flat_file(const char *filename)
|
|
|
|
{
|
|
|
|
assert(filename);
|
|
|
|
|
|
|
|
struct partitioned_file *file = calloc(1, sizeof(*file));
|
|
|
|
if (!file) {
|
|
|
|
ERROR("Failed to allocate partitioned file structure\n");
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (buffer_from_file(&file->buffer, filename)) {
|
|
|
|
free(file);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
file->stream = fopen(filename, "rb+");
|
|
|
|
if (!file->stream) {
|
|
|
|
perror(filename);
|
|
|
|
partitioned_file_close(file);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
return file;
|
|
|
|
}
|
|
|
|
|
|
|
|
partitioned_file_t *partitioned_file_create_flat(const char *filename,
|
|
|
|
size_t image_size)
|
|
|
|
{
|
|
|
|
assert(filename);
|
|
|
|
|
|
|
|
struct partitioned_file *file = calloc(1, sizeof(*file));
|
|
|
|
if (!file) {
|
|
|
|
ERROR("Failed to allocate partitioned file structure\n");
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
file->stream = fopen(filename, "wb");
|
|
|
|
if (!file->stream) {
|
|
|
|
perror(filename);
|
|
|
|
free(file);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (buffer_create(&file->buffer, image_size, filename)) {
|
|
|
|
partitioned_file_close(file);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!fill_ones_through(file)) {
|
|
|
|
partitioned_file_close(file);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
return file;
|
|
|
|
}
|
|
|
|
|
|
|
|
partitioned_file_t *partitioned_file_create(const char *filename,
|
|
|
|
struct buffer *flashmap)
|
|
|
|
{
|
|
|
|
assert(filename);
|
|
|
|
assert(flashmap);
|
|
|
|
assert(flashmap->data);
|
|
|
|
|
|
|
|
if (fmap_find((const uint8_t *)flashmap->data, flashmap->size) != 0) {
|
|
|
|
ERROR("Attempted to create a partitioned image out of something that isn't an FMAP\n");
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
struct fmap *bootstrap_fmap = (struct fmap *)flashmap->data;
|
|
|
|
|
|
|
|
const struct fmap_area *fmap_area =
|
|
|
|
fmap_find_area(bootstrap_fmap, SECTION_NAME_FMAP);
|
|
|
|
if (!fmap_area) {
|
|
|
|
ERROR("Provided FMAP missing '%s' region\n", SECTION_NAME_FMAP);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (count_selected_fmap_entries(bootstrap_fmap,
|
|
|
|
partitioned_file_fmap_select_children_of, fmap_area)) {
|
|
|
|
ERROR("Provided FMAP's '%s' region contains other regions\n",
|
|
|
|
SECTION_NAME_FMAP);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
int fmap_len = fmap_size(bootstrap_fmap);
|
|
|
|
if (fmap_len < 0) {
|
|
|
|
ERROR("Unable to determine size of provided FMAP\n");
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
assert((size_t)fmap_len <= flashmap->size);
|
|
|
|
if ((uint32_t)fmap_len > fmap_area->size) {
|
|
|
|
ERROR("Provided FMAP's '%s' region needs to be at least %d bytes\n",
|
|
|
|
SECTION_NAME_FMAP, fmap_len);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
partitioned_file_t *file = partitioned_file_create_flat(filename,
|
|
|
|
bootstrap_fmap->size);
|
|
|
|
if (!file)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
struct buffer fmap_region;
|
|
|
|
buffer_splice(&fmap_region, &file->buffer, fmap_area->offset, fmap_area->size);
|
|
|
|
memcpy(fmap_region.data, bootstrap_fmap, fmap_len);
|
|
|
|
if (!partitioned_file_write_region(file, &fmap_region)) {
|
|
|
|
partitioned_file_close(file);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
file->fmap = (struct fmap *)(file->buffer.data + fmap_area->offset);
|
|
|
|
|
|
|
|
return file;
|
|
|
|
}
|
|
|
|
|
2015-09-11 18:34:39 +02:00
|
|
|
partitioned_file_t *partitioned_file_reopen(const char *filename)
|
cbfstool: Restructure around support for reading/writing portions of files
The buffer API that cbfstool uses to read and write files only directly supports
one-shot operations on whole files. This adds an intermediate partitioned_file
module that sits on top of the buffer system and has an awareness of FMAP
entries. It provides an easy way to get a buffer for an individual region of a
larger image file based on FMAP section name, as well as incrementally write
those smaller buffers back to the backing file at the appropriate offset. The
module has two distinct modes of operation:
- For new images whose layout is described exclusively by an FMAP section, all
the aforementioned functionality will be available.
- For images in the current format, where the CBFS master header serves as the
root of knowledge of the image's size and layout, the module falls back to a
legacy operation mode, where it only allows manipulation of the entire image
as one unit, but exposes this support through the same interface by mapping
the region named SECTION_NAME_PRIMARY_CBFS ("COREBOOT") to the whole file.
The tool is presently only ported onto the new module running in legacy mode:
higher-level support for true "partitioned" images will be forthcoming. However,
as part of this change, the crusty cbfs_image_from_file() and
cbfs_image_write_file() abstractions are removed and replaced with a single
cbfs_image function, cbfs_image_from_buffer(), as well as centralized image
reading/writing directly in cbfstool's main() function. This reduces the
boilerplate required to implement each new action, makes the create action much
more similar to the others, and will make implementing additional actions and
adding in support for the new format much easier.
BUG=chromium:470407
TEST=Build panther and nyan_big coreboot.rom images with and without this patch
and diff their hexdumps. Ensure that no differences occur at different locations
from the diffs between subsequent builds of an identical source tree. Then flash
a full new build onto nyan_big and watch it boot normally.
BRANCH=None
Change-Id: I25578c7b223bc8434c3074cb0dd8894534f8c500
Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Commit-Id: 7e1c96a48e7a27fc6b90289d35e6e169d5e7ad20
Original-Change-Id: Ia4a1a4c48df42b9ec2d6b9471b3a10eb7b24bb39
Original-Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/265581
Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/10134
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
2015-03-25 21:40:08 +01:00
|
|
|
{
|
|
|
|
assert(filename);
|
|
|
|
|
|
|
|
partitioned_file_t *file = reopen_flat_file(filename);
|
|
|
|
if (!file)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
long fmap_region_offset = fmap_find((const uint8_t *)file->buffer.data,
|
|
|
|
file->buffer.size);
|
|
|
|
if (fmap_region_offset < 0) {
|
|
|
|
INFO("Opening image as a flat file because it doesn't contain any FMAP\n");
|
|
|
|
return file;
|
|
|
|
}
|
|
|
|
file->fmap = (struct fmap *)(file->buffer.data + fmap_region_offset);
|
|
|
|
|
|
|
|
if (file->fmap->size > file->buffer.size) {
|
|
|
|
int fmap_region_size = fmap_size(file->fmap);
|
|
|
|
ERROR("FMAP records image size as %u, but file is only %zu bytes%s\n",
|
|
|
|
file->fmap->size, file->buffer.size,
|
|
|
|
fmap_region_offset == 0 &&
|
|
|
|
(signed)file->buffer.size == fmap_region_size ?
|
|
|
|
" (is it really an image, or *just* an FMAP?)" :
|
|
|
|
" (did something truncate this file?)");
|
|
|
|
partitioned_file_close(file);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
const struct fmap_area *fmap_fmap_entry =
|
|
|
|
fmap_find_area(file->fmap, SECTION_NAME_FMAP);
|
2015-05-09 08:36:28 +02:00
|
|
|
if ((long)fmap_fmap_entry->offset != fmap_region_offset) {
|
cbfstool: Restructure around support for reading/writing portions of files
The buffer API that cbfstool uses to read and write files only directly supports
one-shot operations on whole files. This adds an intermediate partitioned_file
module that sits on top of the buffer system and has an awareness of FMAP
entries. It provides an easy way to get a buffer for an individual region of a
larger image file based on FMAP section name, as well as incrementally write
those smaller buffers back to the backing file at the appropriate offset. The
module has two distinct modes of operation:
- For new images whose layout is described exclusively by an FMAP section, all
the aforementioned functionality will be available.
- For images in the current format, where the CBFS master header serves as the
root of knowledge of the image's size and layout, the module falls back to a
legacy operation mode, where it only allows manipulation of the entire image
as one unit, but exposes this support through the same interface by mapping
the region named SECTION_NAME_PRIMARY_CBFS ("COREBOOT") to the whole file.
The tool is presently only ported onto the new module running in legacy mode:
higher-level support for true "partitioned" images will be forthcoming. However,
as part of this change, the crusty cbfs_image_from_file() and
cbfs_image_write_file() abstractions are removed and replaced with a single
cbfs_image function, cbfs_image_from_buffer(), as well as centralized image
reading/writing directly in cbfstool's main() function. This reduces the
boilerplate required to implement each new action, makes the create action much
more similar to the others, and will make implementing additional actions and
adding in support for the new format much easier.
BUG=chromium:470407
TEST=Build panther and nyan_big coreboot.rom images with and without this patch
and diff their hexdumps. Ensure that no differences occur at different locations
from the diffs between subsequent builds of an identical source tree. Then flash
a full new build onto nyan_big and watch it boot normally.
BRANCH=None
Change-Id: I25578c7b223bc8434c3074cb0dd8894534f8c500
Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Commit-Id: 7e1c96a48e7a27fc6b90289d35e6e169d5e7ad20
Original-Change-Id: Ia4a1a4c48df42b9ec2d6b9471b3a10eb7b24bb39
Original-Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/265581
Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/10134
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
2015-03-25 21:40:08 +01:00
|
|
|
ERROR("FMAP's '%s' section doesn't point back to FMAP start (did something corrupt this file?)\n",
|
|
|
|
SECTION_NAME_FMAP);
|
|
|
|
partitioned_file_close(file);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
return file;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool partitioned_file_write_region(partitioned_file_t *file,
|
|
|
|
const struct buffer *buffer)
|
|
|
|
{
|
|
|
|
assert(file);
|
|
|
|
assert(file->stream);
|
|
|
|
assert(buffer);
|
|
|
|
assert(buffer->data);
|
|
|
|
|
|
|
|
if (buffer->data - buffer->offset != file->buffer.data) {
|
|
|
|
ERROR("Attempted to write a partition buffer back to a different file than it came from\n");
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
if (buffer->offset + buffer->size > file->buffer.size) {
|
|
|
|
ERROR("Attempted to write data off the end of image file\n");
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (fseek(file->stream, buffer->offset, SEEK_SET)) {
|
|
|
|
ERROR("Failed to seek within image file\n");
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
if (!fwrite(buffer->data, buffer->size, 1, file->stream)) {
|
|
|
|
ERROR("Failed to write to image file\n");
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool partitioned_file_read_region(struct buffer *dest,
|
|
|
|
const partitioned_file_t *file, const char *region)
|
|
|
|
{
|
|
|
|
assert(dest);
|
|
|
|
assert(file);
|
|
|
|
assert(file->buffer.data);
|
|
|
|
assert(region);
|
|
|
|
|
|
|
|
if (file->fmap) {
|
|
|
|
const struct fmap_area *area = fmap_find_area(file->fmap,
|
|
|
|
region);
|
|
|
|
if (!area) {
|
|
|
|
ERROR("Image is missing '%s' region\n", region);
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
if (area->offset + area->size > file->buffer.size) {
|
|
|
|
ERROR("Region '%s' runs off the end of the image file\n",
|
|
|
|
region);
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
buffer_splice(dest, &file->buffer, area->offset, area->size);
|
|
|
|
} else {
|
|
|
|
if (strcmp(region, SECTION_NAME_PRIMARY_CBFS) != 0) {
|
|
|
|
ERROR("This is a legacy image that contains only a CBFS\n");
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
buffer_clone(dest, &file->buffer);
|
|
|
|
}
|
|
|
|
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
void partitioned_file_close(partitioned_file_t *file)
|
|
|
|
{
|
|
|
|
if (!file)
|
|
|
|
return;
|
|
|
|
|
|
|
|
file->fmap = NULL;
|
|
|
|
buffer_delete(&file->buffer);
|
|
|
|
if (file->stream) {
|
|
|
|
fclose(file->stream);
|
|
|
|
file->stream = NULL;
|
|
|
|
}
|
|
|
|
free(file);
|
|
|
|
}
|
|
|
|
|
|
|
|
bool partitioned_file_is_partitioned(const partitioned_file_t *file)
|
|
|
|
{
|
|
|
|
return partitioned_file_get_fmap(file) != NULL;
|
|
|
|
}
|
|
|
|
|
2015-05-07 11:39:22 +02:00
|
|
|
size_t partitioned_file_total_size(const partitioned_file_t *file)
|
|
|
|
{
|
|
|
|
assert(file);
|
|
|
|
|
|
|
|
return file->buffer.size;
|
|
|
|
}
|
|
|
|
|
cbfstool: Restructure around support for reading/writing portions of files
The buffer API that cbfstool uses to read and write files only directly supports
one-shot operations on whole files. This adds an intermediate partitioned_file
module that sits on top of the buffer system and has an awareness of FMAP
entries. It provides an easy way to get a buffer for an individual region of a
larger image file based on FMAP section name, as well as incrementally write
those smaller buffers back to the backing file at the appropriate offset. The
module has two distinct modes of operation:
- For new images whose layout is described exclusively by an FMAP section, all
the aforementioned functionality will be available.
- For images in the current format, where the CBFS master header serves as the
root of knowledge of the image's size and layout, the module falls back to a
legacy operation mode, where it only allows manipulation of the entire image
as one unit, but exposes this support through the same interface by mapping
the region named SECTION_NAME_PRIMARY_CBFS ("COREBOOT") to the whole file.
The tool is presently only ported onto the new module running in legacy mode:
higher-level support for true "partitioned" images will be forthcoming. However,
as part of this change, the crusty cbfs_image_from_file() and
cbfs_image_write_file() abstractions are removed and replaced with a single
cbfs_image function, cbfs_image_from_buffer(), as well as centralized image
reading/writing directly in cbfstool's main() function. This reduces the
boilerplate required to implement each new action, makes the create action much
more similar to the others, and will make implementing additional actions and
adding in support for the new format much easier.
BUG=chromium:470407
TEST=Build panther and nyan_big coreboot.rom images with and without this patch
and diff their hexdumps. Ensure that no differences occur at different locations
from the diffs between subsequent builds of an identical source tree. Then flash
a full new build onto nyan_big and watch it boot normally.
BRANCH=None
Change-Id: I25578c7b223bc8434c3074cb0dd8894534f8c500
Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Commit-Id: 7e1c96a48e7a27fc6b90289d35e6e169d5e7ad20
Original-Change-Id: Ia4a1a4c48df42b9ec2d6b9471b3a10eb7b24bb39
Original-Signed-off-by: Sol Boucher <solb@chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/265581
Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/10134
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
2015-03-25 21:40:08 +01:00
|
|
|
bool partitioned_file_region_check_magic(const partitioned_file_t *file,
|
|
|
|
const char *region, const char *magic, size_t magic_len)
|
|
|
|
{
|
|
|
|
struct buffer area;
|
|
|
|
return partitioned_file_read_region(&area, file, region) &&
|
|
|
|
buffer_check_magic(&area, magic, magic_len);
|
|
|
|
}
|
|
|
|
|
|
|
|
bool partitioned_file_region_contains_nested(const partitioned_file_t *file,
|
|
|
|
const char *region)
|
|
|
|
{
|
|
|
|
assert(file);
|
|
|
|
assert(region);
|
|
|
|
|
|
|
|
if (!file->fmap)
|
|
|
|
return false;
|
|
|
|
const struct fmap_area *area = fmap_find_area(file->fmap, region);
|
|
|
|
return area && partitioned_file_fmap_count(file,
|
|
|
|
partitioned_file_fmap_select_children_of, area);
|
|
|
|
}
|
|
|
|
|
|
|
|
const struct fmap *partitioned_file_get_fmap(const partitioned_file_t *file)
|
|
|
|
{
|
|
|
|
assert(file);
|
|
|
|
|
|
|
|
return file->fmap;
|
|
|
|
}
|
|
|
|
|
|
|
|
unsigned partitioned_file_fmap_count(const partitioned_file_t *file,
|
|
|
|
partitioned_file_fmap_selector_t callback, const void *arg)
|
|
|
|
{
|
|
|
|
assert(file);
|
|
|
|
assert(callback);
|
|
|
|
|
|
|
|
if (!file->fmap)
|
|
|
|
return 0;
|
|
|
|
return count_selected_fmap_entries(file->fmap, callback, arg);
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool select_all(unused const struct fmap_area *area,
|
|
|
|
unused const void *arg)
|
|
|
|
{
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
const partitioned_file_fmap_selector_t partitioned_file_fmap_select_all =
|
|
|
|
select_all;
|
|
|
|
|
|
|
|
static bool select_children_of(const struct fmap_area *child, const void *arg)
|
|
|
|
{
|
|
|
|
assert(child);
|
|
|
|
assert(arg);
|
|
|
|
|
|
|
|
const struct fmap_area *parent = (const struct fmap_area *)arg;
|
|
|
|
if (child == arg || (child->offset == parent->offset &&
|
|
|
|
child->size == parent->size))
|
|
|
|
return false;
|
|
|
|
return child->offset >= parent->offset &&
|
|
|
|
child->offset + child->size <= parent->offset + parent->size;
|
|
|
|
}
|
|
|
|
const partitioned_file_fmap_selector_t
|
|
|
|
partitioned_file_fmap_select_children_of = select_children_of;
|
|
|
|
|
|
|
|
static bool select_parents_of(const struct fmap_area *parent, const void *arg)
|
|
|
|
{
|
|
|
|
return select_children_of((const struct fmap_area *)arg, parent);
|
|
|
|
}
|
|
|
|
const partitioned_file_fmap_selector_t partitioned_file_fmap_select_parents_of =
|
|
|
|
select_parents_of;
|