Page MenuHomeFreeBSD

jrtc27 (Jessica Clarke)
User

Projects

User Details

User Since
Jul 4 2018, 7:23 PM (297 w, 5 d)

Recent Activity

Yesterday

jrtc27 accepted D44419: kldxref: Properly handle reading strings near the end of an ELF file.
Mon, Mar 18, 11:45 PM

Sat, Mar 16

jrtc27 committed rG75464941dc17: kldxref: Fix bootstrapping on macOS with Clang 16 / Apple Clang 15 (authored by jrtc27).
kldxref: Fix bootstrapping on macOS with Clang 16 / Apple Clang 15
Sat, Mar 16, 1:51 AM
jrtc27 committed rGd8c84215d767: jevents: Fix bootstrapping on macOS with Clang 16 / Apple Clang 15 (authored by jrtc27).
jevents: Fix bootstrapping on macOS with Clang 16 / Apple Clang 15
Sat, Mar 16, 1:51 AM
jrtc27 committed rGce312ef72d91: mandoc: Fix bootstrapping on macOS with Clang 16 / Apple Clang 15 (authored by jrtc27).
mandoc: Fix bootstrapping on macOS with Clang 16 / Apple Clang 15
Sat, Mar 16, 1:51 AM

Wed, Mar 13

jrtc27 added a comment to D44317: Changed order of arm64 events such that TSLOG-enabled kernels are able to boot.

(The problem this fixes was introduced by me adding profiling to the linker in link_elf_ireloc.)

Wed, Mar 13, 7:39 PM

Tue, Mar 12

jrtc27 added inline comments to D44334: riscv: remove more riscv64sf support.
Tue, Mar 12, 11:23 PM
jrtc27 added a comment to D44333: msun/riscv: expose fe{disable,enable}except.

By ABI break you mean for riscv64sf getting the old symbol versions? That architecture is dead, we should just delete the dead code in the header.

Tue, Mar 12, 10:51 PM
jrtc27 added a comment to D44331: msun/riscv: limit exposure of fegetexcept.

Oh I had it as an in-progress patch to make these *not* inline-only. I don't think it's a bug that they're only inline, but every other architecture exposes them as actual symbols too and GCC at least relies on that. See https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=272759 (comment 5 in particular) and https://cgit.freebsd.org/src/commit/?id=448c505c33cc334193590f3844406d6a74f26e2a.

Ah, I'll grab that and apply it.

Tue, Mar 12, 10:36 PM
jrtc27 added a comment to D44331: msun/riscv: limit exposure of fegetexcept.

Oh I had it as an in-progress patch to make these *not* inline-only. I don't think it's a bug that they're only inline, but every other architecture exposes them as actual symbols too and GCC at least relies on that. See https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=272759 (comment 5 in particular) and https://cgit.freebsd.org/src/commit/?id=448c505c33cc334193590f3844406d6a74f26e2a.

Tue, Mar 12, 9:53 PM

Thu, Mar 7

jrtc27 added a comment to D44270: starfive: add a syscon driver.

I considered extending the existing riscv_syscon for this purpose, but
it seems better to keep the StarFive/JH7110-specific drivers properly
separated.

Thu, Mar 7, 7:53 PM
jrtc27 added inline comments to D44026: JH7110 MMC driver.
Thu, Mar 7, 7:26 PM
jrtc27 added inline comments to D44264: newbus: Create %probe_bias in every devclass.
Thu, Mar 7, 7:15 PM
jrtc27 added inline comments to D44264: newbus: Create %probe_bias in every devclass.
Thu, Mar 7, 7:08 PM

Wed, Mar 6

jrtc27 added a comment to D44207: pci_host_generic: Use SYS_RES_MEMORY for the parent of I/O resource ranges.
In D44207#1009367, @jhb wrote:

ofw_pcib doesn't pass child resources up to simplebus for allocation or activation, it uses bus_space_map directly with a new-bus method to get the bus_space_tag. I think this might be the only one passing SYS_RES_IOPORT up to simplebus and that if this works we might be able to remove all SYS_RES_IOPORT support from simplebus.

Wed, Mar 6, 4:46 PM

Tue, Mar 5

jrtc27 added a comment to D44231: libsys/aarch64: end syscall stubs with newlines.

s/POSIX files/POSIX text files/

Tue, Mar 5, 11:07 PM
jrtc27 added a comment to D44207: pci_host_generic: Use SYS_RES_MEMORY for the parent of I/O resource ranges.

What about all the other PCI controller drivers though? Various use ofw_pcib, for example.

Tue, Mar 5, 5:13 AM

Tue, Feb 27

jrtc27 added inline comments to D44132: pci_host_generic: Enumerate resource ranges under bootverbose.
Tue, Feb 27, 11:21 PM
jrtc27 added inline comments to D44103: conf: deduplicate dwc config logic.
Tue, Feb 27, 6:11 PM
jrtc27 added inline comments to D44104: conf: deduplicate dwmmc config logic.
Tue, Feb 27, 6:04 PM
jrtc27 added inline comments to D44103: conf: deduplicate dwc config logic.
Tue, Feb 27, 6:03 PM
jrtc27 accepted D44105: riscv: add starfive kernelconf template.
Tue, Feb 27, 6:02 PM

Sat, Feb 24

jrtc27 accepted D43918: netlink: Don't use a zero-length array.
Sat, Feb 24, 7:15 PM

Fri, Feb 23

jrtc27 added inline comments to D44042: libsys: fix sleep(3)/usleep(3) cancel behavior.
Fri, Feb 23, 6:06 PM
jrtc27 committed rG09cb8031b43c: efibootmgr: Simplify make_next_boot_var_name and fix cnt == 0 case (authored by jrtc27).
efibootmgr: Simplify make_next_boot_var_name and fix cnt == 0 case
Fri, Feb 23, 2:37 AM
jrtc27 closed D44029: efibootmgr: Simplify make_next_boot_var_name and fix cnt == 0 case.
Fri, Feb 23, 2:37 AM

Thu, Feb 22

jrtc27 requested review of D44029: efibootmgr: Simplify make_next_boot_var_name and fix cnt == 0 case.
Thu, Feb 22, 11:49 PM
jrtc27 added a comment to D44027: libsys: link symbolic.

To confirm I understand this:

Thu, Feb 22, 10:00 PM
jrtc27 added a comment to D43037: jh7110_clkgen: Add StarFive JH7110 clock generator driver.

Initial review comments. Not exhaustive, hard to be for large diffs like this where there are quite a few comments to make, but hopefully I didn't miss anything too fundamental.

Thu, Feb 22, 9:53 PM
jrtc27 added a reverting change for rG5fead429ebb3: Avoid referencing private lib names directly.: rGff6b71c2b214: rescue: Stop using LDADD_zstd.
Thu, Feb 22, 8:57 PM
jrtc27 committed rGff6b71c2b214: rescue: Stop using LDADD_zstd (authored by jrtc27).
rescue: Stop using LDADD_zstd
Thu, Feb 22, 8:56 PM
jrtc27 committed rG121be555997b: arm: Set NEW_PCIB in DEFAULTS rather than a subset of kernel configs (authored by jrtc27).
arm: Set NEW_PCIB in DEFAULTS rather than a subset of kernel configs
Thu, Feb 22, 6:29 PM
jrtc27 closed D43806: arm: Set NEW_PCIB in DEFAULTS rather than a subset of kernel configs.
Thu, Feb 22, 6:29 PM

Feb 16 2024

jrtc27 added a comment to D43922: pcib: Assume a window where both the base and limit are 0 is uninitialized.

Ok, I went and found a copy of the PCI-PCI bridge spec (v1.1), which has this to say:

Feb 16 2024, 8:54 PM
jrtc27 added inline comments to D43910: lib{c,sys}: move auxargs more firmly into libsys.
Feb 16 2024, 8:31 PM

Feb 15 2024

jrtc27 accepted D43860: reboot: Use posix_spawn instead of system.
Feb 15 2024, 11:17 PM
jrtc27 added a comment to D43922: pcib: Assume a window where both the base and limit are 0 is uninitialized.
diff --git a/sys/riscv/sifive/fu740_pci_dw.c b/sys/riscv/sifive/fu740_pci_dw.c
index 13937e283042..d0490d6548f2 100644
--- a/sys/riscv/sifive/fu740_pci_dw.c
+++ b/sys/riscv/sifive/fu740_pci_dw.c
@@ -215,12 +215,6 @@ fupci_phy_init(struct fupci_softc *sc)
                return (error);
        }
Feb 15 2024, 10:55 PM
jrtc27 added a comment to D40474: intrng: call pic_init_secondary on all registered PICs.

@jrtc27 not being on intimate terms with the GICv3 implementation and lacking hardware to confirm how things work, I've got no idea what to do. What I do know is the extra PIC for this device never gets into sc->gic_children and therefore the loop in gic_v3_init_secondary() doesn't work for this device.

Feb 15 2024, 10:52 PM
jrtc27 added a comment to D43922: pcib: Assume a window where both the base and limit are 0 is uninitialized.

Well, I isolated it down to a single bootverbose that makes it work, but this just tells me it's almost certainly just because of the delay:

Feb 15 2024, 8:44 PM
jrtc27 added a comment to D43922: pcib: Assume a window where both the base and limit are 0 is uninitialized.

Perhaps worth mentioning HiFive Unmatched somewhere in the description so there's a breadcrumb to follow for where this came from in the first place?

Feb 15 2024, 7:17 PM
jrtc27 added a comment to D43922: pcib: Assume a window where both the base and limit are 0 is uninitialized.

s/VAR/BAR/ in the message.

Feb 15 2024, 5:59 PM
jrtc27 added inline comments to D43918: netlink: Don't use a zero-length array.
Feb 15 2024, 12:31 AM

Feb 13 2024

jrtc27 added inline comments to D43860: reboot: Use posix_spawn instead of system.
Feb 13 2024, 6:55 PM

Feb 12 2024

jrtc27 added a comment to D43849: riscv: add firmware partitions to GENERICSD.conf.

Doing this for the Unmatched doesn’t seem very useful when we can’t use the SD card for our rootfs due to driver limitations. I’ve generally taken the view that the “correct” way to treat all these dev boards so far is to treat firmware as distinct, whether that means using on-board flash (and a normally-partitioned drive in any form) or a dedicated firmware SD card (and a normally-partitioned drive that’s not the same SD card). The Arm world is a mess with all the special firmware you need, and we should be pushing for standard EFI boot flows where the firmware is part of the board rather than the OS (even if we ship updates to it for convenience).

I really don't have the Unmatched in mind here; on that board it is easiest to use the memstick installer on a USB, and that's what I recommended in the wiki instructions. In other words, driver issues notwithstanding, I still wouldn't recommend GENERICSD for that board.

