Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=df180e369cf54a8ef8440667ab1d13d452fc7215
Commit:     df180e369cf54a8ef8440667ab1d13d452fc7215
Parent:     8d4ef88b5df1afe097e38aef8cab2ed35ca141ea
Author:     Satyam Sharma <[EMAIL PROTECTED]>
AuthorDate: Fri Aug 10 15:32:14 2007 -0700
Committer:  David S. Miller <[EMAIL PROTECTED]>
CommitDate: Wed Oct 10 16:48:04 2007 -0700

    [NET] netconsole: Introduce netconsole_target
    
    Based upon initial work by Keiichi Kii <[EMAIL PROTECTED]>.
    
    Introduce a wrapper structure over netpoll to represent logging targets
    configured in netconsole.  This will get extended with other members in
    further patches.
    
    This is done independent of the (to-be-introduced) NETCONSOLE_DYNAMIC config
    option so that we're able to drastically cut down on the #ifdef complexity 
of
    final netconsole.c.  Also, struct netconsole_target would be required for
    multiple targets support also, and not just dynamic reconfigurability.
    
    Signed-off-by: Satyam Sharma <[EMAIL PROTECTED]>
    Signed-off-by: Keiichi Kii <[EMAIL PROTECTED]>
    Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
    Signed-off-by: David S. Miller <[EMAIL PROTECTED]>
---
 drivers/net/netconsole.c |   36 +++++++++++++++++++++++++-----------
 1 files changed, 25 insertions(+), 11 deletions(-)

diff --git a/drivers/net/netconsole.c b/drivers/net/netconsole.c
index 75cb761..be15ca6 100644
--- a/drivers/net/netconsole.c
+++ b/drivers/net/netconsole.c
@@ -62,24 +62,35 @@ static int __init option_setup(char *opt)
 __setup("netconsole=", option_setup);
 #endif /* MODULE */
 
-static struct netpoll np = {
-       .name           = "netconsole",
-       .dev_name       = "eth0",
-       .local_port     = 6665,
-       .remote_port    = 6666,
-       .remote_mac     = {0xff, 0xff, 0xff, 0xff, 0xff, 0xff},
+/**
+ * struct netconsole_target - Represents a configured netconsole target.
+ * @np:                The netpoll structure for this target.
+ */
+struct netconsole_target {
+       struct netpoll          np;
+};
+
+static struct netconsole_target default_target = {
+       .np             = {
+               .name           = "netconsole",
+               .dev_name       = "eth0",
+               .local_port     = 6665,
+               .remote_port    = 6666,
+               .remote_mac     = {0xff, 0xff, 0xff, 0xff, 0xff, 0xff},
+       },
 };
 
 static void write_msg(struct console *con, const char *msg, unsigned int len)
 {
        int frag, left;
        unsigned long flags;
+       struct netconsole_target *nt = &default_target;
 
-       if (netif_running(np.dev)) {
+       if (netif_running(nt->np.dev)) {
                local_irq_save(flags);
                for (left = len; left;) {
                        frag = min(left, MAX_PRINT_CHUNK);
-                       netpoll_send_udp(&np, msg, frag);
+                       netpoll_send_udp(&nt->np, msg, frag);
                        msg += frag;
                        left -= frag;
                }
@@ -96,17 +107,18 @@ static struct console netconsole = {
 static int __init init_netconsole(void)
 {
        int err = 0;
+       struct netconsole_target *nt = &default_target;
 
        if (!strnlen(config, MAX_PARAM_LENGTH)) {
                printk(KERN_INFO "netconsole: not configured, aborting\n");
                goto out;
        }
 
-       err = netpoll_parse_options(&np, config);
+       err = netpoll_parse_options(&nt->np, config);
        if (err)
                goto out;
 
-       err = netpoll_setup(&np);
+       err = netpoll_setup(&nt->np);
        if (err)
                goto out;
 
@@ -119,8 +131,10 @@ out:
 
 static void __exit cleanup_netconsole(void)
 {
+       struct netconsole_target *nt = &default_target;
+
        unregister_console(&netconsole);
-       netpoll_cleanup(&np);
+       netpoll_cleanup(&nt->np);
 }
 
 module_init(init_netconsole);
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to