Opened 9 months ago

Last modified 6 weeks ago

#26945 assigned defect

Privcount blinding and encryption: always enable i128

Reported by: teor Owned by:
Priority: Medium Milestone: Tor: unspecified
Component: Core Tor/Tor Version:
Severity: Normal Keywords: privcount, 035-roadmap-master, 035-triaged-in-20180711, rust, 040-unreached-20190109, teor-unreached-2019-03-08
Cc: teor, nickm, chelseakomlo Actual Points:
Parent ID: #25669 Points:
Reviewer: Sponsor: SponsorV

Child Tickets

Change History (9)

comment:1 Changed 9 months ago by teor

To test this feature, we should build i686 on Linux, and build and test on OS X:

comment:2 Changed 8 months ago by nickm

Sponsor: SponsorV

comment:3 Changed 7 months ago by nickm

Milestone: Tor: 0.3.5.x-finalTor: 0.3.6.x-final

Deferring privcount tickets in 0.3.5 to 0.3.6

comment:4 Changed 6 months ago by teor

nickm suggests that we just use i128, and wait to see if the unit tests fail.
(In most cases, the compilation will fail, so it will be obvious.)

If we want the fastest code, we should select 32-bit multiplications on some platforms, see:
But that's not a high priority for us.

comment:5 Changed 6 months ago by teor

The relevant i128 lowering rust issue is:

comment:6 Changed 6 months ago by teor

Summary: Privcount blinding and encryption: Conditionally disable i128Privcount blinding and encryption: always enable i128

So we will always enable i128, merge the code, then use the fallback i64 code for any platforms that fail

comment:7 Changed 5 months ago by nickm

Milestone: Tor: 0.3.6.x-finalTor: 0.4.0.x-final

Tor 0.3.6.x has been renamed to 0.4.0.x.

comment:8 Changed 3 months ago by teor

Keywords: 040-unreached-20190109 added
Milestone: Tor: 0.4.0.x-finalTor: unspecified

These tasks aren't essential for a PrivCount proof of concept: moving them to Tor: unspecified.

comment:9 Changed 6 weeks ago by teor

Keywords: teor-unreached-2019-03-08 added
Owner: teor deleted

Unassign the tasks that are in tor: unspecified

Note: See TracTickets for help on using tickets.