File more-reliable-informing-the-kernel.patch of Package parted

Overview Repositories Revisions Requests Users Attributes Meta

File more-reliable-informing-the-kernel.patch of Package parted (Revision 4249f31b9b261a0e4fd3bb65261c66c3)

Currently displaying revision 4249f31b9b261a0e4fd3bb65261c66c3 , Show latest

From: Petr Uzel <petr.uzel@suse.cz>
Subject: parted: Make informing the kernel more reliable
References: bnc#657360
Patch-mainline: no

This patch should make parted's process of informing the kernel
about changes of partition table sligtly more reliable.
It consists of 2 quite unrelated parts:

* increase the max sleep time if partition is busy from 1 to 2 seconds
* do not inform the kernel when only partition flags change - kernel
  does not care about the flags
---
 libparted/arch/linux.c |    2 +-
 parted/parted.c        |   28 +++++++++++++++++++++++++++-
 2 files changed, 28 insertions(+), 2 deletions(-)

Index: parted-3.2/libparted/arch/linux.c
===================================================================
--- parted-3.2.orig/libparted/arch/linux.c
+++ parted-3.2/libparted/arch/linux.c
@@ -3011,7 +3011,7 @@ _disk_sync_part_table (PedDisk* disk)
                 /* Attempt to remove the partition, retrying for
                    up to max_sleep_seconds upon any failure due to EBUSY. */
                 unsigned int sleep_microseconds = 10000;
-                unsigned int max_sleep_seconds = 1;
+                unsigned int max_sleep_seconds = 2;
                 unsigned int n_sleep = (max_sleep_seconds
                                         * 1000000 / sleep_microseconds);
                 do {
Index: parted-3.2/parted/parted.c
===================================================================
--- parted-3.2.orig/parted/parted.c
+++ parted-3.2/parted/parted.c
@@ -266,6 +266,32 @@ _disk_warn_loss (PedDisk* disk)
                 disk->dev->path) == PED_EXCEPTION_YES;
 }
 
+/*
+ * Copied from ped_disk_commit(), but with removed the call to
+ * ped_disk_commit_to_os(). This is used in do_set() - BLKPG* ioctls() do not
+ * care about the partition flags anyway.
+ */
+static int
+ped_disk_commit_just_to_disk (PedDisk* disk)
+{
+        /* Open the device here, so that the underlying fd is not closed
+           between commit_to_dev and commit_to_os (closing causes unwanted
+           udev events to be sent under Linux). */
+	if (!ped_device_open (disk->dev))
+		goto error;
+
+	if (!ped_disk_commit_to_dev (disk))
+		goto error_close_dev;
+
+	ped_device_close (disk->dev);
+	return 1;
+
+error_close_dev:
+	ped_device_close (disk->dev);
+error:
+	return 0;
+}
+
 /* This function changes "sector" to "new_sector" if the new value lies
  * within the required range.
  */
@@ -1735,7 +1761,7 @@ do_set (PedDevice** dev, PedDisk **diskp
 
         if (!ped_partition_set_flag (part, flag, state))
                 goto error;
-        if (!ped_disk_commit (*diskp))
+        if (!ped_disk_commit_just_to_disk (*diskp))
                 goto error;
 
         if ((*dev)->type != PED_DEVICE_FILE)

Places

File more-reliable-informing-the-kernel.patch of Package parted (Revision 4249f31b9b261a0e4fd3bb65261c66c3)

Places