Page MenuHomeFreeBSD

shurd (Stephen Hurd)
User

Projects

User Details

User Since
Jun 19 2014, 6:57 AM (257 w, 4 d)

Recent Activity

Mon, May 13

shurd committed rS347542: MFC r346708:.
MFC r346708:
Mon, May 13, 6:48 PM

Wed, May 8

shurd created D20204: First cut at cleaning up MP_RING_NO_64BIT_ATOMICS code.
Wed, May 8, 10:33 PM

Mon, May 6

shurd accepted D20176: e1000: Correct and optimize interrupt handling.

Ok, I didn't think these functions were hot enough to be worth optimizing, and I'm not sure anything can be optimized around E1000_WRITE_*

Mon, May 6, 11:01 PM
shurd added inline comments to D20176: e1000: Correct and optimize interrupt handling.
Mon, May 6, 10:17 PM
shurd accepted D20175: iflib: Correct and optimize interrupt handling, assorted fixes and improvements.

Phew, great job. I've always been a bit uneasy about the MSI/INTx stuff, but this seems to have cleared everything up.

Mon, May 6, 10:08 PM

Thu, May 2

shurd accepted D20142: iflib: remove assertion that isc_capabilities is nonzero.

Makes sense.

Thu, May 2, 5:51 PM

Tue, Apr 30

shurd created D20110: Fix infinite loop on older hardware.
Tue, Apr 30, 4:55 PM

Mon, Apr 29

shurd updated the diff for D20031: Allow control over initial state of RTS and DTR lines.

Use rtsdtr as the stty argument to set the default mode, and -rtsdtr to
disable automatically asserting them on open().

Mon, Apr 29, 7:12 PM

Apr 26 2019

shurd updated the diff for D20031: Allow control over initial state of RTS and DTR lines.

Address feedback, add support to umcs.

Apr 26 2019, 6:18 PM

Apr 25 2019

shurd committed rS346708: iflib: Better control over queue core assignment.
iflib: Better control over queue core assignment
Apr 25 2019, 9:25 PM
shurd closed D20029: Better control over queue core assignment.
Apr 25 2019, 9:25 PM
shurd added inline comments to D20029: Better control over queue core assignment.
Apr 25 2019, 5:43 PM
shurd updated the diff for D20029: Better control over queue core assignment.

"Reference" count core offsets and free() when all drivers unloaded.

Apr 25 2019, 5:40 PM

Apr 24 2019

shurd added inline comments to D20029: Better control over queue core assignment.
Apr 24 2019, 8:36 PM
shurd updated the diff for D20029: Better control over queue core assignment.

Fix style(), clean up default sysctl values, log message on malloc() failure.

Apr 24 2019, 8:31 PM
shurd added a comment to D20031: Allow control over initial state of RTS and DTR lines.

Tagging @ian as well, as he may be interested in this for $work purposes.
I think this also needs to cover the DTR|RTS setting in ucom_open (^/sys/dev/usb/serial/usb_serial.c) for completeness sake, but I defer to @imp and @ian as they've likely got better intuition of how that should work.

Apr 24 2019, 4:26 PM

Apr 23 2019

shurd created D20031: Allow control over initial state of RTS and DTR lines.
Apr 23 2019, 8:47 PM
shurd added inline comments to D20029: Better control over queue core assignment.
Apr 23 2019, 7:11 PM
shurd created D20029: Better control over queue core assignment.
Apr 23 2019, 7:09 PM

Apr 22 2019

shurd committed rP499604: Fix multi-screen size handling, and fix test for wxWidgets.
Fix multi-screen size handling, and fix test for wxWidgets
Apr 22 2019, 5:45 AM

Apr 19 2019

shurd added a comment to D16402: Add TIOCFBAUD UART ioctl to allow fractional baudrates.

I'd really like to see this part split out into a separate review and committed soon, because it shouldn't see any contention- it adds control that is desirable to the people that know they need it, and doesn't really affect anyone else.

