1. 08 Jul, 2008 3 commits
  2. 07 Jul, 2008 7 commits
  3. 06 Jul, 2008 1 commit
  4. 04 Jul, 2008 10 commits
    • Laurent Pinchart's avatar
      fs_enet: restore promiscuous and multicast settings in restart() · c5a78ac0
      Laurent Pinchart authored
      The restart() function is called when the link state changes and resets
      multicast and promiscuous settings. This patch restores those settings at the
      end of restart().
      Signed-off-by: default avatarLaurent Pinchart <laurentp@cse-semaphore.com>
      Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
      c5a78ac0
    • Sathya Narayanan's avatar
      ibm_newemac: Fixes entry of short packets · 6c688f42
      Sathya Narayanan authored
      Short packets has to be discarded by the driver. So this patch addresses the
      issue of discarding the short packets of size lesser then ethernet header
      size.
      Signed-off-by: default avatarSathya Narayanan <sathyan@teamf1.com>
      Signed-off-by: default avatarStefan Roese <sr@denx.de>
      Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
      6c688f42
    • Sathya Narayanan's avatar
      ibm_newemac: Fixes kernel crashes when speed of cable connected changes · ab9b30cc
      Sathya Narayanan authored
      The descriptor pointers were not initialized to NIL values, so it was
      poiniting to some random addresses which was completely invalid. This
      fix takes care of initializing the descriptor to NIL values and clearing
      the valid descriptors on clean ring operation.
      Signed-off-by: default avatarSathya Narayanan <sathyan@teamf1.com>
      Signed-off-by: default avatarStefan Roese <sr@denx.de>
      Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
      ab9b30cc
    • Roland Dreier's avatar
      pasemi_mac: Access iph->tot_len with correct endianness · 77321233
      Roland Dreier authored
      iph->tot_len is stored in network byte order, so access it using
      ntohs().  This doesn't have any real world impact on pasemi_mac, since
      the device only exists as part of a big-endian system-on-chip, but
      fixing this gets rid of a sparse warning and avoids having a bad example
      in the tree.
      Signed-off-by: default avatarRoland Dreier <rolandd@cisco.com>
      Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
      77321233
    • Roland Dreier's avatar
      ehea: Access iph->tot_len with correct endianness · 3ff2cd23
      Roland Dreier authored
      iph->tot_len is stored in network byte order, so access it using
      ntohs().  This doesn't have any real world impact on ehea, since ehea
      only exists for big-endian platfroms (at the moment at least) but fixing
      this gets rid of a sparse warning and avoids having a bad example in the
      tree.
      Signed-off-by: default avatarRoland Dreier <rolandd@cisco.com>
      Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
      3ff2cd23
    • Jan-Bernd Themann's avatar
      ehea: fix race condition · 2f69ae01
      Jan-Bernd Themann authored
      When ehea_stop is called the function
      cancel_work_sync(&port->reset_task) is used to ensure
      that the reset task is not running anymore. We need an
      additional flag to ensure that it can not be scheduled
      after this call again for a certain time.
      Signed-off-by: default avatarJan-Bernd Themann <themann@de.ibm.com>
      Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
      2f69ae01
    • Jan-Bernd Themann's avatar
      ehea: add MODULE_DEVICE_TABLE · b0afffe8
      Jan-Bernd Themann authored
      Required to allow distros to easily detect when ehea
      module needs to be loaded
      Signed-off-by: default avatarJan-Bernd Themann <themann@de.ibm.com>
      Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
      b0afffe8
    • Jan-Bernd Themann's avatar
      ehea: fix might sleep problem · 5c2cec14
      Jan-Bernd Themann authored
      A mutex has to be replaced by spinlocks as it can be called from
      a context which does not allow sleeping.
      The kzalloc flag GFP_KERNEL has to be replaced by GFP_ATOMIC
      for the same reason.
      Signed-off-by: default avatarJan-Bernd Themann <themann@de.ibm.com>
      Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
      5c2cec14
    • Tobias Diedrich's avatar
      forcedeth: fix lockdep warning on ethtool -s · 97bff095
      Tobias Diedrich authored
      After enabling CONFIG_LOCKDEP and CONFIG_PROVE_LOCKING I get the
      following warning when ethtool -s is first called on one of the
      forcedeth ports:
      
      =================================
      [ INFO: inconsistent lock state ]
      2.6.26-rc4 #28
      ---------------------------------
      inconsistent {in-hardirq-W} -> {hardirq-on-W} usage.
      ethtool/1985 [HC0[0]:SC0[1]:HE1:SE0] takes:
       (&np->lock){++..}, at: [<ffffffffa000c5fd>] nv_set_settings+0xc8/0x3de [forcedeth]
      {in-hardirq-W} state was registered at:
        [<ffffffffffffffff>] 0xffffffffffffffff
      irq event stamp: 3606
      hardirqs last  enabled at (3605): [<ffffffff8068106f>] _spin_unlock_irqrestore+0x3f/0x68
      hardirqs last disabled at (3604): [<ffffffff80680d38>] _spin_lock_irqsave+0x13/0x46
      softirqs last  enabled at (3534): [<ffffffff80246ba5>] __do_softirq+0xbc/0xc5
      softirqs last disabled at (3606): [<ffffffff80680b33>] _spin_lock_bh+0x11/0x41
      
      other info that might help us debug this:
      2 locks held by ethtool/1985:
       #0:  (rtnl_mutex){--..}, at: [<ffffffff80596072>] rtnl_lock+0x12/0x14
       #1:  (_xmit_ETHER){-+..}, at: [<ffffffffa000c5e8>] nv_set_settings+0xb3/0x3de [forcedeth]
      stack backtrace:
      Pid: 1985, comm: ethtool Not tainted 2.6.26-rc4 #28
      Call Trace:
       [<ffffffff8025f190>] print_usage_bug+0x162/0x173
       [<ffffffff8025fa8b>] mark_lock+0x231/0x41f
       [<ffffffff802607cf>] __lock_acquire+0x4e7/0xcac
       [<ffffffff8025fe64>] ? trace_hardirqs_on+0xf1/0x115
       [<ffffffff80272c3a>] ? disable_irq_nosync+0x6f/0x7b
       [<ffffffff80261375>] lock_acquire+0x55/0x6e
       [<ffffffffa000c5fd>] ? :forcedeth:nv_set_settings+0xc8/0x3de
       [<ffffffff80680b15>] _spin_lock+0x2f/0x3c
       [<ffffffffa000c5fd>] :forcedeth:nv_set_settings+0xc8/0x3de
       [<ffffffff8058f8bb>] dev_ethtool+0x186/0xea3
       [<ffffffff8067f446>] ? mutex_lock_nested+0x243/0x275
       [<ffffffff8025df2b>] ? debug_mutex_free_waiter+0x46/0x4a
       [<ffffffff8067f469>] ? mutex_lock_nested+0x266/0x275
       [<ffffffff8058e1ce>] dev_ioctl+0x4eb/0x600
       [<ffffffff8068106f>] ? _spin_unlock_irqrestore+0x3f/0x68
       [<ffffffff80580f91>] sock_ioctl+0x1f5/0x202
       [<ffffffff802a322e>] vfs_ioctl+0x2a/0x77
       [<ffffffff802a34d6>] do_vfs_ioctl+0x25b/0x270
       [<ffffffff806807b6>] ? trace_hardirqs_on_thunk+0x35/0x3a
       [<ffffffff802a352d>] sys_ioctl+0x42/0x65
       [<ffffffff8021fffb>] system_call_after_swapgs+0x7b/0x80
      
      This is caused by the following snippet in nv_set_settings:
      
      	netif_carrier_off(dev);
      	if (netif_running(dev)) {
      		nv_disable_irq(dev);
      		netif_tx_lock_bh(dev);
      		spin_lock(&np->lock);
      		/* stop engines */
      		nv_stop_rxtx(dev);
      		spin_unlock(&np->lock);
      		netif_tx_unlock_bh(dev);
      	}
      
      Because of nv_disable_irq this is probably not really a problem
      though (I guess) and replacing the spin_lock with spin_lock_irqsave
      could keep interrupts disabled for a longer period of time because
      of delays in nv_stop_rx and nv_stop_tx.
      Signed-off-by: default avatarTobias Diedrich <ranma+kernel@tdiedrich.de>
      Cc: Ayaz Abdulla <aabdulla@nvidia.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
      97bff095
    • Krzysztof Halasa's avatar
      Add missing skb->dev assignment in Frame Relay RX code · 54364b75
      Krzysztof Halasa authored
      Commit 4c13eb66 ([ETH]: Make
      eth_type_trans set skb->dev like the other *_type_trans) removed
      skb->dev assignment from hdlc_fr.c:fr_rx(). Unfortunately it was also
      needed for cases other than eth_type_trans().
      
      Adding it back.
      
      It's quite serious and may be a security risk as it causes a wrong
      input interface indication (the physical hdlcX instead of logical
      pvcX). Probably -stable class fix.
      Signed-off-by: default avatarKrzysztof Halasa <khc@pm.waw.pl>
      Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
      54364b75
  5. 03 Jul, 2008 19 commits