Print error and exit if queueing capsule update fails. Mention that most
common case is enabled ME and print link to known issues describing it
in more details along with solution.
Modify cap_upd_tool mocking so we can test failures.
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
source and destination
will be usefull for unit testing of some conditions in romhole_migration
func.
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
Add mockup funtion for amdtool to simualte failures on intel platform.
This is to be consistant with hardware behavior.
Signed-off-by: Mateusz Kusiak <mateusz.kusiak@3mdeb.com>
Emulate the output of `lscpu` more accurately, displaying the actual
line printed with the string "Model name:" and the CPU model, rather
than only the CPU model itself.
Fix workflows, that rely on `$LSCPU | grep -F "Model name"`.
Reuse TEST_CPU_VERSION, rather than having to specify TEST_CPU_MODEL.
Signed-off-by: Kamil Aronowski <kamil.aronowski@3mdeb.com>
Fix futility mocking & add GBB region only if we don't already use
WP_RO otherwise flashrom will complain on write as those 2 regions
overlap
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock-func: set dmidecode to return 0 by default
if test variable isn't defined this likely means that this value
doesn't matter for test purposes.
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock-func: cbmem shouldn't exit in failure in normal check
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock-func: cbfstool: allow specifying fmap regions in binary
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
dts-functions: set_flashrom_update_params: don't use $1 directly
It's hard to know exactly what this argument is for otherwise as this
function isn't documented
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
deploy: don't use mktemp so profile arguments are always the same
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
dts-boot: create TEMP_DIR
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: cbfstool: return 1 if not coreboot
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
cbfstool: don't mock when we check update binary config
Except in some cases where we want to possibly test for identical
config between update binary and current firmware.
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
dasharo-deploy: add 2>>ERR_LOG_FILE redirection
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: cbmem: check if coreboot
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
dts-functions: download_keys: use 'mkdir -p' so it doesn't error out
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: add cbmem_common_mock which returns error if not coreboot
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: add msrtool and mei-amt-check common mock
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: ifdtool: call original ifdtool if used with update binary
Fixes situation on QEMU that results in user not being asked whether to
continue flashing without ME even though we want to test with ME enabled
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: flashrom: verify internal chip use
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: add bootsplash presence mocking
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: both flashrom and cbfstool allow writing over existing file
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: bootsplash region might exist without custom bootsplash
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
check_if_me_disabled: Don't use mocked functions with 'grep -q'
Otherwise grep will break pipe immediately after matching searched
string which will result in mocked function being immediately stopped
even before finishing. It also results in e.g. tool wrapper not writing
used command to profile
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: Add separate ME disabled and ME HAP disabled
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: add cbfstool smmstore mocking
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
common-mock: simplify fuse mock
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
dts-functions: check_if_boot_guard_enabled: hide rdmsr output
remove non-existent command, likely pasted by mistake
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
include: hal: add cbfstool_add_firmware_section_mock
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
use dont_mock instead of cbfstool_add_firmware_section_mock
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
common-mock: rename regions to _regions
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
goal
This functions should check only for payloads. We do have another
function called "check_if_dasharo" for checking if we have Dasharo
firmware flashed.
Change the function names to adhere to the "check_if_dasharo" function.
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
