[PATCH V2 1/3] discover/device-handler: Make boot_status() public

Samuel Mendoza-Jonas sam at mendozajonas.com
Tue Jun 21 13:21:37 AEST 2016


Rename boot_status() to device_handler_boot_status() and make it
accessible by files that include device-handler.h. This enables the boot
status to be updated from additional callers, in particular within
parser functions.

Signed-off-by: Samuel Mendoza-Jonas <sam at mendozajonas.com>
---
 discover/device-handler.c | 21 +++++++++++----------
 discover/device-handler.h |  3 +++
 2 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/discover/device-handler.c b/discover/device-handler.c
index cc48747..1c69de0 100644
--- a/discover/device-handler.c
+++ b/discover/device-handler.c
@@ -396,7 +396,7 @@ void device_handler_remove(struct device_handler *handler,
 	talloc_free(device);
 }
 
-static void boot_status(void *arg, struct boot_status *status)
+void device_handler_boot_status(void *arg, struct boot_status *status)
 {
 	struct device_handler *handler = arg;
 
@@ -448,7 +448,8 @@ static int default_timeout(void *arg)
 	platform_pre_boot();
 
 	handler->pending_boot = boot(handler, handler->default_boot_option,
-			NULL, handler->dry_run, boot_status, handler);
+			NULL, handler->dry_run, device_handler_boot_status,
+			handler);
 	handler->pending_boot_is_default = true;
 	return 0;
 }
@@ -873,7 +874,7 @@ int device_handler_discover(struct device_handler *handler,
 	status->message = talloc_asprintf(status, _("Processing %s device %s"),
 				device_type_display_name(dev->device->type),
 				dev->device->id);
-	boot_status(handler, status);
+	device_handler_boot_status(handler, status);
 
 	process_boot_option_queue(handler);
 
@@ -901,7 +902,7 @@ out:
 	 */
 	status->message = talloc_asprintf(status,_("Processing %s complete"),
 				dev->device->id);
-	boot_status(handler, status);
+	device_handler_boot_status(handler, status);
 
 	talloc_free(status);
 	talloc_free(ctx);
@@ -924,7 +925,7 @@ int device_handler_dhcp(struct device_handler *handler,
 	 */
 	status->message = talloc_asprintf(status, _("Processing dhcp event on %s"),
 				dev->device->id);
-	boot_status(handler, status);
+	device_handler_boot_status(handler, status);
 
 	/* create our context */
 	ctx = device_handler_discover_context_create(handler, dev);
@@ -940,7 +941,7 @@ int device_handler_dhcp(struct device_handler *handler,
 	 */
 	status->message = talloc_asprintf(status,_("Processing %s complete"),
 				dev->device->id);
-	boot_status(handler, status);
+	device_handler_boot_status(handler, status);
 
 	talloc_free(status);
 	talloc_free(ctx);
@@ -958,7 +959,7 @@ int device_handler_conf(struct device_handler *handler,
 	status = talloc_zero(handler, struct boot_status);
 	status->type = BOOT_STATUS_INFO;
 	status->message = talloc_asprintf(status, _("Processing user config"));
-	boot_status(handler, status);
+	device_handler_boot_status(handler, status);
 
 	/* create our context */
 	ctx = device_handler_discover_context_create(handler, dev);
@@ -970,7 +971,7 @@ int device_handler_conf(struct device_handler *handler,
 
 	status->message = talloc_asprintf(status,
 				_("Processing user config complete"));
-	boot_status(handler, status);
+	device_handler_boot_status(handler, status);
 
 	talloc_free(status);
 	talloc_free(ctx);
@@ -1009,7 +1010,7 @@ void device_handler_boot(struct device_handler *handler,
 	platform_pre_boot();
 
 	handler->pending_boot = boot(handler, opt, cmd, handler->dry_run,
-			boot_status, handler);
+			device_handler_boot_status, handler);
 	handler->pending_boot_is_default = false;
 }
 
@@ -1211,7 +1212,7 @@ void device_handler_process_url(struct device_handler *handler,
 	status->message = talloc_asprintf(status, _("Config file %s parsed"),
 					pb_url->file);
 msg:
-	boot_status(handler, status);
+	device_handler_boot_status(handler, status);
 	talloc_free(status);
 }
 
diff --git a/discover/device-handler.h b/discover/device-handler.h
index 12f5ce0..6b1a757 100644
--- a/discover/device-handler.h
+++ b/discover/device-handler.h
@@ -4,6 +4,7 @@
 #include <stdbool.h>
 
 #include <list/list.h>
+#include <types/types.h>
 
 struct device_handler;
 struct discover_device;
@@ -96,6 +97,8 @@ int device_handler_conf(struct device_handler *handler,
 void device_handler_remove(struct device_handler *handler,
 		struct discover_device *device);
 
+void device_handler_boot_status(void *arg, struct boot_status *status);
+
 struct discover_context *device_handler_discover_context_create(
 		struct device_handler *handler,
 		struct discover_device *device);
-- 
2.9.0



More information about the Petitboot mailing list