Page MenuHomeFreeBSD

rlibby (Ryan Libby)
User

Projects

User Details

User Since
Nov 12 2015, 8:14 PM (228 w, 2 d)

Recent Activity

Sat, Mar 7

rlibby updated subscribers of D23997: malloc debugging: detect mismatched mtp in alloc/free with redzone.
Sat, Mar 7, 10:28 PM
rlibby updated the summary of D23997: malloc debugging: detect mismatched mtp in alloc/free with redzone.
Sat, Mar 7, 10:12 PM
rlibby created D23997: malloc debugging: detect mismatched mtp in alloc/free with redzone.
Sat, Mar 7, 10:10 PM
rlibby accepted D23980: Make uma_int.h friendlier to userspace..

Fixups LGTM.

Sat, Mar 7, 12:52 AM

Fri, Mar 6

rlibby accepted D23980: Make uma_int.h friendlier to userspace..

Looks good except SLAB_BITSETS and Jeff's comments.

Fri, Mar 6, 10:20 PM
rlibby accepted D23988: Move SMR pointer type definition and access macros to smr_types.h..
Fri, Mar 6, 8:02 PM

Thu, Mar 5

rlibby added a comment to D23967: emulators/virtualbox-ose: use contemporary GCC instead of old llvm.

USE_GCC= any is not sufficient, as GCC8 doesn't support the feature used by atomic.h.

Thu, Mar 5, 5:10 AM

Tue, Mar 3

rlibby added a comment to D23937: amd64 atomics: guard flag output operand use with feature test.

Okay, I am happiest to go with option #2 also. I'll leave this up for a little in case any one else has a comment.

Tue, Mar 3, 5:00 PM
rlibby created D23937: amd64 atomics: guard flag output operand use with feature test.
Tue, Mar 3, 6:58 AM

Sun, Mar 1

rlibby accepted D23902: vm: add debug to uma_zone_set_smr.

Looks good.

Sun, Mar 1, 3:10 AM

Sat, Feb 29

rlibby accepted D23895: Ensure that arm64 thread structures are allocated from the direct map..

Too bad. Are you planning to pursue this further or to move on?

Sat, Feb 29, 6:07 PM
rlibby added a comment to D23661: Implement a translation from FreeBSD atomic API to gcc intrinsics.
In D23661#521387, @mjg wrote:

I think it's a fair worry that mixing hand-rolled atomics with compiler intrinsics may lead to trouble, but it's not hard to remedy.
Note the 2 mentioned missing testandclear and readandclear can be implemented without significant difficulty (the latter is a oneliner, the former worst case can be a cmpset loop. I grepped only 2 users.)

Sat, Feb 29, 5:39 PM

Fri, Feb 28

rlibby closed D23870: amd64 pmap.c: minor codegen optimization in flag access.
Fri, Feb 28, 6:32 PM
rlibby committed rS358440: amd64 pmap.c: minor codegen optimization in flag access.
amd64 pmap.c: minor codegen optimization in flag access
Fri, Feb 28, 6:32 PM
rlibby committed rS358439: amd64 atomic.h: minor codegen optimization in flag access.
amd64 atomic.h: minor codegen optimization in flag access
Fri, Feb 28, 6:32 PM
rlibby closed D23869: amd64 atomic.h: minor codegen optimization in flag access.
Fri, Feb 28, 6:32 PM
rlibby updated subscribers of D23870: amd64 pmap.c: minor codegen optimization in flag access.
Fri, Feb 28, 4:57 PM
rlibby added a comment to D23869: amd64 atomic.h: minor codegen optimization in flag access.
In D23869#524905, @mjg wrote:

I only have a minor comment that the feature is somewhat new in clang, I don't know if we care [that is, if we should version check]. I also happened to have a complete rewrite here D23661, but I think this should go in regardless of what happens with that review.

Fri, Feb 28, 4:49 PM

Feb 28 2020

rlibby created D23870: amd64 pmap.c: minor codegen optimization in flag access.
Feb 28 2020, 5:55 AM
rlibby created D23869: amd64 atomic.h: minor codegen optimization in flag access.
Feb 28 2020, 5:50 AM

Feb 27 2020

rlibby accepted D23825: Simplify lazy advance and decouple it somewhat from ticks so that it canbe forwarded after other broadcast interrupts..

Looks good.

Feb 27 2020, 10:37 AM

Feb 25 2020

rlibby accepted D23824: A pair of UMA optimizations to improve cache behavior..

Looks good after fixing Mark's comments.

Feb 25 2020, 7:19 AM

Feb 24 2020

rlibby added a comment to D23802: vn_fullpath: const'ify retbuf.

I agree the interface is strange. It would seem simpler to me if it were just
int vn_fullpath(struct thread *td, struct vnode *vp, char **retbuf);

