After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 749867 - Some limits are adjusted by arcane cylinder size amount when copying and resizing in a single operation
Some limits are adjusted by arcane cylinder size amount when copying and resi...
Status: RESOLVED FIXED
Product: gparted
Classification: Other
Component: application
GIT HEAD
Other Linux
: Normal normal
: ---
Assigned To: gparted maintainers alias
gparted maintainers alias
Depends on:
Blocks:
 
 
Reported: 2015-05-25 20:47 UTC by Mike Fleetwood
Modified: 2015-08-03 17:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Remove cylinder sized adjustments to resize limits (v1) (8.96 KB, patch)
2015-05-26 08:25 UTC, Mike Fleetwood
none Details | Review

Description Mike Fleetwood 2015-05-25 20:47:11 UTC
There are some arcane adjustments when performing a combined copy into
unallocated space and resize that copy as a single operation.
1) XFS minimum size is plus 2 * cylinder size.
2) When the available space is larger than the maximum size of the file
   system, the limit is the file system maximum size minus cylinder
   size.
Get rid of these adjustments.

Patches to follow in a while.

Mike
Comment 1 Mike Fleetwood 2015-05-25 20:50:09 UTC
That first sentence should have said:

There are some arcane adjustments when performing a combined copy into
unallocated space and resize as a single operation.
Comment 2 Mike Fleetwood 2015-05-26 08:25:39 UTC
Created attachment 303971 [details] [review]
Remove cylinder sized adjustments to resize limits (v1)

Hi Curtis,

While banging my head against the complexity of trying to understand how
selected_partition member is used in Dialog_Base_Partition and
Dialog_Partition_{Copy,New,Resize_Move} I saw this bit of old crud and
decided I wanted an easy fix to do.

Thanks,
Mike
Comment 3 Curtis Gedak 2015-05-27 16:42:14 UTC
Hi Mike,

Thanks for the detailed patch set in comment #2.  I plan to review in-depth in the next few days.

One question -- in one of the comments, the abbreviation "icw" is used.  What does this represent?

For example,  icw some hacks in the other 2

Curtis
Comment 4 Mike Fleetwood 2015-05-27 18:25:41 UTC
"icw" is actually part of the quoted commit summary written by Bart back
in 2004.  I think in this context it stood for "In Connection With".

Mike
Comment 5 Curtis Gedak 2015-05-28 19:09:30 UTC
Thank you Mike for the clarification.  That Three Letter Acronym (TLA) makes sense in this situation.  :-)

I reviewed the patch set in comment #2 and tested resizing and copying of xfs and ntfs file systems on kubuntu 12.04.  All went well.

I made two small changes to the comment in the second patch.

1)  Change "and" to "an".
From:  This is probably and oversight
  To:  This is probably an oversight

2)  Remove duplicate "maximum".
From: Remove this lower maximum maximum file system size
  To: Remove this lower maximum file system size

The patch set in comment #2 has been committed to the git repository for inclusion in the next release of GParted.

The relevant git commits can be viewed at the following links:

Remove last trace of cylinder size adjustments in the resize dialog (#749867)
https://git.gnome.org/browse/gparted/commit/?id=b9262922a76f985c0502adfa49dc974b5ae52097

Remove cylinder size adjustments in the copy dialog (#749867)
https://git.gnome.org/browse/gparted/commit/?id=8bbb77f1f8c5d15ee2be9db6b69371452f7483d5
Comment 6 Curtis Gedak 2015-08-03 17:30:44 UTC
This enhancement was included in the GParted 0.23.0 release on August 3, 2015.