Re: [PATCH v2] dracut: dmsquash-live: overlay module can be built-in the kernel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Great, thanks. Should I do directly do a pull request next time?

On Tue, Aug 02, 2022 at 07:00:33PM -0400, Frederick Grose wrote:
How about this?
https://github.com/dracutdevs/dracut/pull/1886

On Tue, Aug 2, 2022 at 9:56 AM Federico Vaga <federico.vaga@xxxxxxx> wrote:

Hello,

I've never got a feedback on this patch. Will it be integrated?

Thanks

On Wed, May 18, 2022 at 11:47:07AM +0200, Federico Vaga wrote:
>The dmsquash-live module assumes that `overlay` is always a module.
>Therefore, when `overlay` is built-in the kernel, this module will fail
>to detect the presence of such a feature.
>
>This patch adds a fallback mechanism to check also `/proc/filesystem`.
>This is also what happens with the dracut NSF module when loading the
>kernel module `rpc_pipefs`.
>
>Signed-off-by: Federico Vaga <federico.vaga@xxxxxxx>
>---
> v1 -> v2: do not use unnecessary sub-shell in the if statement
>
> modules.d/90dmsquash-live/dmsquash-live-root.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/modules.d/90dmsquash-live/dmsquash-live-root.sh
b/modules.d/90dmsquash-live/dmsquash-live-root.sh
>index fd8a858a..f68a979c 100755
>--- a/modules.d/90dmsquash-live/dmsquash-live-root.sh
>+++ b/modules.d/90dmsquash-live/dmsquash-live-root.sh
>@@ -170,7 +170,7 @@ do_live_overlay() {
>         fi
>     fi
>     if [ -n "$overlayfs" ]; then
>-        if ! modprobe overlay; then
>+        if ! { modprobe overlay || strstr "$(cat /proc/filesystems)"
overlay; }; then
>             if [ "$overlayfs" = required ]; then
>                 die "OverlayFS is required but not available."
>                 exit 1
>--
>2.27.0
>

--
-------------------------------
Federico Vaga - CERN BE-CEM-EDL


--
-------------------------------
Federico Vaga - CERN BE-CEM-EDL



[Index of Archives]     [Linux Kernel]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux