]> granicus.if.org Git - strace/commitdiff
Move xlat related functions to a separate file
authorDmitry V. Levin <ldv@altlinux.org>
Mon, 12 Jun 2017 15:41:54 +0000 (15:41 +0000)
committerDmitry V. Levin <ldv@altlinux.org>
Mon, 12 Jun 2017 16:44:45 +0000 (16:44 +0000)
* util.c (xlookup, xlat_bsearch_compare, xlat_search, printxvals,
printxval_searchn, addflags, sprintflags, printflags_ex): Move ...
* xlat.c: ... to this new file.
* Makefile.am (strace_SOURCES): Add it.

Makefile.am
util.c
xlat.c [new file with mode: 0644]

index 3d6eba1ea222b34823afd3dcf3bb4aae88d2d086..80f8a3414f2cf5103def8a71f477082f1f0290bc 100644 (file)
@@ -265,6 +265,7 @@ strace_SOURCES =    \
        v4l2.c          \
        wait.c          \
        xattr.c         \
+       xlat.c          \
        xlat.h          \
        xmalloc.c       \
        # end of strace_SOURCES
diff --git a/util.c b/util.c
index 0316744eb9821e10b36ea0c4b27fca1b917bf5dd..90a6f9e2a37a6f01c75daf47e1c3007a57a217c3 100644 (file)
--- a/util.c
+++ b/util.c
@@ -140,33 +140,6 @@ tv_mul(struct timeval *tv, const struct timeval *a, int n)
        tv->tv_usec %= 1000000;
 }
 
-const char *
-xlookup(const struct xlat *xlat, const uint64_t val)
-{
-       for (; xlat->str != NULL; xlat++)
-               if (xlat->val == val)
-                       return xlat->str;
-       return NULL;
-}
-
-static int
-xlat_bsearch_compare(const void *a, const void *b)
-{
-       const uint64_t val1 = *(const uint64_t *) a;
-       const uint64_t val2 = ((const struct xlat *) b)->val;
-       return (val1 > val2) ? 1 : (val1 < val2) ? -1 : 0;
-}
-
-const char *
-xlat_search(const struct xlat *xlat, const size_t nmemb, const uint64_t val)
-{
-       const struct xlat *e =
-               bsearch((const void*) &val,
-                       xlat, nmemb, sizeof(*xlat), xlat_bsearch_compare);
-
-       return e ? e->str : NULL;
-}
-
 #if !defined HAVE_STPCPY
 char *
 stpcpy(char *dst, const char *src)
@@ -225,71 +198,6 @@ next_set_bit(const void *bit_array, unsigned cur_bit, unsigned size_bits)
        }
 }
 
