Hi Mauricio, Thank you for the update. The test plan has been revised accordingly.
** Description changed: [Impact] The shell's read builtin iterates through /proc/mounts one line at a time. This becomes problematic when LDAP automount maps generate a large number of entries in /proc/mounts. It can lead to timeout issues, especially when iterating through the entries twice in the cryptroot hook. [Fix] Applying the following upstream commit [1] can resolve this issue. 95fd4be9b4c6 d/functions: get_mnt_devno(): Speed up execution time on large /proc/mounts. Use awk rather than a `while read; do done` loop here as the /proc/mounts pseudo-file can be many thousands lines long and the shell's `read` builtin traverses it one read(2) at the time which cruelly slows down execution time. See https://salsa.debian.org/cryptsetup- team/cryptsetup/-/merge_requests/36 . [1] https://salsa.debian.org/cryptsetup- team/cryptsetup/-/commit/95fd4be9b4c6471e94c418101e7acfae7e1aa4fc [Test Plan] - (SRU team: pending changes in comment #25) - Our primary objective is to ensure that the output remains consistent when obtaining devnos through iteration. - 1. Execute the /usr/share/initramfs-tools/hooks/cryptroot binary - 2. Confirm that the output does not include the following warning message: + 1. Prepare a VM with its root partition encrypted using LUKS. + 2. Save the content of /etc/crypttab for reference. + 3. Install the patched package and execute the binary located at /usr/share/initramfs-tools/hooks/cryptroot. + 4. Verify that the /etc/crypttab content remains unchanged and ensure the output does not include the following warning message: cryptsetup: WARNING: Couldn't determine root device + + The performance improvement test is outlined as follows: + root@jammy-ptp:~# mkdir src dst + root@jammy-ptp:~# for i in {1..5000}; do touch src/test_${i} dst/test_${i}; mount --bind src/test_${i} dst/test_${i}; done + root@jammy-ptp:~# wc -l /proc/mounts + 5028 /proc/mounts + + [Before] + root@jammy-ptp:~# time /usr/share/initramfs-tools/hooks/cryptroot + real 0m1.415s + user 0m0.975s + sys 0m0.529s + + [After] + root@jammy-ptp:~/cryptsetup# time /usr/share/initramfs-tools/hooks/cryptroot + real 0m0.129s + user 0m0.098s + sys 0m0.037s [Where problems could occur] The patch exclusively modifies the method of extracting information without altering the underlying hook logic. It's crucial to note that the successful generation of the crypttab is contingent upon the accuracy of the information provided by the patch. Any inaccuracies may impede the crypttab generation process. [Other Info] The proposed change [1] is already applied in Oracular and Plucky. ** Changed in: cryptsetup (Ubuntu Noble) Status: Incomplete => In Progress -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/2054390 Title: Refine proc mounts entries traversal To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/cryptsetup/+bug/2054390/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
