netdev - Re: [PATCH 3/6] sky2: reorganize chip revision features

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <46F1B05E.9010209@hp.com>
Date:	Wed, 19 Sep 2007 16:27:26 -0700
From:	Rick Jones <rick.jones2@...com>
To:	Stephen Hemminger <shemminger@...ux-foundation.org>
Cc:	Jeff Garzk <jgarzik@...ox.com>, netdev@...r.kernel.org
Subject: Re: [PATCH 3/6] sky2: reorganize chip revision features

Stephen Hemminger wrote:
> This patch should cause no functional changes in driver behaviour.
> There are (too) many revisions of the Yukon 2 chip now. Instead of
> adding more conditionals based on chip revision; rerganize into a
> set of feature flags so adding new versions is less problematic.


> @@ -311,10 +310,8 @@ static void sky2_phy_init(struct sky2_hw
>  	struct sky2_port *sky2 = netdev_priv(hw->dev[port]);
>  	u16 ctrl, ct1000, adv, pg, ledctrl, ledover, reg;
>  
> -	if (sky2->autoneg == AUTONEG_ENABLE
> -	    && !(hw->chip_id == CHIP_ID_YUKON_XL
> -		 || hw->chip_id == CHIP_ID_YUKON_EC_U
> -		 || hw->chip_id == CHIP_ID_YUKON_EX)) {
> +	if (sky2->autoneg == AUTONEG_ENABLE &&
> +	    !(hw->flags & SKY2_HW_NEWER_PHY)) {

Will something like SKY2_HW_NEWER_PHY age well?  Won't that leave things 
vulnerable to needing a SKY2_HW_NEWER_NEWER_PHY?

> @@ -1436,13 +1428,15 @@ static int sky2_xmit_frame(struct sk_buf
>  	/* Check for TCP Segmentation Offload */
>  	mss = skb_shinfo(skb)->gso_size;
>  	if (mss != 0) {
> -		if (hw->chip_id != CHIP_ID_YUKON_EX)
> +
> +		if (!(hw->flags & SKY2_HW_NEW_LE))
>  			mss += ETH_HLEN + ip_hdrlen(skb) + tcp_hdrlen(skb);

Might the same thing apply with SKY2_HW_NEW_LE?

rick jones
-
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