Mirrors/u-boot
2
0
Fork 0
mirror of https://github.com/AsahiLinux/u-boot synced 2024-11-28 07:31:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

fdt: Align the start of the livetree

Browse source 
Ensure that the block of memory used by live tree is aligned according to
the default for structures. This ensures that the root node appears at
the start of the block, so it can be used with free(), rather than being
4 bytes later in some cases.

This corrects a rather obscure bug in unflatten_device_tree().

Fixes: 8b50d526ea ("dm: Add a function to create a 'live' device tree")

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2023-06-01 10:22:40 -06:00 committed by Tom Rini
parent 822f7a4543
commit 9cf39bbe96
3 changed files with 32 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
include/dm/of.h
View file

@ -63,6 +63,8 @@ struct device_node {
struct device_node *sibling;
};
#define BAD_OF_ROOT 0xdead11e3
#define OF_MAX_PHANDLE_ARGS 16
/**

5
lib/of_live.c
View file

@ -287,9 +287,12 @@ int unflatten_device_tree(const void *blob, struct device_node **mynodes)
debug(" size is %lx, allocating...\n", size);
/* Allocate memory for the expanded device tree */
mem = malloc(size + 4);
mem = memalign(__alignof__(struct device_node), size + 4);
memset(mem, '\0', size);
/* Set up value for dm_test_livetree_align() */
*(u32 *)mem = BAD_OF_ROOT;
*(__be32 *)(mem + size) = cpu_to_be32(0xdeadbeef);
debug(" unflattening %p...\n", mem);

26
test/dm/ofnode.c
View file

@ -1240,3 +1240,29 @@ static int dm_test_ofnode_copy_props_ot(struct unit_test_state *uts)
return 0;
}
DM_TEST(dm_test_ofnode_copy_props_ot, UT_TESTF_SCAN_FDT | UT_TESTF_OTHER_FDT);
/* check that the livetree is aligned to a structure boundary */
static int dm_test_livetree_align(struct unit_test_state *uts)
{
const int align = __alignof__(struct unit_test_state);
struct device_node *node;
u32 *sentinel;
ulong start;
start = (ulong)gd_of_root();
ut_asserteq(start, ALIGN(start, align));
node = gd_of_root();
sentinel = (void *)node - sizeof(u32);
/*
* The sentinel should be overwritten with the root node. If it isn't,
* then the root node is not at the very start of the livetree memory
* area, and free(root) will fail to free the memory used by the
* livetree.
*/
ut_assert(*sentinel != BAD_OF_ROOT);
return 0;
}
DM_TEST(dm_test_livetree_align, UT_TESTF_LIVE_TREE);