Page MenuHomeFreeBSD

aokblast (ShengYi Hung)
AoK_Blast

Projects

User Details

User Since
Jun 21 2022, 7:17 PM (205 w, 2 h)

Recent Activity

Today

aokblast added a comment to D55603: amdrapl: Introduce amdrapl driver.

Hi @aokblast, I've found your rapl driver for amd and I'm interested in continuing the work. It's been sitting since March and I'd like to pick it up. Are you still planning to work on it, or would you be OK with me taking over? Thanks

Tue, May 26, 9:44 AM

Yesterday

aokblast added a comment to D57194: usb: Add SSP and SSPx2 speed and move USB_*_MAX into enum.
In D57194#1311193, @bz wrote:

Fantasic! You are solving one of my local TODOs :)

Can you make all these USB changes belonging together a stack in Phabricator?
It's a bit hard to grasp which all are there or belong together.
I'd be happy to at least scroll through them even if I cannot review them all in detail most likely.

% git show 9dbb804f59533d7d539200367394e4013714b02c
commit 9dbb804f59533d7d539200367394e4013714b02c
Author:     Bjoern A. Zeeb <bz@FreeBSD.org>
AuthorDate: Mon Feb 9 01:11:41 2026 +0000
Commit:     Bjoern A. Zeeb <bz@FreeBSD.org>
CommitDate: Sun May 24 15:53:58 2026 +0000

    TODO usb: add USB_SPEED_SUPER_PLUS to enum usb_dev_speed
    
    While nothing in our native USB stack uses the USB 3.1 "SUPERSPEED+"
    setting yet, a LinuxKPI based wireless driver does check for it.
    
    TODO: The arrays in usbd_get_std_packet_size() likely need to be updated.
    
    Sponsored by:   The FreeBSD Foundation
    MFC after:      3 days

diff --git sys/dev/usb/usb.h sys/dev/usb/usb.h
index 3e972f876c6a..f07eb9adfe40 100644
--- sys/dev/usb/usb.h
+++ sys/dev/usb/usb.h
@@ -755,8 +755,9 @@ enum usb_dev_speed {
        USB_SPEED_FULL,
        USB_SPEED_HIGH,
        USB_SPEED_SUPER,
+       USB_SPEED_SUPER_PLUS,
 };
-#define        USB_SPEED_MAX   (USB_SPEED_SUPER+1)
+#define        USB_SPEED_MAX   (USB_SPEED_SUPER_PLUS+1)
 
 /*
  * The "USB_REV" macros defines all the supported USB revisions.
Mon, May 25, 7:24 AM · USB

Sun, May 24

aokblast added a comment to D56970: usb: Refactor usb_proc_msg.

So I'm starting to understand what you're trying to do here, however just a few things:

  • this code gets used or was used in some bootloader builds, right? So we need to make sure it runs in its standalone mode versus in the kernel
Sun, May 24, 4:19 PM · USB

Sat, May 23

aokblast updated the diff for D51225: libusb: implement libusb_get_max_alt_packet_size.

Minor fixes

Sat, May 23, 4:29 PM
aokblast updated the diff for D50958: libusb: add parser for interface association descriptor.

Minor fixes

Sat, May 23, 4:18 PM
aokblast updated the diff for D51224: libusb: refactor the process of setting option when init the context..

Minor fixes

Sat, May 23, 4:16 PM
aokblast updated the diff for D51224: libusb: refactor the process of setting option when init the context..

Minor fixes

Sat, May 23, 4:11 PM
aokblast updated the diff for D50818: libusb: implement libusb_set_option.

FIxes

Sat, May 23, 3:58 PM
aokblast added inline comments to D46193: sys: implement Kernel CFI from clang.
Sat, May 23, 3:56 PM
aokblast added inline comments to D46193: sys: implement Kernel CFI from clang.
Sat, May 23, 3:55 PM
aokblast added inline comments to D46193: sys: implement Kernel CFI from clang.
Sat, May 23, 3:08 PM
aokblast updated the diff for D46193: sys: implement Kernel CFI from clang.

Rebase to main and use safe_read

Sat, May 23, 3:08 PM
aokblast requested review of D57208: kern: Implement all dead and no prefixed functions in kern_conf.
Sat, May 23, 3:07 PM
aokblast requested review of D57207: random: implement ra_pre_read for fenestrasX.
Sat, May 23, 3:06 PM
aokblast updated the diff for D49167: dtrace: fix signature mismatch for dtrace default handler.

Rebase to main

Sat, May 23, 3:03 PM
aokblast updated the diff for D49170: kcfi: add _NOCFI in kernel.

Repalace _NOCFI with __nosanitizerkcfi

Sat, May 23, 3:00 PM
aokblast updated the diff for D49113: sys/tools: add -s options for KCFI use.

Rebase to main and reflect markj@'s comment

Sat, May 23, 2:56 PM
aokblast added a comment to D49168: null.c: fix signature mismatch for null_read.

Will merge this in next tomorrow.

Sat, May 23, 2:54 PM
aokblast updated the diff for D49168: null.c: fix signature mismatch for null_read.

Rebase to main

Sat, May 23, 2:54 PM
aokblast added a comment to D49166: vfs_subr.c: fix signature mismatch for close method.

@jhb Since it is really trivial. I will merge this next Tuesday if you don't reply.

Sat, May 23, 2:48 PM
aokblast updated the diff for D49166: vfs_subr.c: fix signature mismatch for close method.

Rebase to main

Sat, May 23, 2:47 PM
aokblast added a comment to D48241: usr.sbin/bhyve: add rendezvous layer for initialization step of bhyve.

bhyve wait mode fails to run with or without this path. I am not sure what is the problem...

Sat, May 23, 1:19 PM
aokblast updated the diff for D48241: usr.sbin/bhyve: add rendezvous layer for initialization step of bhyve.

Rebase to main

Sat, May 23, 1:19 PM
aokblast added a comment to D52347: usb: support SSP and SSPx2 xHCI.

Just split the patch with multiple smaller one.

Sat, May 23, 10:17 AM · USB
aokblast retitled D52347: usb: support SSP and SSPx2 xHCI from usb: support usb 3.1 (SuperSpeedPlus) and usb 3.2 (SuperSpeedPlus_x2) addressing and display for xhci. to usb: support SSP and SSPx2 xHCI.
Sat, May 23, 10:16 AM · USB
aokblast updated the diff for D52347: usb: support SSP and SSPx2 xHCI.

Split patches

Sat, May 23, 10:16 AM · USB
aokblast added reviewers for D57204: usb: Add support for SSP and SSPx2 hubs: USB, adrian.
Sat, May 23, 10:14 AM · USB
aokblast requested review of D57204: usb: Add support for SSP and SSPx2 hubs.
Sat, May 23, 10:14 AM · USB
aokblast added reviewers for D57203: usb: Add SSPlus device capability descriptor: adrian, USB.
Sat, May 23, 10:13 AM · USB
aokblast requested review of D57203: usb: Add SSPlus device capability descriptor.
Sat, May 23, 10:13 AM · USB
aokblast added reviewers for D57202: usb: Implement usbd_req_get_bos_descriptor helper: USB, adrian.
Sat, May 23, 10:12 AM · USB
aokblast requested review of D57202: usb: Implement usbd_req_get_bos_descriptor helper.
Sat, May 23, 10:12 AM · USB
aokblast added reviewers for D57201: usb: Add get_ext_port_status helper function: adrian, USB.
Sat, May 23, 10:11 AM · USB
aokblast requested review of D57201: usb: Add get_ext_port_status helper function.
Sat, May 23, 10:11 AM · USB
aokblast added reviewers for D57200: usb: Add extended port status struct: USB, adrian.
Sat, May 23, 10:10 AM · USB
aokblast requested review of D57200: usb: Add extended port status struct.
Sat, May 23, 10:10 AM · USB
aokblast added reviewers for D57199: usb: Add usb_bos_foreach helper: USB, adrian.
Sat, May 23, 10:09 AM · USB
aokblast requested review of D57199: usb: Add usb_bos_foreach helper.
Sat, May 23, 10:09 AM · USB
aokblast added reviewers for D57198: usb: support SSP/SSPx2 devices in all-speed consumers: USB, adrian.
Sat, May 23, 10:08 AM · USB
aokblast requested review of D57198: usb: support SSP/SSPx2 devices in all-speed consumers.
Sat, May 23, 10:08 AM · USB
aokblast added reviewers for D57197: usbdump: Support SSP and SSPx2 speed display: adrian, USB.
Sat, May 23, 10:07 AM · USB
aokblast requested review of D57197: usbdump: Support SSP and SSPx2 speed display.
Sat, May 23, 10:07 AM · USB
aokblast added reviewers for D57196: usbconfig: Add SSPx2 support: USB, adrian.
Sat, May 23, 10:06 AM · USB
aokblast requested review of D57196: usbconfig: Add SSPx2 support.
Sat, May 23, 10:05 AM · USB
aokblast added reviewers for D57195: libusb: Add SSP and SSPx2 enum support: USB, adrian.
Sat, May 23, 10:04 AM · USB
aokblast requested review of D57195: libusb: Add SSP and SSPx2 enum support.
Sat, May 23, 10:04 AM · USB
aokblast added reviewers for D57194: usb: Add SSP and SSPx2 speed and move USB_*_MAX into enum: USB, adrian.
Sat, May 23, 10:03 AM · USB
aokblast requested review of D57194: usb: Add SSP and SSPx2 speed and move USB_*_MAX into enum.
Sat, May 23, 10:02 AM · USB
aokblast updated the diff for D56970: usb: Refactor usb_proc_msg.

Fixes

Sat, May 23, 7:52 AM · USB
aokblast added inline comments to D56970: usb: Refactor usb_proc_msg.
Sat, May 23, 7:41 AM · USB

Fri, May 22

aokblast added a comment to D56970: usb: Refactor usb_proc_msg.

Sorry, on the list to give this a closer look today- it looked nice on the surface and cleans up one annoyance I've had in the past with how this stuff works, but I want to be sure that we don't accidentally regress the serial bits.

Fri, May 22, 2:39 PM · USB
aokblast added a project to D56891: usb_hub: Trigger only chaged port on enumeration: USB.
Fri, May 22, 7:45 AM · USB
aokblast added a project to D56970: usb: Refactor usb_proc_msg: USB.
Fri, May 22, 7:45 AM · USB
aokblast committed rG28d85db46b48: xhci: Do not drop and add bits in xhci (authored by aokblast).
xhci: Do not drop and add bits in xhci
Fri, May 22, 7:42 AM
aokblast closed D57146: xhci: Do not drop and add bits in xhci.
Fri, May 22, 7:41 AM · USB

Thu, May 21

aokblast added a comment to D57146: xhci: Do not drop and add bits in xhci.

This looks fine, is there somewhere in the XHCI specification you can reference for drop bit handling? It'd be good to have a comment there.

Nice catch!

Thu, May 21, 2:47 PM · USB
aokblast added a reviewer for D57146: xhci: Do not drop and add bits in xhci: tomek_cedro.info.
Thu, May 21, 2:02 PM · USB
aokblast added a comment to D52166: bhyve: implement single USB device passthrough support.
In D52166#1273856, @bz wrote:

Woohoo. Happy to see updates here.

I've rebased my main once in between; while my smart card reader still works on Win11 after passthru one of my licensing dongles no longer does. I'll grab an updated stack from here the next days and rebuild to see. I haven't had time to debug it anyway so can as well do it on the latest revision. Thanks for all this work!

Thu, May 21, 1:52 PM
aokblast added a reviewer for D57146: xhci: Do not drop and add bits in xhci: adrian.
Thu, May 21, 1:43 PM · USB
aokblast requested review of D57146: xhci: Do not drop and add bits in xhci.
Thu, May 21, 1:42 PM · USB
aokblast updated the diff for D57130: xhci: Refactor xhci_setup_generic code.
  • xhci: Refactor xhci_generic_setup code
  • AI fixed
  • AI fix: 2
Thu, May 21, 7:30 AM
aokblast updated the diff for D56970: usb: Refactor usb_proc_msg.
  • fix by me
Thu, May 21, 6:44 AM · USB
aokblast updated the diff for D56970: usb: Refactor usb_proc_msg.
  • draft:
  • fix: AI fix
  • fix by me
Thu, May 21, 6:18 AM · USB

Wed, May 20

aokblast added reviewers for D57130: xhci: Refactor xhci_setup_generic code: USB, adrian.
Wed, May 20, 4:13 PM
aokblast requested review of D57130: xhci: Refactor xhci_setup_generic code.
Wed, May 20, 4:13 PM

Tue, May 19

aokblast added inline comments to D57062: bhyve: Check rfb length.
Tue, May 19, 6:51 AM
aokblast updated the diff for D57062: bhyve: Check rfb length.

Remove size limit and relies on the frontend

Tue, May 19, 6:51 AM

Mon, May 18

aokblast added a comment to D57063: bhyve: Sync max {height,width} with pci_fbuf.c.
In D57063#1307650, @bz wrote:

What's the expected bandwidth needed for that with VNC? 30Mbit/s?

Mon, May 18, 4:30 PM
aokblast requested review of D57063: bhyve: Sync max {height,width} with pci_fbuf.c.
Mon, May 18, 7:21 AM
aokblast added a reviewer for D57062: bhyve: Check rfb length: markj.
Mon, May 18, 7:17 AM
aokblast requested review of D57062: bhyve: Check rfb length.
Mon, May 18, 7:17 AM

Fri, May 15

aokblast committed rG776584319fb4: sound: Fix lock order reversal in dsp_poll() (authored by aokblast).
sound: Fix lock order reversal in dsp_poll()
Fri, May 15, 2:53 PM
aokblast closed D57009: dsp: Make witness happy.
Fri, May 15, 2:53 PM
aokblast updated the summary of D57009: dsp: Make witness happy.
Fri, May 15, 2:03 PM
aokblast updated the diff for D57009: dsp: Make witness happy.

Elaborate commit msg again

Fri, May 15, 2:03 PM
aokblast added a comment to D57009: dsp: Make witness happy.

I understand that the change here locks the channels we actually want, instead of both of them. However, the effect is the same; if priv has both channels allocated, then your code will also lock both of them, just not at the same time. But I don't understand why doing this fixes the issue. Could you elaborate further?

Fri, May 15, 1:50 PM
aokblast added a comment to D57009: dsp: Make witness happy.

Please explain in a commit message what this change does. What does WITNESS complain about? Why are you changing the order of dsp_lock_chans()? And why did you remove the dsp_lock_chans() call in dsp_poll()?

Fri, May 15, 1:12 PM
aokblast updated the summary of D57009: dsp: Make witness happy.
Fri, May 15, 1:11 PM
aokblast updated the diff for D57009: dsp: Make witness happy.

Elaborate on commit msg

Fri, May 15, 1:11 PM
aokblast added a reviewer for D57009: dsp: Make witness happy: christos.
Fri, May 15, 8:45 AM
aokblast requested review of D57009: dsp: Make witness happy.
Fri, May 15, 8:44 AM

Thu, May 14

aokblast updated the summary of D56970: usb: Refactor usb_proc_msg.
Thu, May 14, 9:21 AM · USB
aokblast updated the diff for D56970: usb: Refactor usb_proc_msg.

Refactor commit msg

Thu, May 14, 9:20 AM · USB
aokblast updated the diff for D56970: usb: Refactor usb_proc_msg.

Some lock and unlock stuff

Thu, May 14, 9:02 AM · USB

Tue, May 12

aokblast updated the diff for D56970: usb: Refactor usb_proc_msg.

Fix shutdown order

Tue, May 12, 2:51 PM · USB
aokblast requested review of D56970: usb: Refactor usb_proc_msg.
Tue, May 12, 2:30 PM · USB

Sun, May 10

aokblast added a reviewer for D56921: sccache: Update to 0.15.0: pizzamig.
Sun, May 10, 1:31 PM
aokblast requested review of D56921: sccache: Update to 0.15.0.
Sun, May 10, 1:30 PM

Sat, May 9

aokblast committed rG555ffd9022de: pci: use uint32_t for eecp (authored by Gisle Nes <gisle@gisle.net>).
pci: use uint32_t for eecp
Sat, May 9, 9:10 AM
aokblast committed rG072f79d0a287: hwpstate_amd: Initialize CPPC driver type (authored by laurent).
hwpstate_amd: Initialize CPPC driver type
Sat, May 9, 8:39 AM
aokblast added a comment to D55592: acpi: Add acpi_fan.

Just received panic: mtx_lock() of spin mutex Fan mutex @ ... upon sysctl -a.
I may not have all updates from this review in place.

Sat, May 9, 8:28 AM
aokblast updated the diff for D55592: acpi: Add acpi_fan.

Add missing mtx_unlock

Sat, May 9, 8:28 AM

Fri, May 8

aokblast requested review of D56891: usb_hub: Trigger only chaged port on enumeration.
Fri, May 8, 5:15 PM · USB

Tue, May 5

aokblast added a reverting change for rGce9ced951a0b: libusb: change callback register handler to int: rGbf2c6d23ad09: Revert "libusb: change callback register handler to int".
Tue, May 5, 4:49 PM
aokblast committed rGbf2c6d23ad09: Revert "libusb: change callback register handler to int" (authored by aokblast).
Revert "libusb: change callback register handler to int"
Tue, May 5, 4:49 PM
aokblast added a reverting change for D54211: libusb: change callback register handler to int: rGbf2c6d23ad09: Revert "libusb: change callback register handler to int".
Tue, May 5, 4:49 PM
aokblast committed rG06271366982e: libusb: fix incorrect status type when completion on cancel the events (authored by aokblast).
libusb: fix incorrect status type when completion on cancel the events
Tue, May 5, 4:00 PM
aokblast closed D51736: libusb: fix incorrect status type when completion on cancel the events.
Tue, May 5, 3:59 PM
aokblast committed rGce9ced951a0b: libusb: change callback register handler to int (authored by aokblast).
libusb: change callback register handler to int
Tue, May 5, 3:49 PM
aokblast closed D54211: libusb: change callback register handler to int.
Tue, May 5, 3:49 PM