Skip to contentSkip to conversation

i40e: fix merge bug

PATCH New Jesse Brandeburg 2 participants 3 messages Jan 7, 2016 – Jan 8, 2016

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 i40e: fix merge bug

There was a bug (re-)introduced by a merge error when applying patches. This fixes the issue by making the code schedule the work item as originally intended for the fix included in commit 0e4425ed641f ("i40e:
fix: do not sleep in netdev_ops") and broken in commit ea02e90b4b49 ("i40e: propagate properly").

Fixes ea02e90b4b49 ("i40e: propagate properly")
---
 drivers/net/ethernet/intel/i40e/i40e_main.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
drivers/net/ethernet/intel/i40e/i40e_main.c +5 -1
index 5375d3e..590cf0b 100644
@@ -1542,7 +1542,11 @@ static int i40e_set_mac(struct net_device *netdev, void *p)
 
 	ether_addr_copy(netdev->dev_addr, addr->sa_data);
 
-	return i40e_sync_vsi_filters(vsi);
+	/* schedule our worker thread which will take care of
+	 * applying the new filter changes
+	 */
+	i40e_service_event_schedule(vsi->back);
+	return 0;
 }
 
 /**
Signature · 1 line
1.8.3.1
#1 lore raw

PATCH 1/1 i40e: fix merge bug

Quoted 8 lines · On Wed, 6 Jan 2016 17:06:31 -0800 Jesse Brandeburg…

There was a bug (re-)introduced by a merge error when applying patches. This fixes the issue by making the code schedule the work item as originally intended for the fix included in commit 0e4425ed641f ("i40e:
fix: do not sleep in netdev_ops") and broken in commit ea02e90b4b49 ("i40e: propagate properly").

Fixes ea02e90b4b49 ("i40e: propagate properly")

targeted at net-next, tested on next-queue/dev-queue, please send ASAP as this standalone patch applied cleanly to davem's net-next/master when tested just now.

#2 lore raw

PATCH 1/1 i40e: fix merge bug

In reply to Jesse Brandeburg
Quoted 8 lines · Jesse Brandeburg

There was a bug (re-)introduced by a merge error when applying patches.
This fixes the issue by making the code schedule the work item as originally intended for the fix included in commit 0e4425ed641f ("i40e:
fix: do not sleep in netdev_ops") and broken in commit ea02e90b4b49 ("i40e: propagate properly").

Quoted 5 lines
Fixes ea02e90b4b49 ("i40e: propagate properly")

---
?drivers/net/ethernet/intel/i40e/i40e_main.c | 6 +++++- ?1 file changed, 5 insertions(+), 1 deletion(-)

This does not apply because it is already fixed in the following patch that was applied earlier:

commit e91a9d1345bf8a75b988e4bc01a32610ce76084c
Author: Jesse Brandeburg <[email protected]> Date:???Thu Jan 7 22:47:29 2016 -0800

????i40e: fix: do not sleep in netdev_ops
????
????The driver was being called by vlan, bonding, teaming operations
????that expected to be able to hold locks like rcu_read_lock().
????
????This causes the driver to be held to the requirement to not sleep,
????and was found by the kernel debug options for checking sleep
????inside critical section, and the locking validator.
????
????Signed-off-by: Jesse Brandeburg <[email protected]>
????Reviewed-by: Nelson, Shannon <[email protected]>
????Change-ID: Ibc68c835f5ffa8ffe0638ffe910a66fc5649a7f7
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.osuosl.org/pipermail/intel-wired-lan/attachments/20160107/a6c42164/attachment.asc>