Borislav Petkov <bp@xxxxxxxxx> writes:
> On Sun, Mar 12, 2017 at 03:55:08PM +0200, Andy Shevchenko wrote:
>
>> The only change that IMHO matters happened between v4.10 and v4.11-rc1 is this:
>>
>> @@ -6276,8 +6274,8 @@ static int e1000e_pm_freeze(struct device *dev)
>> /* Quiesce the device without resetting the hardware */
>> e1000e_down(adapter, false);
>> e1000_free_irq(adapter);
>> + e1000e_reset_interrupt_capability(adapter);
>> }
>> - e1000e_reset_interrupt_capability(adapter);
>>
>> So, it apparently misses something for the other case, like
>> pci_disable_msi() call or so.
>
> Well, lemme add the people from
>
> 7e54d9d063fa ("e1000e: driver trying to free already-free irq")
>
> to CC then. :-)
Already did that a week ago:
https://www.spinics.net/lists/netdev/msg423379.html
Haven't heard anything back yet. Wondering if they are waiting for
someone else to submit the pretty obvious revert? Don't understand why
that should take more than a minute to figure out. It's not like they
are testing these changes anyway...
Bjørn