Feb 24 2020, 6:31 PM
rlibby added a comment to D23802: vn_fullpath: const'ify retbuf.

I think in terms of const semantics, this is okay, although it does seem accidental and undesirable to prevent callers from modifying the buf on success as a side-effect.

Feb 24 2020, 5:42 PM

Feb 23 2020

rlibby closed D23798: sys/kern: quiet -Wwrite-strings.
Feb 23 2020, 3:32 AM
rlibby committed rS358258: sys/kern: quiet -Wwrite-strings.
sys/kern: quiet -Wwrite-strings
Feb 23 2020, 3:32 AM
rlibby committed rS358257: vfs: quiet -Wwrite-strings.
vfs: quiet -Wwrite-strings
Feb 23 2020, 3:32 AM
rlibby closed D23796: sys/vm: quiet -Wwrite-strings.
Feb 23 2020, 3:32 AM
rlibby committed rS358256: sys/vm: quiet -Wwrite-strings.
sys/vm: quiet -Wwrite-strings
Feb 23 2020, 3:32 AM
rlibby closed D23797: vfs: quiet -Wwrite-strings.
Feb 23 2020, 3:32 AM

Feb 22 2020

rlibby created D23798: sys/kern: quiet -Wwrite-strings.
Feb 22 2020, 9:49 PM
rlibby created D23797: vfs: quiet -Wwrite-strings.
Feb 22 2020, 9:40 PM
rlibby created D23796: sys/vm: quiet -Wwrite-strings.
Feb 22 2020, 9:39 PM
rlibby added inline comments to D23533: Plumb a domain parameter through uma_reclaim() and callees..
Feb 22 2020, 4:57 PM
rlibby added inline comments to D23738: Experimental ticks based SMR..
Feb 22 2020, 6:46 AM

Feb 20 2020

rlibby added inline comments to D23738: Experimental ticks based SMR..
Feb 20 2020, 6:29 AM

Feb 19 2020

rlibby closed D23729: powerpc: unconditionally mark SLB zones UMA_ZONE_CONTIG.
Feb 19 2020, 4:47 AM
rlibby committed rS358094: powerpc: unconditionally mark SLB zones UMA_ZONE_CONTIG.
powerpc: unconditionally mark SLB zones UMA_ZONE_CONTIG
Feb 19 2020, 4:47 AM

Feb 17 2020

rlibby added a comment to D23729: powerpc: unconditionally mark SLB zones UMA_ZONE_CONTIG.

Can you explain why this makes a difference, at least for slbt_zone? I think I am missing something.

I can report observed behavior both ways:
Without removing the conditional structure the old PowerMac G5 "Quads"
do not boot. With the removal they do. The booting has been broken since
head -r357549 when the conditional allocf_flags code structure was put in.
See https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=244118 .
I'm now running with the patch from that bugzilla and have progressed to
-r357979 with it.

Feb 17 2020, 8:07 PM
rlibby added a comment to D23729: powerpc: unconditionally mark SLB zones UMA_ZONE_CONTIG.

Can you explain why this makes a difference, at least for slbt_zone? I think I am missing something.

Feb 17 2020, 7:53 PM
rlibby created D23729: powerpc: unconditionally mark SLB zones UMA_ZONE_CONTIG.
Feb 17 2020, 6:28 PM

Feb 7 2020

rlibby committed rS357644: smr.h: fix build after r357641.
smr.h: fix build after r357641
Feb 7 2020, 12:48 AM

Feb 6 2020

rlibby added a comment to D23464: Fix a race in smr_advance() that could result in unnecessary poll calls..
In D23464#516865, @jkim wrote:

It seems you forgot to define SMR_SEQ_DELTA() macro.

Feb 6 2020, 9:46 PM
rlibby accepted D23532: Optimize keg_drain()..

Fixups look good.

Feb 6 2020, 3:48 PM
rlibby committed rS357611: auditd_stop: wait_for_pids instead of sleeping.
auditd_stop: wait_for_pids instead of sleeping
Feb 6 2020, 8:32 AM
rlibby closed D23223: auditd_stop: wait_for_pids instead of sleeping.
Feb 6 2020, 8:32 AM
rlibby committed rS357610: uma: remove UMA_ZFLAG_CACHEONLY flag.
uma: remove UMA_ZFLAG_CACHEONLY flag
Feb 6 2020, 8:32 AM
rlibby closed D23516: uma: remove UMA_ZFLAG_CACHEONLY flag.
Feb 6 2020, 8:32 AM
rlibby accepted D23532: Optimize keg_drain()..
Feb 6 2020, 6:48 AM

Feb 5 2020

