Andrei Borzenkov schreef op 09-05-2016 8:07:
No, I do not want to change it at all. It should remain as is and
always
probe for FS so that test below actually works
But its output makes no sense, there cannot be a filesystem in a PV. It
could check for existence of VG AND LV AND filesystem b
Andrei Borzenkov schreef op 08-05-2016 11:23:
PV label may be in the first sector. In this case label will be
overwritten later by boot image. This needs additional check.
Pardon my last mail.
ON OLD PV SIGNATURES
If you have an old PV that had a boatloaderarea, then wip
Currently:
- still searches in first sector for label, and reports error if found
there
- get_pvh split into regular and one with 2 extra parameters for
first_sector detection
- some documentation
- VG is not required to install
- does not even check for VG, but function is retained
- is lvm r
Dryden Personalis schreef op 09-05-2016 18:10:
I have reverted to that of the patch (or similar):
if (fs_probe)
{
if (!is_lvm && !fs && !ctx.dest_partmap)
grub_util_error (_("unable to identify a filesystem in %s;
safety check can't be performed"), dest_dev->disk->name);
In partic
Attached my current working patch.
Also attached some housekeeping.
Not sure if it applies to current git version.
Take from it what you like.diff --git a/grub-core/disk/lvm.c b/grub-core/disk/lvm.c
index 22c7b64..37e715a 100644
--- a/grub-core/disk/lvm.c
+++ b/grub-core/disk/lvm.c
@@ -95,6 +95