On 27.02.2015 15:02, nick wrote:
[...]
>> i40e: Fix a bug where Rx would stop after some time
>> [...]
>> diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c
>> b/drivers/net/ethernet/intel/i40e/i40e_main.c
>> index f7464e8..ff6d94d 100644
>> --- a/drivers/net/ethernet/intel/i40e/i40e_main.c
>> +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
>> [...]
>> @@ -9169,6 +9178,13 @@ static int i40e_probe(struct pci_dev *pdev, const
>> struct pci_device_id *ent)
>> if (err)
>> dev_info(&pf->pdev->dev, "set phy mask fail, aq_err %d\n", err);
>>
>> + msleep(75);
>> + err = i40e_aq_set_link_restart_an(&pf->hw, true, NULL);
>> + if (err) {
>> + dev_info(&pf->pdev->dev, "link restart failed, aq_err=%d\n",
>> + pf->hw.aq.asq_last_status);
>> + }
>> +
>> /* The main driver is (mostly) up and happy. We need to set this state
>> * before setting up the misc vector or we get a race and the vector
>> * ends up disabled forever.
>>
>> With this hunk removed the driver successfully unloaded/reloaded a
>> couple of hundred times. Would it be safe to just remove this hunk?
>> I haven't seen any negative effects by removing this yet.
>>
>> Stefan
>>
> Stefan,
> I wouldn't remove them yet as this does look like a valid idea to check to
> see if the link is
> restarting successfully. On the other hand can you try removing the msleep
> line as this one is
> most likely causing the issue due to sleeping for some long in a probe
> function is generally a
> bad idea.
> Thanks,
> Nick
Thanks Nick for the quick reply. I tested removing the msleep but that
didn't make a difference. You actually need to remove the complete hunk
to get a stable driver reload.
Stefan
------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel® Ethernet, visit
http://communities.intel.com/community/wired