The problem I am trying to solve here is more related to the VF2, and conceivably other smaller SBCs where root-and-firmware-on-SD is desirable/common. Even in this development stage, trying to prep an SD card for this board is non-trivial. If I dd the GENERICSD image, I then need to rewrite the partition table in order to insert the firmware partitions at the correct indices (gpart cannot modify a partition's index). So, reserving these partitions is a pragmatic choice aiming to reduce the number of user-required steps to install with GENERICSD.

Btw, the choice to use hifive-fsbl and hifive-bbl as the default types is somewhat arbitrary. gpart has these type aliases, and the Unleashed came first of all, so it seems a fine choice.

Feb 12 2024, 7:37 PM
jrtc27 added a comment to D43849: riscv: add firmware partitions to GENERICSD.conf.

Doing this for the Unmatched doesn’t seem very useful when we can’t use the SD card for our rootfs due to driver limitations.

Which driver ? ours ?

Feb 12 2024, 7:31 PM
jrtc27 added a comment to D43849: riscv: add firmware partitions to GENERICSD.conf.

Doing this for the Unmatched doesn’t seem very useful when we can’t use the SD card for our rootfs due to driver limitations. I’ve generally taken the view that the “correct” way to treat all these dev boards so far is to treat firmware as distinct, whether that means using on-board flash (and a normally-partitioned drive in any form) or a dedicated firmware SD card (and a normally-partitioned drive that’s not the same SD card). The Arm world is a mess with all the special firmware you need, and we should be pushing for standard EFI boot flows where the firmware is part of the board rather than the OS (even if we ship updates to it for convenience).

Feb 12 2024, 6:32 PM

Feb 10 2024

jrtc27 updated the summary of D43806: arm: Set NEW_PCIB in DEFAULTS rather than a subset of kernel configs.
Feb 10 2024, 7:09 PM
jrtc27 added a comment to D43806: arm: Set NEW_PCIB in DEFAULTS rather than a subset of kernel configs.
In D43806#999575, @jhb wrote:

s/SYS_RES_BUS/PCI_RES_BUS/ in commit log

Feb 10 2024, 7:09 PM
jrtc27 committed rG89c3cc20d643: committers-src: Extremely belatedly add myself (authored by jrtc27).
committers-src: Extremely belatedly add myself
Feb 10 2024, 6:08 AM
jrtc27 committed R9:07b41775923e: shared: Extremely belatedly add myself to committers list (authored by jrtc27).
shared: Extremely belatedly add myself to committers list
Feb 10 2024, 5:51 AM

Feb 9 2024

jrtc27 requested review of D43806: arm: Set NEW_PCIB in DEFAULTS rather than a subset of kernel configs.
Feb 9 2024, 7:31 PM
jrtc27 committed rGc994f226bbd3: bsdinstall: Add new Auto option to netconfig interface selection dialog (authored by jrtc27).
bsdinstall: Add new Auto option to netconfig interface selection dialog
Feb 9 2024, 6:26 PM
jrtc27 closed D43731: bsdinstall: Add new Auto option to netconfig interface selection dialog.
Feb 9 2024, 6:25 PM

Feb 7 2024

jrtc27 closed D43785: sysutils/u-boot-bhyve-arm64: Enable MBR and ISO9660 partition support.

Landed in ba5777140ec41aae909456f6da77e1b336f52fcb

Feb 7 2024, 11:30 PM
jrtc27 added a comment to D43784: sysutils/u-boot-bhyve-arm64: Fix after the 2024.01 import.

I suspect this isn't the right approach long-term, but this works in my local testing. This will need U_BOOT_SLAVE_PORTREVISION_2024.01 to be set, but perhaps best to batch that with the patch I have and was trying to test when I encountered this regression?

FYI this is https://reviews.freebsd.org/D43785. I don't mind bumping the version again, but it does seem a little silly.

Oh, yes. Would you like to merge the patches and push? If not I'll take care of it now.

Feb 7 2024, 5:21 PM
jrtc27 added a comment to D43784: sysutils/u-boot-bhyve-arm64: Fix after the 2024.01 import.

I suspect this isn't the right approach long-term, but this works in my local testing. This will need U_BOOT_SLAVE_PORTREVISION_2024.01 to be set, but perhaps best to batch that with the patch I have and was trying to test when I encountered this regression?

Feb 7 2024, 5:04 PM
jrtc27 requested review of D43785: sysutils/u-boot-bhyve-arm64: Enable MBR and ISO9660 partition support.
Feb 7 2024, 5:03 PM
jrtc27 accepted D43784: sysutils/u-boot-bhyve-arm64: Fix after the 2024.01 import.

I suspect this isn't the right approach long-term, but this works in my local testing. This will need U_BOOT_SLAVE_PORTREVISION_2024.01 to be set, but perhaps best to batch that with the patch I have and was trying to test when I encountered this regression?

Feb 7 2024, 4:54 PM
jrtc27 committed rGb8afdda360e5: gicv3_its: Drop a couple of redundant vm_offset_t casts (authored by jrtc27).
gicv3_its: Drop a couple of redundant vm_offset_t casts
Feb 7 2024, 4:40 PM

Feb 6 2024

jrtc27 added a comment to D40474: intrng: call pic_init_secondary on all registered PICs.

@jrtc27 I was hoping for an answer to my question before doing anything.

I lack appropriate hardware to confirm this, but I suspect gic_v3_init_secondary() was already being called a second time for systems with both a normal PIC and a MSI PIC. As long as both were GICv3 implementations and one was a direct child of the other, the loop at the end of gic_v3_init_secondary() was doing this.

Feb 6 2024, 8:16 PM

Feb 5 2024

jrtc27 requested changes to D43755: bsdinstall: separate out dist selection in prep for pkgbase support.

I've tried to give this a more thorough review and found some issues. Please test it more thoroughly for the various different cases that arise.

Feb 5 2024, 10:57 PM

Feb 4 2024

jrtc27 added a comment to D43743: ctf*: exit with error upon terminate().

(and please also note in the commit message that #ifdef __FreeBSD__ for behavioural changes like this in bootstrap tools is incorrect now that we have cross-building from non-FreeBSD, i.e. that cross-building from macOS and Linux can currently fail when FreeBSD ignores the error)

Feb 4 2024, 10:05 PM
jrtc27 added a comment to D43743: ctf*: exit with error upon terminate().

This is long overdue! Though terminate_cleanup needs unifdef'ing (introduced in the import itself in https://cgit.freebsd.org/src/tree/cddl/contrib/opensolaris/tools/ctf/cvt/ctfconvert.c?id=4cc75139b96639698b4e96da3b60cd3d81e9a959). Note this code itself came into the tree in https://cgit.freebsd.org/src/commit/?id=c01977ed3b6d325bae5de5025eb0c6357ee73be5, which is similarly undocumented in the commit message.

Feb 4 2024, 10:03 PM
jrtc27 added a reviewer for D43731: bsdinstall: Add new Auto option to netconfig interface selection dialog: jhb.
Feb 4 2024, 1:43 AM
jrtc27 requested review of D43731: bsdinstall: Add new Auto option to netconfig interface selection dialog.
Feb 4 2024, 1:39 AM

Feb 3 2024

jrtc27 committed rG1e75e61d5c2c: bsdinstall: Fix broken netconfig persistence for installed system (authored by jrtc27).
bsdinstall: Fix broken netconfig persistence for installed system
Feb 3 2024, 11:12 PM
jrtc27 added inline comments to D43293: RISCV: Introduce support for APLIC interrupt controller.
Feb 3 2024, 10:04 AM · riscv
jrtc27 committed rG7414d14bd51d: bsdinstall: Drop Error from title in netconfig no interfaces dialog (authored by jrtc27).
bsdinstall: Drop Error from title in netconfig no interfaces dialog
Feb 3 2024, 2:01 AM
jrtc27 committed rGb809c7d6a269: bsdinstall: Fix netconfig script when no interfaces are present (authored by jrtc27).
bsdinstall: Fix netconfig script when no interfaces are present
Feb 3 2024, 2:01 AM
jrtc27 added a reverting change for rG009d3f66cb5f: bsdinstall: separate out dist selection in prep for pkgbase support: rG7143521f4cbc: Revert "bsdinstall: separate out dist selection in prep for pkgbase support".
Feb 3 2024, 1:32 AM
jrtc27 committed rG7143521f4cbc: Revert "bsdinstall: separate out dist selection in prep for pkgbase support" (authored by jrtc27).
Revert "bsdinstall: separate out dist selection in prep for pkgbase support"
Feb 3 2024, 1:32 AM
jrtc27 added a reverting change for D43621: bsdinstall: separate out dist selection in prep for pkgbase support: rG7143521f4cbc: Revert "bsdinstall: separate out dist selection in prep for pkgbase support".
Feb 3 2024, 1:32 AM

Feb 2 2024

jrtc27 added inline comments to D43293: RISCV: Introduce support for APLIC interrupt controller.
Feb 2 2024, 10:26 PM · riscv
jrtc27 added a reverting change for rGcd19ecdbdc87: Similar to r296013 for NO_ROOT, force SUBDIR_PARALLEL for buildworld WORLDTMP…: rGfbae308319b7: bsd.subdir.mk: Drop broken optimisation for realinstall parallelisation.
Feb 2 2024, 9:19 PM
jrtc27 closed D43705: bsd.subdir.mk: Drop broken optimisation for realinstall parallelisation.
Feb 2 2024, 9:18 PM
jrtc27 committed rGfbae308319b7: bsd.subdir.mk: Drop broken optimisation for realinstall parallelisation (authored by jrtc27).
bsd.subdir.mk: Drop broken optimisation for realinstall parallelisation
Feb 2 2024, 9:18 PM
jrtc27 added a reverting change for rGcd19ecdbdc87: Similar to r296013 for NO_ROOT, force SUBDIR_PARALLEL for buildworld WORLDTMP…: D43705: bsd.subdir.mk: Drop broken optimisation for realinstall parallelisation.
Feb 2 2024, 1:41 AM
jrtc27 requested review of D43705: bsd.subdir.mk: Drop broken optimisation for realinstall parallelisation.
Feb 2 2024, 1:41 AM

Feb 1 2024

jrtc27 added inline comments to D43293: RISCV: Introduce support for APLIC interrupt controller.
Feb 1 2024, 11:28 AM · riscv
jrtc27 added inline comments to D32955: Retire unused non-PCI_RES_BUS code.
Feb 1 2024, 1:29 AM

Jan 31 2024

jrtc27 committed rGb771d5705fc9: tools/build/make.py: Avoid Python 3.7+ subprocess.run capture_output (authored by jrtc27).
tools/build/make.py: Avoid Python 3.7+ subprocess.run capture_output
Jan 31 2024, 7:51 PM
jrtc27 added inline comments to D43293: RISCV: Introduce support for APLIC interrupt controller.
Jan 31 2024, 7:13 PM · riscv
jrtc27 requested changes to D43293: RISCV: Introduce support for APLIC interrupt controller.
Jan 31 2024, 7:13 PM · riscv
jrtc27 added inline comments to D43666: bhyve: Add support for XML register definitions.
Jan 31 2024, 6:34 PM
jrtc27 added inline comments to D43666: bhyve: Add support for XML register definitions.
Jan 31 2024, 6:33 PM
jrtc27 added inline comments to D43666: bhyve: Add support for XML register definitions.
Jan 31 2024, 6:31 PM

Jan 30 2024

jrtc27 committed rG41eb4a4ca299: share/man: Add mandoc.db files to METALOG (authored by jrtc27).
share/man: Add mandoc.db files to METALOG
Jan 30 2024, 10:19 PM
jrtc27 committed rGce047ebf4848: etc: Set DISTBASE during distribute for sub-makes (authored by jrtc27).
etc: Set DISTBASE during distribute for sub-makes
Jan 30 2024, 10:19 PM
jrtc27 closed D43676: share/man: Add mandoc.db files to METALOG.
Jan 30 2024, 10:19 PM
jrtc27 closed D43675: etc: Set DISTBASE during distribute for sub-makes.
Jan 30 2024, 10:19 PM
jrtc27 added a comment to D42968: freebsd-update: Mention upgrading packages, not just ports.

Ping?

Jan 30 2024, 9:23 PM
jrtc27 requested review of D43676: share/man: Add mandoc.db files to METALOG.
Jan 30 2024, 9:22 PM
jrtc27 requested review of D43675: etc: Set DISTBASE during distribute for sub-makes.
Jan 30 2024, 9:22 PM
jrtc27 committed rG722b40377198: riscv: Remove the unused riscv64_cpu driver (authored by jrtc27).
riscv: Remove the unused riscv64_cpu driver
Jan 30 2024, 8:34 PM
jrtc27 closed D43672: riscv: Remove the unused riscv64_cpu driver.
Jan 30 2024, 8:34 PM
jrtc27 requested review of D43672: riscv: Remove the unused riscv64_cpu driver.
Jan 30 2024, 8:01 PM
jrtc27 added inline comments to D43621: bsdinstall: separate out dist selection in prep for pkgbase support.
Jan 30 2024, 7:27 PM
jrtc27 added inline comments to D43621: bsdinstall: separate out dist selection in prep for pkgbase support.
Jan 30 2024, 6:00 PM
jrtc27 requested changes to D43660: Replace boolean_t with bool.

This kind of diff is way too big to review, it needs to be broken up. It's easy to mass rename like this but a lot harder to verify that you've actually done the right thing. The second file I picked to look at the diff at was clearly not correct (see the comment linked to this), so that doesn't bode well for the rest of the diff. Also some of the NIC drivers in the tree are really vendor imports, so any changes like this that get made to them should make their way upstream, otherwise there is a risk they got lost. I have not checked if that applies here or not.

Jan 30 2024, 5:23 AM · Contributor Reviews (src)

Jan 29 2024

jrtc27 requested changes to D43254: libc: Remove support for pre-C11 C standards.
Jan 29 2024, 9:12 PM · Contributor Reviews (src)
jrtc27 added inline comments to D43621: bsdinstall: separate out dist selection in prep for pkgbase support.
Jan 29 2024, 2:55 AM