Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to btrfs-progs 5.4.1 #151

Merged
merged 4 commits into from
Oct 26, 2020

Conversation

boretom
Copy link
Contributor

@boretom boretom commented Oct 26, 2020

Why to version 5.4.1: it's included in the Ubuntu Focal/Linux Mint 20 so it may be stable.

Also:

  • Update toolbox script to use rsync for btrfs-prog code sync. This since it involves subdirectories
  • Update .gitignore to include APFS binary src\partclone.apfs
  • Set file size for btrfs test since it failed because of a minimal size necessary

Build and tests run successfully on Linux Mint 20 (Ubuntu 20.04 Focal based)

The motivation was issue #327 on sourceforge which it didn't solve :)

Thomas Tsai and others added 4 commits October 25, 2020 21:14
Also:
- Update toolbox script to use rsync for btrfs-prog code sync. This since it involves subdirectories
- Update .gitignore to include APFS binary `src\partclone.apfs`
- Set file size for btrfs test since it failed because of a minimal size necessary
@Thomas-Tsai
Copy link
Owner

oops, we are doing the same things, I will roll back to 0.3.14 and merge your pull request first.

@boretom
Copy link
Contributor Author

boretom commented Oct 26, 2020

Sorry about the double-work-on @Thomas-Tsai. I just realized this morning that you updated master to 4.12 and thought I better hurry up so you wouldn't have to do the switch to 5. I was too late then.

@boretom boretom deleted the to-btrfs-progs-5.4.1 branch October 26, 2020 16:37
@boretom
Copy link
Contributor Author

boretom commented Oct 26, 2020

@Thomas-Tsai I also got a branch with the latest btrfs-progs v5.9, not sure how current you want to go. Works as well as 5.4.1 for me but that doesn't say too much.

@Thomas-Tsai
Copy link
Owner

Actually, I just upgrade btrfs-progs to some version, and not yet fishing, so we trully appreciate your contribution. I will test 5.4.1 first, and package it. The latest (v5.9) version could be merge and release in next version, so we can do more test.

Thank you!

@stevenshiau
Copy link
Contributor

I used this merged version Partclone 0.3.16 to take an image for Linux Mint 20 having BTRFS with compress=zstd:9 option, the image was taken successfully and later it was deployed to a VM successfully. Cool!

However, during the saving and restoring, the progress bar did not correctly shown, e.g.,
In the beginning, when the current blocks are about 10% done, no progress bar. The status is still in 1%.

When about 30% done, still no progress bar.

Once all done, the progress bar suddenly went to 100%.

BTW, this Clonezilla live version I created and test with Partclone 0.3.16 is here, if you want to give it a try:

https://stevenshiau.org/clonezilla-live/2.7.0-8/

Steven

@boretom
Copy link
Contributor Author

boretom commented Oct 28, 2020

I did a test with a VM with a compressed and an uncompressed btrfs partition and partclone.btrfs only.

This issue shows up with both of these partitions, doesn't depend on if compressed or not. I'll have a look into it, maybe I messed something up.

/Thomas K

@stevenshiau
Copy link
Contributor

stevenshiau commented Oct 28, 2020

OK, thanks.

I discussed with Thomas Tsai, and he told me the progress issue might be due to this patch:

#148

Steven

@boretom
Copy link
Contributor Author

boretom commented Oct 28, 2020

Ok I see that makes sense, I made a test with an ext2 partition and got the same issue with the progress bar.

I'll check if I can find the root cause for that.

/Thomas K

@boretom
Copy link
Contributor Author

boretom commented Oct 28, 2020

In the mentioned patch there is if ((stop - stop) != 0) { on line 46. I assume that should be if ((stop - start) != 0) { otherwise that's always a happy 0.

/Thomas K.

@stevenshiau
Copy link
Contributor

The weird thing is, only BTRFS has the progress issue. For other file system when running Clonezilla live 2.7.0-8, I do not see this issue.

Steven

@stevenshiau
Copy link
Contributor

I was wrong. Apparently the partition I tested was too small so it ran so fast and the 100% progress was shown. I confirmed that the progress issue exists for all file system, not only BTRFS.

Steven

@Thomas-Tsai
Copy link
Owner

Following the patch(#148), I add isnormal (commit 536bec)) to make sure float is not NaN and remove original (stop - stop) != 0, it looks fine with me.

@stevenshiau
Copy link
Contributor

Great. Thank you very much.

Steven

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.

None yet

4 participants