1. 01 Feb, 2012 10 commits
  2. 31 Jan, 2012 10 commits
  3. 30 Jan, 2012 9 commits
  4. 29 Jan, 2012 1 commit
  5. 28 Jan, 2012 5 commits
  6. 27 Jan, 2012 5 commits
    • Francois Romieu's avatar
      r8169: remove work from irq handler. · da78dbff
      Francois Romieu authored
      The irq handler was a mess.
      
      See 7ab87ff4 ("via-rhine: move work from
      irq handler to softirq and beyond") for similar changes. One can notice:
      - all non-napi tasks are explicitely scheduled trough a single work queue.
      - hiding software tx queue start behind the rtl_hw_start method is mildly
        natural. Move it in the caller where needed.
      - as can be seen from the heavy use of bh disabling locks, the driver is
        not safe for irq context messages with netconsole. It is still quite
        usable for general messaging though. Tested ok with concurrent registers
        dump (ethtool -d) + background traffic + "echo t > /proc/sysrq-trigger".
      
      Tested with old PCI chipset, PCIe 8168 and 810x:
      - XID 0c900800 RTL8168evl/8111evl
      - XID 18000000 RTL8168b/8111b
      - XID 98000000 RTL8169sc/8110sc
      - XID 083000c0 RTL8168d/8111d
      - XID 081000c0 RTL8168d/8111d
      - XID 00b00000 RTL8105e
      - XID 04a00000 RTL8102e
      
      As a side note, the comments in f11a377b
      ("r8169: avoid losing MSI interrupts") does not seem completely clear: if
      I hack the driver further to stop acking the irq link event bit, MSI
      interrupts keep being delivered (RTL8168b/8111b, XID 18000000).
      Signed-off-by: default avatarFrancois Romieu <romieu@fr.zoreil.com>
      Cc: Hayes Wang <hayeswang@realtek.com>
      da78dbff
    • Francois Romieu's avatar
      r8169: missing barriers. · 1e874e04
      Francois Romieu authored
      Signed-off-by: default avatarFrancois Romieu <romieu@fr.zoreil.com>
      Cc: Hayes Wang <hayeswang@realtek.com>
      1e874e04
    • Francois Romieu's avatar
      r8169: irq mask helpers. · 9085cdfa
      Francois Romieu authored
      Signed-off-by: default avatarFrancois Romieu <romieu@fr.zoreil.com>
      Cc: Hayes Wang <hayeswang@realtek.com>
      9085cdfa
    • Francois Romieu's avatar
      r8169: factor out IntrMask writes. · 3e990ff5
      Francois Romieu authored
      Signed-off-by: default avatarFrancois Romieu <romieu@fr.zoreil.com>
      Cc: Hayes Wang <hayeswang@realtek.com>
      3e990ff5
    • Francois Romieu's avatar
      r8169: stop delaying workqueue. · 4422bcd4
      Francois Romieu authored
      Though motivated by the move of the driver to a single work queue of
      sequential events and removal of hard irq processing, it looks safe as
      a standalone change.
      Signed-off-by: default avatarFrancois Romieu <romieu@fr.zoreil.com>
      Cc: Hayes Wang <hayeswang@realtek.com>
      4422bcd4