Skip to content

Conversation

@Arun-Prasad-V
Copy link
Contributor

Tracked by RSDSO-19755

@Arun-Prasad-V
Copy link
Contributor Author

Jammy patches working. Tested with 585.
Noble patches are not getting applied. Debugging it.

@Arun-Prasad-V Arun-Prasad-V requested review from Nir-Az and dmipx August 6, 2024 04:32
@Nir-Az
Copy link
Collaborator

Nir-Az commented Aug 6, 2024

@dmipx please review and help Arun understand why it doesn’t work.
@Arun-Prasad-V please attach dmesg output on the problematic formats

# As a precausion start with unloading the core uvcvideo:
try_unload_module uvcvideo
try_unload_module videobuf2_v4l2
[ ${k_maj_min} -ge 608 ] && try_unload_module videobuf2_memops
Copy link
Contributor Author

@Arun-Prasad-V Arun-Prasad-V Aug 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For some reason, "videobuf2_memops" is not getting removed. So removing manually.

Reference output:

$ lsmod | grep video
videobuf2_memops       16384  0
videobuf2_common       81920  1 videobuf2_memops
mc                     81920  1 videobuf2_common
video                  73728  1 nouveau
wmi                    28672  6 video,intel_wmi_thunderbolt,gigabyte_wmi,wmi_bmof,mxm_wmi,nouveau


# Get the required tools and headers to build the kernel
sudo apt-get install linux-headers-generic linux-headers-$LINUX_BRANCH build-essential git bc -y
sudo apt-get install linux-headers-$LINUX_BRANCH build-essential git bc -y
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed installing linux-headers-generic. Since, it is in development, new versions are released regularly and it leads to installing a kernel version greater than the local kernel version. For example, default kernel for U24 is 6.8.0-31 and this command installs 6.8.0-40.

@Arun-Prasad-V Arun-Prasad-V marked this pull request as ready for review August 13, 2024 11:30
@Arun-Prasad-V Arun-Prasad-V requested a review from Nir-Az August 13, 2024 11:30
Copy link
Collaborator

@Nir-Az Nir-Az left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work Arun!
@dmipx can you also review please?
Thanks

@dmipx
Copy link

dmipx commented Aug 18, 2024

I suggest to verify on one of our NUC setups. They have specific default media setup with loaded sensor driver which conflicts on patched modules loading.

@Nir-Az
Copy link
Collaborator

Nir-Az commented Aug 28, 2024

I suggest to verify on one of our NUC setups. They have specific default media setup with loaded sensor driver which conflicts on patched modules loading.

So you recommend updating the kernel on one of our U22 NUCs?
I believe it was tested on U24 by Arun.

@Arun-Prasad-V Arun-Prasad-V marked this pull request as draft September 3, 2024 04:22
@Arun-Prasad-V Arun-Prasad-V force-pushed the u24_kernel_6.8 branch 2 times, most recently from fc314cd to 73779d6 Compare September 3, 2024 10:30
@Arun-Prasad-V Arun-Prasad-V marked this pull request as ready for review September 3, 2024 10:31
@Arun-Prasad-V Arun-Prasad-V requested a review from Nir-Az September 3, 2024 10:31
@Nir-Az
Copy link
Collaborator

Nir-Az commented Sep 3, 2024

@Arun-Prasad-V looks like a rebase is required to fix CI issues

@Nir-Az Nir-Az merged commit 59bce82 into IntelRealSense:development Sep 9, 2024
@andrejpan
Copy link

I tested the developed branch on Ubuntu 22.04, and it works fine; thanks for the fix!

@betelgeuse009
Copy link

Manual patching from development branch failed with Ubuntu 22.04.01 , kernel release 6.8.0-49-generic . Error code

Patched kernels modules were created successfully

Replacing uvc  -
	Applying the patched module ... modprobe: ERROR: could not insert 'uvc': Key was rejected by service
Failed to insert the patched module. Operation is aborted, the original module is restored
Verify that the current kernel version is aligned to the patched module version
The original  uvc  module was reloaded

@betelgeuse009
Copy link

Edit: problem solved after disabling secure boot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants