mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] boot: support running scripts when doing a dry run
@ 2023-08-03 13:52 Ahmad Fatoum
  2023-08-14 13:09 ` Sascha Hauer
  0 siblings, 1 reply; 2+ messages in thread
From: Ahmad Fatoum @ 2023-08-03 13:52 UTC (permalink / raw)
  To: barebox; +Cc: Ahmad Fatoum

Dry runs are quite useful for debugging or for time measurement, but so
far they weren't useful with scripts as they just did a very early exit.

This is understandable as scripts don't necessarily observe the
convention that everything done before the dry run should not have an
impact on later boot runs. For the cases, where the bootscript is vetted
to behave properly under a dry run, support specifing dryrun twice, so
the script is executed, but the automatic bootm afterwards isn't.

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
 commands/boot.c | 4 ++--
 common/boot.c   | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/commands/boot.c b/commands/boot.c
index 5fd59f8642f8..e4699520e8f5 100644
--- a/commands/boot.c
+++ b/commands/boot.c
@@ -44,7 +44,7 @@ static int do_boot(int argc, char *argv[])
 			do_list = 1;
 			break;
 		case 'd':
-			dryrun = 1;
+			dryrun++;
 			break;
 		case 'M':
 			/* To simplify scripting, an empty string is treated as 1 */
@@ -145,7 +145,7 @@ BAREBOX_CMD_HELP_TEXT("one succeeds.")
 BAREBOX_CMD_HELP_TEXT("")
 BAREBOX_CMD_HELP_TEXT("Options:")
 BAREBOX_CMD_HELP_OPT ("-v","Increase verbosity")
-BAREBOX_CMD_HELP_OPT ("-d","Dryrun. See what happens but do no actually boot")
+BAREBOX_CMD_HELP_OPT ("-d","Dryrun. See what happens but do no actually boot (pass twice to run scripts)")
 BAREBOX_CMD_HELP_OPT ("-l","List available boot sources")
 BAREBOX_CMD_HELP_OPT ("-m","Show a menu with boot options")
 BAREBOX_CMD_HELP_OPT ("-M INDEX","Show a menu with boot options with entry INDEX preselected")
diff --git a/common/boot.c b/common/boot.c
index 4edea682219b..76bf52b52902 100644
--- a/common/boot.c
+++ b/common/boot.c
@@ -75,7 +75,7 @@ static int bootscript_boot(struct bootentry *entry, int verbose, int dryrun)
 
 	struct bootm_data data = {};
 
-	if (dryrun) {
+	if (dryrun == 1) {
 		printf("Would run %s\n", bs->scriptpath);
 		return 0;
 	}
@@ -94,8 +94,8 @@ static int bootscript_boot(struct bootentry *entry, int verbose, int dryrun)
 
 	if (verbose)
 		data.verbose = verbose;
-	if (dryrun)
-		data.dryrun = dryrun;
+	if (dryrun >= 2)
+		data.dryrun = dryrun - 1;
 
 	return bootm_boot(&data);
 }
-- 
2.39.2




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-08-14 13:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-03 13:52 [PATCH] boot: support running scripts when doing a dry run Ahmad Fatoum
2023-08-14 13:09 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox