Page MenuHomeFreeBSD

imp (Warner Losh)
User

Projects (6)

User Details

User Since
Jun 2 2014, 4:20 PM (592 w, 6 d)

Recent Activity

Today

imp added a comment to D53046: vmimage.subr: Sort METALOG before running makefs.

I wondet if we can fix install...

Sun, Oct 12, 4:56 PM
imp added a comment to D53046: vmimage.subr: Sort METALOG before running makefs.

I've not hit the // issue. When does it happen?

Sun, Oct 12, 4:34 PM
imp added a comment to D53046: vmimage.subr: Sort METALOG before running makefs.

I used awk to omit first arg same as last. I think the -u is likely useless.

Sun, Oct 12, 11:41 AM

Yesterday

imp accepted D53036: stand/i386: quiet gcc -Warray-bounds.
Sat, Oct 11, 1:33 AM

Fri, Oct 10

imp accepted D53030: cxgbe: Stop using bus_space_tag/handle directly.
Fri, Oct 10, 11:04 PM
imp added a comment to D53014: release: switch OCI containers to use pkg sets.

I'm cool removing devd. I was cool having people add it back manually if need be.

Fri, Oct 10, 2:49 AM
imp committed rG53825afc5dd1: nvme: There's 64 LBAF descriptors, not 16 (authored by imp).
nvme: There's 64 LBAF descriptors, not 16
Fri, Oct 10, 12:21 AM
imp committed rG498368529011: modules: Simplify some expressions (authored by imp).
modules: Simplify some expressions
Fri, Oct 10, 12:21 AM
imp committed rG3faa5bfece73: kern: Don't need bsd.own.mk (authored by imp).
kern: Don't need bsd.own.mk
Fri, Oct 10, 12:21 AM
imp committed rG8e985774117d: kern: Remove needless kern.opts.mk (authored by imp).
kern: Remove needless kern.opts.mk
Fri, Oct 10, 12:21 AM

Wed, Oct 8

imp accepted D52923: bsd.confs.mk: Support CONFGROUPS.yes.
Wed, Oct 8, 5:56 AM
imp accepted D52974: libexec/rc/rc.d/Makefile: Use CONFGROUPS.yes.
Wed, Oct 8, 5:56 AM
imp added a comment to D52959: kernel: MK_REPRODUCIBLE_PATHS for full paths vs /usr/src.

I'm cool with it off by default too.
But then I'd have to hunt down the builds scripts for the release and add it there :)

Wed, Oct 8, 3:08 AM
imp accepted D52973: nvme: Tweak completion timeout loop.

Not so sure about the expression rewrite, but it's not likely to change to 5 / 4 or 7 /3 any time soon

Wed, Oct 8, 1:21 AM
imp accepted D52958: BSD.root.dist: Add package tag for all directories.

Assuming they are recrusive, this looks perfect.

Wed, Oct 8, 12:27 AM

Tue, Oct 7

imp added a reverting change for rGce89c8f47a91: cam/iosched: Initialize max_lat: rG86becbd8c1ce: Revert "cam/iosched: Initialize max_lat".
Tue, Oct 7, 11:59 PM
imp committed rG86becbd8c1ce: Revert "cam/iosched: Initialize max_lat" (authored by imp).
Revert "cam/iosched: Initialize max_lat"
Tue, Oct 7, 11:59 PM
imp committed rGce89c8f47a91: cam/iosched: Initialize max_lat (authored by imp).
cam/iosched: Initialize max_lat
Tue, Oct 7, 10:27 PM
imp requested review of D52959: kernel: MK_REPRODUCIBLE_PATHS for full paths vs /usr/src.
Tue, Oct 7, 4:43 PM
imp requested changes to D52933: cross-build: Define __GNUC_PREREQ__ in cdefs.h.

I know it's just for cross compiling, but maybe we can use the version we have in cdefs.h to avoid other foot-cannons in the future? The only reason this works is the || <something that works> in the instant use case.

/*
 * Macro to test if we're using a specific version of gcc or later.
 */
#if defined(__GNUC__)
#define __GNUC_PREREQ__(ma, mi) \
        (__GNUC__ > (ma) || __GNUC__ == (ma) && __GNUC_MINOR__ >= (mi))
#else
#define __GNUC_PREREQ__(ma, mi) 0
#endif
Tue, Oct 7, 2:22 PM
imp accepted D52929: Makefile: Force DISTDIR=/ for (update-)packages.
Tue, Oct 7, 9:35 AM

Mon, Oct 6

imp added a comment to D52923: bsd.confs.mk: Support CONFGROUPS.yes.
In D52923#1209247, @ivy wrote:
In D52923#1209241, @imp wrote:

I'd commit bsd.conf.mk as its own commot. I did this to subdirs years ago

