[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 12/15] luks2: grub_cryptodisk_t->total_length is the max n
|
From: |
Daniel Kiper |
|
Subject: |
Re: [PATCH v4 12/15] luks2: grub_cryptodisk_t->total_length is the max number of device native sectors |
|
Date: |
Tue, 17 Nov 2020 15:06:08 +0100 |
|
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Fri, Nov 06, 2020 at 10:44:32PM -0600, Glenn Washburn wrote:
> The total_length field is named confusingly because length usually refers to
total_length? I think by mistake you left original commit message here.
It is confusing right now.
> bytes, whereas in this case its really the total number of sectors on the
> device. Also counter-intuitively, grub_disk_get_size returns the total
> number of device native sectors. We need to convert the sectors from the
> size of the underlying device to the cryptodisk sector size. And
> segment.size is in bytes which need to be converted to cryptodisk sectors.
>
> Also, removed an empty statement.
>
> Signed-off-by: Glenn Washburn <development@efficientek.com>
Otherwise patch LGTM.
Daniel