kmacy (Kip Macy)
User

Projects

User Details

User Since
Jan 24 2015, 12:39 AM (155 w, 4 d)

Recent Activity

Tue, Dec 19

kmacy added a comment to D13531: Test dumping instead of SCHEDULER_STOPPED() since the latter isn'ttrue when doadump is called from ddb..

This seems to fix dumps from panics caused by in-kernel page faults.

Tue, Dec 19, 2:57 AM

Nov 12 2017

kmacy created D13062: Add vmm(4) man page.
Nov 12 2017, 7:10 PM

Oct 31 2017

kmacy added a comment to D12101: swfw_sync DELAY -> sleep conversion.

So let me try to sum up in my own words what's going on here:

  1. We want to be able to sleep when talking to the hardware, rather than poll
  2. We want to change the driver lock from a mutex to an sx lock to allow sleeping when talking to h/w
  3. We can't just change to an SX lock and be done because the multicast code holds a mutex (and maybe an RW lock) when calling ifioctl
  4. We noticed that the multicast code does not check return values from driver ioctls, so it seems safe to defer it into another context
  5. We decided to defer mutlicast (and promisc, since it can be coming from the multicast code) into another context at the iflib level

    While, obviously the best solution would be to refactor the multicast code to avoid holding a lock, I think that I generally agree with the approach, and think it is very clever.

    However, if we're going to defer the mutlicast code into another context, can we please consider *NOT* doing it at the iflib level? I'm thinking we could re-work this hack so this works for all drivers if we made an mcast kproc and submitted mutlicast add/del requests to a global mcast work queue. Then the deferred multicast kproc can call down into drivers with no locks held. Then we assert no locks in the ifioctl entry points.
Oct 31 2017, 4:40 AM · network

Sep 11 2017

kmacy added a comment to D12295: Kill gets().

This is silly to bikeshed. 12.0 wont be released for at least a year and -CURRENT is for API breaks. Users can run the current -STABLE trees in a jail if they have any shitware they need to support for a long time. It's also trivial to maintain the patch in a corporate tree if needed, or a compat lib that doesn't live in src/.

Sep 11 2017, 11:19 PM

Sep 6 2017

kmacy added a comment to D9930: Bhyve cpu topology control.

@rgrimes what is the state of this review?

Sep 6 2017, 11:56 PM

Sep 4 2017

kmacy updated subscribers of D12140: fix iflib netmap rx.

I did a clean world+kernel build on iflib/netmap_rx branch.

Running pkt-gen -i em1 -f rx on receiver and pkt-gen -i em1 -f tx on the sender results in crash on receiver.
(sender and receiver two different machines connected back to back on em1)

db_trace_self_wrapper() at db_trace_self_wrapper+0x2b/frame 0xfffffe03668b2510
vpanic() at vpanic+0x19c/frame 0xfffffe03668b2590
kassert_panic() at kassert_panic+0x126/frame 0xfffffe03668b2600
em_isc_rxd_pkt_get() at em_isc_rxd_pkt_get+0xf1/frame 0xfffffe03668b2670
iflib_netmap_rxsync() at iflib_netmap_rxsync+0x235/frame 0xfffffe03668b2770
netmap_poll() at netmap_poll+0x79c/frame 0xfffffe03668b2870
freebsd_netmap_poll() at freebsd_netmap_poll+0x32/frame 0xfffffe03668b28a0
devfs_poll_f() at devfs_poll_f+0x7f/frame 0xfffffe03668b2900
kern_poll() at kern_poll+0x4fc/frame 0xfffffe03668b2aa0
sys_poll() at sys_poll+0x50/frame 0xfffffe03668b2ac0
amd64_syscall() at amd64_syscall+0x79b/frame 0xfffffe03668b2bf0
Xfast_syscall() at Xfast_syscall+0xfb/frame 0xfffffe03668b2bf0
--- syscall (209, FreeBSD ELF64, sys_poll), rip = 0x800daf2aa, rsp = 0x7fffdfff9e78, rbp = 0x7fffdfff9eb0 ---
KDB: enter: panic
[ thread pid 828 tid 100177 ]
Stopped at      kdb_enter+0x3b: movq    $0,kdb_why

However, if I limit the rate of packets I like so
pkt-gen -i em1 -f rx on receiver and pkt-gen -i em1 -f tx -R 100 on the sender
the receiver stops receiving packets after ~600 packets or so (6 batches received), after that rate on receiver goes to zero but does not crash.

