For the IMSM compatibility add partition checking when creating an array.
If the metadata overwrite the last partition, show the warning to the user.
The patch supports only MBR and GPT partition table types.
Regards,
Maciek.
--- util.c 2009-11-02 16:45:00.000000000 +0100
+++ ../../../../mdadm_neil/vobs/trelliscruzbay/mdadm/util.c 2009-10-26 15:13:32.000000000 +0100
@@ -65,6 +65,39 @@ struct blkpg_partition {
char volname[BLKPG_VOLNAMELTH]; /* volume label */
};
+struct MBR_part_record {
+ __u8 bootable;
+ __u8 first_head;
+ __u8 first_sector;
+ __u8 first_cyl;
+ __u8 part_type;
+ __u8 last_head;
+ __u8 last_sector;
+ __u8 last_cyl;
+ __u32 first_sect_lba;
+ __u32 blocks_num;
+};
+
+struct GPT_part_entry {
+ unsigned char type_guid[16];
+ unsigned char partition_guid[16];
+ unsigned char starting_lba[8];
+ unsigned char ending_lba[8];
+ unsigned char attr_bits[8];
+ unsigned char name[72];
+};
+
+/* MBR/GPT magic numbers */
+#define MBR_SIGNATURE_MAGIC __cpu_to_le16(0xAA55)
+#define GPT_SIGNATURE_MAGIC __cpu_to_le64(0x5452415020494645ULL)
+
+#define MBR_SIGNATURE_OFFSET 510
+#define MBR_PARTITION_TABLE_OFFSET 446
+#define MBR_PARTITIONS 4
+#define MBR_GPT_PARTITION_TYPE 0xEE
+#define GPT_ALL_PARTITIONS_OFFSET 80
+#define GPT_ENTRY_SIZE_OFFSET 84
+
/*
* Parse a 128 bit uuid in 4 integers
* format is 32 hexx nibbles with options :.<space> separator
@@ -1096,6 +1129,142 @@ int get_dev_size(int fd, char *dname, un
return 1;
}
+
+/* Sets endofpart parameter to the last block used by the last GPT partition on the device.
+ Returns: 1 if successful
+ -1 for unknown partition type
+ 0 for other errors
+*/
+int get_gpt_last_partition_end(int fd, unsigned long long *endofpart)
+{
+ unsigned char buf[512];
+ unsigned char empty_gpt_entry[16]= {0};
+ struct GPT_part_entry *part;
+ unsigned long long curr_part_end;
+ unsigned all_partitions, entry_size;
+ int part_nr;
+
+ *endofpart = 0;
+
+ // read GPT header
+ lseek(fd, 512, SEEK_SET);
+ if (read(fd, buf, 512) != 512)
+ return 0;
+
+ // get the number of partition entries and the entry size
+ all_partitions = __le32_to_cpu(buf[GPT_ALL_PARTITIONS_OFFSET]);
+ entry_size = __le32_to_cpu(buf[GPT_ENTRY_SIZE_OFFSET]);
+
+ // Check GPT signature
+ if (*((__u64*)buf) != GPT_SIGNATURE_MAGIC)
+ return -1;
+
+ // read first GPT partition entries
+ if (read(fd, buf, 512) != 512)
+ return 0;
+
+ part = (struct GPT_part_entry*)buf;
+
+ for (part_nr=0; part_nr < all_partitions; part_nr++) {
+ // is this valid partition?
+ if (memcmp(part->type_guid, empty_gpt_entry, 16)) {
+ // check the last lba for the current partition
+ curr_part_end = __le64_to_cpu(*(__u64*)part->ending_lba);
+ if (curr_part_end > *endofpart)
+ *endofpart = curr_part_end;
+ }
+
+ part = (struct GPT_part_entry*)((unsigned char*)part + entry_size);
+
+ if ((unsigned char *)part >= buf + 512) {
+ if (read(fd, buf, 512) != 512)
+ return 0;
+ part = (struct GPT_part_entry*)buf;
+ }
+ }
+ return 1;
+}
+
+/* Sets endofpart parameter to the last block used by the last partition on the device.
+ Returns: 1 if successful
+ -1 for unknown partition type
+ 0 for other errors
+*/
+int get_last_partition_end(int fd, unsigned long long *endofpart)
+{
+ unsigned char boot_sect[512];
+ struct MBR_part_record *part;
+ unsigned long long curr_part_end;
+ int part_nr;
+ int retval = 0;
+
+ *endofpart = 0;
+
+ // read MBR
+ lseek(fd, 0, 0);
+ if (read(fd, boot_sect, 512) != 512)
+ goto abort;
+
+ // check MBP signature
+ if (*((__u16*)(boot_sect + MBR_SIGNATURE_OFFSET))
+ == MBR_SIGNATURE_MAGIC) {
+ retval = 1;
+ // found the correct signature
+ part = (struct MBR_part_record*)
+ (boot_sect + MBR_PARTITION_TABLE_OFFSET);
+
+ for (part_nr=0; part_nr < MBR_PARTITIONS; part_nr++) {
+ // check for GPT type
+ if (part->part_type == MBR_GPT_PARTITION_TYPE) {
+ retval = get_gpt_last_partition_end(fd, endofpart);
+ break;
+ }
+ // check the last used lba for the current partition
+ curr_part_end = __le32_to_cpu(part->first_sect_lba) +
+ __le32_to_cpu(part->blocks_num);
+ if (curr_part_end > *endofpart)
+ *endofpart = curr_part_end;
+
+ part++;
+ }
+ } else {
+ // Unknown partition table
+ retval = -1;
+ }
+ abort:
+ return retval;
+}
+
+int check_partitions(int fd, char *dname, unsigned long long freesize)
+{
+ /*
+ * Check where the last partition ends
+ */
+ unsigned long long endofpart;
+ int ret;
+
+ if ((ret = get_last_partition_end(fd, &endofpart)) > 0) {
+ if (endofpart > freesize) {
+ fprintf(stderr,
+ Name ": Not enough space for metadata on %s.\n",
+ dname);
+ return 1;
+ }
+ } else {
+ if (ret < 0) {
+ fprintf(stderr,
+ Name ": Unknown parition table on %s.\n",
+ dname);
+ } else {
+ fprintf(stderr,
+ Name ": Unknown error while reading %s.\n",
+ dname);
+ }
+ return 1;
+ }
+ return 0;
+}
+
void get_one_disk(int mdfd, mdu_array_info_t *ainf, mdu_disk_info_t *disk)
{
int d;
--- mdadm.h 2009-11-02 16:44:57.000000000 +0100
+++ ../../../../mdadm_neil/vobs/trelliscruzbay/mdadm/mdadm.h 2009-10-26 15:13:23.000000000 +0100
@@ -787,6 +787,7 @@ extern int parse_layout_faulty(char *lay
extern int check_ext2(int fd, char *name);
extern int check_reiser(int fd, char *name);
extern int check_raid(int fd, char *name);
+extern int check_partitions(int fd, char *name, unsigned long long freesize);
extern int get_mdp_major(void);
extern int dev_open(char *dev, int flags);
--- Create.c 2009-11-02 16:44:54.000000000 +0100
+++ ../../../../mdadm_neil/vobs/trelliscruzbay/mdadm/Create.c 2009-10-24 01:50:20.000000000 +0200
@@ -334,8 +334,8 @@ int Create(struct supertype *st, char *m
fail = 1;
continue;
}
- }
-
+ }
+
freesize /= 2; /* convert to K */
if (chunk) {
/* round to chunk size */
@@ -368,6 +368,8 @@ int Create(struct supertype *st, char *m
warn |= check_ext2(fd, dname);
warn |= check_reiser(fd, dname);
warn |= check_raid(fd, dname);
+ // May metadata overwrite any partition?
+ warn |= check_partitions(fd, dname, freesize*2);
close(fd);
}
}
--
---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
z siedziba w Gdansku
ul. Slowackiego 173
80-298 Gdansk
Sad Rejonowy Gdansk Polnoc w Gdansku,
VII Wydzial Gospodarczy Krajowego Rejestru Sadowego,
numer KRS 101882
NIP 957-07-52-316
Kapital zakladowy 200.000 zl
This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
