Skip to content

belt-dwp: migrate to the ghash crate #685

Closed

Description

@newpavlov

belt-dwp currently uses a vendored version of the ghash code with small tweaks required by the specification. We probably should tweak ghash accordingly and migrate belt-dwp to it.

Activity

  1. makavity commented on Mar 29, 2025

    @makavity
    Contributor

    Looks like we should add Le an Be operations, right?

  2. newpavlov commented on Mar 31, 2025

    @newpavlov
    MemberAuthor

    I don't remember. I will need to take a look at the belt-dwp spec and compare it against implementation in ghash.

  3. makavity commented on Mar 31, 2025

    @makavity
    Contributor

    Additional functionality in belt-dwp is only xor_s. If it is ok to add this function in GHash crate and add le or be flavor - can commit it.

  4. newpavlov commented on Apr 1, 2025

    @newpavlov
    MemberAuthor

    Could you open two PRs for ghash and for belt-dwp with migration to it? You can use the dependency patching for the second PR.

  5. makavity commented on Apr 1, 2025

    @makavity
    Contributor

    Yeah, sure.

  6. makavity commented on Jun 4, 2025

    @makavity
    Contributor

    xor_s is useless. It is replaced with just update_padded.
    But now I have some problems with little endian/big endian.
    Should I add flavors for arithmetic in the polyval crate?

  7. newpavlov commented on Jun 4, 2025

    @newpavlov
    MemberAuthor

    Should I add flavors for arithmetic in the polyval crate?

    Yeah, it's probably has to be done.

  8. makavity commented on Sep 9, 2026

    @makavity
    Contributor

    Done in #878

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions