From: Ralf Baechle <ralf@linux-mips.org>
To: "David S. Miller" <davem@davemloft.net>
Cc: pidoux@ccr.jussieu.fr, netdev@vger.kernel.org,
linux-hams@vger.kernel.org
Subject: Re: [PATCH 2/3] Eleminate HZ from NET/ROM kernel interfaces
Date: Tue, 2 May 2006 00:52:35 +0100 [thread overview]
Message-ID: <20060501235235.GA18868@linux-mips.org> (raw)
In-Reply-To: <20060430.172136.99531184.davem@davemloft.net>
On Sun, Apr 30, 2006 at 05:21:36PM -0700, David S. Miller wrote:
> > With such extensive patches for netrom and rose modules that will go
> > into a future 2.6.x kernel, I think it would be justified to update the
> > following banners in af_rose.c and af_netrom.c respectively for they
> > appear during boot :
>
> It might be worthwhile to remove the messages altogether.
>
> IPV4 and the core networking used to output similar initialization
> log messages and it really doesn't really add anything but pollute
> the already voluminous kernel log.
>
> So I say we just remove it.
Agreed, patch to remove the messages is below.
Ralf
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
net/netrom/af_netrom.c | 3 ---
net/rose/af_rose.c | 3 ---
2 files changed, 6 deletions(-)
Index: linux-net.git/net/netrom/af_netrom.c
===================================================================
--- linux-net.git.orig/net/netrom/af_netrom.c 2006-04-30 10:28:32.000000000 +0100
+++ linux-net.git/net/netrom/af_netrom.c 2006-05-02 00:48:59.000000000 +0100
@@ -1370,8 +1370,6 @@ static struct notifier_block nr_dev_noti
static struct net_device **dev_nr;
-static char banner[] __initdata = KERN_INFO "G4KLX NET/ROM for Linux. Version 0.7 for AX25.037 Linux 2.4\n";
-
static int __init nr_proto_init(void)
{
int i;
@@ -1419,7 +1417,6 @@ static int __init nr_proto_init(void)
}
register_netdevice_notifier(&nr_dev_notifier);
- printk(banner);
ax25_protocol_register(AX25_P_NETROM, nr_route_frame);
ax25_linkfail_register(nr_link_failed);
Index: linux-net.git/net/rose/af_rose.c
===================================================================
--- linux-net.git.orig/net/rose/af_rose.c 2006-04-30 13:29:58.000000000 +0100
+++ linux-net.git/net/rose/af_rose.c 2006-05-02 00:48:28.000000000 +0100
@@ -1469,8 +1469,6 @@ static struct notifier_block rose_dev_no
static struct net_device **dev_rose;
-static const char banner[] = KERN_INFO "F6FBB/G4KLX ROSE for Linux. Version 0.62 for AX25.037 Linux 2.4\n";
-
static int __init rose_proto_init(void)
{
int i;
@@ -1519,7 +1517,6 @@ static int __init rose_proto_init(void)
sock_register(&rose_family_ops);
register_netdevice_notifier(&rose_dev_notifier);
- printk(banner);
ax25_protocol_register(AX25_P_ROSE, rose_route_frame);
ax25_linkfail_register(rose_link_failed);
next prev parent reply other threads:[~2006-05-01 23:52 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-29 14:16 [PATCH 2/3] Eleminate HZ from NET/ROM kernel interfaces Ralf Baechle
2006-04-30 19:34 ` Bernard Pidoux
2006-05-01 0:21 ` David S. Miller
2006-05-01 23:52 ` Ralf Baechle [this message]
2006-05-06 0:19 ` David S. Miller
2006-05-04 6:27 ` David S. Miller
-- strict thread matches above, loose matches on Subject: below --
2005-08-18 18:13 Ralf Baechle DL5RB
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20060501235235.GA18868@linux-mips.org \
--to=ralf@linux-mips.org \
--cc=davem@davemloft.net \
--cc=linux-hams@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pidoux@ccr.jussieu.fr \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.