On Mon, Sep 07, 2020 at 05:27:41PM +0200, Patrick Steinhardt wrote:
> From: Glenn Washburn
It seems to me this patch should be split into two and and begs for
commit message improvement. In general it would be nice to know why
we need these fixes.
Daniel
On Mon, Sep 07, 2020 at 05:27:46PM +0200, Patrick Steinhardt wrote:
> From: Glenn Washburn
>
> The total_length field is named confusingly because length usually refers to
> bytes, whereas in this case its really the total number of sectors on the
> device. Also counter-intuitively, grub_disk_get_
On Mon, Sep 07, 2020 at 05:27:55PM +0200, Patrick Steinhardt wrote:
> From: Glenn Washburn
>
> Signed-off-by: Glenn Washburn
> Reviewed-by: Patrick Steinhardt
> ---
> grub-core/disk/cryptodisk.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/grub-core/disk/cryptodisk.c b/grub-core/dis
On Mon, Sep 07, 2020 at 05:28:04PM +0200, Patrick Steinhardt wrote:
> From: Glenn Washburn
Could explain here why this patch is needed? I know why but I would like
to have this explicitly stated in the commit message.
Daniel
___
Grub-devel mailing lis