Message ID | 20180808095433.230882-3-sjg@chromium.org |
---|---|
State | Superseded, archived |
Delegated to: | Alexander Graf |
Headers | show |
Series | efi: Enable sandbox support for EFI loader | expand |
On 08.08.18 11:54, Simon Glass wrote: > This does not work at present and gives the following error: > > output: 'ld.bfd: read in flex scanner failed > scripts/Makefile.lib:390: recipe for target 'lib/efi_selftest/efi_selftest_miniapp_return_efi.so' failed > > It may be possible to figure this out with suitable linker magic but it > does not seem to be easy. Also, we will be able to run the tests on > sandbox without using the miniapp. > > So for now at least, disable this option. > > Signed-off-by: Simon Glass <sjg@chromium.org> Didn't Heinrich have some magic for this in the works? Alex > --- > > Changes in v9: None > Changes in v8: None > Changes in v7: None > Changes in v6: None > Changes in v5: > - Add new patch to disallow CMD_BOOTEFI_SELFTEST on sandbox > > Changes in v4: None > Changes in v3: None > Changes in v2: None > > lib/efi_selftest/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/efi_selftest/Kconfig b/lib/efi_selftest/Kconfig > index 59f9f36801c..b52696778dd 100644 > --- a/lib/efi_selftest/Kconfig > +++ b/lib/efi_selftest/Kconfig > @@ -1,6 +1,6 @@ > config CMD_BOOTEFI_SELFTEST > bool "Allow booting an EFI efi_selftest" > - depends on CMD_BOOTEFI > + depends on CMD_BOOTEFI && !SANDBOX > imply FAT > imply FAT_WRITE > help >
Hi Alex, On 26 August 2018 at 18:53, Alexander Graf <agraf@suse.de> wrote: > > > On 08.08.18 11:54, Simon Glass wrote: >> This does not work at present and gives the following error: >> >> output: 'ld.bfd: read in flex scanner failed >> scripts/Makefile.lib:390: recipe for target 'lib/efi_selftest/efi_selftest_miniapp_return_efi.so' failed >> >> It may be possible to figure this out with suitable linker magic but it >> does not seem to be easy. Also, we will be able to run the tests on >> sandbox without using the miniapp. >> >> So for now at least, disable this option. >> >> Signed-off-by: Simon Glass <sjg@chromium.org> > > Didn't Heinrich have some magic for this in the works? Yes I'll rewrite the commit message for this and enable it in the last patch of the series. Regards, Simon
diff --git a/lib/efi_selftest/Kconfig b/lib/efi_selftest/Kconfig index 59f9f36801c..b52696778dd 100644 --- a/lib/efi_selftest/Kconfig +++ b/lib/efi_selftest/Kconfig @@ -1,6 +1,6 @@ config CMD_BOOTEFI_SELFTEST bool "Allow booting an EFI efi_selftest" - depends on CMD_BOOTEFI + depends on CMD_BOOTEFI && !SANDBOX imply FAT imply FAT_WRITE help
This does not work at present and gives the following error: output: 'ld.bfd: read in flex scanner failed scripts/Makefile.lib:390: recipe for target 'lib/efi_selftest/efi_selftest_miniapp_return_efi.so' failed It may be possible to figure this out with suitable linker magic but it does not seem to be easy. Also, we will be able to run the tests on sandbox without using the miniapp. So for now at least, disable this option. Signed-off-by: Simon Glass <sjg@chromium.org> --- Changes in v9: None Changes in v8: None Changes in v7: None Changes in v6: None Changes in v5: - Add new patch to disallow CMD_BOOTEFI_SELFTEST on sandbox Changes in v4: None Changes in v3: None Changes in v2: None lib/efi_selftest/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)