|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
From 4b8bcadc43ed78a0eedb8e330684c626ef8b9d89 Mon Sep 17 00:00:00 2001
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
From: David Lehman <dlehman@redhat.com>
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
Date: Wed, 21 Jun 2017 15:36:42 -0400
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
Subject: [PATCH] Stop enforcing obsolete limits on partition count. (#1460668)
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
---
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
blivet/partitioning.py | 10 ++++------
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
tests/partitioning_test.py | 20 +++++++++-----------
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
2 files changed, 13 insertions(+), 17 deletions(-)
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
diff --git a/blivet/partitioning.py b/blivet/partitioning.py
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
index 05cb4c6..0a973e9 100644
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
--- a/blivet/partitioning.py
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
+++ b/blivet/partitioning.py
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
@@ -136,8 +136,6 @@ def get_next_partition_type(disk, no_primary=None):
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
part_type = None
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
extended = disk.getExtendedPartition()
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
supports_extended = disk.supportsFeature(parted.DISK_TYPE_EXTENDED)
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
- logical_count = len(disk.getLogicalPartitions())
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
- max_logicals = disk.getMaxLogicalPartitions()
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
primary_count = disk.primaryPartitionCount
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
if primary_count < disk.maxPrimaryPartitionCount:
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
@@ -153,17 +151,17 @@ def get_next_partition_type(disk, no_primary=None):
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
# there is an extended and a free primary
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
if not no_primary:
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
part_type = parted.PARTITION_NORMAL
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
- elif logical_count < max_logicals:
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
- # we have an extended with logical slots, so use one.
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
+ else:
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
+ # we have an extended, so use it.
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
part_type = parted.PARTITION_LOGICAL
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
else:
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
# there are two or more primary slots left. use one unless we're
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
# not supposed to make primaries.
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
if not no_primary:
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
part_type = parted.PARTITION_NORMAL
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
- elif extended and logical_count < max_logicals:
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
+ elif extended:
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
part_type = parted.PARTITION_LOGICAL
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
- elif extended and logical_count < max_logicals:
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
+ elif extended:
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
part_type = parted.PARTITION_LOGICAL
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
return part_type
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
--
|
|
![](https://seccdn.libravatar.org/avatar/76d41fbb4664a40a6be7ca6e366d9167862e2010da12503e27fa920b566c820f?s=16&d=retro) |
e8cad13 |
1.8.3.1
|