Breaking the process might result in crash:

cpuid = 2                 
time = 1504082223         
KDB: stack backtrace:     
db_trace_self_wrapper() at db_trace_self_wrapper+0x2b/frame 0xfffffe02398c0730                           
vpanic() at vpanic+0x19c/frame 0xfffffe02398c07b0   
kassert_panic() at kassert_panic+0x126/frame 0xfffffe02398c0820                                          
iflib_fl_bufs_free() at iflib_fl_bufs_free+0x1c2/frame 0xfffffe02398c0870                                
iflib_stop() at iflib_stop+0x478/frame 0xfffffe02398c08c0                                                
iflib_netmap_register() at iflib_netmap_register+0x1a4/frame 0xfffffe02398c0900                          
netmap_hw_reg() at netmap_hw_reg+0x2c/frame 0xfffffe02398c0930                                           
netmap_do_unregif() at netmap_do_unregif+0x16a/frame 0xfffffe02398c0960                                  
netmap_priv_delete() at netmap_priv_delete+0x31/frame 0xfffffe02398c0980                                 
netmap_dtor() at netmap_dtor+0x2b/frame 0xfffffe02398c09a0                                               
devfs_destroy_cdevpriv() at devfs_destroy_cdevpriv+0x8b/frame 0xfffffe02398c09c0                         
devfs_close_f() at devfs_close_f+0x65/frame 0xfffffe02398c09f0                                           
closef() at closef+0x1f5/frame 0xfffffe02398c0a80   
closefp() at closefp+0x9f/frame 0xfffffe02398c0ac0  
amd64_syscall() at amd64_syscall+0x79b/frame 0xfffffe02398c0bf0                                          
Xfast_syscall() at Xfast_syscall+0xfb/frame 0xfffffe02398c0bf0

igb0 interface works flawlessly at 1.4 Mpps so problem limited to em

Sep 4 2017, 5:49 PM

Aug 30 2017

kmacy abandoned D6694: Update intel_gtt_get to its modern incarnation.
Aug 30 2017, 11:09 PM
kmacy abandoned D6590: Fix seq namespace collisions.
Aug 30 2017, 11:08 PM
kmacy abandoned D1963: reduce cache coherence traffic on br_ring accesses.
Aug 30 2017, 11:08 PM
kmacy abandoned D6589: AcpiGetDataFull / AcpiGetTableWithSize.
Aug 30 2017, 11:07 PM

Aug 29 2017

kmacy added a comment to D12132: Avoid spinning in random_harvest_queue.
In D12132#252205, @mjg wrote:

since this is a spin mutex even failed trylock adds a trip through disabling/enabling preemption + interrupts. add a cacheline fetch. "fortunately" it so happens that spin trylock does not dirty it if it sees a taken lock so this bit is not extremely slow. the sum is definitely significantly slower than necessary.

I think a much better approach would be to have this rate-limited instead. use a per-cpu counter and grab every n'th packet.

longer term this should probably gather per-cpu and rollup once a second or similar

Aug 29 2017, 4:02 AM
kmacy added a comment to D12137: Adaptively enable/disable entropy collection from packets.

@sbruno I've created an iflib/adaptive_entropy branch with this change. It's dependent on the master-bogofix branch.

Aug 29 2017, 1:25 AM
kmacy updated the diff for D12140: fix iflib netmap rx.

fix merge error

Aug 29 2017, 12:49 AM
kmacy updated the diff for D12142: Don't start grouptaskqueue threads for unused grouptaskqueues.

Only launch as many threads as we actually need
@sbruno dedicated branch vs iflib/ithread_dispatch called iflib/pollution reduction

Aug 29 2017, 12:41 AM
kmacy updated the diff for D12138: Add support for using ithread style dispatch for grouptaskqueue threads.
Aug 29 2017, 12:39 AM
kmacy added a comment to D12140: fix iflib netmap rx.

@sbruno @johalun0_gmail.com I've created a dedicated branch for this fix ifilb/netmap_rx

Aug 29 2017, 12:37 AM
kmacy updated the diff for D12140: fix iflib netmap rx.

avoid gratuitous ithread dispatch

Aug 29 2017, 12:36 AM
kmacy abandoned D4490: inpcb rtentry/l2 prepend caching.
Aug 29 2017, 12:25 AM · transport
kmacy abandoned D1677: defer inactive processing.
Aug 29 2017, 12:25 AM
kmacy abandoned D1676: remove vm object cache.
Aug 29 2017, 12:24 AM
kmacy abandoned D7016: Add CK.
Aug 29 2017, 12:23 AM
kmacy abandoned D7017: Add basic wrapper for CK's EBR.
Aug 29 2017, 12:23 AM

Aug 28 2017

kmacy added a comment to D12140: fix iflib netmap rx.
em0@pci0:0:31:6:	class=0x020000 card=0x06db1028 chip=0x156f8086 rev=0x21 hdr=0x00
    device     = 'Ethernet Connection I219-LM'

Tested with netmap's pkt-gen

On receive I get crash:

#7  0xffffffff80a72006 in kassert_panic (
   fmt=0xffffffff810482e7 "Assertion %s failed at %s:%d")
   at /usr/home/johannes/dev/freebsd/freebsd-base-graphics/sys/kern/kern_shutdown.c:669
#8  0xffffffff8053a20f in em_isc_rxd_pkt_get (arg=<unavailable>,
   ri=<optimized out>)
   at /usr/home/johannes/dev/freebsd/freebsd-base-graphics/sys/dev/e1000/em_txrx.c:697
#9  0xffffffff80b81288 in iflib_netmap_rxsync (kring=<optimized out>,
   flags=<optimized out>)
   at /usr/home/johannes/dev/freebsd/freebsd-base-graphics/sys/net/iflib.c:1008
#10 0xffffffff806c12fc in netmap_poll (priv=<optimized out>,
   events=<optimized out>, sr=0xfffff80220420560)
   at /usr/home/johannes/dev/freebsd/freebsd-base-graphics/sys/dev/netmap/netmap.c:2724
#11 0xffffffff806c3a02 in freebsd_netmap_poll (cdevi=<optimized out>,
   events=1, td=0xfffff80220420560)
   at /usr/home/johannes/dev/freebsd/freebsd-base-graphics/sys/dev/netmap/netmap_freebsd.c:1393
#12 0xffffffff80941eef in devfs_poll_f (fp=0xfffff8006cdd9c30, events=1,
   cred=0xfffff8000c4bba00, td=0xfffff80220420560)

On transmit I get:

621.256106 sender_body [1181] start, fd 3 main_fd 3
622.257327 main_thread [2056] 1.022 Kpps (1.023 Kpkts 491.040 Kbps in 1001225 usec) 511.50 avg_batch 0 min_space
623.258329 main_thread [2056] 0.000 pps (0.000 pkts 0.000 bps in 1001002 usec) 0.00 avg_batch 99999 min_space
623.260011 sender_body [1250] poll error/timeout on queue 0: No error: 0
624.259329 main_thread [2056] 1.023 Kpps (1.024 Kpkts 491.520 Kbps in 1000999 usec) 341.33 avg_batch 99999 min_space
625.263889 main_thread [2056] 0.000 pps (0.000 pkts 0.000 bps in 1004548 usec) 0.00 avg_batch 99999 min_space
625.263876 sender_body [1250] poll error/timeout on queue 0: No error: 0
626.265335 main_thread [2056] 1.023 Kpps (1.024 Kpkts 491.520 Kbps in 1001458 usec) 341.33 avg_batch 99999 min_space
627.265335 sender_body [1250] poll error/timeout on queue 0: No error: 0
627.276066 main_thread [2056] 1.013 Kpps (1.024 Kpkts 491.520 Kbps in 1010731 usec) 341.33 avg_batch 99999 min_space
628.276338 main_thread [2056] 0.000 pps (0.000 pkts 0.000 bps in 1000273 usec) 0.00 avg_batch 99999 min_space

The packets that do get transmitted are received by receiving machine.

Aug 28 2017, 4:29 PM
kmacy updated subscribers of D12142: Don't start grouptaskqueue threads for unused grouptaskqueues.
Aug 28 2017, 2:20 AM
kmacy created D12142: Don't start grouptaskqueue threads for unused grouptaskqueues.
Aug 28 2017, 2:19 AM
kmacy created D12140: fix iflib netmap rx.
Aug 28 2017, 12:11 AM

Aug 27 2017

kmacy created D12138: Add support for using ithread style dispatch for grouptaskqueue threads.
Aug 27 2017, 8:00 PM
kmacy created D12137: Adaptively enable/disable entropy collection from packets.
Aug 27 2017, 7:55 PM
kmacy added a comment to D12132: Avoid spinning in random_harvest_queue.

