[PATCH 2/3] discover: Be more verbose about boot failures

Samuel Mendoza-Jonas sam at mendozajonas.com
Mon Jul 30 11:48:25 AEST 2018


Signed-off-by: Samuel Mendoza-Jonas <sam at mendozajonas.com>
---
 discover/boot.c  | 6 ++++++
 discover/paths.c | 2 ++
 2 files changed, 8 insertions(+)

diff --git a/discover/boot.c b/discover/boot.c
index 2a0d333..04c32a1 100644
--- a/discover/boot.c
+++ b/discover/boot.c
@@ -341,6 +341,8 @@ static int check_load(struct boot_task *task, const char *name,
 		return 0;
 	}
 
+	pb_log("Failed to load %s from %s\n", name,
+			pb_url_to_string(result->url));
 	update_status(task->status_fn, task->status_arg,
 			STATUS_ERROR,
 			_("Couldn't load %s from %s"), name,
@@ -451,6 +453,8 @@ no_load:
 					STATUS_ERROR,
 					_("kexec reboot failed"));
 		}
+	} else {
+		pb_log("Failed to load all boot resources\n");
 	}
 }
 
@@ -462,6 +466,8 @@ static int start_url_load(struct boot_task *task, struct boot_resource *res)
 	res->result = load_url_async(task, res->url, boot_process,
 				 task, NULL, task->status_arg);
 	if (!res->result) {
+		pb_log("Error starting load for %s at %s\n",
+				res->name, pb_url_to_string(res->url));
 		update_status(task->status_fn, task->status_arg,
 				STATUS_ERROR, _("Error loading %s"),
 				res->name);
diff --git a/discover/paths.c b/discover/paths.c
index dd54ba2..dcd7b49 100644
--- a/discover/paths.c
+++ b/discover/paths.c
@@ -118,6 +118,8 @@ static void load_url_process_exit(struct process *process)
 	} else {
 		result->status = LOAD_ERROR;
 		load_url_result_cleanup_local(result);
+		pb_debug("Download client stdout buffer:\n%s\n",
+				process->stdout_buf);
 	}
 
 	if (result->status == LOAD_OK && process->stdout_data)
-- 
2.18.0



More information about the Petitboot mailing list