Page MenuHomeFreeBSD

cem (Conrad Meyer)
User

Projects

User Details

User Since
Jul 9 2015, 9:56 PM (310 w, 4 d)

Recent Activity

May 11 2021

cem added a comment to D30170: sort: Cache value of MB_CUR_MAX.

Is there a way we could make MB_CUR_MAX or __mb_cur_max suck less, in a more general way? Like, isn't it basically always four or one?

May 11 2021, 12:45 AM

May 1 2021

cem added a comment to D26743: Factor out code from fstyp into libfstyp.

I think the idea is very reasonable and could be done in a way that preserves the safety of the sandbox, at least for fstyp. I don't think the popen approach makes sense. There may be a lot of work required to sandbox the suggested consumer (mount).

May 1 2021, 7:03 PM

Apr 12 2021

cem added inline comments to D29736: OCF: Remove support for asymmetric cryptographic operations..
Apr 12 2021, 9:12 PM
cem accepted D29736: OCF: Remove support for asymmetric cryptographic operations..

Nice cleanup. I didn't review the man page language updates super thoroughly, but the parts I skimmed looked good.

Apr 12 2021, 9:11 PM

Apr 1 2021

cem accepted D29518: cryptocheck: Expand the set of sizes tested by -z..

Merci

Apr 1 2021, 10:16 PM

Mar 31 2021

cem edited P491 'pciconf -lc pci0:0:0:2' ryzen 5800x.
Mar 31 2021, 10:43 PM
cem added inline comments to D29518: cryptocheck: Expand the set of sizes tested by -z..
Mar 31 2021, 10:30 PM
cem accepted D29517: ossl: Don't encryt/decrypt too much data for chacha20..
Mar 31 2021, 6:09 PM
cem added inline comments to D29518: cryptocheck: Expand the set of sizes tested by -z..
Mar 31 2021, 6:07 PM

Mar 23 2021

cem accepted D29385: Make random() behaves as documented: "result is uniform in [0, 2^31 - 1]".

Thanks!

Mar 23 2021, 5:05 PM
cem accepted D29385: Make random() behaves as documented: "result is uniform in [0, 2^31 - 1]".

Nice find. I'd encourage using prng32_bounded instead, but I don't think this is mechanically wrong.

Mar 23 2021, 5:52 AM

Mar 16 2021

cem accepted D29173: gdb: allow setting/removing hardware watchpoints.

Awesome, thanks!

Mar 16 2021, 4:25 PM

Mar 9 2021

cem added a comment to D29173: gdb: allow setting/removing hardware watchpoints.

Seems reasonable to me!

Mar 9 2021, 9:31 PM
cem added a comment to D29174: gdb: report specific stop reason for watchpoints.

Generally seems good to me.

Mar 9 2021, 9:25 PM

Mar 3 2021

cem resigned from D29031: fusefs: fix two bugs regarding fcntl file locks.

Nah

Mar 3 2021, 5:38 AM
cem resigned from D27946: fusefs: protect fufh table and cached attributes with the vnode lock.
Mar 3 2021, 5:37 AM
cem requested changes to D27946: fusefs: protect fufh table and cached attributes with the vnode lock.
Mar 3 2021, 5:36 AM
cem added a comment to D29007: address a performance problem w/ partial sscanf on long strings....
In D29007#649973, @jmg wrote:
In D29007#649700, @cem wrote:

Is 128 a reasonable chunk here? How does it relate to the internal buffering of FILE?

I'm going to run some perf numbers on ARM, but w/ the attached file w/ benchmarks, it looks like the chunk should be closer to 16.

Mar 3 2021, 5:17 AM
cem accepted D28754: ossl: Add Poly1305 digest support..
Mar 3 2021, 5:12 AM
cem accepted D28755: The ChaCha20 counter is little endian, not big endian..

Certainly not a regression :-)

Mar 3 2021, 5:04 AM

Mar 2 2021

cem added a comment to D29007: address a performance problem w/ partial sscanf on long strings....

Is 128 a reasonable chunk here? How does it relate to the internal buffering of FILE?

Mar 2 2021, 6:28 PM

Mar 1 2021

cem added a comment to D27946: fusefs: protect fufh table and cached attributes with the vnode lock.

Two small issues remaining, but otherwise I think it's in good shape

Mar 1 2021, 3:12 AM

Feb 28 2021

cem added a comment to D12330: "Long" filename support changes.

@cem Hi there! Is there a link to the v2 ?

Feb 28 2021, 7:20 PM

Feb 23 2021

cem accepted D28889: Fix ptrace_test:ptrace__syscall_args after ATF upgrade.
Feb 23 2021, 4:33 PM

Feb 18 2021

cem accepted D28700: nvdimm(4): Export NVDIMM health flags via sysctl.

Still accepted from earlier