It occurred to me that we can both have our way by enabling the driver to disable entropy collection on packets.

Aug 27 2017, 6:49 PM
kmacy added a comment to D12132: Avoid spinning in random_harvest_queue.
Aug 27 2017, 5:44 PM
kmacy added a comment to D12132: Avoid spinning in random_harvest_queue.
Aug 27 2017, 5:33 PM

Aug 26 2017

kmacy added a comment to D12101: swfw_sync DELAY -> sleep conversion.

Huh .... Testing this review by itself this morning. I see the following startup panic:

Sleeping on "e1000_delay" with the following non-sleepable locks held:
exclusive sleep mutex em2:tx(0):callo (em2:tx(0):callo) r = 0 (0xfffff80004175068) locked @ /home/sbruno/bsd/fbsd_head/sys/kern/kern_mutex.c:182
stack backtrace:
#0 0xffffffff80ac85d3 at witness_debugger+0x73
#1 0xffffffff80ac99cf at witness_warn+0x43f
#2 0xffffffff80a708fc at _sleep+0x6c
#3 0xffffffff80a71117 at pause_sbt+0x117
#4 0xffffffff805620e1 at e1000_read_phy_reg_mdic+0xf1
#5 0xffffffff80562b5b at e1000_read_phy_reg_igp+0x5b
#6 0xffffffff80535d6c at em_if_timer+0xcc
#7 0xffffffff80b76279 at iflib_timer+0x69
#8 0xffffffff80a7e375 at softclock_call_cc+0x155
#9 0xffffffff80a7e75c at softclock+0x7c
#10 0xffffffff80a2bed1 at intr_event_execute_handlers+0x91
#11 0xffffffff80a2c5d6 at ithread_loop+0xb6
#12 0xffffffff80a29304 at fork_exit+0x84
#13 0xffffffff80ecf7ce at fork_trampoline+0xe
panic: sleepq_add: td 0xfffff80003d59000 to sleep on wchan 0xffffffff81c7d260 with sleeping prohibited
cpuid = 0
time = 4
KDB: stack backtrace:
db_trace_self_wrapper() at db_trace_self_wrapper+0x2b/frame 0xfffffe00f55f4730
vpanic() at vpanic+0x19c/frame 0xfffffe00f55f47b0
kassert_panic() at kassert_panic+0x126/frame 0xfffffe00f55f4820
sleepq_add() at sleepq_add+0x34f/frame 0xfffffe00f55f4870
_sleep() at _sleep+0x26c/frame 0xfffffe00f55f4910
pause_sbt() at pause_sbt+0x117/frame 0xfffffe00f55f4960
e1000_read_phy_reg_mdic() at e1000_read_phy_reg_mdic+0xf1/frame 0xfffffe00f55f49b0
e1000_read_phy_reg_igp() at e1000_read_phy_reg_igp+0x5b/frame 0xfffffe00f55f49e0
em_if_timer() at em_if_timer+0xcc/frame 0xfffffe00f55f4a10
iflib_timer() at iflib_timer+0x69/frame 0xfffffe00f55f4a40
softclock_call_cc() at softclock_call_cc+0x155/frame 0xfffffe00f55f4af0
softclock() at softclock+0x7c/frame 0xfffffe00f55f4b20
intr_event_execute_handlers() at intr_event_execute_handlers+0x91/frame 0xfffffe00f55f4b60
ithread_loop() at ithread_loop+0xb6/frame 0xfffffe00f55f4bb0
fork_exit() at fork_exit+0x84/frame 0xfffffe00f55f4bf0
fork_trampoline() at fork_trampoline+0xe/frame 0xfffffe00f55f4bf0
--- trap 0, rip = 0, rsp = 0, rbp = 0 ---
KDB: enter: panic
[ thread pid 12 tid 100023 ]
Stopped at      kdb_enter+0x3b: movq    $0,kdb_why
Aug 26 2017, 11:44 PM · network
kmacy updated the diff for D12101: swfw_sync DELAY -> sleep conversion.

move more blocking work to deferred context

Aug 26 2017, 11:43 PM · network
kmacy added a comment to D12132: Avoid spinning in random_harvest_queue.
Aug 26 2017, 8:29 PM

Aug 25 2017

kmacy added inline comments to D11232: ixgbe(4): Update HEAD (p3) to 3.2.12-k.
Aug 25 2017, 10:55 PM

