Page MenuHomeFreeBSD
Differential D16428

iflib: Changes for ixlv(4) update
AbandonedPublic
Actions

Authored by erj on Jul 24 2018, 6:34 PM.
Tags
Referenced Files
Unknown Object (File)
Sun, Dec 22, 6:32 AM
Unknown Object (File)
Sun, Dec 22, 6:31 AM
Unknown Object (File)
Fri, Dec 20, 8:03 PM
Unknown Object (File)
Fri, Dec 20, 7:54 PM
Unknown Object (File)
Fri, Dec 20, 7:51 PM
Unknown Object (File)
Fri, Dec 20, 6:44 PM
Unknown Object (File)
Fri, Dec 20, 2:10 PM
Unknown Object (File)
Sun, Dec 8, 1:30 AM
View All 78 Files
Subscribers
ae
imp

Details

Reviewers
shurd
sbruno
mmacy
Group Reviewers
Intel Networking
Summary
  • Add IFLIB_ADMIN_ALWAYS_RUN check to IOV task function so PF driver does not need to be up to handle VFLR events from VF
  • Add public iflib_request_reset() function so ixlv(4) can request a full up/down cycle after a HW VF reset; this just sets the IFC_DO_RESET flag so init runs again when _task_fn_admin next runs

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 19832
Build 19364: arc lint + arc unit

Event Timeline

erj created this revision.Jul 24 2018, 6:34 PM
Herald added subscribers: ae, imp. · View Herald TranscriptJul 24 2018, 6:34 PM
Harbormaster completed remote builds in B18307: Diff 45798.Jul 24 2018, 6:34 PM
erj mentioned this in D16429: ixlv(4): Update to use iflib; change name to iavf(4).Jul 24 2018, 6:37 PM
erj added a reviewer: mmacy.Jul 25 2018, 4:25 PM
sbruno accepted this revision.Jul 25 2018, 6:16 PM
This revision is now accepted and ready to land.Jul 25 2018, 6:16 PM
erj planned changes to this revision.Aug 2 2018, 5:07 PM

I need to add the state lock to iflib_request_reset

erj updated this revision to Diff 46481.Aug 9 2018, 6:39 PM
  • iflib: Prevent kernel panic on detach in _task_fn_admin by checking if in detach
  • iflib: Add STATE_LOCK() around flag setting in iflib_request_reset()
Harbormaster completed remote builds in B18681: Diff 46481.Aug 9 2018, 6:39 PM
erj updated this revision to Diff 46641.Aug 14 2018, 12:02 AM
  • iflib: Free remaining interrupt resources on failure in iflib_device_register()
  • iflib: Free RX queue TCP LRO memory on unload
Harbormaster completed remote builds in B18779: Diff 46641.Aug 14 2018, 12:02 AM
erj added inline comments.Aug 23 2018, 11:54 PM
sys/net/iflib.c
258–259

Can I get feedback on whether it's ok to use an atomic flag like this?

264

Ditto

mmacy added a comment.EditedAug 25 2018, 7:11 PM

I'm confused; this is just doing a normal store / load for the in detach - you need a lock to serialize reading it with other operations.

erj updated this revision to Diff 47510.Aug 31 2018, 12:05 AM
  • iflib: Destroy state lock at end of iflib_device_deregister()
  • iflib: Move ifc_in_detach variable to flag in if_ctx_t's ifc_flags, and lock around it
Harbormaster completed remote builds in B19308: Diff 47510.Aug 31 2018, 12:05 AM
erj added a comment.Sep 12 2018, 9:42 PM
In D16428#360259, @mmacy wrote:

I'm confused; this is just doing a normal store / load for the in detach - you need a lock to serialize reading it with other operations.

Does moving the in_detach value to ifc_flags and making sure to use the STATE_LOCK() when accessing it work?

Though honestly, I wonder if this whole "IFLIB_ADMIN_ALWAYS_RUN" in _task_fn_admin is a bad idea. Maybe we need something like iflib_timer(), but one that doesn't get started/stopped when the driver goes up or down. But 12 is coming soon, and ixlv(4) currently depends on these changes.

erj updated this revision to Diff 48494.Sep 27 2018, 12:57 AM
  • Revert "iflib: Free RX queue TCP LRO memory on unload"
Harbormaster completed remote builds in B19832: Diff 48494.Sep 27 2018, 12:57 AM
erj mentioned this in D17404: iflib: drain enqueued tasks before detaching from taskqgroup.Oct 5 2018, 8:16 PM
erj added a child revision: D16429: ixlv(4): Update to use iflib; change name to iavf(4).Oct 5 2018, 8:21 PM
erj removed a child revision: D16429: ixlv(4): Update to use iflib; change name to iavf(4).Oct 12 2018, 5:28 PM
erj abandoned this revision.Oct 12 2018, 10:59 PM

Revision Contents
Changeset List

PathSize
sys/
net/
4 lines
101 lines
1 line

Diff 48494

View Options

sys/net/iflib.h

Loading...
View Options

sys/net/iflib.c

Loading...
View Options

sys/net/iflib_private.h

Loading...