rlibby added inline comments to D22963: Expand generic subword atomic primitives.
Feb 5 2020, 7:24 PM
rlibby added inline comments to D22963: Expand generic subword atomic primitives.
Feb 5 2020, 5:32 PM
rlibby created D23516: uma: remove UMA_ZFLAG_CACHEONLY flag.
Feb 5 2020, 9:08 AM

Feb 4 2020

rlibby closed D23487: uma: multipage chicken switch.
Feb 4 2020, 10:40 PM
rlibby committed rS357550: uma: multipage chicken switch.
uma: multipage chicken switch
Feb 4 2020, 10:40 PM
rlibby committed rS357549: uma: grow slabs to enforce minimum memory efficiency.
uma: grow slabs to enforce minimum memory efficiency
Feb 4 2020, 10:40 PM
rlibby committed rS357548: uma: convert mbuf_jumbo_alloc to UMA_ZONE_CONTIG & tag others.
uma: convert mbuf_jumbo_alloc to UMA_ZONE_CONTIG & tag others
Feb 4 2020, 10:40 PM
rlibby closed D23239: [uma-multipage 3/3] uma: grow slabs to enforce minimum memory efficiency.
Feb 4 2020, 10:40 PM
rlibby closed D23238: [uma-multipage 2/3] uma: convert mbuf_jumbo_alloc & pmap_pdpt_alloc to UMA_ZONE_CONTIG.
Feb 4 2020, 10:40 PM
rlibby closed D23237: [uma-multipage 1/3] uma: add UMA_ZONE_CONTIG, and a default contig_alloc.
Feb 4 2020, 10:40 PM
rlibby committed rS357547: uma: add UMA_ZONE_CONTIG, and a default contig_alloc.
uma: add UMA_ZONE_CONTIG, and a default contig_alloc
Feb 4 2020, 10:40 PM
rlibby closed D23496: uma: pcpu_page_free needs to startup_free pages from startup_alloc.
Feb 4 2020, 10:40 PM
rlibby committed rS357546: uma: pcpu_page_free needs to startup_free pages from startup_alloc.
uma: pcpu_page_free needs to startup_free pages from startup_alloc
Feb 4 2020, 10:40 PM
rlibby added a comment to D23223: auditd_stop: wait_for_pids instead of sleeping.
In D23223#515942, @cem wrote:

I don't see any reason auditd cannot just use the default stop action, which sends SIGTERM. auditd handles sigterm and performs more or less the same shutdown actions as those triggered by audit -t (AUDIT_TRIGGER_CLOSE_AND_DIE). I would suggest just removing the auditd_stop() function entirely.

Feb 4 2020, 10:29 PM
rlibby added a comment to D23223: auditd_stop: wait_for_pids instead of sleeping.
In D23223#515931, @cem wrote:

Seems like there is a possible race condition here where $rc_pid exits and is recycled before we check for exit.

Feb 4 2020, 10:12 PM
rlibby updated the diff for D23487: uma: multipage chicken switch.

Rename from vm.multipage_slabs to vm.debug.uma_multipage_slabs

Feb 4 2020, 7:55 PM
rlibby updated the diff for D23239: [uma-multipage 3/3] uma: grow slabs to enforce minimum memory efficiency.

Fix CTR/KASSERT after dropping some local variables

Feb 4 2020, 7:52 PM
rlibby added a comment to D23238: [uma-multipage 2/3] uma: convert mbuf_jumbo_alloc & pmap_pdpt_alloc to UMA_ZONE_CONTIG.

Fix CTR/KASSERT after dropping some local variables

Feb 4 2020, 7:50 PM
rlibby updated the diff for D23238: [uma-multipage 2/3] uma: convert mbuf_jumbo_alloc & pmap_pdpt_alloc to UMA_ZONE_CONTIG.

Fix CTR/KASSERT after dropping some local variables

Feb 4 2020, 7:49 PM
rlibby updated the diff for D23237: [uma-multipage 1/3] uma: add UMA_ZONE_CONTIG, and a default contig_alloc.

Drop early-boot PCPU assert. It works now.

Feb 4 2020, 7:47 PM
rlibby added inline comments to D23487: uma: multipage chicken switch.
Feb 4 2020, 8:26 AM
rlibby updated subscribers of D23487: uma: multipage chicken switch.
Feb 4 2020, 8:25 AM
rlibby created D23496: uma: pcpu_page_free needs to startup_free pages from startup_alloc.
Feb 4 2020, 12:58 AM

Feb 3 2020

rlibby added inline comments to D23487: uma: multipage chicken switch.
Feb 3 2020, 10:32 PM
rlibby added inline comments to D23487: uma: multipage chicken switch.
Feb 3 2020, 6:37 PM
rlibby added inline comments to D23487: uma: multipage chicken switch.
Feb 3 2020, 6:22 PM
rlibby created D23487: uma: multipage chicken switch.
Feb 3 2020, 5:59 PM
rlibby updated the diff for D23237: [uma-multipage 1/3] uma: add UMA_ZONE_CONTIG, and a default contig_alloc.

