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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).