All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.