-/**
- * Print entry in struct xlat table, if there.
- *
- * @param val  Value to search a literal representation for.
- * @param dflt String (abbreviated in comment syntax) which should be emitted
- *             if no appropriate xlat value has been found.
- * @param xlat (And the following arguments) Pointers to arrays of xlat values.
- *             The last argument should be NULL.
- * @return     1 if appropriate xlat value has been found, 0 otherwise.
- */
-int
-printxvals(const uint64_t val, const char *dflt, const struct xlat *xlat, ...)
-{
-       va_list args;
-
-       va_start(args, xlat);
-       for (; xlat; xlat = va_arg(args, const struct xlat *)) {
-               const char *str = xlookup(xlat, val);
-
-               if (str) {
-                       tprints(str);
-                       va_end(args);
-                       return 1;
-               }
-       }
-       /* No hits -- print raw # instead. */
-       tprintf("%#" PRIx64, val);
-       tprints_comment(dflt);
-
-       va_end(args);
-
-       return 0;
-}
-
-/**
- * Print entry in sorted struct xlat table, if it is there.
- *
- * @param xlat      Pointer to an array of xlat values (not terminated with
- *                  XLAT_END).
- * @param xlat_size Number of xlat elements present in array (usually ARRAY_SIZE
- *                  if array is declared in the unit's scope and not
- *                  terminated with XLAT_END).
- * @param val       Value to search literal representation for.
- * @param dflt      String (abbreviated in comment syntax) which should be
- *                  emitted if no appropriate xlat value has been found.
- * @return          1 if appropriate xlat value has been found, 0
- *                  otherwise.
- */
-int
-printxval_searchn(const struct xlat *xlat, size_t xlat_size, uint64_t val,
-       const char *dflt)
-{
-       const char *s = xlat_search(xlat, xlat_size, val);
-
-       if (s) {
-               tprints(s);
-               return 1;
-       }
-
-       tprintf("%#" PRIx64, val);
-       tprints_comment(dflt);
-
-       return 0;
-}
-
 /*
  * Fetch 64bit argument at position arg_no and
  * return the index of the next argument.
@@ -351,101 +259,6 @@ printllval(struct tcb *tcp, const char *format, int arg_no)
        return arg_no;
 }
 
-/*
- * Interpret `xlat' as an array of flags
- * print the entries whose bits are on in `flags'
- */
-void
-addflags(const struct xlat *xlat, uint64_t flags)
-{
-       for (; xlat->str; xlat++) {
-               if (xlat->val && (flags & xlat->val) == xlat->val) {
-                       tprintf("|%s", xlat->str);
-                       flags &= ~xlat->val;
-               }
-       }
-       if (flags) {
-               tprintf("|%#" PRIx64, flags);
-       }
-}
-
-/*
- * Interpret `xlat' as an array of flags.
- * Print to static string the entries whose bits are on in `flags'
- * Return static string.
- */
-const char *
-sprintflags(const char *prefix, const struct xlat *xlat, uint64_t flags)
-{
-       static char outstr[1024];
-       char *outptr;
-       int found = 0;
-
-       outptr = stpcpy(outstr, prefix);
-
-       if (flags == 0 && xlat->val == 0 && xlat->str) {
-               strcpy(outptr, xlat->str);
-               return outstr;
-       }
-
-       for (; xlat->str; xlat++) {
-               if (xlat->val && (flags & xlat->val) == xlat->val) {
-                       if (found)
-                               *outptr++ = '|';
-                       outptr = stpcpy(outptr, xlat->str);
-                       found = 1;
-                       flags &= ~xlat->val;
-                       if (!flags)
-                               break;
-               }
-       }
-       if (flags) {
-               if (found)
-                       *outptr++ = '|';
-               outptr += sprintf(outptr, "%#" PRIx64, flags);
-       }
-
-       return outstr;
-}
-
-int
-printflags_ex(uint64_t flags, const char *dflt, const struct xlat *xlat, ...)
-{
-       unsigned int n = 0;
-       va_list args;
-
-       va_start(args, xlat);
-       for (; xlat; xlat = va_arg(args, const struct xlat *)) {
-               for (; (flags || !n) && xlat->str; ++xlat) {
-                       if ((flags == xlat->val) ||
-                           (xlat->val && (flags & xlat->val) == xlat->val)) {
-                               tprintf("%s%s", (n++ ? "|" : ""), xlat->str);
-                               flags &= ~xlat->val;
-                       }
-                       if (!flags)
-                               break;
-               }
-       }
-       va_end(args);
-
-       if (n) {
-               if (flags) {
-                       tprintf("|%#" PRIx64, flags);
-                       n++;
-               }
-       } else {
-               if (flags) {
-                       tprintf("%#" PRIx64, flags);
-                       tprints_comment(dflt);
-               } else {
-                       if (dflt)
-                               tprints("0");
-               }
-       }
-
-       return n;
-}
-
 void
 printaddr(const kernel_ulong_t addr)
 {
diff --git a/xlat.c b/xlat.c
new file mode 100644 (file)
index 0000000..95ba1b0
--- /dev/null
+++ b/xlat.c
@@ -0,0 +1,220 @@
+/*
+ * Copyright (c) 1991, 1992 Paul Kranenburg <pk@cs.few.eur.nl>
+ * Copyright (c) 1993 Branko Lankester <branko@hacktic.nl>
+ * Copyright (c) 1993, 1994, 1995, 1996 Rick Sladkey <jrs@world.std.com>
+ * Copyright (c) 1996-1999 Wichert Akkerman <wichert@cistron.nl>
+ * Copyright (c) 1999-2017 The strace developers.
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ * 3. The name of the author may not be used to endorse or promote products
+ *    derived from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
+ * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+ * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
+ * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+ * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+ * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "defs.h"
+#include <stdarg.h>
+
+const char *
+xlookup(const struct xlat *xlat, const uint64_t val)
+{
+       for (; xlat->str != NULL; xlat++)
+               if (xlat->val == val)
+                       return xlat->str;
+       return NULL;
+}
+
+static int
+xlat_bsearch_compare(const void *a, const void *b)
+{
+       const uint64_t val1 = *(const uint64_t *) a;
+       const uint64_t val2 = ((const struct xlat *) b)->val;
+       return (val1 > val2) ? 1 : (val1 < val2) ? -1 : 0;
+}
+
+const char *
+xlat_search(const struct xlat *xlat, const size_t nmemb, const uint64_t val)
+{
+       const struct xlat *e =
+               bsearch((const void*) &val,
+                       xlat, nmemb, sizeof(*xlat), xlat_bsearch_compare);
+
+       return e ? e->str : NULL;
+}
+
+/**
+ * Print entry in struct xlat table, if there.
+ *
+ * @param val  Value to search a literal representation for.
+ * @param dflt String (abbreviated in comment syntax) which should be emitted
+ *             if no appropriate xlat value has been found.
+ * @param xlat (And the following arguments) Pointers to arrays of xlat values.
+ *             The last argument should be NULL.
+ * @return     1 if appropriate xlat value has been found, 0 otherwise.
+ */
+int
+printxvals(const uint64_t val, const char *dflt, const struct xlat *xlat, ...)
+{
+       va_list args;
+
+       va_start(args, xlat);
+       for (; xlat; xlat = va_arg(args, const struct xlat *)) {
+               const char *str = xlookup(xlat, val);
+
+               if (str) {
+                       tprints(str);
+                       va_end(args);
+                       return 1;
+               }
+       }
+       /* No hits -- print raw # instead. */
+       tprintf("%#" PRIx64, val);
+       tprints_comment(dflt);
+
+       va_end(args);
+
+       return 0;
+}
+
+/**
+ * Print entry in sorted struct xlat table, if it is there.
+ *
+ * @param xlat      Pointer to an array of xlat values (not terminated with
+ *                  XLAT_END).
+ * @param xlat_size Number of xlat elements present in array (usually ARRAY_SIZE
+ *                  if array is declared in the unit's scope and not
+ *                  terminated with XLAT_END).
+ * @param val       Value to search literal representation for.
+ * @param dflt      String (abbreviated in comment syntax) which should be
+ *                  emitted if no appropriate xlat value has been found.
+ * @return          1 if appropriate xlat value has been found, 0
+ *                  otherwise.
+ */
+int
+printxval_searchn(const struct xlat *xlat, size_t xlat_size, uint64_t val,
+       const char *dflt)
+{
+       const char *s = xlat_search(xlat, xlat_size, val);
+
+       if (s) {
+               tprints(s);
+               return 1;
+       }
+
+       tprintf("%#" PRIx64, val);
+       tprints_comment(dflt);
+
+       return 0;
+}
+
+/*
+ * Interpret `xlat' as an array of flags
+ * print the entries whose bits are on in `flags'
+ */
+void
+addflags(const struct xlat *xlat, uint64_t flags)
+{
+       for (; xlat->str; xlat++) {
+               if (xlat->val && (flags & xlat->val) == xlat->val) {
+                       tprintf("|%s", xlat->str);
+                       flags &= ~xlat->val;
+               }
+       }
+       if (flags) {
+               tprintf("|%#" PRIx64, flags);
+       }
+}
+
+/*
+ * Interpret `xlat' as an array of flags.
+ * Print to static string the entries whose bits are on in `flags'
+ * Return static string.
+ */
+const char *
+sprintflags(const char *prefix, const struct xlat *xlat, uint64_t flags)
+{
+       static char outstr[1024];
+       char *outptr;
+       int found = 0;
+
+       outptr = stpcpy(outstr, prefix);
+
+       if (flags == 0 && xlat->val == 0 && xlat->str) {
+               strcpy(outptr, xlat->str);
+               return outstr;
+       }
+
+       for (; xlat->str; xlat++) {
+               if (xlat->val && (flags & xlat->val) == xlat->val) {
+                       if (found)
+                               *outptr++ = '|';
+                       outptr = stpcpy(outptr, xlat->str);
+                       found = 1;
+                       flags &= ~xlat->val;
+                       if (!flags)
+                               break;
+               }
+       }
+       if (flags) {
+               if (found)
+                       *outptr++ = '|';
+               outptr += sprintf(outptr, "%#" PRIx64, flags);
+       }
+
+       return outstr;
+}
+
+int
+printflags_ex(uint64_t flags, const char *dflt, const struct xlat *xlat, ...)
+{
+       unsigned int n = 0;
+       va_list args;
+
+       va_start(args, xlat);
+       for (; xlat; xlat = va_arg(args, const struct xlat *)) {
+               for (; (flags || !n) && xlat->str; ++xlat) {
+                       if ((flags == xlat->val) ||
+                           (xlat->val && (flags & xlat->val) == xlat->val)) {
+                               tprintf("%s%s", (n++ ? "|" : ""), xlat->str);
+                               flags &= ~xlat->val;
+                       }
+                       if (!flags)
+                               break;
+               }
+       }
+       va_end(args);
+
+       if (n) {
+               if (flags) {
+                       tprintf("|%#" PRIx64, flags);
+                       n++;
+               }
+       } else {
+               if (flags) {
+                       tprintf("%#" PRIx64, flags);
+                       tprints_comment(dflt);
+               } else {
+                       if (dflt)
+                               tprints("0");
+               }
+       }
+
+       return n;
+}