Blame SOURCES/0010-libparted-device-mapper-uses-512b-sectors.patch

003ee8
From f98f791e19669b900345dad7d96ea4df974e4596 Mon Sep 17 00:00:00 2001
003ee8
From: "Brian C. Lane" <bcl@redhat.com>
003ee8
Date: Wed, 4 Feb 2015 16:46:07 -0800
003ee8
Subject: [PATCH 10/11] libparted: device mapper uses 512b sectors
003ee8
003ee8
device mapper doesn't use the device's sector size when creating a
003ee8
table. It always uses 512b units. This causes partitions to be created
003ee8
8x smaller than expected on devices with 4906b sectors.
003ee8
---
003ee8
 NEWS                   |  4 ++++
003ee8
 libparted/arch/linux.c | 21 +++++++++++++++++----
003ee8
 2 files changed, 21 insertions(+), 4 deletions(-)
003ee8
003ee8
diff --git a/NEWS b/NEWS
003ee8
index 10c9a6e..96135ed 100644
003ee8
--- a/NEWS
003ee8
+++ b/NEWS
003ee8
@@ -4,6 +4,10 @@ GNU parted NEWS                                    -*- outline -*-
003ee8
 
003ee8
 ** Bug Fixes
003ee8
 
003ee8
+  Use 512b sector size when communicating with device-mapper. Fixes
003ee8
+  problems with partitions being created too small on dm devices
003ee8
+  with sector sizes > 5121b
003ee8
+
003ee8
   Don't crash in the disk_set command when a disk label is not found
003ee8
 
003ee8
   libparted-fs-resize: Prevent crash resizing FAT16 file systems.
003ee8
diff --git a/libparted/arch/linux.c b/libparted/arch/linux.c
003ee8
index 0f18904..90ab21d 100644
003ee8
--- a/libparted/arch/linux.c
003ee8
+++ b/libparted/arch/linux.c
003ee8
@@ -2763,6 +2763,12 @@ _dm_get_partition_start_and_length(PedPartition const *part,
003ee8
         if (sscanf (params, "%d:%d %Ld", &major, &minor, start) != 3)
003ee8
                 goto err;
003ee8
         rc = 1;
003ee8
+
003ee8
+        /* device-mapper uses 512b units, make sure we return length and start in terms of the device's
003ee8
+         * sector size.
003ee8
+         */
003ee8
+        *start /= (part->disk->dev->sector_size / PED_SECTOR_SIZE_DEFAULT);
003ee8
+        *length /= (part->disk->dev->sector_size / PED_SECTOR_SIZE_DEFAULT);
003ee8
 err:
003ee8
         free (path);
003ee8
         dm_task_destroy(task);
003ee8
@@ -2810,8 +2816,10 @@ _dm_add_partition (PedDisk* disk, const PedPartition* part)
003ee8
         /* Caution: dm_task_destroy frees dev_name.  */
003ee8
         dm_task_destroy (task);
003ee8
         task = NULL;
003ee8
+        /* device-mapper uses 512b units, not the device's sector size */
003ee8
         if ( ! (params = zasprintf ("%d:%d %lld", arch_specific->major,
003ee8
-                                    arch_specific->minor, part->geom.start)))
003ee8
+                                    arch_specific->minor,
003ee8
+                                    part->geom.start * (disk->dev->sector_size / PED_SECTOR_SIZE_DEFAULT))))
003ee8
                 goto err;
003ee8
 
003ee8
         task = dm_task_create (DM_DEVICE_CREATE);
003ee8
@@ -2821,7 +2829,8 @@ _dm_add_partition (PedDisk* disk, const PedPartition* part)
003ee8
         dm_task_set_name (task, vol_name);
003ee8
         if (vol_uuid)
003ee8
                 dm_task_set_uuid (task, vol_uuid);
003ee8
-        dm_task_add_target (task, 0, part->geom.length,
003ee8
+        /* device-mapper uses 512b units, not the device's sector size */
003ee8
+        dm_task_add_target (task, 0, part->geom.length * (disk->dev->sector_size / PED_SECTOR_SIZE_DEFAULT),
003ee8
                 "linear", params);
003ee8
         if (!dm_task_set_cookie (task, &cookie, 0))
003ee8
                 goto err;
003ee8
@@ -2878,8 +2887,11 @@ _dm_resize_partition (PedDisk* disk, const PedPartition* part)
003ee8
         /* Caution: dm_task_destroy frees dev_name.  */
003ee8
         dm_task_destroy (task);
003ee8
         task = NULL;
003ee8
+
003ee8
+        /* device-mapper uses 512b units, not the device's sector size */
003ee8
         if ( ! (params = zasprintf ("%d:%d %lld", arch_specific->major,
003ee8
-                                    arch_specific->minor, part->geom.start)))
003ee8
+                                    arch_specific->minor,
003ee8
+                                    part->geom.start * (disk->dev->sector_size / PED_SECTOR_SIZE_DEFAULT))))
003ee8
                 goto err;
003ee8
 
003ee8
         task = dm_task_create (DM_DEVICE_RELOAD);
003ee8
@@ -2887,7 +2899,8 @@ _dm_resize_partition (PedDisk* disk, const PedPartition* part)
003ee8
                 goto err;
003ee8
 
003ee8
         dm_task_set_name (task, vol_name);
003ee8
-        dm_task_add_target (task, 0, part->geom.length,
003ee8
+        /* device-mapper uses 512b units, not the device's sector size */
003ee8
+        dm_task_add_target (task, 0, part->geom.length * (disk->dev->sector_size / PED_SECTOR_SIZE_DEFAULT),
003ee8
                 "linear", params);
003ee8
         if (!dm_task_set_cookie (task, &cookie, 0))
003ee8
                 goto err;
003ee8
-- 
003ee8
2.1.0
003ee8