Page MenuHomeFreeBSD

imp (Warner Losh)
User

Projects (6)

User Details

User Since
Jun 2 2014, 4:20 PM (603 w, 18 h)

Recent Activity

Today

imp added a comment to D54289: ixl(4): Enable building driver on ARM64.

I'd still be tempted to put it in files and just spike 32bit builds. Sure, you can put it in the config file and yhe error is delayed until you build, but that's ok imho. Lots of things in files that don't work evertwhere. No need for a files.64.

Tue, Dec 23, 8:23 AM

Yesterday

imp accepted D54285: camcontrol.8: Descriptions before examples.

I guess this is OK.... It's consistent.

Mon, Dec 22, 9:07 PM

Sat, Dec 20

imp added a comment to D54327: geom(9): struct bio KBI fix.

So a few words about why this is useful. Our KBI commitments on 15.x is fuzzy at best. What does this enable?

Sat, Dec 20, 8:29 PM
imp accepted D54324: kern: document rationale for PRIV_REBOOT usage in kexec_load.
Sat, Dec 20, 6:00 AM

Fri, Dec 19

imp accepted D54319: nanobsd: remove pandaboard.cfg.
Fri, Dec 19, 10:52 PM · ARM
imp accepted D54320: reboot: Fix halt -p behavior.
Fri, Dec 19, 10:49 PM
imp accepted D54313: acpi: Statically initialize acpi_ioctl_hooks.
Fri, Dec 19, 4:18 PM
imp accepted D54311: acpi: Reject duplicate handlers for ioctl commands.
Fri, Dec 19, 4:17 PM
imp accepted D54312: acpi: Use M_WAITOK in acpi_register_ioctl.
Fri, Dec 19, 4:16 PM
imp accepted D54310: acpi_apei: Remove the hest member from the softc.
Fri, Dec 19, 4:16 PM
imp accepted D54309: acpi: Free ivars in bus_child_deleted method.
Fri, Dec 19, 4:15 PM
imp accepted D54308: acpidump: Extend the decoding of entries in the EINJ and ERST tables.
Fri, Dec 19, 4:14 PM
imp accepted D54239: usr.bin: Remove intrinsic utilities.
Fri, Dec 19, 4:07 PM
imp accepted D54314: sys/modules/acpi/Makefile: One line per subdirectory.

More than 5, this is overdue

Fri, Dec 19, 3:36 PM
imp accepted D54315: Build acpi kernel modules on aarch64.
Fri, Dec 19, 3:28 PM

Wed, Dec 17

imp accepted D54255: sdhci: Try to complete the last transaction if dumping.

Ok. This makes good sense. I'd be tempted to return 0 rather than err, but both are fine.

Wed, Dec 17, 5:33 AM
imp added inline comments to D54255: sdhci: Try to complete the last transaction if dumping.
Wed, Dec 17, 12:36 AM

Tue, Dec 16

imp accepted D54213: libpathconv: Document library in the man pages.
Tue, Dec 16, 5:31 PM · manpages

Sat, Dec 13

imp added inline comments to D54213: libpathconv: Document library in the man pages.
Sat, Dec 13, 6:33 PM · manpages
imp added inline comments to D54213: libpathconv: Document library in the man pages.
Sat, Dec 13, 5:04 PM · manpages
imp accepted D54196: Revert "devinfo: Add support for libxo".

Yes. We need a way to export all the data for libxo in one path, not the twisty maze which we've gotten wrong so many times.

Sat, Dec 13, 8:35 AM
imp accepted D54197: RELNOTES: Add entries from this week's scan of the commit logs.
Sat, Dec 13, 8:30 AM

Fri, Dec 12

imp added a reviewer for D54187: nanobsd.sh: Start the transition away from 'legacy' config: jlduran.
Fri, Dec 12, 7:34 AM
imp requested review of D54187: nanobsd.sh: Start the transition away from 'legacy' config.
Fri, Dec 12, 7:33 AM
imp committed rG478ac7dc2afd: cam: Also reduce timeout for wlun probing (authored by imp).
cam: Also reduce timeout for wlun probing
Fri, Dec 12, 4:25 AM
imp closed D54184: cam: Also reduce timeout for wlun probing.
Fri, Dec 12, 4:24 AM
imp added a comment to D48763: kernel: Add `%pV` to kernel's printf format string.

