Provide a little bit more context behind some of the klp-build failure modes clarify which of the user-provided patches is unsupported, doesn't apply, and which kernel build failed.
Signed-off-by: Joe Lawrence <[email protected]> --- scripts/livepatch/klp-build | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/scripts/livepatch/klp-build b/scripts/livepatch/klp-build index df3a0fa031a6..f8ce456523fe 100755 --- a/scripts/livepatch/klp-build +++ b/scripts/livepatch/klp-build @@ -343,7 +343,7 @@ check_unsupported_patches() { for file in "${files[@]}"; do case "$file" in lib/*|*.S) - die "unsupported patch to $file" + die "$patch unsupported patch to $file" ;; esac done @@ -359,7 +359,7 @@ apply_patch() { cd "$SRC" # The sed strips the version signature from 'git format-patch'. sed -n '/^-- /q;p' "$patch" | \ - patch -p1 --no-backup-if-mismatch -r /dev/null + patch -p1 --no-backup-if-mismatch -r /dev/null || die "$patch doesn't apply" ) APPLIED_PATCHES+=("$patch") @@ -500,6 +500,7 @@ clean_kernel() { } build_kernel() { + local build="$1" local log="$TMP_DIR/build.log" local objtool_args=() local cmd=() @@ -538,7 +539,7 @@ build_kernel() { "${cmd[@]}" \ 1> >(tee -a "$log") \ 2> >(tee -a "$log" | grep0 -v "modpost.*undefined!" >&2) - ) + ) || die "$build kernel build failed" } find_objects() { @@ -815,7 +816,7 @@ fi if (( SHORT_CIRCUIT <= 1 )); then status "Building original kernel" clean_kernel - build_kernel + build_kernel "original" status "Copying original object files" copy_orig_objects fi @@ -825,7 +826,7 @@ if (( SHORT_CIRCUIT <= 2 )); then fix_patches apply_patches status "Building patched kernel" - build_kernel + build_kernel "patched" revert_patches status "Copying patched object files" copy_patched_objects -- 2.52.0