Aug 24 2017

kmacy abandoned D11969: refactoring in support of *future* change to cope with slow configuration path on INTC and BRCM drivers.

Folded in to D12101

Aug 24 2017, 10:07 PM
kmacy updated the diff for D12101: swfw_sync DELAY -> sleep conversion.
  • Eliminate remaining mutex use in os independent e1000 code.
  • Fold in iflib updates.
Aug 24 2017, 10:06 PM · network
kmacy added a comment to D12101: swfw_sync DELAY -> sleep conversion.
In D12101#251488, @erj wrote:

The busy waiting would cause 10% CPU usage on my system when no cable connected to em0 (traced to em_if_update_admin_status).
This patch also fixes this. I can no longer detect any unusual CPU usage.

Getting a lot of witness output. Like a few per second or so.

I think this revision depends on D11969. It changes the core lock from a mutex to an sx lock, which would allow sleeping while its held.

Aug 24 2017, 10:02 PM · network
kmacy added a comment to D12101: swfw_sync DELAY -> sleep conversion.

@jtl any comments?

Aug 24 2017, 4:24 AM · network
kmacy added a reviewer for D12101: swfw_sync DELAY -> sleep conversion: jtl.
Aug 24 2017, 4:23 AM · network

Aug 23 2017

kmacy added a comment to D12101: swfw_sync DELAY -> sleep conversion.

@erj I updated the summary - is there anything more I should add?

Aug 23 2017, 8:49 PM · network
kmacy updated the summary of D12101: swfw_sync DELAY -> sleep conversion.
Aug 23 2017, 8:49 PM · network
kmacy updated subscribers of D12101: swfw_sync DELAY -> sleep conversion.
Aug 23 2017, 8:45 PM · network
kmacy added a comment to D11232: ixgbe(4): Update HEAD (p3) to 3.2.12-k.

Also, can we stop massively refactoring the drivers until after the iflib version goes in to HEAD? And in general, there's a lot of non-productive function shuffling that goes on in Intel drivers that makes it a lot more work for downstream users maintaining their own branches.

Aug 23 2017, 6:35 AM
kmacy added a reviewer for D11232: ixgbe(4): Update HEAD (p3) to 3.2.12-k: shurd.
Aug 23 2017, 6:33 AM
kmacy updated subscribers of D11232: ixgbe(4): Update HEAD (p3) to 3.2.12-k.

In the future when you find yourselves implementing synchronization primitives like swfw_sync or your hand rolled locks in if_bypass.c please make sure you get the attention and sign off from @markj, @jhb, or myself.

Aug 23 2017, 6:32 AM

Aug 22 2017

kmacy updated the summary of D12102: add tunable to eliminate conflict misses when frequently acccessing the first cache line of mbuf & cluster.
Aug 22 2017, 6:11 PM · network
kmacy added a comment to D12101: swfw_sync DELAY -> sleep conversion.
In D12101#251141, @erj wrote:

Can we also get a description of what this fixes, for future reference?

Aug 22 2017, 6:02 PM · network
kmacy created D12102: add tunable to eliminate conflict misses when frequently acccessing the first cache line of mbuf & cluster.
Aug 22 2017, 5:48 AM · network
kmacy updated the diff for D12101: swfw_sync DELAY -> sleep conversion.

update to reflect potential for brokenness

Aug 22 2017, 5:04 AM · network
kmacy created D12101: swfw_sync DELAY -> sleep conversion.
Aug 22 2017, 2:46 AM · network

Aug 12 2017

kmacy retitled D11969: refactoring in support of *future* change to cope with slow configuration path on INTC and BRCM drivers from incremental fixes towards coping with slow configuration path on INTC and BRCM drivers to refactoring in support of *future* change to cope with slow configuration path on INTC and BRCM drivers.
Aug 12 2017, 7:42 PM
kmacy added a comment to D11969: refactoring in support of *future* change to cope with slow configuration path on INTC and BRCM drivers.
In D11969#248789, @jtl wrote:

Please review the FreeBSD locking hierarchy. SX locks can't be acquired after default mutexes.

