Skip to content

Commit

Permalink
llext: Remove simple from llext test
Browse files Browse the repository at this point in the history
The simple path and naming wasn't really right, there are no other tests
and the test suite has become quite automated and extensable to cover
most things we'd like to test with llext. Building, loading, and linking
of course.

Signed-off-by: Tom Burdick <thomas.burdick@intel.com>
  • Loading branch information
teburd authored and kartben committed Jan 2, 2025
1 parent bc5aff3 commit 73f1156
Show file tree
Hide file tree
Showing 28 changed files with 11 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@

cmake_minimum_required(VERSION 3.20.0)
find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})
project(llext_simple_test)
project(llext_test)

target_sources(app PRIVATE
src/test_llext_simple.c
src/test_llext.c
)

target_include_directories(app PRIVATE
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
#include "threads_kernel_objects_ext.h"


LOG_MODULE_REGISTER(test_llext_simple);
LOG_MODULE_REGISTER(test_llext);


#ifdef CONFIG_LLEXT_STORAGE_WRITABLE
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,14 +24,14 @@ tests:
# without actually running it to load some extension, let's keep it in
# good shape and ready to be used by additional architectures in the
# future.
llext.simple.loader_build:
llext.loader_build:
build_only: true

# Run the suite with all combinations of core Kconfig options for the LLEXT
# subsystem (storage type, ELF type, MPU/MMU etc). To focus on LLEXT issues,
# most tests include no_mem_protection.conf, which disables memory protection
# hardware completely.
llext.simple.readonly:
llext.readonly:
arch_allow: # Xtensa needs writable storage
- arm
- riscv
Expand All @@ -40,14 +40,14 @@ tests:
extra_conf_files: ['no_mem_protection.conf']
extra_configs:
- CONFIG_LLEXT_STORAGE_WRITABLE=n
llext.simple.readonly_mpu:
llext.readonly_mpu:
min_ram: 128
arch_allow: arm # Xtensa needs writable storage, currently not supported on RISC-V
filter: CONFIG_ARCH_HAS_USERSPACE
extra_configs:
- CONFIG_USERSPACE=y
- CONFIG_LLEXT_STORAGE_WRITABLE=n
llext.simple.readonly_mmu:
llext.readonly_mmu:
arch_allow:
- arm64
- arm
Expand All @@ -58,7 +58,7 @@ tests:
extra_configs:
- CONFIG_LLEXT_HEAP_SIZE=128 # qemu_cortex_a9 requires larger heap
- CONFIG_LLEXT_STORAGE_WRITABLE=n
llext.simple.writable:
llext.writable:
arch_allow:
- arm
- xtensa
Expand All @@ -70,7 +70,7 @@ tests:
extra_conf_files: ['no_mem_protection.conf']
extra_configs:
- CONFIG_LLEXT_STORAGE_WRITABLE=y
llext.simple.writable_relocatable:
llext.writable_relocatable:
arch_allow:
- arm
- xtensa
Expand All @@ -91,7 +91,7 @@ tests:

# Test the Symbol Link Identifier (SLID) linking feature on writable
# storage to cover both ARM and Xtensa architectures on the same test.
llext.simple.writable_slid_linking:
llext.writable_slid_linking:
arch_allow:
- arm
- xtensa
Expand All @@ -104,7 +104,7 @@ tests:
extra_configs:
- CONFIG_LLEXT_STORAGE_WRITABLE=y
- CONFIG_LLEXT_EXPORT_BUILTINS_BY_SLID=y
llext.simple.writable_relocatable_slid_linking:
llext.writable_relocatable_slid_linking:
arch_allow:
- arm
- xtensa
Expand Down

0 comments on commit 73f1156

Please sign in to comment.