Skip to content

modules/partitioning: remove xcp workaround#1593

Merged
brianmcgillion merged 1 commit intomainfrom
disko-xcp
Nov 27, 2025
Merged

modules/partitioning: remove xcp workaround#1593
brianmcgillion merged 1 commit intomainfrom
disko-xcp

Conversation

@Mic92
Copy link
Copy Markdown
Contributor

@Mic92 Mic92 commented Nov 27, 2025

Disko now uses parallel cp instead of xcp, which hopefully will the errors we see with virtiofs. Upstream disko pr: nix-community/disko#1166

Description of Changes

Type of Change

  • New Feature
  • Bug Fix
  • Improvement / Refactor

Related Issues / Tickets

Checklist

  • Clear summary in PR description
  • Detailed and meaningful commit message(s)
  • Commits are logically organized and squashed if appropriate
  • Contribution guidelines followed
  • Ghaf documentation updated with the commit - https://tiiuae.github.io/ghaf/
  • Author has run make-checks and it passes
  • All automatic GitHub Action checks pass - see actions
  • Author has added reviewers and removed PR draft status

Testing Instructions

Applicable Targets

  • Orin AGX aarch64
  • Orin NX aarch64
  • Lenovo X1 x86_64
  • Dell Latitude x86_64
  • System 76 x86_64

Installation Method

  • Requires full re-installation
  • Can be updated with nixos-rebuild ... switch
  • Other:

Test Steps To Verify:

  1. ...

Disko now uses parallel cp instead of xcp, which hopefully will not show
the file descriptor exhaustion issue with virtiofs.

Signed-off-by: Jörg Thalheim <joerg@thalheim.io>
@Mic92
Copy link
Copy Markdown
Contributor Author

Mic92 commented Nov 27, 2025

If this works for some days, I will merge the PR in disko as well.

@brianmcgillion
Copy link
Copy Markdown
Collaborator

will merge it as the only way to get visability is if the PRs are utilizing it.

@brianmcgillion brianmcgillion merged commit 9f9b78b into main Nov 27, 2025
28 checks passed
@brianmcgillion brianmcgillion deleted the disko-xcp branch November 27, 2025 13:32
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.

2 participants