[Parted-maintainers] Bug#706023: parted: mkpart screws up value if start is 1 plus a unit

Len Sorensen lennartsorensen at ruggedcom.com
Thu Apr 25 15:55:34 UTC 2013


On Tue, Apr 23, 2013 at 04:15:41PM -0400, Phillip Susi wrote:
> It looks like this was fixed upstream in commit 18e727d4 - libparted:
> fix snap radius so that it is using half, which is a simple one line
> change:
> 
> diff --git a/libparted/unit.c b/libparted/unit.c
> index dc4205b..c2921e3 100644
> - --- a/libparted/unit.c
> +++ b/libparted/unit.c
> @@ -533,7 +533,7 @@ ped_unit_parse_custom (const char* str, const
> PedDevice* dev, PedUnit unit,
>         }
> 
>         unit_size = ped_unit_get_size (dev, unit);
> - -       radius = ped_div_round_up (unit_size, dev->sector_size) - 1;
> +       radius = (ped_div_round_up (unit_size, dev->sector_size) / 2) - 1;
>         if (radius < 0)
>                 radius = 0;
>         /* If the user specifies units in a power of 2, e.g., 4MiB, as in
> 
> 

And what are the chances of that patch making it into Debian any time
soon?  It sure wasted a lot of time wondering why it kept creating
partions wrong.  What a strange bug this was.

Hopefully the installer can get updated to handle something newer than
parted 2.3 for the next release after wheezy.

-- 
Len Sorensen



More information about the Parted-maintainers mailing list