Apr 19 2019, 5:50 PM
shurd committed rP499373: Update to 4.1.26.
Update to 4.1.26
Apr 19 2019, 3:59 PM
shurd committed rP499342: Update to 1.3.2.
Update to 1.3.2
Apr 19 2019, 9:24 AM
shurd committed rP499340: Update to 0.9.7.2.
Update to 0.9.7.2
Apr 19 2019, 9:13 AM
shurd committed rP499338: Update to 1.3.5.
Update to 1.3.5
Apr 19 2019, 8:57 AM
shurd committed rP499337: Update to version 1.3.43.
Update to version 1.3.43
Apr 19 2019, 8:47 AM
shurd committed rP499336: Update to 1.2.6.
Update to 1.2.6
Apr 19 2019, 8:38 AM
shurd committed rP499334: Update to 4.1.02.
Update to 4.1.02
Apr 19 2019, 8:36 AM
shurd committed rP499330: Update to 9.3.3.
Update to 9.3.3
Apr 19 2019, 7:43 AM

Apr 17 2019

shurd committed rP499145: MFH: r499037.
MFH: r499037
Apr 17 2019, 3:01 AM

Apr 15 2019

shurd committed rP499037: Fix build when hamlib is installed..
Fix build when hamlib is installed.
Apr 15 2019, 5:52 PM

Mar 27 2019

shurd accepted D19652: iflib: return ENETDOWN when the network device is down.

lgtm

Mar 27 2019, 5:47 PM
shurd accepted D19645: add pfil hooks to iflib.
Mar 27 2019, 5:45 PM
shurd accepted D19604: iflib: hold the CTX lock in iflib_pseudo_register.

Looks good to me.

Mar 27 2019, 5:31 PM

Mar 14 2019

shurd accepted D19468: iflib: prevent possible infinite loop in iflib_encap.

lgtm

Mar 14 2019, 10:25 PM
Herald added a reviewer for D19468: iflib: prevent possible infinite loop in iflib_encap: iflib.
Mar 14 2019, 5:17 PM
shurd accepted D19489: iflib: expose the Rx mbuf buffer size to drivers.

And again now that I've joined iflib...

Mar 14 2019, 5:02 PM
shurd added a member for iflib: shurd.
Mar 14 2019, 5:01 PM
shurd accepted D19489: iflib: expose the Rx mbuf buffer size to drivers.

Accepted. (derp)

Mar 14 2019, 5:00 PM
Herald added a reviewer for D19489: iflib: expose the Rx mbuf buffer size to drivers: iflib.

lgtm

Mar 14 2019, 4:59 PM

Feb 28 2019

shurd accepted D19402: ixv(4): Add missing IFLIB_IS_VF flag in iflib shared ctx.

LGTM

Feb 28 2019, 6:01 PM

Feb 22 2019

shurd committed rS344472: MFC r333131:.
MFC r333131:
Feb 22 2019, 6:25 PM
shurd committed rS344471: MFC r344162:.
MFC r344162:
Feb 22 2019, 6:03 PM

Feb 15 2019

shurd committed rS344162: iflib: Improve return values of interrupt handlers..
iflib: Improve return values of interrupt handlers.
Feb 15 2019, 6:52 PM
shurd closed D19201: iflib: Return the correct filter result in the interrupt handler..
Feb 15 2019, 6:52 PM

Feb 14 2019

shurd updated the diff for D19201: iflib: Return the correct filter result in the interrupt handler..

style(9) nits.

Feb 14 2019, 10:10 PM
shurd updated the diff for D19201: iflib: Return the correct filter result in the interrupt handler..

FILTER_SCHEDULE_THREAD is a flag, not a discrete value. If it's set, schedule the
gtask and return FILTER_HANDLED.

Feb 14 2019, 10:08 PM
shurd updated the diff for D19201: iflib: Return the correct filter result in the interrupt handler..

Return FILTER_HANDLED rather than FILTER_SCHEDULE_THREAD when gtask (potentially) scheduled
per @waddlesplash_gmail.com.

Feb 14 2019, 9:56 PM
shurd updated the diff for D19201: iflib: Return the correct filter result in the interrupt handler..

Some bits of the upstream patch didn't paply cleanly... added manually.

Feb 14 2019, 9:24 PM
shurd updated the diff for D19201: iflib: Return the correct filter result in the interrupt handler..

Put the result variable back in. *facepalm*

Feb 14 2019, 7:49 PM
shurd updated the diff for D19201: iflib: Return the correct filter result in the interrupt handler..

Unpack the conditional for easier reading.

Feb 14 2019, 7:43 PM
shurd updated the diff for D19201: iflib: Return the correct filter result in the interrupt handler..

Remove unrelated stuff.

Feb 14 2019, 7:36 PM
shurd created D19201: iflib: Return the correct filter result in the interrupt handler..
Feb 14 2019, 7:36 PM
shurd accepted D19199: remove references to CONTIGMALLOC_WORKS in iflib.

