Add a hexdump printf specifier for userspace
authorBarret Rhoden <brho@cs.berkeley.edu>
Mon, 19 Sep 2016 18:57:40 +0000 (14:57 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Wed, 21 Sep 2016 21:27:42 +0000 (17:27 -0400)
You need to register it, then your program can use it, like so:

printf("foo %p: %.*H", foo, sizeof(*foo), foo);

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
tests/printf-ext.c
user/parlib/include/parlib/printf-ext.h
user/parlib/printf-ext.c

index 5d36836..cc94863 100644 (file)
@@ -44,6 +44,8 @@ int main(int argc, char **argv)
                printf("Failed to register 'M'\n");
        if (register_printf_specifier('E', printf_ethaddr, printf_ethaddr_info))
                printf("Failed to register 'E'\n");
+       if (register_printf_specifier('H', printf_hexdump, printf_hexdump_info))
+               printf("Failed to register 'H'\n");
 
        printf("IPv4 addr %i\n", v4addr);
        printf("IPv6 addr %i\n", v6addr);
@@ -52,6 +54,8 @@ int main(int argc, char **argv)
        printf("IPv6 addr as mask %M\n", v6addr);
        printf("ethaddr %E\n", ethaddr);
        printf("ethaddr null %E\n", 0);
+       printf("IPv6 addr as hex (4 bytes only): %.4H\n", v6addr);
+       printf("IPv6 addr as hex: %.*H\n", COUNT_OF(v6addr), v6addr);
 
        ret = open("/9/proc/no/such/file", 0, 0);
        printf("Open ret %d, errstr: %r\n", ret);
index 14e3c6d..b99410a 100644 (file)
@@ -34,10 +34,17 @@ int printf_ethaddr(FILE *stream, const struct printf_info *info,
 int printf_ethaddr_info(const struct printf_info* info, size_t n, int *argtypes,
                         int *size);
 
+/* Commonly used as %H, will print out a pointer as hex.  Pass it a precision,
+ * as in ("%.5H", ptr) (5 bytes) or ("%.*H", x, ptr) (x bytes). */
+int printf_hexdump(FILE *stream, const struct printf_info *info,
+                   const void *const *args);
+int printf_hexdump_info(const struct printf_info* info, size_t n, int *argtypes,
+                        int *size);
+
 /* Installed by default, will print the errstr for %r */
 int printf_errstr(FILE *stream, const struct printf_info *info,
                   const void *const *args);
-int printf_errstr_info(const struct printf_infoinfo, size_t n, int *argtypes,
+int printf_errstr_info(const struct printf_info *info, size_t n, int *argtypes,
                        int *size);
 
 __END_DECLS
index 8317863..db59e5e 100644 (file)
@@ -39,6 +39,7 @@
 #include <stdio.h>
 #include <string.h>
 #include <errno.h>
+#include <stdlib.h>
 
 static bool is_ipv4(uint8_t *ipaddr)
 {
@@ -187,3 +188,40 @@ int printf_errstr_info(const struct printf_info* info, size_t n, int *argtypes,
        /* errstr consumes no arguments */
        return 0;
 }
+
+static char num_to_nibble(unsigned int x)
+{
+       return "0123456789abcdef"[x & 0xf];
+}
+
+int printf_hexdump(FILE *stream, const struct printf_info *info,
+                   const void *const *args)
+{
+       uint8_t *arg = *(uint8_t**)args[0];
+       char *buf, *p;
+       int ret;
+
+       /* 3 chars per byte, one for the space */
+       buf = malloc(3 * info->prec);
+       p = buf;
+       for (int i = 0; i < info->prec; i++) {
+               if (i)
+                       *p++ = ' ';
+               *p++ = num_to_nibble(*arg >> 4);
+               *p++ = num_to_nibble(*arg);
+               arg++;
+       }
+       ret =  fwrite(buf, 1, p - buf, stream);
+       free(buf);
+       return ret;
+}
+
+int printf_hexdump_info(const struct printf_info *info, size_t n, int *argtypes,
+                        int *size)
+{
+       if (n > 0) {
+               argtypes[0] = PA_POINTER;
+               size[0] = sizeof(uint8_t*);
+       }
+       return 1;
+}