Re: [PATCH 2/2] netfilter: cleanup printk messages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Stephen Hemminger wrote:
> On Wed, 12 May 2010 18:52:02 +0200 (CEST)
> Jan Engelhardt <jengelh@xxxxxxxxxx> wrote:
> 
>> On Wednesday 2010-05-12 18:29, Stephen Hemminger wrote:
>>
>>> Make sure all printk messages have a severity level.
>>>
>>> --- a/net/netfilter/nf_conntrack_amanda.c	2010-05-12 09:28:26.447006714 -0700
>>> +++ b/net/netfilter/nf_conntrack_amanda.c	2010-05-12 09:28:44.287371327 -0700
>>> @@ -108,7 +108,7 @@ static int amanda_help(struct sk_buff *s
>>> 	dataoff = protoff + sizeof(struct udphdr);
>>> 	if (dataoff >= skb->len) {
>>> 		if (net_ratelimit())
>>> -			printk("amanda_help: skblen = %u\n", skb->len);
>>> +			printk(KERN_ERR "amanda_help: skblen = %u\n", skb->len);
>> A good idea, but perhaps we should just use pr_{err,andsoon} while at it.
> 
> I did some places, but code that was already using printk(KERN_XXX form,
> I just duplicated existing style in that file.

We can clean this up in follow-up patches. Applied.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Netfitler Users]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux