lkml.org 
[lkml]   [2018]   [Sep]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v3] efi: take size of partition entry from GPT header
On Wed, 12 Sep 2018, Eugene Korenevsky wrote:
> /**
> * efi_partition(struct parsed_partitions *state)
> * @state: disk parsed partitions
>@@ -704,32 +710,36 @@ int efi_partition(struct parsed_partitions *state)
>
> pr_debug("GUID Partition Table is valid! Yea!\n");
>
>- for (i = 0; i < le32_to_cpu(gpt->num_partition_entries) && i < state->limit-1; i++) {
>+ for (i = 0;
>+ i < le32_to_cpu(gpt->num_partition_entries) && i < state->limit-1;
>+ i++) {

Please rearrange this like:

for (i = 0; < le32_to_cpu(gpt->num_partition_entries) &&
i < state->limit-1; i++)

>+ gpt_entry *pte = get_gpt_entry(gpt, ptes, i);
> struct partition_meta_info *info;
> unsigned label_count = 0;
> unsigned label_max;
>- u64 start = le64_to_cpu(ptes[i].starting_lba);
>- u64 size = le64_to_cpu(ptes[i].ending_lba) -
>- le64_to_cpu(ptes[i].starting_lba) + 1ULL;
>+ u64 start = le64_to_cpu(pte->starting_lba);
>+ u64 size = le64_to_cpu(pte->ending_lba) -
>+ le64_to_cpu(pte->starting_lba) + 1ULL;
>
>- if (!is_pte_valid(&ptes[i], last_lba(state->bdev)))
>+ if (!is_pte_valid(pte, last_lba(state->bdev)))
> continue;
>
> put_partition(state, i+1, start * ssz, size * ssz);
>
> /* If this is a RAID volume, tell md */
>- if (!efi_guidcmp(ptes[i].partition_type_guid, PARTITION_LINUX_RAID_GUID))
>+ if (!efi_guidcmp(
>+ pte->partition_type_guid, PARTITION_LINUX_RAID_GUID))

This is ugly. If you are worried about 80-chars, please do:

if (!efi_guidcmp(pte->partition_type_guid,
PARTITION_LINUX_RAID_GUID))

Thanks,
Davidlohr

\
 
 \ /
  Last update: 2018-09-12 20:55    [W:0.046 / U:1.552 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site