lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <0c3a6ad937c8a1eaf3ca914b2edc4a1cd6ffc18e.1242407227.git.joe@perches.com>
Date:	Fri, 15 May 2009 10:59:40 -0700
From:	Joe Perches <joe@...ches.com>
To:	linux-kernel@...r.kernel.org
Cc:	Karsten Keil <isdn@...ux-pingi.de>
Subject: [PATCH 06/11] drivers/isdn: Convert #ifdef DEBUG printk(KERN_DEBUG to pr_debug(

From: Joe Perches <joe@...ches.com>

Signed-off-by: Joe Perches <joe@...ches.com>
---
 drivers/isdn/i4l/isdn_bsdcomp.c |    4 +---
 drivers/isdn/pcbit/callbacks.c  |    9 ++-------
 drivers/isdn/pcbit/capi.c       |   16 ++++------------
 drivers/isdn/pcbit/drv.c        |   30 +++++++++---------------------
 drivers/isdn/pcbit/layer2.c     |   30 +++++++++---------------------
 5 files changed, 25 insertions(+), 64 deletions(-)

diff --git a/drivers/isdn/i4l/isdn_bsdcomp.c b/drivers/isdn/i4l/isdn_bsdcomp.c
index 02d9918..ab77384 100644
--- a/drivers/isdn/i4l/isdn_bsdcomp.c
+++ b/drivers/isdn/i4l/isdn_bsdcomp.c
@@ -769,10 +769,8 @@ static int bsd_decompress (void *state, struct sk_buff *skb_in, struct sk_buff *
 		if( skb_tailroom(skb_out) < codelen + extra) {
 			if (db->debug) {
 				printk(KERN_DEBUG "bsd_decomp%d: ran out of mru\n", db->unit);
-#ifdef DEBUG
-				printk(KERN_DEBUG "  len=%d, finchar=0x%x, codelen=%d,skblen=%d\n",
+				pr_debug("  len=%d, finchar=0x%x, codelen=%d,skblen=%d\n",
 					ilen, finchar, codelen, skb_out->len);
-#endif
 			}
 			return DECOMP_FATALERROR;
 		}
diff --git a/drivers/isdn/pcbit/callbacks.c b/drivers/isdn/pcbit/callbacks.c
index 43ecd0f..57a065c 100644
--- a/drivers/isdn/pcbit/callbacks.c
+++ b/drivers/isdn/pcbit/callbacks.c
@@ -48,10 +48,7 @@ void cb_out_1(struct pcbit_dev * dev, struct pcbit_chan* chan,
         ushort refnum;
 
 
-#ifdef DEBUG
-        printk(KERN_DEBUG "Called Party Number: %s\n", 
-               cbdata->data.setup.CalledPN);
-#endif
+        pr_debug("Called Party Number: %s\n", cbdata->data.setup.CalledPN);
         /*
          * hdr - kmalloc in capi_conn_req
          *     - kfree   when msg has been sent
@@ -164,9 +161,7 @@ void cb_in_1(struct pcbit_dev * dev, struct pcbit_chan* chan,
 	ictl.parm.setup.plan = 0;
 	ictl.parm.setup.screen = 0;
 
-#ifdef DEBUG
-	printk(KERN_DEBUG "statstr: %s\n", ictl.num);
-#endif
+	pr_debug("statstr: %s\n", ictl.num);
 
         dev->dev_if->statcallb(&ictl);
 
diff --git a/drivers/isdn/pcbit/capi.c b/drivers/isdn/pcbit/capi.c
index ac5a91c..4a812e7 100644
--- a/drivers/isdn/pcbit/capi.c
+++ b/drivers/isdn/pcbit/capi.c
@@ -162,9 +162,7 @@ int capi_conn_active_req(struct pcbit_chan* chan, struct sk_buff **skb)
 
         *((ushort*) skb_put(*skb, 2) ) = chan->callref;  
 
-#ifdef DEBUG
-	printk(KERN_DEBUG "Call Reference: %04x\n", chan->callref); 
-#endif
+	pr_debug("Call Reference: %04x\n", chan->callref); 
 
         *(skb_put(*skb, 1)) = 0;       /*  BC.Length = 0;          */
         *(skb_put(*skb, 1)) = 0;       /*  ConnectedPN.Length = 0  */
@@ -226,9 +224,7 @@ int capi_select_proto_req(struct pcbit_chan *chan, struct sk_buff **skb,
 		*(skb_put(*skb, 1)) = 0x06;
 		break;
         default:
-#ifdef DEBUG 
-                printk(KERN_DEBUG "Transparent\n");
-#endif
+                pr_debug("Transparent\n");
                 *(skb_put(*skb, 1)) = 0x03;
                 break;
         }
@@ -388,9 +384,7 @@ int capi_decode_conn_ind(struct pcbit_chan * chan,
         chan->callref = *((ushort*) skb->data);
         skb_pull(skb, 2);
 
-#ifdef DEBUG
-	printk(KERN_DEBUG "Call Reference: %04x\n", chan->callref); 
-#endif
+	pr_debug("Call Reference: %04x\n", chan->callref); 
 
         /* Channel Identification */
 
@@ -420,9 +414,7 @@ int capi_decode_conn_ind(struct pcbit_chan * chan,
 	if (len > 0) {
 		int count = 1;
 		
-#ifdef DEBUG
-		printk(KERN_DEBUG "CPN: Octect 3 %02x\n", skb->data[1]);
-#endif
+		pr_debug("CPN: Octect 3 %02x\n", skb->data[1]);
 		if ((skb->data[1] & 0x80) == 0)
 			count = 2;
 		
diff --git a/drivers/isdn/pcbit/drv.c b/drivers/isdn/pcbit/drv.c
index 8c66bcb..8ec12b0 100644
--- a/drivers/isdn/pcbit/drv.c
+++ b/drivers/isdn/pcbit/drv.c
@@ -313,9 +313,8 @@ static void pcbit_block_timer(unsigned long data)
 	del_timer(&chan->block_timer);
 	chan->block_timer.function = NULL;
 
-#ifdef DEBUG
-	printk(KERN_DEBUG "pcbit_block_timer\n");
-#endif	
+	pr_debug("pcbit_block_timer\n");
+
 	chan->queued = 0;
 	ictl.driver = dev->id;
 	ictl.command = ISDN_STAT_BSENT;
@@ -565,11 +564,9 @@ void pcbit_l3_receive(struct pcbit_dev * dev, ulong msg,
 		 * reference number. The current version of the firmware
 		 * doesn't sent the ref number correctly.
 		 */
-#ifdef DEBUG
-		printk(KERN_DEBUG "refnum=%04x b1=%04x b2=%04x\n", refnum, 
-		       dev->b1->s_refnum, 
-		       dev->b2->s_refnum);
-#endif
+		pr_debug("refnum=%04x b1=%04x b2=%04x\n", refnum, 
+			 dev->b1->s_refnum, 
+			 dev->b2->s_refnum);
 		/* We just try to find a channel in the right state */
 
 		if (dev->b1->fsm_state == ST_CALL_INIT)
@@ -676,9 +673,7 @@ void pcbit_l3_receive(struct pcbit_dev * dev, ulong msg,
 			printk(KERN_WARNING "capi_decode_disc_conf - error\n");
 		break;
 	case MSG_INFO_IND:
-#ifdef DEBUG
-		printk(KERN_DEBUG "received Info Indication - discarded\n");
-#endif
+		pr_debug("received Info Indication - discarded\n");
 		break;
 #ifdef DEBUG
 	case MSG_DEBUG_188:
@@ -794,9 +789,7 @@ static void set_running_timeout(unsigned long ptr)
 {
 	struct pcbit_dev * dev;
 
-#ifdef DEBUG
-	printk(KERN_DEBUG "set_running_timeout\n");
-#endif
+	pr_debug("set_running_timeout\n");
 	dev = (struct pcbit_dev *) ptr;
 
 	wake_up_interruptible(&dev->set_running_wq);
@@ -847,10 +840,7 @@ static int set_protocol_running(struct pcbit_dev * dev)
 
 		dev->l2_state = L2_DOWN;
 
-#ifdef DEBUG
-		printk(KERN_DEBUG "Bank3 = %02x\n", 
-		       readb(dev->sh_mem + BANK3));
-#endif
+		pr_debug("Bank3 = %02x\n", readb(dev->sh_mem + BANK3));
 		writeb(0x40, dev->sh_mem + BANK4);
 
 		/* warn the upper layer */
@@ -1048,9 +1038,7 @@ static void pcbit_set_msn(struct pcbit_dev *dev, char *list)
 		memcpy(ptr->msn, sp, len - 1);
 		ptr->msn[len] = 0;
 
-#ifdef DEBUG
-		printk(KERN_DEBUG "msn: %s\n", ptr->msn);
-#endif
+		pr_debug("msn: %s\n", ptr->msn);
 		if (dev->msn_list == NULL)
 			dev->msn_list = ptr;
 		else
diff --git a/drivers/isdn/pcbit/layer2.c b/drivers/isdn/pcbit/layer2.c
index e075e8d..7f3ab02 100644
--- a/drivers/isdn/pcbit/layer2.c
+++ b/drivers/isdn/pcbit/layer2.c
@@ -267,11 +267,9 @@ pcbit_transmit(struct pcbit_dev *dev)
 		spin_unlock_irqrestore(&dev->lock, flags);
 	} else {
 		spin_unlock_irqrestore(&dev->lock, flags);
-#ifdef DEBUG
-		printk(KERN_DEBUG "unacked %d free %d write_queue %s\n",
-		     unacked, dev->free, dev->write_queue ? "not empty" :
-		       "empty");
-#endif
+		pr_debug("unacked %d free %d write_queue %s\n",
+			 unacked, dev->free,
+			 dev->write_queue ? "not empty" : "empty");
 	}
 }
 
@@ -385,20 +383,15 @@ pcbit_receive(struct pcbit_dev *dev)
 
 		if (frame->hdr_len == 0) {
 			kfree(frame);
-#ifdef DEBUG
-			printk(KERN_DEBUG "0 sized frame\n");
-#endif
+			pr_debug("0 sized frame\n");
 			pcbit_firmware_bug(dev);
 			return;
 		}
 		/* sanity check the length values */
 		if (frame->hdr_len > 1024 || frame->dt_len > 2048) {
-#ifdef DEBUG
-			printk(KERN_DEBUG "length problem: ");
-			printk(KERN_DEBUG "TH=%04x TD=%04x\n",
-			       frame->hdr_len,
-			       frame->dt_len);
-#endif
+			pr_debug("length problem: TH=%04x TD=%04x\n",
+				 frame->hdr_len,
+				 frame->dt_len);
 			pcbit_l2_error(dev);
 			kfree(frame);
 			return;
@@ -521,9 +514,7 @@ pcbit_irq_handler(int interrupt, void *devptr)
 		return IRQ_HANDLED;
 	}
 	if (info & 0x40U) {     /* E bit set */
-#ifdef DEBUG
-		printk(KERN_DEBUG "pcbit_irq_handler: E bit on\n");
-#endif
+		pr_debug("pcbit_irq_handler: E bit on\n");
 		pcbit_l2_error(dev);
 		dev->interrupt = 0;
 		return IRQ_HANDLED;
@@ -562,10 +553,7 @@ pcbit_l2_active_conf(struct pcbit_dev *dev, u_char info)
 
 	state = dev->l2_state;
 
-#ifdef DEBUG
-	printk(KERN_DEBUG "layer2_active_confirm\n");
-#endif
-
+	pr_debug("layer2_active_confirm\n");
 
 	if (info & 0x80U) {
 		dev->rcv_seq = info & 0x07U;
-- 
1.6.3.1.9.g95405b.dirty

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