yea, I'm generally OK with it, but my question about the code remains... Though I have a vague memory of another review that was more complete somehow. I was worried it would break FreeBSD code, but no code in the tree would be broken by it.

Fri, Dec 12, 3:18 AM
imp added a reviewer for D54184: cam: Also reduce timeout for wlun probing: jaeyoon.
Fri, Dec 12, 3:07 AM
imp requested review of D54184: cam: Also reduce timeout for wlun probing.
Fri, Dec 12, 3:06 AM

Thu, Dec 11

imp committed rG8ac7a3801c6a: cam: Reduce overly long timeout values for initial device probing (authored by imp).
cam: Reduce overly long timeout values for initial device probing
Thu, Dec 11, 7:08 PM
imp closed D52427: cam: Reduce overly long timeout values for initial device probing.
Thu, Dec 11, 7:08 PM

Wed, Dec 10

imp committed rG0a19464bf7af: nvme: Only attach to storage NVMe devices (authored by imp).
nvme: Only attach to storage NVMe devices
Wed, Dec 10, 10:54 PM
imp committed rG848602856f49: nvme: remove now-redundant consumer interface (authored by imp).
nvme: remove now-redundant consumer interface
Wed, Dec 10, 10:54 PM
imp committed rGb6b52a9b8011: nvme: Notify failure with newbus call (authored by imp).
nvme: Notify failure with newbus call
Wed, Dec 10, 10:54 PM
imp committed rG9b170dc457a4: nvme: Use new method to do async notifications (authored by imp).
nvme: Use new method to do async notifications
Wed, Dec 10, 10:54 PM
imp committed rGaed44717a160: nvd: Connect nvme_if methods (authored by imp).
nvd: Connect nvme_if methods
Wed, Dec 10, 10:54 PM
imp committed rGa21f12840f5f: nvme_sim: Connect to events broadcast with nvme_if (authored by imp).
nvme_sim: Connect to events broadcast with nvme_if
Wed, Dec 10, 10:54 PM
imp committed rGb32470bc6621: nvd: Attach as a child of nvme (authored by imp).
nvd: Attach as a child of nvme
Wed, Dec 10, 10:54 PM
imp committed rGe5c770dc7ff3: nvme: Nvme controller generated events (authored by imp).
nvme: Nvme controller generated events
Wed, Dec 10, 10:54 PM
imp closed D54095: nvme: remove now-redundant consumer interface.
Wed, Dec 10, 10:54 PM
imp committed rG1e39b5d4833e: nvme_sim: Attach as a child of nvme (authored by imp).
nvme_sim: Attach as a child of nvme
Wed, Dec 10, 10:54 PM
imp closed D51391: nvme: Notify failure with newbus call.
Wed, Dec 10, 10:54 PM
imp committed rG743fbb549b17: nvme: Add child device for each controller (authored by imp).
nvme: Add child device for each controller
Wed, Dec 10, 10:54 PM
imp closed D51390: nvme: Use new method to do async notifications.
Wed, Dec 10, 10:54 PM
imp closed D51388: nvd: Connect nvme_if methods.
Wed, Dec 10, 10:54 PM
imp closed D51387: nvme_sim: Connect to events broadcast with nvme_if.
Wed, Dec 10, 10:54 PM
imp closed D51385: nvd: Attach as a child of nvme.
Wed, Dec 10, 10:54 PM
imp closed D51386: nvme: Nvme controller generated events.
Wed, Dec 10, 10:54 PM
imp closed D51384: nvme_sim: Attach as a child of nvme.
Wed, Dec 10, 10:54 PM
imp closed D51383: nvme: Add child device for each controller.
Wed, Dec 10, 10:54 PM
imp committed rGb19e5e7ce4c7: cam: Expand the parts of the sense buffer we report (authored by imp).
cam: Expand the parts of the sense buffer we report
Wed, Dec 10, 9:42 PM
imp committed rGd3e79182960a: cam: decode and print direct accecss block device sense data (authored by imp).
cam: decode and print direct accecss block device sense data
Wed, Dec 10, 9:42 PM
imp committed rG84d198e137a5: mpr: Partially revert 332096ebb638 (authored by imp).
mpr: Partially revert 332096ebb638
Wed, Dec 10, 9:13 PM
imp added a reverting change for rG332096ebb638: kboot: Explicitly use host:/proc: rG84d198e137a5: mpr: Partially revert 332096ebb638.
Wed, Dec 10, 9:13 PM
imp committed rG332096ebb638: kboot: Explicitly use host:/proc (authored by imp).
kboot: Explicitly use host:/proc
Wed, Dec 10, 7:05 PM
imp accepted D54122: shutdown: Adapt to changes in reboot.
Wed, Dec 10, 2:38 PM
imp accepted D54155: acpi_gpiobus: Reuse the existing IVAR index for an ACPI handle.
Wed, Dec 10, 5:47 AM
imp accepted D54117: reboot: Default to a clean shutdown.