i could split this into two commits if you insist, but i'd rather not. the changes to both files are self-contained, so it will look fine in the history.

Mon, Oct 6, 1:58 PM
imp accepted D52923: bsd.confs.mk: Support CONFGROUPS.yes.

I'd commit bsd.conf.mk as its own commot. I did this to subdirs years ago

Mon, Oct 6, 7:26 AM

Sun, Oct 5

imp added inline comments to D52911: release: Add /boot/efi to METALOG.
Sun, Oct 5, 8:33 PM
imp added a comment to D52911: release: Add /boot/efi to METALOG.
In D52911#1208992, @imp wrote:

So why aren't we using this mtree in the process? It should have added to the metalog in make distribute since that's what uses this...

We're now building cloudware from pkgbase, so we don't run make distribute. So I guess the real question is why /boot/efi didn't end up in any of the pkgbase packages.

Sun, Oct 5, 8:29 PM
imp committed rG106951f09fe3: Makefile.inc1: Add back missing if (authored by imp).
Makefile.inc1: Add back missing if
Sun, Oct 5, 3:15 PM
imp added a comment to D52904: build: don't create duplicate bootstrap-tools targets.

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=289997

Sun, Oct 5, 3:05 PM
imp committed rG28b858f5059c: Makefile.inc1: Remove svn support (authored by imp).
Makefile.inc1: Remove svn support
Sun, Oct 5, 2:27 PM
imp closed D52912: Makefile.inc1: Remove svn support.
Sun, Oct 5, 2:26 PM

Sat, Oct 4

imp added a comment to D52911: release: Add /boot/efi to METALOG.

Except it is in BSD.root.dist:

...
    boot
...
       efi
...
Sat, Oct 4, 11:34 PM
imp added inline comments to D52911: release: Add /boot/efi to METALOG.
Sat, Oct 4, 11:33 PM
imp committed rGd8e73f45fc5f: bsdinstall: Add loader.efi to all ESPs we create (authored by imp).
bsdinstall: Add loader.efi to all ESPs we create
Sat, Oct 4, 11:19 PM
imp requested review of D52912: Makefile.inc1: Remove svn support.
Sat, Oct 4, 11:15 PM
imp accepted D52907: Replace some spaces with tabs to match the rest of this struct.
Sat, Oct 4, 11:04 PM
imp accepted D52911: release: Add /boot/efi to METALOG.
Sat, Oct 4, 11:04 PM
imp accepted D52906: mkimg: Add a -h option and improve usage output to be more expressive.
Sat, Oct 4, 7:19 PM
imp accepted D52904: build: don't create duplicate bootstrap-tools targets.

This is obscure enough we may want to have a comment..

Sat, Oct 4, 6:49 PM
imp committed rG4b734c477446: bsdinstall: Add loader.efi to all ESPs we create (authored by imp).
bsdinstall: Add loader.efi to all ESPs we create
Sat, Oct 4, 12:28 AM
imp accepted D52881: generate-ucl.lua: Add missing copyright header.
Sat, Oct 4, 12:11 AM

Fri, Oct 3

imp accepted D52763: geom: zero: Let sysctls .byte and .clear to be set in loader.
Fri, Oct 3, 11:35 PM
imp added inline comments to D52885: (draft) style.9: Encourage style changes when doing significant modifications.
Fri, Oct 3, 5:01 PM
imp accepted D52889: ahci: Don't fail attach if the MSI-X BARs cannot be allocated.
Fri, Oct 3, 4:52 PM
imp accepted D52888: ahci: Use pci_msix_*_bar instead of reading config registers directly.
Fri, Oct 3, 4:51 PM
imp added inline comments to D52866: nda.4: Add a HARDWARE section, tag SPDX.
Fri, Oct 3, 2:02 PM
imp added inline comments to D52881: generate-ucl.lua: Add missing copyright header.
Fri, Oct 3, 4:25 AM

Thu, Oct 2

imp added a comment to D52860: sdhci.4: Improve HARDWARE.

Looking at the hardware driver suggests this page is even more out of date than I thought.
I don't have time to enumerate everything here.

Thu, Oct 2, 8:57 PM

Wed, Oct 1

imp added a comment to D52833: bsd.cpu.mk: Fix whitespace appending to MACHINE_ABI.
In D52833#1207103, @imp wrote:

Fun. Commit message could be better at explaining why this happens with the old code

There's no functional change. It's just that I copied and pasted without tabs.

Wed, Oct 1, 8:48 PM
imp accepted D52833: bsd.cpu.mk: Fix whitespace appending to MACHINE_ABI.

Fun. Commit message could be better at explaining why this happens with the old code

Wed, Oct 1, 4:06 PM
imp accepted D52815: isa_common: add all listed devices.

I suspect this is fine. ISA is all the on boatd IO that's ioport mapped that's not part of pci. It most likely is lpc based and the ACPI enumeration of that jardware should hang off this bus, not the acpi bus for the software model. But practically, this is easier and there's not enough difference to mater.

Wed, Oct 1, 10:16 AM
imp accepted D52817: arch.7: Remove mention of PDP11_ENDIAN.

PDP_ENDIAN was the spelling. It was also the order for a subset of pdp-11 compilers.

Wed, Oct 1, 10:09 AM

Tue, Sep 30

imp committed rG29810aa6ae70: libc: Move tzset.3 to stdtime (authored by imp).
libc: Move tzset.3 to stdtime
Tue, Sep 30, 7:29 PM
imp added a comment to D52798: build.7: Clarify TARGET and TARGET_ARCH.

So internally to the build system, MACHINE and MACHINE_ARCH are used. When cross building, for various technical reasons, you have to set TARGET and/or TARGET_ARCH. When one can be guessed from the other (as is the case for all architectures except powerpc), you need only set one, the other one will be inferred. For powerpc, setting TARGET_ARCH alone is sufficient, so maybe we should recommend that for all cases. I'd put a cross reference to arch(7) since MACHINE and MACHINE_ARCH, and their allowed values, are explained in detail. The make target only lists a subset of allowed values at times when we've deprecated some targets but haven't yet removed them .

Tue, Sep 30, 2:50 PM
imp committed rG494de51bc007: bsdinstall: Add loader.efi to all ESPs we create (authored by imp).
bsdinstall: Add loader.efi to all ESPs we create
Tue, Sep 30, 2:58 AM
imp closed D52780: bsdinstall: Add loader.efi to all ESPs we create.
Tue, Sep 30, 2:57 AM
imp accepted D52726: wlanstat(8): Follow-ups after moving to /usr/sbin and renaming.
Tue, Sep 30, 1:50 AM
imp accepted D52724: wlanstat(8): Rename from wlanstats(8) to keep consistency.
Tue, Sep 30, 1:46 AM

Mon, Sep 29

imp updated the diff for D52780: bsdinstall: Add loader.efi to all ESPs we create.

update comments, some minor tweaking

Mon, Sep 29, 11:24 PM
imp accepted D52197: gpio: implement bus_setup_intr and bus_teardown_intr.

This looks good to me.

Mon, Sep 29, 11:08 PM
imp added inline comments to D52780: bsdinstall: Add loader.efi to all ESPs we create.
Mon, Sep 29, 9:39 AM
imp updated the test plan for D52780: bsdinstall: Add loader.efi to all ESPs we create.
Mon, Sep 29, 9:38 AM
imp requested review of D52780: bsdinstall: Add loader.efi to all ESPs we create.
Mon, Sep 29, 9:37 AM
imp committed rGebc32e74b6f9: core.lua: Consistently check for single user (authored by imp).
core.lua: Consistently check for single user
Mon, Sep 29, 1:49 AM
imp committed rGb7473a5d68b8: loader: make disable-device more rebust (authored by imp).
loader: make disable-device more rebust
Mon, Sep 29, 1:49 AM
imp committed rG55025f42f6fa: stand: Remove support for FreeBSD 12.2 and earlier (authored by imp).
stand: Remove support for FreeBSD 12.2 and earlier
Mon, Sep 29, 1:49 AM

Sun, Sep 28

imp added inline comments to D52734: release: Fix booting on aarch64 in Azure by adding the BOOTPARTSOFFSET.
Sun, Sep 28, 2:45 PM
imp accepted D52760: mpr.4: Add Broadcom LSI 3808 to support list.
Sun, Sep 28, 1:01 AM · manpages
imp accepted D52125: mrsas.4: Cleaning.
Sun, Sep 28, 12:57 AM

Fri, Sep 26

imp accepted D52736: mdconfig_test: determine page size at run time rather than assuming 4k.
Fri, Sep 26, 10:06 PM
imp added a comment to D52734: release: Fix booting on aarch64 in Azure by adding the BOOTPARTSOFFSET.

So this is a bit tricky for why we're doing this and how it solves the problem. See suggestion for a comment.

Fri, Sep 26, 5:46 PM
imp accepted D52742: packages: Add efi-tools to set-minimal.

Aarch64 too

Fri, Sep 26, 2:54 AM

Thu, Sep 25

imp added a comment to D52735: mmap_test: determine page size at run time rather than compile time.

This is needed also for arm64

Thu, Sep 25, 8:18 PM
imp accepted D52735: mmap_test: determine page size at run time rather than compile time.
Thu, Sep 25, 8:18 PM
imp committed R9:9b95e33e870f: pgp: Update my expiration date (authored by imp).
pgp: Update my expiration date
Thu, Sep 25, 6:34 PM
imp accepted D52728: pci: add quirk to disable FLR.
Thu, Sep 25, 1:06 PM
imp added a comment to D50825: libexec/kgdb: Add a new VNET function and add more scaffolding.

