kernel: Nuke legacy FreeBSD "%r" format support.
authorzrj <rimvydas.jasinskas@gmail.com>
Sat, 3 Feb 2018 19:27:51 +0000 (21:27 +0200)
committerzrj <zrj@dragonflybsd.org>
Wed, 14 Feb 2018 08:14:02 +0000 (10:14 +0200)
Makes a quite a cleanup in subr_prf.c and less hits against things
like "%rip". Only one custom format remains - "%b".

sys/ddb/db_output.c
sys/dev/drm/drm_dragonfly.c
sys/dev/drm/include/drm/drmP.h
sys/dev/raid/vinum/vinumconfig.c
sys/kern/kern_conf.c
sys/kern/subr_prf.c
sys/kern/subr_sbuf.c
sys/sys/systm.h
sys/vfs/devfs/devfs_core.c

index 709aa00..d1c36a6 100644 (file)
@@ -184,7 +184,7 @@ db_printf(const char *fmt, ...)
        __va_list listp;
 
        __va_start(listp, fmt);
-       kvcprintf (fmt, db_putchar, NULL, db_radix, listp);
+       kvcprintf (fmt, db_putchar, NULL, listp);
        __va_end(listp);
 /*     DELAY(100000);*/
 }
@@ -192,7 +192,7 @@ db_printf(const char *fmt, ...)
 void
 db_vprintf(const char *fmt, __va_list va)
 {
-       kvcprintf (fmt, db_putchar, NULL, db_radix, va);
+       kvcprintf (fmt, db_putchar, NULL, va);
 /*     DELAY(100000);*/
 }
 
@@ -209,7 +209,7 @@ db_iprintf(const char *fmt,...)
        while (--i >= 0)
                db_printf(" ");
        __va_start(listp, fmt);
-       kvcprintf (fmt, db_putchar, NULL, db_radix, listp);
+       kvcprintf (fmt, db_putchar, NULL, listp);
        __va_end(listp);
 }
 
@@ -312,7 +312,7 @@ kprintf0(const char *fmt, ...)
        __va_list ap;
 
        __va_start(ap, fmt);
-       kvcprintf(fmt, PCHAR_, NULL, 10, ap);
+       kvcprintf(fmt, PCHAR_, NULL, ap);
        __va_end(ap);
 }
 
index 542e02e..4c4b724 100644 (file)
@@ -59,7 +59,7 @@ fb_get_options(const char *connector_name, char **option)
 }
 
 /*
- * Implement simplified version of kvasnrprintf() for drm needs using
+ * Implement simplified version of kvasnprintf() for drm needs using
  * M_DRM and kvsnprintf(). Since it is unclear what string size is
  * optimal thus use of an actual length.
  */
@@ -82,7 +82,7 @@ char *drm_vasprintf(int flags, const char *format, __va_list ap)
        return str;
 }
 
