netdev - Re: [Cbe-oss-dev] [PATCH 5/5] spidernet: revise link status logging

Open Source and information security mailing list archives

Message-Id: <200801231455.57262.jens@de.ibm.com>
Date:	Wed, 23 Jan 2008 14:55:57 +0100
From:	Jens Osterkamp <jens@...ibm.com>
To:	cbe-oss-dev@...abs.org
Cc:	Ishizaki Kou <kou.ishizaki@...hiba.co.jp>, netdev@...r.kernel.org
Subject: Re: [Cbe-oss-dev] [PATCH 5/5] spidernet: revise link status logging

On Friday 11 January 2008, Ishizaki Kou wrote:
> This patch revises the logging for link informations of spidernet.
> 
>   - The link down message is too verbose because auto-negotiation timeout
>     occurs periodically while an ethernet cable is not connected. 
>   - We want to see the link result, and we think it should be displayed. 
> 
> Signed-off-by: Kou Ishizaki <kou.ishizaki@...hiba.co.jp>

Acked-by: Jens Osterkamp <jens@...ibm.com>

> ---
> 
> Index: linux-powerpc-git/drivers/net/spider_net.c
> ===================================================================
> --- linux-powerpc-git.orig/drivers/net/spider_net.c
> +++ linux-powerpc-git/drivers/net/spider_net.c
> @@ -2045,7 +2045,8 @@ static void spider_net_link_phy(unsigned
>  	/* if link didn't come up after SPIDER_NET_ANEG_TIMEOUT tries, setup phy again */
>  	if (card->aneg_count > SPIDER_NET_ANEG_TIMEOUT) {
> 
> -		pr_info("%s: link is down trying to bring it up\n", card->netdev->name);
> +		pr_debug("%s: link is down trying to bring it up\n",
> +			 card->netdev->name);
> 
>  		switch (card->medium) {
>  		case BCM54XX_COPPER:
> @@ -2096,9 +2097,10 @@ static void spider_net_link_phy(unsigned
> 
>  	card->aneg_count = 0;
> 
> -	pr_debug("Found %s with %i Mbps, %s-duplex %sautoneg.\n",
> -		phy->def->name, phy->speed, phy->duplex==1 ? "Full" : "Half",
> -		phy->autoneg==1 ? "" : "no ");
> +	pr_info("%s: link up, %i Mbps, %s-duplex %sautoneg.\n",
> +		card->netdev->name, phy->speed,
> +		phy->duplex == 1 ? "Full" : "Half",
> +		phy->autoneg == 1 ? "" : "no ");
> 
>  	return;
>  }
> _______________________________________________
> cbe-oss-dev mailing list
> cbe-oss-dev@...abs.org
> https://ozlabs.org/mailman/listinfo/cbe-oss-dev
> 


-- 

Gruß,

Jens

IBM Deutschland Entwicklung GmbH
Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Herbert Kircher 
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives