Attention is currently required from: Paul Fagerburg, Julius Werner, Jan Dabros.
4 comments:
File tests/lib/bootmem-test.c:
Patch Set #2, Line 171: static int setup_bootmem_walk(void **state)
This is because in my concept I would need to use non-constant initializers (linker symbols), which […]
Done. Only required regions symbols are set now from inside of the test source.
struct lb_memory *lb_mem;
will_return_always(search_global_resources, &mem_device_mock);
/* Allocate space for 5 lb_mem entries to be safe */
lb_mem = malloc(sizeof(*lb_mem) + 5 * sizeof(struct lb_memory_range));
/* We need to call this only to initialize library */
bootmem_write_memory_table(lb_mem);
free(lb_mem);
Makes sense.
Done. Less code, more readability.
To be fixed.
Done
File tests/lib/bootmem-test.ld:
Nice trick with using asm() here. […]
File removed. Symbol setup moved to the test source file.
To view, visit change 43510. To unsubscribe, or for help writing mail filters, visit settings.