I'm familiar with the FreeBSD locking hierarchy. What I don't see (and, since I may be missing it, I'm asking you to point it out) is where the code changes actually require the unbounded sleep that SX locks allow. Understanding this will help me to better understand the rationale for this change.

Aug 12 2017, 7:24 PM

Aug 11 2017

kmacy added a comment to D11969: refactoring in support of *future* change to cope with slow configuration path on INTC and BRCM drivers.
In D11969#248529, @erj wrote:

Can you remind me what lock is held?

I don't know about other ioctls, but I remember there being a lock held when IPv6 multicast MAC address are added.

Unrelated to that, for ixl(4), nvmupdate uses ioctls to the update, but it needs the returned error code from the ioctl call (from the driver) to work. So ixl(4) wouldn't work with this change.

Aug 11 2017, 7:32 PM
kmacy added a comment to D11969: refactoring in support of *future* change to cope with slow configuration path on INTC and BRCM drivers.

I'm also unsure about the benefit of deferring some of the work to run asynchronously in a different thread. Again, I may be missing something, but it looks to me like the same code runs, but it is deferred. In some cases, this has user-visible side effects. For example, there could be a period of time after an ioctl call has "successfully" completed when the behavior will be different than expected. This might be OK if the behavior was well-documented and there was a good reason for it. But, I'm not sure the change description explains why this is necessary. (Again, I might be missing something.)

Aug 11 2017, 7:21 PM
kmacy added a comment to D11969: refactoring in support of *future* change to cope with slow configuration path on INTC and BRCM drivers.

I talked to Kevin. I may need to clarify my thinking a bit more.

Aug 11 2017, 7:15 PM
kmacy added inline comments to D11969: refactoring in support of *future* change to cope with slow configuration path on INTC and BRCM drivers.
Aug 11 2017, 7:06 PM

Aug 10 2017

kmacy updated the summary of D11969: refactoring in support of *future* change to cope with slow configuration path on INTC and BRCM drivers.
Aug 10 2017, 10:46 PM
kmacy created D11969: refactoring in support of *future* change to cope with slow configuration path on INTC and BRCM drivers.
Aug 10 2017, 10:42 PM

Jul 7 2017

kmacy added inline comments to D11476: iflib - try m_defrag when m_collapse fails.
Jul 7 2017, 12:30 AM
kmacy added a comment to D11476: iflib - try m_defrag when m_collapse fails.
In D11476#238004, @ae wrote:

From a quick look, the iflib code does not bind irq to CPU cores. The old em/igb drivers did that and I guess, if you add bus_bind_intr() again, this will increase the performance.

It does. The problem is that FreeBSD goes through the entire forwarding path for every single packet, not supporting any sort of batching. There is a doorbell coalesce optimization for content serving that is a pessimization for forwarding on FreeBSD.

Jul 7 2017, 12:28 AM
kmacy added inline comments to D11476: iflib - try m_defrag when m_collapse fails.
Jul 7 2017, 12:27 AM
kmacy added a comment to D11476: iflib - try m_defrag when m_collapse fails.
In D11476#238004, @ae wrote:

From a quick look, the iflib code does not bind irq to CPU cores. The old em/igb drivers did that and I guess, if you add bus_bind_intr() again, this will increase the performance.

Jul 7 2017, 12:20 AM

Jul 5 2017

kmacy added a comment to D11476: iflib - try m_defrag when m_collapse fails.

Thanks for your review.

Jul 5 2017, 7:09 PM

May 26 2017

kmacy added a comment to D8556: Fix spurious retransmit recovery on low latency networks.

@rstone have you committed an equivalent fix already?

May 26 2017, 10:43 PM · network
kmacy updated the summary of D10894: Hystart.
May 26 2017, 5:54 AM

May 25 2017

kmacy created D10894: Hystart.
May 25 2017, 2:51 AM

May 24 2017

kmacy added a comment to D10556: Update cubic constants.

Thanks!

May 24 2017, 3:00 AM · network

May 23 2017

kmacy added a comment to D10556: Update cubic constants.

I updated my primary e-mail. I didn't see your update. Thanks.

May 23 2017, 4:39 AM · network
kmacy added inline comments to D10556: Update cubic constants.
May 23 2017, 4:37 AM · network
kmacy updated the diff for D10556: Update cubic constants.

Update for comments

May 23 2017, 4:37 AM · network

May 1 2017

kmacy created D10556: Update cubic constants.
May 1 2017, 2:20 AM · network

Jan 5 2017

kmacy abandoned D9019: Transition extended linuxkpi to pg_populate.
Jan 5 2017, 12:27 AM

Jan 2 2017

kmacy added a comment to D9019: Transition extended linuxkpi to pg_populate.

FYI - this is a backtrace of where X hangs when running the patched driver:
#0 0x000000080250115a in ioctl () from /lib/libc.so.7
#1 0x00000008012763d5 in drmIoctl (fd=7, request=3223348297, arg=0x7fffffffd830) at xf86drm.c:183
#2 0x0000000807a5a1b4 in amdgpu_ioctl_wait_cs (context=0x80409c000, ip=0, ip_instance=0, ring=0, handle=1,

timeout_ns=18446744073709551615, flags=1, busy=0x7fffffffd8d7) at amdgpu_cs.c:403

