summaryrefslogtreecommitdiffstats
path: root/meta/recipes-extended/parted
diff options
context:
space:
mode:
authorOvidiu Panait <ovidiu.panait@windriver.com>2017-09-26 12:00:46 +0300
committerRichard Purdie <richard.purdie@linuxfoundation.org>2017-10-07 23:20:39 +0100
commita2201f192aa79bf8ebde54d764307325a3f0f09c (patch)
tree47526654188aebe0b75efe4893b5ee9b54439e33 /meta/recipes-extended/parted
parent59a238f3d75cbdd7a22e12445e440d5c29eaf6ee (diff)
downloadpoky-a2201f192aa79bf8ebde54d764307325a3f0f09c.tar.gz
libparted: Use read only when probing devices on linux
When a device is opened for RW closing it can trigger other actions, like udev scanning it for partition changes. Use read only for the init_* methods and RW for actual changes to the device. This adds _device_open which takes mode flags as an argument and turns linux_open into a wrapper for it with RW_MODE. _device_open_ro is added to open the device with RD_MODE and increment the open_counter. This is used in the init_* functions. _device_close is a wrapper around linux_close that decrements the open_counter and is used in the init_* functions. All of these changes are self-contained with no external API changes. The only visible change in behavior is that when a new PedDevice is created the device is opened in RO_MODE instead of RW_MODE. Resolves: rhbz#1245144 Reference: https://bugzilla.redhat.com/show_bug.cgi?id=1283112 Upstream patch: http://git.savannah.gnu.org/cgit/parted.git/commit/?id=44d5ae0115c4ecfe3158748309e9912c5aede92d (From OE-Core rev: f68fca4776a38a1bdf7ed37ce87cddbb21ff74f8) Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com> Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-extended/parted')
-rw-r--r--meta/recipes-extended/parted/files/0001-libparted-Use-read-only-when-probing-devices-on-linu.patch224
-rw-r--r--meta/recipes-extended/parted/parted_3.2.bb1
2 files changed, 225 insertions, 0 deletions
diff --git a/meta/recipes-extended/parted/files/0001-libparted-Use-read-only-when-probing-devices-on-linu.patch b/meta/recipes-extended/parted/files/0001-libparted-Use-read-only-when-probing-devices-on-linu.patch
new file mode 100644
index 0000000000..e522e1c6ef
--- /dev/null
+++ b/meta/recipes-extended/parted/files/0001-libparted-Use-read-only-when-probing-devices-on-linu.patch
@@ -0,0 +1,224 @@
1From d6e15a60e84c1511523aa81272b7db7a6ec441d0 Mon Sep 17 00:00:00 2001
2From: Ovidiu Panait <ovidiu.panait@windriver.com>
3Date: Tue, 26 Sep 2017 08:04:58 +0000
4Subject: [PATCH] libparted: Use read only when probing devices on linux
5 (#1245144)
6
7When a device is opened for RW closing it can trigger other actions,
8like udev scanning it for partition changes. Use read only for the
9init_* methods and RW for actual changes to the device.
10
11This adds _device_open which takes mode flags as an argument and turns
12linux_open into a wrapper for it with RW_MODE.
13
14_device_open_ro is added to open the device with RD_MODE and increment
15the open_counter. This is used in the init_* functions.
16
17_device_close is a wrapper around linux_close that decrements the
18open_counter and is used in the init_* functions.
19
20All of these changes are self-contained with no external API changes.
21The only visible change in behavior is that when a new PedDevice is
22created the device is opened in RO_MODE instead of RW_MODE.
23
24Resolves: rhbz#1245144
25
26Upstream-Status: Backport
27
28Author: Brian C. Lane <bcl@redhat.com>
29Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com>
30---
31 libparted/arch/linux.c | 62 +++++++++++++++++++++++++++++++++++---------------
32 1 file changed, 44 insertions(+), 18 deletions(-)
33
34diff --git a/libparted/arch/linux.c b/libparted/arch/linux.c
35index f612617..0a06a54 100644
36--- a/libparted/arch/linux.c
37+++ b/libparted/arch/linux.c
38@@ -294,7 +294,9 @@ struct blkdev_ioctl_param {
39 static char* _device_get_part_path (PedDevice const *dev, int num);
40 static int _partition_is_mounted_by_path (const char* path);
41 static unsigned int _device_get_partition_range(PedDevice const* dev);
42-
43+static int _device_open (PedDevice* dev, int flags);
44+static int _device_open_ro (PedDevice* dev);
45+static int _device_close (PedDevice* dev);
46
47 static int
48 _read_fd (int fd, char **buf)
49@@ -913,7 +915,7 @@ init_ide (PedDevice* dev)
50 if (!_device_stat (dev, &dev_stat))
51 goto error;
52
53- if (!ped_device_open (dev))
54+ if (!_device_open_ro (dev))
55 goto error;
56
57 if (ioctl (arch_specific->fd, HDIO_GET_IDENTITY, &hdi)) {
58@@ -982,11 +984,11 @@ init_ide (PedDevice* dev)
59 if (!_device_probe_geometry (dev))
60 goto error_close_dev;
61
62- ped_device_close (dev);
63+ _device_close (dev);
64 return 1;
65
66 error_close_dev:
67- ped_device_close (dev);
68+ _device_close (dev);
69 error:
70 return 0;
71 }
72@@ -1119,7 +1121,7 @@ init_scsi (PedDevice* dev)
73 char* vendor;
74 char* product;
75
76- if (!ped_device_open (dev))
77+ if (!_device_open_ro (dev))
78 goto error;
79
80 if (ioctl (arch_specific->fd, SCSI_IOCTL_GET_IDLUN, &idlun) < 0) {
81@@ -1133,7 +1135,7 @@ init_scsi (PedDevice* dev)
82 goto error_close_dev;
83 if (!_device_probe_geometry (dev))
84 goto error_close_dev;
85- ped_device_close (dev);
86+ _device_close (dev);
87 return 1;
88 }
89
90@@ -1155,11 +1157,11 @@ init_scsi (PedDevice* dev)
91 if (!_device_probe_geometry (dev))
92 goto error_close_dev;
93
94- ped_device_close (dev);
95+ _device_close (dev);
96 return 1;
97
98 error_close_dev:
99- ped_device_close (dev);
100+ _device_close (dev);
101 error:
102 return 0;
103 }
104@@ -1171,7 +1173,7 @@ init_file (PedDevice* dev)
105
106 if (!_device_stat (dev, &dev_stat))
107 goto error;
108- if (!ped_device_open (dev))
109+ if (!_device_open_ro (dev))
110 goto error;
111
112 dev->sector_size = PED_SECTOR_SIZE_DEFAULT;
113@@ -1198,7 +1200,7 @@ init_file (PedDevice* dev)
114 goto error_close_dev;
115 }
116
117- ped_device_close (dev);
118+ _device_close (dev);
119
120 dev->bios_geom.cylinders = dev->length / 4 / 32;
121 dev->bios_geom.heads = 4;
122@@ -1209,7 +1211,7 @@ init_file (PedDevice* dev)
123 return 1;
124
125 error_close_dev:
126- ped_device_close (dev);
127+ _device_close (dev);
128 error:
129 return 0;
130 }
131@@ -1225,7 +1227,7 @@ init_dasd (PedDevice* dev, const char* model_name)
132 if (!_device_stat (dev, &dev_stat))
133 goto error;
134
135- if (!ped_device_open (dev))
136+ if (!_device_open_ro (dev))
137 goto error;
138
139 LinuxSpecific* arch_specific = LINUX_SPECIFIC (dev);
140@@ -1265,11 +1267,11 @@ init_dasd (PedDevice* dev, const char* model_name)
141
142 dev->model = strdup (model_name);
143
144- ped_device_close (dev);
145+ _device_close (dev);
146 return 1;
147
148 error_close_dev:
149- ped_device_close (dev);
150+ _device_close (dev);
151 error:
152 return 0;
153 }
154@@ -1284,7 +1286,7 @@ init_generic (PedDevice* dev, const char* model_name)
155 if (!_device_stat (dev, &dev_stat))
156 goto error;
157
158- if (!ped_device_open (dev))
159+ if (!_device_open_ro (dev))
160 goto error;
161
162 ped_exception_fetch_all ();
163@@ -1332,11 +1334,11 @@ init_generic (PedDevice* dev, const char* model_name)
164
165 dev->model = strdup (model_name);
166
167- ped_device_close (dev);
168+ _device_close (dev);
169 return 1;
170
171 error_close_dev:
172- ped_device_close (dev);
173+ _device_close (dev);
174 error:
175 return 0;
176 }
177@@ -1623,12 +1625,27 @@ retry:
178 }
179
180 static int
181+_device_open_ro (PedDevice* dev)
182+{
183+ int rc = _device_open (dev, RD_MODE);
184+ if (rc)
185+ dev->open_count++;
186+ return rc;
187+}
188+
189+static int
190 linux_open (PedDevice* dev)
191 {
192+ return _device_open (dev, RW_MODE);
193+}
194+
195+static int
196+_device_open (PedDevice* dev, int flags)
197+{
198 LinuxSpecific* arch_specific = LINUX_SPECIFIC (dev);
199
200 retry:
201- arch_specific->fd = open (dev->path, RW_MODE);
202+ arch_specific->fd = open (dev->path, flags);
203
204 if (arch_specific->fd == -1) {
205 char* rw_error_msg = strerror (errno);
206@@ -1697,6 +1714,15 @@ linux_refresh_close (PedDevice* dev)
207 return 1;
208 }
209
210+static int
211+_device_close (PedDevice* dev)
212+{
213+ int rc = linux_close (dev);
214+ if (dev->open_count > 0)
215+ dev->open_count--;
216+ return rc;
217+}
218+
219 #if SIZEOF_OFF_T < 8
220
221 #if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,20)
222--
2232.11.0
224
diff --git a/meta/recipes-extended/parted/parted_3.2.bb b/meta/recipes-extended/parted/parted_3.2.bb
index 2bfdfb5f8f..ab30108bf2 100644
--- a/meta/recipes-extended/parted/parted_3.2.bb
+++ b/meta/recipes-extended/parted/parted_3.2.bb
@@ -18,6 +18,7 @@ SRC_URI = "${GNU_MIRROR}/parted/parted-${PV}.tar.xz \
18 file://parted-3.2-sysmacros.patch \ 18 file://parted-3.2-sysmacros.patch \
19 file://run-ptest \ 19 file://run-ptest \
20 file://Makefile \ 20 file://Makefile \
21 file://0001-libparted-Use-read-only-when-probing-devices-on-linu.patch \
21" 22"
22 23
23SRC_URI[md5sum] = "0247b6a7b314f8edeb618159fa95f9cb" 24SRC_URI[md5sum] = "0247b6a7b314f8edeb618159fa95f9cb"