Feb 18 2021, 10:12 PM
cem added a comment to D28756: ossl: Add ChaCha20 cipher support..

Outside of ossl_chacha20.c, everything looks fine.

Feb 18 2021, 2:23 AM
cem added a comment to D28755: The ChaCha20 counter is little endian, not big endian..

Hm, it is little endian, but I'm not confident about the two sentences prior.

Feb 18 2021, 2:17 AM
cem added inline comments to D28754: ossl: Add Poly1305 digest support..
Feb 18 2021, 2:14 AM
cem accepted D28752: cryptosoft: Support per-op keys for AES-GCM and AES-CCM..
Feb 18 2021, 2:00 AM
cem accepted D27838: cryptocheck: Add Chacha20-Poly1305 AEAD coverage..
Feb 18 2021, 1:58 AM
cem accepted D28700: nvdimm(4): Export NVDIMM health flags via sysctl.
Feb 18 2021, 1:57 AM

Feb 17 2021

cem accepted D28753: cryptocheck: Free generated IV after each GMAC test..
Feb 17 2021, 11:59 PM
cem added inline comments to D28700: nvdimm(4): Export NVDIMM health flags via sysctl.
Feb 17 2021, 4:19 PM
cem accepted D28684: libc: Fix t_spawn_fileactions test after ATF update.
Feb 17 2021, 4:03 PM

Feb 16 2021

cem edited P483 Masterwork From Distant Lands.
Feb 16 2021, 1:52 AM

Feb 15 2021

cem accepted D28690: Export various 128 bit long double functions from libgcc_s.so.1.
Feb 15 2021, 10:14 PM
cem accepted D28683: Fix two failing tests after ATF update.

The analysis makes sense and the fix looks fine to me. You could allocate two fds and check socketpair against both (instead of just low) in the final test, but I don't think there's much marginal value in that. (Frankly, I'm not a fan of unix' must-assign-lowest-fd behavior in general, but we have to abide by it.)

Feb 15 2021, 6:54 PM

Feb 13 2021

cem accepted D28605: fusefs: set d_off during VOP_READDIR.
Feb 13 2021, 4:44 AM
cem added inline comments to D28605: fusefs: set d_off during VOP_READDIR.
Feb 13 2021, 3:57 AM
cem added inline comments to D28605: fusefs: set d_off during VOP_READDIR.
Feb 13 2021, 1:46 AM

Feb 12 2021

cem added a comment to D28605: fusefs: set d_off during VOP_READDIR.

In the commit summary, is this a typo?

Feb 12 2021, 5:18 PM
cem added inline comments to D28607: fusefs: forward the server's reported optimal transfer size to statfs.
Feb 12 2021, 4:49 PM

Feb 8 2021

cem added inline comments to D27946: fusefs: protect fufh table and cached attributes with the vnode lock.
Feb 8 2021, 4:44 AM
cem added inline comments to D27946: fusefs: protect fufh table and cached attributes with the vnode lock.
Feb 8 2021, 2:12 AM
cem added a comment to D28528: cherry pick openpam update.

No objection in principle, but the diff is basically unreviewable as-is for me; it's like 90% autoconf generated shell.

Feb 8 2021, 1:48 AM

Feb 2 2021

cem accepted D28461: contribreadelf: do not trucate section name with -W.

LGTM, thanks.

Feb 2 2021, 3:11 PM

Feb 1 2021

cem accepted D28395: tests/sys/kern/crc32: Check for SSE4.2 before using it.

Looks great to me!

Feb 1 2021, 3:56 PM
cem added a comment to D28395: tests/sys/kern/crc32: Check for SSE4.2 before using it.

Turns out the bug was that I messed up the size being passed to the SSE4.2 version. It does work correctly.

Feb 1 2021, 3:48 PM
cem added inline comments to D28395: tests/sys/kern/crc32: Check for SSE4.2 before using it.
Feb 1 2021, 3:22 PM
cem added inline comments to D28395: tests/sys/kern/crc32: Check for SSE4.2 before using it.
Feb 1 2021, 3:14 PM

Jan 29 2021

cem added inline comments to D28395: tests/sys/kern/crc32: Check for SSE4.2 before using it.
Jan 29 2021, 4:43 PM

Jan 28 2021

cem added a comment to D28375: Fix getblk() with GB_NOCREAT returning false-negatives..
In D28375#634909, @cem wrote:

incore has always been able to produce false negatives, FWIW. It's fundamentally racy. I think it's mostly used as a heuristic for avoiding unnecessary prefetch.

That's true, but it's possible for callers to depend on the instantaneous result being correct because some external synchronization guarantees that the result is stable. For instance, some lock might serialize calls to getblk() for a particular vnode, so holding that lock ensures that incore() returns a stable result. I'm worried about false negatives caused by reassignbuf() in cases like this.