From a historical perspective, I'd prefer fastboot == reboot -f. However, that ship has sailed and we're stuck with either preserving a lot of the current behavior with reboot -f checking or also overloading it with fastboot. Or moving the force to -F. I hate myself that I added -f, but it can't be helped. So fastboot is the least bad choice, imho.

Wed, Dec 10, 3:10 AM

Tue, Dec 9

imp accepted D54124: sys/_types.h: recognise char8_t as a builtin type in C++20.
Tue, Dec 9, 5:29 AM
imp accepted D54147: zfs: Reuse ZINCDIR variable from kmod.mk.
Tue, Dec 9, 1:57 AM

Mon, Dec 8

imp accepted D54120: nextboot: Reimplement missing -a option.

I have a vague notion of doing this before...

Mon, Dec 8, 6:39 AM
imp accepted D54118: cleanvar: Fix startup order.
Mon, Dec 8, 6:39 AM
imp added a comment to D54117: reboot: Default to a clean shutdown.
In D54117#1236396, @des wrote:
In D54117#1236374, @imp wrote:

This is what I've wanted to do for some time now. I didn't look closely enough, but so long as fastboot gives us the current behavior, (aka the -f behavior), I'm good with all this.

That is not currently the case, but that's fixable.

Mon, Dec 8, 12:29 AM
imp accepted D54117: reboot: Default to a clean shutdown.

This is what I've wanted to do for some time now. I didn't look closely enough, but so long as fastboot gives us the current behavior, (aka the -f behavior), I'm good with all this.

Mon, Dec 8, 12:12 AM

Sun, Dec 7

imp added a comment to D52011: depend-cleanup: Force a clean build when options change.

Offline we talked. I'm totally cool with whatever so long as I can override the forced clean.

Sun, Dec 7, 5:25 PM
imp requested changes to D52011: depend-cleanup: Force a clean build when options change.

So I need to make sure there's not a blast radius on this one.

Sun, Dec 7, 4:22 PM

Fri, Dec 5

imp updated the diff for D54095: nvme: remove now-redundant consumer interface.

Update per review and rebase over resize changes... woof.

Fri, Dec 5, 11:29 PM
imp updated the diff for D51391: nvme: Notify failure with newbus call.

Update per review and rebase over resize changes... woof.

Fri, Dec 5, 11:28 PM
imp updated the diff for D51390: nvme: Use new method to do async notifications.

Update per review and rebase over resize changes... woof.

Fri, Dec 5, 11:28 PM
imp updated the diff for D51388: nvd: Connect nvme_if methods.

Update per review and rebase over resize changes... woof.

Fri, Dec 5, 11:28 PM
imp updated the diff for D51387: nvme_sim: Connect to events broadcast with nvme_if.

Update per review and rebase over resize changes... woof.

Fri, Dec 5, 11:28 PM
imp updated the diff for D51385: nvd: Attach as a child of nvme.

Update per review and rebase over resize changes... woof.

Fri, Dec 5, 11:28 PM
imp updated the diff for D51384: nvme_sim: Attach as a child of nvme.

Update per review and rebase over resize changes... woof.

Fri, Dec 5, 11:27 PM
imp updated the diff for D51383: nvme: Add child device for each controller.

Update per review and rebase over resize changes... woof.

Fri, Dec 5, 11:27 PM
imp updated the diff for D51386: nvme: Nvme controller generated events.

Update per review and rebase over resize changes... woof.

Fri, Dec 5, 11:27 PM
imp added reviewers for D54095: nvme: remove now-redundant consumer interface: jhb, mav, chs, chuck.
Fri, Dec 5, 11:27 PM
imp requested review of D54095: nvme: remove now-redundant consumer interface.
Fri, Dec 5, 11:24 PM
imp committed rG10efa7c30209: nvme: Rename NVME_NS_ADDED to NVME_NS_ALIVE and _CHANGED to _DELTA (authored by imp).
nvme: Rename NVME_NS_ADDED to NVME_NS_ALIVE and _CHANGED to _DELTA
Fri, Dec 5, 11:19 PM
imp committed rGf5b812331db5: nvme: Improve error message for invalid format value (authored by imp).
nvme: Improve error message for invalid format value
Fri, Dec 5, 11:19 PM
imp closed D51393: nvme: Improve error message for invalid format value.
Fri, Dec 5, 11:18 PM
imp accepted D54093: libkern: Avoid a one-byte OOB access in strndup().
Fri, Dec 5, 9:41 PM
imp committed rGc3ea7bfc72b7: cam: set reladr in scsi_read_capacity_16 (authored by imp).
cam: set reladr in scsi_read_capacity_16
Fri, Dec 5, 7:20 AM

Thu, Dec 4

imp accepted D54024: git-arc: Add patch -b option.
Thu, Dec 4, 5:48 PM
imp committed rGa8b0208e7e98: nvme: Fix devd logging of SMART_ERROR (authored by imp).
nvme: Fix devd logging of SMART_ERROR
Thu, Dec 4, 6:16 AM
imp added inline comments to D53950: Add libxo support to `gpart show` and document libxo flags in geom and gpart man pages.
Thu, Dec 4, 12:19 AM

Wed, Dec 3

imp accepted D53897: xhci: Add header values for Debug Capability (DbC).
Wed, Dec 3, 8:22 PM
imp added inline comments to D52303: build: Use language-agnostic (ISO 8601) timestamp format.
Wed, Dec 3, 8:10 PM
imp added a comment to D54041: stddef.h: hide from C++.

ISO_C_VISIBLE: Interfaces available. It's set unconditionally based on the dialect. It's an undocumented interface.
STDC_VERSION: the exact dialect being compiled for. This is standards defined.

Wed, Dec 3, 7:57 PM
imp committed rGb033e3684fd1: armv7: Tweak some armv6 removals (authored by imp).
armv7: Tweak some armv6 removals
Wed, Dec 3, 5:34 AM
imp committed rG7b71f57f4e51: netinet: Remove left-over sys/cdefs.h (authored by imp).
netinet: Remove left-over sys/cdefs.h
Wed, Dec 3, 5:05 AM
imp added reviewers for D54043: headers: Remove redundant checks for __BSD_VISIBLE: jrtc27, fuz.
Wed, Dec 3, 4:47 AM
imp requested review of D54043: headers: Remove redundant checks for __BSD_VISIBLE.
Wed, Dec 3, 4:46 AM
imp committed rG618347747494: kqueue(9): document f_copy (authored by imp).
kqueue(9): document f_copy
Wed, Dec 3, 4:41 AM
imp closed D53845: kqueue(9): document f_copy.
Wed, Dec 3, 4:41 AM
imp accepted D54041: stddef.h: hide from C++.
Wed, Dec 3, 4:19 AM
imp accepted D54003: ufshci: Support UIC hibernation enter/exit.
Wed, Dec 3, 12:45 AM
imp accepted D54004: ufshci: Support UIC Auto Hibernation.
Wed, Dec 3, 12:45 AM
imp accepted D54002: ufshci: Support suspend/resume.
Wed, Dec 3, 12:44 AM

Tue, Dec 2

imp added a comment to D54037: PR 291273 - p9fs module missing symbol exports -- dependent modules fail to load with module loader local symbol resolution disabled.

Why can't we make virtio_p9fs.ko depend on p9fs.ko? That would also cover this case, no? And would confine their visibility to that one module, no? It's how we do pci, etc.

Tue, Dec 2, 10:00 PM
imp added a comment to D54024: git-arc: Add patch -b option.

Is it possible to create a branch for the whole chain somehow?

Tue, Dec 2, 8:41 PM