#3 0x0000000807a5a0bf in amdgpu_cs_query_fence_status (fence=0x804170010, timeout_ns=18446744073709551615, flags=1,

expired=0x7fffffffd944) at amdgpu_cs.c:436

#4 0x0000000808d213bb in amdgpu_fence_wait (fence=0x804170000, timeout=18446744073709551615, absolute=true)

at amdgpu_cs.c:117

#5 0x0000000808d1ea3a in amdgpu_bo_wait (_buf=0x8041e2880, timeout=18446744073709551615, usage=RADEON_USAGE_WRITE)

at amdgpu_bo.c:117

#6 0x0000000808d1fa95 in amdgpu_bo_map (buf=0x8041e2880, rcs=0x0, usage=PIPE_TRANSFER_READ) at amdgpu_bo.c:264
#7 0x0000000808d4406d in r600_buffer_map_sync_with_rings (ctx=0x804171000, resource=0x804038b40, usage=1)

at r600_buffer_common.c:99

#8 0x0000000808d4c3c2 in r600_query_init_backend_mask (ctx=0x804171000) at r600_query.c:1611
#9 0x0000000808c1353c in si_create_context (screen=0x80409b800, priv=0x0, flags=0) at si_pipe.c:253
#10 0x0000000808c13042 in radeonsi_screen_create (ws=0x8040e5600) at si_pipe.c:848
#11 0x0000000808d26c23 in amdgpu_winsys_create (fd=8, screen_create=0x808c12c70 <radeonsi_screen_create>)

at amdgpu_winsys.c:590

#12 0x00000008084671ca in pipe_radeonsi_create_screen (fd=8)

at ../../../../src/gallium/auxiliary/target-helpers/drm_helper.h:151

#13 0x0000000808aa04f3 in pipe_loader_drm_create_screen (dev=0x80415f7c0) at pipe_loader_drm.c:347
#14 0x0000000808a9f618 in pipe_loader_create_screen (dev=0x80415f7c0) at pipe_loader.c:79
#15 0x00000008088ccb15 in dri2_init_screen (sPriv=0x8040385a0) at dri2.c:1885
#16 0x00000008088c7c79 in driCreateNewScreen2 (scrn=0, fd=6, extensions=0x80463aaa0 <gbm_dri_screen_extensions>,

driver_extensions=0x8091b8910 <galliumdrm_driver_extensions>, driver_configs=0x804042570, data=0x804042400)
at dri_util.c:145

#17 0x0000000804435b0f in dri_screen_create_dri2 (dri=0x804042400, driver_name=0x804029120 "radeonsi")

at backends/dri/gbm_dri.c:448

#18 0x0000000804435480 in dri_screen_create (dri=0x804042400) at backends/dri/gbm_dri.c:523
#19 0x00000008044342b2 in dri_device_create (fd=6) at backends/dri/gbm_dri.c:1069
#20 0x000000080443396e in _gbm_create_device (fd=6) at main/backend.c:100
#21 0x0000000804433baf in gbm_create_device (fd=6) at main/gbm.c:123
#22 0x0000000807840dbe in ?? () from /usr/local/lib/xorg/modules/drivers/amdgpu_drv.so
#23 0x000000000047ce3f in InitOutput ()
#24 0x000000000043a4d4 in ?? ()
#25 0x0000000000424e0f in _start ()

Jan 2 2017, 12:12 AM
kmacy retitled D9019: Transition extended linuxkpi to pg_populate from to Transition extended linuxkpi to pg_populate.
Jan 2 2017, 12:10 AM

Dec 23 2016

kmacy added a comment to D8860: Add some additional ACPI methods for DRM.

Why are you abandoning this?

Dec 23 2016, 8:23 AM

Dec 11 2016

kmacy abandoned D1675: enable more aggressive contiguous memory reclamation .