Looks good to me.

Feb 14 2019, 7:33 PM

Feb 5 2019

shurd committed rP492276: Remove obsolete features, fixed blacklist.
Remove obsolete features, fixed blacklist
Feb 5 2019, 11:31 PM

Feb 4 2019

shurd accepted D19067: Change iflib(4) to employ the usual convention of "descriptor".

I love everything about this. If you plan to use the summary as the commit message though you should modify "change the M_NOWAIT from malloc(9) calls into M_NOWAIT." bit to mention M_WAITOK.

Feb 4 2019, 8:18 PM

Jan 28 2019

shurd accepted D18980: Stop iflib(4) from leaking MSI messages and along with drivers let it use the correct RIDs when releasing resources.

Looks good. This may also fix where bar == -1 && pci_msix_count(dev) == 0 (or break it less... or something)

Jan 28 2019, 11:59 PM
shurd added a comment to D18980: Stop iflib(4) from leaking MSI messages and along with drivers let it use the correct RIDs when releasing resources.

I think that the inability to map an MSI-X table should likely not be bootverbose... this is an allocation failure which will significantly impact device performance. While the user should know that MSI-X is disabled, I'm not sure the user should be expected to know that the allocation will fail.

Jan 28 2019, 6:58 PM

Jan 25 2019

shurd accepted D16164: Make options IFLIB a real configuration option..

No objections, though I do agree with cem@ in principle regarding putting it in the GENERICs. When 3rd-party drivers start relying on it, it would violate POLA removing Intel NICs from the kernel would break a 3rd-party driver. For now it's likely "fine".

Jan 25 2019, 6:50 PM
shurd committed rS343441: MFC r343047:.
MFC r343047:
Jan 25 2019, 6:30 PM

Jan 18 2019

shurd abandoned D18861: Attempt to address lagg race in PR234864.
Jan 18 2019, 5:42 PM

Jan 17 2019

shurd added a comment to D18861: Attempt to address lagg race in PR234864.

So the basic goal is to have the test at sys/net/if_ethersubr.c:583 see a consistent state of the changes from the /* Change the interface type */ line to the lagg_proto_addport() line.

Jan 17 2019, 5:45 PM

Jan 16 2019

shurd updated the diff for D18861: Attempt to address lagg race in PR234864.

RLOCK() before setting ifp->if_type

Jan 16 2019, 10:36 PM
shurd created D18861: Attempt to address lagg race in PR234864.
Jan 16 2019, 10:34 PM
shurd committed rS343099: MFC r342855:.
MFC r342855:
Jan 16 2019, 7:20 PM

Jan 15 2019

shurd closed D18821: Fix window update issue when scaling disabled.
Jan 15 2019, 5:40 PM
shurd committed rS343047: Fix window update issue when scaling disabled.
Fix window update issue when scaling disabled
Jan 15 2019, 5:40 PM

Jan 14 2019

shurd committed rS343024: MFC r342855:.
MFC r342855:
Jan 14 2019, 6:40 PM

Jan 11 2019

shurd created D18821: Fix window update issue when scaling disabled.
Jan 11 2019, 5:16 PM

Jan 8 2019

shurd accepted D18761: Convert vmx(4) to iflib.

My only concern here is making IFLIB_MAX_RX_SEGS available to the driver...

In the current iflib implementation, the reality of course is that IFLIB_MAX_RX_SEGS is the upper limit of what a driver can use. My approach is to just give it as much runway as the current iflib implementation allows.

Jan 8 2019, 5:12 PM

Jan 7 2019

shurd committed rS342855: Use iflib_if_init_locked() during resume instead of iflib_init_locked()..
Use iflib_if_init_locked() during resume instead of iflib_init_locked().
Jan 7 2019, 11:47 PM
shurd added a comment to D18761: Convert vmx(4) to iflib.

My only concern here is making IFLIB_MAX_RX_SEGS available to the driver...

Jan 7 2019, 11:41 PM
shurd accepted D18760: Fix error-path resource leaks in iflib registration routines.

Looks good, only a tiny nitpick on the fail label.

Jan 7 2019, 9:03 PM
shurd added a comment to D18532: Use byte-counting rather than packet counting for TX batch size.

Hi,
I've got igb only NIC, not em, so I can't use my lab to bench this new review version.