Jan 28 2021, 7:18 PM
cem requested changes to D28395: tests/sys/kern/crc32: Check for SSE4.2 before using it.
Jan 28 2021, 4:29 PM
cem accepted D28395: tests/sys/kern/crc32: Check for SSE4.2 before using it.

I think QEMU provides a basic amd64 CPU by default which does not include any newer features (wikipedia suggests SSE4.2 was introduced ~2008).

Jan 28 2021, 2:31 PM
cem requested changes to D28395: tests/sys/kern/crc32: Check for SSE4.2 before using it.

I think the idea here is that CRC32 is part of SSE4.2, which is part of the base feature set of amd64. If you're hitting SIGILL in QEMU in amd64 mode, that suggests QEMU's amd64 emulation is somewhat invalid. However, it's certainly optional on at least some early models of i386.

Jan 28 2021, 1:51 PM
cem added a comment to D28375: Fix getblk() with GB_NOCREAT returning false-negatives..

Hmm. It's not really obvious to me that all callers handle false negatives correctly. Some callers simply call getblk() in that case, but others like ufs_bmaparray() are more complicated.

Jan 28 2021, 12:15 AM

Jan 27 2021

cem accepted D28375: Fix getblk() with GB_NOCREAT returning false-negatives..

I think incore was already always racy? The race scenarios may have changed slightly.

Jan 27 2021, 8:34 PM

Jan 26 2021

cem accepted D27542: stand: lua: enhance lfs.dir() to speed up kernels_autodetect.

Excellent. Thanks!

Jan 26 2021, 3:08 PM

Jan 23 2021

cem added inline comments to D25782: Add unlocked/SMR fast path to getblk().
Jan 23 2021, 5:25 AM
cem added inline comments to D25782: Add unlocked/SMR fast path to getblk().
Jan 23 2021, 12:16 AM
cem added inline comments to D25782: Add unlocked/SMR fast path to getblk().
Jan 23 2021, 12:14 AM

Jan 22 2021

cem added a comment to D28260: Define PNP info after defining driver modules.

I think maybe we should revert the change, if it is truly unfounded and was only observed on 12. If this wasn't ever a problem on CURRENT, the commit message rationale is wrong and misleading. Reverting is the canonical way to correct that kind of thing.

Jan 22 2021, 9:32 PM
cem added a comment to D28260: Define PNP info after defining driver modules.

Hmm, the kldxref change doesn't fully fix the problem as I understand it (not that my commit does). The real problem is that the pnp info structure doesn't contain the module name, so devmatch(8) doesn't know which PNP info goes with which module. The kldxref change at least ensures that devmatch won't try to process a PNP info definition without having seen an MDT_MODULE record.

Jan 22 2021, 1:56 PM
cem added a comment to D28260: Define PNP info after defining driver modules.

Wait, kldxref should be ordering these correctly since rS348309. Why isn't it?

Jan 22 2021, 1:41 AM

Jan 17 2021

cem committed R10:ddf61156132b: fstyp(8): fix exfat detection (authored by cem).
fstyp(8): fix exfat detection
Jan 17 2021, 7:58 PM

Jan 14 2021

cem accepted D28159: opencrypto: Fix assignment of crypto completions to worker threads.

πŸ‘

Jan 14 2021, 5:37 PM
cem accepted D28158: opencrypto: Embed the driver softc in the session structure.

Sure, seems fine to me.

Jan 14 2021, 5:08 PM

Jan 9 2021

cem added inline comments to D27838: cryptocheck: Add Chacha20-Poly1305 AEAD coverage..
Jan 9 2021, 1:47 AM

Jan 8 2021

cem accepted D28047: gitignore: update cscope rules.

I don’t know anything about cscope but it looks fine. Ideally files that would only be found in the root (eg tinderbox crap) are β€œ/β€œ prefixed to reduce the search cost in subdirectories. Orthogonal to this change.

Jan 8 2021, 11:11 PM

Jan 4 2021

cem added inline comments to D24933: implement W^X for mmap and mprotect.
Jan 4 2021, 7:14 PM
cem added inline comments to D27946: fusefs: protect fufh table and cached attributes with the vnode lock.
Jan 4 2021, 2:51 PM
cem added inline comments to D27947: Fix LK_TRYUPGRADE.
Jan 4 2021, 2:37 PM
cem added inline comments to D27947: Fix LK_TRYUPGRADE.
Jan 4 2021, 2:36 PM
cem added inline comments to D27946: fusefs: protect fufh table and cached attributes with the vnode lock.
Jan 4 2021, 1:03 AM

Jan 3 2021