Fixed by PQ_LAUNDER

Dec 11 2016, 11:17 PM
kmacy abandoned D6620: pass fault type to vm_pager_get_pages.

Replaced by populate, which although it still insists on doing a great deal of accounting not needed for device mappings (COW cargo culting AFAICT) that complicates mapping the same device memory at different VAs - it is quite close to what is expected by graphics drivers. Thanks go to Kostik and Alan.

Dec 11 2016, 5:14 AM · multimedia

Nov 23 2016

kmacy updated the diff for D8556: Fix spurious retransmit recovery on low latency networks.

add missed check

Nov 23 2016, 9:56 PM · network

Nov 18 2016

kmacy updated the diff for D8556: Fix spurious retransmit recovery on low latency networks.

Add comment to tcp_timer.c as requested by hiren@

Nov 18 2016, 4:31 AM · network
kmacy updated the diff for D8556: Fix spurious retransmit recovery on low latency networks.

simplify badrxtwin assignment

Nov 18 2016, 3:22 AM · network
kmacy updated the diff for D8558: iflib fixes.

fix derp

Nov 18 2016, 12:58 AM
kmacy added a comment to D5213: convert ixgbe to iflib.

This is now dependent on D8558

Nov 18 2016, 12:33 AM
kmacy added a dependent revision for D8558: iflib fixes: D5213: convert ixgbe to iflib.
Nov 18 2016, 12:33 AM
kmacy added a dependency for D5213: convert ixgbe to iflib: D8558: iflib fixes.
Nov 18 2016, 12:33 AM
kmacy retitled D8558: iflib fixes from to iflib fixes.
Nov 18 2016, 12:31 AM
kmacy accepted D5213: convert ixgbe to iflib.
Nov 18 2016, 12:04 AM

Nov 17 2016

kmacy retitled D8556: Fix spurious retransmit recovery on low latency networks from to Fix spurious retransmit recovery on low latency networks.
Nov 17 2016, 11:40 PM · network
kmacy added a comment to D5213: convert ixgbe to iflib.

The following 1 line change fixes this issue:

Nov 17 2016, 10:37 PM

Oct 27 2016

kmacy added a comment to D5213: convert ixgbe to iflib.

@sbruno what is this gated on now?

Oct 27 2016, 1:18 AM
kmacy added a comment to D7986: Simplify cubic_ack_received.

With regards to:

The one exception is that - if it is possible to reach a state where min_rtt_ticks is zero but we are *not* in slowstart the ack will be processed whereas before it would have been ignored.

Oct 27 2016, 1:15 AM

Oct 19 2016

kmacy added a comment to D5213: convert ixgbe to iflib.

That makes sense, as the device id is the same as a standard X520-DA2 adapter.

Oct 19 2016, 9:26 PM
kmacy added a comment to D5213: convert ixgbe to iflib.

@jeffrey.e.pieper_intel.com are all your test systems multi-socket or just this one?

Oct 19 2016, 9:01 PM
kmacy added a comment to D5213: convert ixgbe to iflib.

@jeffrey.e.pieper_intel.com The panic you've hit means that the admin task was not assigned a taskqueue. In all likelihood we're hitting an initialization edge case on the system itself, not that particular adapter.

Oct 19 2016, 8:19 PM
kmacy added a comment to D5213: convert ixgbe to iflib.

@jeffrey.e.pieper_intel.com and please post dmesg for the system it crashes on.

Oct 19 2016, 8:18 PM
kmacy added a comment to D5213: convert ixgbe to iflib.

@jeffrey.e.pieper_intel.com Please run with INVARIANTS. No one else has the offending hardware.

Oct 19 2016, 8:16 PM
kmacy added a comment to D5213: convert ixgbe to iflib.

Please try running worth INVARIANTS on the offending hardware.

Oct 19 2016, 7:02 PM
kmacy added a comment to D5213: convert ixgbe to iflib.

Is this with the driver in kernel or as a module? And with INVARIANTS or
without?

Oct 19 2016, 5:01 PM
kmacy added a comment to D5213: convert ixgbe to iflib.

What svn rev are you using?

Oct 19 2016, 4:59 PM

Oct 13 2016

kmacy added a comment to D5213: convert ixgbe to iflib.

@sbruno A fine exemplar of the modern FreeBSD work flow. Thanks for the prompt response.

Oct 13 2016, 2:14 PM