Opened 4 years ago
Closed 4 years ago
#1505 closed defect (fixed)
Add new Bloom filter setting, raise BW limit
Reported by: | zzz | Owned by: | zzz |
---|---|---|---|
Priority: | minor | Milestone: | 0.9.21 |
Component: | router/general | Version: | 0.9.18 |
Keywords: | Cc: | ||
Parent Tickets: |
Description
Related: #1447
Subtickets
Change History (4)
comment:1 Changed 4 years ago by zzz
- Status changed from new to testing
comment:2 Changed 4 years ago by zzz
- Resolution set to fixed
- Status changed from testing to closed
propped early in 0.9.19-x cycle
comment:3 Changed 4 years ago by zzz
- Milestone changed from 0.9.20 to 0.9.21
- Resolution fixed deleted
- Status changed from closed to reopened
Somebody on IRC has hit the new 8 MBps limit and requested and is testing a higher limit (w/o a Bloom filter change)
Doubling the Bloom filter size again would be really huge. Psi proposes a per-tunnel filter instead, presumably a HashSet?, but without any data that a) cross-tunnel (or out-of-tunnel) replay attacks are not part of our threat model; or b) per-tunnel filters would be smaller. For further research.
The next step up for the DecayingBloomFilter? would be 32 MB.
I briefly looked at using memory off-heap with a direct ByteBuffer? but BB.array() is optional and apparently not usually implemented. But we could perhaps use get/put.
comment:4 Changed 4 years ago by zzz
- Resolution set to fixed
- Status changed from reopened to closed
In c051f1b4a8ff522b94a2be9da03e1601b873f983 to be 0.9.20-15, upped to m=27 k=9 (32 MB) max=16384KBps
In 616a55fe09c9f3c5649ab86bfb004a38b58dd391 i2p.i2p.zzz.test2 to be propped for 0.9.20