Bug#642911: TX watchdog timeout on RTL8168B

[Jonathan Nieder <jrnieder@gmail.com>]

Francois Romieu wrote:

> It's part of the "no processing" policy: the driver only acks what
> it cares about. 

While you're here, a few more questions. ;-)  No more after these,
I promise.

 - Is the PCI commit after writing to IntrStatus needed in the
   context of 3.0.y which does not have 92fc43b4159b ("r8169: modify
   the flow of the hw reset") but does call irq_mask_and_ack from time
   to time without a PCI commit immediately afterward?  What symptoms
   does it address?

 - Does init_one need the same "don't ack uninteresting events" change
   (0xffff -> tp->intr_event)?

 - Am I understanding correctly that patch 3/4 (remove now-redundant
   case statements) is a code cleanup / no-op?

Thanks very much for your help.
Jonathan