-/* mimic ksnrprintf(), return pointer to char* and match drm api */
+/* mimic ksnprintf(), return pointer to char* and match drm api */
 char *drm_asprintf(int flags, const char *format, ...)
 {
        char *str;
index ae20022..07b2292 100644 (file)
@@ -1210,7 +1210,7 @@ int ttm_bo_mmap_single(struct ttm_bo_device *bdev, vm_ooffset_t *offset,
 struct ttm_buffer_object;
 void ttm_bo_release_mmap(struct ttm_buffer_object *bo);
 
-/* simplified version of kvasnrprintf() for drm needs. */
+/* simplified version of kvasnprintf() for drm needs. */
 char *drm_vasprintf(int flags, const char *format, __va_list ap) __printflike(2, 0);
 char *drm_asprintf(int flags, const char *format, ...) __printflike(2, 3);
 
index f100bb1..3a66a14 100644 (file)
@@ -111,7 +111,7 @@ throw_rude_remark(int error, char *msg,...)
         * We can't just format to ioctl_reply, since it
         * may contain our input parameters
         */
-           kvasnrprintf(&text, MSG_MAX, 10, msg, ap);
+           kvasnprintf(&text, MSG_MAX, msg, ap);
            strcpy(ioctl_reply->msg, text);
            ioctl_reply->error = error;                     /* first byte is the error number */
            kvasfree(&text);
index 51f69aa..f6efd10 100644 (file)
@@ -199,8 +199,7 @@ make_dev(struct dev_ops *ops, int minor, uid_t uid, gid_t gid,
 
        devfs_dev = devfs_new_cdev(ops, minor, NULL);
        __va_start(ap, fmt);
-       kvsnrprintf(devfs_dev->si_name, sizeof(devfs_dev->si_name),
-                   32, fmt, ap);
+       kvsnprintf(devfs_dev->si_name, sizeof(devfs_dev->si_name), fmt, ap);
        __va_end(ap);
 
        devfs_debug(DEVFS_DEBUG_INFO,
@@ -230,8 +229,7 @@ make_dev_covering(struct dev_ops *ops, struct dev_ops *bops, int minor,
 
        devfs_dev = devfs_new_cdev(ops, minor, bops);
        __va_start(ap, fmt);
-       kvsnrprintf(devfs_dev->si_name, sizeof(devfs_dev->si_name),
-                   32, fmt, ap);
+       kvsnprintf(devfs_dev->si_name, sizeof(devfs_dev->si_name), fmt, ap);
        __va_end(ap);
 
        devfs_debug(DEVFS_DEBUG_INFO,
@@ -261,8 +259,7 @@ make_only_devfs_dev(struct dev_ops *ops, int minor, uid_t uid, gid_t gid,
         * Set additional fields (XXX DEVFS interface goes here)
         */
        __va_start(ap, fmt);
-       kvsnrprintf(devfs_dev->si_name, sizeof(devfs_dev->si_name),
-                   32, fmt, ap);
+       kvsnprintf(devfs_dev->si_name, sizeof(devfs_dev->si_name), fmt, ap);
        __va_end(ap);
 
        devfs_create_dev(devfs_dev, uid, gid, perms);
@@ -290,8 +287,7 @@ make_only_dev(struct dev_ops *ops, int minor, uid_t uid, gid_t gid,
         * Set additional fields (XXX DEVFS interface goes here)
         */
        __va_start(ap, fmt);
-       kvsnrprintf(devfs_dev->si_name, sizeof(devfs_dev->si_name),
-                   32, fmt, ap);
+       kvsnprintf(devfs_dev->si_name, sizeof(devfs_dev->si_name), fmt, ap);
        __va_end(ap);
 
        reference_dev(devfs_dev);
@@ -319,8 +315,7 @@ make_only_dev_covering(struct dev_ops *ops, struct dev_ops *bops, int minor,
         * Set additional fields (XXX DEVFS interface goes here)
         */
        __va_start(ap, fmt);
-       kvsnrprintf(devfs_dev->si_name, sizeof(devfs_dev->si_name),
-                   32, fmt, ap);
+       kvsnprintf(devfs_dev->si_name, sizeof(devfs_dev->si_name), fmt, ap);
        __va_end(ap);
 
        reference_dev(devfs_dev);
@@ -382,7 +377,7 @@ make_dev_alias(cdev_t target, const char *fmt, ...)
        char *name;
 
        __va_start(ap, fmt);
-       kvasnrprintf(&name, PATH_MAX, 32, fmt, ap);
+       kvasnprintf(&name, PATH_MAX, fmt, ap);
        __va_end(ap);
 
        devfs_make_alias(name, target);
@@ -398,7 +393,7 @@ destroy_dev_alias(cdev_t target, const char *fmt, ...)
        char *name;
 
        __va_start(ap, fmt);
-       kvasnrprintf(&name, PATH_MAX, 32, fmt, ap);
+       kvasnprintf(&name, PATH_MAX, fmt, ap);
        __va_end(ap);
 
        devfs_destroy_alias(name, target);
@@ -418,7 +413,7 @@ make_autoclone_dev(struct dev_ops *ops, struct devfs_bitmap *bitmap,
        char *name;
 
        __va_start(ap, fmt);
-       kvasnrprintf(&name, PATH_MAX, 32, fmt, ap);
+       kvasnprintf(&name, PATH_MAX, fmt, ap);
        __va_end(ap);
 
        if (bitmap != NULL)
index c3cc6cf..21b2dc3 100644 (file)
@@ -148,7 +148,7 @@ uprintf(const char *fmt, ...)
                pca.tty = p->p_session->s_ttyp;
                pca.flags = TOTTY;
 
-               retval = kvcprintf(fmt, kputchar, &pca, 10, ap);
+               retval = kvcprintf(fmt, kputchar, &pca, ap);
                __va_end(ap);
        }
        return (retval);
@@ -193,7 +193,7 @@ tprintf(tpr_t tpr, const char *fmt, ...)
        pca.tty = tp;
        pca.flags = flags;
        pca.pri = LOG_INFO;
-       retval = kvcprintf(fmt, kputchar, &pca, 10, ap);
+       retval = kvcprintf(fmt, kputchar, &pca, ap);
        __va_end(ap);
        msgbuftrigger = 1;
        return (retval);
@@ -214,7 +214,7 @@ ttyprintf(struct tty *tp, const char *fmt, ...)
        __va_start(ap, fmt);
        pca.tty = tp;
        pca.flags = TOTTY;
-       retval = kvcprintf(fmt, kputchar, &pca, 10, ap);
+       retval = kvcprintf(fmt, kputchar, &pca, ap);
        __va_end(ap);
        return (retval);
 }
@@ -239,7 +239,7 @@ log(int level, const char *fmt, ...)
                pca.flags = TOCONS;
 
        __va_start(ap, fmt);
-       retval = kvcprintf(fmt, kputchar, &pca, 10, ap);
+       retval = kvcprintf(fmt, kputchar, &pca, ap);
        __va_end(ap);
 
        msgbuftrigger = 1;
@@ -308,7 +308,7 @@ kprintf(const char *fmt, ...)
        pca.tty = NULL;
        pca.flags = kprintf_logging & ~TOTTY;
        pca.pri = -1;
-       retval = kvcprintf(fmt, kputchar, &pca, 10, ap);
+       retval = kvcprintf(fmt, kputchar, &pca, ap);
        __va_end(ap);
        if (!panicstr)
                msgbuftrigger = 1;
@@ -328,7 +328,7 @@ kvprintf(const char *fmt, __va_list ap)
        pca.tty = NULL;
        pca.flags = kprintf_logging & ~TOTTY;
        pca.pri = -1;
-       retval = kvcprintf(fmt, kputchar, &pca, 10, ap);
+       retval = kvcprintf(fmt, kputchar, &pca, ap);
        if (!panicstr)
                msgbuftrigger = 1;
        consintr = savintr;             /* reenable interrupts */
@@ -401,7 +401,7 @@ ksprintf(char *buf, const char *cfmt, ...)
        __va_list ap;
 
        __va_start(ap, cfmt);
-       retval = kvcprintf(cfmt, NULL, buf, 10, ap);
+       retval = kvcprintf(cfmt, NULL, buf, ap);
        buf[retval] = '\0';
        __va_end(ap);
        return (retval);
@@ -415,7 +415,7 @@ kvsprintf(char *buf, const char *cfmt, __va_list ap)
 {
        int retval;
 
-       retval = kvcprintf(cfmt, NULL, buf, 10, ap);
+       retval = kvcprintf(cfmt, NULL, buf, ap);
        buf[retval] = '\0';
        return (retval);
 }
@@ -446,41 +446,14 @@ kvsnprintf(char *str, size_t size, const char *format, __va_list ap)
 
        info.str = str;
        info.remain = size;
-       retval = kvcprintf(format, snprintf_func, &info, 10, ap);
+       retval = kvcprintf(format, snprintf_func, &info, ap);
        if (info.remain >= 1)
                *info.str++ = '\0';
        return (retval);
 }
 
 int
-ksnrprintf(char *str, size_t size, int radix, const char *format, ...)
-{
-       int retval;
-       __va_list ap;
-
-       __va_start(ap, format);
-       retval = kvsnrprintf(str, size, radix, format, ap);
-       __va_end(ap);
-       return(retval);
-}
-
-int
-kvsnrprintf(char *str, size_t size, int radix, const char *format, __va_list ap)
-{
-       struct snprintf_arg info;
-       int retval;
-
-       info.str = str;
-       info.remain = size;
-       retval = kvcprintf(format, snprintf_func, &info, radix, ap);
-       if (info.remain >= 1)
-               *info.str++ = '\0';
-       return (retval);
-}
-
-int
-kvasnrprintf(char **strp, size_t size, int radix,
-            const char *format, __va_list ap)
+kvasnprintf(char **strp, size_t size, const char *format, __va_list ap)
 {
        struct snprintf_arg info;
        int retval;
@@ -488,7 +461,7 @@ kvasnrprintf(char **strp, size_t size, int radix,
        *strp = kmalloc(size, M_TEMP, M_WAITOK);
        info.str = *strp;
        info.remain = size;
-       retval = kvcprintf(format, snprintf_func, &info, radix, ap);
+       retval = kvcprintf(format, snprintf_func, &info, ap);
        if (info.remain >= 1)
                *info.str++ = '\0';
        return (retval);
@@ -562,8 +535,7 @@ ksprintn(char *nbuf, uintmax_t num, int base, int *lenp, int upper)
 #define PCHAR(c) {int cc=(c); if(func) (*func)(cc,arg); else *d++=cc; retval++;}
 
 int
-kvcprintf(char const *fmt, void (*func)(int, void*), void *arg,
-         int radix, __va_list ap)
+kvcprintf(char const *fmt, void (*func)(int, void*), void *arg, __va_list ap)
 {
        char nbuf[MAXNBUF];
        char *d;
@@ -600,9 +572,6 @@ kvcprintf(char const *fmt, void (*func)(int, void*), void *arg,
        if (fmt == NULL)
                fmt = "(fmt null)\n";
 
-       if (radix < 2 || radix > 36)
-               radix = 10;
-
        usespin = (func == kputchar &&
                   (kprintf_logging & TONOSPIN) == 0 &&
                   panic_cpu_gd != mycpu &&
@@ -748,11 +717,6 @@ reswitch:
                case 'q':
                        qflag = 1;
                        goto reswitch;
-               case 'r':
-                       base = radix;
-                       if (sign)
-                               goto handle_sign;
-                       goto handle_nosign;
                case 's':
                        p = __va_arg(ap, char *);
                        if (p == NULL)
index 17254e4..9d7fe08 100644 (file)
@@ -550,7 +550,7 @@ sbuf_vprintf(struct sbuf *s, const char *fmt, __va_list ap)
        KASSERT(fmt != NULL,
            ("%s called with a NULL format string", __func__));
 
-       (void)kvcprintf(fmt, sbuf_putc_func, s, 10, ap);
+       (void)kvcprintf(fmt, sbuf_putc_func, s, ap);
        if (s->s_error != 0)
                return (-1);
        return (0);
index a25c558..e6ada9c 100644 (file)
@@ -192,7 +192,7 @@ uint32_t calculate_crc32c(uint32_t crc32c, const unsigned char *buffer,
 void   init_param1 (void);
 void   init_param2 (int physpages);
 void   tablefull (const char *);
-int    kvcprintf (char const *, void (*)(int, void*), void *, int,
+int    kvcprintf (char const *, void (*)(int, void*), void *,
                      __va_list) __printf0like(1, 0);
 void   kvcreinitspin(void);
 int    log (int, const char *, ...) __printflike(2, 3);
@@ -202,16 +202,13 @@ int       kprintf (const char *, ...) __printflike(1, 2);
 void   kprintf0 (const char *, ...) __printflike(1, 2);
 void   krateprintf (struct krate *, const char *, ...) __printflike(2, 3);
 int    ksnprintf (char *, size_t, const char *, ...) __printflike(3, 4);
-int    ksnrprintf (char *, size_t, int, const char *, ...) __printflike(4, 5);
 int    ksprintf (char *buf, const char *, ...) __printflike(2, 3);
 int    uprintf (const char *, ...) __printflike(1, 2);
 int    kvprintf (const char *, __va_list) __printflike(1, 0);
 int    kvsnprintf (char *, size_t, const char *,
                        __va_list) __printflike(3, 0);
-int    kvsnrprintf (char *, size_t, int, const char *,
-                       __va_list) __printflike(4, 0);
-int    kvasnrprintf (char **, size_t, int, const char *,
-                       __va_list) __printflike(4, 0);
+int    kvasnprintf (char **, size_t, const char *,
+                       __va_list) __printflike(3, 0);
 int    kvsprintf (char *buf, const char *,
                        __va_list) __printflike(2, 0);
 int    ttyprintf (struct tty *, const char *, ...) __printflike(2, 3);
index dbd5976..1f9f52e 100644 (file)
@@ -827,7 +827,7 @@ devfs_find_device_by_name(const char *fmt, ...)
                return NULL;
 
        __va_start(ap, fmt);
-       kvasnrprintf(&target, PATH_MAX, 10, fmt, ap);
+       kvasnprintf(&target, PATH_MAX, fmt, ap);
        __va_end(ap);
 
        msg = devfs_msg_get();
@@ -1989,7 +1989,7 @@ devfs_create_device_node(struct devfs_node *root, cdev_t dev,
 
        if (path_fmt != NULL) {
                __va_start(ap, path_fmt);
-               kvasnrprintf(&path, PATH_MAX, 10, path_fmt, ap);
+               kvasnprintf(&path, PATH_MAX, path_fmt, ap);
                __va_end(ap);
        }