Jan 7 2019, 8:42 PM

Dec 27 2018

shurd committed rP488507: Update to 2.0.0.
Update to 2.0.0
Dec 27 2018, 4:46 AM

Dec 20 2018

shurd updated the diff for D18532: Use byte-counting rather than packet counting for TX batch size.

In addition to trying to keep the TXQ full, use a mp_ring size that's half the
number of descriptors. Previously, the mp_ring was a fixed size which happened
to be twice the default size of the txq for my em devices.

Dec 20 2018, 7:59 PM

Dec 18 2018

shurd committed rS342194: MFC r341824:.
MFC r341824:
Dec 18 2018, 5:31 PM
shurd accepted D18545: intel: correct initialization of tx_cidx_processed.

lgtm

Dec 18 2018, 4:30 AM

Dec 14 2018

shurd added a comment to D18545: intel: correct initialization of tx_cidx_processed.

I'm not entirely certain how to reproduce the original setup that triggered the bug, so some help with that would be appreciated.

Dec 14 2018, 8:36 PM
shurd updated the diff for D18532: Use byte-counting rather than packet counting for TX batch size.

Remove the drain limit completely and instead try to fill the TX queue on each drain.

Dec 14 2018, 6:03 AM
shurd added a comment to D18532: Use byte-counting rather than packet counting for TX batch size.

tx_abdicate still brings lot's major gain against D18532.
Theory invalided ?

Dec 14 2018, 6:02 AM

Dec 12 2018

shurd created D18532: Use byte-counting rather than packet counting for TX batch size.
Dec 12 2018, 7:09 PM

Dec 11 2018

shurd committed rS341824: Fix !tx_abdicate error from r336560.
Fix !tx_abdicate error from r336560
Dec 11 2018, 5:46 PM

Dec 5 2018

shurd added a comment to D18402: netmap: ixgbe: add support for jumbo frames.

I didn't think this code was still in use... AFAIK, ixgbe uses the iflib netmap support now.

Dec 5 2018, 3:17 PM

Dec 3 2018

shurd committed rS341427: MFC r341156:.
MFC r341156:
Dec 3 2018, 3:21 PM

Nov 28 2018

shurd closed D18368: Fix first-packet completion.
Nov 28 2018, 5:38 PM
shurd committed rS341156: Fix first-packet completion.
Fix first-packet completion
Nov 28 2018, 5:38 PM
shurd created D18368: Fix first-packet completion.
Nov 28 2018, 3:25 PM

Nov 21 2018

shurd committed rS340735: MFS r340729:.
MFS r340729:
Nov 21 2018, 5:33 PM
shurd committed rS340729: MFC r340435:.
MFC r340435:
Nov 21 2018, 5:15 PM

Nov 19 2018

shurd committed rS340634: MFS r340629.
MFS r340629
Nov 19 2018, 4:40 PM
shurd committed rS340629: MFC r340434, r340445.
MFC r340434, r340445
Nov 19 2018, 3:18 PM

Nov 14 2018

shurd committed rS340445: Clear RX completion queue state veriables in iflib_stop().
Clear RX completion queue state veriables in iflib_stop()
Nov 14 2018, 8:36 PM
shurd committed rS340435: Prevent POLA violation with TSO/CSUM offload.
Prevent POLA violation with TSO/CSUM offload
Nov 14 2018, 3:23 PM
shurd closed D17801: TSO inherently requires checksum offload. In the case where TSO is in use, force the checksum offload on as well for that packet..
Nov 14 2018, 3:23 PM
shurd committed rS340434: Fix leaks caused by ifc_nhwtxqs never being initialized.
Fix leaks caused by ifc_nhwtxqs never being initialized
Nov 14 2018, 3:17 PM
shurd closed D17979: Fix leaks caused by ifc_nhwtxqs never being initialized.
Nov 14 2018, 3:17 PM

Nov 13 2018

shurd added a comment to D17979: Fix leaks caused by ifc_nhwtxqs never being initialized.

Style-wise, would it be better to set up an sctx local at the top of these routines and then dereference that to avoid the double-dereference noise in the code?

Nov 13 2018, 8:09 PM
shurd updated the diff for D17979: Fix leaks caused by ifc_nhwtxqs never being initialized.

Address feedback.

Nov 13 2018, 8:09 PM
shurd created D17979: Fix leaks caused by ifc_nhwtxqs never being initialized.
Nov 13 2018, 7:17 PM