Re: [PATCH v3 0/2] btrfs: Add zstd support to grub btrfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> On Oct 10, 2018, at 12:34 AM, Paul Menzel <pmenzel@xxxxxxxxxxxxx> wrote:
> 
> Sorry for being ignorant, but you explain, why the library needs to be imported and it is not enough to use that library as an external dependency?
> 
> Importing the library means, it has to be maintained in the GRUB repository, which will result in some maintenance burden.

I've imported zstd because thats the way the rest of the decompressors are imported.

We could potentially use libzstd as an external dependency, since its only dependency is libc
and GRUB provides the definitions of the libc functions that zstd needs to decompress
(memcpy, and memmove). Theres some other stuff in the library that requires libc functionality
that GRUB doesn't provide, but that isn't used during decompression. We strip those files
out in the import.

Let me know if you want me to switch to an external dependency.

Best,
Nick



[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux