[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/5] Support Argon2 KDF in LUKS2
From: |
Patrick Steinhardt |
Subject: |
Re: [PATCH 0/5] Support Argon2 KDF in LUKS2 |
Date: |
Wed, 12 Feb 2020 08:47:49 +0100 |
On Tue, Feb 11, 2020 at 10:53:59PM +0100, Daniel Kiper wrote:
> > GRUB's codebase. This included both stripping off unneeded
> > functionality as well as converting the code to use our own coding
>
> Stripping unneeded functionality is OK. However, I think that it does
> not make sense to convert coding style to the GRUB one. Especially if we
> do not do that for other modules. So, I would leave coding style in
> Argon2 module as is and save your precious minutes for something more
> productive... ;-)
Fair enough, I'll send out a v2 with the original coding style. I
thought as much when I was ready with v1, but was too lazy to do the
work and change back the coding style.
Anyway, to save myself another roundtrip: would you prefer to merge
Argon2 functionality into a single file like I've done it right now or
to retain the original set of files? The reason why I've opted for the
latter is mainly to be able to annotate more functions as static.
> > style. While it makes importing upstream fixes harder, I'd argue
> > the code is still very similar in its structure and thus
> > backporting should be easy enough.
> >
> > Anyway. With these changes I'm able to successfully decrypt LUKS2
> > partitions making use of either PBKDF2, Argon2i or Argon2id.
>
> I will take deeper dive into the code if we hammer out things listed above.
Cool, thanks!
Patrick
signature.asc
Description: PGP signature
- Re: [PATCH 1/5] efi: Allocate half of available memory by default, (continued)
- [PATCH v2 0/6] Support Argon2 KDF in LUKS2, Patrick Steinhardt, 2020/02/20
- [PATCH v2 2/6] types.h: add UINT-related macros needed for Argon2, Patrick Steinhardt, 2020/02/20
- [PATCH v2 1/6] efi: Allocate half of available memory by default, Patrick Steinhardt, 2020/02/20
- [PATCH v2 4/6] luks2: Add missing newline to debug message, Patrick Steinhardt, 2020/02/20
- [PATCH v2 5/6] luks2: Discern Argon2i and Argon2id, Patrick Steinhardt, 2020/02/20
- [PATCH v2 3/6] argon2: Import Argon2 from cryptsetup, Patrick Steinhardt, 2020/02/20