Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Change memory usage of dual attack #89

Merged
merged 8 commits into from
Oct 31, 2023
Merged

Change memory usage of dual attack #89

merged 8 commits into from
Oct 31, 2023

Conversation

ludopulles
Copy link
Contributor

Resolves #88

  • The parameter use_lll in lwe_dual.py was unused.
  • The short_vectors methods in reduction.py now consistently return a 4-tuple, containing sieving dimension.
  • The memory cost of dual attack considers memory for storing the dual vectors, AND the FFT table.
  • The runtime cost also takes into account building the initial FFT table, based on all the enumeration targets.
  • Various typo fixes.
  • Added note that the number of dual vectors required is assuming the Independence Heuristic.

@ludopulles
Some typos
a2a4d5f
@ludopulles
Remove unused "use_lll" variable
0c77d35
@ludopulles
Make short_vectors consistently return sieve dim
cc891e4
@ludopulles
Take more costs into account for dual attack:
185b8d5 
- initializing FFT table, and
- running BKZ,
- running sieve

In addition, the memory cost is also computed differently, by assuming
that memory consist of:

- storing all the short dual vectors,
- storing 1 FFT table (which is overwritten in between FFT executions for different enumeration targets)
@ludopulles
Move enumeration cost to fft_solver & use m_required consistently
a2027f9 
In particular, one if-else was simplified to if, because that one raises
an error.
@ludopulles ludopulles marked this pull request as ready for review October 30, 2023 14:22
@malb malb requested a review from blowfish880 October 30, 2023 14:41
blowfish880
blowfish880 approved these changes Oct 31, 2023
View reviewed changes
Copy link
Collaborator

@blowfish880 blowfish880 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks! @malb I'd be happy with a merge if you are.

@malb malb merged commit 0dba276 into malb:main Oct 31, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blowfish880 blowfish880 blowfish880 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Memory usage of DualHybrid attack
3 participants
@ludopulles @blowfish880 @malb