Skip to content

use squashfuse_ll if it is available.#354

Merged
rchincha merged 1 commit intoproject-stacker:mainfrom
smoser:feature/squashfuse-cleanups
Dec 13, 2022
Merged

use squashfuse_ll if it is available.#354
rchincha merged 1 commit intoproject-stacker:mainfrom
smoser:feature/squashfuse-cleanups

Conversation

@smoser
Copy link
Contributor

@smoser smoser commented Dec 8, 2022

feat: Use squashfuse_ll if available for faster squashfs.

squashfuse README says of squashfuse_ll:

Like squashfuse, but implemented using the low-level
FUSE API. It's a tiny bit faster, but less portable.

But some reports say things like "30x times faster."
canonical/snapd#7477

@smoser smoser force-pushed the feature/squashfuse-cleanups branch from d3b388c to 783d3a1 Compare December 9, 2022 14:36
@smoser smoser force-pushed the feature/squashfuse-cleanups branch from 783d3a1 to dbf0371 Compare December 13, 2022 15:08
@smoser smoser marked this pull request as ready for review December 13, 2022 15:10
@smoser smoser requested a review from rchincha as a code owner December 13, 2022 15:10
@smoser smoser force-pushed the feature/squashfuse-cleanups branch from dbf0371 to ef4d16f Compare December 13, 2022 15:14
squashfuse README says of squashfuse_ll:

   Like `squashfuse`, but implemented using the low-level
   FUSE API. It's a tiny bit faster, but less portable.

But some reports say things like "30x times faster."
canonical/snapd#7477

Signed-off-by: Scott Moser <scmoser@cisco.com>
@smoser smoser changed the title Feature/squashfuse cleanups use squashfuse_ll if it is available. Dec 13, 2022
@smoser smoser force-pushed the feature/squashfuse-cleanups branch from ef4d16f to 4afcd48 Compare December 13, 2022 15:15
@smoser
Copy link
Contributor Author

smoser commented Dec 13, 2022

I dropped the "real root kernel mount" support from this PR.

I think that is useful on its own, but lets do it on its own.

Copy link
Contributor

@rchincha rchincha left a comment

Choose a reason for hiding this comment

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

lgtm

@rchincha rchincha merged commit dd512cc into project-stacker:main Dec 13, 2022
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.

3 participants

Comments