-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rebased and re-implemented USBIP configs #3983
Conversation
I can try it locally. But why are there so many changes in aarch64? I would have thought just the USBIP-related ones? |
I was wondering that too - I followed the exact same steps as I did for |
So I switched back to
I'd expect no changes but it introduced most of what we're seeing. Maybe something changed in |
I can confirm that amd64 on your branch is fine, and arm64 is fine on |
I can confirm that a change similar to x86_64 works. Here is my diff on the aarch64 file (working with USBIP vs master). If you apply this to the master and commit it here instead of yours, we should be able to push it all out and run it. diff --git a/kernel/config-6.6.x-aarch64 b/kernel/config-6.6.x-aarch64
index 947af3a06..d6561cc4a 100644
--- a/kernel/config-6.6.x-aarch64
+++ b/kernel/config-6.6.x-aarch64
@@ -3709,7 +3709,12 @@ CONFIG_USB_STORAGE=m
#
# CONFIG_USB_MDC800 is not set
# CONFIG_USB_MICROTEK is not set
-# CONFIG_USBIP_CORE is not set
+CONFIG_USBIP_CORE=m
+CONFIG_USBIP_VHCI_HCD=m
+CONFIG_USBIP_VHCI_HC_PORTS=8
+CONFIG_USBIP_VHCI_NR_HCS=1
+# CONFIG_USBIP_HOST is not set
+# CONFIG_USBIP_DEBUG is not set
#
# USB dual-mode controller drivers |
Okay! I also wanted to make sure that USB/IP doesn't enable any transitive dependencies, so I verified that against main. Now with the manual changes I still can't build when I run |
I suspect that has more to do with an error in cross-compilation. They work fine on Switch this from draft to ready? |
Successfully built and pushed out. Mark this as ready and we can run CI. |
Signed-off-by: Jonathan Beri <jberi@golioth.io> Revert to master Signed-off-by: Jonathan Beri <jberi@golioth.io> Modify USBIP modules by hand Signed-off-by: Jonathan Beri <jberi@golioth.io>
Ready and squashed! |
CI running, let it rip! |
Thanks for the patience as we worked this out. I pushed out the images, so download it and check that it all works as you expect? |
Verified with QEMU! Planning on testing in Virtualization Framework next, but QEMU is enough to prove it's there. Thank you for shepherding this through! |
Great! One of these lifetimes, we will get the kernel process part of the CI build. All packages used to be like that, so there s hope. |
Old PR autoclosed so created a new one.
amd64
built butarm64
is still having issues.Fixes #609