Re: [PATCH 0/3 v8] LVM Cachevol and Integrity volumes break entire LVM VG

[Patrick Plenefisch]

Hello, yet another friendly poke that this has been waiting around for several
months for a review, and I'd love to get this bug fixed sooner rather than later
Patrick


On Tue, Oct 22, 2024 at 10:46 AM Patrick Plenefisch <simonpatp@gmail.com> wrote:
>
> Hello, a friendly poke that this has been waiting around for several
> months for a review
> Patrick
>
> On Mon, Sep 16, 2024 at 7:07 PM Patrick Plenefisch <simonpatp@gmail.com> 
> wrote:
> >
> > Hello, I'd love to get this latest version reviewed at some point
> >
> > Thanks,
> > Patrick
> >
> >
> > On Tue, Aug 13, 2024 at 8:50 PM Patrick Plenefisch <simonpatp@gmail.com> 
> > wrote:
> > >
> > > In an effort to solve
> > > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1061168 for myself,
> > > I implemented basic support for cachevol and integrity volumes in LVM.
> > >
> > > This is just an extension of the cachepool support that already
> > > existed, and just like that support, I ignore all of the metadata and
> > > just go for the data. This obviously means that writing to "writeback"
> > > cachepools, and any raidintegrity volumes will cause corruption as
> > > before, but as grub only reads files, I think that should be fine.
> > >
> > > Without these patches, a system with /boot on a LV with cachevol or
> > > integrity volumes will fail to boot
> > >
> > > These patches are also available attached and at
> > > https://github.com/byteit101/grub2/commits/grub-lvmintegrity/
> > >
> > > Please ignore the ones not labeled "v8", as I had a cherry picking
> > > error. How do you edit patches sanely?
> > >
> > > Patrick Plenefisch (3):
> > >   disk/lvm: Make cache_lv more generic as ignored_feature_lv
> > >   disk/lvm: Remove unused cache_pool
> > >   lvm: Add support for cachevol and integrity lv
> > >
> > >  grub-core/disk/diskfilter.c |   6 +-
> > >  grub-core/disk/lvm.c        | 267 ++++++++++++++++++------------------
> > >  2 files changed, 136 insertions(+), 137 deletions(-)
> > >
> > > --
> > > 2.39.2