drivers/isdn/mISDN: Use printf extension %pV
authorJoe Perches <joe@perches.com>
Tue, 9 Nov 2010 14:35:16 +0000 (14:35 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 16 Nov 2010 18:22:58 +0000 (10:22 -0800)
Using %pV reduces the number of printk calls and
eliminates any possible message interleaving from
other printk calls.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/isdn/mISDN/layer1.c
drivers/isdn/mISDN/layer2.c
drivers/isdn/mISDN/tei.c

index ac4aa18c632b0f2040cef841a4e5fd98a0cd7f5a..5cc7c001c523e3e7c73730c4822fe317d7ca1ef4 100644 (file)
@@ -99,12 +99,16 @@ static void
 l1m_debug(struct FsmInst *fi, char *fmt, ...)
 {
        struct layer1 *l1 = fi->userdata;
+       struct va_format vaf;
        va_list va;
 
        va_start(va, fmt);
-       printk(KERN_DEBUG "%s: ", dev_name(&l1->dch->dev.dev));
-       vprintk(fmt, va);
-       printk("\n");
+
+       vaf.fmt = fmt;
+       vaf.va = &va;
+
+       printk(KERN_DEBUG "%s: %pV\n", dev_name(&l1->dch->dev.dev), &vaf);
+
        va_end(va);
 }
 
index c97371788764df3af6a45ad9f402ece12f59a1b3..4ae75053c9d2a9e572f0fb760caabac9ce4b3e28 100644 (file)
@@ -95,14 +95,20 @@ static void
 l2m_debug(struct FsmInst *fi, char *fmt, ...)
 {
        struct layer2 *l2 = fi->userdata;
+       struct va_format vaf;
        va_list va;
 
        if (!(*debug & DEBUG_L2_FSM))
                return;
+
        va_start(va, fmt);
-       printk(KERN_DEBUG "l2 (sapi %d tei %d): ", l2->sapi, l2->tei);
-       vprintk(fmt, va);
-       printk("\n");
+
+       vaf.fmt = fmt;
+       vaf.va = &va;
+
+       printk(KERN_DEBUG "l2 (sapi %d tei %d): %pV\n",
+              l2->sapi, l2->tei, &vaf);
+
        va_end(va);
 }
 
index 1b85d9d274965e583a5c972eb7c1a597af879dab..687c9b6264ab027fee9d1ed355239711236f34be 100644 (file)
@@ -79,14 +79,19 @@ static void
 da_debug(struct FsmInst *fi, char *fmt, ...)
 {
        struct manager  *mgr = fi->userdata;
+       struct va_format vaf;
        va_list va;
 
        if (!(*debug & DEBUG_L2_TEIFSM))
                return;
+
        va_start(va, fmt);
-       printk(KERN_DEBUG "mgr(%d): ", mgr->ch.st->dev->id);
-       vprintk(fmt, va);
-       printk("\n");
+
+       vaf.fmt = fmt;
+       vaf.va = &va;
+
+       printk(KERN_DEBUG "mgr(%d): %pV\n", mgr->ch.st->dev->id, &vaf);
+
        va_end(va);
 }
 
@@ -223,14 +228,20 @@ static void
 tei_debug(struct FsmInst *fi, char *fmt, ...)
 {
        struct teimgr   *tm = fi->userdata;
+       struct va_format vaf;
        va_list va;
 
        if (!(*debug & DEBUG_L2_TEIFSM))
                return;
+
        va_start(va, fmt);
-       printk(KERN_DEBUG "sapi(%d) tei(%d): ", tm->l2->sapi, tm->l2->tei);
-       vprintk(fmt, va);
-       printk("\n");
+
+       vaf.fmt = fmt;
+       vaf.va = &va;
+
+       printk(KERN_DEBUG "sapi(%d) tei(%d): %pV\n",
+              tm->l2->sapi, tm->l2->tei, &vaf);
+
        va_end(va);
 }