| 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
| ||
|
Message-ID: <52A01361.1060501@gmail.com>
Date: Thu, 05 Dec 2013 13:47:13 +0800
From: Yi Li <lovelylich@...il.com>
To: Wang Weidong <wangweidong1@...wei.com>,
David Miller <davem@...emloft.net>, andrew.hendry@...il.com
CC: linux-x25@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [PATCH] x25: convert printks to pr_<level>
于 2013年12月05日 11:06, Wang Weidong 写道:
> use pr_<level> instead of printk(LEVEL)
>
> Signed-off-by: Wang Weidong <wangweidong1@...wei.com>
> ---
> net/x25/af_x25.c | 2 +-
> net/x25/x25_dev.c | 8 ++++----
> net/x25/x25_facilities.c | 12 ++++++------
> net/x25/x25_forward.c | 2 +-
> net/x25/x25_in.c | 2 +-
> net/x25/x25_link.c | 4 ++--
> net/x25/x25_subr.c | 4 ++--
> 7 files changed, 17 insertions(+), 17 deletions(-)
>
> diff --git a/net/x25/af_x25.c b/net/x25/af_x25.c
> index 7622789..5142a40 100644
> --- a/net/x25/af_x25.c
> +++ b/net/x25/af_x25.c
> @@ -1809,7 +1809,7 @@ static int __init x25_init(void)
> if (rc != 0)
> goto out_sock;
>
> - printk(KERN_INFO "X.25 for Linux Version 0.2\n");
> + pr_info("X.25 for Linux Version 0.2\n");
>
> x25_register_sysctl();
> rc = x25_proc_init();
> diff --git a/net/x25/x25_dev.c b/net/x25/x25_dev.c
> index a8a2363..ccf8fd9 100644
> --- a/net/x25/x25_dev.c
> +++ b/net/x25/x25_dev.c
> @@ -89,7 +89,7 @@ static int x25_receive_data(struct sk_buff *skb, struct x25_neigh *nb)
> */
>
> if (frametype != X25_CLEAR_CONFIRMATION)
> - printk(KERN_DEBUG "x25_receive_data(): unknown frame type %2x\n",frametype);
> + pr_debug("x25_receive_data(): unknown frame type %2x\n",frametype);
>
> return 0;
> }
> @@ -114,7 +114,7 @@ int x25_lapb_receive_frame(struct sk_buff *skb, struct net_device *dev,
> */
> nb = x25_get_neigh(dev);
> if (!nb) {
> - printk(KERN_DEBUG "X.25: unknown neighbour - %s\n", dev->name);
> + pr_debug("X.25: unknown neighbour - %s\n", dev->name);
> goto drop;
> }
>
> @@ -154,7 +154,7 @@ void x25_establish_link(struct x25_neigh *nb)
> switch (nb->dev->type) {
> case ARPHRD_X25:
> if ((skb = alloc_skb(1, GFP_ATOMIC)) == NULL) {
> - printk(KERN_ERR "x25_dev: out of memory\n");
> + pr_err("x25_dev: out of memory\n");
> return;
> }
> ptr = skb_put(skb, 1);
> @@ -189,7 +189,7 @@ void x25_terminate_link(struct x25_neigh *nb)
>
> skb = alloc_skb(1, GFP_ATOMIC);
> if (!skb) {
> - printk(KERN_ERR "x25_dev: out of memory\n");
> + pr_err("x25_dev: out of memory\n");
> return;
> }
>
> diff --git a/net/x25/x25_facilities.c b/net/x25/x25_facilities.c
> index b825325..a4c2a2a 100644
> --- a/net/x25/x25_facilities.c
> +++ b/net/x25/x25_facilities.c
> @@ -109,7 +109,7 @@ int x25_parse_facilities(struct sk_buff *skb, struct x25_facilities *facilities,
> case X25_MARKER:
> break;
> default:
> - printk(KERN_DEBUG "X.25: unknown facility "
> + pr_debug("X.25: unknown facility "
> "%02X, value %02X\n",
> p[0], p[1]);
> break;
> @@ -132,7 +132,7 @@ int x25_parse_facilities(struct sk_buff *skb, struct x25_facilities *facilities,
> *vc_fac_mask |= X25_MASK_WINDOW_SIZE;
> break;
> default:
> - printk(KERN_DEBUG "X.25: unknown facility "
> + pr_debug("X.25: unknown facility "
> "%02X, values %02X, %02X\n",
> p[0], p[1], p[2]);
> break;
> @@ -143,7 +143,7 @@ int x25_parse_facilities(struct sk_buff *skb, struct x25_facilities *facilities,
> case X25_FAC_CLASS_C:
> if (len < 4)
> return -1;
> - printk(KERN_DEBUG "X.25: unknown facility %02X, "
> + pr_debug("X.25: unknown facility %02X, "
> "values %02X, %02X, %02X\n",
> p[0], p[1], p[2], p[3]);
> p += 4;
> @@ -172,7 +172,7 @@ int x25_parse_facilities(struct sk_buff *skb, struct x25_facilities *facilities,
> *vc_fac_mask |= X25_MASK_CALLED_AE;
> break;
> default:
> - printk(KERN_DEBUG "X.25: unknown facility %02X,"
> + pr_debug("X.25: unknown facility %02X,"
> "length %d\n", p[0], p[1]);
> break;
> }
> @@ -341,12 +341,12 @@ void x25_limit_facilities(struct x25_facilities *facilities,
>
> if (!nb->extended) {
> if (facilities->winsize_in > 7) {
> - printk(KERN_DEBUG "X.25: incoming winsize limited to 7\n");
> + pr_debug("X.25: incoming winsize limited to 7\n");
> facilities->winsize_in = 7;
> }
> if (facilities->winsize_out > 7) {
> facilities->winsize_out = 7;
> - printk( KERN_DEBUG "X.25: outgoing winsize limited to 7\n");
> + pr_debug("X.25: outgoing winsize limited to 7\n");
> }
> }
> }
> diff --git a/net/x25/x25_forward.c b/net/x25/x25_forward.c
> index c541b62..a8b0778 100644
> --- a/net/x25/x25_forward.c
> +++ b/net/x25/x25_forward.c
> @@ -51,7 +51,7 @@ int x25_forward_call(struct x25_address *dest_addr, struct x25_neigh *from,
> list_for_each(entry, &x25_forward_list) {
> x25_frwd = list_entry(entry, struct x25_forward, node);
> if (x25_frwd->lci == lci) {
> - printk(KERN_WARNING "X.25: call request for lci which is already registered!, transmitting but not registering new pair\n");
> + pr_warn("X.25: call request for lci which is already registered!, transmitting but not registering new pair\n");
> same_lci = 1;
> }
> }
> diff --git a/net/x25/x25_in.c b/net/x25/x25_in.c
> index a49cd4e..20e850e 100644
> --- a/net/x25/x25_in.c
> +++ b/net/x25/x25_in.c
> @@ -317,7 +317,7 @@ static int x25_state3_machine(struct sock *sk, struct sk_buff *skb, int frametyp
> break;
>
> default:
> - printk(KERN_WARNING "x25: unknown %02X in state 3\n", frametype);
> + pr_warn("x25: unknown %02X in state 3\n", frametype);
> break;
> }
>
> diff --git a/net/x25/x25_link.c b/net/x25/x25_link.c
> index 4acacf3..b5f44d3 100644
> --- a/net/x25/x25_link.c
> +++ b/net/x25/x25_link.c
> @@ -93,13 +93,13 @@ void x25_link_control(struct sk_buff *skb, struct x25_neigh *nb,
> if (!pskb_may_pull(skb, X25_STD_MIN_LEN + 4))
> break;
>
> - printk(KERN_WARNING "x25: diagnostic #%d - %02X %02X %02X\n",
> + pr_warn("x25: diagnostic #%d - %02X %02X %02X\n",
> skb->data[3], skb->data[4],
> skb->data[5], skb->data[6]);
> break;
>
> default:
> - printk(KERN_WARNING "x25: received unknown %02X with LCI 000\n",
> + pr_warn("x25: received unknown %02X with LCI 000\n",
> frametype);
> break;
> }
> diff --git a/net/x25/x25_subr.c b/net/x25/x25_subr.c
> index 5170d52..5e6d96b 100644
> --- a/net/x25/x25_subr.c
> +++ b/net/x25/x25_subr.c
> @@ -148,7 +148,7 @@ void x25_write_internal(struct sock *sk, int frametype)
> case X25_RESET_CONFIRMATION:
> break;
> default:
> - printk(KERN_ERR "X.25: invalid frame type %02X\n", frametype);
> + pr_err("X.25: invalid frame type %02X\n", frametype);
> return;
> }
>
> @@ -338,7 +338,7 @@ int x25_decode(struct sock *sk, struct sk_buff *skb, int *ns, int *nr, int *q,
> }
> }
>
> - printk(KERN_DEBUG "X.25: invalid PLP frame %02X %02X %02X\n",
> + pr_debug("X.25: invalid PLP frame %02X %02X %02X\n",
> frame[0], frame[1], frame[2]);
>
> return X25_ILLEGAL;
I just wonder why we need to change? What's your reason and isn't OK for
now?
regards,
Yi
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists