Skip to content

Commit

Permalink
build.rs: Add feature 'lowmemory' to reduce memory usage
Browse files Browse the repository at this point in the history
Currently, this only set `ECMULT_WINDOW_SIZE` to 3 instead of 15.

Fixes #139.
  • Loading branch information
laanwj committed Aug 7, 2019
1 parent b005089 commit 09037fc
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 2 deletions.
1 change: 1 addition & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ script:
- cargo build --verbose --no-default-features --features="rand"
- cargo build --verbose --no-default-features --features="rand serde recovery endomorphism"
- cargo build --verbose --no-default-features --features="fuzztarget recovery"
- cargo build --verbose --no-default-features --features="lowmemory"
- cargo build --verbose
- cargo test --verbose
- cargo build --release
Expand Down
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,10 @@
# 0.15.1 - ????-??-??

- Add feature `lowmemory` that reduces the EC mult window size to require
significantly less memory for the validation context (~340B instead of
~520kB), at the cost of slower validation. It does not affect signing, nor
the size of the signing context.

# 0.15.0 - 2019-07-25

* Implement hex human-readable serde for PublicKey
Expand Down
1 change: 1 addition & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ fuzztarget = []
std = ["rand/std"]
recovery = []
endomorphism = []
lowmemory = []

[dev-dependencies]
rand = "0.6"
Expand Down
8 changes: 6 additions & 2 deletions build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,13 @@ fn main() {
.define("USE_FIELD_INV_BUILTIN", Some("1"))
.define("USE_SCALAR_INV_BUILTIN", Some("1"))
.define("ENABLE_MODULE_ECDH", Some("1"))
.define("USE_EXTERNAL_DEFAULT_CALLBACKS", Some("1"))
.define("ECMULT_WINDOW_SIZE", Some("15")); // This is the default in the configure file (`auto`)
.define("USE_EXTERNAL_DEFAULT_CALLBACKS", Some("1"));

if cfg!(feature = "lowmemory") {
base_config.define("ECMULT_WINDOW_SIZE", Some("3")); // This is the lowest accepted value by configure
} else {
base_config.define("ECMULT_WINDOW_SIZE", Some("15")); // This is the default in the configure file (`auto`)
}
#[cfg(feature = "endomorphism")]
base_config.define("USE_ENDOMORPHISM", Some("1"));
#[cfg(feature = "recovery")]
Expand Down

0 comments on commit 09037fc

Please sign in to comment.