src/patches/igb-5.3.4.4_fix_hung_at_dev_get_stats.patch

   1 Don't read phy_idle_errors at dev_get_stats() because this can
   2 hung the userspace.
   3
   4 https://sourceforge.net/p/e1000/bugs/500/
   5 Why this is set to wont-fix?
   6
   7 I think it is better when phy errors are not reportet in the stats
   8 than a random hanging of the whole sytem.
   9
  10 diff -Naur igb-5.3.4.4.org/src/igb_main.c igb-5.3.4.4/src/igb_main.c
  11 --- igb-5.3.4.4.org/src/igb_main.c      2016-02-26 01:48:39.000000000 +0100
  12 +++ igb-5.3.4.4/src/igb_main.c  2016-03-31 09:22:10.671794282 +0200
  13 @@ -6032,14 +6032,14 @@
  14
  15         /* Tx Dropped needs to be maintained elsewhere */
  16
  17 -       /* Phy Stats */
  18 -       if (hw->phy.media_type == e1000_media_type_copper) {
  19 -               if ((adapter->link_speed == SPEED_1000) &&
  20 -                  (!e1000_read_phy_reg(hw, PHY_1000T_STATUS, &phy_tmp))) {
  21 -                       phy_tmp &= PHY_IDLE_ERROR_COUNT_MASK;
  22 -                       adapter->phy_stats.idle_errors += phy_tmp;
  23 -               }
  24 -       }
  25 +//     /* Phy Stats */
  26 +//     if (hw->phy.media_type == e1000_media_type_copper) {
  27 +//             if ((adapter->link_speed == SPEED_1000) &&
  28 +//                (!e1000_read_phy_reg(hw, PHY_1000T_STATUS, &phy_tmp))) {
  29 +//                     phy_tmp &= PHY_IDLE_ERROR_COUNT_MASK;
  30 +//                     adapter->phy_stats.idle_errors += phy_tmp;
  31 +//             }
  32 +//     }
  33
  34         /* Management Stats */
  35         adapter->stats.mgptc += E1000_READ_REG(hw, E1000_MGTPTC);