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