Python looks ok

Thu, Sep 25, 1:04 PM

Wed, Sep 24

imp committed rGd466cb731a23: make packages: Fix edge case of no kernel (authored by imp).
make packages: Fix edge case of no kernel
Wed, Sep 24, 7:57 PM
imp committed rG2352b89d2f73: nanobsd: Fix regressions from last cleanup (authored by imp).
nanobsd: Fix regressions from last cleanup
Wed, Sep 24, 4:45 PM
imp added inline comments to D52596: nanobsd: minor formatting cleanup.
Wed, Sep 24, 3:48 PM
imp added inline comments to D52706: geom_zero.4: Document.
Wed, Sep 24, 1:54 PM

Mon, Sep 22

imp added inline comments to D52644: packages.7: New manpage describing pkgbase.
Mon, Sep 22, 1:32 AM
imp added a comment to D52644: packages.7: New manpage describing pkgbase.

I like base way better than packages, which has the sane issues as pkgbase, and it's too generic.

Mon, Sep 22, 1:17 AM

Sun, Sep 21

imp added a comment to D52644: packages.7: New manpage describing pkgbase.

So is this the place to document the config files you need to fetch a custom repo?
Or what the distdir settings are for -current and -stable and how to get releases?
Or that if you pkg install all these things, you still need to do some flavoring to the system (fstab, rc.conf, loader.conf and a few others)

Sun, Sep 21, 7:38 PM

Sat, Sep 20

imp accepted D52645: packages: Move powerd to its own package.

Spammy how? Unlike ports, UPDATING isn't automatically displayed. I imagine that they'd slow way down once we have the deck chairs just so...

Sat, Sep 20, 11:10 PM
imp accepted D52648: elfdump: Move to the toolchain package.
Sat, Sep 20, 11:07 PM

Fri, Sep 19

imp added inline comments to D52617: release: Rework oci image handling using metalogs.
Fri, Sep 19, 2:50 PM
imp added a comment to D52617: release: Rework oci image handling using metalogs.

You might want to look at ~imp/armv7-pkgbase-14.3-exp.sh which generates a bootable image w/o root for armv7 with the correct perms using pkgbase + pkg to create the system.

Fri, Sep 19, 2:20 PM
imp accepted D48693: build/development.7: System building examples.

Might be a few more nits, but we're getting close to time to just accept what's done and do further refinement later.

Fri, Sep 19, 2:02 AM

Thu, Sep 18

imp added a comment to D52085: bnxt_en: Periodically synchronize host time with firmware.

W/o measuring it, or knowing what the NIC is doing, it's hard to say how good the results from this will be.

Thu, Sep 18, 6:16 PM
imp accepted D52440: ufshci: revisit controller reset path and add I/O timeout handling.

This looks better now. Doing things better next time is fine.

Thu, Sep 18, 12:59 AM
imp accepted D52570: committers-src: add myself (jaeyoon@).

this is ready too!

Thu, Sep 18, 12:58 AM
imp accepted D52569: new committer (src): Jaeyoon Choi (jaeyoon).

This is ready!

Thu, Sep 18, 12:58 AM
imp added inline comments to D51471: release: Add 2 image layers for almost all base.txz packages excluding toolchain, and one with toolchain.
Thu, Sep 18, 12:19 AM
imp added inline comments to D51471: release: Add 2 image layers for almost all base.txz packages excluding toolchain, and one with toolchain.
Thu, Sep 18, 12:06 AM

Wed, Sep 17

imp accepted D52596: nanobsd: minor formatting cleanup.
Wed, Sep 17, 6:36 PM
imp accepted D52596: nanobsd: minor formatting cleanup.

There's a few of these elsewhere in the nanobsd tree.
It's also the only place in the tree we do this style, so this is good, I think.

Wed, Sep 17, 6:30 PM

Mon, Sep 15

imp accepted D52559: boot/universe.sh: toggle LOADER_BIOS_TEXTONLY.
Mon, Sep 15, 9:22 PM
imp accepted D52557: if_re: Add PNP info for module.

I'm surprised this hasn't been done sooner

Mon, Sep 15, 9:21 PM
imp committed rGe239f49146e0: UPDATING: fix typos (authored by Harry Fellowes <harryfellowes1@gmail.com>).
UPDATING: fix typos
Mon, Sep 15, 3:17 AM
imp committed rG5e0a4859f28a: iwlwifi: Don't compile for gcc before 14 (authored by imp).
iwlwifi: Don't compile for gcc before 14
Mon, Sep 15, 3:07 AM

Sun, Sep 14

imp accepted D51619: kern: Introduce kexec system feature (MI).

I didn't see anything in the implementation of the MI parts to give me heartburn.

Sun, Sep 14, 11:03 PM