Date: Tue, 20 Jul 2021 14:35:10 +0200
From: Michael Siegel <msi@malbolge.net>
To: caskd <caskd@redxen.eu>
Cc: ~alpine/users@lists.alpinelinux.org
Subject: Re: "LVM on LUKS" wiki page: Questions and suggestions
Message-ID: <20210720143510.12c89045@moon>
In-Reply-To: <20210719210325.5jno55lrja76vrdu@navi>
References: <20210719215114.531515a2@moon>
	<20210719210325.5jno55lrja76vrdu@navi>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Am Mon, 19 Jul 2021 21:03:25 +0000
schrieb caskd <caskd@redxen.eu>:

> >Then, when using dd(1), going with the default block size will take
> >ages to complete. Adding `bs=3D1M` is generally much faster, in my
> >experience. =20
> This is entirely dependent on the device you have and a few other
> factors, so it makes more sense to keep the default.

I see. I still think it should be mentioned that choosing a more
adequate block size might speed things up considerably.
=20
> >So, I suggest the following:
> >
> >  * Remove section "Optional: Overwrite LUKS Partition with Random
> >    Data" entirely. =20
> It should be replaced with "Overwrite disk with Random Data" before
> creating the partition table instead.

Okay, but this is already covered in "Preparing the Temporary
Installation Environment". And that is where these instructions belong,
I'd say.

> >This section is obviously specific to the case of using GRUB with
> >UEFI.
> >
> >The version for Syslinux with BIOS would have to look like this, if
> >I'm not mistaken:
> >
> >  # cd
> >  # umount /mnt/boot
> >  # swapoff /dev/vg0/swap
> >  # umount /mnt
> >  # vgchange -a n
> >  # cryptsetup luksClose lvmcrypt
> >  # reboot
> >
> >So, there should be two sub-sections here:
> >
> >  * "Syslinux with BIOS", containing the above command block.
> >  * "Grub with UEFI", containing what's already there. =20
> This has nothing to do with the bootloader. If syslinux supported
> UEFI, it would be similar if not the same. It should instead keep the
> `umount /mnt/boot/efi` with a comment or a note that says it is UEFI
> specific.

The problem is that the whole tutorial is based on going about things
one of two ways: either BIOS/MBR/Syslinux or UEFI/GPT/GRUB. And lines
2=E2=80=935 in the code block as it is on the page are specific to what the
tutorial describes for going with UEFI/GPT/GRUB. So, for consistency,
there should be two sub-sections, just like in some of the previous
sections.


--
Michael