[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: |
Glenn Washburn |
|
Subject: |
Re: [PATCH v4 12/15] luks2: grub_cryptodisk_t->total_length is the max number of device native sectors |
|
Date: |
Fri, 20 Nov 2020 02:41:28 -0600 |
On Tue, 17 Nov 2020 15:06:08 +0100
Daniel Kiper <dkiper@net-space.pl> wrote:
> 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
Damn, I thought I changed that. Good catch.
Glenn