netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] e1000: fix two mispatches
@ 2006-04-18 19:39 Kok, Auke
  2006-04-18 19:40 ` [PATCH 1/2] e1000: fix mispatch for media type detect Kok, Auke
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Kok, Auke @ 2006-04-18 19:39 UTC (permalink / raw)
  To: Garzik, Jeff, netdev, Miller, David
  Cc: Ronciak, John, Brandeburg, Jesse, Kirsher, Jeff, Kok, Auke, akpm


Hi,

This patch series implements two e100 fixes for an old and new patch mishap.

[1] fix mispatch for media type detect.
[2] fix mismerge skb_put.


These changes are available through git.

git://63.64.152.142/~ahkok/git/netdev-2.6 e1000-7.0.38-k2-fixes

these patches are against
 netdev-2.6#upstream 8fc65162a8f25929be80c8d6321a3479e92b5aae


Cheers,

Auke


---
 drivers/net/e1000/e1000_main.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)


--
Auke Kok <auke-jan.h.kok@intel.com>
Intel Pro Ethernet Driver Group
LAN Access Division / Digital Enterprise Group 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] e1000: fix mispatch for media type detect.
  2006-04-18 19:39 [PATCH 0/2] e1000: fix two mispatches Kok, Auke
@ 2006-04-18 19:40 ` Kok, Auke
  2006-04-18 19:40 ` [PATCH 2/2] e1000: fix mismerge skb_put Kok, Auke
  2006-04-20 21:31 ` [PATCH 0/2] e1000: fix two mispatches Jeff Garzik
  2 siblings, 0 replies; 4+ messages in thread
From: Kok, Auke @ 2006-04-18 19:40 UTC (permalink / raw)
  To: Garzik, Jeff, netdev, Miller, David
  Cc: Ronciak, John, Brandeburg, Jesse, Kirsher, Jeff, Kok, Auke, akpm


Recent patch was mismerged in the miitool path. e1000_media_type_copper
was being compared with the phy type instead of the media type.


Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: Auke Kok <auke-jan.h.kok@intel.com>
---

 drivers/net/e1000/e1000_main.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/net/e1000/e1000_main.c b/drivers/net/e1000/e1000_main.c
index 82d443b..2b8bced 100644
--- a/drivers/net/e1000/e1000_main.c
+++ b/drivers/net/e1000/e1000_main.c
@@ -4174,7 +4174,7 @@ e1000_mii_ioctl(struct net_device *netde
 			spin_unlock_irqrestore(&adapter->stats_lock, flags);
 			return -EIO;
 		}
-		if (adapter->hw.phy_type == e1000_media_type_copper) {
+		if (adapter->hw.media_type == e1000_media_type_copper) {
 			switch (data->reg_num) {
 			case PHY_CTRL:
 				if (mii_reg & MII_CR_POWER_DOWN)



--
Auke Kok <auke-jan.h.kok@intel.com>
Intel Pro Ethernet Driver Group
LAN Access Division / Digital Enterprise Group 

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] e1000: fix mismerge skb_put.
  2006-04-18 19:39 [PATCH 0/2] e1000: fix two mispatches Kok, Auke
  2006-04-18 19:40 ` [PATCH 1/2] e1000: fix mispatch for media type detect Kok, Auke
@ 2006-04-18 19:40 ` Kok, Auke
  2006-04-20 21:31 ` [PATCH 0/2] e1000: fix two mispatches Jeff Garzik
  2 siblings, 0 replies; 4+ messages in thread
From: Kok, Auke @ 2006-04-18 19:40 UTC (permalink / raw)
  To: Garzik, Jeff, netdev, Miller, David
  Cc: Ronciak, John, Brandeburg, Jesse, Kirsher, Jeff, Kok, Auke, akpm


Seems there was a bit of a fix needed to due a bad merge in the legacy
receive path.  Fixes a panic due to skb_over_panic.

Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: Auke Kok <auke-jan.h.kok@intel.com>
---

 drivers/net/e1000/e1000_main.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/drivers/net/e1000/e1000_main.c b/drivers/net/e1000/e1000_main.c
index 2b8bced..fb8cef6 100644
--- a/drivers/net/e1000/e1000_main.c
+++ b/drivers/net/e1000/e1000_main.c
@@ -3585,8 +3585,7 @@ e1000_clean_rx_irq(struct e1000_adapter 
 				buffer_info->skb = skb;
 				goto next_desc;
 			}
-		} else
-			skb_put(skb, length);
+		}
 
 		/* code added for copybreak, this should improve
 		 * performance for small packets with large amounts



--
Auke Kok <auke-jan.h.kok@intel.com>
Intel Pro Ethernet Driver Group
LAN Access Division / Digital Enterprise Group 

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 0/2] e1000: fix two mispatches
  2006-04-18 19:39 [PATCH 0/2] e1000: fix two mispatches Kok, Auke
  2006-04-18 19:40 ` [PATCH 1/2] e1000: fix mispatch for media type detect Kok, Auke
  2006-04-18 19:40 ` [PATCH 2/2] e1000: fix mismerge skb_put Kok, Auke
@ 2006-04-20 21:31 ` Jeff Garzik
  2 siblings, 0 replies; 4+ messages in thread
From: Jeff Garzik @ 2006-04-20 21:31 UTC (permalink / raw)
  To: Kok, Auke
  Cc: netdev, Miller, David, Ronciak, John, Brandeburg, Jesse,
	Kirsher, Jeff, Kok, Auke, akpm

Kok, Auke wrote:
> Hi,
> 
> This patch series implements two e100 fixes for an old and new patch mishap.
> 
> [1] fix mispatch for media type detect.
> [2] fix mismerge skb_put.
> 
> 
> These changes are available through git.
> 
> git://63.64.152.142/~ahkok/git/netdev-2.6 e1000-7.0.38-k2-fixes

applied



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2006-04-20 21:31 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-04-18 19:39 [PATCH 0/2] e1000: fix two mispatches Kok, Auke
2006-04-18 19:40 ` [PATCH 1/2] e1000: fix mispatch for media type detect Kok, Auke
2006-04-18 19:40 ` [PATCH 2/2] e1000: fix mismerge skb_put Kok, Auke
2006-04-20 21:31 ` [PATCH 0/2] e1000: fix two mispatches Jeff Garzik

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).