cem committed R10:98689d0ffb0a: libprivatezstd: Restrict symbol visibility similar to upstream (authored by cem).
libprivatezstd: Restrict symbol visibility similar to upstream
Jan 3 2021, 7:27 PM
cem accepted D27944: bitset.9: add missing MLINKS.
Jan 3 2021, 6:54 PM
cem added a comment to D27943: ccp: add new PCI ID for Zen 2/3.

Well, wouldn't mixing an extra source be better?

Jan 3 2021, 6:30 PM
cem removed reviewers for D27943: ccp: add new PCI ID for Zen 2/3: cem, Contributor Reviews (base).

See d7b665dfd7a6 / rS368542 / 7cda7375e629 / rS366943.

Jan 3 2021, 5:19 PM
cem accepted D27938: Fix vnode locking bug in fuse_vnop_copy_file_range.
Jan 3 2021, 4:50 PM
cem committed R10:bcae12b591d9: zstd: Fix non-FreeBSD CI libzstd build (authored by cem).
zstd: Fix non-FreeBSD CI libzstd build
Jan 3 2021, 3:43 PM
cem edited P471 git log -1 --pretty=raw.
Jan 3 2021, 1:16 PM
cem accepted D27936: Fix a vnode locking bug in fuse_vnop_advlock..

Ok, so the bug was that we never took any lock at all? But we're also moving a block of code surrounding the new vn_lock(); operation, which isn't related to our not-locking-the-vp-bug. It looks like we want to reject EINVAL flags before we start locking, which is fine. I would just like to see a better commit message than "fix a [non-specific] bug." Something like "add missing lock in fuse advlock. also, reject invalid parameters sooner (style)."

Jan 3 2021, 12:58 PM
cem accepted D27939: fusefs: only check vnode locks with DEBUG_VFS_LOCKS.

The other nice thing about this routine (ok, conditional on debug.vfs_badlock_vnode=1, but it is the default) β€” or VNASSERT(), VPASS() β€” is that they'll print out details of the vp. MPASS can't do that.

Jan 3 2021, 12:48 PM
cem added inline comments to D27938: Fix vnode locking bug in fuse_vnop_copy_file_range.
Jan 3 2021, 12:43 PM

Dec 30 2020

cem accepted D27818: fusefs: implement FUSE_COPY_FILE_RANGE..
Dec 30 2020, 4:23 PM
cem accepted D27804: fusefs: update FUSE protocol to 7.24 and implement FUSE_LSEEK.

Thanks!

Dec 30 2020, 2:54 PM
cem added a comment to D27837: Add an implementation of CHACHA20_POLY1035 to cryptosoft..

The make-bits and other integration look fine. I didn't have time to verify the meat of the implementation (and probably won't).

Dec 30 2020, 2:34 AM
cem accepted D27836: Add an OCF algorithm for ChaCha20-Poly1035 AEAD..
Dec 30 2020, 2:26 AM
cem accepted D27835: Tidy some crypto-related lines in sys/conf/files..
Dec 30 2020, 2:17 AM
cem accepted D27834: Reject AES-GCM and AES-CCM sessions with per-op keys..
Dec 30 2020, 2:16 AM
cem added a comment to D27804: fusefs: update FUSE protocol to 7.24 and implement FUSE_LSEEK.

Mostly LGTM, thanks!

Dec 30 2020, 1:58 AM
cem added inline comments to D27804: fusefs: update FUSE protocol to 7.24 and implement FUSE_LSEEK.
Dec 30 2020, 1:58 AM

Dec 27 2020

cem removed a reviewer for D27787: tcgetwinsize(3): man page: cem.
Dec 27 2020, 9:55 PM

Dec 25 2020

cem committed R10:f7cd7fe51c41: sys/contrib/zstd: Import zstd 1.4.8 (authored by cem).
sys/contrib/zstd: Import zstd 1.4.8
Dec 25 2020, 3:57 PM
cem committed R10:f6ae97673c28: Import zstd 1.4.8 (authored by cem).
Import zstd 1.4.8
Dec 25 2020, 3:32 PM

Dec 24 2020

cem accepted D27770: fusefs: delete some dead code.

Awesome, thanks!

Dec 24 2020, 10:03 PM
cem accepted D27770: fusefs: delete some dead code.
Dec 24 2020, 9:15 PM

Dec 22 2020

cem accepted D27644: gdb(4): allow bulk write and single read of registers.

Thanks!

Dec 22 2020, 8:37 PM

Dec 17 2020

cem accepted D27642: amd64: allow gdb(4) to write to most registers.
Dec 17 2020, 9:01 PM
cem added a comment to D27633: Initial version of git commit message preparation hook.

Re: hyphenated metadata attribute names β€” I'd just keep our annotations the same as they are now. If we were starting from scratch as a git project, sure, we should adopt the common git scheme. But the annotations are not special in git (just part of the commit message), and any FreeBSD tooling has to understand our previous annotations anyway.

Dec 17 2020, 12:08 AM