Rebase over r357392.

Feb 3 2020, 5:49 PM
rlibby added inline comments to D23462: Deferred write seq advancement.
Feb 3 2020, 2:03 AM

Feb 2 2020

rlibby accepted D23464: Fix a race in smr_advance() that could result in unnecessary poll calls..

Looks good.

Feb 2 2020, 8:37 PM
rlibby accepted D23463: Add counters to smr..

Logic looks fine. I trust you've checked any performance effects.

Feb 2 2020, 8:23 PM

Jan 30 2020

rlibby added a comment to D22586: Implement safe memory reclamation in UMA..

Just dropping some nitpick comments before heading to bed. I looked through the interdiff for the latest update, but I was still intending to take another pass through the rest of the uma_core.c changes.

Jan 30 2020, 10:05 AM

Jan 27 2020

rlibby added inline comments to D22586: Implement safe memory reclamation in UMA..
Jan 27 2020, 6:08 PM
rlibby added inline comments to D22586: Implement safe memory reclamation in UMA..
Jan 27 2020, 1:24 AM
rlibby added inline comments to D22586: Implement safe memory reclamation in UMA..
Jan 27 2020, 1:04 AM

Jan 25 2020

rlibby accepted D22963: Expand generic subword atomic primitives.

_atomic_subword.h LGTM.

Jan 25 2020, 6:38 PM

Jan 24 2020

rlibby updated the diff for D23239: [uma-multipage 3/3] uma: grow slabs to enforce minimum memory efficiency.

Try to clean up size selection loop. Does this seem more clear, or
still obscure?

Jan 24 2020, 8:25 AM

Jan 23 2020

rlibby accepted D23330: Fix r356919..

Looks good. Thanks. I applied this patch and rebooted about ten times and saw no issues (previous frequency for me was maybe 1/3).

Jan 23 2020, 4:56 PM
rlibby closed D23318: uma: fix zone domain overlaying pcpu cache with disabled cpus.
Jan 23 2020, 4:56 AM
rlibby committed rS357019: uma: fix zone domain overlaying pcpu cache with disabled cpus.
uma: fix zone domain overlaying pcpu cache with disabled cpus
Jan 23 2020, 4:56 AM
rlibby closed D23275: uma: report leaks more accurately.
Jan 23 2020, 4:56 AM
rlibby committed rS357018: uma: report leaks more accurately.
uma: report leaks more accurately
Jan 23 2020, 4:56 AM

Jan 22 2020

rlibby added a comment to D23318: uma: fix zone domain overlaying pcpu cache with disabled cpus.
In D23318#511196, @jeff wrote:

mea culpa. Can you commit to 12.x as well?

Jan 22 2020, 6:40 PM
rlibby created D23318: uma: fix zone domain overlaying pcpu cache with disabled cpus.
Jan 22 2020, 6:38 PM

Jan 20 2020

rlibby created D23275: uma: report leaks more accurately.
Jan 20 2020, 3:21 AM

Jan 18 2020

rlibby updated the diff for D23238: [uma-multipage 2/3] uma: convert mbuf_jumbo_alloc & pmap_pdpt_alloc to UMA_ZONE_CONTIG.

review feedback: leave pmap_pdpt_allocf alone, just tag the zone.

Jan 18 2020, 8:25 PM
rlibby updated the diff for D23237: [uma-multipage 1/3] uma: add UMA_ZONE_CONTIG, and a default contig_alloc.

Review feedback: man page and comment wording. Also s/0x%#b/0x%b/.

Jan 18 2020, 6:46 PM
rlibby added inline comments to D23237: [uma-multipage 1/3] uma: add UMA_ZONE_CONTIG, and a default contig_alloc.
Jan 18 2020, 5:57 PM
rlibby added inline comments to D23237: [uma-multipage 1/3] uma: add UMA_ZONE_CONTIG, and a default contig_alloc.
Jan 18 2020, 5:05 PM

Jan 17 2020

rlibby added inline comments to D23238: [uma-multipage 2/3] uma: convert mbuf_jumbo_alloc & pmap_pdpt_alloc to UMA_ZONE_CONTIG.
Jan 17 2020, 8:54 PM
rlibby added a comment to D23238: [uma-multipage 2/3] uma: convert mbuf_jumbo_alloc & pmap_pdpt_alloc to UMA_ZONE_CONTIG.

Yes, that's right. In fact, the smaller mbufs are actually still okay with multipage slabs due to their power-of-2 size, but I thought it was better to make the constraint explicit.

Jan 17 2020, 8:08 PM