[PATCH 1/1] efi_loader: use '%#x' instead of '0x%x'
Ilias Apalodimas
ilias.apalodimas at linaro.org
Thu Nov 7 10:57:13 CET 2024
On Sat, 2 Nov 2024 at 07:07, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> We can save a few bytes by using '%#x' instead of '0x%x' in printf
> statements.
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
> lib/efi_loader/efi_boottime.c | 20 ++++++++++----------
> lib/efi_loader/efi_device_path_to_text.c | 20 ++++++++++----------
> lib/efi_loader/efi_hii_config.c | 2 +-
> lib/efi_loader/efi_image_loader.c | 10 +++++-----
> lib/efi_loader/efi_memory.c | 4 ++--
> lib/efi_loader/efi_signature.c | 2 +-
> lib/efi_loader/efi_variable_tee.c | 2 +-
> 7 files changed, 30 insertions(+), 30 deletions(-)
>
> diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c
> index 4f52284b4c6..01a5182b417 100644
> --- a/lib/efi_loader/efi_boottime.c
> +++ b/lib/efi_loader/efi_boottime.c
> @@ -372,7 +372,7 @@ static unsigned long EFIAPI efi_raise_tpl(efi_uintn_t new_tpl)
> {
> efi_uintn_t old_tpl = efi_tpl;
>
> - EFI_ENTRY("0x%zx", new_tpl);
> + EFI_ENTRY("%#zx", new_tpl);
>
> if (new_tpl < efi_tpl)
> EFI_PRINT("WARNING: new_tpl < current_tpl in %s\n", __func__);
> @@ -395,7 +395,7 @@ static unsigned long EFIAPI efi_raise_tpl(efi_uintn_t new_tpl)
> */
> static void EFIAPI efi_restore_tpl(efi_uintn_t old_tpl)
> {
> - EFI_ENTRY("0x%zx", old_tpl);
> + EFI_ENTRY("%#zx", old_tpl);
>
> if (old_tpl > efi_tpl)
> EFI_PRINT("WARNING: old_tpl > current_tpl in %s\n", __func__);
> @@ -431,7 +431,7 @@ static efi_status_t EFIAPI efi_allocate_pages_ext(int type, int memory_type,
> {
> efi_status_t r;
>
> - EFI_ENTRY("%d, %d, 0x%zx, %p", type, memory_type, pages, memory);
> + EFI_ENTRY("%d, %d, %#zx, %p", type, memory_type, pages, memory);
> r = efi_allocate_pages(type, memory_type, pages, memory);
> return EFI_EXIT(r);
> }
> @@ -453,7 +453,7 @@ static efi_status_t EFIAPI efi_free_pages_ext(uint64_t memory,
> {
> efi_status_t r;
>
> - EFI_ENTRY("%llx, 0x%zx", memory, pages);
> + EFI_ENTRY("%llx, %#zx", memory, pages);
> r = efi_free_pages(memory, pages);
> return EFI_EXIT(r);
> }
> @@ -797,7 +797,7 @@ efi_status_t EFIAPI efi_create_event_ex(uint32_t type, efi_uintn_t notify_tpl,
> {
> efi_status_t ret;
>
> - EFI_ENTRY("%d, 0x%zx, %p, %p, %pUs", type, notify_tpl, notify_function,
> + EFI_ENTRY("%d, %#zx, %p, %p, %pUs", type, notify_tpl, notify_function,
> notify_context, event_group);
>
> /*
> @@ -839,7 +839,7 @@ static efi_status_t EFIAPI efi_create_event_ext(
> void *context),
> void *notify_context, struct efi_event **event)
> {
> - EFI_ENTRY("%d, 0x%zx, %p, %p", type, notify_tpl, notify_function,
> + EFI_ENTRY("%d, %#zx, %p, %p", type, notify_tpl, notify_function,
> notify_context);
> return EFI_EXIT(efi_create_event(type, notify_tpl, notify_function,
> notify_context, NULL, event));
> @@ -2337,7 +2337,7 @@ static efi_status_t EFIAPI efi_set_watchdog_timer(unsigned long timeout,
> unsigned long data_size,
> uint16_t *watchdog_data)
> {
> - EFI_ENTRY("%ld, 0x%llx, %ld, %p", timeout, watchdog_code,
> + EFI_ENTRY("%ld, %#llx, %ld, %p", timeout, watchdog_code,
> data_size, watchdog_data);
> return EFI_EXIT(efi_set_watchdog(timeout));
> }
> @@ -2982,7 +2982,7 @@ static void EFIAPI efi_copy_mem(void *destination, const void *source,
> */
> static void EFIAPI efi_set_mem(void *buffer, size_t size, uint8_t value)
> {
> - EFI_ENTRY("%p, %ld, 0x%x", buffer, (unsigned long)size, value);
> + EFI_ENTRY("%p, %ld, %#x", buffer, (unsigned long)size, value);
> memset(buffer, value, size);
> EFI_EXIT(EFI_SUCCESS);
> }
> @@ -3123,7 +3123,7 @@ static efi_status_t EFIAPI efi_open_protocol
> struct efi_handler *handler;
> efi_status_t r = EFI_INVALID_PARAMETER;
>
> - EFI_ENTRY("%p, %pUs, %p, %p, %p, 0x%x", handle, protocol,
> + EFI_ENTRY("%p, %pUs, %p, %p, %p, %#x", handle, protocol,
> protocol_interface, agent_handle, controller_handle,
> attributes);
>
> @@ -3496,7 +3496,7 @@ static efi_status_t EFIAPI efi_exit(efi_handle_t image_handle,
> if (image_obj->image_type == IMAGE_SUBSYSTEM_EFI_APPLICATION) {
> ret = efi_tcg2_measure_efi_app_exit();
> if (ret != EFI_SUCCESS) {
> - log_warning("tcg2 measurement fails(0x%lx)\n",
> + log_warning("tcg2 measurement fails (%#lx)\n",
> ret);
> }
> }
> diff --git a/lib/efi_loader/efi_device_path_to_text.c b/lib/efi_loader/efi_device_path_to_text.c
> index 0c7b30a26e7..51732cc0a48 100644
> --- a/lib/efi_loader/efi_device_path_to_text.c
> +++ b/lib/efi_loader/efi_device_path_to_text.c
> @@ -53,7 +53,7 @@ static char *dp_hardware(char *s, struct efi_device_path *dp)
> case DEVICE_PATH_SUB_TYPE_MEMORY: {
> struct efi_device_path_memory *mdp =
> (struct efi_device_path_memory *)dp;
> - s += sprintf(s, "MemoryMapped(0x%x,0x%llx,0x%llx)",
> + s += sprintf(s, "MemoryMapped(%#x,%#llx,%#llx)",
> mdp->memory_type,
> mdp->start_address,
> mdp->end_address);
> @@ -79,7 +79,7 @@ static char *dp_hardware(char *s, struct efi_device_path *dp)
> struct efi_device_path_controller *cdp =
> (struct efi_device_path_controller *)dp;
>
> - s += sprintf(s, "Ctrl(0x%0x)", cdp->controller_number);
> + s += sprintf(s, "Ctrl(%#0x)", cdp->controller_number);
> break;
> }
> default:
> @@ -152,7 +152,7 @@ static char *dp_msging(char *s, struct efi_device_path *dp)
> case DEVICE_PATH_SUB_TYPE_MSG_USB: {
> struct efi_device_path_usb *udp =
> (struct efi_device_path_usb *)dp;
> - s += sprintf(s, "USB(0x%x,0x%x)", udp->parent_port_number,
> + s += sprintf(s, "USB(%#x,%#x)", udp->parent_port_number,
> udp->usb_interface);
> break;
> }
> @@ -174,7 +174,7 @@ static char *dp_msging(char *s, struct efi_device_path *dp)
> struct efi_device_path_usb_class *ucdp =
> (struct efi_device_path_usb_class *)dp;
>
> - s += sprintf(s, "UsbClass(0x%x,0x%x,0x%x,0x%x,0x%x)",
> + s += sprintf(s, "UsbClass(%#x,%#x,%#x,%#x,%#x)",
> ucdp->vendor_id, ucdp->product_id,
> ucdp->device_class, ucdp->device_subclass,
> ucdp->device_protocol);
> @@ -185,7 +185,7 @@ static char *dp_msging(char *s, struct efi_device_path *dp)
> struct efi_device_path_sata *sdp =
> (struct efi_device_path_sata *) dp;
>
> - s += sprintf(s, "Sata(0x%x,0x%x,0x%x)",
> + s += sprintf(s, "Sata(%#x,%#x,%#x)",
> sdp->hba_port,
> sdp->port_multiplier_port,
> sdp->logical_unit_number);
> @@ -197,7 +197,7 @@ static char *dp_msging(char *s, struct efi_device_path *dp)
> u32 ns_id;
>
> memcpy(&ns_id, &ndp->ns_id, sizeof(ns_id));
> - s += sprintf(s, "NVMe(0x%x,", ns_id);
> + s += sprintf(s, "NVMe(%#x,", ns_id);
>
> /* Display byte 7 first, byte 0 last */
> for (int i = 0; i < 8; ++i)
> @@ -264,18 +264,18 @@ static char *dp_media(char *s, struct efi_device_path *dp)
>
> memcpy(&signature, sig, sizeof(signature));
> s += sprintf(
> - s, "HD(%d,MBR,0x%08x,0x%llx,0x%llx)",
> + s, "HD(%d,MBR,%#08x,%#llx,%#llx)",
> hddp->partition_number, signature, start, end);
> break;
> }
> case SIG_TYPE_GUID:
> s += sprintf(
> - s, "HD(%d,GPT,%pUl,0x%llx,0x%llx)",
> + s, "HD(%d,GPT,%pUl,%#llx,%#llx)",
> hddp->partition_number, sig, start, end);
> break;
> default:
> s += sprintf(
> - s, "HD(%d,0x%02x,0,0x%llx,0x%llx)",
> + s, "HD(%d,%#02x,0,%#llx,%#llx)",
> hddp->partition_number, hddp->partmap_type,
> start, end);
> break;
> @@ -286,7 +286,7 @@ static char *dp_media(char *s, struct efi_device_path *dp)
> case DEVICE_PATH_SUB_TYPE_CDROM_PATH: {
> struct efi_device_path_cdrom_path *cddp =
> (struct efi_device_path_cdrom_path *)dp;
> - s += sprintf(s, "CDROM(%u,0x%llx,0x%llx)", cddp->boot_entry,
> + s += sprintf(s, "CDROM(%u,%#llx,%#llx)", cddp->boot_entry,
> cddp->partition_start, cddp->partition_size);
> break;
> }
> diff --git a/lib/efi_loader/efi_hii_config.c b/lib/efi_loader/efi_hii_config.c
> index ae0f3ecd3b1..0c8e2445d7a 100644
> --- a/lib/efi_loader/efi_hii_config.c
> +++ b/lib/efi_loader/efi_hii_config.c
> @@ -127,7 +127,7 @@ form_callback(const struct efi_hii_config_access_protocol *this,
> union efi_ifr_type_value *value,
> efi_browser_action_request_t *action_request)
> {
> - EFI_ENTRY("%p, 0x%zx, 0x%x, 0x%x, %p, %p", this, action,
> + EFI_ENTRY("%p, %#zx, %#x, %#x, %p, %p", this, action,
> question_id, type, value, action_request);
>
> return EFI_EXIT(EFI_DEVICE_ERROR);
> diff --git a/lib/efi_loader/efi_image_loader.c b/lib/efi_loader/efi_image_loader.c
> index 0ddf69a0918..2cea7a81849 100644
> --- a/lib/efi_loader/efi_image_loader.c
> +++ b/lib/efi_loader/efi_image_loader.c
> @@ -72,7 +72,7 @@ static efi_status_t efi_print_image_info(struct efi_loaded_image_obj *obj,
> image->image_base, image->image_base + image->image_size - 1);
> if (pc && pc >= image->image_base &&
> pc < image->image_base + image->image_size)
> - printf(" pc=0x%zx", pc - image->image_base);
> + printf(" pc=%#zx", pc - image->image_base);
> if (image->file_path)
> printf(" '%pD'", image->file_path);
> printf("\n");
> @@ -459,7 +459,7 @@ bool efi_image_parse(void *efi, size_t len, struct efi_image_regions **regp,
> efi_image_region_add(regs, efi + sorted[i]->PointerToRawData,
> efi + sorted[i]->PointerToRawData + size,
> 0);
> - log_debug("section[%d](%s): raw: 0x%x-0x%x, virt: %x-%x\n",
> + log_debug("section[%d](%s): raw: %#x-%#x, virt: %x-%x\n",
> i, sorted[i]->Name,
> sorted[i]->PointerToRawData,
> sorted[i]->PointerToRawData + size,
> @@ -493,7 +493,7 @@ bool efi_image_parse(void *efi, size_t len, struct efi_image_regions **regp,
> }
> *auth = efi + authoff;
> *auth_len = authsz;
> - log_debug("WIN_CERTIFICATE: 0x%x, size: 0x%x\n", authoff,
> + log_debug("WIN_CERTIFICATE: %#x, size: %#x\n", authoff,
> authsz);
> } else {
> *auth = NULL;
> @@ -648,7 +648,7 @@ static bool efi_image_authenticate(void *efi, size_t efi_size)
> continue;
> }
>
> - log_debug("WIN_CERTIFICATE_TYPE: 0x%x\n",
> + log_debug("WIN_CERTIFICATE_TYPE: %#x\n",
> wincert->wCertificateType);
>
> auth = (u8 *)wincert + sizeof(*wincert);
> @@ -859,7 +859,7 @@ efi_status_t efi_load_pe(struct efi_loaded_image_obj *handle,
> }
>
> if (!supported) {
> - log_err("Machine type 0x%04x is not supported\n",
> + log_err("Machine type %#04x is not supported\n",
> nt->FileHeader.Machine);
> return EFI_LOAD_ERROR;
> }
> diff --git a/lib/efi_loader/efi_memory.c b/lib/efi_loader/efi_memory.c
> index d2f5d563f2a..c42c9cccd79 100644
> --- a/lib/efi_loader/efi_memory.c
> +++ b/lib/efi_loader/efi_memory.c
> @@ -278,7 +278,7 @@ efi_status_t efi_add_memory_map_pg(u64 start, u64 pages,
> uint64_t carved_pages = 0;
> struct efi_event *evt;
>
> - EFI_PRINT("%s: 0x%llx 0x%llx %d %s\n", __func__,
> + EFI_PRINT("%s: %#llx %#llx %d %s\n", __func__,
> start, pages, memory_type, overlap_conventional ?
> "yes" : "no");
>
> @@ -530,7 +530,7 @@ efi_status_t efi_free_pages(uint64_t memory, efi_uintn_t pages)
>
> /* Sanity check */
> if (!memory || (memory & EFI_PAGE_MASK) || !pages) {
> - printf("%s: illegal free 0x%llx, 0x%zx\n", __func__,
> + printf("%s: illegal free %#llx, %#zx\n", __func__,
> memory, pages);
> return EFI_INVALID_PARAMETER;
> }
> diff --git a/lib/efi_loader/efi_signature.c b/lib/efi_loader/efi_signature.c
> index 184eac8cddb..29ef2681c06 100644
> --- a/lib/efi_loader/efi_signature.c
> +++ b/lib/efi_loader/efi_signature.c
> @@ -468,7 +468,7 @@ static bool efi_signature_check_revocation(struct pkcs7_signed_info *sinfo,
>
> memcpy(&revoc_time, sig_data->data + len,
> sizeof(revoc_time));
> - EFI_PRINT("revocation time: 0x%llx\n", revoc_time);
> + EFI_PRINT("revocation time: %#llx\n", revoc_time);
> /*
> * TODO: compare signing timestamp in sinfo
> * with revocation time
> diff --git a/lib/efi_loader/efi_variable_tee.c b/lib/efi_loader/efi_variable_tee.c
> index 8b6b0a39086..c5069186f99 100644
> --- a/lib/efi_loader/efi_variable_tee.c
> +++ b/lib/efi_loader/efi_variable_tee.c
> @@ -257,7 +257,7 @@ static int ffa_discover_mm_sp_id(void)
>
> mm_sp_id = descs[0].info.id;
>
> - log_info("EFI: MM partition ID 0x%x\n", mm_sp_id);
> + log_info("EFI: MM partition ID %#x\n", mm_sp_id);
>
> return 0;
> }
> --
> 2.45.2
>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>
More information about the U-Boot
mailing list